code-review

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

Updated Jul 12, 2025
One-click install
npx skills add https://github.com/kpayakv2/check-products --skill code-review-kpayakv2
Or copy as Structured Prompt for Agent▼
Please help me install this Agent Skill.
Skill: code-review
Source: https://github.com/kpayakv2/check-products/tree/main/.agents/skills/code-review
Command: npx skills add https://github.com/kpayakv2/check-products --skill code-review-kpayakv2

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 originating issue or spec asked for. Doing both in one pass lets one axis mask the other, so findings get lost or misranked. ## Core Features & Use Cases - Two-axis review: Runs a Standards review (repo conventions 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 and validates it before spawning reviewers, so bad refs fail fast. - 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: After finishing a feature branch, ask to "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 since main using the code-review skill, pointing it at the issue or spec the branch implements.

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 branch against coding standards and its 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 merge-base comparison. It then runs a Standards sub-agent and a Spec sub-agent in parallel and reports both sets of findings side by side.

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

It first checks commit messages for issue references like #123 or Closes #45 and fetches them via the configured issue tracker workflow. If none exist, it looks for a user-supplied path or a spec file under docs/, specs/, or .scratch/ matching the branch or feature.

What happens if the repo has no documented coding standards?▼

The Standards axis still applies a fixed baseline of Fowler code smells such as Duplicated Code, Feature Envy, and Shotgun Surgery. These baseline findings are always labelled judgement calls, and any documented repo standard overrides the baseline where they conflict.

Can I review changes if there is no spec at all?▼

Yes. If no spec is found and the user confirms none exists, the Spec sub-agent is skipped and the final report notes that no spec was available. The Standards review still runs normally against the diff.

Why are standards and spec findings reported separately?▼

A change can pass one axis and fail the other, such as clean code implementing the wrong requirement. Keeping the reports separate prevents one axis from masking the other and avoids reranking findings across fundamentally different criteria.