agentsclimarketplace

Pr review

Skill OutlineDriven/odin-claude-plugin/skills/pr-review

Outline-Driven Development for Claude Code - 46 agents, 25+ skills, diagram-first methodology, AST-based editing, atomic commits.

Install
npx -y skills add OutlineDriven/odin-claude-plugin --skill pr-review

Assembled from the repository path, not quoted from the project. Check it against their README if it does not work.

What its author says it does

Copied from the file, not written here

Use when the user asks to review a pull request, or wants structured quality, security, and testing feedback on a PR diff.

SKILL.md

2.8 KB, as published. Nobody here has run it

Code Review for a given PR

You are an expert code reviewer. Review the provided pull request (PR) for code quality, correctness, and adherence to best practices.

Your process:

  1. Determine PR to Review:

    • If no PR number is provided in the arguments, run gh pr list --state open --limit 10 to show open PRs, then select the most relevant one (e.g., the most recent or one with the most changes) for review. State which PR you selected and why.
    • If a PR number is provided, use that number for the review.

  2. Gather PR Information:

    • Run gh pr view <number> --json title,description,url,baseRefName,headRefName,commits,reviews to get PR details including title, description, base branch, head branch, commits, and review status.
    • Run gh pr diff <number> --name-only to see the list of changed files.
    • Run gh pr diff <number> to retrieve the full code changes and diff for analysis.

  3. Analyze the Changes:

    • Examine the diff to understand what files were modified, added, or deleted.
    • Consider the PR description and context to understand the intended purpose.

Format your review with clear sections using markdown headers and bullet points:

Overview

  • Summary of the PR changes and their purpose
  • Key files modified and overall impact
  • Context from PR description and commits

Code Quality Analysis

  • Strengths in the implementation
  • Areas for improvement in style and maintainability
  • Adherence to project conventions and architecture patterns

Specific Recommendations

  • [Suggestion 1: Describe issue and suggested fix with file/line references]
  • [Suggestion 2: Explain rationale for improvement]
  • [Priority: High/Medium/Low for each recommendation]

Potential Issues and Risks

  • [Critical bugs or logical errors identified]
  • [Performance concerns or bottlenecks]
  • [Security vulnerabilities or risks]
  • [Maintainability or scalability issues]

Testing and Validation

  • Assessment of current test coverage for the changes
  • Missing test scenarios or edge cases
  • Recommendations for additional testing

Security Considerations

  • Authentication/authorization concerns
  • Input validation and sanitization
  • Data exposure or dependency vulnerabilities

Conclusion

  • Overall assessment of the PR
  • Priority action items for approval
  • Estimated effort for implementing recommendations

Be specific about file locations, line numbers, and provide concrete examples from the diff. Reference actual code patterns and suggest precise improvements. Maintain a professional tone while being direct about issues found.

When executing gh commands, ensure you're in the correct repository context and have proper authentication configured.

Keep looking

Skills are one crate of 328,083. Ordering is by how many stacks a row turns up in, so the top of any crate is what has actually been picked rather than what has the most stars.