@arbiterforge/ca-pi 0.14.10 → 0.14.16

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
Files changed (44) hide show
  1. package/README.md +33 -9
  2. package/package.json +2 -1
  3. package/plugins/ca-pi/CHANGELOG.md +67 -0
  4. package/plugins/ca-pi/LICENSE +661 -0
  5. package/plugins/ca-pi/SKILLS.md +2 -2
  6. package/plugins/ca-pi/SPRINT.md +47 -0
  7. package/plugins/ca-pi/THIRD_PARTY_NOTICES.md +40 -0
  8. package/plugins/ca-pi/generated/command-catalog.json +2 -2
  9. package/plugins/ca-pi/helpers/artifacts/ca-artifact-darwin-amd64 +0 -0
  10. package/plugins/ca-pi/helpers/artifacts/ca-artifact-darwin-arm64 +0 -0
  11. package/plugins/ca-pi/helpers/artifacts/ca-artifact-linux-amd64 +0 -0
  12. package/plugins/ca-pi/helpers/artifacts/ca-artifact-linux-arm64 +0 -0
  13. package/plugins/ca-pi/helpers/artifacts/ca-artifact-windows-amd64.exe +0 -0
  14. package/plugins/ca-pi/helpers/artifacts/ca-artifact-windows-arm64.exe +0 -0
  15. package/plugins/ca-pi/helpers/artifacts/release.json +6 -6
  16. package/plugins/ca-pi/hooks/_approvallib.py +58 -22
  17. package/plugins/ca-pi/hooks/_artifactauthoritylib.py +2135 -0
  18. package/plugins/ca-pi/hooks/_artifactlib.py +164 -7
  19. package/plugins/ca-pi/hooks/_artifactpromptlib.py +177 -0
  20. package/plugins/ca-pi/hooks/_host.py +1 -1
  21. package/plugins/ca-pi/hooks/_overrideappendlib.py +5 -1
  22. package/plugins/ca-pi/hooks/_prerequisitelib.py +44 -22
  23. package/plugins/ca-pi/hooks/_reconciliationlib.py +472 -0
  24. package/plugins/ca-pi/hooks/_releaselib.py +345 -111
  25. package/plugins/ca-pi/hooks/_sprintapprovallib.py +302 -0
  26. package/plugins/ca-pi/hooks/artifact-authority-hook.py +127 -0
  27. package/plugins/ca-pi/hooks/artifact-authority.py +102 -0
  28. package/plugins/ca-pi/hooks/artifact-reconcile.py +57 -0
  29. package/plugins/ca-pi/hooks/hostapi.py +1 -1
  30. package/plugins/ca-pi/hooks/prompt-submit.py +20 -3
  31. package/plugins/ca-pi/hooks/releasehash.py +36 -23
  32. package/plugins/ca-pi/includes/artifacts.md +120 -13
  33. package/plugins/ca-pi/includes/sprint-authority.md +69 -0
  34. package/plugins/ca-pi/package.json +1 -1
  35. package/plugins/ca-pi/routines/INDEX.md +7 -4
  36. package/plugins/ca-pi/routines/brainstorming/SKILL.md +97 -29
  37. package/plugins/ca-pi/routines/commit-gate/SKILL.md +7 -1
  38. package/plugins/ca-pi/routines/executing-plans/SKILL.md +32 -15
  39. package/plugins/ca-pi/routines/release/SKILL.md +220 -70
  40. package/plugins/ca-pi/routines/skill-author/SKILL.md +26 -5
  41. package/plugins/ca-pi/routines/writing-plans/SKILL.md +22 -10
  42. package/plugins/ca-pi/skills/ca-commit/SKILL.md +170 -16
  43. package/plugins/ca-pi/skills/ca-new-skill/SKILL.md +116 -21
  44. package/plugins/ca-pi/skills/ca-release/SKILL.md +6 -4
@@ -1,6 +1,6 @@
1
1
  ---
2
2
  name: brainstorming
3
- description: The Socratic spec-refinement front of /feature, and the planning front of /sprint. Routed to BEFORE any code — it takes a one-line idea and drives it to an approved, concrete spec with testable acceptance criteria. Five gated phases — frame, shape, refine, write, review-and-approve. No implementation and no handoff to tdd until the spec is on disk and approved; each acceptance criterion becomes one tdd Phase 1 obligation.
3
+ description: The Socratic spec-refinement front of /feature, and the planning front of /sprint. Routed to BEFORE any code — it turns a one-line idea into a concrete spec with testable criteria, ready for initial combined sprint review or explicit sequential approval. Five gated phases — frame, shape, refine, write, review-and-approve. No implementation before an approved spec; the caller retains required planning, execution and delivery ordering. Each acceptance criterion becomes one tdd Phase 1 obligation.
4
4
  disable-model-invocation: true
