reset-framework-cli 1.1.2 → 1.1.4

package/src/lib/process.js

@@ -1,206 +1,213 @@
-import { spawn } from "node:child_process"
-
-import { logger } from "./logger.js"
-
-function sleep(ms) {
-  return new Promise((resolve) => {
-    setTimeout(resolve, ms)
-  })
-}
-
-export function resolveNpmCommand() {
-  return process.platform === "win32" ? "npm.cmd" : "npm"
-}
-
-export function resolvePackageManagerCommand(packageManager = "npm") {
-  if (packageManager === "npm") {
-    return resolveNpmCommand()
-  }
-
-  if (packageManager === "yarn") {
-    return process.platform === "win32" ? "yarn.cmd" : "yarn"
-  }
-
-  if (packageManager === "pnpm") {
-    return process.platform === "win32" ? "pnpm.cmd" : "pnpm"
-  }
-
-  if (packageManager === "bun") {
-    return process.platform === "win32" ? "bun.exe" : "bun"
-  }
-
-  throw new Error(`Unsupported package manager: ${packageManager}`)
-}
-
-export function getInstallCommandArgs(packageManager = "npm") {
-  if (packageManager === "npm") {
-    return ["install", "--include=optional"]
-  }
-
-  if (packageManager === "yarn") {
-    return ["install"]
-  }
-
-  return ["install"]
-}
-
+import { spawn } from "node:child_process"
+
+import { logger } from "./logger.js"
+
+function sleep(ms) {
+  return new Promise((resolve) => {
+    setTimeout(resolve, ms)
+  })
+}
+
+export function resolveNpmCommand() {
+  return process.platform === "win32" ? "npm.cmd" : "npm"
+}
+
+export function resolvePackageManagerCommand(packageManager = "npm") {
+  if (packageManager === "npm") {
+    return resolveNpmCommand()
+  }
+
+  if (packageManager === "yarn") {
+    return process.platform === "win32" ? "yarn.cmd" : "yarn"
+  }
+
+  if (packageManager === "pnpm") {
+    return process.platform === "win32" ? "pnpm.cmd" : "pnpm"
+  }
+
+  if (packageManager === "bun") {
+    return process.platform === "win32" ? "bun.exe" : "bun"
+  }
+
+  throw new Error(`Unsupported package manager: ${packageManager}`)
+}
+
+export function getInstallCommandArgs(packageManager = "npm") {
+  if (packageManager === "npm") {
+    return ["install", "--include=optional"]
+  }
+
+  if (packageManager === "yarn") {
+    return ["install"]
+  }
+
+  return ["install"]
+}
+
 export function formatCommand(command, args = []) {
   return [command, ...args].join(" ")
 }
 
+function shouldUseShell(command) {
+  return process.platform === "win32" && typeof command === "string" && /\.(cmd|bat)$/i.test(command)
+}
+
 export async function runCommand(command, args = [], options = {}) {
   const { cwd, dryRun = false, env } = options
 
   logger.info(`$ ${formatCommand(command, args)}`)
-
-  if (dryRun) {
-    return
-  }
-
+
+  if (dryRun) {
+    return
+  }
+
   await new Promise((resolve, reject) => {
     const child = spawn(command, args, {
       cwd,
       env: env ? { ...process.env, ...env } : process.env,
-      stdio: "inherit"
+      stdio: "inherit",
+      shell: shouldUseShell(command)
     })
 
     child.once("error", reject)
-    child.once("exit", (code, signal) => {
-      if (code === 0) {
-        resolve(undefined)
-        return
-      }
-
-      reject(
-        new Error(
-          signal
-            ? `Command terminated by signal: ${signal}`
-            : `Command exited with code ${code}`
-        )
-      )
-    })
-  })
-}
-
-export async function captureCommandOutput(command, args = [], options = {}) {
-  const { cwd, env } = options
-
-  await Promise.resolve()
-
+    child.once("exit", (code, signal) => {
+      if (code === 0) {
+        resolve(undefined)
+        return
+      }
+
+      reject(
+        new Error(
+          signal
+            ? `Command terminated by signal: ${signal}`
+            : `Command exited with code ${code}`
+        )
+      )
+    })
+  })
+}
+
+export async function captureCommandOutput(command, args = [], options = {}) {
+  const { cwd, env } = options
+
+  await Promise.resolve()
+
   return await new Promise((resolve, reject) => {
     const child = spawn(command, args, {
       cwd,
       env: env ? { ...process.env, ...env } : process.env,
-      stdio: ["ignore", "pipe", "pipe"]
-    })
-
-    let stdout = ""
-    let stderr = ""
-
-    child.stdout?.on("data", (chunk) => {
-      stdout += String(chunk)
+      stdio: ["ignore", "pipe", "pipe"],
+      shell: shouldUseShell(command)
     })
-
-    child.stderr?.on("data", (chunk) => {
-      stderr += String(chunk)
-    })
-
-    child.once("error", reject)
-    child.once("exit", (code, signal) => {
-      if (code === 0) {
-        resolve(stdout.trim())
-        return
-      }
-
-      reject(
-        new Error(
-          signal
-            ? `Command terminated by signal: ${signal}`
-            : (stderr.trim() || stdout.trim() || `Command exited with code ${code}`)
-        )
-      )
-    })
-  })
-}
-
-export function spawnCommand(command, args = [], options = {}) {
-  const { cwd, dryRun = false, env } = options
-
-  logger.info(`$ ${formatCommand(command, args)}`)
-
-  if (dryRun) {
-    return null
-  }
-
+
+    let stdout = ""
+    let stderr = ""
+
+    child.stdout?.on("data", (chunk) => {
+      stdout += String(chunk)
+    })
+
+    child.stderr?.on("data", (chunk) => {
+      stderr += String(chunk)
+    })
+
+    child.once("error", reject)
+    child.once("exit", (code, signal) => {
+      if (code === 0) {
+        resolve(stdout.trim())
+        return
+      }
+
+      reject(
+        new Error(
+          signal
+            ? `Command terminated by signal: ${signal}`
+            : (stderr.trim() || stdout.trim() || `Command exited with code ${code}`)
+        )
+      )
+    })
+  })
+}
+
+export function spawnCommand(command, args = [], options = {}) {
+  const { cwd, dryRun = false, env } = options
+
+  logger.info(`$ ${formatCommand(command, args)}`)
+
+  if (dryRun) {
+    return null
+  }
+
   return spawn(command, args, {
     cwd,
     env: env ? { ...process.env, ...env } : process.env,
-    stdio: "inherit"
+    stdio: "inherit",
+    shell: shouldUseShell(command)
   })
 }
