chore: improve bootstrap-cli and snap-test output handling (#323)

Author: fengmk2

package.json

@@ -8,7 +8,7 @@
   },
   "type": "module",
   "scripts": {
-    "bootstrap-cli": "pnpm --filter=@voidzero-dev/vite-plus build && pnpm --filter=@voidzero-dev/global build && pnpm install-global-cli",
+    "bootstrap-cli": "pnpm --filter=@voidzero-dev/* build && pnpm install-global-cli",
     "bootstrap-cli:ci": "pnpm --filter=@voidzero-dev/global build && pnpm install-global-cli",
     "install-global-cli": "npm install -g ./packages/global",
     "typecheck": "tsc -b tsconfig.json",

packages/tools/src/snap-test.ts

@@ -184,9 +184,9 @@ async function runTestCase(name: string, tempTmpDir: string, casesDir: string) {
         stdin: null,
         // Declared to be `Writable` but `FileHandle` works too.
         // @ts-expect-error
-        stderr: cmd.ignoreOutput ? null : outputStream,
+        stderr: outputStream,
         // @ts-expect-error
-        stdout: cmd.ignoreOutput ? null : outputStream,
+        stdout: outputStream,
         glob: {
           // Disable glob expansion. Pass args like '--filter=*' as-is.
           isGlobPattern: () => false,
@@ -198,11 +198,17 @@ async function runTestCase(name: string, tempTmpDir: string, casesDir: string) {
 
     await outputStream.close();
 
-    const output = readFileSync(outputStreamPath, 'utf-8');
+    let output = readFileSync(outputStreamPath, 'utf-8');
 
     let commandLine = `> ${cmd.command}`;
     if (exitCode !== 0) {
-      commandLine = (exitCode === undefined ? '[timeout]' : `[${exitCode}]`) + commandLine;
+      commandLine =
+        (exitCode === undefined ? '[timeout]' : `[${exitCode}]`) + commandLine;
+    } else {
+      // only allow ignore output if the command is successful
+      if (cmd.ignoreOutput) {
+        output = '';
+      }
     }
     newSnap.push(commandLine);
     if (output.length > 0) {

vite-task.json

@@ -7,7 +7,10 @@
         "rolldown#build-binding:release",
         "rolldown#build-node",
         "rolldown-vite#build-types",
-        "@voidzero-dev/vite-plus#build"
+        "@voidzero-dev/vite-plus-test#build",
+        "@voidzero-dev/vite-plus-core#build",
+        "@voidzero-dev/vite-plus#build",
+        "@voidzero-dev/global#build"
       ],
       "cacheable": false
     }