5
5
  ---
6
6
 
@@ -23,7 +23,21 @@ structured-artifact engine and default to canonical `.html`. HTML `--farm`
23
23
  dispatch remains blocked.
24
24
 
25
25
 
26
- Refine the idea before it touches code. Routed to by `/feature` (before `tdd`) and by `/sprint` (the planning front).
26
+ Refine the idea before it touches code. Routed to by `/feature` and `/sprint` as their specification step.
27
+
28
+ ## Caller scope and authority
29
+
30
+ Read the active caller's workflow, selected lane, scope and existing delegation before refining.
31
+ Initial sprint planning is not an approved sprint: this skill does not grant its own delegation.
32
+ A full-lane feature requires `writing-plans` after spec approval. Initial HTML sprint planning instead
33
+ returns a ready draft identity for the combined spec/plan review before either is approved; a
34
+ subordinate specification step returns its result to the existing caller. Preserve direct handoffs when they
35
+ satisfy that caller's ordering, not an unconditional terminal route to implementation.
36
+
37
+ A sprint may contain several related features under one coherent goal. Keep their boundaries and
38
+ criteria explicit without converting the sprint into repeated feature interviews. Parameter and
39
+ method choices follow safety-core; material product decisions and missing authority are not inferred.
40
+ No new registered surface, runtime state or copied procedure is needed for this call contract.
27
41
 
28
42
  ## Pre-flight
29
43
 
@@ -41,7 +55,9 @@ when the feature touches component boundaries or integrations). Index-first, nev
41
55
  A missing file or directory here is NOT a gap to surface and never a STOP: record
42
56
  `intent: silent — no decomposition record` in the running notes and proceed.
43
57
 
44
- Per-feature and light. NOT decompose's whole-project six-layer interview — one feature, five phases. Depth scales with the feature, not the ceremony: a small feature earns a short pass through every phase, never a skipped one. A spec can be five sentences; it cannot be zero — smallness changes the spec's length, not its existence.
58
+ Scope-sized and light. NOT decompose's whole-project six-layer interview — five phases for the
59
+ caller's feature or coherent sprint. Depth scales with that scope, not the ceremony. The deliberately
60
+ inline small-feature lane remains owned by `/feature`; do not manufacture an HTML pair for it.
45
61
 
46
62
  ## Phase 1 — Frame the problem · gate: BLOCK
47
63
 
@@ -52,52 +68,74 @@ Take the one-line idea and pin its boundaries before designing or asking anythin
52
68
  - Name what this feature explicitly does NOT do — the boundary that keeps scope honest.
53
69
  - Check the framing against `CONTEXT.md`: it never contradicts the NOT-building list or redefines domain vocabulary. A contradiction is a conflict — surface it, do not reconcile it silently.
54
70
  - Check the framing against the recorded intent (pre-flight, when present): an idea that resurrects a capability `plans/02` records as deferred, or that duplicates a backlog item, is a fork to ask — lead with the recorded deferral rationale and your recommendation; never proceed on it silently (ADR-0025).
55
- - **Split before you spend.** If the idea bundles more than one independently shippable concern — distinct callers, acceptance criteria that would partition into disjoint sets, separable data or surfaces — say so NOW and negotiate the split with the user before any refinement effort is spent on one piece. Each surviving piece is its own spec and its own run of this skill. A bundle discovered in Phase 4 has already wasted every question asked about the piece that gets cut.
71
+ - **Respect the caller's unit of work.** For a feature, split unrelated independently shippable
72
+ concerns before refining them; record the selected piece and its exclusions. For a sprint, retain
73
+ related features within the stated sprint goal and identify their separable criteria and dependencies
74
+ for planning. Do not force one new user interview per feature or silently absorb unrelated work.
75
+ Ask only when the intended scope or necessary product decision remains unresolved.
56
76
 
57
- Gate: problem, caller, and out-of-scope boundary stated and consistent with `CONTEXT.md`, and the idea is confirmed to be ONE feature — or the split is agreed and this run proceeds on exactly one piece.
77
+ Gate: problem, caller, and out-of-scope boundary stated and consistent with `CONTEXT.md`; the unit is
78
+ the caller's selected feature or coherent sprint, with unrelated scope explicitly excluded.
58
79
 
