fix(codex): handle stdout/stderr stream errors in CLI session exec resume

lsr911 · claude · lsr911 · commit c698a8163985 · 2026-07-09T11:23:17.000+08:00
The exec resume child process pipes stdout/stderr for data collection but does not register error handlers on the stdio streams. If a stream errors (e.g. pipe broken), the Promise never settles because only process-level errors are caught. Add child.stdout.on('error', reject) and child.stderr.on('error', reject) inside the Promise constructor alongside the existing child.on('error', reject) to catch stream-level errors. This follows the same pattern as #101505 (mushuiyu886), #101597 (Alix-007), #101401 (cxbAsDev), and #101088 (masatohoshino). Co-Authored-By: Claude <noreply@anthropic.com>
diff --git a/extensions/codex/src/node-cli-sessions.ts b/extensions/codex/src/node-cli-sessions.ts
@@ -296,6 +296,8 @@ async function runCodexExecResume(params: {
     child.stdin.end(params.prompt);
     const exitCode = await new Promise<number | null>((resolve, reject) => {
       child.on("error", reject);
+      child.stdout.on("error", reject);
+      child.stderr.on("error", reject);
       child.on("exit", (code) => resolve(code));
     }).finally(() => {
       clearTimeout(timeout);
