关于 code-review
用于依据已交付差异和现有要求检查指定审查范围的代码审查技能。它规定审查范围、证据标准、并行的规范与规格检查,以及问题处置方式。
主要功能
- 01选择审查范围
- 02规范与规格检查
- 03基于证据的问题发现
- 04修复审查模式
- 05问题处置指导
适用场景
- 01审查完整的已交付差异
- 02对照验收标准检查变更
- 03验证指定问题的修复
- 04报告后续事项和可选问题
安装
复制下面的命令,在终端中运行。
npx skills add https:/ / github.com/ gabriel-lafrance/ skills --skill code-review安装后在 Claude Code 中输入/code-review确认已加载。
附带文件doctrine.mdexamples.md
详细介绍
代码审查
必读: ../pack-shared/standards.md。本轮裁定标准前,先应用 AGENTS.md 的 Taste 和 Architecture 章节。不得跳过。
阅读: doctrine.md · examples.md
标准始终适用 AGENTS.md 的 Taste 和 Architecture 章节。Worker 输出使用 review-contract 的 review output 围栏(Principles、Architecture、Correctness hunt)。面向用户的发现使用通俗措辞
(plain-language.md)。
输入
确定请求的固定点,并检查已交付的差异。
如果父级已提供结果、完成条件、非目标、工单或 PR、固定点、通道、阶段、活动规则、当前切片和修复待办,则将该聊天上下文视为有约束力的交接。不要从隐藏文件中重构意图。
否则,按以下顺序确定规范轴:
- 用户陈述的结果和验收标准
- 指定的 PR、工单及其可用讨论
- 相关的仓库代码、规则和已提交文档
如果没有可用的规范,请明确说明,并在不臆造要求的情况下执行标准审查。所有发现、决定和修复备忘录都保留在聊天中。
仅在确实违反活动规则时引用该规则;否则引用相关验收标准,或说明没有适用规则。
范围
有意选择共享审查模式:
initial会并行执行完整已交付差异的标准审查和规范审查。没有设计轴,也没有第二轮对抗性审查。remediation仅接收具名发现 ID、修复差异、直接涉及的路径以及直接调用方。full-rescan需要明确请求在发生重大更改后重新进行完整深度审查。
在任何修复工作之前,应用 code-review:axes、code-review:blocker-vs-follow-up、code-review:naming-alignment、review-contract 的证据门槛以及修复分析。标准审查 Worker 必须返回 review output 围栏。标准审查本轮必须应用 AGENTS.md 的 Taste 和 Architecture 章节。不得派发设计审查 Task。
对于 remediation,请在该范围内核实相关发现和回归问题;不要重新开启大范围审查。
交接
父级(本聊天,或嵌套时的 /task / /just-do-it)负责设置固定点、派发 Worker、审查完成情况、验收证据和审查关卡。标准审查和规范审查必须作为并行 Task 执行;如果差异涉及独立范围,则添加额外 Task。没有设计轴,也没有第二轮对抗性审查。实现 Worker 不负责运行这些关卡,也不得扩大修复审查范围。在聊天中报告稳定的发现 ID,以及“立即修复 / 后续处理 / 可选的小问题”处置结果。
如果父级已负责下一步
任何修复工作开始前,将选定的“立即修复”发现以 review-remediation 模式发送给 /analyze。其备忘录须与稳定的发现 ID 关联,并要求明确晋级(或适用文档说明的 /just-do-it 例外)。晋级后的通道仍须限定在这些发现及所提供的当前切片内。
如果这是用户的一次性请求
在聊天中报告处置结果。除非用户要求执行下一步,否则不要臆造父级审查轮次,也不要晋级修复。