code-review

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

2|Updated Jun 21, 2026
One-click install
npx skills add https://github.com/IsKenKenYa/skills --skill code-review-iskenkenya
Or copy as Structured Prompt for Agent▼
Please help me install this Agent Skill.
Skill: code-review
Source: https://github.com/IsKenKenYa/skills/tree/main/skills/engineering/code-review
Command: npx skills add https://github.com/IsKenKenYa/skills --skill code-review-iskenkenya

SYSTEM DOCUMENTATION & REQUIREMENTS

What problem does it solve? Reviewing a branch or PR requires checking two separate concerns at once—whether the code follows the repo's documented standards and whether it faithfully implements the original spec—and mixing them in one pass lets one axis mask failures in the other. ## Core Features & Use Cases - Two-axis review: Runs a Standards review (documented repo conventions plus a 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 reviews the three-dot diff against HEAD, validating the ref and non-empty diff before spawning agents. - Spec source discovery: Locates the originating spec from issue references in commit messages, a user-provided path, or files under docs/, specs/, or .scratch/. - Use Case: Before merging a feature branch, ask for a review since main; you receive separate Standards and Spec reports with per-file findings, quoted spec lines, and a one-line summary of the worst issue per axis. ## 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 a git branch against coding standards and a spec?▼

Provide a fixed point such as a commit SHA, branch, tag, or main, and the skill diffs it against HEAD using a three-dot comparison. Two parallel sub-agents then check standards compliance and spec fidelity, reporting findings side by side.

What does review since a commit mean in git?▼

It compares the current HEAD against the merge-base of the given ref using git diff <ref>...HEAD. This shows only changes introduced on your branch, excluding unrelated commits on the other side.

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

The skill searches commit messages for issue references, user-provided paths, and 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.

Which code smells does the standards review check for?▼

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

Why are standards and spec reviews kept separate?▼

A change can pass one axis and fail the other: code may follow every convention but implement the wrong thing, or meet the spec while breaking project conventions. Separate reports prevent one axis from masking failures in the other.