fix: use CLI args instead of inline env vars for snap-test-global on Windows

Windows cmd.exe doesn't support `VAR=val command` syntax, causing
snap-test-global to fail with 'SNAP_TEST_DIR is not recognized'.
Replace with --dir and --bin-dir CLI args with ~ expansion via os.homedir().

Author: fengmk2

packages/cli/package.json

@@ -291,7 +291,7 @@
     "build-native": "oxnode -C dev ./build.ts --skip-ts",
     "snap-test": "pnpm snap-test-local && pnpm snap-test-global",
     "snap-test-local": "tool snap-test",
-    "snap-test-global": "SNAP_TEST_DIR=snap-tests-global SNAP_TEST_BIN_DIR=$HOME/.vite-plus/current/bin tool snap-test",
+    "snap-test-global": "tool snap-test --dir snap-tests-global --bin-dir ~/.vite-plus/current/bin",
     "publish-native": "node ./publish-native-addons.ts",
     "test": "vitest run"
   },

packages/tools/src/snap-test.ts

@@ -62,10 +62,18 @@ async function runWithConcurrencyLimit(
   }
 }
 
+function expandHome(p: string): string {
+  return p.startsWith('~') ? path.join(homedir(), p.slice(1)) : p;
+}
+
 export async function snapTest() {
-  const { positionals } = parseArgs({
+  const { positionals, values } = parseArgs({
     allowPositionals: true,
     args: process.argv.slice(3),
+    options: {
+      dir: { type: 'string' },
+      'bin-dir': { type: 'string' },
+    },
   });
 
   const filter = positionals[0] ?? ''; // Optional filter to run specific test cases
@@ -115,15 +123,15 @@ export async function snapTest() {
   // Clean up the temporary directory on exit
   process.on('exit', () => fs.rmSync(tempTmpDir, { recursive: true, force: true }));
 
-  const casesDir = path.resolve(process.env.SNAP_TEST_DIR || 'snap-tests');
+  const casesDir = path.resolve(values.dir || 'snap-tests');
 
   const taskFunctions: (() => Promise<void>)[] = [];
   for (const caseName of fs.readdirSync(casesDir)) {
     if (caseName.startsWith('.')) {
       continue;
     } // Skip hidden files like .DS_Store
     if (caseName.includes(filter)) {
-      taskFunctions.push(() => runTestCase(caseName, tempTmpDir, casesDir));
+      taskFunctions.push(() => runTestCase(caseName, tempTmpDir, casesDir, values['bin-dir']));
     }
   }
 
@@ -165,7 +173,7 @@ interface Steps {
   after?: string[];
 }
 
-async function runTestCase(name: string, tempTmpDir: string, casesDir: string) {
+async function runTestCase(name: string, tempTmpDir: string, casesDir: string, binDir?: string) {
   const steps: Steps = JSON.parse(
     await fsPromises.readFile(`${casesDir}/${name}/steps.json`, 'utf-8'),
   );
@@ -210,9 +218,9 @@ async function runTestCase(name: string, tempTmpDir: string, casesDir: string) {
   }
   env['PATH'] = [
     // Extend PATH to include the package's bin directory
-    // SNAP_TEST_BIN_DIR overrides the default for cases like global CLI tests
+    // --bin-dir overrides the default for cases like global CLI tests
     // where vp should resolve to the Rust binary instead of the Node.js script
-    path.resolve(process.env.SNAP_TEST_BIN_DIR || 'bin'),
+    path.resolve(expandHome(binDir || 'bin')),
     ...env['PATH'].split(path.delimiter),
   ].join(path.delimiter);