Skip to content

Commit d85d690

Browse files
committed
refactor(scripts): use platform-conditional shell option instead of hardcoded true
Changed two spawn calls in claude.mjs from hardcoded `shell: true` to platform-conditional `...(WIN32 && { shell: true })` for consistency with the rest of the codebase. This follows the established pattern used throughout the codebase where shell is only enabled on Windows when needed for .cmd/.bat file execution.
1 parent 2a1b3d0 commit d85d690

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

packages/cli/scripts/claude.mjs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4716,7 +4716,7 @@ Fix all issues by making necessary file changes. Be direct, don't ask questions.
47164716
const child = spawn(scriptCmd, [], {
47174717
stdio: 'inherit',
47184718
cwd: rootPath,
4719-
shell: true,
4719+
...(WIN32 && { shell: true }),
47204720
})
47214721

47224722
// Handle Ctrl+C gracefully
@@ -5036,7 +5036,7 @@ Fix the issue by making necessary file changes. Be direct, don't ask questions.`
50365036
const child = spawn(scriptCmd, [], {
50375037
stdio: 'inherit',
50385038
cwd: rootPath,
5039-
shell: true,
5039+
...(WIN32 && { shell: true }),
50405040
})
50415041

50425042
// Handle Ctrl+C gracefully

0 commit comments

Comments
 (0)