mcpField
P
pr-review

pr-review

github.com/proffesor-for-testing测试与质量520151 次安装今天更新
代码仓库

关于 pr-review

用于审查 AQE 范围内 GitHub 拉取请求的工作流技能,涵盖范围核对、信任等级、代码质量和评审沟通。

主要功能

  1. 01阅读完整 PR 差异与说明
  2. 02排除 AQE 范围外的平台技能
  3. 03核对信任等级分配
  4. 04检查测试、安全与生产风险
  5. 05发布可执行的评审意见

适用场景

  1. 01审查 AQE 范围内的拉取请求
  2. 02核对技能变更及信任等级
  3. 03查找安全或测试覆盖问题

安装

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

$npx skills add https://github.com/proffesor-for-testing/agentic-qe --skill pr-review

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

详细介绍

PR 审查工作流

在正确界定 AQE 范围、清晰沟通并提供可执行反馈的前提下审查拉取请求。

参数

  • <pr-number> — 要审查的 GitHub PR 编号。如果省略,请提示用户输入。

步骤

1. 阅读完整差异
gh pr diff <pr-number>
gh pr view <pr-number>

阅读完整差异和 PR 描述。不要略读——逐一阅读每个有改动的文件。

2. 范围检查
  • 仅分析 AQE/QE 技能(不包括 Claude Flow 平台技能)
  • 应排除的平台技能:v3-, flow-nexus-, agentdb-, reasoningbank-, swarm-, github-, hive-mind-advanced、hooks-automation、iterative-loop、stream-chain、skill-builder、sparc-methodology、pair-programming、release、debug-loop
  • 如果 PR 涉及技能,核实数量和范围是否符合预期(约 82 个 AQE 技能)
  • 标记可能混入以 AQE 为重点的 PR 中的平台技能改动
3. 总结改动

以用户易于理解的方式总结改动及原因:

  • 关注结果,而非实现细节
  • 避免使用过于技术化的术语
  • 控制在 3–5 个项目符号内
4. 信任级别验证

对于任何技能改动,验证 trust_tier 赋值:

  • tier 3 = 具备评估基础设施(evals/、schemas/、scripts/)
  • tier 2 = 已测试,但没有评估框架
  • tier 1 = 未测试
  • 标记不一致之处(例如,具备 evals/ 的技能却被标为 tier 2 时,应为 tier 3)
5. 代码质量审查

检查以下内容:

  • 硬编码的版本字符串
  • 生产环境安全问题(适配器改动、破坏性改动)
  • 新代码缺少测试覆盖
  • 安全问题(泄露的密钥、注入风险)
6. 发布审查意见
gh pr review <pr-number> --body "review comments"

沟通规则

  • 保持建设性,并提供可执行的建议
  • 以结果为导向:说明作者应该做什么,而不是哪里有问题
  • 将相关评论归为一组,而不是发布许多零散评论
  • 如果批准时只有一些小建议,请使用 APPROVE 并附上评论,而不是使用 REQUEST_CHANGES

评论

支持 Markdown