jlengrand/Maestro
1
0
Fork 0
mirror of https://github.com/jlengrand/Maestro.git synced 2026-03-10 08:31:19 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #302 from dmaynor/fix/windows-cmd-length-limit-301

Browse Source 
Fix Windows wizard failure by sending prompt via stdin for stream-json mode
This commit is contained in:
Pedram Amini
2026-02-05 14:39:38 -06:00
committed by GitHub
parent 7daa5d7cd2 2d227ed02c
commit d29bb36346
1 changed files with 5 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
src/main/process-manager/spawners/ChildProcessSpawner.ts
View File

@@ -73,7 +73,11 @@ export class ChildProcessSpawner {
// Check if prompt will be sent via stdin instead of command line
// This is critical for SSH remote execution to avoid shell escaping issues
const promptViaStdin = sendPromptViaStdin || sendPromptViaStdinRaw;
// Also critical on Windows: when using stream-json output mode, the prompt is sent
// via stdin (see stream-json stdin write below). Adding it as a CLI arg too would
// exceed cmd.exe's ~8191 character command line limit, causing immediate exit code 1.
const argsHaveStreamJson = args.some((arg) => arg.includes('stream-json'));
const promptViaStdin = sendPromptViaStdin || sendPromptViaStdinRaw || argsHaveStreamJson;
// Build final args based on batch mode and images
let finalArgs: string[];