CI 检查未通过

以下 job 在本次自动化 review 时未通过，请修复：

本次自动化 review 暂缓，待 CI 全部通过后将重新处理。

Code Review：fix(gemini): ensure workspace dir exists before initialization (#1784)

变更概述

本 PR 修复了 Sentry 高频问题 ELECTRON-6W（949 次），根因是 aioncli-core 的 loadServerHierarchicalMemory 内部直接调用 fs.realpath(workspace) 而不捕获异常。当临时工作目录在创建后到 agent 初始化前被 OS 清理程序或杀毒软件删除时，会抛出未处理的 ENOENT rejection。修复方案是在 GeminiAgent.initialize() 开头调用 fs.promises.mkdir(path, { recursive: true }) 确保目录存在。新增测试文件 tests/unit/geminiWorkspaceRecovery.test.ts 验证修复行为。

方案评估

结论：✅ 方案合理

修复直接命中根因：在第三方代码的 realpath 调用前确保目录存在。{ recursive: true } 使调用幂等——目录已存在时不抛异常、不清除内容，对正常流程无任何影响。修复位置正确，置于所有依赖 workspace 路径的操作之前。无更优替代方案——在调用者侧修复比 fork aioncli-core 更合理。

问题清单

🔵 LOW — 测试验证的是 Node.js fs 行为而非 GeminiAgent 的调用

文件：tests/unit/geminiWorkspaceRecovery.test.ts，全文

问题说明：4 个测试用例均直接调用 Node.js fs API，验证的是 mkdir/realpath 的内置行为，而非 GeminiAgent.initialize() 实际调用了 mkdir。若日后有人不小心删除 initialize() 里的 mkdir 调用，这批测试仍会全部通过，无法起到回归防护作用。

理想的补充测试应 mock fs.promises.mkdir 并断言它在 initialize() 中被调用（但这需要 mock aioncli-core 的整条依赖链，成本较高）。现有测试作为修复行为的文档说明仍有价值，不属于阻塞问题。

汇总

结论

✅ 批准合并 — 修复精准、幂等，无阻塞性问题。

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

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