code-review

Reviews a git diff against coding standards and the originating spec using parallel sub-agents.

1|Updated Feb 2, 2026
One-click install
npx skills add https://github.com/rockcookies/skills --skill code-review-rockcookies
Or copy as Structured Prompt for Agent▼
Please help me install this Agent Skill.
Skill: code-review
Source: https://github.com/rockcookies/skills/tree/main/skills/mattpocock/code-review
Command: npx skills add https://github.com/rockcookies/skills --skill code-review-rockcookies

SYSTEM DOCUMENTATION & REQUIREMENTS

What problem does it solve? Reviewing a branch or PR requires checking two separate things — whether the code follows the repo's documented standards and whether it actually implements what the issue or spec asked for — and mixing both in one pass lets one axis mask failures in the other. ## Core Features & Use Cases - Two-axis review: Runs a Standards review (documented repo standards plus a fixed Fowler code-smell baseline) and a Spec review (requirements coverage, scope creep, incorrect implementations) as parallel sub-agents, then reports them side by side. - Fixed-point diffing: Compares HEAD against any user-supplied commit, branch, tag, or merge-base using three-dot git diff, with upfront validation that the ref resolves and the diff is non-empty. - Spec discovery: Locates the originating spec from issue references in commit messages, a user-provided path, or spec files under docs/, specs/, or .scratch/. - Use Case: Before merging a feature branch, ask for a review since main — you get a Standards report citing violated conventions and code smells, plus a Spec report quoting requirement lines that are missing, partial, or out of scope. ## Quick Start Ask the agent to review the changes on this branch since main against the coding standards and the originating issue spec.

Frequently Asked Questions about code-review

High-intent search queries and answers about installing and using this skill.

FAQPage Schema
How do I review a branch against coding standards and its spec?▼

Provide a fixed point such as a commit SHA, branch, tag, or main, and the review compares HEAD against it using a three-dot git diff. Two parallel sub-agents check standards compliance and spec fidelity, then report findings side by side.

How to review changes since a specific commit in git?▼

Pass the commit SHA, tag, or expression like HEAD~5 as the fixed point. The skill validates the ref with git rev-parse, captures the diff and commit list, and fails early if the ref is bad or the diff is empty.

What happens if there is no spec for the code being reviewed?▼

The skill searches commit messages for issue references, checks a user-provided path, and looks under docs/, specs/, or .scratch/. If no spec exists, the Spec sub-agent is skipped and the final report notes that no spec was available.

Does the standards review work without documented coding standards?▼

Yes, it applies a fixed baseline of Fowler code smells such as Duplicated Code, Feature Envy, and Shotgun Surgery even when the repo documents nothing. Documented repo standards always override the baseline, and smells are reported as judgement calls rather than hard violations.

Why are standards and spec findings reported separately?▼

A change can pass one axis and fail the other — code can follow every standard but implement the wrong thing, or match the spec while breaking conventions. Keeping the reports separate prevents one axis from masking failures in the other.