mcpField
G
git-pr-reviewer

git-pr-reviewer

github.com/onewave-ai测试与质量393324 次安装1 周前更新
代码仓库

关于 git-pr-reviewer

面向开发者的 Pull Request 审查技能,可检查差异中的逻辑、安全、性能、代码质量、错误处理和测试覆盖问题,并按严重程度整理审查结果。

主要功能

  1. 01检查变更文件
  2. 02发现逻辑错误
  3. 03审查安全风险
  4. 04评估性能与代码质量
  5. 05识别测试覆盖缺口

适用场景

  1. 01对比 main 审查分支
  2. 02查找 PR 安全问题
  3. 03检查变更代码的测试

安装

复制下面的命令,在终端中运行。

$npx skills add https://github.com/onewave-ai/claude-skills --skill git-pr-reviewer

安装后在 Claude Code 中输入/git-pr-reviewer确认已加载。

详细介绍

Git PR 审查器

说明

审查拉取请求时:

  1. 获取差异:运行 git diff main...HEAD 或 git diff <base-branch>...HEAD
  2. 分析变更文件:识别所有已修改、新增和删除的文件
  3. 审查每个文件,检查以下方面:
    • 逻辑错误和漏洞
    • 安全漏洞(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