code-review

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

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

SYSTEM DOCUMENTATION & REQUIREMENTS

What problem does it solve? Reviewing a branch or PR requires checking two independent things at once: whether the code follows the repo's documented standards and whether it actually implements what the originating issue or spec asked for. Doing both in one pass lets one concern mask the other, and findings get lost. ## Core Features & Use Cases - Two-axis review: Runs a Standards review (documented repo conventions plus a Fowler code-smell baseline) and a Spec review (requirements coverage, scope creep, incorrect implementations) as parallel sub-agents, then reports them side by side without merging or reranking. - Fixed-point diffing: Pins any commit, branch, tag, or merge-base the user supplies and reviews git diff <fixed-point>...HEAD, validating the ref and diff before spawning reviewers. - Automatic 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 and receive a ## Standards report citing violated conventions and code smells, plus a ## Spec report quoting unmet or over-delivered requirements. ## Quick Start Ask the assistant to review the changes on this 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 a branch against coding standards before merging?▼

Provide a fixed point such as main, a commit SHA, or a tag, and the skill diffs HEAD against it using a three-dot merge-base comparison. It then checks the diff against documented repo standards plus a baseline of Fowler code smells and reports findings per file or hunk.

How do I check if code matches the original issue or spec?▼

The Spec axis locates the originating spec from issue references in commit messages, a path you provide, or files under docs/, specs/, or .scratch/. It reports missing requirements, scope creep, and implementations that look wrong, quoting the spec line for each finding.

What code smells does the review check for?▼

The baseline covers twelve Fowler smells including Duplicated Code, Feature Envy, Data Clumps, Primitive Obsession, Shotgun Surgery, Divergent Change, and Speculative Generality. Each is reported as a labelled judgement call, and documented repo standards always override the baseline.

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

If no spec is found through commit messages, user input, or spec directories, the skill asks you where it is. If you confirm none exists, the Spec sub-agent is skipped and the final report notes that no spec was available while still delivering the Standards review.

Why does the review report Standards and Spec results separately?▼

A change can follow every convention yet implement the wrong thing, or match the spec while breaking project standards. Keeping the axes separate prevents one passing axis from masking failures in the other, so findings are never merged or reranked across axes.