| name | 507-review |
| description | 交付审查:围绕明确的 commit、branch、diff 或路径检查 Standards、Spec 与 Code Quality,只报告问题、证据和路由。Use when user says review this change, code review, review 一下, 检查, 审查, 审查改动, 审查这次改动, 检查提交, 检查分支, 过一遍代码, 把关, 需求符合度, 交付审查. 不用于主动寻找全库架构机会。 |
交付审查(review)
对一个明确的代码或文档变更范围检查:
- Standards(项目规范):项目规则、既有决策与文档约束;
- Spec(需求符合度):需求、验收标准、范围和行为;
- Code Quality(代码质量):真实的 bug、安全、错误处理、性能和可维护性风险。
本 skill 只读审查:只给发现、证据和路由,不修改文件。
确定审查范围
按顺序选择:
- 用户明确给出的 commit(提交)、范围、branch(分支)、tag(标签)或路径;
- 当前 staged(已暂存)、unstaged(未暂存)和相关 untracked(未跟踪)变更;
- 当前分支相对默认主干的差异;
- 已提交且工作区干净时,仍审当前分支相对主干的差异;
- 没有可用基准时,临时审最近一个 commit 并明确说明;
- 仍无法判断时,只询问一次范围并给出推荐基准。
工作区干净不等于没有可审内容。开始前说明范围、基准和选择原因。
审查依据
优先读取相关 AGENTS.md、根及目标目录 README.md、doc/README.md、术语表、决策档案、贡献/风格规范、测试与构建配置,以及用户提供的 issue(问题单)、PRD(需求文档)、spec(规格)或验收标准。
没有明确需求来源时,将 Spec 标为“无明确 spec”,仍检查范围蔓延、行为风险和可验证性。只把已有证据当作规范,不把个人偏好包装成 Standards。
审查重点
Standards
- 是否违反项目规范、既有决策、不做边界或稳定术语;
- 职责、入口、运行方式、配置或权限变化是否同步文档;
- 用户可见变化是否同步项目要求的变更记录和版本信息。
Spec
- 需求是否完整实现,有无漏做、误做或范围蔓延;
- 行为、文档、测试和验收标准是否一致;
- 验收是否可复验,入口 README、项目地图和发布产物是否完整。
Code Quality
只报告真实或有明确证据的风险:边界/空值/竞态/状态错误,注入/权限/泄密,静默失败,不必要重算或泄漏,以及复杂度、误导命名、死代码和重复逻辑。不要重复 formatter(格式器)或 linter(静态检查器)已经明确捕获的表面问题。
输出格式
## Review Scope
- 范围:...
- 基准:...
- 原因:...
- 依据:...
## Summary
- Standards:通过 / 有问题 N 个
- Spec:通过 / 有问题 N 个 / 无明确 spec
- Code Quality:通过 / 有问题 N 个
- 最严重问题:...
## Findings
### 🔴 严重 / 🟡 警告 / 🔵 信息
- `file:line`:问题
- 依据:...
- 影响:...
- 建议:...
## Recommended Next Mode
- 默认下一步:...
- 路由理由:...
## Verification
- 已检查 / 已运行 / 未运行 / 建议验证:...
没有发现阻塞问题时明确写“未发现阻塞问题”,并说明范围和验证限制。
后续路由
- bug、逻辑或安全问题 →
507-fix;
- 测试缺口/失败 →
507-test;明确 test-first → 507-tdd;
- 行为不变内部简化 →
507-simplify;架构摩擦 → 507-inspect;
- 地图文档失真 →
507-map;完整项目初始化或规范巡检 → 507-setup;
- 漏做需求 →
507-prd 或 507-issue;
- 已完成且用户明确要求 Git 提交 →
507-commit。
红线
- 不直接修改、修复、重构、补测试或提交;
- 不把无证据风险断言为事实;
- 不因工作区干净跳过已提交差异;
- 不大段复述 diff(差异),只列有行动价值的问题。
完成与接力
- 完成信号:明确范围已按 Standards、Spec 与 Code Quality 审完,给出带证据的 findings(发现)和清晰 verdict(结论)。
- 产物:只读审查报告与问题路由,不直接修改。
- 候选出口:按发现进入
507-fix、507-test、507-simplify、507-map、507-setup、507-prd 或 507-issue;没有阻塞且用户明确要求提交时进入 507-commit;只需审查结论时直接结束。
- 回退条件:审查基线或目标规格不明确时先返回
507-explore 或 507-grill,不对猜测中的需求判通过/失败。