/silver:review-triad

Catalog-backed reusable workflow component (WF-REVIEW-TRIAD) for structured code and artifact review — request scope, run review, triage findings. Composed inside delivery workflows immediately after EXECUTE.

Overview

/silver:review-triad documents WF-REVIEW-TRIAD, a reusable workflow component in the v0.48.3 APO catalog. It encapsulates the three-step review chain that every delivery workflow runs after implementation completes.

Catalog slug: review-triad. Type: reusable_component. Allowed dynamic operations: loop (re-run triage until findings are resolved or explicitly accepted).

REVIEW_REQUEST → REVIEW → REVIEW_TRIAGE

Individual atomic flows are invokable directly via their public routes (/silver:review-request, /silver:review, /silver:review-triage), but the triad only runs as an ordered composition inside parent workflows.

Reusable component: Unlike top-level composer workflows such as /silver:feature, this chain is composed automatically inside delivery workflows when post-execution review begins. Individual atomic flows remain invokable directly via their own routes.

Catalog Composition

Atomic flowSlugCommandWork product
AF-REVIEW-REQUESTREVIEW_REQUEST/silver:review-requestReview request package with scope and evidence pointers
AF-REVIEWREVIEW/silver:reviewStructured artifact and code review findings
AF-REVIEW-TRIAGEREVIEW_TRIAGE/silver:review-triageTriaged findings: MUST-FIX / NICE-TO-HAVE / DISMISS with fix assignments

Where it is composed

WF-REVIEW-TRIAD expands inside delivery workflows after EXECUTE (and optionally after UI_QUALITY for UI work):

EXECUTE → [UI_QUALITY if UI] → REVIEW_REQUEST → REVIEW → REVIEW_TRIAGE → VERIFY (verify-tests) → SECURE (VALIDATE as needed) → …

Parent workflows that include this component:

The larger post-execution block (WF-POST-EXEC-GATES) wraps review triad with verification, security, validation, and ship readiness.

Atomic Flow Execution

AF-REVIEW-REQUEST — Package review scope

Invoke /silver:review-request. Define what is under review: changed files, artifacts, acceptance criteria coverage, and evidence pointers. Produces ART-REVIEW_REQUEST.

AF-REVIEW — Run structured review

Invoke /silver:review. Execute artifact reviewers (SPEC, REQUIREMENTS, UAT, cross-artifact) and code review against the request package. Findings are evidence-backed, not impressionistic.

AF-REVIEW-TRIAGE — Triage and assign fixes

Invoke /silver:review-triage. Classify each finding as MUST-FIX, NICE-TO-HAVE, or DISMISS. MUST-FIX items loop back through EXECUTE or targeted fix skills before the parent workflow advances to VERIFY.

V-loop behavior

Each atomic flow in the triad owns a local V-loop that must pass before the parent flow gate advances. AF-REVIEW-TRIAGE may loop until all MUST-FIX items are resolved or explicitly accepted with rationale.

Flow Log rows in .planning/workflows/<id>.md record each triad pass. The orchestrator enforcement queue in hooks/lib/orchestrator-state.sh tracks runtime proof that review steps executed.