关于 pr-review
用于依据关联议题和提交代码评估 Pull Request 的审查技能。它按五项检查顺序给出结论,并在需要修改时说明具体改动。
主要功能
- 01审查标题、描述和测试
- 02对照议题与提交代码核实声明
- 03评估价值、方案、正确性和范围
- 04按首个未通过项确定结论
- 05提供证据和修改要求
适用场景
- 01审查指定的 Pull Request
- 02检查仓库中符合条件的未关闭 PR
- 03决定合并、修改、分流或关闭
- 04为作者提供具体修改指引
安装
复制下面的命令,在终端中运行。
$
npx skills add https:/ / github.com/ wkentaro/ agent-skills --skill pr-review安装后在 Claude Code 中输入/pr-review确认已加载。
附带文件agents/openai.yaml
详细介绍
PR 审查
如果指定了 PR,请逐一检查。若没有指定,请检查当前仓库中所有未关闭、非草稿且没有 recommend-* 标签的 PR,每个 PR 分配一个子代理。
对照关联的 issue 和头部提交中的代码,检查标题、描述和测试;它们都只是声明。
按顺序回答以下五个问题,每个问题提供一行证据:
- 有意义:这个问题是真实存在的,并且在基分支上仍然存在吗?
- 值得添加:它带来的价值是否足以证明其范围和持续维护成本是合理的?
- 方法正确:你会从根本原因入手,以这种方式解决问题吗?只有在指出一种替代方案及其优势时才判定不通过:采用不同的设计,或先进行重构或架构调整,从而缩小或消除这项改动。
- 正确:它是否解决了问题,且不会破坏调用方或边界情况?它为行为变更新增的测试是否会在基分支上失败?
- 最小化:每一行改动都是必需的吗?是否没有重复实现现有代码已具备的功能?
根据第一个未通过项给出结论:
| 未通过项 | 结论 |
|---|---|
| 有意义,或出于技术原因值得添加 | recommend-close |
| 出于产品或范围考量值得添加 | recommend-triage |
| 方法正确、正确性或最小化 | recommend-revise |
| 全部通过 | recommend-merge |
向调用方报告结论和五个问题的答案;若结论为 revise,还需说明需要进行的具体更改。由调用方决定 PR 最终是否合并。
评论
支持 Markdown