Codecov Report

❌ Patch coverage is 81.81818% with 2 lines in your changes missing coverage. Please review.

📢 Thoughts on this report? Let us know!

Code Review：feat(agent): support non-YOLO session modes at ACP start (#1894)

变更概述

本 PR 为 Claude 后端在 ACP 会话启动时增加了对非 YOLO 权限模式（acceptEdits、auto、dontAsk、plan）的支持。核心改动是提取了 applySessionMode 私有方法来复用 YOLO/非 YOLO 两条路径的会话模式设置逻辑，并通过 AcpAgentManager 将用户在 Guid 页面选择的模式传递给 AcpAgent。同时补充了 6 个语言包的 i18n 翻译和两个单元测试文件。

方案评估

结论：✅ 方案合理

applySessionMode 的 fatal 参数设计干净——YOLO 模式必须成功（fatal=true），非 YOLO 模式失败时优雅降级（fatal=false），符合两类模式不同的重要性。else if 分支对所有后端生效但 fatal=false，非 Claude 后端如果不支持该模式只会打一条 warn，不影响功能。PR 描述中已提及 "double-apply" 的跟进 issue (#1892)，属于已知限制，不阻塞本次合并。

问题清单

🔵 LOW — 测试 mock 中存在空构造函数（lint 警告）

文件：tests/unit/acpApplySessionMode.test.ts，第 39–48 行

问题代码：

vi.mock('../../src/process/agent/acp/AcpAdapter', () => ({
  AcpAdapter: class {
    constructor() {}   // lint: no-useless-constructor / no-extraneous-class
  },
}));

vi.mock('../../src/process/agent/acp/ApprovalStore', () => ({
  AcpApprovalStore: class {
    constructor() {}   // lint: no-useless-constructor / no-extraneous-class
  },
  createAcpApprovalKey: vi.fn(),
}));

问题说明：oxlint 报 no-useless-constructor 和 no-extraneous-class 两条警告，空构造函数可省略。

修复建议：

vi.mock('../../src/process/agent/acp/AcpAdapter', () => ({
  AcpAdapter: class {},
}));

vi.mock('../../src/process/agent/acp/ApprovalStore', () => ({
  AcpApprovalStore: class {},
  createAcpApprovalKey: vi.fn(),
}));

汇总

结论

✅ 批准合并 — 无阻塞性问题，主体逻辑和测试均干净，lint 警告仅在测试 mock 中。

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

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