关于 git-pr-reviewer
面向开发者的 Pull Request 审查技能,可检查差异中的逻辑、安全、性能、代码质量、错误处理和测试覆盖问题,并按严重程度整理审查结果。
主要功能
- 01检查变更文件
- 02发现逻辑错误
- 03审查安全风险
- 04评估性能与代码质量
- 05识别测试覆盖缺口
适用场景
- 01对比 main 审查分支
- 02查找 PR 安全问题
- 03检查变更代码的测试
安装
复制下面的命令,在终端中运行。
$
npx skills add https:/ / github.com/ onewave-ai/ claude-skills --skill git-pr-reviewer安装后在 Claude Code 中输入/git-pr-reviewer确认已加载。
详细介绍
Git PR 审查器
说明
审查拉取请求时:
- 获取差异:运行
git diff main...HEAD或git diff <base-branch>...HEAD - 分析变更文件:识别所有已修改、新增和删除的文件
- 审查每个文件,检查以下方面:
- 逻辑错误和漏洞
- 安全漏洞(SQL 注入、XSS、硬编码机密)
- 性能问题(N+1 查询、不必要的重新渲染、内存泄漏)
- 代码风格和一致性
- 缺少错误处理
- 测试覆盖率缺口
审查清单
安全性
- 未硬编码凭据或 API 密钥
- 对用户数据进行输入验证
- 具备适当的身份验证和授权检查
- 不存在 SQL 注入漏洞
- 已采取 XSS 防护措施
代码质量
- 函数简短且职责明确
- 没有代码重复
- 变量和函数命名清晰
- 错误处理得当
- 没有未使用的导入或无效代码
性能
- 没有不必要的数据库查询
- 使用了高效的算法
- 在需要时进行了适当的缓存
- 没有内存泄漏
测试
- 新代码有对应的测试
- 已覆盖边界情况
- 测试有实际意义,而非仅为提高覆盖率
输出格式
## PR Review Summary
### Overview
[Brief summary of changes]
### Issues Found
#### Critical
- [Issue description + file:line]
#### Warnings
- [Issue description + file:line]
#### Suggestions
- [Improvement ideas]
### Approval Status
[APPROVE / REQUEST CHANGES / NEEDS DISCUSSION]
示例
# Review current branch against main
git diff main...HEAD --stat
git diff main...HEAD
评论
支持 Markdown