fix(ci): skip snap tests that install vite-plus from npm on musl

Published vite-plus 0.1.13 doesn't include musl NAPI bindings yet.
Skip tests that depend on the npm-published package on musl targets:
- migration tests that install vite-plus@latest
- command-pack-exe (uses tsdown which loads rolldown binding)
- command-upgrade-check

Remove debug steps from CI now that root cause is identified.

Author: fengmk2

.github/workflows/ci.yml

@@ -669,13 +669,6 @@ jobs:
               cp ./package/rolldown-binding.linux-x64-musl.node ./rolldown/packages/rolldown/dist/shared/
               rm -rf package *.tgz
 
-              # Debug: verify rolldown musl binding exists
-              echo '=== Verify rolldown musl binding ==='
-              ls -la ./packages/core/dist/rolldown/shared/rolldown-binding.linux-x64-musl.node
-              ls -la ./rolldown/packages/rolldown/dist/shared/rolldown-binding.linux-x64-musl.node
-              # Also check the rolldown src dir (used by rolldown source builds)
-              ls -la ./rolldown/packages/rolldown/src/rolldown-binding.*.node
-
               pnpm bootstrap-cli:ci
               export PATH=\"/root/.vite-plus/bin:\$PATH\"
 
@@ -693,32 +686,15 @@ jobs:
               tsc --version
               vp uninstall -g typescript
 
