review-pr

What this skill does

# 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

1. Fetches pull request details using `get_pull_request`
2. Retrieves commit history
3. Analyzes file changes and diffs
4. Checks CI/CD status from GitHub Actions
5. 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 format
- `pr-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