Review Skill: 质量审查 + 归档 Skill

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 — 按顺序

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

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

Path C — 全部

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

Path D — 全部 + E2E

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

未来扩展

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

@evaluator 评审

委托 @evaluator 时提供:

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

4 维评分: | 维度 | 说明 | |------|------| | Functionality | 验收标准逐条检查 | | Spec Compliance | 对照 design.md | | Craft | 命名、结构、可维护性 | | Robustness | 异常处理、边界、安全 |

VERDICT 和后续动作

| VERDICT | 均分 | 动作 | |---------|------|------| | PASS | ≥4.0 | → V 归档 | | CONCERNS | 3.0-3.9 | → 修复问题 → 重新评分 | | REWORK | 2.0-2.9 | → 回 E 阶段重做 | | FAIL | <2.0 | → 回 D 阶段重新设计 |

V 归档 (PASS 后)

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

Gotchas

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

Agent Skills: Review Skill: 质量审查 + 归档

Install this agent skill to your local

Skill Files