-              # Debug: run failing snap tests manually to see full output
-              echo '=== Debug: command-pack-exe ==='
-              cd packages/cli/snap-tests-global/command-pack-exe
-              VITE_DISABLE_AUTO_INSTALL=1 vp pack src/index.ts --exe 2>&1 || true
-              ls dist 2>&1 || true
-              ls build 2>&1 || true
-              ./build/index 2>&1 || true
-              rm -rf dist build
-              cd /workspace
-
-              echo '=== Debug: migration-auto-create-vite-config ==='
-              TMPDIR=\$(mktemp -d)
-              cp -r packages/cli/snap-tests-global/migration-auto-create-vite-config/* \$TMPDIR/
-              cd \$TMPDIR
-              echo '--- vp migrate --no-interactive ---'
-              vp migrate --no-interactive 2>&1 || true
-              echo '--- cat vite.config.ts ---'
-              cat vite.config.ts 2>&1 || true
-              echo '--- cat .oxlintrc.json ---'
-              cat .oxlintrc.json 2>&1 || true
-              echo '--- cat .oxfmtrc.json ---'
-              cat .oxfmtrc.json 2>&1 || true
-              echo '--- cat package.json ---'
-              cat package.json 2>&1 || true
-              cd /workspace
-              rm -rf \$TMPDIR
+              # Run snap tests
+              git config --global --add safe.directory /workspace
+              RUST_BACKTRACE=1 pnpm test
+              if ! git diff --exit-code; then
+                echo '::error::Snapshot diff detected. Run pnpm -F vite-plus snap-test locally and commit the updated snap.txt files.'
+                git diff --stat
+                git diff
+                exit 1
+              fi
             "
 
   install-e2e-test:

packages/cli/snap-tests-global/command-pack-exe/steps.json

@@ -1,8 +1,22 @@
 {
-  "ignoredPlatforms": ["win32"],
+  "ignoredPlatforms": [
+    "win32",
+    {
+      "os": "linux",
+      "libc": "musl"
+    }
+  ],
   "env": {
     "VITE_DISABLE_AUTO_INSTALL": "1"
   },
-  "commands": ["vp pack src/index.ts --exe 2>&1", "ls dist", "ls build", "./build/index"],
-  "after": ["rm -rf dist", "rm -rf build"]
+  "commands": [
+    "vp pack src/index.ts --exe 2>&1",
+    "ls dist",
+    "ls build",
+    "./build/index"
+  ],
+  "after": [
+    "rm -rf dist",
+    "rm -rf build"
+  ]
 }

packages/cli/snap-tests-global/migration-already-vite-plus-with-husky-hookspath/steps.json

@@ -1,10 +1,22 @@
 {
   "commands": [
-    { "command": "git init", "ignoreOutput": true },
-    { "command": "git config core.hooksPath .husky/_", "ignoreOutput": true },
+    {
+      "command": "git init",
+      "ignoreOutput": true
+    },
+    {
+      "command": "git config core.hooksPath .husky/_",
+      "ignoreOutput": true
+    },
     "vp migrate --no-interactive # should override husky's core.hooksPath and migrate hooks",
     "cat package.json # husky/lint-staged should be removed, prepare should be vp config",
     "cat .vite-hooks/pre-commit # pre-commit hook should be rewritten",
     "git config --local core.hooksPath # should be .vite-hooks/_"
+  ],
+  "ignoredPlatforms": [
+    {
+      "os": "linux",
+      "libc": "musl"
+    }
   ]
 }

packages/cli/snap-tests-global/migration-already-vite-plus-with-husky-lint-staged/steps.json

@@ -1,9 +1,18 @@
 {
   "commands": [
-    { "command": "git init", "ignoreOutput": true },
+    {
+      "command": "git init",
+      "ignoreOutput": true
+    },
     "vp migrate --no-interactive # should still migrate husky/lint-staged even though vite-plus exists",
     "cat package.json # husky/lint-staged should be removed, prepare should be vp config",
     "cat .vite-hooks/pre-commit # pre-commit hook should be rewritten",
     "test -d .husky && echo '.husky directory exists' || echo 'No .husky directory' # .husky should be removed"
+  ],
+  "ignoredPlatforms": [
+    {
+      "os": "linux",
+      "libc": "musl"
+    }
   ]
 }

packages/cli/snap-tests-global/migration-already-vite-plus/steps.json

@@ -1,3 +1,11 @@
 {
-  "commands": ["vp migrate --no-interactive # should detect existing vite-plus and exit"]
+  "commands": [
+    "vp migrate --no-interactive # should detect existing vite-plus and exit"
+  ],
+  "ignoredPlatforms": [
+    {
+      "os": "linux",
+      "libc": "musl"
+    }
+  ]
 }

packages/cli/snap-tests-global/migration-eslint-legacy-already-vite-plus/steps.json

@@ -1,5 +1,11 @@
 {
   "commands": [
     "vp migrate --no-interactive # should show legacy config warning and already using Vite+"
+  ],
+  "ignoredPlatforms": [
+    {
+      "os": "linux",
+      "libc": "musl"
+    }
   ]
 }

packages/cli/snap-tests-global/migration-eslint-lint-staged-mjs/steps.json

@@ -2,5 +2,11 @@
   "commands": [
     "vp migrate --no-interactive # migration should warn about non-JSON lint-staged config",
     "cat lint-staged.config.mjs # verify non-JSON lint-staged config is preserved unchanged"
+  ],
+  "ignoredPlatforms": [
+    {
+      "os": "linux",
+      "libc": "musl"
+    }
   ]
 }

packages/cli/snap-tests-global/migration-eslint-rerun-dual-config/steps.json

@@ -5,5 +5,11 @@
     "cat eslint.config.mjs && exit 1 || true # check flat config is removed",
     "cat .eslintrc && exit 1 || true # check legacy config is also removed",
     "cat vite.config.ts # check oxlint config merged into vite.config.ts"
+  ],
+  "ignoredPlatforms": [
+    {
+      "os": "linux",
+      "libc": "musl"
+    }
   ]
 }

packages/cli/snap-tests-global/migration-eslint-rerun-mjs/steps.json

@@ -4,5 +4,11 @@
     "cat package.json # check eslint removed from devDependencies and scripts rewritten",
     "cat eslint.config.mjs && exit 1 || true # check eslint config is removed",
     "cat vite.config.mjs # check oxlint config merged into existing vite.config.mjs (not creating vite.config.ts)"
+  ],
+  "ignoredPlatforms": [
+    {
+      "os": "linux",
+      "libc": "musl"
+    }
   ]
 }

packages/cli/snap-tests-global/migration-eslint-rerun/steps.json

@@ -4,5 +4,11 @@
     "cat package.json # check eslint removed from devDependencies and scripts rewritten",
     "cat eslint.config.mjs && exit 1 || true # check eslint config is removed",
     "cat vite.config.ts # check oxlint config merged into vite.config.ts"
+  ],
+  "ignoredPlatforms": [
+    {
+      "os": "linux",
+      "libc": "musl"
+    }
   ]
 }