13
Review Loop · workflow

review-fix-regression-loop

13-review-fix-regression-loop.md
🔁
P0 source: codex-sessions created: 2026-06-13
◉

WHAT IS THIS

定位

把 code review finding 转成 regression test，再做最小 patch 的闭环 workflow。适合高风险模块，比如本地 executor、权限边界、文件系统、agent tool、API contract。

⚡

TRIGGERS

触发场景
▸Review 发现了真实 bug 或安全风险。
▸Finding 多，需要分组修。
▸用户要求“把 review feedback 都处理掉”。
▸改动涉及权限、路径、幂等、大小限制、错误桥接。
▸Review finding 指向“真实副作用已发生，但最终展示失败”。
▸Finding 来自线上 session、audit log、tool call，而不是静态 diff。
↹

INPUT & OUTPUT

输入 / 产出
↘ INPUT
  • Review findings。
  • 当前 diff。
  • 相关 session id、audit log、tool call record，可选。
  • 测试命令。
  • 风险模块路径。
  • 用户要求的修复范围。
↗ OUTPUT
  • Finding 分组。
  • 每组 regression test。
  • 最小修复 patch。
  • 验证结果。
  • Review reply 草稿。
🪜

STEPS

编排步骤
  1. 1
    过滤 finding
    区分 bug、risk、style、question。只把真实 bug 和高风险 risk 放入修复闭环。
  2. 2
    分组
    按根因分组，不按评论数量分组。
  3. 3
    先写 regression
    每个 bug 至少有一个失败测试或可执行复现。
  4. 4
    最小 patch
    修 guard、状态、错误处理、边界，不做顺手重构。
  5. 5
    定向验证
    跑对应测试，确认 regression 从失败变通过。
  6. 6
    全量验证
    跑相关全量 test/build/lint。
  7. 7
    回写 review
    每条 finding 标 fixed、not applicable、deferred，并说明证据。
  8. 8
    分流运行时问题
    如果 finding 需要先还原对话、route decision、child run 或 tool audit，转 →#18。如果已经确认写操作成功但 UI/assistant 说失败，转 →#19。
⚙

AGENT ROLES

Agent 分工
⚙
Finding Triage Agent
判断 finding 有效性。
⚙
Regression Agent
写测试。
⚙
Fix Agent
做最小 patch。
⚙
Verifier Agent
独立验证。
⚙
Reply Agent
整理回复。
✓

ACCEPTANCE GATE

验收 gate
⊘

FAILURE HANDLING

失败处理
⊘
如果 finding 无法复现，不直接改，标为 risk 并说明缺证据。
⊘
如果修复需要架构决策，转 ADR 或 plan。
⤳
如果修复暴露更大 contract 问题，转 →#01。
⊘
如果 finding 依赖线上 DB 或真实会话，不用 mock 先行代替运行时证据。
⌘

TEMPLATE

回复模板
TEMPLATE
### Review Resolution

| Finding | Status | Fix | Test |
| --- | --- | --- | --- |