Review

Review Skill: 质量审查 + 归档

核心原则: 写代码的模型不应该独自审查自己的代码。

当前状态

!cat .ai_state/project.json 2>/dev/null | head -5

已完成的 Task

!grep -c "\[x\]" .ai_state/tasks.md 2>/dev/null

可用审查工具

!which codex 2>/dev/null !npx ecc-agentshield --version 2>/dev/null

前置检查

• Path A: 确认修复代码已完成即可
• Path B+: tasks.md 中有 "完成" 的 Task
• project.json stage 更新为 "T"
• 确保 mkdir -p .ai_state/reviews (写入报告前)

审查调度 (按 Path 深度)

Path A — 选一即可

• /codex:review 或 /review (CC 内置)
• 通过就可交付, 不需要 @evaluator 评分

Path B — 按顺序

1. /codex:review --background → /codex:status → /codex:result
2. 运行项目测试: npm test / pytest 等
3. @evaluator 综合评分 (附带 codex:result + 测试结果)
4. 结果写入 .ai_state/reviews/sprint-N.md

重要: 必须等 /codex:result 拿到结果后再调 @evaluator。

Path C — 全部

1. /codex:review --background
2. /codex:adversarial-review --background (可与 1 并行)
3. /codex:status → /codex:result (等 1+2 完成)
4. npx ecc-agentshield scan
5. 运行项目测试
6. @evaluator 综合评分 (附带所有审查结果 + 测试结果)
7. 结果写入 .ai_state/reviews/sprint-N.md

Path D — 全部 + E2E

• 同 Path C
• 如有前端: playwright-skill E2E 测试

未来扩展

当其他模型 CLI 可用时 (如 Gemini CLI), 加入审查链。当前只有 codex-plugin-cc。

@evaluator 评审

委托 @evaluator 时提供:

1. .ai_state/design.md — 验收标准
2. 代码变更 (git diff 或文件列表)
3. 前置审查结果 (codex:result + ECC)
4. 测试结果

4 维评分:

VERDICT 和后续动作

V 归档 (PASS 后)

1. project.json: 追加新发现的 conventions 和 gotchas
2. lessons.md: 追加本 Sprint 教训
3. project.json: stage="", sprint+1
4. 告知用户: "Sprint N 完成, 交付了 X 个 Feature。"

Gotchas

• ❌ 只用 /review 自审就交付 → ✅ Path B+ 必须有跨模型审查
• ❌ adversarial-review 不指定方向 → ✅ 给方向: "challenge the caching design"
• ❌ @evaluator CONCERNS 就放过 → ✅ 必须修复后重新评分
• ❌ 忘记写审查报告 → ✅ 结果必须写入 .ai_state/reviews/