code-review

Reviews a change set against coding standards and its originating spec on two independent axes.

6|2|Updated May 6, 2026
One-click install
npx skills add https://github.com/sek788432/Stock-Back-Test-System --skill code-review-sek788432
Or copy as Structured Prompt for Agent▼
Please help me install this Agent Skill.
Skill: code-review
Source: https://github.com/sek788432/Stock-Back-Test-System/tree/main/.agents/skills/code-review
Command: npx skills add https://github.com/sek788432/Stock-Back-Test-System --skill code-review-sek788432

SYSTEM DOCUMENTATION & REQUIREMENTS

What problem does it solve? Code reviews often conflate two separate questions: whether code follows project conventions and whether it actually implements what was requested. This Skill reviews a complete change set since a fixed point (commit, branch, tag, or merge-base) along independent Standards and Spec axes so one axis cannot mask failures in the other. ## Core Features & Use Cases - Four-part scope capture: Collects committed diffs, staged changes, unstaged changes, and untracked files relative to a resolved merge-base, so nothing in the working tree escapes review. - Standards axis with smell baseline: Checks documented repo standards plus a fixed set of Fowler code smells (Duplicated Code, Feature Envy, Shotgun Surgery, and more), with repo standards always overriding the baseline. - Spec axis with source resolution: Locates the originating issue, PRD, or spec file (including Docs/Specs/) and reports missing requirements, scope creep, and incorrect implementations with quoted spec lines. - Use Case: Before opening a pull request, run this Skill against main to get separate ## Standards and ## Spec reports, then follow up with the repository's comprehensive review skill for merge readiness. ## Quick Start Review all changes since origin/main against the repo's coding standards and the spec for this branch.

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 since a specific commit or branch?▼

Specify a fixed point such as a commit SHA, branch, tag, or HEAD~5. The review resolves the merge-base and captures committed diffs, staged changes, unstaged changes, and untracked files so the entire change set is covered.

What is the difference between the Standards and Spec review axes?▼

The Standards axis checks conformance to documented repo coding standards plus a baseline of Fowler code smells. The Spec axis verifies the diff faithfully implements the originating issue, PRD, or spec. They are reported separately so one cannot mask the other.

What happens if no spec is found for the change?▼

The review searches issue references, user-provided paths, and spec locations like Docs/Specs/. If nothing is found it asks the user; if no spec exists, the Spec axis is skipped and reported as "no spec available" rather than treated as a pass.

Does this focused review replace a full pre-merge review?▼

No. This is a focused two-axis review of a change set since a fixed point. Before a commit, pull request, merge, or handoff, the repository's comprehensive review skill and Definition of Done still apply.

Which code smells does the Standards axis check?▼

It applies a fixed Fowler baseline including Mysterious Name, Duplicated Code, Feature Envy, Data Clumps, Primitive Obsession, Repeated Switches, Shotgun Surgery, Divergent Change, Speculative Generality, Message Chains, Middle Man, and Refused Bequest. Documented repo standards override the baseline.