test: execute steps using @yarnpkg/shell (#292)

### What changed?

- Removed `2>&1` redirections from test commands as we can capture stdout/stderr stably.
- Update snapshots with stdout/stderr interleaved.
- Refactored the snap test runner to use `@yarnpkg/shell`

### Why make this change?

To get prepared to run on Windows

Author: wan9chi

.gitmodules

@@ -2,23 +2,21 @@
 
   vitepress v<semver>
 
-build complete in <variable>ms.
-
-
 - building client + server bundles...
 ✓ building client + server bundles...
 - rendering pages...
 ✓ rendering pages...
+build complete in <variable>ms.
+
 
 > vite doc build # build documentation again should hit the cache
 ✓ cache hit, replaying
 
   vitepress v<semver>
 
-build complete in <variable>ms.
-
-
 - building client + server bundles...
 ✓ building client + server bundles...
 - rendering pages...
 ✓ rendering pages...
+build complete in <variable>ms.
+

mise.toml

@@ -1,4 +1,4 @@
-> vite run build 2>&1 # initial run, create the cache
+> vite run build # initial run, create the cache
 $ node build.js
 [build.js] --------------------------------
 [build.js] start
@@ -108,7 +108,7 @@ Task Details:
       → Cache miss: no previous cache entry found
 ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
 
-> vite run build 2>&1 # should hit the cache
+> vite run build # should hit the cache
 $ node build.js (✓ cache hit, replaying)
 [build.js] --------------------------------
 [build.js] start
@@ -218,7 +218,7 @@ Task Details:
       → Cache hit - output replayed - <variable>ms saved
 ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
 
-> vite run build 2>&1 # should hit the cache and make sure the replay order is chronological
+> vite run build # should hit the cache and make sure the replay order is chronological
 $ node build.js (✓ cache hit, replaying)
 [build.js] --------------------------------
 [build.js] start
@@ -328,7 +328,7 @@ Task Details:
       → Cache hit - output replayed - <variable>ms saved
 ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
 
-> vite run build 2>&1 # should hit the cache and make sure the replay order is chronological again
+> vite run build # should hit the cache and make sure the replay order is chronological again
 $ node build.js (✓ cache hit, replaying)
 [build.js] --------------------------------
 [build.js] start
@@ -438,7 +438,7 @@ Task Details:
       → Cache hit - output replayed - <variable>ms saved
 ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
 
-> vite run build 2>&1 # should hit the cache and make sure the replay order is chronological again
+> vite run build # should hit the cache and make sure the replay order is chronological again
 $ node build.js (✓ cache hit, replaying)
 [build.js] --------------------------------
 [build.js] start

packages/cli/snap-tests/command-doc/snap.txt

@@ -3,10 +3,10 @@
     "VITE_DISABLE_AUTO_INSTALL": "1"
   },
   "commands": [
-    "vite run build 2>&1 # initial run, create the cache",
-    "vite run build 2>&1 # should hit the cache",
-    "vite run build 2>&1 # should hit the cache and make sure the replay order is chronological",
-    "vite run build 2>&1 # should hit the cache and make sure the replay order is chronological again",
-    "vite run build 2>&1 # should hit the cache and make sure the replay order is chronological again"
+    "vite run build # initial run, create the cache",
+    "vite run build # should hit the cache",
+    "vite run build # should hit the cache and make sure the replay order is chronological",
+    "vite run build # should hit the cache and make sure the replay order is chronological again",
+    "vite run build # should hit the cache and make sure the replay order is chronological again"
   ]
 }

packages/cli/snap-tests/replay-logs-chronological-order/snap.txt

@@ -145,14 +145,12 @@ packages:
 
 [1]> vp add --filter app test-vite-plus-package-optional --save-catalog-name v1 # should error because save-catalog-name is not supported at pnpm@9
 Running: pnpm --filter app add --save-catalog-name=v1 test-vite-plus-package-optional
-
  ERROR  Unknown option: 'save-catalog-name'
 Did you mean 'save-optional'? Use "--config.unknown=value" to force an unknown option.
 For help, run: pnpm help add
 
 [1]> vp add --filter=./packages/utils test-vite-plus-package-optional -O --save-catalog v2 # should error because save-catalog is not supported at pnpm@9
 Running: pnpm --filter ./packages/utils add --save-optional --save-catalog test-vite-plus-package-optional v2
