code-review

Reviews git diffs against coding standards and spec requirements using parallel sub-agents.

Updated Jul 16, 2026
One-click install
npx skills add https://github.com/MaksymilianCzadowski/skills --skill code-review-maksymilianczadowski
Or copy as Structured Prompt for Agent▼
Please help me install this Agent Skill.
Skill: code-review
Source: https://github.com/MaksymilianCzadowski/skills/tree/main/skills/engineering/code-review
Command: npx skills add https://github.com/MaksymilianCzadowski/skills --skill code-review-maksymilianczadowski

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 originating issue or PRD asked for — and doing both in one pass lets one concern mask the other. ## Core Features & Use Cases - Two-axis review: Runs a Standards review (documented repo standards 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: Reviews the changes between HEAD and any commit, branch, tag, or merge-base the user supplies, validating the ref and diff before spawning agents. - 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 — 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 assistant to review the changes on this branch since main against the repo's coding standards and the originating issue.

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 git branch against coding standards and a spec?▼

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

How to review changes since a specific commit or tag?▼

Pass any ref — a commit SHA, tag, branch name, or expression like HEAD~5 — as the fixed point. The ref is validated with git rev-parse and the diff is confirmed non-empty before the review agents run.

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

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

Does the code review work when a repo has no documented standards?▼

Yes. A built-in baseline of Fowler code smells from Refactoring chapter 3 applies, covering Duplicated Code, Feature Envy, Shotgun Surgery, and others. Documented repo standards always override the baseline, and smells are reported as judgement calls, not hard violations.

Why are standards and spec findings reported separately instead of merged?▼

A change can pass one axis and fail the other — standards-compliant code can implement the wrong thing, and correct implementations can break conventions. Keeping the reports separate prevents one axis from masking failures in the other.