-
-export function waitForProcessExit(child, label) {
-  return new Promise((resolve, reject) => {
-    child.once("error", reject)
-    child.once("exit", (code, signal) => {
-      if (code === 0 || signal === "SIGINT" || signal === "SIGTERM") {
-        resolve(undefined)
-        return
-      }
-
-      reject(
-        new Error(
-          signal
-            ? `${label} terminated by signal: ${signal}`
-            : `${label} exited with code ${code}`
-        )
-      )
-    })
-  })
-}
-
-export function registerChildCleanup(children) {
-  const activeChildren = children.filter(Boolean)
-
-  const cleanup = () => {
-    for (const child of activeChildren) {
-      if (child.exitCode === null && !child.killed) {
-        child.kill("SIGTERM")
-      }
-    }
-  }
-
-  const handleSignal = () => {
-    cleanup()
-    process.exit(0)
-  }
-
-  process.on("SIGINT", handleSignal)
-  process.on("SIGTERM", handleSignal)
-
-  return () => {
-    process.off("SIGINT", handleSignal)
-    process.off("SIGTERM", handleSignal)
-    cleanup()
-  }
-}
-
-export async function waitForUrl(url, options = {}) {
-  const { intervalMs = 250, timeoutMs = 15000 } = options
-  const deadline = Date.now() + timeoutMs
-
-  while (Date.now() < deadline) {
-    try {
-      const response = await fetch(url)
-      if (response.status < 500) {
-        return
-      }
-    } catch {
-      // Retry until timeout.
-    }
-
-    await sleep(intervalMs)
-  }
-
-  throw new Error(`Timed out waiting for ${url}`)
-}
+
+export function waitForProcessExit(child, label) {
+  return new Promise((resolve, reject) => {
+    child.once("error", reject)
+    child.once("exit", (code, signal) => {
+      if (code === 0 || signal === "SIGINT" || signal === "SIGTERM") {
+        resolve(undefined)
+        return
+      }
+
+      reject(
+        new Error(
+          signal
+            ? `${label} terminated by signal: ${signal}`
+            : `${label} exited with code ${code}`
+        )
+      )
+    })
+  })
+}
+
+export function registerChildCleanup(children) {
+  const activeChildren = children.filter(Boolean)
+
+  const cleanup = () => {
+    for (const child of activeChildren) {
+      if (child.exitCode === null && !child.killed) {
+        child.kill("SIGTERM")
+      }
+    }
+  }
+
+  const handleSignal = () => {
+    cleanup()
+    process.exit(0)
+  }
+
+  process.on("SIGINT", handleSignal)
+  process.on("SIGTERM", handleSignal)
+
+  return () => {
+    process.off("SIGINT", handleSignal)
+    process.off("SIGTERM", handleSignal)
+    cleanup()
+  }
+}
+
+export async function waitForUrl(url, options = {}) {
+  const { intervalMs = 250, timeoutMs = 15000 } = options
+  const deadline = Date.now() + timeoutMs
+
+  while (Date.now() < deadline) {
+    try {
+      const response = await fetch(url)
+      if (response.status < 500) {
+        return
+      }
+    } catch {
+      // Retry until timeout.
+    }
+
+    await sleep(intervalMs)
+  }
+
+  throw new Error(`Timed out waiting for ${url}`)
+}