GitLab合并请求全面审查Skill review-mr

Name: GitLab合并请求全面审查Skill
Rating: 5 (1 reviews)
Author: TheBushidoCollective

名称: review-mr 描述: 获取GitLab合并请求的全面审查

审查GitLab合并请求

名称

gitlab:review-mr - 获取GitLab合并请求的全面审查

概要

/review-mr [参数]

描述

获取GitLab合并请求的全面审查

实现

获取GitLab合并请求的全面审查，包括提交、差异和管道状态。

用法

/review-mr [项目ID] [合并请求编号]

此命令对合并请求进行彻底审查。

此命令的功能

使用get_merge_request获取合并请求详情
使用get_merge_request_commits检索所有提交
使用get_merge_request_diffs分析文件更改
使用get_merge_request_pipelines检查管道状态
提供结构化审查，包括：
- MR元数据（标题、作者、状态、日期）
- 提交历史和消息
- 文件更改摘要
- 代码审查建议
- CI/CD管道状态
- 批准或更改的建议

先决条件

必须安装和配置GitLab MCP服务器
您必须有权访问项目和合并请求
必须完成OAuth认证

示例交互

用户: /review-mr 456 15

Claude: 正在审查项目456中的合并请求!15...

## 合并请求审查: !15 - 添加基于JWT的用户认证

### 概述
- **作者**: @johndoe
- **状态**: 开放
- **创建时间**: 2025-01-15
- **目标分支**: main ← feature/user-authentication
- **更改**: +347行, -12行，跨8个文件

### 提交（共3个）
1. feat: 添加JWT令牌生成和验证
2. feat: 实现用户登录和注册端点
3. test: 添加认证流程的单元测试

### 文件更改
✓ 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) - 单元测试
⚠ src/config/keys.ts (+0, -7) - 移除硬编码密钥（很好！）

### 代码审查观察

**优点:**
- 使用专用认证模块清晰分离关注点
- 登录/注册流程中的全面错误处理
- 良好覆盖了正常和错误路径的测试
- 正确的密码哈希实现
- 移除了硬编码密钥

**改进建议:**
1. **安全**: 考虑在登录端点添加速率限制
2. **文档**: 为JWT工具函数添加JSDoc注释
3. **测试**: 可以从集成测试中受益
4. **错误消息**: 登录错误可以更具体以便调试

### CI/CD管道状态
管道 #1234 - ✓ 通过 (4分32秒)
- build: ✓ 通过 (1分23秒)
- test: ✓ 通过 (2分45秒)
- lint: ✓ 通过 (24秒)

### 建议
✅ **批准并附带次要建议**

此MR安全地实现了认证并遵循最佳实践。上述建议将进一步提高代码质量，但不是阻碍。管道通过了所有检查。

考虑在后续MR或合并前解决速率限制和文档建议。

参数

项目ID（必需）: 数字项目ID或URL编码路径
合并请求编号（必需）: 合并请求编号（GitLab中的!NUMBER）

提示

定期审查合并请求以维护代码质量
查找安全问题，特别是在认证代码中
检查新功能的测试覆盖
在批准前验证CI/CD管道通过
考虑大型更改的架构影响
在实际MR页面上留下建设性反馈
使用此功能进行详细代码审查前的预分析

名称: review-mr 描述: 获取GitLab合并请求的全面审查

审查GitLab合并请求

名称

概要

描述

实现

用法

此命令的功能

先决条件

示例交互

参数

提示

相关命令