fix: update vite-task rev and fix CI snap test failures

branchseer · branchseer · commit 2d0c478a9806 · 2026-02-25T11:42:50.000+08:00
- Update vite-task to 7d49f24f (summary-output e2e tests, fspy revert)
- Strip ANSI escape codes in replaceUnstableOutput — third-party tools
  like rolldown emit ANSI on stderr even with NO_COLOR, causing [33m
  fragments in snap output on Linux/Windows
- Add .gitignore with 'node_modules' to ignore_dist, plain-terminal-ui-nested,
  and oxlint-typeaware fixtures to prevent oxlint from traversing
  node_modules/.vite/task-cache (causes spurious cache misses)
- Update ignore_dist snap: now correctly shows cache hit after adding .gitignore
- Update command-helper/command-vp-alias snaps: ANSI stripped from clap help output
diff --git a/Cargo.lock b/Cargo.lock
diff --git a/Cargo.toml b/Cargo.toml
@@ -82,7 +82,7 @@ dunce = "1.0.5"
 fast-glob = "1.0.0"
 flate2 = { version = "=1.1.5", features = ["zlib-rs"] }
 form_urlencoded = "1.2.1"
-fspy = { git = "ssh://git@github.com/voidzero-dev/vite-task.git", rev = "fcc662b94410e6f68dececfe8ebf099eb7487ab6" }
+fspy = { git = "ssh://git@github.com/voidzero-dev/vite-task.git", rev = "7d49f24fb40ada5d86fbb7541421d516c9f4dfef" }
 futures = "0.3.31"
 futures-util = "0.3.31"
 glob = "0.3.2"
@@ -181,14 +181,14 @@ vfs = "0.12.1"
 vite_command = { path = "crates/vite_command" }
 vite_error = { path = "crates/vite_error" }
 vite_js_runtime = { path = "crates/vite_js_runtime" }
-vite_glob = { git = "ssh://git@github.com/voidzero-dev/vite-task.git", rev = "fcc662b94410e6f68dececfe8ebf099eb7487ab6" }
+vite_glob = { git = "ssh://git@github.com/voidzero-dev/vite-task.git", rev = "7d49f24fb40ada5d86fbb7541421d516c9f4dfef" }
 vite_install = { path = "crates/vite_install" }
 vite_migration = { path = "crates/vite_migration" }
 vite_shared = { path = "crates/vite_shared" }
-vite_path = { git = "ssh://git@github.com/voidzero-dev/vite-task.git", rev = "fcc662b94410e6f68dececfe8ebf099eb7487ab6" }
-vite_str = { git = "ssh://git@github.com/voidzero-dev/vite-task.git", rev = "fcc662b94410e6f68dececfe8ebf099eb7487ab6" }
-vite_task = { git = "ssh://git@github.com/voidzero-dev/vite-task.git", rev = "fcc662b94410e6f68dececfe8ebf099eb7487ab6" }
-vite_workspace = { git = "ssh://git@github.com/voidzero-dev/vite-task.git", rev = "fcc662b94410e6f68dececfe8ebf099eb7487ab6" }
+vite_path = { git = "ssh://git@github.com/voidzero-dev/vite-task.git", rev = "7d49f24fb40ada5d86fbb7541421d516c9f4dfef" }
+vite_str = { git = "ssh://git@github.com/voidzero-dev/vite-task.git", rev = "7d49f24fb40ada5d86fbb7541421d516c9f4dfef" }
+vite_task = { git = "ssh://git@github.com/voidzero-dev/vite-task.git", rev = "7d49f24fb40ada5d86fbb7541421d516c9f4dfef" }
+vite_workspace = { git = "ssh://git@github.com/voidzero-dev/vite-task.git", rev = "7d49f24fb40ada5d86fbb7541421d516c9f4dfef" }
 walkdir = "2.5.0"
 wax = "0.6.0"
 which = "8.0.0"
diff --git a/packages/cli/snap-tests/command-helper/snap.txt b/packages/cli/snap-tests/command-helper/snap.txt
@@ -1,21 +1,21 @@
 > vp -h # help message
 Vite+/<semver>
 
-[1;4mUsage:[0m [1mvp[0m <COMMAND>
-
-[1;4mCore Commands:[0m
-  [1mdev[0m        Run the development server
-  [1mbuild[0m      Build for production
-  [1mtest[0m       Run tests
-  [1mlint[0m       Lint code
-  [1mfmt[0m        Format code
-  [1mpack[0m       Build library
-  [1mrun[0m        Run tasks
-  [1mpreview[0m    Preview production build
-  [1mcache[0m      Manage the task cache
-
-[1;4mPackage Manager Commands:[0m
-  [1minstall[0m    Install all dependencies, or add packages if package names are provided
+Usage: vp <COMMAND>
+
+Core Commands:
+  dev        Run the development server
+  build      Build for production
+  test       Run tests
+  lint       Lint code
+  fmt        Format code
+  pack       Build library
+  run        Run tasks
+  preview    Preview production build
+  cache      Manage the task cache
+
+Package Manager Commands:
+  install    Install all dependencies, or add packages if package names are provided
 
 Options:
   -h, --help  Print help
diff --git a/packages/cli/snap-tests/command-vp-alias/snap.txt b/packages/cli/snap-tests/command-vp-alias/snap.txt
@@ -1,21 +1,21 @@
 > vp -h # vp should show help same as vite
 Vite+/<semver>
 
-[1;4mUsage:[0m [1mvp[0m <COMMAND>
-
-[1;4mCore Commands:[0m
-  [1mdev[0m        Run the development server
-  [1mbuild[0m      Build for production
-  [1mtest[0m       Run tests
-  [1mlint[0m       Lint code
-  [1mfmt[0m        Format code
-  [1mpack[0m       Build library
-  [1mrun[0m        Run tasks
-  [1mpreview[0m    Preview production build
-  [1mcache[0m      Manage the task cache
-
-[1;4mPackage Manager Commands:[0m
-  [1minstall[0m    Install all dependencies, or add packages if package names are provided
+Usage: vp <COMMAND>
+
+Core Commands:
+  dev        Run the development server
+  build      Build for production
+  test       Run tests
+  lint       Lint code
+  fmt        Format code
+  pack       Build library
+  run        Run tasks
+  preview    Preview production build
+  cache      Manage the task cache
+
+Package Manager Commands:
+  install    Install all dependencies, or add packages if package names are provided
 
 Options:
   -h, --help  Print help
diff --git a/packages/cli/snap-tests/ignore_dist/.gitignore b/packages/cli/snap-tests/ignore_dist/.gitignore
@@ -0,0 +1,7 @@
+# Prevent oxlint from scanning node_modules.
+# Without this, oxlint reads files in node_modules/.vite/task-cache, causing
+# fspy to fingerprint the cache directory as an input. The cache directory
+# changes between runs (DB writes, last-summary.json), producing flaky
+# cache-miss reasons that alternate between '' and
+# 'node_modules/.vite/task-cache'.
+node_modules
diff --git a/packages/cli/snap-tests/ignore_dist/snap.txt b/packages/cli/snap-tests/ignore_dist/snap.txt
@@ -6,7 +6,9 @@ Finished in <variable>ms on 1 file with <variable> rules using <variable> thread
 
 > mkdir dist
 > vp run lint # new dist folder should not invalidate cache
-$ vp lint ✗ cache miss: content of input 'node_modules/.vite/task-cache' changed, executing
+$ vp lint ✓ cache hit, replaying
 Found 0 warnings and 0 errors.
 Finished in <variable>ms on 1 file with <variable> rules using <variable> threads.
 
+---
+[vp run] cache hit, <variable>ms saved.
diff --git a/packages/cli/snap-tests/oxlint-typeaware/.gitignore b/packages/cli/snap-tests/oxlint-typeaware/.gitignore
@@ -0,0 +1,7 @@
+# Prevent oxlint from scanning node_modules.
+# Without this, oxlint reads files in node_modules/.vite/task-cache, causing
+# fspy to fingerprint the cache directory as an input. The cache directory
+# changes between runs (DB writes, last-summary.json), producing flaky
+# cache-miss reasons that alternate between '' and
+# 'node_modules/.vite/task-cache'.
+node_modules
diff --git a/packages/cli/snap-tests/plain-terminal-ui-nested/.gitignore b/packages/cli/snap-tests/plain-terminal-ui-nested/.gitignore
@@ -0,0 +1,7 @@
+# Prevent oxlint from scanning node_modules.
+# Without this, oxlint reads files in node_modules/.vite/task-cache, causing
+# fspy to fingerprint the cache directory as an input. The cache directory
+# changes between runs (DB writes, last-summary.json), producing flaky
+# cache-miss reasons that alternate between '' and
+# 'node_modules/.vite/task-cache'.
+node_modules
diff --git a/packages/tools/src/utils.ts b/packages/tools/src/utils.ts
@@ -4,6 +4,11 @@ import path from 'node:path';
 import { Minimatch } from 'minimatch';
 
 export function replaceUnstableOutput(output: string, cwd?: string) {
+  // Strip ANSI escape codes first. Tests set NO_COLOR=1 so vp output is clean,
+  // but third-party tools (e.g. rolldown) may still emit ANSI codes on stderr
+  // that leak into the captured output.
+  // eslint-disable-next-line no-control-regex
+  output = output.replaceAll(/\x1b\[[0-9;]*m/g, '');
   if (cwd) {
     output = output.replaceAll(cwd, '<cwd>');
     if (path.dirname(cwd) !== '/') {
