test(e2e): verify old output archive is removed on re-run

Adds `vtt list-dir <dir> [--ext <suffix>]` for inspecting cache state
from e2e tests, plus a UUID redactor so archive filenames (which are
random per run) stay stable across snapshot regenerations.

The new `output_globs___old_archive_removed_on_rewrite` fixture asserts
that after two cache-missing runs of the same task, the cache directory
contains exactly one `.tar.zst` archive — proving the cleanup path in
`ExecutionCache::update` actually runs.

Author: wan9chi

crates/vite_task_bin/src/vtt/list_dir.rs

@@ -0,0 +1,44 @@
+/// List entries in a directory, sorted by name, one per line.
+///
+/// Usage: `vtt list-dir <dir> [--ext <suffix>]`
+///
+/// With `--ext`, only entries whose filename ends with the given suffix are
+/// printed (the leading `.` is part of the suffix you pass, e.g. `.tar.zst`).
+///
+/// Used by e2e tests to assert on cache directory contents (e.g. exactly one
+/// `.tar.zst` archive after a re-run that should have cleaned up the prior
+/// archive).
+pub fn run(args: &[String]) -> Result<(), Box<dyn std::error::Error>> {
+    let mut dir: Option<&str> = None;
+    let mut ext: Option<&str> = None;
+    let mut i = 0;
+    while i < args.len() {
+        match args[i].as_str() {
+            "--ext" => {
+                i += 1;
+                ext = Some(args.get(i).ok_or("--ext requires a value")?.as_str());
+            }
+            other if dir.is_none() => dir = Some(other),
+            other => return Err(format!("unexpected argument: {other}").into()),
+        }
+        i += 1;
+    }
+    let dir = dir.ok_or("Usage: vtt list-dir <dir> [--ext <suffix>]")?;
+
+    let mut names: Vec<String> = Vec::new();
+    for entry in std::fs::read_dir(dir)? {
+        let entry = entry?;
+        let name = entry.file_name().to_string_lossy().into_owned();
+        if let Some(suffix) = ext
+            && !name.ends_with(suffix)
+        {
+            continue;
+        }
+        names.push(name);
+    }
+    names.sort();
+    for name in names {
+        println!("{name}");
+    }
+    Ok(())
+}

crates/vite_task_bin/src/vtt/main.rs

@@ -11,6 +11,7 @@ mod check_tty;
 mod cp;
 mod exit;
 mod exit_on_ctrlc;
+mod list_dir;
 mod mkdir;
 mod pipe_stdin;
 mod print;
@@ -28,7 +29,7 @@ fn main() {
     if args.len() < 2 {
         eprintln!("Usage: vtt <subcommand> [args...]");
         eprintln!(
-            "Subcommands: barrier, check-tty, cp, exit, exit-on-ctrlc, mkdir, pipe-stdin, print, print-cwd, print-env, print-file, read-stdin, replace-file-content, rm, touch-file, write-file"
+            "Subcommands: barrier, check-tty, cp, exit, exit-on-ctrlc, list-dir, mkdir, pipe-stdin, print, print-cwd, print-env, print-file, read-stdin, replace-file-content, rm, touch-file, write-file"
         );
         std::process::exit(1);
     }
@@ -42,6 +43,7 @@ fn main() {
         "cp" => cp::run(&args[2..]),
         "exit" => exit::run(&args[2..]),
         "exit-on-ctrlc" => exit_on_ctrlc::run(),
+        "list-dir" => list_dir::run(&args[2..]),
         "mkdir" => mkdir::run(&args[2..]),
         "pipe-stdin" => pipe_stdin::run(&args[2..]),
         "print" => {

crates/vite_task_bin/tests/e2e_snapshots/fixtures/output_cache_test/snapshots.toml

@@ -18,6 +18,27 @@ steps = [
   ["vtt", "print-file", "dist/output.txt"],
 ]
 
+[[e2e]]
+name = "output_globs___old_archive_removed_on_rewrite"
+comment = """
+When a cached task re-runs (cache miss because an input changed), it
+writes a new archive and the previous archive file is cleaned up. After
+two runs of the same task the cache directory still contains only one
+\\\".tar.zst\\\" file.
+"""
+steps = [
+  # First run - cache miss, produces archive A
+  ["vt", "run", "build"],
+  # Exactly one archive on disk
+  ["vtt", "list-dir", "node_modules/.vite/task-cache", "--ext", ".tar.zst"],
+  # Modify the input so the next run is a cache miss
+  ["vtt", "write-file", "src/main.ts", "changed"],
+  # Second run - cache miss, produces archive B; archive A is deleted
+  ["vt", "run", "build"],
+  # Still only one archive on disk (A removed, B written)
+  ["vtt", "list-dir", "node_modules/.vite/task-cache", "--ext", ".tar.zst"],
+]
+
 [[e2e]]
 name = "output_globs___negative_excludes_files_from_archive"
 comment = """

crates/vite_task_bin/tests/e2e_snapshots/fixtures/output_cache_test/snapshots/output_globs___old_archive_removed_on_rewrite.md

@@ -0,0 +1,35 @@
+# output_globs___old_archive_removed_on_rewrite
+
+When a cached task re-runs (cache miss because an input changed), it
+writes a new archive and the previous archive file is cleaned up. After
+two runs of the same task the cache directory still contains only one
+\".tar.zst\" file.
+
+## `vt run build`
+
+```
+$ vtt write-file dist/output.txt built
+```
+
+## `vtt list-dir node_modules/.vite/task-cache --ext .tar.zst`
+
+```
+<uuid>.tar.zst
+```
+
+## `vtt write-file src/main.ts changed`
+
+```
+```
+
+## `vt run build`
+
+```
+$ vtt write-file dist/output.txt built ○ cache miss: 'src/main.ts' modified, executing
+```
+
+## `vtt list-dir node_modules/.vite/task-cache --ext .tar.zst`
+
+```
+<uuid>.tar.zst
+```

crates/vite_task_bin/tests/e2e_snapshots/redact.rs

@@ -64,6 +64,11 @@ pub fn redact_e2e_output(mut output: String, workspace_root: &str) -> String {
 
     redact_string(&mut output, &redactions);
 
+    // Redact UUIDs (e.g. cache archive filenames `<uuid>.tar.zst`) to "<uuid>"
+    let uuid_regex =
+        regex::Regex::new(r"[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}").unwrap();
+    output = uuid_regex.replace_all(&output, "<uuid>").into_owned();
+
     // Redact durations like "0ns", "123ms" or "1.23s" to "<duration>"
     let duration_regex = regex::Regex::new(r"\d+(\.\d+)?(ns|ms|s)").unwrap();
     output = duration_regex.replace_all(&output, "<duration>").into_owned();