Code Review：fix(codex): handle signal-killed process exit during startup (#1963)

变更概述

本 PR 修复了 CodexConnection.start() 中进程退出处理的逻辑缺陷。原有条件 code !== 0 && code !== null 在进程被信号杀死时（code=null）求值为 false，导致 handleProcessExit 未被调用，promise 一直等到 5 秒超时才以通用错误消息 reject。修复后，启动窗口内的任何退出都会立即 reject 并附带详细的错误信息（包含 code 和 signal）。同时重写了相关测试，使用 fake timers 确保测试确定性。

涉及模块：src/process/agent/codex/connection/、tests/unit/。

方案评估

结论：✅ 方案合理

方案正确解决了 Sentry ELECTRON-E4 报告的问题。启动阶段进程退出属于异常情况，无论 exit code 是非零还是 null（信号杀死），都应立即 reject 并清理状态。移除条件判断后逻辑更简洁，且不影响正常启动流程（5 秒 timeout resolve 在进程存活时仍然生效）。handleProcessExit 将 this.child 置为 null 后，后续的 setTimeout resolve 分支因 this.child 为 null 而被跳过，不会产生双重 settlement 问题。

问题清单

✅ 未发现明显问题，代码质量良好，建议批准合并。

汇总

结论

✅ 批准合并 — 修复逻辑正确，覆盖了信号杀死（code=null）和非零退出两种场景。测试重写后结构更清晰、确定性更强。代码干净，无阻塞性问题。

本报告由本地 pr-review skill 生成，包含完整项目上下文，无截断限制。

CONCLUSION: APPROVED
IS_CRITICAL_PATH: false
PR_NUMBER: 1963

✅ 已自动 review，无阻塞性问题，正在触发自动合并。