code-review

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

7|1|Updated Jul 17, 2026
One-click install
npx skills add https://github.com/J0Jng/MathModelingAgents --skill code-review-j0jng
Or copy as Structured Prompt for Agent▼
Please help me install this Agent Skill.
Skill: code-review
Source: https://github.com/J0Jng/MathModelingAgents/tree/main/.claude/skills/engineering/code-review
Command: npx skills add https://github.com/J0Jng/MathModelingAgents --skill code-review-j0jng

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 PRD asked for — and mixing both in one pass causes one axis to mask 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, wrong implementations) as parallel sub-agents, then reports them side by side. - Fixed-point diffing: Reviews changes since any commit, branch, tag, or merge-base using git diff <fixed-point>...HEAD, 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-supplied path, or PRD files under docs/, specs/, or .scratch/. - Use Case: Before merging a feature branch, ask for a review since main and receive separate Standards and Spec reports, each with cited findings and a per-axis summary. ## Quick Start Ask the assistant to review the changes on the current branch since main using the code-review skill.

Frequently Asked Questions about code-review

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

FAQPage Schema
How do I review all changes on a branch since main?▼

Provide the fixed point (e.g., main) and the skill runs git diff main...HEAD using the three-dot merge-base comparison. It then spawns two parallel sub-agents to review the diff against coding standards and the originating spec, reporting both axes separately.

How does the review check code against a spec or PRD?▼

The Spec sub-agent receives the spec located from issue references in commit messages, a user-supplied path, or PRD files under docs/, specs/, or .scratch/. It reports missing or partial requirements, unrequested behavior, and incorrect implementations, quoting the spec line for each finding.

What coding standards does the review check against?▼

It uses any standards documented in the repo, such as CODING_STANDARDS.md or CONTRIBUTING.md, plus a fixed baseline of Fowler code smells like Duplicated Code, Feature Envy, and Shotgun Surgery. Documented repo standards always override the baseline, and anything tooling already enforces is skipped.

What happens if there is no spec for the changes?▼

If no spec is found and the user confirms none exists, the Spec sub-agent is skipped and the final report notes that no spec was available. The Standards review still runs normally against the diff.

Why does the review fail before the sub-agents run?▼

The skill validates the fixed point with git rev-parse and confirms the diff is non-empty before spawning sub-agents. An unresolvable ref or empty diff fails fast at this step rather than inside the parallel review agents.