code-review

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

Updated Sep 19, 2026
One-click install
npx skills add https://github.com/Ab0umar/selrs.cc.BU --skill code-review-ab0umar
Or copy as Structured Prompt for Agent▼
Please help me install this Agent Skill.
Skill: code-review
Source: https://github.com/Ab0umar/selrs.cc.BU/tree/main/.agents/skills/code-review
Command: npx skills add https://github.com/Ab0umar/selrs.cc.BU --skill code-review-ab0umar

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 issue or spec asked for. Doing both in one pass lets one concern mask the other, and doing it manually is slow and inconsistent. ## 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 without merging or reranking. - Fixed-point diffing: Pins any commit, branch, tag, or merge-base the user supplies, validates the ref and non-empty diff before spawning reviewers, and uses three-dot git diff semantics. - Automatic spec discovery: Locates the originating spec from issue references in commit messages, a user-supplied 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 smells, plus a ## Spec report quoting unmet or over-built 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 all changes on a branch since main?▼

Provide a fixed point such as main, a commit SHA, tag, or HEAD~5, and the skill diffs it against HEAD using three-dot merge-base semantics. It then runs standards and spec reviews in parallel sub-agents and reports both side by side.

How does the code review check against the original issue or spec?▼

The Spec sub-agent locates the spec from issue references in commit messages, a user-supplied path, or files under docs/, specs/, or .scratch/. It reports missing requirements, scope creep, and incorrect implementations, quoting the spec line for each finding.

What coding standards does the review check if the repo documents none?▼

The Standards axis always applies a fixed baseline of Fowler code smells such as Duplicated Code, Feature Envy, and Shotgun Surgery. Documented repo standards override the baseline, and baseline smells are reported as judgement calls rather than hard violations.

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 none exists, 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 reviewers. An unresolvable ref or empty diff fails fast at this step instead of inside the parallel sub-agents.