59
80
  ## Phase 2 — Shape the approach · gate: BLOCK
60
81
 
61
82
  Before drilling into details, establish WHICH design the details belong to:
62
83
 
63
- - Propose the genuine candidate approaches — usually two or three — each with its real trade-off stated (`X gives you A but costs B`). Recommend exactly one, with the reasoning that picks it. The user chooses under `/feature`; under `/sprint`, SMARTS chooses and the choice is logged with its scoring.
84
+ - Propose the genuine candidate approaches — usually two or three — each with its real trade-off stated (`X gives you A but costs B`). Recommend exactly one, with the reasoning that picks it. The user rules on material product decisions during initial feature or sprint planning; within an already-approved sprint, its existing delegated decision rules apply and the choice is logged. A recommendation made before approval does not authorize execution.
64
85
  - **Never manufacture alternatives.** When only one sane approach exists, say so and say why — a padded list of straw options is noise wearing the costume of rigor, and it trains the reader to skim the one section that matters.
65
86
  - **Check each candidate against accepted ADRs** (the pre-flight index; ADR-0025). A contradicting candidate is surfaced WITH the ADR citation, never silently dropped — and it may not be recommended except paired with a supersession fork via `/adr`. When the contradicting candidate is the only sane approach, that IS the fork: present it (the user rules under `/feature`; under `/sprint` this surfaces at the interactive Phase 1 gate, where the user is present to rule).
66
87
  - Apply the isolation lens to the recommended shape while it is still soft: can each part be understood without reading its internals, and can its internals change without breaking its consumers? A design that fails this here fails it again in review, after the code exists.
67
88
  - Apply the YAGNI lens: strike anything the Phase 1 problem statement does not demand. A capability the caller never asked for is scope creep with a head start.
68
89
 
69
- Gate: one approach chosen (user-chosen, or SMARTS-chosen and logged), its trade-off stated, and nothing in it the problem does not require.
90
+ Gate: one approach selected under the caller's existing authority and recorded with its trade-off;
91
+ nothing in it exceeds the stated problem. Selection during initial planning is not execution authority.
70
92
 
71
93
  ## Phase 3 — Socratic refinement loop · gate: BLOCK
72
94
 
73
95
  Turn the chosen approach concrete. The discipline is decide-or-ask, never drift:
74
96
 
75
- - **Decide the parameters; ask the forks.** A parameter-level choice — a naming, a default, a threshold, an ordering with one sensible answer — is yours to make: decide it and record it in this phase's running notes; every recorded decision lands in the spec's **Decided parameters** section when Phase 4 writes the file, which is where the user reads them at approval. A genuine fork — a real trade-off, a scope boundary, anything irreversible or user-visible — is the user's: ask it IN FULL the first time, leading with your recommendation and its reasoning. Never merely mention that an open decision exists; a named-but-unasked question is an omission wearing a disclaimer.
97
+ - **Choose methods; ask missing decisions.** Select reversible parameters and implementation methods
98
+ within safety-core's decision-authority limits; multiple reasonable methods alone do not require a
99
+ user choice. Record material decisions in the running notes and the spec's **Decided parameters**.
100
+ Use the existing delegated decision rules only where the approved sprint covers the choice. A
101
+ missing product decision, unresolved `[CONFIRM-NN]`, scope expansion or irreversible operation is
102
+ not a method choice: ask the actual question with its recommendation and counter-consideration.
76
103
  - **Batch what is independent.** Forks whose answers do not depend on each other are asked together, not serialized into rounds. One round of three real questions beats three rounds of one.
77
104
  - Run every answer through three lenses:
78
- - **Vague language** — force concrete nouns, numbers, and verbs. "Manage", "handle", "support" are not verbs. "Fast", "secure", "scalable" are not specifications. "We'll figure it out later" is not an answer — every "later" becomes a `[CONFIRM-NN]`.
105
+ - **Vague language** — make the required outcome observable: "fast", "secure" and "scalable"
106
+ need acceptance boundaries. Only unresolved product requirements or missing user-owned decisions
107
+ become `[CONFIRM-NN]`; a reversible implementation detail deliberately left to planning within
108
+ the agreed constraints is not an unanswered user question. Do not invent unknowns to fill a template.
79
109
  - **Hidden complexity** — name what the user assumes is easy but is hard: state, concurrency, edge cases, failure modes, validation, idempotency, migration of existing data. Surface it now or it surfaces in `tdd`.
