requesting-code-review

Review git commit ranges for bugs, design flaws, and missing tests.

Updated Mar 3, 2026
One-click install
npx skills add https://github.com/jacexh/skills --skill requesting-code-review-jacexh
Or copy as Structured Prompt for Agent
Please help me install this Agent Skill.
Skill: requesting-code-review
Source: https://github.com/jacexh/skills/tree/main/skills/requesting-code-review
Command: npx skills add https://github.com/jacexh/skills --skill requesting-code-review-jacexh

SYSTEM DOCUMENTATION & REQUIREMENTS

What problem does it solve?

This Skill provides a structured process and reviewer template to catch bugs, design flaws, and missing tests before they propagate into mainline branches or production releases.

Core Features & Use Cases

  • Structured Review Template: A code-reviewer checklist that categorizes findings into Critical, Important, and Minor with guidance on fixes.
  • Git Range Focused: Reviews are scoped to a base and head SHA so reviewers only evaluate the intended changes.
  • Workflow Integration: Designed for subagent-driven development, per-task reviews, pre-merge checks, and ad-hoc feature validations.

Quick Start

Dispatch the code-reviewer subagent with BASE_SHA, HEAD_SHA, a brief DESCRIPTION of what was implemented, and the relevant PLAN_OR_REQUIREMENTS to request a focused review.

Frequently Asked Questions about requesting-code-review

High-intent search queries and answers about installing and using this skill.

FAQPage Schema
How do I automate code review for a specific git commit range?

Yes, pre-merge code review can validate implementations against plans by dispatching a reviewer subagent with the base and head SHAs along with the original plan or requirements. This ensures the code changes align with the intended design before reaching mainline branches.

What is the best way to catch design flaws and missing tests before merging?

The best way to catch design flaws and missing tests before merging is using a structured review checklist scoped to the exact git changes. This process evaluates code quality, architecture, testing coverage, and production readiness to prevent issues from propagating into production releases.

Can I use a subagent to validate feature implementations against requirements?

You can use a subagent to validate feature implementations by supplying the base SHA, head SHA, implementation description, and the original requirements. The subagent assesses the code changes against the plan to verify completeness and production readiness.

Does pre-merge code review work without a git base and head SHA?

Pre-merge code review requires a git base and head SHA to scope the evaluation to the intended changes. Without these commit identifiers, the reviewer cannot isolate the specific code diff to assess code quality, architecture, and testing coverage.

How do I structure code review feedback for production readiness?

To structure code review feedback for production readiness, categorize findings into Critical, Important, and Minor levels with specific guidance on fixes. This structured template ensures thorough validation of code quality, architecture, and testing coverage.