refactor(cli): replace verbose/quiet bool pairs with Verbosity enum

exarch-cli's output-formatter and progress-selection code took separate
verbose/quiet bool parameters, allowing the nonsensical verbose=true,
quiet=true state and, in extract, resolving it inconsistently between
the formatter (quiet-wins) and the progress reporter (verbose-wins).

create_formatter, HumanFormatter::new, HumanFormatter::with_writers,
commands::extract::execute, and commands::create::execute now take a
single output::Verbosity (Quiet | Normal | Verbose), resolved once via
impl From<&cli::Cli> for Verbosity and threaded through every call
site, closing the split-ArgMatches-level gap where clap's
conflicts_with does not catch --verbose and --quiet passed at
different argument levels.

Closes #550

Author: bug-ops

CHANGELOG.md

@@ -18,6 +18,25 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ### Changed
 
+- **`exarch-cli`'s output-formatter and progress-selection code took separate `verbose: bool, quiet: bool`
+  parameters, allowing the nonsensical `verbose: true, quiet: true` state and, in `extract`, resolving it
+  inconsistently between the formatter (quiet-wins) and the progress reporter (verbose-wins) (#550)**:
+  `output::create_formatter`, `HumanFormatter::new`, `HumanFormatter::with_writers`,
+  `commands::extract::execute`, and `commands::create::execute` now take a single `output::Verbosity`
+  (`Quiet` | `Normal` | `Verbose`) instead, resolved once via `impl From<&cli::Cli> for Verbosity` in
+  `main.rs` and threaded through every call site. `--verbose`/`--quiet` remain independent `clap` flags
+  with `conflicts_with` on `--quiet`; that check only rejects the pair when both land in the same
+  `ArgMatches` level (e.g. `exarch list --verbose --quiet`), so a split-level combination such as
+  `exarch --verbose extract archive.tar.gz out --quiet` still parses with both `true` —
+  `Verbosity::from_flags` resolves that case deterministically to `Quiet`, the same tie-break the
+  formatter already used.
+
+  **Behavior change**: `extract`'s progress reporter previously selected `VerboseProgress` whenever
+  `--verbose` was passed, even together with `--quiet` (verbose-wins), while the formatter suppressed the
+  summary (quiet-wins) — so `--verbose extract ... --quiet` printed per-entry progress lines but no
+  summary. Both now consistently resolve to `Quiet` (no progress output, no summary) through the same
+  `Verbosity` value.
+
 - **`exarch-cli`'s `--atomic --force` swap path now bundles the pinned temp directory's
   `pin`/`name`/`id`/`parent_display` identifiers into a `TempOrphanRef` struct (#538)** instead of
   threading the same four parameters through `move_destination_to_backup` (7 params, down to 4) and

crates/exarch-cli/src/commands/create.rs

@@ -2,14 +2,19 @@
 
 use crate::cli::CreateArgs;
 use crate::output::OutputFormatter;
+use crate::output::Verbosity;
 use crate::progress::CliProgress;
 use anyhow::Context;
 use anyhow::Result;
 use exarch_core::CreationConfig;
 use exarch_core::NoopProgress;
 use exarch_core::create_archive_with_progress;
 
-pub fn execute(args: &CreateArgs, formatter: &mut dyn OutputFormatter, quiet: bool) -> Result<()> {
+pub fn execute(
+    args: &CreateArgs,
+    formatter: &mut dyn OutputFormatter,
+    verbosity: Verbosity,
+) -> Result<()> {
     // Check if output exists
     if args.output.exists() && !args.force {
         anyhow::bail!(
@@ -42,7 +47,7 @@ pub fn execute(args: &CreateArgs, formatter: &mut dyn OutputFormatter, quiet: bo
     config.exclude_patterns.extend(args.exclude.iter().cloned());
 
     // Create archive with progress if TTY is detected
-    let report = if !quiet && CliProgress::should_show() {
+    let report = if verbosity != Verbosity::Quiet && CliProgress::should_show() {
         let mut progress = CliProgress::new(100, "Creating");
         create_archive_with_progress(&args.output, &args.sources, &config, &mut progress)
             .with_context(|| format!("Failed to create archive: {}", args.output.display()))?
@@ -186,7 +191,7 @@ mod tests {
         let args = make_args(out, src);
         let mut formatter = AlwaysCallSpyFormatter::new();
 
-        execute(&args, &mut formatter, true).unwrap();
+        execute(&args, &mut formatter, Verbosity::Quiet).unwrap();
 
         assert!(
             formatter.was_called(),
@@ -204,7 +209,7 @@ mod tests {
         let args = make_args(out, src);
         let mut formatter = AlwaysCallSpyFormatter::new();
 
-        execute(&args, &mut formatter, false).unwrap();
+        execute(&args, &mut formatter, Verbosity::Normal).unwrap();
 
         assert!(formatter.was_called());
     }
@@ -221,7 +226,7 @@ mod tests {
         let args = make_args(out, src);
         let mut formatter = SpyFormatter::new(true);
 
-        execute(&args, &mut formatter, true).unwrap();
+        execute(&args, &mut formatter, Verbosity::Quiet).unwrap();
 
         assert!(
             !formatter.was_called(),

crates/exarch-cli/src/commands/extract.rs

@@ -7,6 +7,7 @@ use crate::commands::atomic_swap::DestEntryKind;
 use crate::commands::atomic_swap::PinnedDir;
 use crate::error::add_archive_context;
 use crate::output::OutputFormatter;
+use crate::output::Verbosity;
 use crate::progress::CliProgress;
 use crate::progress::VerboseProgress;
 use anyhow::Context;
@@ -719,8 +720,7 @@ fn filter_banned_components(raw: &[String]) -> Vec<String> {
 pub fn execute(
     args: &ExtractArgs,
     formatter: &mut dyn OutputFormatter,
-    verbose: bool,
-    quiet: bool,
+    verbosity: Verbosity,
 ) -> Result<()> {
     let output_dir = match &args.output_dir {
         Some(dir) => dir.clone(),
@@ -817,12 +817,12 @@ pub fn execute(
         .with_atomic(args.atomic && atomic_force_target.is_none())
         .with_skip_duplicates(!args.force);
 
-    let mut progress: Box<dyn ProgressCallback> = if verbose {
-        Box::new(VerboseProgress::new())
-    } else if !quiet && CliProgress::should_show() {
-        Box::new(CliProgress::new(entry_count, "Extracting"))
-    } else {
-        Box::new(NoopProgress)
+    let mut progress: Box<dyn ProgressCallback> = match verbosity {
+        Verbosity::Verbose => Box::new(VerboseProgress::new()),
+        Verbosity::Normal if CliProgress::should_show() => {
+            Box::new(CliProgress::new(entry_count, "Extracting"))
+        }
+        Verbosity::Quiet | Verbosity::Normal => Box::new(NoopProgress),
     };
 
     let report = if let Some(target) = atomic_force_target {

crates/exarch-cli/src/main.rs

@@ -11,16 +11,21 @@ use clap::Parser;
 use error::StrictWarning;
 use error::VerificationFailed;
 use output::OutputFormatter;
+use output::Verbosity;
 use std::process;
 
-fn run(cli: &cli::Cli, formatter: &mut dyn OutputFormatter) -> (anyhow::Result<()>, &'static str) {
+fn run(
+    cli: &cli::Cli,
+    verbosity: Verbosity,
+    formatter: &mut dyn OutputFormatter,
+) -> (anyhow::Result<()>, &'static str) {
     match &cli.command {
         cli::Commands::Extract(args) => (
-            commands::extract::execute(args, formatter, cli.verbose, cli.quiet),
+            commands::extract::execute(args, formatter, verbosity),
             "extract",
         ),
         cli::Commands::Create(args) => (
-            commands::create::execute(args, formatter, cli.quiet),
+            commands::create::execute(args, formatter, verbosity),
             "create",
         ),
         cli::Commands::List(args) => (commands::list::execute(args, formatter), "list"),
@@ -34,9 +39,10 @@ fn run(cli: &cli::Cli, formatter: &mut dyn OutputFormatter) -> (anyhow::Result<(
 
 fn main() {
     let cli = cli::Cli::parse();
-    let mut formatter = output::create_formatter(cli.json, cli.verbose, cli.quiet);
+    let verbosity = Verbosity::from(&cli);
+    let mut formatter = output::create_formatter(cli.json, verbosity);
 
-    let (result, operation) = run(&cli, formatter.as_mut());
+    let (result, operation) = run(&cli, verbosity, formatter.as_mut());
     if let Err(err) = result {
         if err.is::<StrictWarning>() {
             process::exit(2);