approve
SolidThe approve (approve|deny) decision gate of the canonical workflow — read a spec's critique findings and decide go/no-go BEFORE any GitHub-side state exists, then emit a single APPROVED/DENIED verdict. Auto-gates on critic SEVERITY (any unmitigated high → deny), with an optional human override for borderline findings. This is the decision half of the critique→approve pair; the critics that produce the findings are the separate `/critique` node. TRIGGER when: a plan/PRD has been critiqued and needs a commitment decision before implementation; "approve this spec", "should we build <slug>", "gate <slug> on its critique".
Install
Quality Score: 83/100
Skill Content
Details
- Author
- mifunedev
- Repository
- mifunedev/openharness
- Created
- 5 months ago
- Last Updated
- today
- Language
- TypeScript
- License
- Apache-2.0
Similar Skills
Semantically similar based on skill content — not just same category
critique
The critique node of the canonical workflow — run two adversarial critics in parallel (implementer lens + user lens) against a freshly-planned spec, write their findings to .oh/tasks/<slug>/critique.md, and hand off to the approve gate. This is the EVIDENCE half of the critique→approve pair; it produces findings, it does not decide (the /approve gate decides). Runs on local artifacts only, before any GitHub-side state. TRIGGER when: a PRD/plan exists and needs adversarial review before the commitment gate; "critique this spec", "run the critics on <slug>", "review prd before we build".
approve
Human approval gate — seal an atomic run as approved, unlocking the git commit gate. Only the user can invoke this.
rootnode-critic-gate
Independent re-derivation gate for proposed changes during autonomous execution. Evaluates a change (code diff, config edit, engine evolution, schema change) against the work's authority matrix and a 4-check protocol: invariant compliance, scope authorization, detection narrowness, and regression risk. Returns structured JSON with pass/fail per check, an overall verdict (APPROVE / REQUEST_CHANGES / REJECT), and blockers. Profile-driven thresholds (strict for unattended runs, lenient for desk supervision). Use when an autonomous agent proposes an engine change, when reviewing a Claude Code-authored modification before merge, or when an unattended profile requires independent re-derivation. Trigger on: "review/approve this proposed change," "critic-gate this," "is this change safe," "is this safe to merge," "should I let this land," "run the critic." Do NOT use for handoff-readiness checks (use rootnode-handoff-trigger-check), general code review, or design-time correctness. Authority matrix required.