80
- - **Trade-off forcing** — when a real decision exists, frame it with a recommendation and let the user rule. Do not pick for the user; do not hide that a pick exists.
81
- - **The loop has a breaker.** When refinement stops converging — an answer contradicts an earlier one, or the same fork resurfaces after being answered — STOP the loop: restate the contradiction plainly, force one adjudication (a user ruling, or a recorded `[CONFIRM-NN]`), and only then continue. A loop that circles is not being thorough; it is failing to close.
110
+ - **Trade-off forcing** — identify the actual choice and its counter-consideration. Apply the caller's existing authority: ask material user-owned decisions, or choose and log an authorized delegated method. Never hide a choice or widen delegation.
111
+ - **The loop has a breaker.** When refinement stops converging, stop repeating the same question or
112
+ hypothesis. Reuse an applicable recorded ruling; within delegation, diagnose and choose a different
113
+ evidence-supported method. A genuine unresolved user decision or rule conflict is surfaced once
114
+ through its owner, never guessed or treated as permission. Do not manufacture `[CONFIRM-NN]` for
115
+ the agent's internal choice of method.
82
116
  - **Closure is a checklist, not a feeling.** The reasons to stop asking that do not count, each with why:
83
117
  - *"The feature is simple enough."* — Simplicity shortens the spec; it does not waive it. The five-sentence spec still names its criteria.
84
118
  - *"I've asked enough questions."* — Question fatigue is not an exit condition. The exit is the Phase 5 review passing.
85
119
  - *"The user seems eager to move on."* — Eagerness is a signal to compress the asking (decide more parameters, batch harder), never to skip an unresolved fork.
86
- - *"We can settle it during implementation."* — Every deferral is a `[CONFIRM-NN]` on record, or it is a guess that `tdd` will faithfully implement.
120
+ - *"We can settle it during implementation."* — Missing acceptance requirements cannot be deferred
121
+ as implementation freedom. Record genuine user-owned unknowns; retain method selection within
122
+ the existing planning and delegated-execution contract.
87
123
 
88
- Record every genuinely-unresolved unknown as `[CONFIRM-NN]` in `<project-root>/.codearbiter/open-questions.md`, numbered sequentially. A finding that belongs to a different feature or a future scope gets an inline `[NEEDS-TRIAGE]` marker in the notes — never route it to a ticket.
124
+ Record every genuinely-unresolved user-owned requirement or decision as `[CONFIRM-NN]` in `<project-root>/.codearbiter/open-questions.md`, numbered sequentially. A finding that belongs to a different feature or a future scope gets an inline `[NEEDS-TRIAGE]` marker in the notes — never route it to a ticket.
89
125
 
