code-review

Compare HEAD against a fixed point to review code standards and spec alignment.

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

SYSTEM DOCUMENTATION & REQUIREMENTS

What problem does it solve?

This skill solves the problem of inconsistent code quality and scope creep by providing a structured, two-axis review process that evaluates code against both technical standards and functional requirements.

Core Features & Use Cases

  • Standards Review: Automatically checks code against documented repository standards and a baseline of common code smells (Fowler's Refactoring).
  • Spec Alignment: Verifies that the implementation matches the originating issue, PRD, or spec, identifying missing requirements or scope creep.
  • Use Case: Use this when you need to review a complex pull request or a set of work-in-progress changes to ensure the code is both clean and functionally correct before merging.

Quick Start

Ask the agent to review the changes since the main branch by providing the branch name or commit hash as the fixed point.

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 pull request for both coding standards and spec alignment?

To perform a dual-axis code review, compare the current HEAD against a fixed branch name or commit hash. This evaluates technical quality and requirement fulfillment independently to ensure adherence to coding standards and functional specifications.

What is dual-axis code review and how does it handle functional specifications?

Dual-axis code review is a mechanism that spawns parallel sub-agents to independently analyze technical quality and spec alignment. It verifies implementation against originating issues, PRDs, or specs to identify missing requirements and scope creep.

Does this automated code review require git access to check for code smells?

Yes, this code review process requires git access to resolve references and compare code changes. It checks for code smells by analyzing diffs against documented repository standards and Fowler's Refactoring principles.

What is the best way to check work-in-progress changes against a main branch?

The best way to check work-in-progress changes is to provide the main branch name as the fixed point for comparison. The process analyzes the diff to ensure the code is clean and functionally correct before merging.

When do I need to evaluate diffs for scope creep and missing requirements?

You need to evaluate diffs for scope creep and missing requirements when reviewing complex pull requests or work-in-progress changes before merging. This ensures the implementation matches the originating issue, PRD, or functional spec.

Can I use static analysis to verify implementation against a PRD?

Yes, static analysis is used to verify implementation against a PRD by spawning parallel sub-agents. One axis checks technical quality against coding standards while the other verifies functional requirement fulfillment independently.