executing-plans · diff
git:20260917.2ea35d0 to git:20260917.8f29537
48 added, 16 removed. Audit A to A.
---
name: executing-plans
description: The checkpoint coordinator for /feature. Routed to by /feature once a writing-plans plan exists. Groups tasks into batches, delegates each batch to subagent-driven-development (fresh author agent per task, full review chain, fresh verification), then stops for a human checkpoint before the next batch. The checkpointed counterpart to /sprint's autonomous run.
disable-model-invocation: true
---
# executing-plans
- ## Typed-artifact pilot boundary
+ ## Structured-artifact format boundary
- For an existing HTML spec/plan or an explicitly requested typed-HTML pilot, load
- `${CLAUDE_PLUGIN_ROOT}/includes/artifacts.md` before artifact I/O. Its typed ID, binding,
- readiness, receipt, contextual-read and scope-state rules replace the legacy
- Markdown parsing and direct status-cell edits below for that pilot only. Keep
- all other workflow gates, including human checkpoints, unchanged. Missing or
- invalid HTML capability is a STOP for this path, not a fallback to Markdown.
- HTML `--farm` dispatch is blocked in this candidate. Default legacy workflows
- remain unchanged until native host qualification and cutover approval.
+ With a qualified normal package, new full feature and sprint workflows use HTML
+ by default. Load `${CLAUDE_PLUGIN_ROOT}/includes/artifacts.md` before HTML artifact I/O;
+ its typed identity, binding, receipt, contextual-read, and scope-state rules
+ replace rendered-document parsing and direct status-cell edits. Existing
+ authoritative Markdown pairs stay on their exact legacy path without conversion.
+ A missing, invalid, or unqualified HTML capability must fail closed rather than
+ fall back to new Markdown authority. HTML `--farm` dispatch remains disabled.
+ Keep every other workflow gate, including human checkpoints, unchanged.
Coordinate the plan in small, user-acknowledged batches. Routed to by `/feature` after
`writing-plans` has produced a plan. Each batch is executed by `subagent-driven-development` — fresh
author agent per task, spec-compliance review, quality review, fresh verification — and the user
checkpoints between batches. The orchestrator never implements; it schedules and checkpoints.
## Pre-flight
Read these, or STOP and surface the gap — never guess a path, a command, or a task boundary:
- - `${CLAUDE_PROJECT_DIR}/.codearbiter/plans/<slug>.md` — the approved plan. The ordered task list and each task's verification command are the contract. No plan, no execution.
- - `${CLAUDE_PROJECT_DIR}/.codearbiter/specs/<slug>.md` — the spec the plan implements.
+ - The discovered authoritative spec/plan pair. Existing Markdown pairs retain
+ their current ledger. For an HTML pair, load
+ `${CLAUDE_PLUGIN_ROOT}/includes/artifacts.md`, retain the exact artifact identities
+ and engine-written receipts, and obtain task, scope, dependency, and binding
+ state only through `identity`, `index`, `outline`, symbol-scoped `read`, and
+ `eligible`. The engine `eligible` response is authoritative; the coordinator
+ must not infer authority from rendered text, checkboxes, status words,
+ digests, or caller-supplied labels, and must not read or write a shadow
+ Markdown ledger.
- `${CLAUDE_PROJECT_DIR}/.codearbiter/CONTEXT.md` — the `stage:` frontmatter (the maturity value) and project context.
- `${CLAUDE_PROJECT_DIR}/.codearbiter/tech-stack.md` — the exact verification, test, and build invocations.
## Phase 1 — Batch plan · gate: BLOCK
Group the plan's tasks into small batches. Keep batches tight — three to five tasks is a ceiling, not
a target. Respect the plan's ordering and dependencies: a task never lands before the task it depends on.
- **Resume is the normal re-entry.** A task already `ACCEPTED` in the plan's status column was verified
- before an earlier interruption — exclude it from batching and say so ("resuming: T-01–T-03 already
- ACCEPTED"). Batch only the non-`ACCEPTED` tasks, starting at the first. Never re-execute an `ACCEPTED`
- task, and never restart the pipeline at brainstorming because the session died mid-plan.
+ **Resume is the normal re-entry.** For Markdown, a task already `ACCEPTED` in
+ the plan's status column was verified before an earlier interruption. For HTML,
+ derive the same decision from the current `eligible` result and its exact task
+ identities. Exclude accepted tasks from batching and say so ("resuming: T-01–T-03
+ already ACCEPTED"). Batch only the remaining tasks, starting at the first. Never
+ re-execute an accepted task, and never restart the pipeline at brainstorming
+ because the session died mid-plan.
+ After process recreation on an HTML pair, rediscover the same authoritative pair
+ and query `identity` and `eligible` again. An `IN_PROGRESS` task is not proof of
+ completion: capture the existing reconciliation authority and apply
+ `task-reconcile` before redispatch through `task-start` with a fresh complete
+ context ticket. A task in `REVIEW` still requires fresh evidence and the current
+ scope review before `accept-scope`; conversation memory cannot advance it.
+
For each task, confirm the plan names its exact target paths and its verification command. A task
missing either is underspecified — return it to `writing-plans`, do not improvise the gap.
Present the batch breakdown to the user as information, not a question: which tasks land in batch 1,
and what follows. Do NOT stop for a separate acknowledgment — the user approved the plan in
`writing-plans`, and the first Phase 3 checkpoint arrives after batch 1; a breakdown objection
surfaces there (or the user interrupts). Each unresolved unknown is a `[CONFIRM-NN]` in
`open-questions.md` — surface it, do not guess past it.
Gate: a batch sequence exists and every task has a target path and a verification command. A
`[CONFIRM-NN]` that blocks batch 1 is the only reason to stop here.
## Phase 2 — Execute batch · gate: BLOCK
Invoke `subagent-driven-development` (`${CLAUDE_PLUGIN_ROOT}/skills/subagent-driven-development/SKILL.md`) with `scope = [current batch task IDs]`. Pass the plan slug and
spec slug so it can read its own pre-flight files. Do not implement anything here — the author agents,
review chain, and verification all run inside that skill.
+ For HTML, pass the selected spec/plan identities and task IDs, not copied task
+ labels or prose. The nested workflow records dispatch with `task-start`, records
+ the separate verification and spec-review receipts with `task-review`, and
+ records whole-scope acceptance once with `accept-scope`. Preserve every returned
+ identity and receipt for the checkpoint and later commit proof.
+
`subagent-driven-development` returns when every task in the scope is `ACCEPTED` (Phase 3–5 green,
verification passed on a fresh run). A `tdd` BLOCK, a security CRITICAL finding, or an unresolved
`[CONFIRM-NN]` surfaced inside that skill halts the loop — surface it to the user and do not proceed.
Gate: `subagent-driven-development` signals batch complete with all scoped tasks `ACCEPTED`. Any
unresolved halt from inside the skill is a gate failure here.
## Phase 3 — Checkpoint · gate: STOP
Every task in the batch is done and verified. STOP and checkpoint with the user before any further
work:
- **Landed** — the tasks completed this batch, each verified by `subagent-driven-development` Phase 5.
- **Next** — the tasks in the upcoming batch.
- **Open** — any `[NEEDS-TRIAGE]` marker raised inside the batch, any `[CONFIRM-NN]` still blocking.
Do not begin the next batch until the user acknowledges. This gate is the point of `/feature` — the
human checkpoint that separates it from `/sprint`'s autonomous run.
Gate: the user has acknowledged the checkpoint. On acknowledgement, loop to Phase 2 for the next
batch. When the final batch is acknowledged and the plan is fully verified, route to `commit-gate` —
do not commit from here.
+ Before claiming that an HTML plan's final batch completed, use the installed
+ bridge's `_preflight_current_acceptance` with the exact route returned by
+ `_resolve_workflow_pair`, the repository-bound client, and the engine-returned
+ spec and plan artifact IDs. Pass no caller status, digest, receipt, or label.
+ Retain the returned engine identities and receipt set for `commit-gate`. Any
+ refusal leaves the batch incomplete and blocks the handoff.
+
## Hard rules
- MUST NOT execute tasks inline — delegate every batch to `subagent-driven-development` with an explicit `scope`.
- MUST NOT advance past a checkpoint the user has not acknowledged.
- - MUST NOT call `commit-gate` until all batches are acknowledged and every plan task is `ACCEPTED`.
+ - MUST NOT call `commit-gate` until all batches are acknowledged and every plan
+ task is accepted; for HTML this means a fresh `eligible` result reports
+ `all_accepted_and_current: true` for the selected plan identity.
- MUST surface any halt from `subagent-driven-development` (tdd BLOCK, CRITICAL, CONFIRM-NN) to the user — do not swallow it or re-dispatch around it.
- MUST NOT absorb scope the plan does not name — mark it `[NEEDS-TRIAGE]`.
- MUST NOT commit; route to `commit-gate` when the plan is fully verified.
- MUST return an underspecified task (missing path or verification command) to `writing-plans` — do not improvise the gap.
- MUST surface a plan-versus-code conflict via `/conflict`, never silently reconcile it.