/athena-review — Review (v9.9.9)
触发
impl 完成且测试通过。Refactor/System:先 runtime-verify,再完成会改代码的 polish,最后才 review。
稳定审查提示在本 skill 的 REVIEW.md(Claude Code skill 支持文件,不是 ~/.pi/agent/skills/athena-review/REVIEW.md)。sprint 合同用 review-packet.md。
一次请求(异步里程碑)
Athena 只发起 一次 review 请求:
| 端 | 入口 |
|---|---|
| CC | /code-review(/review 为其别名,2.1.223+)+ 本 skill REVIEW.md + sprint packet |
| CX | 原生 /review;不可用则单个只读 reviewer agent 读同一 REVIEW.md |
| 皆无 | 单个 reviewer,schema 相同 |
官方 harness 内部可并行多个 reviewer,不计入 Athena 轮次。Athena CC agent 统一 maxTurns: 70;原生内置 reviewer 是否受该字段控制须按入口能力区分,到限返回未完成进度。
发起后本轮 正常结束,设 _index.next_action = await-review-result。Stop / continuator 对该信号 放行且不注入续跑。完成通知轮:校验并写入 reviews/implementation-review.md。
使用本端 review-binding.cjs 的 prepare → 原生派发 → bind → 原生完成 → accept。两次原生工具返回 JSON 原样保存为 receipt;参数与异常处理只见 执行合同的 CC CLI,不手写 session-log 绑定、不补造 receipt。
CC 原生 review 可用则用,否则 Agent 只读 reviewer/独立会话;不要求 CX 或 Grok。仅真实后台入口进入 await-review-result,前台返回直接接收,等待不重复派发。
维度
Spec coverage · Correctness · Security · Test risk · Over-engineering · Evidence(R/S)
测试是否执行、越界、hash 由 gate 判。
可选:测试已绿且存在多个合格 diff 时,可先跑 /llm-as-a-verifier 写 verifier_rank;reviewer 可以读排名,不能把排名当 VERDICT。
结果
单文件 reviews/implementation-review.md,YAML frontmatter 含 review_run_id、native_output_ref、packet_sha256、reviewed_diff_sha256、verdict。缺任一字段 ship block。转录不得改定级。
目标复核:diff hash 变了才再审 open findings。同因新 P0 ×2 → 交还用户,禁止无界 passN。
禁止
- spawn critic / evaluator / spec-compliance
- 同轮死等后台
/code-review - 主 agent 编造 findings(无 native_output_ref)
- 把
~/.pi/agent/skills/athena-review/REVIEW.md当 skill 入口
例外
- Hotfix:默认可跳过
- Quick:用户或风险触发才跑
- Bugfix / Feature / R/S:一次 review