wenjunduan /athena-review — Review stage (Codex, v9.9.6)
触发
impl 完成、checklist 全绿、测试通过; Refactor/System 还须先完成 runtime-verify.
6 维度
| 维度 | 执行者 |
|---|---|
| Correctness / Security / Test / Design / Quality | reviewer |
| Spec Compliance (MISSING/EXTRA/DEVIATED) | spec-compliance |
工作流 (2 + 1)
- 主 agent 并行调用两次
spawn_agent: reviewer 与 spec-compliance. 任务消息包含当前 sprint 绝对路径、design 路径、diff 基线与输出格式. - 两个 read-only agent 只返回完整 markdown 段, 不创建或修改文件.
- 主 agent 用
wait_agent收齐结果, 串行写入sprints/{current_sprint_slug}/reviews/passN.md. - 主 agent 再用
spawn_agent调 evaluator; evaluator 读取已合并的 passN.md、design、checklist 与 evidence, 返回 Evidence Cross-Check + VERDICT. - 主 agent 追加 evaluator 结果并更新
_index.next_action.
原生工具 schema: https://github.com/openai/codex/blob/rust-v0.144.1/codex-rs/core/src/tools/handlers/multi_agents_spec.rs#L78-L112
VERDICT
| VERDICT | 下一步 |
|---|---|
| PASS | polish (Refactor/System) 或 ship |
| CONCERNS | 修复或明确 defer 后生成 passN+1; 不得直接 ship (gate 只接受最终 PASS) |
| REWORK / FAIL | 回 impl, 修复后生成 passN+1 |
门禁
ship 时 delivery-gate 检查 passN.md、Spec Compliance、Evidence Cross-Check (Refactor/System) 与 reviewer/evaluator 结果. review agent 的启动记录不能代替返回结果或落盘产物.
禁止
- reviewer/spec-compliance/evaluator 并发写同一文件
- evaluator 早于前两份结果合并
- read-only agent 更新
_index.md - 主 agent 伪造 agent findings; 主 agent 只负责合并、去重与落盘
例外
- Hotfix: 可跳过 review
- Bugfix / Quick: 仅 reviewer
- Feature / Refactor / System: 完整 2+1 流程