code-review

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

Updated Jul 25, 2026
One-click install
npx skills add https://github.com/regisleandro/clara-financas --skill code-review-regisleandro
Or copy as Structured Prompt for Agent
Please help me install this Agent Skill.
Skill: code-review
Source: https://github.com/regisleandro/clara-financas/tree/main/.agents/skills/code-review
Command: npx skills add https://github.com/regisleandro/clara-financas --skill code-review-regisleandro

SYSTEM DOCUMENTATION & REQUIREMENTS

What problem does it solve? Reviewing a branch or PR requires checking two independent things — whether the code follows the repo's documented standards and whether it actually implements what the issue or PRD asked for. Doing both in one pass lets one concern mask the other, and findings get lost or reranked arbitrarily. ## Core Features & Use Cases - Two-axis review: Runs a Standards review (documented repo conventions plus a fixed Fowler code-smell baseline) and a Spec review (requirements coverage, scope creep, incorrect implementations) as parallel sub-agents that don't pollute each other's context. - Fixed-point diffing: Pins any commit, branch, tag, or merge-base the user supplies and validates the ref and non-empty diff before spawning reviewers. - 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 smells, and a Spec report quoting unmet requirements, side by side. ## Quick Start Ask the assistant to review the changes since main using the code-review skill and point it to the issue or spec file for the feature.

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 skill diffs HEAD against it using a three-dot merge-base comparison. It then runs two parallel sub-agents: one checks documented standards and code smells, the other checks the originating issue or PRD.

How does the skill find the spec for a code review?

It looks for issue references in commit messages first, then a user-supplied path, then PRD or spec files under docs/, specs/, or .scratch/ matching the branch or feature. If nothing is found, it asks the user, and the Spec axis is skipped when no spec exists.

What code smells does the standards review check for?

It applies a fixed baseline of Fowler smells including Duplicated Code, Feature Envy, Data Clumps, Primitive Obsession, Shotgun Surgery, Divergent Change, and Message Chains. Documented repo standards always override the baseline, and smells are reported as judgement calls, not hard violations.

Can I review work-in-progress changes instead of a finished PR?

Yes, any fixed point works — a commit SHA, tag, HEAD~5, or a branch name. The skill validates that the ref resolves and the diff is non-empty before spawning the review sub-agents, so bad refs fail early.

Why are standards and spec findings reported separately?

A change can follow every convention but implement the wrong thing, or match the spec while breaking project conventions. Keeping the two axes separate prevents one axis from masking the other and avoids arbitrary cross-axis reranking of findings.