requesting-code-review

Automates pre-commit verification of staged Git code with security scans, tests, linting, and JSON reviewer verdicts.

Updated Mar 25, 2026
One-click install
npx skills add https://github.com/afel6/scal-ai-pipeline --skill requesting-code-review-afel6
Or copy as Structured Prompt for Agent
Please help me install this Agent Skill.
Skill: requesting-code-review
Source: https://github.com/afel6/scal-ai-pipeline/tree/main/hermes_skills_library/software-development/requesting-code-review
Command: npx skills add https://github.com/afel6/scal-ai-pipeline --skill requesting-code-review-afel6

SYSTEM DOCUMENTATION & REQUIREMENTS

What problem does it solve?

This Skill enforces a pre-commit verification pipeline that catches security issues, regressions, and quality regressions before code is committed or pushed, reducing broken builds and accidental leaks.

Core Features & Use Cases

  • Staged-diff scanning: Scans added lines for hardcoded secrets, shell injection, unsafe eval/exec, unsafe deserialization, and SQL injection patterns.
  • Baseline-aware tests and linting: Captures pre-change baseline failures then compares results to detect regressions introduced by the current edits.
  • Independent reviewer & auto-fix loop: Delegates an isolated reviewer subagent that returns a JSON verdict, and optionally runs up to two fix-and-reverify cycles with a separate fix agent.
  • Use Case: Use this pipeline after implementing a feature spanning multiple files to ensure no new security or logic regressions reach the commit history.

Quick Start

Run the pre-commit verification pipeline on your staged changes to scan for security issues, run baseline-aware tests and linters, obtain an independent JSON reviewer verdict, and auto-fix reported issues before committing.

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 staged git changes before committing?

To run a pre-commit security scan on staged git changes, this pipeline automatically checks added lines for hardcoded secrets, shell injection, unsafe eval/exec, unsafe deserialization, and SQL injection patterns before a commit is allowed.

Can I automatically fix linting and test regressions before a git commit?

Yes, you can automatically fix regressions before a git commit using an independent reviewer subagent that returns a JSON verdict and optionally runs up to two fix-and-reverify cycles with a separate fix agent.

What is baseline-aware testing and how does it prevent code regressions?

Baseline-aware testing prevents code regressions by capturing pre-change test and linting failures, then comparing post-change results to detect only the new regressions introduced by your current staged edits.

Does this pre-commit verification pipeline work for multi-file feature edits?

Yes, this pre-commit verification pipeline works for multi-file feature edits in git repositories, applying staged-diff scanning and baseline-aware checks to gate commits before pushing or opening a pull request.

What are the limitations of running an automated pre-commit code review?

A limitation of this automated pre-commit code review is that the auto-fix loop is capped at two fix-and-reverify cycles, meaning complex or persistent logic and security issues may require manual intervention before committing.

Why does my pre-commit hook allow bypassing security checks for staged code?

Your pre-commit hook may bypass security checks if not configured as a comprehensive pipeline; this approach enforces staged-diff scanning and an independent JSON reviewer verdict to strictly gate commits against security vulnerabilities.