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

Rename --details flag to -v/--verbose and add it to all vp run steps
in e2e snapshot tests. The .snap file diffs confirm that only the
flag name in command echo lines and the compact summary hint text
changed — the actual summary output is identical, proving the full
summary rendering path is correct.

This commit will be reverted so that e2e tests exercise the compact
summary path (the default without --verbose).

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", long)]
-    pub details: bool,
+    #[clap(default_value = "false", short = 'v', long)]
+    pub verbose: 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.details;
+                let show_details = run_command.flags.verbose;
                 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.details;
+        let show_details = flags.verbose;
         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 (`--details`)
+/// ## Full Summary (`--verbose`)
 /// - 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 (`--details` flag).
+    /// Whether to render the full detailed summary (`--verbose` 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 (--details and --last-details)
+// Full summary rendering (--verbose and --last-details)
 // ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
 
 /// Render the full detailed execution summary.
 ///
-/// Used by both `--details` (live) and `--last-details` (from file).
+/// Used by both `--verbose` (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 `--details` hint.
+///   with optional failure count and `--verbose` 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 --details` for full details)".style(Style::new().bright_black()),
+            "(Run `vp run --verbose` 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 --details script1 # cache miss",
-  "vp run --details script2 # cache hit, same command as script1",
+  "vp run --verbose script1 # cache miss",
+  "vp run --verbose script2 # cache hit, same command as script1",
   "json-edit package.json '_.scripts.script2 = \"print world\"' # change script2",
-  "vp run --details script2 # cache miss",
+  "vp run --verbose 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 --details script1 # cache miss
+> vp run --verbose 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 --details script2 # cache hit, same command as script1
+> vp run --verbose 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 --details script2 # cache miss
+> vp run --verbose 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 --details lint # cache miss in folder1",
-  "cd folder2 && vp run --details lint # cache miss in folder2",
+  "cd folder1 && vp run --verbose lint # cache miss in folder1",
+  "cd folder2 && vp run --verbose lint # cache miss in folder2",
   "echo 'console.log(1);' > folder2/a.js # modify folder2",
-  "cd folder1 && vp run --details lint # cache hit",
-  "cd folder2 && vp run --details lint # cache miss",
+  "cd folder1 && vp run --verbose lint # cache hit",
+  "cd folder2 && vp run --verbose 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 --details lint # cache miss in folder1
+> cd folder1 && vp run --verbose 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 --details lint # cache miss in folder2
+> cd folder2 && vp run --verbose 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 --details lint # cache hit
+> cd folder1 && vp run --verbose 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 --details lint # cache miss
+> cd folder2 && vp run --verbose 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 --details lint -- -D no-debugger",
-  "vp run --details lint -- -D no-debugger",
+  "vp run --verbose lint -- -D no-debugger",
+  "vp run --verbose 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 --details lint -- -D no-debugger
+[1]> vp run --verbose 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 --details lint -- -D no-debugger
+[1]> vp run --verbose lint -- -D no-debugger
 $ vp lint -D no-debugger
 
   x eslint(no-debugger): `debugger` statement is not allowed