code-review

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

1|Updated Oct 29, 2025
One-click install
npx skills add https://github.com/jwh3times/holland-vip --skill code-review-jwh3times
Or copy as Structured Prompt for Agent▼
Please help me install this Agent Skill.
Skill: code-review
Source: https://github.com/jwh3times/holland-vip/tree/main/.agents/skills/code-review
Command: npx skills add https://github.com/jwh3times/holland-vip --skill code-review-jwh3times

SYSTEM DOCUMENTATION & REQUIREMENTS

What problem does it solve? Code reviews often conflate two separate questions — does the code follow the project's conventions, and does it actually implement what was asked for — letting one axis mask failures in the other. This Skill separates them into independent parallel reviews so neither hides the other's findings. ## Core Features & Use Cases - Two-axis review: Runs a Standards review (repo conventions plus a Fowler code-smell baseline) and a Spec review (fidelity to the originating issue or spec) as parallel sub-agents, then reports them side by side. - Fixed-point diffing: Reviews changes since any commit, branch, tag, or merge-base using a three-dot git diff, with upfront validation that the ref resolves and the diff is non-empty. - Smell baseline: Applies a labelled set of Fowler code smells (Duplicated Code, Feature Envy, Shotgun Surgery, and more) as judgement-call heuristics that documented repo standards always override. - 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, plus a per-axis summary. ## 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 runs git diff main...HEAD using the three-dot merge-base comparison. It then reviews that diff against repo standards and the originating spec in parallel sub-agents.

How does a spec-based code review work?▼

The skill locates the originating spec from issue references in commit messages, a user-supplied path, or tracked planning documents. A dedicated sub-agent then reports missing requirements, scope creep, and incorrect implementations, quoting the spec line for each finding.

What coding standards does the review check against?▼

It reads the repo's own instruction files such as AGENTS.md and CLAUDE.md first, then applies a fixed baseline of Fowler code smells like Duplicated Code and Feature Envy. Documented repo standards always override the baseline, and anything tooling already enforces is skipped.

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

If no spec is found through commit references, user input, or repo documents, the skill asks the user directly. 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 sub-agents. An unresolvable ref or empty diff fails at this step rather than inside the parallel reviews.