Skip to content

fix: pass prompt via stdin to avoid E2BIG error #12

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
kevinfealey wants to merge 1 commit into
base: main
Choose a base branch
from
+9 −5
Open

fix: pass prompt via stdin to avoid E2BIG error #12

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
14 changes: 9 additions & 5 deletions src/subprocess/manager.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,7 @@ export class ClaudeSubprocess extends EventEmitter {
* Start the Claude CLI subprocess with the given prompt
*/
async start(prompt: string, options: SubprocessOptions): Promise<void> {
const args = this.buildArgs(prompt, options);
const args = this.buildArgs(options);
const timeout = options.timeout || DEFAULT_TIMEOUT;

return new Promise((resolve, reject) => {
Expand Down Expand Up @@ -81,8 +81,12 @@ export class ClaudeSubprocess extends EventEmitter {
}
});

// Close stdin since we pass prompt as argument
this.process.stdin?.end();
// Write prompt to stdin instead of passing as CLI argument
// This avoids E2BIG errors when prompts exceed the OS argument size limit
if (this.process.stdin) {
this.process.stdin.write(prompt);
this.process.stdin.end();
}

console.error(`[Subprocess] Process spawned with PID: ${this.process.pid}`);

Expand Down Expand Up @@ -126,8 +130,9 @@ export class ClaudeSubprocess extends EventEmitter {

/**
* Build CLI arguments array
* Note: prompt is passed via stdin to avoid E2BIG errors with large prompts
*/
private buildArgs(prompt: string, options: SubprocessOptions): string[] {
private buildArgs(options: SubprocessOptions): string[] {
const args = [
"--print", // Non-interactive mode
"--output-format",
Expand All @@ -137,7 +142,6 @@ export class ClaudeSubprocess extends EventEmitter {
"--model",
options.model, // Model alias (opus/sonnet/haiku)
"--no-session-persistence", // Don't save sessions
prompt, // Pass prompt as argument (more reliable than stdin)
];

if (options.sessionId) {
Expand Down