Skip to content
cccskills
FreePublished on GitHub

code-review

Review GitHub pull requests with specialized agents that check for clear bugs and violations of project instructions in CLAUDE.md. Validate bug findings before posting focused comments on the affected lines, with fix suggestions where appropriate.

OriginalShow the original description

Automated code review for pull requests using multiple specialized agents

Works withGitHub
See how to install

Good for

  • Reviewing a pull request for clear bugs
  • Checking changes against CLAUDE.md rules
  • Posting validated findings on affected lines

Files included (1)

  • SKILL.md6.8 KB

SKILL.md (original)

Review the instructions the agent will receive before you install.

Provide a code review for the given pull request.

Agent assumptions (applies to all agents and subagents):

  • All tools are functional and will work without error. Do not test tools or make exploratory calls. Make sure this is clear to every subagent that is launched.
  • Only call a tool if it is required to complete the task. Every tool call should have a clear purpose.

To do this, follow these steps precisely:

  1. Launch a haiku agent to check if any of the following are true:

    • The pull request is closed
    • The pull request is a draft
    • The pull request does not need code review (e.g. automated PR, trivial change that is obviously correct)
    • Claude has already commented on this PR (check gh pr view <PR> --comments for comments left by claude)

    If any condition is true, stop and do not proceed.

Note: Still review Claude generated PR's.

  1. Launch a haiku agent to return a list of file paths (not their contents) for all relevant CLAUDE.md files including:

    • The root CLAUDE.md file, if it exists
    • Any CLAUDE.md files in directories containing files modified by the pull request
  2. Launch a sonnet agent to view the pull request and return a summary of the changes

  3. Launch 4 agents in parallel to independently review the changes. Each agent should return the list of issues, where each issue includes a description and the reason it was flagged (e.g. "CLAUDE.md adherence", "bug"). The agents should do the following:

    Agents 1 + 2: CLAUDE.md compliance sonnet agents Audit changes for CLAUDE.md compliance in parallel. Note: When evaluating CLAUDE.md compliance for a file, you should only consider CLAUDE.md files that share a file path with the file or parents.

    Agent 3: Opus bug agent (parallel subagent with agent 4) Scan for obvious bugs. Focus only on the diff itself without reading extra context. Flag only significant bugs; ignore nitpicks and likely false positives. Do not flag issues that you cannot validate without looking at context outside of the git diff.

    Agent 4: Opus bug agent (parallel subagent with agent 3) Look for problems that exist in the introduced code. This could be security issues, incorrect logic, etc. Only look for issues that fall within the changed code.

    CRITICAL: We only want HIGH SIGNAL issues. Flag issues where:

    • The code will fail to compile or parse (syntax errors, type errors, missing imports, unresolved references)
    • The code will definitely produce wrong results regardless of inputs (clear logic errors)
    • Clear, unambiguous CLAUDE.md violations where you can quote the exact rule being broken

    Do NOT flag:

    • Code style or quality concerns
    • Potential issues that depend on specific inputs or state
    • Subjective suggestions or improvements

    If you are not certain an issue is real, do not flag it. False positives erode trust and waste reviewer time.

    In addition to the above, each subagent should be told the PR title and description. This will help provide context regarding the author's intent.

  4. For each issue found in the previous step by agents 3 and 4, launch parallel subagents to validate the issue. These subagents should get the PR title and description along with a description of the issue. The agent's job is to review the issue to validate that the stated issue is truly an issue with high confidence. For example, if an issue such as "variable is not defined" was flagged, the subagent's job would be to validate that is actually true in the code. Another example would be CLAUDE.md issues. The agent should validate that the CLAUDE.md rule that was violated is scoped for this file and is actually violated. Use Opus subagents for bugs and logic issues, and sonnet agents for CLAUDE.md violations.

  5. Filter out any issues that were not validated in step 5. This step will give us our list of high signal issues for our review.

  6. If issues were found, skip to step 8 to post comments.

    If NO issues were found, post a summary comment using gh pr comment (if --comment argument is provided): "No issues found. Checked for bugs and CLAUDE.md compliance."

  7. Create a list of all comments that you plan on leaving. This is only for you to make sure you are comfortable with the comments. Do not post this list anywhere.

  8. Post inline comments for each issue using gh pr review with inline comments. For each comment:

    • Provide a brief description of the issue
    • For small, self-contained fixes, include a committable suggestion block
    • For larger fixes (6+ lines, structural changes, or changes spanning multiple locations), describe the issue and suggested fix without a suggestion block
    • Never post a committable suggestion UNLESS committing the suggestion fixes the issue entirely. If follow up steps are required, do not leave a committable suggestion.

    IMPORTANT: Only post ONE comment per unique issue. Do not post duplicate comments.

