src/orchestrator/skills/fast-review/SKILL.md
Mandatory post-delegation gate: checks output completeness, verifies acceptance criteria compliance, flags regressions, produces PASS/FAIL verdict. Use when checking delegated work against acceptance criteria, running post-delegation gate, validating agent output before acceptance, verifying sub-agent completed its assignment, or running post-delegation QA check.
npx skillsauth add monkilabs/opencastle fast-reviewInstall this skill globally with one command. Works with Claude Code, Cursor, and Windsurf.
3 of 9 scanners reported clean
Some scanners were skipped, did not run, or reported a non-clean status. Review each row below.
| Rule | Detail | |------|--------| | Trigger | After every delegation — no exceptions | | Reviewer | Single sub-agent; Economy tier (Standard for premium/security work) | | Verdict | PASS or FAIL with structured feedback | | Retry | ≤2 retries on FAIL; 3rd FAIL → panel review |
Issue + acceptance criteria, file diff, file partition, deterministic results (lint/test/build), agent self-report.
Single runSubagent. Context = acceptance criteria, diff, partition, deterministic results only — no session history, no delegation prompt.
runSubagent({ agentName: 'Reviewer', prompt: `Review against ACs:\n${criteria}\nDiff:\n${diff}\nGates: lint ✅ test ✅ build ✅` });
VERDICT: PASS | FAIL
ISSUES:
- [severity:critical|major|minor] Description
FEEDBACK: Actionable feedback.
CONFIDENCE: low | medium | high
Auto-PASS (skip reviewer): pure research/no code changes; docs-only .md changes; ≤10 lines across ≤2 non-sensitive files with all deterministic gates passing.
Sensitive override: Auth/middleware, DB migrations, RLS policies, security headers, CSP, env var schemas, CI/CD config always require review — even 1-line changes.
| Outcome | Action |
|---------|--------|
| PASS | Log review; continue |
| FAIL 1–2 | Log; re-delegate same agent: "Retry N/2 — address listed issues" |
| FAIL 3 | Log escalated: true; load panel-majority-vote skill |
| Panel BLOCK ×3 | Dispute in .opencastle/DISPUTES.md (see team-lead-reference § Dispute Protocol) |
See REFERENCE.md for full reviewer prompt template.
⛔ HARD GATE — Log the review before proceeding.
npx opencastle log review --skill <name> --outcome pass|fail --reviewer "Reviewer" --mechanism sub-agent
on-post-delegate Gate 5 (after deterministic Gates 1–4), ~5–15% token overhead. Overnight: upgrade one tier, escalate after 2 FAILs, checkpoint before panel.
development
Defines 10 sequential validation gates: secret scanning, lint/test/build checks, blast radius analysis, dependency auditing, browser testing, cache management, regression checks, smoke tests. Use when running pre-deploy validation or CI checks, CI/CD pipelines, deployment pipeline validation, pre-merge checks, continuous integration, or pull request validation.
development
Generates test plans, writes unit/integration/E2E test files, identifies coverage gaps, flags common testing anti-patterns. Use when writing tests, creating test suites, planning test strategies, mocking dependencies, measuring code coverage, or test planning.
development
Provides model routing rules, validates delegation prerequisites, supplies cost tracking templates, defines dead-letter queue formats for Team Lead orchestration. Load when assigning tasks to agents, choosing model tiers, starting delegation session, running multi-agent workflow, delegating work, choosing which model to use, or assigning tasks.
testing
Saves, restores session state including task progress, file changes, delegation history. Use when saving progress, resuming interrupted work, picking up where you left off, or checkpointing current work.