code-review

技能简介

这个技能用于审查从某个固定点（commit、branch、tag 或 merge-base）到当前 HEAD 的代码变更。它会从两个维度并行审查：Standards（代码是否符合仓库文档化的编码标准）和 Spec（代码是否忠实实现了原始 issue/spec 的要求），最后并排输出两份报告，帮助快速定位规范性和需求对齐问题。

使用场景

• 审查一个分支或 PR 相对主分支的改动
• 审查“自某次 commit/tag 以来”的变更
• 检查代码是否符合仓库编码规范和常见代码坏味道
• 核对实现是否与 issue/spec 一致，发现遗漏或范围蔓延
• 对进行中的工作（WIP）做快速 review

使用方法

1. 确保仓库已配置 issue tracker（docs/agents/issue-tracker.md）。若缺失，运行： /setup-matt-pocock-skills
2. 调用技能时提供固定点，例如 commit SHA、分支名、tag、main、HEAD~5 等。如果未指定，技能会要求你提供。
3. 技能会先解析固定点并确认 diff 非空，然后并行启动两个子代理：
   • Standards 子代理：查找并对照仓库内的 CODING_STANDARDS.md、CONTRIBUTING.md 等标准文件，以及内置的 Fowler 代码坏味道基线，报告违规点。
   • Spec 子代理：从 commit message 中的 issue 引用（#123、Closes #45 等）、用户传入路径或 docs/、specs/、.scratch/ 下查找 spec 来源；找不到时会询问你。
4. 最终报告以 ## Standards 和 ## Spec 两节呈现，不会合并或排序。

注意事项

• 固定点必须有效（git rev-parse 可解析）且 diff 非空，否则技能会直接失败。
• 如果找不到 spec，Spec 子代理会跳过，并在报告中注明“无 spec”。
• Standards 审查中，仓库文档化的标准优先于内置坏味道基线；工具已能强制检查的项不会重复报告。
• 坏味道属于启发式判断（如“可能 Feature Envy”），并非硬性违规。