Use this list when evaluating issues in Steps 4 and 5 (these are false positives, do NOT flag):

  • Pre-existing issues
  • Something that appears to be a bug but is actually correct
  • Pedantic nitpicks that a senior engineer would not flag
  • Issues that a linter will catch (do not run the linter to verify)
  • General code quality concerns (e.g., lack of test coverage, general security issues) unless explicitly required in CLAUDE.md
  • Issues mentioned in CLAUDE.md but explicitly silenced in the code (e.g., via a lint ignore comment)

Notes:

  • Use gh CLI to interact with GitHub (e.g., fetch pull requests, create comments). Do not use web fetch.
  • Create a todo list before starting.
  • You must cite and link each issue in inline comments (e.g., if referring to a CLAUDE.md, include a link to it).
  • If no issues are found, post a comment with the following format:

Code review

No issues found. Checked for bugs and CLAUDE.md compliance.


  • When linking to code in inline comments, follow the following format precisely, otherwise the Markdown preview won't render correctly: https://github.com/OWNER/REPO/blob/FULL_SHA/path/to/file.py#L10-L15
    • Requires full git sha
    • You must provide the full sha. Commands like https://github.com/owner/repo/blob/$(git rev-parse HEAD)/foo/bar will not work, since your comment will be directly rendered in Markdown.
    • Repo name must match the repo you're code reviewing
    • sign after the file name

    • Line range format is L[start]-L[end]
    • Provide at least 1 line of context before and after, centered on the line you are commenting about (eg. if you are commenting about lines 5-6, you should link to L4-7)

Reviews

No reviews yet. Tell others what you thought.

Comments

No login needed. Share how it worked for you, ask a question, or note where it did not run.

No comments yet. Be the first to write one.

Comments are published right away. Do not include personal information such as email addresses or phone numbers. Up to 2 URLs.

0 / 1,000 characters

This form is protected by reCAPTCHA, and the Google Privacy Policy and Terms of Service apply.

More skills from this repository

changelog

FreeSummary

Create changelog files for significant changes in a pull request, a proposed code change. Describe effects users will notice and organize entries by PR number and change type, such as features or fixes.

  • Writing changelog entries for a pull request
  • Separating new features from bug fixes
  • Explaining changes in user-facing language
pipecat-ai/pipecat16.6KUpdated Oct 19, 2026

cleanup

FreeSummary

Review and improve code changes on the current branch using Pipecat conventions. Refine names and structure, add type hints and documentation, check example patterns, and make conservative performance improvements while preserving compatibility.

  • Cleaning up branch changes before review
  • Adding missing type hints and documentation
  • Checking Pipecat service and example patterns
pipecat-ai/pipecat16.6KUpdated Oct 19, 2026

docstring

FreeSummary

Add or improve Google-style docstrings, the explanations embedded in Python code, for modules and classes. Document constructor arguments, public methods, configuration fields, and enum values while preserving existing documentation that already meets the project’s conventions.

  • Document a Python module
  • Explain constructor arguments and public methods
  • Describe configuration fields and enum values
pipecat-ai/pipecat16.6KUpdated Oct 19, 2026

pr-description

FreeSummary

Draft or update GitHub pull request descriptions using the actual changes and related context. Explain the problem, changed behavior, and review concerns in plain language, including breaking changes and explicit issue-closing references when applicable.

  • Draft a description for a new pull request
  • Rewrite a description to reflect the final implementation
  • Explain behavior changes and review concerns
pipecat-ai/pipecat16.6KUpdated Oct 19, 2026

pr-submit

FreeSummary

Submit current work as a GitHub pull request, handling relevant uncommitted changes and reviewing prose before pushing. After creating the PR, run companion skills to generate changelog files, update the description, and return its URL.

  • Submitting current work as a GitHub pull request
  • Creating a branch and commits for uncommitted changes
  • Reviewing prose and adding a changelog before handoff
pipecat-ai/pipecat16.6KUpdated Oct 19, 2026

prose-review

FreeSummary

Review newly added comments, docstrings, and changelog entries so future readers can understand the current code. Revise development history, redundant explanations, and unclear shorthand while preserving constraints that the code cannot explain on its own.

  • Review comments before committing
  • Check prose added across a branch
  • Clarify function documentation for future readers
pipecat-ai/pipecat16.6KUpdated Oct 19, 2026

See all skills by pipecat-ai

Report a problem with this skill