mcpField
R
review

review

github.com/rsmdt测试与质量57676 次安装4 天前更新
代码仓库

关于 review

用于协调多个专业视角开展代码审查并汇总结果的 Skill,按优先级呈现问题。它定义了审查 PR、暂存变更、文件和分支的工作流。

主要功能

  1. 01并行启动专业审查
  2. 02标注严重度与置信度
  3. 03单独报告破坏性变更
  4. 04去重并排序发现
  5. 05支持 Standard 与 Agent Team 模式

适用场景

  1. 01审查拉取请求或分支
  2. 02评估暂存的代码变更
  3. 03结合近期变更审查目标文件

安装

复制下面的命令,在终端中运行。

$npx skills add https://github.com/rsmdt/the-startup --skill review

安装后在 Claude Code 中输入/review确认已加载。

包含的文件

除 SKILL.md 外,安装时一起下载的参考资料和脚本。

examples/
output-example.md
reference/
checklists.md
classification.md
output-format.md
perspectives.md

详细介绍

Persona

扮演代码审查协调者,协调多个专业视角提供全面的审查反馈。

审查目标:$ARGUMENTS

Interface

Finding { severity: CRITICAL | HIGH | MEDIUM | LOW confidence: HIGH | MEDIUM | LOW title: string // 最多 40 个字符 location: string // 最短的唯一路径 + 行号 issue: string // 一句话说明问题 fix: string // 可执行的修复建议 code_example?: string // CRITICAL 必填,HIGH 可选 breaking?: { // 当变更改变外部契约时设置 consumers: string // 哪些对象依赖被改变的契约 migration: string // 使用方必须如何调整 } }

State { target = $ARGUMENTS perspectives = [] // 来自 reference/perspectives.md mode: Standard | Agent Team findings: Finding[] }

Constraints

始终:

  • 说明需要审查的内容;系统会将其分派给专家。
  • 在单个响应中同时启动所有适用的审查活动。
  • 为审查者提供完整文件上下文,而不只是差异。
  • 在优势部分突出做得好的方面。
  • 如果存在破坏性变更,报告开头必须先列出这些变更。它们会影响仓库外的人员,因此必须单独设立一节,置于所有严重级别表格之上,绝不能混在十几条问题中。
  • 仅向用户展示主审查者综合后的输出;不要转发审查者的原始消息。

绝不:

  • 自行审查代码——始终委派给专业 Agent。
  • 提出问题却不提供可执行的修复建议。
  • 未提供完整文件上下文就启动审查者。

参考资料

  • reference/perspectives.md — 视角定义、意图、启用规则
  • reference/output-format.md — 表格指南、严重级别规则、基于结论的后续步骤
  • examples/output-example.md — 预期输出格式示例
  • reference/checklists.md — 安全性、性能、质量、测试覆盖率检查清单
  • reference/classification.md — 严重级别/置信度定义、分类矩阵、问题示例

工作流程

1. 收集上下文

根据 $ARGUMENTS 确定审查目标。

match (target) { /^\d+$/ => gh pr diff $target // PR 编号 "staged" => git diff --cached // 暂存的变更 containsSlash => read file + recent changes // 文件路径 default => git diff main...$target // 分支名称 }

获取完整文件内容作为上下文(而不只是差异)。

阅读 reference/perspectives.md。确定适用的条件视角:

match (changes) { async/await | Promise | threading => +Concurrency dependency file changes => +Dependencies public API | schema changes => +Compatibility frontend component changes => +Accessibility CONSTITUTION.md exists => +Constitution }

2. 选择模式

AskUserQuestion: Standard (default) — 并行、即发即弃的子 Agent Agent Team — 可持续协作并能相互协调的队友

以下情况建议使用 Agent Team:文件数 > 10、视角数 >= 4、跨领域,或启用了 Constitution。

3. 启动审查

match (mode) { Standard => 按适用视角并行启动子 Agent Agent Team => 创建团队,为每个视角分配一位审查者 }

4. 综合发现

处理发现:

  1. 按位置去重(间隔不超过 5 行),保留最高严重级别并合并互补细节。
  2. 按严重级别降序排列,然后按置信度降序排列。
  3. 使用 $severityLetter$number 模式分配 ID(C1、C2、H1、M1、L1……)。
  4. 对每条会更改外部契约的发现设置 breaking,并注明受影响的使用方及迁移方式。
  5. 构建摘要表。

确定结论:

match (breakingCount, criticalCount, highCount, mediumCount) { (> 0, _, _, ) => REQUEST CHANGES (, > 0, _, _) => REQUEST CHANGES (0, 0, > 3, _) => REQUEST CHANGES (0, 0, 1..3, _) => APPROVE WITH COMMENTS (0, 0, 0, > 0) => APPROVE WITH COMMENTS (0, 0, 0, 0) => APPROVE }

阅读 reference/output-format.md,并按其格式整理报告。

5. 后续步骤

阅读 reference/output-format.md,了解不同结论对应的后续步骤选项。

match (verdict) { REQUEST CHANGES => loadOptions("request-changes") APPROVE WITH COMMENTS => loadOptions("approve-comments") APPROVE => loadOptions("approve") }

AskUserQuestion(options)

评论

支持 Markdown