mcpField
C
code-review

code-review

github.com/gabriel-lafrance测试与质量081 次安装2 天前更新
代码仓库

关于 code-review

用于依据已交付差异和现有要求检查指定审查范围的代码审查技能。它规定审查范围、证据标准、并行的规范与规格检查,以及问题处置方式。

主要功能

  1. 01选择审查范围
  2. 02规范与规格检查
  3. 03基于证据的问题发现
  4. 04修复审查模式
  5. 05问题处置指导

适用场景

  1. 01审查完整的已交付差异
  2. 02对照验收标准检查变更
  3. 03验证指定问题的修复
  4. 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 章节。不得跳过。

共享契约: review · 执行上下文

阅读: doctrine.md · examples.md

标准始终适用 AGENTS.md 的 Taste 和 Architecture 章节。Worker 输出使用 review-contract 的 review output 围栏(Principles、Architecture、Correctness hunt)。面向用户的发现使用通俗措辞 (plain-language.md)。

输入

确定请求的固定点,并检查已交付的差异。

如果父级已提供结果、完成条件、非目标、工单或 PR、固定点、通道、阶段、活动规则、当前切片和修复待办,则将该聊天上下文视为有约束力的交接。不要从隐藏文件中重构意图。

否则,按以下顺序确定规范轴:

  1. 用户陈述的结果和验收标准
  2. 指定的 PR、工单及其可用讨论
  3. 相关的仓库代码、规则和已提交文档

如果没有可用的规范,请明确说明,并在不臆造要求的情况下执行标准审查。所有发现、决定和修复备忘录都保留在聊天中。

仅在确实违反活动规则时引用该规则;否则引用相关验收标准,或说明没有适用规则。

范围

有意选择共享审查模式:

  • 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 例外)。晋级后的通道仍须限定在这些发现及所提供的当前切片内。

如果这是用户的一次性请求

在聊天中报告处置结果。除非用户要求执行下一步,否则不要臆造父级审查轮次,也不要晋级修复。

评论

支持 Markdown