review

Reviews branch changes, documentation, tests, and build checks against repository standards.

6|2|Updated May 6, 2026
One-click install
npx skills add https://github.com/sek788432/Stock-Back-Test-System --skill review-sek788432
Or copy as Structured Prompt for Agent▼
Please help me install this Agent Skill.
Skill: review
Source: https://github.com/sek788432/Stock-Back-Test-System/tree/main/.agents/skills/review
Command: npx skills add https://github.com/sek788432/Stock-Back-Test-System --skill review-sek788432

SYSTEM DOCUMENTATION & REQUIREMENTS

What problem does it solve? It provides a rigorous, evidence-based review of a branch's complete change set—including committed, staged, unstaged, and untracked work—so defects, documentation drift, and missing test coverage are caught before a commit, pull request, merge, or handoff. ## Core Features & Use Cases - Full-range change review: Pins a fixed merge-base and reviews the union of committed diffs plus all uncommitted and untracked files, building an impact closure of callers, build wiring, and related specs. - Three independent review axes: Evaluates code correctness against five repository C++ skills, verifies documentation-to-implementation alignment, and maps every affected behavior to positive, negative, boundary, and regression tests. - Verified check execution: Runs the checked-in CMake/CTest presets, sanitizer builds, and the project standards checker, recording each command as Passed, Failed, Blocked, Not available, or Not applicable. - Use Case: Before opening a pull request on a C++ feature branch, run this Skill to receive a severity-ordered findings list (P0–P3), a behavior-to-test coverage matrix, and a final Pass, Needs changes, or Blocked verdict. ## Quick Start Use $review to review this branch and all uncommitted changes before I open a pull request.

Frequently Asked Questions about review

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

FAQPage Schema
How do I review all uncommitted and committed branch changes before a pull request?▼

Invoke the review Skill, which pins a merge-base against your fixed point and reviews the union of committed diffs, staged, unstaged, and untracked files. It produces severity-ordered findings and a final verdict without modifying any code.

What does a comprehensive code review checklist include for C++ projects?▼

This review covers functional correctness, modern C++20 style, RAII, ownership, thread safety, error handling, and performance rules from five repository C++ skills. It also checks documentation alignment, test intent, and build/test registration.

Does the review run tests and sanitizers automatically?▼

Yes, it runs the checked-in CMake configure, build, and CTest presets plus the sanitizer preset when applicable. Each command is recorded as Passed, Failed, Blocked, Not available, or Not applicable, and it never claims success for suites that did not run.

Can the review fix the issues it finds?▼

No, the review is strictly read-only and never modifies code, documentation, tests, configuration, or git state. Fixes are only made when the user separately requests them after reviewing the findings report.

What happens if the review cannot resolve a fixed point or merge-base?▼

The review stops and asks for a fixed point rather than guessing. It tries the user-supplied ref, the branch upstream, then origin/main or local main, and reports the problem if none resolves.