关于 review-pr-changes
用于按代码质量、测试、文档、安全和 Git 清单开展结构化 PR 审查的技能。它涵盖检查背景与差异、测试、发表评论到给出结论的流程。
主要功能
- 01涵盖关键领域的审查清单
- 02分步审查工作流
- 03规定结果报告结构
- 04常见审查问题的处理建议
适用场景
- 01合并前审查 PR
- 02检查变更中的测试与边界情况
- 03报告安全或文档问题
安装
复制下面的命令，在终端中运行。
$
npx skills add https:/安装后在 Claude Code 中输入/review-pr-changes确认已加载。
附带文件.claude-plugin/plugin.json
详细介绍
审查 PR 更改
根据项目质量标准，对 PR 更改进行结构化审查。
快速参考
# Get PR files changed
gh pr diff <pr> --name-only
# View specific file diff
gh pr diff <pr> -- path/to/file
# Get PR review status
gh pr view <pr> --json reviews
# Check file statistics
gh pr diff <pr> | diffstat
# Get PR body/description
gh pr view <pr> --json body
审查清单
代码质量：
- 代码清晰易读、结构良好
- 函数／类用途明确
- 没有代码重复（DRY 原则）
- 错误处理得当
- 没有不必要的复杂性
测试：
- 新功能有对应的测试
- 测试通过（CI 显示绿色）
- 已覆盖边缘情况
- 更改有充分的测试覆盖
文档：
- 公共 API 有文档字符串
- 已添加类型提示
- 对不明显的代码添加了注释
安全与 Git：
- 没有硬编码的密钥／令牌
- 已进行输入验证
- PR 已关联 issue（在描述中）
- 没有包含非预期文件
审查工作流
- 检查上下文：查看 PR 描述和关联的 issue
- 浏览更改：查看文件列表和统计信息
- 阅读代码：仔细检查实际更改
- 执行清单：逐项检查各个类别
- 本地测试：如有需要，拉取并测试更改
- 创建评论：以代码评论的形式标记问题
- 给出结论：批准、请求更改或发表评论
输出格式
审查结果必须严格包含以下部分：
- Summary — 用 1-2 句话进行整体评估
- Issues Found — 必须修复的问题（如果没有则省略）
- Verdict — Approve / Request Changes / Comment
错误处理
| 问题 | 解决方案 |
|---|---|
| 无法访问 PR | 检查 gh auth status |
| 无法理解代码 | 在评论中提出澄清问题 |
| 需要本地测试 | 使用 worktree-create 技能进行测试 |
| 存在多个问题 | 优先处理关键问题，其次处理可选问题 |
| 对代码风格存在分歧 | 参阅 CLAUDE.md 中的标准 |
参考资料
- 项目标准请参阅 CLAUDE.md
- 合并就绪情况检查请参阅 verify-pr-ready
- 批量审查工作流请参阅 gh-batch-merge-by-labels
评论
支持 Markdown