Skip to content

Commit 542c407

Browse files
branchseerclaude
andcommitted
fix: update snaps for version numbers exposed by ANSI stripping
- command-env-which: versions in "Version:" and "Node:" fields are now matched by semver regex after ANSI codes that previously hid them are stripped - cache-scripts-default: remove trailing blank line to match snap test output Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent 10151d5 commit 542c407

File tree

2 files changed

+4
-5
lines changed
  • packages/cli
    • snap-tests-global/command-env-which
    • snap-tests/cache-scripts-default

2 files changed

+4
-5
lines changed

packages/cli/snap-tests-global/command-env-which/snap.txt

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -3,17 +3,17 @@ v20.18.0
33

44
> vp env which node # Core tool - shows resolved Node.js binary path
55
<vite-plus-home>/js_runtime/node/<semver>/bin/node
6-
Version: 20.18.0
6+
Version: <semver>
77
Source: .node-version
88

99
> vp env which npm # Core tool - shows resolved npm binary path
1010
<vite-plus-home>/js_runtime/node/<semver>/bin/npm
11-
Version: 20.18.0
11+
Version: <semver>
1212
Source: .node-version
1313

1414
> vp env which npx # Core tool - shows resolved npx binary path
1515
<vite-plus-home>/js_runtime/node/<semver>/bin/npx
16-
Version: 20.18.0
16+
Version: <semver>
1717
Source: .node-version
1818

1919
> vp install -g cowsay@1.6.0 # Install a global package via vp
@@ -31,7 +31,7 @@ added 41 packages in <variable>ms
3131
<vite-plus-home>/packages/cowsay/lib/node_modules/cowsay/./cli.js
3232
Package: cowsay@<semver>
3333
Binaries: cowsay, cowthink
34-
Node: 20.18.0
34+
Node: <semver>
3535
Installed: <date>
3636

3737
> vp remove -g cowsay # Cleanup

packages/cli/snap-tests/cache-scripts-default/snap.txt

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,4 +3,3 @@ $ node hello.mjs ⊘ cache disabled: no cache config
33

44
> vp run hello 2>&1 | grep 'cache disabled' # second run should also show cache disabled
55
$ node hello.mjs ⊘ cache disabled: no cache config
6-

0 commit comments

Comments
 (0)