adversarial review gate — a dedicated, read-only skeptic dispatched after an impl wave, before merge-to-main / bead-close, whose only job is to disprove "done." Hunts stub bodies, mock-the-unit tests, unverified acceptance criteria, unreproducible runtime claims, composition gaps. Distinct from /check (which interrogates the plan, pre-impl).

Audit post-implementation code for false completion claims with file:line evidence.

8|1|Updated Jan 21, 2024
One-click install
npx skills add https://github.com/azigler/dotfiles --skill adversarial-review-gate-a-dedicated-read-only-skeptic-dispatched-after-an-impl-wave-before-merge-to-main-bead-close-whose-only-job-is-to-disprove-done-hunts-stub-bodies-mock-the-unit-tests-unverified-acceptance-criteria-unreproducible-runtime-claims-composition-gaps-distinct-from-check-which-interrogates-the-plan-pre-impl
Or copy as Structured Prompt for Agent
Please help me install this Agent Skill.
Skill: adversarial review gate — a dedicated, read-only skeptic dispatched after an impl wave, before merge-to-main / bead-close, whose only job is to disprove "done." Hunts stub bodies, mock-the-unit tests, unverified acceptance criteria, unreproducible runtime claims, composition gaps. Distinct from /check (which interrogates the plan, pre-impl).
Source: https://github.com/azigler/dotfiles/tree/main/agents/skills/scrutinize
Command: npx skills add https://github.com/azigler/dotfiles --skill adversarial-review-gate-a-dedicated-read-only-skeptic-dispatched-after-an-impl-wave-before-merge-to-main-bead-close-whose-only-job-is-to-disprove-done-hunts-stub-bodies-mock-the-unit-tests-unverified-acceptance-criteria-unreproducible-runtime-claims-composition-gaps-distinct-from-check-which-interrogates-the-plan-pre-impl

SYSTEM DOCUMENTATION & REQUIREMENTS

What problem does it solve?

It prevents teams from shipping or closing an implementation wave when the delivered code, tests, or acceptance evidence are misleading, incomplete, or unverifiable.

Core Features & Use Cases

  • Adversarial post-implementation audit: reviews delivered code to disprove the claim that the work is complete.
  • Evidence-first verdicts: requires findings with file:line locations and a clear SHIP / FIX-FIRST / REJECT outcome.
  • Targets common quality failures: hunts stub bodies, mocked tests that don’t test real code, missing error paths, and composition gaps in user-facing surfaces.

Quick Start

Dispatch the /scrutinize gate right after an impl wave with real implementation, then require the reviewer’s verdict and file:line findings to be appended to the impl bead notes before merging.

Frequently Asked Questions about adversarial review gate — a dedicated, read-only skeptic dispatched after an impl wave, before merge-to-main / bead-close, whose only job is to disprove "done." Hunts stub bodies, mock-the-unit tests, unverified acceptance criteria, unreproducible runtime claims, composition gaps. Distinct from /check (which interrogates the plan, pre-impl).

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

FAQPage Schema
How do I prevent merging incomplete code with stub bodies before closing an implementation wave?

Dispatch a read-only adversarial review gate post-implementation to disprove completion claims. It hunts stub bodies, mocked tests, and composition gaps, routing verdicts to SHIP, FIX-FIRST, or REJECT before merging to main.

What is evidence-based pre-merge validation for test integrity?

Evidence-based pre-merge validation is an adversarial review process that requires file:line locations to support findings. It targets unverified acceptance criteria and unreproducible runtime claims, ensuring test reports accurately reflect implementation scope.

How do I run a post-implementation audit to catch mocked tests that don't test real code?

Run an adversarial review gate after your implementation wave to scrutinize test integrity. The read-only skeptic identifies mocked unit tests and missing error paths, appending file:line findings and a verdict to impl bead notes before merge.

Does an adversarial quality gate write code or modify files during pre-merge validation?

No, an adversarial quality gate requires read-only execution. It performs code scrutiny to disprove false completion without modifying files, routing a SHIP, FIX-FIRST, or REJECT verdict based strictly on evidence-based findings.

When do I need an adversarial review gate instead of a standard plan interrogation check?

You need an adversarial review gate after implementation, right before merge-to-main. It is distinct from pre-implementation checks, specifically targeting composition gaps and unreproducible runtime claims in delivered code rather than interrogating the plan.

What are the limitations of using an adversarial review gate for code scrutiny?

The gate cannot invent issues and only validates post-implementation, pre-merge surfaces. It requires real implementation to scrutinize and relies on file:line evidence, meaning it cannot catch quality failures in code paths lacking explicit acceptance criteria.