mcpField
R
review-team

review-team

github.com/cline测试与质量403.4k 次安装15 周前更新
代码仓库

关于 review-team

review-team 协调独立的审查子 Agent 检查代码变更,并汇总可执行的问题。它支持单轮审查,也支持用户明确要求的修复与复审循环。

主要功能

  1. 01选择专门审查角色
  2. 02并行独立审查
  3. 03去重并排列审查发现
  4. 04单轮与迭代模式
  5. 05按需重跑相关审查

适用场景

  1. 01从代码质量和安全角度审查变更
  2. 02针对特定问题开展审查
  3. 03修复问题并重复审查直至通过

安装

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

$npx skills add https://github.com/cline/skills --skill review-team

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

包含的文件

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

reviewers/
api-compatibility-reviewer.md
architecture-reviewer.md
code-quality-conventions-reviewer.md
correctness-reviewer.md
documentation-dx-reviewer.md
performance-reliability-reviewer.md
product-ux-accessibility-reviewer.md
security-abuse-reviewer.md
simplicity-scope-reviewer.md
telemetry-observability-reviewer.md
testing-strategy-reviewer.md

详细介绍

审查团队

针对同一项更改运行一组专用审查子代理。每位审查者都刻意不知晓其他审查者的工作,并且必须严格遵守分配给自己的审查范围。

审查者提示词位于 reviewers/ 中。只阅读你计划启动的审查者文件。

模式

  • single:默认模式。启动审查者一次,汇总发现、去重,并报告最终审查结果。
  • iterate:启动审查者,修复已接受的发现,重新运行相关审查者,并重复此过程,直到审查没有问题,或剩余发现被有意延后处理。

如果用户未指定模式,使用 single。仅当用户明确要求持续修复、让审查者满意或反复运行直至没有问题时,才使用 iterate。

审查者集合

核心审查者:

  • reviewers/correctness-reviewer.md
  • reviewers/security-abuse-reviewer.md
  • reviewers/architecture-reviewer.md
  • reviewers/code-quality-conventions-reviewer.md
  • reviewers/simplicity-scope-reviewer.md

专业审查者:

  • reviewers/product-ux-accessibility-reviewer.md
  • reviewers/performance-reliability-reviewer.md
  • reviewers/telemetry-observability-reviewer.md
  • reviewers/testing-strategy-reviewer.md
  • reviewers/api-compatibility-reviewer.md
  • reviewers/documentation-dx-reviewer.md

用户要求完整团队时,默认启动所有审查者。对于范围较窄的请求,只启动相关审查者。

启动模式

为每位审查子代理提供:

  1. 其 Markdown 文件中的完整审查者提示词。
  2. 任务描述、PR/问题背景或用户请求。
  3. 要审查的差异或已更改文件。
  4. 任何测试结果、日志、截图或相关仓库背景。

将此包装内容添加到每条审查者提示词中:

You are one reviewer in a fleet of independent code review agents. You cannot see the other reviewers. Stay strictly within your assigned role. Return only concrete, actionable findings with file and line references. If you find no issues in your area, say so clearly. Do not provide general commentary, praise, summaries, or duplicate concerns outside your mandate.

优先并行运行相互独立的审查者。不要让多位审查者以相同方式解决同一个宽泛任务;审查者提示词就是他们的职责边界。

汇总

审查者完成后:

  1. 合并重复发现。
  2. 丢弃缺少具体失败模式、用户影响、安全风险或维护成本的发现。
  3. 根据代码库、测试和明确要求提供的证据解决冲突。
  4. 按严重程度和实际重要性排列发现。
  5. 首先列出发现,并附上文件和行号引用。

除非用户要求,否则最终审查结果不应公开审查者的原始对话记录。

迭代模式

在 iterate 模式下:

  1. 运行初始审查者集合。
  2. 判断哪些发现有效且应当修复。
  3. 直接实施修复。
  4. 运行适用于这些更改的测试或检查。
  5. 仅重新运行与已更改区域或未解决发现相关的审查者。
  6. 持续进行,直到没有可采取行动的发现,或剩余发现被明确记录为超出范围、误报或已接受的权衡。

在迭代之间及时告知用户进展。不要无限循环;如果后续迭代没有带来实质性新发现,就应停止。

最终回复

对于 single 模式,返回汇总后的审查结果。

对于 iterate 模式,返回:

  • 修复了哪些问题。
  • 运行了哪些验证。
  • 重新运行了哪些审查者。
  • 任何剩余发现或已接受的权衡。

评论

支持 Markdown