Review 修复阶段失败：Failed to start dialog turn: Unknown session mode: ReviewFixer（#2106 修复不完整）

[YodonTan]

现象

review 会话可以正常创建并产出 review 结果（#2100 已修复），但从 review 结果点击「开始修复」时报错：

审核失败
Failed to start dialog turn: Unknown session mode: ReviewFixer

根因：PR #2106 的修复不完整

#2106（修复 #2100）只放行了会话创建/首轮使用的 CodeReview / DeepReview 两个 agent。但 review 流程的「修复阶段」由前端以 agentType='ReviewFixer' 提交（DeepReviewActionBar.handleStartFixing → sendMessage(..., 'ReviewFixer', 'agentic')），后端 turn 启动路径的 primary-agent 解析（resolve_primary_agent_for_turn）白名单中不包含 ReviewFixer，导致：

Failed to start dialog turn: Unknown session mode: ReviewFixer

与原 issue #2100 的关联

• 两个问题同源：回归均来自 commit ca94825ad（feat(extensions): activate external primary agent profiles，2026-08-04）。该 commit 把 external primary agent 解析接入会话创建与 turn 启动路径；此前 turn 启动直接用 agent_type，不经过 registry 解析，ReviewFixer 一直可用。
• /review 无法进入 review：Failed to create session: Validation error: Unknown session mode: CodeReview #2100 是回归的第一半（创建会话，agent 类型 CodeReview/DeepReview 被拦）；本 issue 是回归的第二半（修复轮，agent 类型 ReviewFixer 被拦）。fix(review): restore CodeReview/DeepReview session creation and turns #2106 合入后 /review 无法进入 review：Failed to create session: Validation error: Unknown session mode: CodeReview #2100 关闭，但未覆盖此路径。

影响面（同一解析函数的全部路径）

• turn 启动：scheduler start_turn → coordinator.rs resolve_session_primary_agent 报错（用户所见）
• 手动压缩：coordinator.rs 手动压缩路径同样失败
• 会话恢复：session_manager.rs 恢复路径解析失败后会把会话 agent_type 静默改写为 agentic（修复中途重启会丢失 ReviewFixer 绑定，后续修复轮以 agentic 运行且持久化改写）

修复建议

在后端共享解析层放行 ReviewFixer（is_builtin_session_primary_agent 白名单加入 REVIEW_FIXER_AGENT_TYPE，常量已存在于 bitfun-agent-runtime::deep_review::constants）。ReviewFixer 注册为 hidden(["CodeReview","DeepReview"])，普通模式不可选，放行不会暴露给普通会话；ReviewWorker/ReviewJudge 等 team 角色保持受限。

[bobleer]

这条定位是对的，ReviewFixer 确实漏在 #2106 的 builtin session-primary 白名单之外。你提到的 restore 静默回退成 agentic 比入口报错更危险，这个不能只补 start_turn。

我把它保留为独立回归，不重新塞回 #2100。修复时至少要一起覆盖 start turn、manual compaction、restore 三条路径，并加一组正负测试：ReviewFixer 可以作为 builtin hidden session primary；ReviewWorker / ReviewJudge 仍不能越过这条边界。

谢谢把链路追完整。

[bobleer]

已合入 #2127（rebase 到 main：6aa5f123f / c4a301e20）。审过了，代码没有风险，回一下你问的风险点。

改在唯一收敛点上。 is_builtin_session_primary_agent 是 resolve_primary_agent_for_turn 的唯一判定入口，我核对了受影响的路径全部经过它——turn 启动（coordinator.rs:5151）、手动压缩（coordinator.rs:4718）、会话恢复（session_manager.rs:5371）、会话创建与模式切换（coordinator.rs:2454 / 10527）。一处放行确实覆盖全部，我之前要求的三条路径都在里面。

不会暴露给普通会话。 模式列表走 AgentCategory::Mode 过滤（get_modes_info_for_workspace），和这个白名单是两套逻辑；ReviewFixer 是 SubAgent 类别，模式选择器不会列出它。

负向边界仍然守得住，而且有测试。 ReviewWorker/ReviewJudge 在 fallback 和显式 Local route 下都返回 None；同名非 Builtin 条目（AgentSource::User）fail closed；external-owner 守卫对三个 review agent 都保持拒绝。

is_review_agent_type 把 ReviewFixer 排除在 manifest 注入之外是对的，不是漏改。 不注入时 GetFileDiff::target_evidence 返回 Ok(None) 退化为工作区 diff，live_repository_context_allowed 返回 true——这正是「改本地文件」的修复阶段需要的语义。manifest 那套闸门是给只读 review 目标用的。

这是恢复 ca94825ad 之前的状态，不是新增能力。 ReviewFixer 的权限姿态没变（Edit/Write 仍走正常审批），review 子会话的 safeMode: true 也没被绕开。另外扫了 web-ui 全量，前端提交的 session primary agent 类型只有 CodeReview / DeepReview / ReviewFixer 三个，没有遗漏的第四个。

一个不影响合入的观察，记一下：update_session_mode 现在也会接受 "ReviewFixer"。只能靠手工构造本地 IPC 触发（UI 模式列表不提供），而且 CodeReview/DeepReview 从 #2106 起就是同样性质，不是这个 PR 引入的。

CI 全绿。之前那次 Rust Build Check (ubuntu-latest) 红是 GitHub 解析 action 时的 502/503，不是代码问题，重跑已通过。

谢谢把链路追完整。