code-review

Reviews a git diff against coding standards and the originating spec using parallel isolated agents.

Updated Aug 27, 2026
One-click install
npx skills add https://github.com/Fatih0234/mattpocock-skills-pi --skill code-review-fatih0234
Or copy as Structured Prompt for Agent▼
Please help me install this Agent Skill.
Skill: code-review
Source: https://github.com/Fatih0234/mattpocock-skills-pi/tree/main/skills/engineering/code-review
Command: npx skills add https://github.com/Fatih0234/mattpocock-skills-pi --skill code-review-fatih0234

SYSTEM DOCUMENTATION & REQUIREMENTS

What problem does it solve? Reviewing a branch or PR requires checking two separate things—whether the code follows the repo's documented standards and whether it actually implements what the issue or spec asked for—and doing both in one pass lets one concern mask the other. ## Core Features & Use Cases - Two-axis review: Evaluates the diff since a fixed point (commit, branch, tag, or merge-base) separately for Standards conformance and Spec fidelity, reporting them side by side without merging or reranking findings. - Isolated parallel reviewers: Delegates each axis to a separate Pi subagent with a fresh context and read-only tools, writing results to dedicated artifact files under .scratch/pi-agents/. - Fowler smell baseline: Applies a fixed set of code smells from Refactoring (Duplicated Code, Feature Envy, Shotgun Surgery, and more) even when the repo documents no standards, with repo rules always taking precedence. - Use Case: Before merging a feature branch, ask for a review since main and receive a Standards report and a Spec report, each with its own findings and worst issue, so a spec-compliant but sloppy change (or vice versa) is never hidden. ## Quick Start Ask the agent to review the changes since main on the current branch 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, or tag, and the skill diffs it against HEAD using a three-dot merge-base comparison. It then runs a Standards review and a Spec review as isolated parallel agents and reports both sets of findings separately.

How to review code changes since a specific commit in git?▼

Pass the commit reference as the fixed point, for example main, HEAD~5, or a tag. The skill verifies the ref resolves and the diff is non-empty before delegating the review, so bad refs fail early.

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, it skips the Spec axis, runs only the Standards review, and reports "no spec available".

Does the code review modify my files or git state?▼

No. Reviewer agents receive inspection-only tools and cannot mutate project files, git state, or tracker state. The skill compares git status before and after delegation and stops if any unexpected change appears.

What code smells does the standards review check for?▼

It applies a baseline of Fowler smells from Refactoring chapter 3, including Duplicated Code, Feature Envy, Data Clumps, Shotgun Surgery, and Primitive Obsession. Documented repo standards always override the baseline, and each smell is a labelled heuristic rather than a hard violation.