requesting-code-review

Verify staged git diffs with security scans, tests, and fail-closed review.

Updated Apr 9, 2026
One-click install
npx skills add https://github.com/MarbleSodas/Mavis --skill requesting-code-review-marblesodas
Or copy as Structured Prompt for Agent
Please help me install this Agent Skill.
Skill: requesting-code-review
Source: https://github.com/MarbleSodas/Mavis/tree/main/skills/software-development/requesting-code-review
Command: npx skills add https://github.com/MarbleSodas/Mavis --skill requesting-code-review-marblesodas

SYSTEM DOCUMENTATION & REQUIREMENTS

What problem does it solve?

It helps prevent insecure, low-quality, or logically broken code from being committed by running a pre-commit verification pipeline and an independent review before changes land.

Core Features & Use Cases

  • Security scanning on the added diff: Detects common red flags like hardcoded secrets, shell injection patterns, eval/exec usage, unsafe deserialization, and SQL injection risks.
  • Baseline-aware tests and linting: Runs relevant test and lint/type-check tools when available, blocking only new regressions introduced by the change.
  • Fail-closed independent reviewer: Delegates the diff to a separate reviewer subagent that returns a strict JSON verdict to determine pass/fail.
  • Two-cycle auto-fix loop: If issues are found, a separate fix agent patches only the reported items, then re-runs the verification pipeline.
  • Commit with verification marker: Commits only after passing verification, using a “[verified]” prefix.

Quick Start

After you finish editing your code and before you commit, ask your agent to run the requesting-code-review skill on your staged git diff to verify and auto-fix issues if needed.

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 pre-commit code review and security scanning on my staged git diff?

Pre-commit code review on staged git diffs combines static security scanning for hardcoded secrets and injection risks with baseline-aware lint checks. An independent reviewer evaluates the changes and returns a strict pass or fail verdict before committing.

How does baseline-aware test gating work for pre-commit verification?

Baseline-aware test gating runs relevant test suites and blocks commits only for new regressions introduced by your staged changes. It counts pre-existing baseline failures separately, ensuring your diff is evaluated solely on the new issues it creates.

Can I automatically fix lint and security issues found in my staged changes?

Yes, a bounded auto-fix loop patches reported security and lint issues by delegating to a separate fix agent. After patching, the verification pipeline re-runs static scans, tests, and lint checks to confirm the fixes resolved the original problems.

What is a fail-closed independent reviewer for git diff verification?

A fail-closed independent reviewer is a separate subagent that analyzes your staged git diff and outputs a strict JSON verdict. If the verdict indicates issues or the format is invalid, the pipeline blocks the commit to prevent unverified code from landing.

When should I use automated pre-commit verification in my git workflow?

Automated pre-commit verification is ideal for git-based workflows after feature or bugfix tasks involving multiple file edits. It is especially useful following subagent-driven development steps to ensure code quality, security, and logic are validated before changes land.

Does pre-commit security scanning detect SQL injection and unsafe deserialization?

Pre-commit security scanning detects common red flags like hardcoded secrets, shell injection patterns, eval/exec usage, unsafe deserialization, and SQL injection risks. It specifically targets the added diff to prevent these vulnerabilities from being committed.