code-review

Reviews git diffs against coding standards and originating specs using parallel sub-agents.

1|Updated Jun 2, 2026
One-click install
npx skills add https://github.com/SevWren/Daily-Motivation-Brain-Helper --skill code-review-sevwren
Or copy as Structured Prompt for Agent▼
Please help me install this Agent Skill.
Skill: code-review
Source: https://github.com/SevWren/Daily-Motivation-Brain-Helper/tree/main/CLAUDE/skills/engineering/code-review
Command: npx skills add https://github.com/SevWren/Daily-Motivation-Brain-Helper --skill code-review-sevwren

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 the diff between HEAD and any commit, branch, tag, or merge-base the user supplies, 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, user-provided paths, 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 citing the violated rule or spec line. ## 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 (e.g., main, a commit SHA, or a tag) and the skill diffs it against HEAD using a three-dot merge-base comparison. It then runs Standards and Spec reviews in parallel sub-agents and reports both axes separately.

How does the spec review find the originating issue or PRD?▼

It looks for issue references in commit messages first, then a user-provided path, then PRD files under docs/, specs/, or .scratch/ matching the branch or feature. If nothing is found, it asks the user or reports that no spec is available.

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.

Can it review work-in-progress changes that are not committed?▼

The review operates on the diff between HEAD and a fixed point, so uncommitted working-tree changes are not included. Commit your work-in-progress first, then specify the fixed point to review against.

Why are standards and spec findings reported separately?▼

A change can pass one axis and fail the other — correct code for the wrong requirement, or the right feature written against conventions. Keeping them separate prevents one axis from masking failures in the other.