diff options
| author | nsfisis <nsfisis@gmail.com> | 2026-02-23 01:36:57 +0900 |
|---|---|---|
| committer | nsfisis <nsfisis@gmail.com> | 2026-02-23 01:36:57 +0900 |
| commit | 82c89125dc6a1620ecd80949c155295a96db408c (patch) | |
| tree | 9479cea7534cf01c7a8c69540cd93dfb94ec5da8 | |
| parent | a0c17873aeb88cc81be769317fcad37161bb516c (diff) | |
| download | php-mozart-82c89125dc6a1620ecd80949c155295a96db408c.tar.gz php-mozart-82c89125dc6a1620ecd80949c155295a96db408c.tar.zst php-mozart-82c89125dc6a1620ecd80949c155295a96db408c.zip | |
fix(remove): wire apcu autoloader flags, add audit-format default, and match output messages
- Pass --apcu-autoloader and --apcu-autoloader-prefix through to
InstallConfig instead of hardcoding false/None
- Set --audit-format default to "summary" matching Composer behavior
- Print "./composer.json has been updated" after modification
- Print "Running composer update <packages>" before resolution step
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
| -rw-r--r-- | crates/mozart/src/commands/remove.rs | 16 |
1 files changed, 11 insertions, 5 deletions
diff --git a/crates/mozart/src/commands/remove.rs b/crates/mozart/src/commands/remove.rs index d047ad3..acf878c 100644 --- a/crates/mozart/src/commands/remove.rs +++ b/crates/mozart/src/commands/remove.rs @@ -36,7 +36,7 @@ pub struct RemoveArgs { pub no_audit: bool, /// Audit output format - #[arg(long, value_parser = ["table", "plain", "json", "summary"])] + #[arg(long, value_parser = ["table", "plain", "json", "summary"], default_value = "summary")] pub audit_format: Option<String>, /// Do not block on security advisories @@ -188,6 +188,7 @@ pub async fn execute( } else if any_removed { package::write_to_file(&raw, &composer_path)?; } + eprintln!("./composer.json has been updated"); // Step 7: Handle --no-update early return if args.no_update { @@ -252,6 +253,11 @@ pub async fn execute( }; // Print header messages + let pkg_names = args.packages.join(" "); + console.info(&console_format!( + "<info>Running composer update {}</info>", + pkg_names + )); console.info("Loading composer repositories with package information"); if dev_mode { console.info("Updating dependencies (including require-dev)"); @@ -428,8 +434,8 @@ pub async fn execute( ignore_platform_req: args.ignore_platform_req.clone(), optimize_autoloader: args.optimize_autoloader, classmap_authoritative: args.classmap_authoritative, - apcu_autoloader: false, - apcu_autoloader_prefix: None, + apcu_autoloader: args.apcu_autoloader || args.apcu_autoloader_prefix.is_some(), + apcu_autoloader_prefix: args.apcu_autoloader_prefix.clone(), download_only: false, }, ) @@ -566,8 +572,8 @@ async fn remove_unused( ignore_platform_req: args.ignore_platform_req.clone(), optimize_autoloader: args.optimize_autoloader, classmap_authoritative: args.classmap_authoritative, - apcu_autoloader: false, - apcu_autoloader_prefix: None, + apcu_autoloader: args.apcu_autoloader || args.apcu_autoloader_prefix.is_some(), + apcu_autoloader_prefix: args.apcu_autoloader_prefix.clone(), download_only: false, }, ) |