关于 pr-review
用于审查 AQE 范围内 GitHub 拉取请求的工作流技能,涵盖范围核对、信任等级、代码质量和评审沟通。
主要功能
- 01阅读完整 PR 差异与说明
- 02排除 AQE 范围外的平台技能
- 03核对信任等级分配
- 04检查测试、安全与生产风险
- 05发布可执行的评审意见
适用场景
- 01审查 AQE 范围内的拉取请求
- 02核对技能变更及信任等级
- 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