code-review

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

Updated Sep 6, 2026
One-click install
npx skills add https://github.com/malikkotb/shellpluscore --skill code-review-malikkotb
Or copy as Structured Prompt for Agent
Please help me install this Agent Skill.
Skill: code-review
Source: https://github.com/malikkotb/shellpluscore/tree/main/.agents/skills/code-review
Command: npx skills add https://github.com/malikkotb/shellpluscore --skill code-review-malikkotb

SYSTEM DOCUMENTATION & REQUIREMENTS

What problem does it solve? Reviewing a branch or PR requires checking two separate things at once: whether the code follows the repo's coding standards and whether it actually implements what the issue or spec asked for. Mixing both in one review lets one axis mask the other, so problems slip through. ## Core Features & Use Cases - Two-axis review: Runs a Standards review (repo conventions plus a Fowler code-smell baseline) and a Spec review (requirements coverage, scope creep, wrong implementations) as parallel sub-agents, then reports them side by side. - Fixed-point diffing: Compares HEAD against any commit, branch, tag, or merge-base the user supplies, validating the ref and diff before spawning reviewers. - Spec discovery: Locates the originating spec from issue references in commit messages, a user-provided path, or spec files under docs/, specs/, or .scratch/. - 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. ## 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 branch against coding standards and its spec?

Provide a fixed point such as a commit SHA, branch, tag, or main, and the skill diffs HEAD against it using a three-dot merge-base comparison. It then runs a Standards review and a Spec review as parallel sub-agents and reports both side by side.

What coding standards does an automated code review check?

It checks any standards documented in the repo, such as CODING_STANDARDS.md or CONTRIBUTING.md, plus a built-in baseline of Fowler code smells like Duplicated Code, Feature Envy, and Shotgun Surgery. Documented repo standards always override the baseline.

Can code review run without a spec or issue?

Yes. The skill looks for the spec via issue references in commit messages, a user-provided path, or files under docs/, specs/, or .scratch/. If none exists, the Spec sub-agent is skipped and the report notes that no spec was available.

Why does the review fail before starting the sub-agents?

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 that step rather than inside the parallel sub-agents.

What are the limitations of diff-based automated code review?

It only evaluates what changed between the fixed point and HEAD, so pre-existing issues outside the diff are not flagged. Baseline smell findings are judgement calls rather than hard violations, and anything tooling already enforces is skipped.