Help us improve
Share bugs, ideas, or general feedback.
From gitlab
Reviews GitLab merge requests including metadata, commits, diffs, pipeline status, code suggestions, and approval recommendations. Invoke via /review-mr.
npx claudepluginhub thebushidocollective/han --plugin gitlabHow this skill is triggered — by the user, by Claude, or both
Slash command
/gitlab:review-mrThe summary Claude sees in its skill listing — used to decide when to auto-load this skill
gitlab:review-mr - Get comprehensive review of a GitLab merge request
Reviews GitHub pull requests with metadata, commits, file diffs, code suggestions, CI/CD status, and approval recommendations. Use /review-pr owner/repo pr-number.
Manages GitLab merge requests via glab: create from commits, list/view/checkout, merge with auto-handling, block MRs, handle reviews/discussions/stacks.
Searches, retrieves, and installs Agent Skills from prompts.chat registry using MCP tools like search_skills and get_skill. Activates for finding skills, browsing catalogs, or extending Claude.
Share bugs, ideas, or general feedback.
gitlab:review-mr - Get comprehensive review of a GitLab merge request
/review-mr [arguments]
Get comprehensive review of a GitLab merge request
Get a comprehensive review of a GitLab merge request including commits, diffs, and pipeline status.
/review-mr [project-id] [mr-number]
This command performs a thorough review of a merge request.
get_merge_requestget_merge_request_commitsget_merge_request_diffsget_merge_request_pipelinesUser: /review-mr 456 15
Claude: Reviewing merge request !15 in project 456...
## Merge Request Review: !15 - Add JWT-based user authentication
### Overview
- **Author**: @johndoe
- **Status**: Open
- **Created**: 2025-01-15
- **Target Branch**: main ← feature/user-authentication
- **Changes**: +347 lines, -12 lines across 8 files
### Commits (3 total)
1. feat: add JWT token generation and validation
2. feat: implement user login and registration endpoints
3. test: add unit tests for authentication flows
### Files Changed
✓ src/auth/jwt.ts (+145, -0) - New JWT utilities
✓ src/api/login.ts (+89, -5) - Login endpoint implementation
✓ src/api/register.ts (+67, -0) - Registration endpoint
✓ src/middleware/auth.ts (+23, -0) - Authentication middleware
✓ tests/auth.test.ts (+23, -0) - Unit tests
⚠ src/config/keys.ts (+0, -7) - Removed hardcoded secrets (good!)
### Code Review Observations
**Strengths:**
- Clean separation of concerns with dedicated auth module
- Comprehensive error handling in login/register flows
- Good test coverage for happy and error paths
- Proper password hashing implementation
- Removed hardcoded secrets
**Suggestions for Improvement:**
1. **Security**: Consider adding rate limiting to login endpoint
2. **Documentation**: Add JSDoc comments to JWT utility functions
3. **Testing**: Could benefit from integration tests
4. **Error Messages**: Login errors could be more specific for debugging
### CI/CD Pipeline Status
Pipeline #1234 - ✓ Passed (4m 32s)
- build: ✓ Passed (1m 23s)
- test: ✓ Passed (2m 45s)
- lint: ✓ Passed (24s)
### Recommendation
✅ **Approve with minor suggestions**
This MR implements authentication securely and follows best practices. The suggestions above would further improve code quality but aren't blocking. The pipeline passes all checks.
Consider addressing the rate limiting and documentation suggestions in a follow-up MR or before merging.
project-id (required): Numeric project ID or URL-encoded pathmr-number (required): Merge request number (the !NUMBER from GitLab)/create-mr: Create a new merge request/view-mr: View basic MR details without full review/search-code: Find similar implementations for comparison