-
  ERROR  Unknown option: 'save-catalog'
 Did you mean 'save-exact', or 'save-prod'? Use "--config.unknown=value" to force an unknown option.
 For help, run: pnpm help add

packages/cli/snap-tests/replay-logs-chronological-order/steps.json

@@ -78,7 +78,6 @@ Done in <variable>ms using pnpm v<semver>
 
 [1]> vp add testnpm2 test-vite-plus-install --allow-build=test-vite-plus-install # should error because allow-build is not supported at pnpm@9
 Running: pnpm add --allow-build=test-vite-plus-install testnpm2 test-vite-plus-install
-
  ERROR  Unknown option: 'allow-build'
 For help, run: pnpm help add
 

packages/global/snap-tests/command-add-pnpm9-with-workspace/snap.txt

@@ -32,6 +32,7 @@ up to date in <variable>ms
 
 > vp up -D --filter app -- --no-audit && cat packages/app/package.json # should update dev dependencies in app
 Running: npm update --workspace app --include=dev --no-audit
+npm warn workspaces app in filter set, but no workspace folder present
 
 up to date in <variable>ms
 {
@@ -45,10 +46,10 @@ up to date in <variable>ms
   }
 }
 
-npm warn workspaces app in filter set, but no workspace folder present
-
 > vp update --filter "*" -- --no-audit && cat packages/app/package.json packages/utils/package.json # should update in all packages
 Running: npm update --workspace * --no-audit
+npm warn workspaces app in filter set, but no workspace folder present
+npm warn workspaces @vite-plus-test/utils in filter set, but no workspace folder present
 
 up to date in <variable>ms
 {
@@ -69,11 +70,10 @@ up to date in <variable>ms
   }
 }
 
-npm warn workspaces app in filter set, but no workspace folder present
-npm warn workspaces @vite-plus-test/utils in filter set, but no workspace folder present
-
 > vp update -r --no-save -- --no-audit && cat package.json packages/app/package.json # should update recursively without saving
 Running: npm update --include-workspace-root --workspaces --no-save --no-audit
+npm warn workspaces app in filter set, but no workspace folder present
+npm warn workspaces @vite-plus-test/utils in filter set, but no workspace folder present
 
 up to date in <variable>ms
 {
@@ -98,9 +98,6 @@ up to date in <variable>ms
   }
 }
 
-npm warn workspaces app in filter set, but no workspace folder present
-npm warn workspaces @vite-plus-test/utils in filter set, but no workspace folder present
-
 > vp update --workspace --filter app @vite-plus-test/utils -- --no-audit && cat packages/app/package.json # should update workspace dependency
 Running: npm update --workspace app --no-audit @vite-plus-test/utils
 

packages/global/snap-tests/command-add-pnpm9/snap.txt

@@ -84,6 +84,10 @@ Running: npm install --save-optional --no-audit testnpm2@<semver>
 added 1 package in <variable>ms
 Warning: npm doesn't support --latest flag. Updating within semver range only.
 Running: npm update --no-optional --no-audit
+npm warn config optional Use `--omit=optional` to exclude optional dependencies, or
+npm warn config `--include=optional` to include them.
+npm warn config
+npm warn config       Default value does install optional deps unless otherwise omitted.
 
 changed 1 package in <variable>ms
 {
@@ -99,11 +103,6 @@ changed 1 package in <variable>ms
   }
 }
 
-npm warn config optional Use `--omit=optional` to exclude optional dependencies, or
-npm warn config `--include=optional` to include them.
-npm warn config
-npm warn config       Default value does install optional deps unless otherwise omitted.
-
 > vp update -- --no-audit && cat package.json # should update all packages but won't change the package.json
 Running: npm update --no-audit
 

packages/global/snap-tests/command-update-npm10-with-workspace/snap.txt

@@ -9,12 +9,16 @@
   "devDependencies": {
     "@oxc-node/cli": "catalog:",
     "@oxc-node/core": "catalog:",
-    "@types/semver": "catalog:",
     "@std/yaml": "catalog:",
+    "@types/semver": "catalog:",
     "minimatch": "catalog:",
     "semver": "catalog:"
   },
   "scripts": {
     "snap-test": "tool snap-test"
+  },
+  "dependencies": {
+    "@yarnpkg/fslib": "catalog:",
+    "@yarnpkg/shell": "catalog:"
   }
 }