90
126
  **Before closing this loop, check the criteria decided so far against their own stated intent, not merely against each other** (#566: `writing-plans` Phase 4 proves BIJECTION between a plan's tasks and the `AC-NN` ledger — every criterion has a task, every task has a criterion — and that proves the two AGREE, never that the ledger itself is COMPLETE. A criterion missed by both sides passes that check cleanly; catching the miss belongs here, at the source, before Phase 4 writes anything to freeze it in place.) Assemble a scratch file outside the working tree (`mktemp`) shaped like the spec sections Phase 4 is about to write — a `## Scope` heading over the Phase 1 framing, and an `## Acceptance criteria` heading over every criterion decided in this phase's running notes so far — and, when this feature traces to a GitHub issue, its body into a second scratch file (`gh issue view <N> --json body -q .body > <scratch-file>`; omitted when there is no linked issue). Run `"$PY" "<plugin-root>/hooks/_intentlib.py" uncovered-intent <scope-and-criteria-file> [--issue-body <scratch-file>]`. A non-empty result names an in-scope bullet or an issue acceptance checkbox no criterion's text cites — add the missing criterion now, or record it as a `[CONFIRM-NN]` if it is genuinely undecided, never carry it forward uncited. This mechanical pass is the CHEAP half; it catches only an uncited bullet or checkbox.
91
127
 
92
128
  Then ask the half it cannot mechanize, explicitly, as its own step: **if every criterion decided so far passed and nothing else changed, what would still be broken?** A real answer names a criterion the ledger is still missing even though every scope bullet and checkbox is technically cited — a bullet reading "fix all three contaminated skills" against criteria for two of the three passes the citation check above cleanly, and only this question catches it. Finding nothing broken is a reportable result, stated in one line, never a silent skip.
93
129
 
94
- Gate: every vague term made concrete; every fork resolved by the user or recorded as `[CONFIRM-NN]`; every parameter decision recorded in the running notes that seed Phase 4's **Decided parameters** section; no unresolved "later" outside a `[CONFIRM-NN]`; the `uncovered_intent` backstop returns empty or every finding is resolved; and the negative question above has been asked and answered, not merely named. A blocking `[CONFIRM-NN]` that gates the spec's core stops the loop — surface it and STOP.
130
+ Gate: every vague term made concrete; every material choice resolved under the caller's authority or recorded as a genuine `[CONFIRM-NN]`; every parameter decision recorded in the running notes that seed Phase 4's **Decided parameters** section; no unresolved user-owned acceptance requirement outside a `[CONFIRM-NN]`; the `uncovered_intent` backstop returns empty or every finding is resolved; and the negative question above has been asked and answered, not merely named. A blocking `[CONFIRM-NN]` that gates the spec's core stops the loop — surface it and STOP.
95
131
 
96
132
  ## Phase 4 — Write the spec · gate: BLOCK
97
133
 
98
- Create the agreed spec through the installed structured-artifact engine at
99
- `<project-root>/.codearbiter/specs/<slug>.html`; do not write the HTML or its
100
- embedded model directly. The slug is derived from the feature. The spec holds:
134
+ Create or revise the agreed spec at the route-selected spec path, in its selected format.
135
+ New full-lane work uses `<project-root>/.codearbiter/specs/<slug>.html` through the installed
136
+ structured-artifact engine; do not write HTML or its embedded model directly. An existing `.md`
137
+ spec stays on its exact legacy path; never create a counterpart or implicitly convert it.
138
+ The spec holds:
101
139
 
102
140
  - **Problem** — the Phase 1 framing in final form.
103
141
  - **Approach** — the Phase 2 choice and the trade-off that picked it, in two or three sentences.
@@ -113,29 +151,59 @@ Gate: the spec file exists on disk under `specs/`, with at least one acceptance
113
151
 
114
152
  The spec earns its approval; it is not waved through. Two passes over the FILE as written, then the stop:
115
153
 
116
- 1. **Mechanical self-review** — one pass, fix inline, no re-review: no placeholder text (`TBD`, `???`, an unfilled section); no criterion that contradicts the scope; nothing that crosses the `CONTEXT.md` NOT-building boundary; no criterion that contradicts an accepted ADR or `plans/01`'s recorded component boundaries (ADR-0025); every criterion still testable by one or more focused tests after the edits; the vague-language lens applied to the spec's own prose.
154
+ 1. **Mechanical self-review** — one pass, correct through the selected format's owning writer, no re-review: no placeholder text (`TBD`, `???`, an unfilled section); no criterion that contradicts the scope; nothing that crosses the `CONTEXT.md` NOT-building boundary; no criterion that contradicts an accepted ADR or `plans/01`'s recorded component boundaries (ADR-0025); every criterion still testable by one or more focused tests after the edits; the vague-language lens applied to the spec's own prose.
117
155
  2. **Adversarial pass** — build the strongest case AGAINST the design before the user reads it: the failure mode most likely to be real, the criterion most likely to be wrong, the assumption that would invalidate the approach if false. Present what survives WITH the spec at approval — the user rules on a challenged design, not a defended one. Finding nothing is a reportable result, stated in one line, not a silent skip.
118
156
  3. **Approval:**
119
157
  - **Under `/feature`** — present the spec and the adversarial findings, and request explicit user approval. Iterate on the file in place until the user approves. A blocking `[CONFIRM-NN]` must be resolved by the user before approval — never auto-resolve it.
120
- - **Under `/sprint`** — approval may be granted automatically by SMARTS scoring, logged to the `.codearbiter/` audit trail. A blocking `[CONFIRM-NN]` is never auto-approvable; it escalates to the user and STOPs the sprint flow.
121
-
122
- On approval, hand off to the `tdd` skill (`<plugin-root>/routines/tdd/SKILL.md`), which enters Phase 1 against the approved spec — one obligation per acceptance criterion.
123
-
124
- Gate: both review passes run against the file on disk, adversarial findings presented, and the spec approved (by the user under `/feature`, or by logged SMARTS auto-approval under `/sprint`) with no unresolved blocking `[CONFIRM-NN]`. Only then does control pass to `tdd`.
158
+ - **Initial `/sprint` planning** — prepare the ready draft spec and adversarial findings for
159
+ explicit user approval with its complete draft plan at the caller's one initial review gate.
160
+ For HTML, return the current draft identity to `writing-plans` in `draft_for_pair` mode;
161
+ the combined `arm-sprint` / `sprint-approve` path in `<plugin-root>/includes/artifacts.md`
162
+ observes the exact returned reply and binds/approves both documents atomically. Do not separately approve
163
+ the spec, synthesize a SMARTS receipt, or execute draft tasks. Do not reinterpret one generic reply
164
+ as approval of either artifact or delegation. Autonomy begins only after the
165
+ caller's spec AND plan approval requirement is met. A legacy Markdown caller retains its
166
+ recorded approval mechanism; there is no implicit conversion.
167
+
168
+ - **Within an already-approved sprint** — use its existing delegation only for an in-scope choice.
169
+ For HTML, a supported policy-owned adapter must supply the actual event and engine receipt;
170
+ follow `<plugin-root>/includes/artifacts.md`. A SMARTS score or log entry is not an approval receipt.
171
+ The private `smarts-apply` producer covers only existing plan-method steps under an explicit
172
+ paired user grant; it cannot approve a specification amendment or widen its protected scope.
173
+ Use only a qualified production producer for the requested authority and host; do not invent a capture event
174
+ or reuse inspection-only evidence. A missing producer is not delegation: retain the pending state and use
175
+ only the existing supported authority path. The native engine's presence does not qualify a host adapter.
176
+ Blocking `[CONFIRM-NN]` decisions are never auto-approved in either phase.
177
+
178
+ **Continuation follows the caller.** For a full-lane feature or sprint with approved spec authority,
179
+ return the approved spec identity and review outcome to its planning step; for initial HTML sprint
180
+ review, return the ready draft identity without execution authority. Either may forward directly to
181
+ `writing-plans` (`<plugin-root>/routines/writing-plans/SKILL.md`) when that is the caller's declared
182
+ next step. Do not skip required plan creation, binding or approval by jumping to `tdd`. A subordinate
183
+ invocation returns to the named checkpoint in its caller instead of launching another full workflow.
184
+ The inline small feature route belongs to `/feature` and can reach `tdd` without invoking this full-lane
185
+ specification procedure. Neither path grants commit, PR, merge or publication authority.
186
+
187
+ Gate: both review passes completed against the authoritative file, adversarial findings presented,
188
+ and no unresolved blocking `[CONFIRM-NN]`. Require actual approval verified before execution.
189
+ Only initial combined HTML review may return a ready unapproved draft to its constrained planner.
190
+ The next operation obeys the active caller's sequence and authority.
125
191
 
126
192
  ## Hard rules
127
193
 
128
194
  - MUST NOT write implementation code or route to `tdd` before the spec is on disk under `specs/` AND approved.
129
- - MUST NOT refine a bundle — an idea holding more than one independently shippable concern splits in Phase 1, before any question budget is spent.
195
+ - MUST preserve the caller's feature or coherent sprint boundary; unrelated scope is split or excluded, not silently absorbed.
130
196
  - MUST NOT manufacture alternatives in Phase 2 — when one sane approach exists, say so and why.
131
197
  - MUST NOT write an acceptance criterion that cannot be verified by one or more focused tests.
132
- - MUST decide parameter-level choices and record them in the spec; MUST ask genuine forks in full, with a recommendation, the first time — never name an open decision without asking it.
133
- - MUST stop a non-converging refinement loop and force one adjudication — a contradiction or a resurfacing fork is a breaker, not a prompt to circle.
198
+ - MUST record material parameter choices; ask genuine missing user decisions in full, not internal method-selection questions already covered by delegation.
199
+ - MUST stop a non-converging refinement strategy; reuse recorded rulings or authorized diagnosis rather than repeat an answered question. Genuine conflicts still route to their owner.
134
200
  - MUST run the `uncovered_intent` mechanical backstop and ask the negative-judgment question ("if every criterion passed and nothing else changed, what would still be broken?") before closing Phase 3 — bijective coverage in `writing-plans` proves the plan and the ledger agree with each other, never that the ledger is complete (#566).
135
201
  - MUST run the Phase 5 mechanical self-review and adversarial pass against the file as written, and present the adversarial findings at approval.
136
202
  - MUST NOT resolve a `[CONFIRM-NN]` by guessing — surface it and record it in `open-questions.md`.
137
203
  - MUST NOT auto-approve a spec carrying a blocking `[CONFIRM-NN]`, even under `/sprint` — it escalates to the user.
138
204
  - MUST NOT contradict the NOT-building list or redefine domain vocabulary in `CONTEXT.md` — a contradiction is a conflict to surface, not reconcile.
139
- - MUST NOT run decompose's six-layer whole-project interview — this is one feature, five phases.
140
- - MUST log a `/sprint` auto-approval to the `.codearbiter/` audit trail.
141
- - MUST, at exit, run the follow-up harvest (`<plugin-root>/includes/harvest.md`) over any `[NEEDS-TRIAGE]` notes raised this run — batch-confirm promoting them to `open-tasks.md` (work) or `open-questions.md` (decisions) so out-of-scope ideas don't vanish.
205
+ - MUST NOT run decompose's six-layer whole-project interview or force a coherent sprint through repeated one-feature interviews.
206
+ - MUST NOT use SMARTS to approve the initial sprint; record delegated decisions in the existing audit trail and verify actual authority for the selected artifact format.
207
+ - MUST, at exit, run the follow-up harvest (`<plugin-root>/includes/harvest.md`) in the active caller's mode:
208
+ batch-confirm interactive work; use autonomous promotion only under existing delegation. A blocking
209
+ decision is never promoted into authority or silently demoted to a follow-up.
@@ -1,12 +1,18 @@
1
1
  ---
2
2
  name: commit-gate
3
- description: The only path to a commit. Routed to when the user invokes /commit or otherwise instructs codeArbiter to persist staged changes. Nine gated phases — permission, branch, classification, verification (test/lint/secrets), behavioral proof, diff review, selective stage, message, commit. Nothing reaches version control without clearing every gate; "it looks good" is not authorization.
3
+ description: Create a verified local Git commit when committing changes is requested. Not for explaining commit history, drafting a message only, or postponing a commit. Applies every commit gate and never implies a push or PR.
4
+ argument-hint: (none)
4
5
  ---
5
6
 
6
7
  # commit-gate
7
8
 
8
9
  The only permitted path to a commit. Bypassing it is a hard-rule violation. Routed to when the user invokes `/commit` or any equivalent instruction to persist staged changes.
9
10
 
11
+ No command arguments are required; inspect current Git state. Test-only requests use the
12
+ recorded invocation in `tech-stack.md` without entering this committing procedure. PR
13
+ readiness belongs to the existing PR workflow; an informational or message-only request
14
+ must not reach staging, commit, or this procedure's writing close.
15
+
10
16
  ## Pre-flight
11
17
 
12
18
  Read these, or STOP and surface the gap — never guess a command:
@@ -1,6 +1,6 @@
1
1
  ---
2
2
  name: executing-plans
3
- 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.
3
+ description: The checkpoint coordinator for /feature. Routed to by /feature once a writing-plans plan exists. Uses authored HTML checkpoints or bounded legacy batches, delegates each scope 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.
4
4
  disable-model-invocation: true
5
5
  ---
6
6
 
@@ -23,6 +23,8 @@ Coordinate the plan in small, user-acknowledged batches. Routed to by `/feature`
23
23
  `writing-plans` has produced a plan. Each batch is executed by `subagent-driven-development` — fresh
24
24
  author agent per task, spec-compliance review, quality review, fresh verification — and the user
25
25
  checkpoints between batches. The orchestrator never implements; it schedules and checkpoints.
26
+ An HTML checkpoint is an acceptance scope, not a new user checkpoint in `/sprint`. Sprint routes
27
+ directly to `subagent-driven-development` and does not inherit this attended coordinator's pauses.
26
28
 
27
29
  ## Pre-flight
28
30
 
@@ -42,22 +44,34 @@ Read these, or STOP and surface the gap — never guess a path, a command, or a
42
44
 
43
45
  ## Phase 1 — Batch plan · gate: BLOCK
44
46
 
45
- Group the plan's tasks into small batches. Keep batches tight — three to five tasks is a ceiling, not
46
- a target. Respect the plan's ordering and dependencies: a task never lands before the task it depends on.
47
+ For Markdown, group remaining tasks into small batches; the three-to-five task ceiling is legacy
48
+ presentation guidance, not a reason to rewrite typed scope. For HTML, read the authored checkpoint partition
49
+ through the installed engine. Build the sequence from the plan's exact checkpoint membership and ordering,
50
+ retaining each checkpoint's full task-ID set as its acceptance scope. Skip a checkpoint only when current
51
+ engine evidence establishes its accepted state. Do not remove `REVIEW` tasks from an unfinished checkpoint:
52
+ they still need the combined review and whole-scope acceptance. Likewise, do not drop `BLOCKED` members:
53
+ excluding one does not make the checkpoint complete. Resume it only through the supported reconciliation path
54
+ and refreshed eligibility. Respect explicit dependencies in either format.
55
+
56
+ A smaller display grouping must not split an HTML acceptance scope, combine checkpoints, or change their
57
+ membership to fit a task-count ceiling. Execute and accept one authored scope at a time. If the partition
58
+ itself is unsuitable, return the actual problem to `writing-plans` for the existing amendment/approval path;
59
+ do not mutate scope identity in the coordinator or add a new batching registry.
47
60
 
48
61
  **Resume is the normal re-entry.** For Markdown, a task already `ACCEPTED` in
49
62
  the plan's status column was verified before an earlier interruption. For HTML,
50
- derive the same decision from the current `eligible` result and its exact task
51
- identities. Exclude accepted tasks from batching and say so ("resuming: T-01–T-03
52
- already ACCEPTED"). Batch only the remaining tasks, starting at the first. Never
53
- re-execute an accepted task, and never restart the pipeline at brainstorming
54
- because the session died mid-plan.
63
+ derive resume from current `eligible` evidence and the authored checkpoint partition above.
64
+ For Markdown, exclude accepted tasks from new batches. For HTML, retain checkpoint membership while
65
+ selecting only eligible work, and say which current accepted checkpoints are already complete.
66
+ Stale proof requires its existing reconciliation and revalidation, not automatic reimplementation.
67
+ Never restart the pipeline at brainstorming merely because the session died mid-plan.
55
68
 
56
69
  After process recreation on an HTML pair, rediscover the same authoritative pair
57
70
  and query `identity` and `eligible` again. An `IN_PROGRESS` task is not proof of
58
- completion: capture the existing reconciliation authority and apply
59
- `task-reconcile` before redispatch through `task-start` with a fresh complete
60
- context ticket. A task in `REVIEW` still requires fresh evidence and the current
71
+ completion. Use the installed reconciliation adapter in `<plugin-root>/includes/artifacts.md` for the exact task,
72
+ including its genuine host-observed decision and exact stored retry identity; do not construct
73
+ an authority event in this coordinator. A supported `task-reconcile` transition must precede
74
+ redispatch through `task-start` with a fresh complete context ticket. A task in `REVIEW` still requires fresh evidence and the current
61
75
  scope review before `accept-scope`; conversation memory cannot advance it.
62
76
 
63
77
  For each task, confirm the plan names its exact target paths and its verification command. A task
@@ -69,8 +83,10 @@ and what follows. Do NOT stop for a separate acknowledgment — the user approve
69
83
  surfaces there (or the user interrupts). Each unresolved unknown is a `[CONFIRM-NN]` in
70
84
  `open-questions.md` — surface it, do not guess past it.
71
85
 
72
- Gate: a batch sequence exists and every task has a target path and a verification command. A
73
- `[CONFIRM-NN]` that blocks batch 1 is the only reason to stop here.
86
+ Gate: a batch sequence exists and every task has a target path and a verification command.
87
+ Missing current authority or capability, a stale prerequisite, or a `[CONFIRM-NN]` blocking the
88
+ first scope is a gate failure. Follow the existing diagnostic or reconciliation owner; do not
89
+ invent a plan, drop blocked members, or restart the interview to clear it.
74
90
 
75
91
  ## Phase 2 — Execute batch · gate: BLOCK
76
92
 
@@ -78,8 +94,9 @@ Invoke `subagent-driven-development` (`<plugin-root>/routines/subagent-driven-de
78
94
  spec slug so it can read its own pre-flight files. Do not implement anything here — the author agents,
79
95
  review chain, and verification all run inside that skill.
80
96
 
81
- For HTML, pass the selected spec/plan identities and task IDs, not copied task
82
- labels or prose. The nested workflow records dispatch with `task-start`, records
97
+ For HTML, pass the selected spec/plan identities and the complete task-ID set for one authored checkpoint;
98
+ do not pass a partial or cross-checkpoint task set. Labels and presentation batches do not confer acceptance.
99
+ The nested workflow uses engine eligibility to select work inside that scope, records dispatch with `task-start`, records
83
100
  the separate verification and spec-review receipts with `task-review`, and
84
101
  records whole-scope acceptance once with `accept-scope`. Preserve every returned
85
102
  identity and receipt for the checkpoint and later commit proof.