From d6e0c6d34449224ac3687daf551a0acfd15cee32 Mon Sep 17 00:00:00 2001 From: nsfisis Date: Mon, 23 Feb 2026 15:11:36 +0900 Subject: refactor(cli): route command output through Console abstraction Replace direct println\!/eprintln\! calls with console.write(), console.info(), and console.write_stdout() across all command handlers to respect verbosity settings. Co-Authored-By: Claude Opus 4.6 --- crates/mozart/src/commands/remove.rs | 35 ++++++++++++++++++----------------- 1 file changed, 18 insertions(+), 17 deletions(-) (limited to 'crates/mozart/src/commands/remove.rs') diff --git a/crates/mozart/src/commands/remove.rs b/crates/mozart/src/commands/remove.rs index 4808dd0..2529391 100644 --- a/crates/mozart/src/commands/remove.rs +++ b/crates/mozart/src/commands/remove.rs @@ -1,4 +1,5 @@ use clap::Args; +use mozart_core::console::Verbosity; use mozart_core::console_format; use mozart_core::package; use mozart_core::validation; @@ -148,9 +149,9 @@ pub async fn execute( if args.dev { // Only look in require-dev if raw.require_dev.contains_key(&name) { - println!( - "{}", - console_format!("Removing {name} from require-dev") + console.write_stdout( + &console_format!("Removing {name} from require-dev"), + Verbosity::Normal, ); raw.require_dev.remove(&name); any_removed = true; @@ -160,16 +161,16 @@ pub async fn execute( } else { // Auto-detect: look in require first, then require-dev if raw.require.contains_key(&name) { - println!( - "{}", - console_format!("Removing {name} from require") + console.write_stdout( + &console_format!("Removing {name} from require"), + Verbosity::Normal, ); raw.require.remove(&name); any_removed = true; } else if raw.require_dev.contains_key(&name) { - println!( - "{}", - console_format!("Removing {name} from require-dev") + console.write_stdout( + &console_format!("Removing {name} from require-dev"), + Verbosity::Normal, ); raw.require_dev.remove(&name); any_removed = true; @@ -181,22 +182,22 @@ pub async fn execute( // Step 6: Write updated composer.json (unless --dry-run) if args.dry_run { - println!( - "{}", - console_format!("Dry run: composer.json not modified.") + console.write_stdout( + &console_format!("Dry run: composer.json not modified."), + Verbosity::Normal, ); } else if any_removed { package::write_to_file(&raw, &composer_path)?; } - eprintln!("./composer.json has been updated"); + console.info("./composer.json has been updated"); // Step 7: Handle --no-update early return if args.no_update { - println!( - "{}", - console_format!( + console.write_stdout( + &console_format!( "Not updating dependencies, only modifying composer.json." - ) + ), + Verbosity::Normal, ); return Ok(()); } -- cgit v1.3.1