Revert "test: add --verbose to all e2e test runs to prove full summary output is unchanged"

This reverts commit 86019bc.

Author: branchseer

crates/vite_task/src/cli/mod.rs

@@ -32,8 +32,8 @@ pub struct RunFlags {
     pub ignore_depends_on: bool,
 
     /// Show full detailed summary after execution.
-    #[clap(default_value = "false", short = 'v', long)]
-    pub verbose: bool,
+    #[clap(default_value = "false", long)]
+    pub details: bool,
 }
 
 // ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━

crates/vite_task/src/session/mod.rs

@@ -237,7 +237,7 @@ impl<'a> Session<'a> {
 
                 // Save task name and flags before consuming run_command
                 let task_name = run_command.task_specifier.as_ref().map(|s| s.task_name.clone());
-                let show_details = run_command.flags.verbose;
+                let show_details = run_command.flags.details;
                 let flags = run_command.flags;
                 let additional_args = run_command.additional_args.clone();
 
@@ -398,7 +398,7 @@ impl<'a> Session<'a> {
         // Interactive: run the selected task
         let selected_label = &select_items[selected_index].label;
         let task_specifier = TaskSpecifier::parse_raw(selected_label);
-        let show_details = flags.verbose;
+        let show_details = flags.details;
         let run_command =
             RunCommand { task_specifier: Some(task_specifier), flags, additional_args };
 

crates/vite_task/src/session/reporter/labeled.rs

@@ -49,12 +49,12 @@ struct SharedReporterState {
 /// - Single task + cache hit → thin line + "[vp run] cache hit, {duration} saved."
 /// - Multi-task → thin line + one-liner with stats
 ///
-/// ## Full Summary (`--verbose`)
+/// ## Full Summary (`--details`)
 /// - Shows full Statistics, Performance, and Task Details sections
 pub struct LabeledReporterBuilder {
     workspace_path: Arc<AbsolutePath>,
     writer: Box<dyn AsyncWrite + Unpin>,
-    /// Whether to render the full detailed summary (`--verbose` flag).
+    /// Whether to render the full detailed summary (`--details` flag).
     show_details: bool,
     /// Callback to persist the summary (e.g., write `last-summary.json`).
     /// `None` when persistence is not needed (e.g., nested script execution, tests).

crates/vite_task/src/session/reporter/summary.rs

@@ -459,12 +459,12 @@ impl TaskResult {
 }
 
 // ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
-// Full summary rendering (--verbose and --last-details)
+// Full summary rendering (--details and --last-details)
 // ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
 
 /// Render the full detailed execution summary.
 ///
-/// Used by both `--verbose` (live) and `--last-details` (from file).
+/// Used by both `--details` (live) and `--last-details` (from file).
 #[expect(
     clippy::too_many_lines,
     reason = "summary formatting is inherently verbose with many write calls"
@@ -647,7 +647,7 @@ pub fn format_full_summary(summary: &LastRunSummary) -> Vec<u8> {
 /// - Single task + not cache hit → empty (no summary at all)
 /// - Single task + cache hit → thin line + "[vp run] cache hit, {duration} saved."
 /// - Multi-task → thin line + "[vp run] {hits}/{total} cache hit ({rate}%), {duration} saved."
-///   with optional failure count and `--verbose` hint.
+///   with optional failure count and `--details` hint.
 pub fn format_compact_summary(summary: &LastRunSummary) -> Vec<u8> {
     let stats = SummaryStats::compute(&summary.tasks);
 
@@ -713,7 +713,7 @@ pub fn format_compact_summary(summary: &LastRunSummary) -> Vec<u8> {
         let _ = write!(
             buf,
             ". {}",
-            "(Run `vp run --verbose` for full details)".style(Style::new().bright_black()),
+            "(Run `vp run --details` for full details)".style(Style::new().bright_black()),
         );
         let _ = writeln!(buf);
     }

crates/vite_task_bin/tests/e2e_snapshots/fixtures/associate-existing-cache/snapshots.toml

@@ -3,8 +3,8 @@
 [[e2e]]
 name = "associate existing cache"
 steps = [
-  "vp run --verbose script1 # cache miss",
-  "vp run --verbose script2 # cache hit, same command as script1",
+  "vp run --details script1 # cache miss",
+  "vp run --details script2 # cache hit, same command as script1",
   "json-edit package.json '_.scripts.script2 = \"print world\"' # change script2",
-  "vp run --verbose script2 # cache miss",
+  "vp run --details script2 # cache miss",
 ]

crates/vite_task_bin/tests/e2e_snapshots/fixtures/associate-existing-cache/snapshots/associate existing cache.snap

@@ -2,7 +2,7 @@
 source: crates/vite_task_bin/tests/e2e_snapshots/main.rs
 expression: e2e_outputs
 ---
-> vp run --verbose script1 # cache miss
+> vp run --details script1 # cache miss
 $ print hello
 hello
 
@@ -19,7 +19,7 @@ Task Details:
   [1] script1: $ print hello ✓
       → Cache miss: no previous cache entry found
 ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
-> vp run --verbose script2 # cache hit, same command as script1
+> vp run --details script2 # cache hit, same command as script1
 $ print hello ✓ cache hit, replaying
 hello
 
@@ -38,7 +38,7 @@ Task Details:
 ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
 > json-edit package.json '_.scripts.script2 = "print world"' # change script2
 
-> vp run --verbose script2 # cache miss
+> vp run --details script2 # cache miss
 $ print world ✗ cache miss: args changed, executing
 world
 

crates/vite_task_bin/tests/e2e_snapshots/fixtures/builtin-different-cwd/snapshots.toml

@@ -3,9 +3,9 @@
 [[e2e]]
 name = "builtin different cwd"
 steps = [
-  "cd folder1 && vp run --verbose lint # cache miss in folder1",
-  "cd folder2 && vp run --verbose lint # cache miss in folder2",
+  "cd folder1 && vp run --details lint # cache miss in folder1",
+  "cd folder2 && vp run --details lint # cache miss in folder2",
   "echo 'console.log(1);' > folder2/a.js # modify folder2",
-  "cd folder1 && vp run --verbose lint # cache hit",
-  "cd folder2 && vp run --verbose lint # cache miss",
+  "cd folder1 && vp run --details lint # cache hit",
+  "cd folder2 && vp run --details lint # cache miss",
 ]

crates/vite_task_bin/tests/e2e_snapshots/fixtures/builtin-different-cwd/snapshots/builtin different cwd.snap

@@ -2,7 +2,7 @@
 source: crates/vite_task_bin/tests/e2e_snapshots/main.rs
 expression: e2e_outputs
 ---
-> cd folder1 && vp run --verbose lint # cache miss in folder1
+> cd folder1 && vp run --details lint # cache miss in folder1
 $ vp lint
 
   ! eslint-plugin-unicorn(no-empty-file): Empty files are not allowed.
@@ -35,7 +35,7 @@ Task Details:
   [1] lint: $ vp lint ✓
       → Cache miss: no previous cache entry found
 ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
-> cd folder2 && vp run --verbose lint # cache miss in folder2
+> cd folder2 && vp run --details lint # cache miss in folder2
 $ vp lint ✓ cache hit, replaying
 
   ! eslint-plugin-unicorn(no-empty-file): Empty files are not allowed.
@@ -70,7 +70,7 @@ Task Details:
 ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
 > echo 'console.log(1);' > folder2/a.js # modify folder2
 
-> cd folder1 && vp run --verbose lint # cache hit
+> cd folder1 && vp run --details lint # cache hit
 $ vp lint ✗ cache miss: content of input 'folder2/a.js' changed, executing
 
   ! eslint-plugin-unicorn(no-empty-file): Empty files are not allowed.
@@ -96,7 +96,7 @@ Task Details:
   [1] lint: $ vp lint ✓
       → Cache miss: content of input 'folder2/a.js' changed
 ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
-> cd folder2 && vp run --verbose lint # cache miss
+> cd folder2 && vp run --details lint # cache miss
 $ vp lint ✓ cache hit, replaying
 
   ! eslint-plugin-unicorn(no-empty-file): Empty files are not allowed.

crates/vite_task_bin/tests/e2e_snapshots/fixtures/builtin-non-zero-exit/snapshots.toml

@@ -1,6 +1,6 @@
 [[e2e]]
 name = "builtin command with non-zero exit does not show cache not updated"
 steps = [
-  "vp run --verbose lint -- -D no-debugger",
-  "vp run --verbose lint -- -D no-debugger",
+  "vp run --details lint -- -D no-debugger",
+  "vp run --details lint -- -D no-debugger",
 ]

crates/vite_task_bin/tests/e2e_snapshots/fixtures/builtin-non-zero-exit/snapshots/builtin command with non-zero exit does not show cache not updated.snap

@@ -2,7 +2,7 @@
 source: crates/vite_task_bin/tests/e2e_snapshots/main.rs
 expression: e2e_outputs
 ---
-[1]> vp run --verbose lint -- -D no-debugger
+[1]> vp run --details lint -- -D no-debugger
 $ vp lint -D no-debugger
 
   x eslint(no-debugger): `debugger` statement is not allowed
@@ -28,7 +28,7 @@ Task Details:
   [1] builtin-non-zero-exit-test#lint: $ vp lint -D no-debugger ✗ (exit code: 1)
       → Cache miss: no previous cache entry found
 ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
-[1]> vp run --verbose lint -- -D no-debugger
+[1]> vp run --details lint -- -D no-debugger
 $ vp lint -D no-debugger
 
   x eslint(no-debugger): `debugger` statement is not allowed