review-fix-signoff-loop
Installation
SKILL.md
Review Fix Signoff Loop
Purpose
Use this pattern for high-stakes implementation workflows where a normal "implement, test, review once" flow is not enough. The workflow must keep repairing and re-reviewing until independent signoff agents agree the spec is fully wired end to end.
Pair this with writing-agent-relay-workflows for SDK syntax and relay-80-100-workflow for deterministic validation gates.
Required Shape
- Run deterministic preflight before agents start.
- Confirm repository root, required specs, declared write scope, credentials needed for PR comments, and whether commit/push/PR creation is in scope.
- For cross-repo or package-release work, write a scope matrix before implementation: repositories, branches, PRs, packages, providers/features touched, published versions, consuming package manifests, lockfiles, and expected downstream bumps.
- Probe the CLIs used by later agent steps. For Codex,
codex login statusis not enough; run a tinycodex exec --ephemeral --json --sandbox read-only -m <supported-model>prompt and fail early with a clear re-login instruction if it cannot return the expected token. - Write preflight evidence to
.workflow-artifacts/<workflow>/iteration-N/preflight.md.