requesting-code-review

Scan code diffs for security issues and run baseline-aware tests before commit.

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

SYSTEM DOCUMENTATION & REQUIREMENTS

What problem does it solve?

It prevents insecure or broken code from being merged by running a security-focused diff scan, executing baseline-aware tests and linting, and enforcing an independent reviewer verdict before a commit.

Core Features & Use Cases

  • Diff-based security scan: Flags added lines that may contain secrets, shell injection, eval/exec patterns, unsafe deserialization, or common SQL injection strings.
  • Baseline-aware quality gates: Captures pre-existing failures as a baseline so only regressions introduced by your changes block the commit.
  • Independent subagent review + fail-closed: Delegates the diff to a separate reviewer that must return valid JSON, blocking on unparseable output.
  • Optional auto-fix loop: Up to two fix-and-reverify cycles to address only the reported issues.

Real-world use case: After you implement a feature and edit multiple files, run this pipeline to confirm your changes don’t introduce security risks or new failing tests before you push.

Quick Start

Ask the agent to verify your staged changes before committing and include an auto-fix loop if any security or logic issues are found.

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 security scan on my git diff before committing?

A diff-based security scan checks your staged changes for secrets, shell injection, eval/exec patterns, unsafe deserialization, and SQL injection before you commit. This ensures insecure code is flagged and blocked before merging.

How does baseline-aware testing work for new code changes?

Baseline-aware testing captures pre-existing test and linting failures before running your checks, so only regressions introduced by your new changes block the commit rather than old issues.

Can I automatically fix code issues found during a pre-commit review?

Yes, an optional auto-fix loop runs up to two fix-and-reverify cycles to address only the reported issues. This iterates on your code until all security and quality gates pass before committing.

What is a fail-closed independent reviewer verdict in code review?

A fail-closed independent reviewer verdict delegates your diff to a separate subagent that must return valid JSON. If the output is unparseable or issues are found, the commit is blocked automatically.

Does this code review workflow work with subagent-driven development?

Yes, this workflow applies to local feature development and subagent-driven workflows where you intend to commit, push, or ship after multiple file edits have been completed.

What types of security vulnerabilities are detected in a diff scan?

The diff scan detects secret leakage, shell injection, eval/exec patterns, unsafe deserialization, and common SQL injection strings within the added lines of your code changes.