requesting-code-review

Review completed code changes for correctness, quality, and merge readiness.

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

SYSTEM DOCUMENTATION & REQUIREMENTS

What problem does it solve?

This Skill helps you catch defects, requirement mismatches, and quality gaps before code is merged or handed off, reducing rework and late-stage bugs.

Core Features & Use Cases

  • Pre-merge validation: Checks completed work against requirements before integrating to main.
  • Reviewer delegation: Encourages dispatching a fresh reviewer perspective to uncover issues you may miss.
  • Use Cases: Ideal after major features, bug fixes, refactors, or any task where correctness, tests, security, and maintainability matter.

Quick Start

Ask the assistant to review the completed change for correctness, edge cases, test coverage, and code quality before you merge it.

Frequently Asked Questions about requesting-code-review

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

FAQPage Schema
How do I review code changes for correctness before merging a pull request?

A pre-merge code review validates completed changes against requirements by performing systematic self-review, diff inspection, and test verification. It applies to major features, bug fixes, and refactors to ensure correctness and readiness before integrating to main.

What is the best way to validate a subagent-driven development handoff?

Validating subagent-driven development handoffs requires a fresh reviewer to perform systematic self-review, diff inspection, and test verification. This fresh perspective catches defects and quality gaps the original developer missed before the code is merged.

How do I check completed code for security, performance, and edge cases?

Checking completed code for security, performance, and edge cases requires explicit assessment during a systematic self-review. You inspect the diff for vulnerabilities, verify test coverage for boundary conditions, and validate efficiency before merging to main.

Does this pre-merge validation work for both major features and minor bug fixes?

Pre-merge validation works for major features, bug fixes, and refactors. It applies to any completed code change where correctness, test coverage, security, and maintainability matter, catching defects and requirement mismatches before integration.

Why should I dispatch a fresh reviewer perspective instead of doing a self-review?

Dispatching a fresh reviewer perspective uncovers defects and quality gaps you may miss during self-review. A new set of eyes systematically inspects the diff and verifies tests without prior bias, reducing late-stage bugs before integration.