关于 qe-pr-review
qe-pr-review 是面向 AQE/QE 技能贡献者的 GitHub 拉取请求审查流程,涵盖范围检查、信任级别验证和代码质量检查,并指导撰写可执行的审查意见。
主要功能
- 01检查完整差异和 PR 描述
- 02核对 AQE/QE 范围边界
- 03验证信任级别分配
- 04检查代码质量与安全问题
- 05指导撰写建设性审查意见
适用场景
- 01审查聚焦 AQE 的 GitHub 拉取请求
- 02检查技能范围和信任级别
- 03发现测试、安全或生产风险
安装
复制下面的命令,在终端中运行。
$
npx skills add https:/ / github.com/ proffesor-for-testing/ agentic-qe --skill qe-pr-review安装后在 Claude Code 中输入/qe-pr-review确认已加载。
详细介绍
PR Review 工作流
在正确划定 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、aqe-v2-v3-migration
- 如果 PR 涉及技能,请确认数量和范围符合预期(约 78 个 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