code-review

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

10|3|Updated Jul 18, 2019
One-click install
npx skills add https://github.com/tanqimin/MyFavsORM --skill code-review-tanqimin
Or copy as Structured Prompt for Agent
Please help me install this Agent Skill.
Skill: code-review
Source: https://github.com/tanqimin/MyFavsORM/tree/main/.agents/skills/code-review
Command: npx skills add https://github.com/tanqimin/MyFavsORM --skill code-review-tanqimin

SYSTEM DOCUMENTATION & REQUIREMENTS

What problem does it solve? Reviewing a branch or PR requires checking two separate 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 axis mask the other, so findings get lost or misranked. ## 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: Reviews the diff between HEAD and any user-supplied commit, branch, tag, or merge-base using three-dot git diff semantics. - Automatic spec discovery: Locates the originating spec from issue references in commit messages, user-supplied paths, or spec 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 citing the violated rule or spec line, with hard violations distinguished from judgement calls. ## 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 all changes on a branch since main?

Provide the fixed point (for example, main) and the skill computes a three-dot git diff against HEAD, then runs Standards and Spec reviews in parallel sub-agents. It reports findings under separate headings without merging or reranking them.

What coding standards does an automated code review check?

It checks any standards documented in the repo, such as CODING_STANDARDS.md or CONTRIBUTING.md, plus a built-in baseline of Fowler code smells like Duplicated Code, Feature Envy, and Shotgun Surgery. Documented repo standards always override the baseline.

Can a code review verify the code matches the original issue or spec?

Yes. The Spec axis locates the originating spec from issue references in commit messages, a user-supplied path, or spec files in the repo, then reports missing requirements, scope creep, and incorrect implementations with quoted spec lines.

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

If no spec is found after checking commit messages, user arguments, and spec directories, the skill asks the user where it is. If there is none, the Spec sub-agent is skipped and the final report notes that no spec was available.

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 reviews.