approve-plan

Solid

Atomically lock IMPLEMENTATION_PLAN.md as the approved execution baseline and stamp TASK_STATE.md with the approval signal. Symmetric counterpart to approve-proposed but for the plan itself, not for arbitrary PROPOSED artifacts. Use when IMPLEMENTATION_PLAN.md has been produced (or revised via self-critique-and-revise) and the user authorizes execution to begin. Do not use when the plan is still draft, when implementation-plan returned NO_OP, when the plan contains [NEEDS CLARIFICATION:] markers, or when revisions are still pending review.

AI & Automation 6 stars 0 forks Updated 5 days ago MIT

Install

View on GitHub

Quality Score: 81/100

Stars 20%
28
Recency 20%
100
Frontmatter 20%
70
Documentation 15%
100
Issue Health 10%
80
License 10%
100
Description 5%
100

Skill Content

Act as a senior/staff engineer atomically locking the current `IMPLEMENTATION_PLAN.md` as the approved execution baseline for the active task. Goal: Verify the plan is ready to be locked, append an explicit `## Approval log` entry to `IMPLEMENTATION_PLAN.md`, update `TASK_STATE.md` to reflect the approval, and emit a Handoff to the correct execution command for the first wave (waves-aware per ADR-0042). Closes the gap that previously left plan approval implicit (the user signaled approval by invoking `implement-approved-slice` directly, which left no audit trail of when the plan crossed from "draft" to "locked"). **Waves-aware routing rule (ADR-0042, stated verbatim wherever execution is routed):** when the approved plan's `## Execution waves` section shows a remaining wave of size 2 or more whose slices declare `Scope` and `Depends-on`, route to `implement-fleet`; otherwise route to `implement-approved-slice`. This puts the parallel-execution offer at the moment the plan is locked, instead of leaving the operator to discover it. **Test-strategy gate (before the waves-aware rule):** WHEN no `TEST_STRATEGY.md` exists for the task AND the plan's changes affect important behavior, contracts, data flow, or regression risk (the `test-strategy` command's own use-when test), route to `test-strategy` first and name the execution command that follows it per the waves-aware rule. Approval and a test-strategy pass are not in tension: locking the plan and deciding the protecting tests...

Details

Author
Mozurok
Repository
Mozurok/fhorja.dev
Created
1 months ago
Last Updated
5 days ago
Language
Python
License
MIT

Similar Skills

Semantically similar based on skill content — not just same category