requesting-code-review

Scan staged git diffs, run baseline tests, and delegate JSON-only review.

Updated May 4, 2026
One-click install
npx skills add https://github.com/JamesFincher/gengar --skill requesting-code-review-jamesfincher
Or copy as Structured Prompt for Agent
Please help me install this Agent Skill.
Skill: requesting-code-review
Source: https://github.com/JamesFincher/gengar/tree/main/skills/software-development/requesting-code-review
Command: npx skills add https://github.com/JamesFincher/gengar --skill requesting-code-review-jamesfincher

SYSTEM DOCUMENTATION & REQUIREMENTS

What problem does it solve?

This Skill prevents broken or insecure code from being committed by running a pre-commit verification pipeline and an independent review before your changes land.

Core Features & Use Cases

  • Diff-first workflow: Pulls your staged (cached) changes and handles empty diffs and large diffs by file splitting.
  • Security scanning with quality gates: Runs targeted static checks on added lines for common issues (secrets, injection vectors, dangerous calls).
  • Baseline-aware testing and linting: Detects the project test/lint stack, captures baseline failures, and blocks only regressions introduced by your changes.
  • Independent reviewer subagent + fail-closed: Delegates the reviewer with only the diff and scan results, requiring JSON and auto-failing on unparseable output.

Quick Start

Ask the agent to verify your staged changes with a pre-commit security scan, baseline tests/lint, and an independent reviewer verdict before you run git commit.

Frequently Asked Questions about requesting-code-review

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

FAQPage Schema
How do I run a pre-commit security scan on staged git changes?

To run a pre-commit security scan on staged git changes, you can trigger an agent to verify your cached diff, executing targeted static checks on added lines to block secrets and injection vectors before committing.

How does baseline-aware testing prevent regressions in my git diff?

Baseline-aware testing prevents regressions in your git diff by capturing existing project test and lint failures first, then blocking only the new test failures or lint errors introduced by your staged changes.

What is the best way to auto-fail code review on unparseable output?

The best way to auto-fail code review on unparseable output is using a fail-closed independent reviewer subagent that requires strict JSON output, automatically failing the review if the response cannot be parsed.

How do I handle large diffs during a pre-commit code review?

To handle large diffs during a pre-commit code review, the verification pipeline splits the staged changes by file, ensuring the independent reviewer and security scans process manageable chunks without missing added lines.

Can I detect project test and lint tools automatically before pushing?

Yes, you can detect project test and lint tools automatically before pushing, as the pre-commit workflow identifies your project's existing test stack and lint configurations to run baseline-aware checks against your staged changes.

Does the independent reviewer subagent see the entire repository or just the diff?

The independent reviewer subagent sees just the diff and scan results, receiving only your newly staged changes to provide an isolated security and logic check without full repository context.