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

fix: use shell mode on Windows for command spawning

Browse Source 
On Windows, spawn("npm", [...]) fails with ENOENT because Windows
requires .cmd/.exe extensions or shell-based PATH resolution.
Adding shell: true on Windows enables proper PATH lookup for npm
and other commands spawned via runCommandWithTimeout.

Fixes #4557

Co-Authored-By: Ayush Ojha <ayushozha@outlook.com>
This commit is contained in:
Ayush Ojha 2026-01-30 05:15:33 -08:00
parent da71eaebd2
commit 9d5abc6354
1 changed files with 2 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/process/exec.ts
View File

@ -80,11 +80,13 @@ export async function runCommandWithTimeout(
} }
const stdio = resolveCommandStdio({ hasInput, preferInherit: true }); const stdio = resolveCommandStdio({ hasInput, preferInherit: true });
const isWindows = process.platform === "win32";
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: isWindows,
}); });
// 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) => {