Review GitHub Pull Request
Name
github:review-pr - Get comprehensive review of a GitHub pull request
Synopsis
/review-pr [arguments]Description
Get comprehensive review of a GitHub pull request
Implementation
Get a comprehensive review of a pull request including commits, diffs, and checks status.
Usage
/review-pr [owner/repo] [pr-number]This command performs a thorough review of a pull request.
What This Command Does
- Fetches pull request details using
get_pull_request - Retrieves commit history
- Analyzes file changes and diffs
- Checks CI/CD status from GitHub Actions
- Provides a structured review with:
- PR metadata (title, author, status, dates) - Commit history and messages - File changes summary - Code review suggestions - CI/CD workflow status - Recommendations for approval or changes
Prerequisites
- GitHub MCP server must be installed and configured
- You must have access to the repository
- OAuth authentication must be completed
Example Interaction
User: /review-pr my-org/my-repo 15
Claude: Reviewing pull request #15 in my-org/my-repo...
## Pull 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
### 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
**Suggestions for Improvement:**
1. Consider adding rate limiting to login endpoint
2. Add JSDoc comments to JWT utility functions
3. Could benefit from integration tests
### CI/CD Status
- build: Passed (1m 23s)
- test: Passed (2m 45s)
- lint: Passed (24s)
### Recommendation
**Approve with minor suggestions**
This PR implements authentication securely and follows best practices.Arguments
owner/repo(required): Repository in owner/repo formatpr-number(required): Pull request number
Tips
- Review PRs regularly to maintain code quality
- Look for security issues, especially in authentication code
- Check test coverage for new functionality
- Verify CI passes before approving
- Consider architectural impact of large changes
- Use this for pre-review analysis before detailed code review
Related Commands
/create-pr: Create a new pull request/search-code: Find similar implementations for comparison/view-workflow: Check detailed workflow run status