refactor(scripts): use @socketsecurity/lib/spawn for child processes

Update scripts to use spawn from @socketsecurity/lib for consistent
cross-platform behavior.

Author: jdalton

scripts/utils/run-command.mjs

@@ -14,11 +14,21 @@ const logger = getDefaultLogger()
  * @returns {Promise<number>} Exit code
  */
 export async function runCommand(command, args = [], options = {}) {
-  const result = await spawn(command, args, {
-    stdio: 'inherit',
-    ...options,
-  })
-  return result.code
+  try {
+    const result = await spawn(command, args, {
+      stdio: 'inherit',
+      ...(process.platform === 'win32' && { shell: true }),
+      ...options,
+    })
+    return result.code
+  } catch (error) {
+    // spawn() from @socketsecurity/lib throws on non-zero exit
+    // Return the exit code from the error
+    if (error && typeof error === 'object' && 'code' in error) {
+      return error.code
+    }
+    throw error
+  }
 }
 
 /**
@@ -75,23 +85,43 @@ export async function runParallel(commands) {
 }
 
 /**
- * Run a command and capture output.
+ * Run a command and suppress output.
  * @param {string} command - The command to run
  * @param {string[]} args - Arguments to pass to the command
  * @param {object} options - Spawn options
  * @returns {Promise<{exitCode: number, stdout: string, stderr: string}>}
  */
 export async function runCommandQuiet(command, args = [], options = {}) {
-  const result = await spawn(command, args, {
-    ...options,
-    stdio: ['inherit', 'pipe', 'pipe'],
-    stdioString: true,
-  })
+  try {
+    const result = await spawn(command, args, {
+      ...options,
+      ...(process.platform === 'win32' && { shell: true }),
+      stdio: 'pipe',
+      stdioString: true,
+    })
 
-  return {
-    exitCode: result.code,
-    stderr: result.stderr,
-    stdout: result.stdout,
+    return {
+      exitCode: result.code,
+      stderr: result.stderr,
+      stdout: result.stdout,
+    }
+  } catch (error) {
+    // spawn() from @socketsecurity/lib throws on non-zero exit
+    // Return the exit code and output from the error
+    if (
+      error &&
+      typeof error === 'object' &&
+      'code' in error &&
+      'stdout' in error &&
+      'stderr' in error
+    ) {
+      return {
+        exitCode: error.code,
+        stderr: error.stderr,
+        stdout: error.stdout,
+      }
+    }
+    throw error
   }
 }