romtuck/openclaw
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(exec): scope Windows shell to npm/pnpm/yarn/npx

Browse Source
This commit is contained in:
ThanhNguyxn 2026-01-30 22:36:49 +07:00
parent aaefee78de
commit 59f6fc3748
1 changed files with 4 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
src/process/exec.ts
View File

@ -80,12 +80,15 @@ export async function runCommandWithTimeout(
} }
const stdio = resolveCommandStdio({ hasInput, preferInherit: true }); const stdio = resolveCommandStdio({ hasInput, preferInherit: true });
const cmdTool = path.parse(argv[0] ?? "").name;
const needsShell =
process.platform === "win32" && ["npm", "pnpm", "yarn", "npx"].includes(cmdTool);
const child = spawn(argv[0], argv.slice(1), { const child = spawn(argv[0], argv.slice(1), {
stdio, stdio,
cwd, cwd,
env: resolvedEnv, env: resolvedEnv,
windowsVerbatimArguments, windowsVerbatimArguments,
shell: process.platform === "win32", shell: needsShell,
}); });
// Spawn with inherited stdin (TTY) so tools like `pi` stay interactive when needed. // Spawn with inherited stdin (TTY) so tools like `pi` stay interactive when needed.
return await new Promise((resolve, reject) => { return await new Promise((resolve, reject) => {