Back to Skills
Skill
Community

review-pr

From github
106
Install
1
Install the plugin
$
npx claudepluginhub thebushidocollective/han --plugin github

Want just this skill?

Then install: npx claudepluginhub u/[userId]/[slug]

Description

Get comprehensive review of a GitHub pull request

Tool Access

This skill uses the workspace's default tool permissions.

Skill Content

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
Stats
Stars106
Forks13
Last CommitFeb 11, 2026

Similar Skills

cache-components

Expert guidance for Next.js Cache Components and Partial Prerendering (PPR). **PROACTIVE ACTIVATION**: Use this skill automatically when working in Next.js projects that have `cacheComponents: true` in their next.config.ts/next.config.js. When this config is detected, proactively apply Cache Components patterns and best practices to all React Server Component implementations. **DETECTION**: At the start of a session in a Next.js project, check for `cacheComponents: true` in next.config. If enabled, this skill's patterns should guide all component authoring, data fetching, and caching decisions. **USE CASES**: Implementing 'use cache' directive, configuring cache lifetimes with cacheLife(), tagging cached data with cacheTag(), invalidating caches with updateTag()/revalidateTag(), optimizing static vs dynamic content boundaries, debugging cache issues, and reviewing Cache Component implementations.

cache-components
138.4k