← ClaudeAtlas

pre-execution-reviewlisted

Internal owner of the pre-execution review cycle and the planning ledgers: independence, unioned findings, counter-evidence dismissal, no-progress, `CONVERGENCE-ANOMALY`, and the evidence/obligation/findings tables. Consumed by `review-spec`, `review-plan`, and the authoring skills. Not a menu entry.
gtrabanco/agentic-workflow · ★ 21 · Code & Development · score 80
Install: claude install-skill gtrabanco/agentic-workflow
# Pre-Execution Review Policy (internal) One owner for the rules that both pre-execution reviewers apply, so a Product review and a Plan review cannot drift into two different definitions of independence, union, or convergence. It states **policy**; it runs nothing, writes nothing, and emits no verdict of its own. ```text evidence-grounding = how an author prepares and self-checks an artifact. pre-execution-review = how any pre-execution reviewer judges it, and what the frozen ledgers look like. review-spec / review-plan = the only skills that emit a pre-execution verdict. ``` ## When to use - `review-spec` / `review-plan` — before the first check and again before any repeat, repair, or synthesis step. - `design-feature`, `plan-feature`, `plan-feature-scaffold`, `plan-fix` — when writing the ledgers and when a review comes back failed. - Nothing else. Candidate-source review (`review-change`), merge gating (`audit-pr`) and execution (`execute-phase`) keep their own contracts; this skill adds no authority over them. ## Hard rule — no verdicts, no edits This skill never prints `SPEC-REVIEW-PASS`, `PLAN-REVIEW-PASS`, `SPEC-REVIEW-FAIL`, `PLAN-REVIEW-FAIL` or `NEEDS-DESIGN` as a result of its own reading, and it never writes a unit artifact. Quoting a verdict shape here is a definition, not an issuance. A policy summary that reads like an approval is a contract violation — the verdict belongs to the reviewer turn that binds the snapshot. ##