名称: 审查-pr 描述: 获取GitHub拉取请求的全面审查
审查GitHub拉取请求
名称
github:review-pr - 获取GitHub拉取请求的全面审查
摘要
/review-pr [参数]
描述
获取GitHub拉取请求的全面审查
实现
获取拉取请求的全面审查，包括提交、差异和检查状态。
用法
/review-pr [所有者/仓库] [pr-编号]
此命令对拉取请求执行彻底的审查。
该命令的功能
- 使用
get_pull_request获取拉取请求详情 - 检索提交历史
- 分析文件变更和差异
- 从GitHub Actions检查CI/CD状态
- 提供结构化审查，包括：
- PR元数据（标题、作者、状态、日期）
- 提交历史和消息
- 文件变更摘要
- 代码审查建议
- CI/CD工作流状态
- 批准或更改建议
先决条件
- GitHub MCP服务器必须安装并配置
- 您必须有权访问仓库
- OAuth认证必须完成
示例交互
用户: /review-pr my-org/my-repo 15
Claude: 正在审查my-org/my-repo中的拉取请求 #15...
## 拉取请求审查: #15 - 添加基于JWT的用户认证
### 概述
- **作者**: @johndoe
- **状态**: 开放
- **创建时间**: 2025-01-15
- **目标分支**: main <- feature/user-authentication
- **变更**: +347行, -12行，跨8个文件
### 提交（共3个）
1. 功能: 添加JWT令牌生成和验证
2. 功能: 实现用户登录和注册端点
3. 测试: 添加认证流程的单元测试
### 文件变更
+ src/auth/jwt.ts (+145, -0) - 新JWT工具
+ src/api/login.ts (+89, -5) - 登录端点实现
+ src/api/register.ts (+67, -0) - 注册端点
+ src/middleware/auth.ts (+23, -0) - 认证中间件
+ tests/auth.test.ts (+23, -0) - 单元测试
### 代码审查观察
**优点:**
- 使用专用认证模块实现清晰的关注点分离
- 登录/注册流程中全面的错误处理
- 对正常和错误路径的良好测试覆盖
- 正确的密码哈希实现
**改进建议:**
1. 考虑向登录端点添加速率限制
2. 为JWT工具函数添加JSDoc注释
3. 可以从集成测试中受益
### CI/CD状态
- 构建: 通过（1分23秒）
- 测试: 通过（2分45秒）
- 代码检查: 通过（24秒）
### 推荐
**批准并附上次要建议**
此PR安全地实现认证并遵循最佳实践。
参数
所有者/仓库(必需): 所有者/仓库格式的仓库pr-编号(必需): 拉取请求编号
提示
- 定期审查PR以维护代码质量
- 寻找安全问题，特别是在认证代码中
- 检查新功能的测试覆盖
- 在批准前验证CI通过
- 考虑大型变更的架构影响
- 在详细代码审查前使用此进行预分析
相关命令
/create-pr: 创建新拉取请求/search-code: 查找类似实现进行比较/view-workflow: 检查详细工作流运行状态