@gtrabanco/pi-agentic-workflow 0.13.3 → 0.15.0

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 (118) hide show
  1. package/README.md +3 -9
  2. package/dist/conductor/closeout.d.ts +18 -0
  3. package/dist/conductor/closeout.js +24 -0
  4. package/dist/conductor/decide.d.ts +25 -0
  5. package/dist/conductor/decide.js +104 -0
  6. package/dist/conductor/index.d.ts +16 -0
  7. package/dist/conductor/index.js +15 -0
  8. package/dist/conductor/invoke.d.ts +15 -0
  9. package/dist/conductor/invoke.js +59 -0
  10. package/dist/conductor/loop.d.ts +25 -0
  11. package/dist/conductor/loop.js +236 -0
  12. package/dist/conductor/sensor.d.ts +12 -0
  13. package/dist/conductor/sensor.js +150 -0
  14. package/dist/conductor/snapshot.d.ts +13 -0
  15. package/dist/conductor/snapshot.js +60 -0
  16. package/dist/conductor/types.d.ts +248 -0
  17. package/dist/conductor/types.js +47 -0
  18. package/dist/conductor/urgency.d.ts +16 -0
  19. package/dist/conductor/urgency.js +70 -0
  20. package/dist/config/merge.js +21 -1
  21. package/dist/config/schema.js +45 -2
  22. package/dist/config/types.d.ts +22 -0
  23. package/dist/config/types.js +7 -0
  24. package/dist/extension/conductor-command.d.ts +23 -0
  25. package/dist/extension/conductor-command.js +93 -0
  26. package/dist/extension/factory.js +6 -1
  27. package/dist/routing/types.d.ts +2 -0
  28. package/dist/routing/types.js +2 -0
  29. package/dist/runtime.d.ts +45 -0
  30. package/dist/runtime.js +62 -0
  31. package/package.json +4 -2
  32. package/skills/audit-docs/SKILL.md +6 -5
  33. package/skills/audit-pr/SKILL.md +1 -1
  34. package/skills/audit-pr/references/PORTABILITY.md +1 -1
  35. package/skills/execute-phase/SKILL.md +207 -116
  36. package/skills/execute-phase/references/DESCOPE.md +22 -18
  37. package/skills/execute-phase/references/EXECUTION_CONTRACT.md +69 -77
  38. package/skills/execute-phase/references/FOLDING.md +34 -47
  39. package/skills/execute-phase/references/PREFLIGHT.md +103 -114
  40. package/skills/execute-phase/references/PRE_EXECUTION_GATE.md +53 -60
  41. package/skills/init-workspace/SKILL.md +35 -5
  42. package/skills/init-workspace/references/BOOTSTRAP_DISCOVERY.md +5 -5
  43. package/skills/init-workspace/references/BOOTSTRAP_WRITE.md +15 -8
  44. package/skills/init-workspace/references/PORTABILITY.md +1 -1
  45. package/skills/init-workspace/references/UPGRADE.md +17 -7
  46. package/skills/log-session/SKILL.md +4 -3
  47. package/skills/orchestration-envelope/references/TURN_CONTRACT.md +1 -1
  48. package/skills/pre-execution-review/references/LEDGERS.md +39 -39
  49. package/skills/pre-execution-review/references/POLICY.md +1 -0
  50. package/skills/review-change/SKILL.md +1 -1
  51. package/skills/review-change/references/PORTABILITY.md +1 -1
  52. package/skills/unit-lane/SKILL.md +154 -0
  53. package/skills/unit-lane/references/DESIGN.md +35 -0
  54. package/skills/unit-lane/references/DOCS.md +36 -0
  55. package/skills/unit-lane/references/EVIDENCE.md +34 -0
  56. package/skills/unit-lane/references/IMPLEMENT.md +45 -0
  57. package/skills/unit-lane/references/PLAN.md +38 -0
  58. package/skills/unit-lane/references/RELEASE.md +45 -0
  59. package/skills/unit-lane/references/RESEARCH.md +37 -0
  60. package/skills/unit-lane/references/REVIEW.md +45 -0
  61. package/skills/unit-lane/references/STEPS.md +23 -0
  62. package/skills/unit-lane/references/TESTS.md +32 -0
  63. package/skills/workflow-status/references/ENVELOPE_FIELDS.md +5 -6
  64. package/skills/workflow-status/references/SENSOR_CORE.md +16 -20
  65. package/skills/workflow-status/references/SENSOR_SIGNALS.md +5 -7
  66. package/skills/design-feature/SKILL.md +0 -231
  67. package/skills/design-feature/references/INTERVIEW.md +0 -74
  68. package/skills/design-feature/references/PORTABILITY.md +0 -15
  69. package/skills/design-feature/references/REPAIR.md +0 -88
  70. package/skills/design-feature/references/UPSERT_EXAMPLE.md +0 -8
  71. package/skills/design-feature/references/WRITE_AND_UPSERT.md +0 -115
  72. package/skills/evidence-grounding/SKILL.md +0 -174
  73. package/skills/evidence-grounding/references/DELEGATION.md +0 -131
  74. package/skills/evidence-grounding/references/READINESS.md +0 -130
  75. package/skills/evidence-grounding/references/ROWS.md +0 -93
  76. package/skills/execute-phase/references/BATCH_AND_PORTABILITY.md +0 -57
  77. package/skills/execute-phase/references/CLOSEOUT.md +0 -77
  78. package/skills/execute-phase/references/HANDOFF.md +0 -21
  79. package/skills/execute-phase/references/OPPORTUNISTIC_FINDING.md +0 -45
  80. package/skills/execute-phase/references/UNIT_LOOP.md +0 -88
  81. package/skills/execute-phase/references/WORKFLOWS_FEATURE.md +0 -37
  82. package/skills/execute-phase/references/WORKFLOWS_FIX.md +0 -43
  83. package/skills/execute-phase/references/WORKFLOWS_LEGACY.md +0 -26
  84. package/skills/execute-phase/references/WORKFLOWS_SMALL_PHASED.md +0 -38
  85. package/skills/generate-docs/SKILL.md +0 -145
  86. package/skills/generate-docs/references/ADAPTERS.md +0 -15
  87. package/skills/generate-docs/references/ADAPTER_DISCOVERY.md +0 -33
  88. package/skills/generate-docs/references/GENERATION_PROCESS.md +0 -90
  89. package/skills/implementation-discovery/SKILL.md +0 -209
  90. package/skills/plan-feature/SKILL.md +0 -208
  91. package/skills/plan-feature/references/ROUTING.md +0 -155
  92. package/skills/plan-feature-from-issue/SKILL.md +0 -162
  93. package/skills/plan-feature-scaffold/SKILL.md +0 -112
  94. package/skills/plan-feature-scaffold/references/SCAFFOLD_PROCESS.md +0 -122
  95. package/skills/plan-fix/SKILL.md +0 -161
  96. package/skills/plan-fix/references/PLANNING_PROCESS.md +0 -148
  97. package/skills/plan-fix/references/SPEC_CONTRACT.md +0 -22
  98. package/skills/planning-preflight/SKILL.md +0 -130
  99. package/skills/replan-findings/SKILL.md +0 -131
  100. package/skills/replan-findings/references/PHASE_APPEND.md +0 -89
  101. package/skills/review-plan/SKILL.md +0 -154
  102. package/skills/review-plan/references/CHECKS.md +0 -111
  103. package/skills/review-plan/references/ENG-CHECKS.md +0 -30
  104. package/skills/review-plan/references/OUTPUT.md +0 -166
  105. package/skills/review-spec/SKILL.md +0 -145
  106. package/skills/review-spec/references/CHECKS.md +0 -108
  107. package/skills/review-spec/references/OUTPUT.md +0 -160
  108. package/skills/ship-roadmap/SKILL.md +0 -120
  109. package/skills/ship-roadmap/references/ADVANCE.md +0 -117
  110. package/skills/ship-roadmap/references/AUDIT_AND_MERGE.md +0 -64
  111. package/skills/ship-roadmap/references/CLOSEOUT_AND_LOG.md +0 -27
  112. package/skills/ship-roadmap/references/FOUNDING.md +0 -101
  113. package/skills/ship-roadmap/references/GUARDRAILS.md +0 -56
  114. package/skills/ship-roadmap/references/MODEL_ROUTING.md +0 -16
  115. package/skills/ship-roadmap/references/PORTABILITY.md +0 -34
  116. package/skills/ship-roadmap/references/RECOVERY_AND_SELECTION.md +0 -94
  117. package/skills/ship-roadmap/references/STOP_CONDITIONS.md +0 -9
  118. package/skills/ship-roadmap/references/TERMINAL_REPORT.md +0 -34
@@ -1,120 +0,0 @@
1
- ---
2
- name: ship-roadmap
3
- user-invocable: true
4
- version: 5.2.3
5
- author: "Gabriel Trabanco <1969593+gtrabanco@users.noreply.github.com>"
6
- license: MIT
7
- argument-hint: "[--fullauto] | --continue [--fullauto]"
8
- description: >
9
- Found or continue a roadmap autopilot one stage per invocation. Default:
10
- human merge. --fullauto is invocation-scoped and uses the transient wrapper
11
- only after a fresh audit. Triggers: "ship-roadmap", "ship the roadmap",
12
- "autopilot this project".
13
- ---
14
-
15
- # Ship the roadmap (autopilot)
16
-
17
- Found once, then run the driver-fired loop (Claude `/loop`, external driver, or
18
- manual re-invocation): plan, implement, review, open and optionally merge one PR
19
- per roadmap unit. After the roadmap, sweep existing issues, ship fix-now work,
20
- report residue as proposals (never create backlog automatically), and print the
21
- final report. Use strong tiers for judgment, cheap tiers for typing, and humans
22
- at expensive-to-undo decisions.
23
-
24
- `ship-roadmap` is the conductor of the loop. It emits its native fixed `SHIP:`
25
- banner and closing `→ Next:` block; the package-owned machine-result profiles
26
- apply to the worker and sensor skills that a driver invokes inside the loop.
27
-
28
- ## Turn contract — verify before ending the turn
29
-
30
- ```
31
- ✓ Exactly ONE stage advanced (or a terminal banner printed) and ONE line appended to the run log
32
- ✓ Nothing was merged outside the active --fullauto wrapper; direct merge
33
- commands remained blocked and no authorization survived the iteration
34
- ✓ Artifact language: explicit user instruction > the project's declared docs language > English. The CONVERSATION language never decides — a Spanish prompt still produces English PRs/issues/commits/SPECs unless one of the first two says otherwise
35
- ✓ The closing `→ Next:` block is printed as the ABSOLUTE last output
36
- ```
37
-
38
- Any unchecked box means the turn is not done.
39
-
40
- ## When to use
41
-
42
- Use for a locked roadmap with supervision at merge/end points. For one feature,
43
- one bug, or exploratory work use the manual planning/execution flow instead.
44
-
45
- ## Step 0 — Discover the project (always first)
46
-
47
- Read before acting: `AGENTS.md` Workflow conventions, documentation
48
- map, `docs/features/ROADMAP.md`, fix index, architecture doc, and `.github/`
49
- templates. Then establish:
50
-
51
- 1. **Substrate:** if the guide, map, roadmap and fix index exist, skip founding
52
- and confirm their answers; otherwise founding creates missing pieces.
53
- 2. **Skills:** verify `plan-feature`, `execute-phase`, `review-change`, and
54
- `audit-pr` are installed and record their directory for worker prompts. If
55
- absent, stop with `npx skills add gtrabanco/agentic-workflow`. With
56
- `--fullauto`, also require executable `.agentic-workflow/hooks/fullauto-merge.sh`
57
- and its platform guard; otherwise route to `init-workspace`, never direct merge.
58
- 3. **Run:** existing `docs/features/SHIP_DECISIONS.md` or an open
59
- `docs/ship-founding` PR means resume with `--continue`; bare invocation reports
60
- status instead of founding again.
61
- 4. **Shape:** greenfield/existing, branch, and dirty tree; unexplained dirty
62
- default branch stops (never clean it silently).
63
-
64
-
65
- ## Progressive loading — select the invocation route
66
-
67
- The allowlist is exactly the linked paths below. Never invent another reference.
68
- Every route starts with [guardrails](references/GUARDRAILS.md), then loads only
69
- its matching row:
70
-
71
- **Hard rule for `--continue` at AUDIT:** LOAD exactly, in this order,
72
- `references/GUARDRAILS.md`, `references/RECOVERY_AND_SELECTION.md`,
73
- `references/STOP_CONDITIONS.md`, `references/ADVANCE.md`,
74
- `references/MODEL_ROUTING.md`, and
75
- `references/AUDIT_AND_MERGE.md`; after the stage, load
76
- `references/CLOSEOUT_AND_LOG.md`. When the run is not terminal and all named
77
- primitives exist, every other reference is forbidden for that turn.
78
-
79
- | Condition now | LOAD complete route in this order | SKIP now |
80
- |---|---|---|
81
- | Found or inspect a run, no `--continue` (default mode or greenfield `--fullauto`) | [guardrails](references/GUARDRAILS.md) → [founding](references/FOUNDING.md) | recovery, stop conditions, advance, model routing, audit/merge, terminal report, portability |
82
- | Existing-repo founding with `--fullauto` | [guardrails](references/GUARDRAILS.md) → [founding](references/FOUNDING.md) → [audit and merge policy](references/AUDIT_AND_MERGE.md) | recovery, stop conditions, advance, model routing, terminal report, portability |
83
- | Continue one non-AUDIT iteration | [guardrails](references/GUARDRAILS.md) → [recovery and selection](references/RECOVERY_AND_SELECTION.md) → [stop conditions](references/STOP_CONDITIONS.md) → [advance](references/ADVANCE.md) → [model routing](references/MODEL_ROUTING.md) before stage execution → [closeout and log](references/CLOSEOUT_AND_LOG.md) after it | founding, audit/merge, terminal report, portability |
84
- | Continue an AUDIT/fullauto iteration | [guardrails](references/GUARDRAILS.md) → [recovery and selection](references/RECOVERY_AND_SELECTION.md) → [stop conditions](references/STOP_CONDITIONS.md) → [advance](references/ADVANCE.md) → [model routing](references/MODEL_ROUTING.md) → [audit and merge policy](references/AUDIT_AND_MERGE.md) before the AUDIT stage → [closeout and log](references/CLOSEOUT_AND_LOG.md) after it | founding, terminal report, portability |
85
- | Terminal stop/report | the active row above, then [terminal report](references/TERMINAL_REPORT.md) | unrelated rows |
86
- | A named platform primitive is absent | the active row above, then [portability](references/PORTABILITY.md) | unrelated rows |
87
-
88
- Do not load terminal reporting before terminal state or portability when all
89
- primitives exist. Model routing precedes every stage; audit/merge precedes every
90
- AUDIT stage, including non-`--fullauto` runs.
91
-
92
- Selected resources are one hop from this file. Fixed banners, transitions, floor
93
- checks and output blocks are normative; an unreadable required resource stops the
94
- run—never improvise from an older run.
95
-
96
- ## Portability
97
-
98
- The workflow is driver-neutral. Use the exact [portability](references/PORTABILITY.md)
99
- fallback only when a primitive is unavailable; keep stage order and safety floors.
100
-
101
- ## Relationship to other skills
102
-
103
- | Relation | Skills/policy |
104
- |---|---|
105
- | Compose (same/lower tier) | `init-workspace`; JIT `design-feature` + `plan-feature-scaffold`; `plan-feature`; the review→fold manual path (`/review-change` → `/fold-findings` on `REVIEW-FAIL` → re-run `/review-change`); verdict-only `audit-pr`; `audit-docs`. |
106
- | Cheap workers | `execute-phase` (fresh context/phase), mechanical folds, audit-blocker fixes; use the validated worker model. |
107
- | Human hand-off | Default merges, `product-audit` (higher effort), and report issue batches via `triage-issue`. |
108
- | Sibling | `workflow-status` supplies the strict sensor result; worker skills use the package-owned output profiles, while the conductor keeps its native `SHIP:` turn contract. |
109
-
110
- Manual feature-by-feature flow remains the default; this skill packages it with
111
- the human at its edges.
112
-
113
- ## Done when
114
-
115
- - A terminal banner and final report exist with an open PR; roadmap statuses are
116
- true; every PR is merged, open/audited, or parked with a reason. `SHIP: COMPLETE`
117
- also accounts for the issue sweep and reports residue as proposals.
118
- - Locked answers, iteration evidence, outcomes and recommendations are traceable
119
- in `SHIP_DECISIONS.md`, the run log and report; the report states exact next
120
- human actions (merges, triage, proposals, product-audit timing).
@@ -1,117 +0,0 @@
1
- ## Advance exactly one stage
2
-
3
- 4. **ADVANCE exactly one stage:**
4
- - **DESIGN (mid-run `idea`/`defined` unit only)** — compose `design-feature`
5
- in-turn (equal tier, opus/high — within the ≥ rule already stated for
6
- founding), **deriving the product half strictly from the locked
7
- `SHIP_DECISIONS.md` record — no new questions, ever** (the "no further
8
- questions after the interview" contract this skill already enforces for
9
- every other stage applies here identically). Walk capability closure
10
- from what rounds 2–4 already answered for this feature; stamp `## Design
11
- status: designed` and promote the roadmap row `idea → defined` on
12
- success. Then compose `plan-feature-scaffold` in the same iteration to
13
- promote `defined → planned` (both writes are the same JIT pass — a
14
- feature never sits mid-promotion between iterations). **Undesignable
15
- from the locked record** (the feature as scoped contradicts a locked
16
- decision, or needs an answer rounds 2–4 never covered) → do **not**
17
- guess and do **not** ask: emit `NEEDS_INPUT`, **park the unit** with the
18
- specific gap recorded (mirrors a red-gate park — see Stop conditions),
19
- and SELECT continues with the next startable unit this same run (or ends
20
- the iteration per the capacity guard if none remain). A parked
21
- undesignable unit is picked back up only by a human answering the
22
- recorded question and re-running `/design-feature <slug> "<answer>"`
23
- directly — the autopilot never re-asks it.
24
- - **REVIEW-SPEC** (only between DESIGN and PLAN) — compose `review-spec` in a
25
- clean context at the routed tier (opus/high), never in the turn that wrote the
26
- product half. `SPEC-REVIEW-PASS` releases the unit into PLAN. A FAIL whose
27
- findings are common-root-cause or wording-only returns to the same unit's
28
- author for one root-caused repair batch, then a fresh review. `NEEDS-DESIGN`
29
- means a product choice is open that this run's locked record cannot answer:
30
- **park the unit** with the exact question (`NEEDS_INPUT`, same shape as the
31
- undesignable park above) — the autopilot never answers a product question
32
- itself, and `SELECT` continues with the next startable unit.
33
- - **PLAN** — compose `plan-feature` in-turn via its scoped path (equal
34
- tier). Every unit reaching PLAN holds a current `SPEC-REVIEW-PASS` (REVIEW-SPEC
35
- ran first, or the unit was already planned before this gate existed and its
36
- own Product receipt is current), so `plan-feature`'s PRODUCT-REVIEW gate passes
37
- here. The interview path is **forbidden** mid-run:
38
- SPEC gaps are resolved silently from the decision record and logged. JIT
39
- planning that reveals the feature's premise is wrong (obsolete, absorbed,
40
- impossible on this stack) → mark it blocked with the contradiction
41
- recorded; never re-ask.
42
- - **REVIEW-PLAN** (only between PLAN and EXECUTE) — compose `review-plan` in a
43
- clean context at the routed tier over the plan the previous stage just froze.
44
- `PLAN-REVIEW-PASS` releases EXECUTE. A FAIL routes by root cause: plan-local
45
- (bad phase cut, blank validator, ledger drift) → root-caused re-cut by the
46
- planning author and a review; product-rooted or an assumption this record
47
- cannot settle → PLAN-REVIEW-FAIL with class: product → design-feature. A second local cycle
48
- that changes nothing stops editing and reports `CONVERGENCE-ANOMALY` instead of
49
- burning a third budget. **No stage between PLAN and EXECUTE may create a forge
50
- issue or defer an obligation to one** — an unmet obligation fails this stage, it
51
- is never exported.
52
- - **EXECUTE** — run each unfinished **implementation phase** in a **fresh cheap-tier context**: on Claude
53
- Code, spawn one subagent per phase with `model: sonnet` (the override is
54
- the only mechanism that runs *below* the conductor's turn tier); on an
55
- agent without subagents, the equivalent is one headless invocation per
56
- phase driven from outside (see `docs/workflow/ORCHESTRATION.md`), or —
57
- last resort — executing the phase in THIS turn while noting in the run
58
- log that it ran at the conductor's tier. Whatever the mechanism, the
59
- phase executor is instructed to read the **installed `execute-phase`
60
- SKILL.md first** (at the skills directory
61
- located in Step 0 and recorded in the decision record — e.g.
62
- `.claude/skills/execute-phase/SKILL.md` in Claude Code) and follow it for
63
- exactly one explicit phase: tests-first where it
64
- applies, gate green, one commit, per-phase docs. Two autopilot overrides
65
- to its recipe: (a) **never ask** — SPEC ambiguity is resolved from the
66
- committed decision record with the most conservative reading, and the
67
- assumption is surfaced in the phase docs so the conductor logs it;
68
- (b) the **P1 planning commit also carries `ROADMAP.md`** (the feature's
69
- `in-progress` flip rides it; the **`done` flip rides the PR-stage commit**
70
- when the PR opens; any flips left at run end ride the report commit).
71
- Stop before the literal final `Hardening & PR` phase: PR owns that explicit
72
- phase so close-out has one owner. Never bundle phases into one worker.
73
- - **REVIEW** — compose the review→fold path (equal tier) once: `/review-change`, then `/fold-findings` (on `REVIEW-FAIL`), then re-run `/review-change`. It reuses a current exact-SHA receipt or runs `review-change`
74
- context-clean, batches compatible fixes through `fold-findings`, and
75
- re-reviews only changed HEADs. Default correction budget is two; unchanged
76
- evidence stops NO-PROGRESS. L/sensitive features
77
- forward `--adversarial 2` (security/auth: `3`) as the unattended floor;
78
- other units use one final reviewer. There are no intermediate phase
79
- reviews in autopilot: phase gates + frozen acceptance guard execution,
80
- and the independent final loop judges one complete candidate. Independent
81
- future work remains proposals; no review/fold stage creates issues.
82
- - **PR** — run the unit's explicit final `Hardening & PR` phase through one
83
- fresh cheap-tier `execute-phase` worker. That canonical close-out flips the
84
- feature to `done` (built, not merged), pushes, creates the PR with `Closes
85
- #N` where issue-born, prints its URL, links it from the roadmap, commits the
86
- link, and pushes. Reconcile an already-open matching PR idempotently; never
87
- create a second one. The stage is incomplete until the row carries its PR
88
- link and the branch is clean/current.
89
- - **AUDIT** — compose `audit-pr` in-turn (equal tier); bind the verdict to
90
- the PR's head SHA in the run log, and **print the PR's full URL next to
91
- the verdict in the iteration output** (the human merging works from the
92
- chat, not from a CI monitor). MERGE-READY → default mode logs and
93
- moves on; `--fullauto` treats `audit-pr` as verdict/comment-only, checks
94
- the floors, **records the merge intent in the run log first**, then calls
95
- `.agentic-workflow/hooks/fullauto-merge.sh` with only the PR number and
96
- run id. The wrapper derives and verifies head, base, decision, and audit
97
- evidence from the forge. Never invoke `gh pr merge` directly.
98
- BLOCKED → in-scope blockers go to a
99
- cheap-tier worker next iteration (max 2 audit cycles, then the feature is
100
- parked and the loop moves on); the fixer's cycle ends committed AND
101
- pushed (step 5), so the re-audit judges the real branch.
102
-
103
- Fix units take the same pair in miniature: **plan-fix → REVIEW-PLAN → EXECUTE
104
- (`--fix`)** — a fix has no Product hop to wait on (D6), so its plan review is the
105
- only pre-execution gate it can have. The stage sequence is per-feature and size-dependent — always **one stage
106
- per iteration**: a feature starting at `idea`/`defined` gets a DESIGN stage
107
- first; one already `planned` (including the founding-scaffolded feature 01)
108
- goes to its missing review stage, and skips to EXECUTE only when a current
109
- `PLAN-REVIEW-PASS` is bound to its bytes. Every size follows
110
- **[DESIGN → REVIEW-SPEC] → PLAN → REVIEW-PLAN → EXECUTE**
111
- (implementation phases, fresh cheap worker per phase) → PR (explicit
112
- `Hardening & PR`) → REVIEW (bounded final loop) → AUDIT. Risk changes final
113
- review strength, not phase cadence. The two review stages are the pre-execution
114
- pair: they judge documents in a clean context and are **not** the post-code
115
- REVIEW loop, which stays exactly as described below. Merge policy is unchanged —
116
- the human (or the `--fullauto` wrapper behind its recorded floors) still owns the
117
- merge, and neither review stage may merge, close, or file anything.
@@ -1,64 +0,0 @@
1
- ### Merge policy
2
-
3
- **Default — the human merges.** The autopilot opens PRs and never merges. It
4
- continues with the next feature whose dependencies are all merged (new branches
5
- always cut from the freshly pulled default branch); when everything remaining
6
- waits on human merges, it stops with `SHIP: BLOCKED` + the unblock map. After
7
- merging, re-run the same launch command (`/loop /ship-roadmap --continue`, plus
8
- `--fullauto` on fullauto runs) — recovery records the merges (the rows are already
9
- `done` from PR-open), unblocks the dependents, and resumes.
10
-
11
- **`--fullauto` — dual-keyed.** Auto-merge requires a fresh SHA-bound
12
- `audit-pr:merge-ready` comment and `merge: fullauto` in the decision file at
13
- the PR's current head — stale or caller-supplied local evidence can never
14
- enable it. The **first
15
- feature PR of a greenfield run is always human-merged** (calibration: inspect
16
- one complete artifact — code, tests, docs, review trail — before delegating).
17
-
18
- This AUDIT stage is the **sole automated merge authority**. Authorization lasts
19
- for this invocation and this merge attempt only:
20
-
21
- 1. `audit-pr` returns a fresh SHA-bound MERGE-READY verdict and never merges.
22
- 2. Record merge intent in `.ship-run.log` before execution.
23
- 3. Invoke exactly:
24
-
25
- ```sh
26
- .agentic-workflow/hooks/fullauto-merge.sh \
27
- --pr <number> --run-id <run-id>
28
- ```
29
-
30
- The wrapper derives the PR head and forge default base itself. A project
31
- with no forge-reported checks fails closed.
32
- 4. The wrapper re-checks the current PR head, forge default base, SHA-bound
33
- audit comment, decision file fetched from that head, clean/synchronized
34
- branch, mergeability, and CI evidence. It
35
- creates a namespaced marker under the git common directory only after those
36
- checks, installs cleanup with `trap`, and removes the marker on success,
37
- failure, signal, or already-merged recovery.
38
- 5. After the forge reports MERGED, the wrapper posts one idempotent comment
39
- marked `<!-- agentic-workflow:automerge head=<sha> -->`, containing the run,
40
- audited head, and merge commit. That PR comment is the durable automerge log;
41
- no repository log grows.
42
-
43
- Direct `gh pr merge`, `glab mr merge`, `git merge`, and forge-API merge calls
44
- remain blocked by the command guard at all times. There is no `.automerge` file
45
- and no persistent exception for a later manual `audit-pr` or shell session.
46
-
47
- Non-negotiable floors, evaluated fresh immediately before every merge —
48
- **fail-closed: a floor that cannot be evaluated counts as breached**:
49
-
50
- 1. **Never merge red** — re-verify CI status via the forge CLI at merge time;
51
- the audit verdict is evidence, fresh green CI is the precondition. In a
52
- no-CI project the accepted evidence is a **fresh local verification-gate run
53
- on the PR's exact head SHA**, recorded in the run log — without one of the
54
- two, the floor is unevaluable and therefore breached.
55
- 2. **Verdict freshness** — MERGE-READY must reference the PR's current head
56
- SHA; any later commit forces a re-audit.
57
- 3. **Sensitive-area pause** — PRs touching the declared sensitive set are
58
- never auto-merged; the run continues around them and the report flags them.
59
- 4. **Destructive-operation pause** — data-deleting or schema-destructive
60
- diffs pause even when not in the declared set (users forget to declare it).
61
- 5. **Forge refusal is a signal** — never bypass branch protection, never
62
- force-push, never merge to anything but the default branch; a refused merge
63
- parks the PR and is reported.
64
- 6. **Budget floors still bind** — no cap is exempted by `--fullauto`.
@@ -1,27 +0,0 @@
1
- ## Close out and log the iteration
2
-
3
- 5. **CLEAN CLOSE-OUT — verify before logging the stage complete.** A stage
4
- only counts as advanced when the conductor has RUN and checked, on the
5
- unit's branch: `git status --porcelain` → empty (no tracked modification
6
- left behind — **docs included**: progress/testing/known-issues/roadmap
7
- edits ride the stage's commit, never linger), and — once the unit's PR
8
- exists — `git status -sb` after `git fetch` → not ahead of the remote
9
- (every commit pushed; the PR and CI must see what was actually done).
10
- A subagent that "finished" but left the tree dirty or the branch unpushed
11
- did NOT finish: the conductor commits/pushes the remainder itself (same
12
- stage, same iteration) or marks the stage partial. This check is
13
- unconditional for REVIEW-SPEC/REVIEW-PLAN repairs, EXECUTE, REVIEW fix cycles,
14
- PR, and AUDIT fix cycles.
15
- 6. **LOG** one line to `.ship-run.log`; print `→ Next: <unit> (CONTINUE)` (the
16
- canonical next-step shape; `CONTINUE` stays the loop's keep-going signal).
17
- **Say WHY the turn is ending** — one explicit
18
- line before the `→ Next:` block, always one of: "iteration complete — one
19
- stage advanced (normal; re-invoke to continue)", "parked <unit>: <exact cap
20
- hit — red-gate retries / review ping-pong / audit ping-pong / 3 partials /
21
- planning contradiction>", or the terminal banner's reason. A turn ending
22
- silently reads as a crash on agents without `/loop` — never leave the stop
23
- unexplained.
24
-
25
- **Capacity guard:** an iteration that cannot finish its stage in one turn
26
- (e.g. an oversized review) writes a partial-stage marker and ends cleanly;
27
- three consecutive partials on the same stage parks the feature as blocked.
@@ -1,101 +0,0 @@
1
- ## Process
2
-
3
- ### Mode A — Found & launch (interactive): `/ship-roadmap [--fullauto]`
4
-
5
- **1. The interview — all questions up front, then silence.** Small batched
6
- rounds; recommended defaults on every question; skip what discovery already
7
- answered. After Round 6 locks, **no further questions for the entire run** —
8
- every later decision is made silently and logged with a one-line rationale.
9
-
10
- | Round | Covers |
11
- |---|---|
12
- | 1 — Product | What it is, for whom; scale ceiling (solo / team / thousands of customers); lifespan & ambition (throwaway, internal, long-lived production). Calibrates every ceremony decision downstream. |
13
- | 2 — Features | The feature list (or "elicit" → draft one from the goal); must-have vs can-wait; ordering constraints; explicit out-of-scope. |
14
- | 3 — Stack & architecture | Stack decided? else recommend from features/constraints. Architecture chosen? else recommend the **lightest structure proportional to Round 1** — a solo tool gets a flat modular layout, a thousands-of-customers system gets enforced boundaries; never default to DDD, hexagonal, or any named pattern. Platform/runtime constraints, library vetoes. |
15
- | 4 — Quality & ops | Test depth (smoke / workflow default / strict); whether a11y, SEO, i18n, perf budgets apply (proposed from platform type); deploy target + scaffold CI?; secrets posture; **confirm the proposed verification gate commands** — they become the gate every phase must pass. |
16
- | 5 — Workflow & autonomy | Docs language (default English); forge + CLI (**verify with a real authenticated call now**, e.g. `gh auth status` — not mid-loop); **git workflow** (default `branches`: one active unit, sequential, no worktrees — `worktrees` only if the user declares it and their tooling manages them; recorded in the Workflow conventions and honored by every stage); merge policy (default human-merge vs `--fullauto`); the sensitive-area list (defaults: auth, payments, destructive migrations/data deletion, secrets, CI config — **seeded with every integration named in rounds 2–4**, e.g. the payment processor or auth provider the user mentioned); budget caps (default: max iterations = 4× roadmap feature count; 2 retries per red gate; 2 review-fix and 2 audit-fix cycles; optional "pause after N shipped features" checkpoint and milestone stop lines); model-routing confirmation; recommend enabling `ultracode` for the loop. |
17
- | 6 — Confirm & launch | The drafted roadmap (numbers, order, deps, sizes) and the full decision record, presented for **one last edit**. Then: founding artifacts written, exact `/loop` command printed. |
18
-
19
- **2. Founding (only what's missing).** Compose `init-workspace`'s process
20
- in-turn (both opus/high — within the ≥ rule), **pre-fed with the interview
21
- answers** so it asks nothing. Branch discipline:
22
-
23
- - **Empty repo:** the scaffold (AGENTS.md, docs/, .github/, completed
24
- ROADMAP.md, decision record) is the repo's **initial commit on the default
25
- branch** — there is no history to protect and no base for a PR yet.
26
- - **Existing repo:** founding goes on a `docs/ship-founding` branch as a PR.
27
- Default mode: **stop after the interview** — print the PR and require it
28
- merged before the loop starts (building features against an unmerged
29
- substrate would stack PRs). `--fullauto`: gate the founding PR with
30
- `audit-pr` like every other PR, then merge it.
31
-
32
- **3. The roadmap — founding IS batch design.** The interview's rounds 2–4
33
- (features, quality/ops, workflow) already collected every product-definition
34
- answer a `design-feature` capability closure would ask — founding is design
35
- for every feature it names, not a shortcut around it. Adopt existing entries
36
- (never renumber), fill gaps the interview surfaced, append elicited features.
37
- If absent, write the complete table: NN in dependency-respecting order, slug,
38
- **status `idea`** (the locked founding decisions are the design record, but
39
- no per-feature `SPEC.md`/capability-closure artifact exists yet — that gets
40
- written JIT per feature, see ADVANCE → DESIGN below), depends-on, one-line
41
- summary with a **provisional XS/S/M/L size in the summary text** (the
42
- template's five-state legend and column schema stay exactly as they are —
43
- `plan-feature`/`plan-feature-scaffold` re-size authoritatively at planning
44
- time; a size change is logged silently). Greenfield: **feature 01 is always
45
- the project skeleton** (stack init, gate wiring, CI if requested), sized S —
46
- founding scaffolds it immediately (composing `design-feature` +
47
- `plan-feature-scaffold` in-turn, pre-fed from the interview, no questions) so
48
- it lands directly at **status `planned`**, never left at `idea` — and **every
49
- other feature's depends-on closure must include 01** (directly or
50
- transitively), so SELECT can never start a feature on a default branch that
51
- lacks the skeleton.
52
-
53
- **4. The run state — two artifacts, deliberately split:**
54
-
55
- - `docs/features/SHIP_DECISIONS.md` — **committed** (rides the founding
56
- commit/PR): run mode, safety floors, sensitive-area list, budget caps, stop
57
- lines, model routing, docs language, and a digest of every locked interview
58
- answer. It is the durable, auditable policy: a crash, another machine, or a
59
- fresh clone recovers the full run policy without re-interviewing.
60
- - `docs/features/.ship-run.log` — **untracked** (founding appends it to
61
- `.gitignore`): the append-only iteration log — one line per iteration
62
- (`date | NN-slug | stage | outcome | evidence: SHA / PR# / verdict`), silent
63
- decisions with rationale, partial-stage markers, verdict↔SHA bindings.
64
- Machine-local mechanics; committing it would conflict across every open PR.
65
-
66
- **5. Print the launch contract** — detect which driver this environment has
67
- and print the matching command. **Three equivalent drivers** (the loop is the
68
- contract; who re-invokes it is an implementation detail):
69
-
70
- | Driver | When | Launch |
71
- |---|---|---|
72
- | **`/loop`** (Claude Code) | The agent has a self-re-invoking loop primitive | `/loop /ship-roadmap --continue` |
73
- | **External orchestrator** | Any agent invocable headless (a shell loop, CI, your own program) | loop: invoke `/ship-roadmap --continue`, parse its native `SHIP:`/`→ Next:` contract, and re-invoke while the turn is non-terminal — see `docs/workflow/ORCHESTRATION.md` |
74
- | **Manual** | Neither of the above | re-run `/ship-roadmap --continue` yourself after each iteration; each ends with the exact next command |
75
-
76
- Default launch contract text (adapt the first line to the detected driver):
77
-
78
- ```
79
- Founded. Start the autopilot with:
80
-
81
- /loop /ship-roadmap --continue (Claude Code)
82
- — or loop `/ship-roadmap --continue` from your orchestrator/by hand;
83
- parse the native `SHIP:` banner and `→ Next:` block, then re-invoke
84
- while the turn is non-terminal (see docs/workflow/ORCHESTRATION.md)
85
-
86
- Stop when an iteration's first line is SHIP: COMPLETE, SHIP: BLOCKED, or
87
- SHIP: STOPPED (envelope state OK, BLOCKED, or FAILED). Iterations are
88
- idempotent and resume cleanly; stopping at any time is safe.
89
- ```
90
-
91
- For a fullauto run the command is `/ship-roadmap --continue --fullauto` (under
92
- whichever driver) — the flag must ride every iteration, because auto-merge is
93
- dual-keyed: the flag on the running command **and** `merge: fullauto` in the
94
- committed decision record (see Merge policy). One key without the other runs
95
- in default mode.
96
-
97
- Each firing is a fresh `/ship-roadmap --continue` turn (on Claude Code's
98
- `#claude` branch, at this skill's pinned tier; elsewhere, at whatever tier the
99
- driver chose — judgment iterations deserve your strongest model). Iterations
100
- after a terminal banner are cheap no-ops that re-print the same banner — so a
101
- missed stop costs tokens, never correctness.
@@ -1,56 +0,0 @@
1
- ## Guardrails
2
-
3
- - **Never work on the default branch** — the empty-repo initial scaffold commit
4
- is the single exception. One PR per unit, never stacked; roadmap status flips
5
- ride PR-bound commits only.
6
- - **Forge bodies are Markdown, not shell — never hand-escape.** Every explicitly
7
- authorized issue, PR, or comment the run creates (subagent PRs, triage
8
- comments) carries a body of **real Markdown**: backticks / `*` / `_` are
9
- formatting, and a `\` before them renders literally (`` \`code\` `` instead
10
- of `` `code` ``). Write the body to a file and pass **`--body-file <path>`**
11
- to `gh issue create` / `gh pr create` / `gh issue comment` (or the declared
12
- forge's equivalent) — never inline `--body "…"` or a quoted heredoc. Verify
13
- with `gh … --json body` that no literal `` \` `` survived. (execute-phase
14
- subagents already follow this; the conductor must too.)
15
- - **Never commit red; never merge red.** The gate and the floors are
16
- unconditional — no flag, mode, or interview answer disables them.
17
- - **Never request or retain a direct-merge permission.** Fullauto calls only
18
- the repository wrapper with command-scoped variables; a missing wrapper or
19
- active guard blocks the run and routes to `init-workspace` upgrade.
20
- - **No stage ends dirty or unpushed.** The clean close-out check (Mode B
21
- step 5) is part of every stage: tracked modifications — docs included — are
22
- committed with the stage, and a PR-backed branch is pushed before the
23
- iteration logs the stage complete. Merging while anything is uncommitted,
24
- unpushed, or unpulled is forbidden: push, wait for CI, re-audit, then merge.
25
- - **The conductor never writes application code.** All implementation flows
26
- through fresh cheap-tier `execute-phase` workers, one phase per context —
27
- that keeps the cost model honest and `execute-phase` the single pathway.
28
- - **Tier discipline.** Compose in-turn only skills at ≤ opus/high;
29
- implementation goes below the turn tier via explicit subagent model
30
- overrides; `product-audit` is never run by this skill. `ultracode` is a
31
- user-owned session setting — recommended, never claimed.
32
- - **Interview once, then silence.** Mid-run gaps — including a mid-run
33
- `idea`/`defined` unit's product-half gaps in the DESIGN stage — are resolved
34
- from the decision record and logged; contradictions park the feature with
35
- the evidence recorded. Re-interviewing mid-run is forbidden, in DESIGN as in
36
- every other stage: an undesignable unit is parked (`NEEDS_INPUT` on that
37
- unit, `state: CONTINUE` on the run), never asked about. The recovery from a
38
- wrong founding call is a reported stop and a human-restarted run.
39
- - **Scope discipline.** Defects and ideas discovered mid-run become report
40
- proposals — never automatically-created issues or in-run side quests. Only
41
- existing user/forge issues enter the issue sweep.
42
- - **Stack/architecture/forge agnostic; English artifacts** regardless of the
43
- interview language; recommendations proportional to the interviewed scale,
44
- recorded in the project's own docs so every sub-skill discovers them through
45
- its normal Step 0.
46
-
47
- **Known limits (stated, not hidden):** subagent overrides pin the model but
48
- not the effort, so execution subagents inherit the session's effort — cost can
49
- drift if the session runs high. `/loop`'s stop-on-banner matching should be
50
- treated as a convenience, not a guarantee — iterations after a terminal banner
51
- are idempotent no-ops, and the loop can always be stopped manually. Budget caps
52
- count iterations, not tokens — and the count lives in the machine-local log, so
53
- it bounds each machine's run, not the run's lifetime across machines. Verdicts
54
- persist in the run log and feature docs,
55
- but a crash between PR close-out and review may re-run one receipt check —
56
- accepted cost, never a correctness risk.
@@ -1,16 +0,0 @@
1
- ### Model routing
2
-
3
- | Stage | Tier | Mechanism |
4
- |---|---|---|
5
- | Interview, founding, roadmap creation | opus/high | this skill's frontmatter; composes `init-workspace` (equal tier), answers pre-fed |
6
- | Recovery, routing, logging | opus/high | in-turn (tiny token volume; a subagent would add cost, not save it) |
7
- | JIT feature design (mid-run `idea`/`defined` unit) | opus/high | compose `design-feature` + `plan-feature-scaffold` in-turn (equal tier, deriving only from `SHIP_DECISIONS.md` — no new questions) |
8
- | JIT feature planning | opus/high | compose `plan-feature` in-turn (its internals are opus/high–medium: ≥ holds) |
9
- | Product review (REVIEW-SPEC) | opus/high | `review-spec` in a **clean context** (fresh subagent or outside headless call) — never the turn that wrote the product half; tier equals or exceeds the author's, so the ≥ rule holds |
10
- | Plan review (REVIEW-PLAN) | opus/high | `review-plan` in a **clean context**, same rule; it judges the plan the previous stage froze |
11
- | Phase execution, single-pass, fixes | **cheap worker** | fresh context per phase following `execute-phase`; Claude branch maps this role to `sonnet`, portable drivers use their validated worker tier |
12
- | Final review/correction loop | opus/high | compose the manual path `/review-change`, then `/fold-findings` (only on `REVIEW-FAIL`), then re-run `/review-change`; it reuses a current exact-SHA receipt or reviews context-clean, and routes unresolved findings to `triage-issue --prioritize-now` |
13
- | Merge gate | opus/high | compose `audit-pr` in-turn (the highest-stakes automated verdict; must share one turn with the floor checks) |
14
- | Forge/git mechanics | — | Bash tool calls; no model judgment involved |
15
- | Final-report evidence gathering | haiku (optional) | fan-out subagents for grep-shaped per-feature log collection when ultracode is on; synthesis stays opus |
16
- | `product-audit` | opus/max | **never composed, never imitated by a subagent** — its effort (max) exceeds the conductor's (high) and a subagent override cannot carry `effort: max`. Hand-off only: the report prescribes when to run it. |
@@ -1,34 +0,0 @@
1
- ## Portability (agents other than Claude Code)
2
-
3
- The workflow is the contract; Claude Code features are conveniences. This skill
4
- leans on them harder than any other — here is the manual equivalent of each:
5
-
6
- - **No `/loop`** — two equivalent replacements, both vendor-neutral: (a) an
7
- **external orchestrator** loops `/ship-roadmap --continue` headless and
8
- routes its native fixed `SHIP:` banner plus closing `→ Next:` block (full
9
- protocol + driver skeleton in `docs/workflow/ORCHESTRATION.md`); (b) manual
10
- re-invocation after each iteration. Iterations are
11
- stateless-by-reconstruction, so any driver is exactly equivalent; stop when
12
- the first line is a terminal `SHIP:` banner. **Every iteration ending is
13
- announced** — "iteration complete (normal; re-invoke to continue)" vs a
14
- parked/terminal reason — so a stop is never ambiguous.
15
- - **No subagents** — execute phases sequentially: for each phase, one **fresh
16
- headless invocation (or conversation) on a cheaper model** following the
17
- installed `execute-phase` SKILL.md for exactly one phase (same two autopilot
18
- overrides) — the external-driver pattern in `docs/workflow/ORCHESTRATION.md`
19
- does exactly this. The conductor stages (recover/plan/PR/review/audit) stay
20
- on your strongest model.
21
- - **No slash-command menu** — where this skill says `/<skill>`, open that
22
- skill's `SKILL.md` (wherever your agent installed the skills) and follow it
23
- literally in the conversation the routing table assigns it (in-turn = this
24
- conversation; subagent/hand-off = a fresh one).
25
- - **No per-skill `model:`/`effort:`** — the `#claude` branch's routing table pins these tiers; here, pick tiers yourself:
26
- judgment stages on your **strongest** model, implementation on a cheaper one,
27
- and `product-audit` always as its own maximum-effort run.
28
- - **Provider concurrency limits** — when spawning parallel subagents (or
29
- parallel headless invocations), cap concurrency at the provider's documented
30
- parallel-request limit per API key, leaving one slot free for the conductor
31
- (e.g. a provider allowing 5 concurrent requests → at most 3–4 parallel
32
- executors). An agentic loop spends one request per tool round-trip, so
33
- parallel executors also eat the per-minute request budget fast; on a 429,
34
- reduce parallelism before retrying rather than hammering the limit.
@@ -1,94 +0,0 @@
1
- ### Mode B — One loop iteration: `/ship-roadmap --continue [--fullauto]`
2
-
3
- Every iteration is stateless-by-reconstruction — no memory is assumed between
4
- turns:
5
-
6
- 1. **RECOVER.** Read `SHIP_DECISIONS.md` (missing → `SHIP: STOPPED — no run
7
- policy; run /ship-roadmap first`) and `.ship-run.log` (missing on this
8
- machine → recreate empty; policy lives in the committed record). **Verify
9
- the substrate landed:** `SHIP_DECISIONS.md` must exist on the default
10
- branch — an open `docs/ship-founding` PR means the substrate isn't merged
11
- yet → `SHIP: BLOCKED` with "merge the founding PR" as the unblock map.
12
- Read ROADMAP.md; query the forge for open/merged PRs on `feat/*`, `fix/*`,
13
- `docs/ship-founding` and `docs/ship-report` heads; check git state.
14
- Reconcile: a feature flips to `done` when its **PR opens** (built, not merged
15
- — see the PR stage), so a `done` row with an open PR is awaiting a human merge
16
- (default mode), not finished shipping. A **merged** PR needs no status change
17
- (already `done`) — it means *shipped*, and **unblocks its dependents** + counts
18
- toward `SHIP: COMPLETE`. The done-flip rides the PR-bound commit, never a lone
19
- commit on the default branch. A dirty feature branch from a crashed phase
20
- is handed to the next phase subagent to finish or restart (counts against
21
- the red-gate retry cap). Uncommitted changes on the default branch confined
22
- to `docs/features/<NN-slug>/` + ROADMAP.md that match an in-flight roadmap
23
- row are the loop's own planning output — resume that feature; the
24
- dirty-default stop fires only for changes matching no roadmap unit.
25
- 2. **STOP-CHECK.** Evaluate the stop conditions (below). Terminal → write or
26
- refresh the final report, open the report PR, print the `SHIP:` banner +
27
- status table, end the turn.
28
- 3. **SELECT one unit.** Priority order, first match wins:
29
- 1. **Urgency labels first (feature 15, injection-safe).** Read
30
- `workflow-status`'s `detail.urgent` (labels-only, presence-only —
31
- never derived from issue text; see `skills/triage-issue/SKILL.md`, the
32
- sole owner of the `urgent`/`fix-next` vocabulary):
33
- - Any open issue labeled **`fix-next`** → its fix jumps to the head of
34
- the queue exactly like a blocking fix below (`plan-fix` →
35
- `execute-phase --fix`), **no interrupt** of the in-flight unit — it
36
- waits for the current stage to finish this iteration, then is
37
- SELECTed next.
38
- - Any open issue labeled **`urgent`** → run the canonical pause-vs-finish
39
- rubric in `docs/workflow/ORCHESTRATION.md` **by reference, never
40
- forked here** — feed it the issue content plus `detail.urgent
41
- .interruptibility` for the current in-flight unit.
42
- `INTERRUPT_NOW` → park the in-flight unit (WIP commit + `progress.md`
43
- note, same as any voluntary park) and SELECT the urgent issue's fix as
44
- this iteration's unit instead. `FINISH_FIRST` → this iteration
45
- proceeds normally (steps below); the urgent fix is queued head-of-line
46
- for the **next** iteration, same as `fix-next`.
47
- - No `urgent`/`fix-next` issues in `detail.urgent` → fall through to the
48
- normal priority order below.
49
- 2. **Blocking fixes first.** A fix-index entry classified fix-now whose
50
- subject blocks the next startable feature (same module, a dependency's
51
- defect, or a red gate cause) → its fix is the selected unit
52
- (`plan-fix` → `execute-phase --fix` through the normal stages). Fixes
53
- that block nothing wait for the report's triage batch.
54
- 3. An in-progress feature's next pending stage.
55
- 4. The next feature at status `idea` **or** `planned` whose depends-on rows
56
- are all **merged** (forge state, not merely `done` — a `done` dep with
57
- an open PR isn't on the default branch yet, so a dependent cut from it
58
- would lack its code). A `defined`-but-not-`planned` row is treated the
59
- same as `idea` here — its design exists but its planning artifacts
60
- don't, so it still needs a scaffold pass before PLAN.
61
- **Verify the closure transitively:** a dep row marked merged whose own
62
- dependencies aren't merged means the roadmap's statuses are inconsistent
63
- → `SHIP: STOPPED` (substrate invariant broken), never build on top of it.
64
- → `idea`/`defined`: DESIGN first (see ADVANCE). `planned`: → REVIEW-SPEC/
65
- REVIEW-PLAN if its receipt for that stage is missing or stale, else PLAN/EXECUTE
66
- directly.
67
- 5. **Issue sweep — features exhausted, run NOT over.** Every roadmap feature
68
- is `done` **and merged** but the sweep hasn't completed → the run
69
- continues with issues; finishing the features is not finishing the run:
70
- 1. **INVENTORY (once per run, its own iteration).** Enumerate (a) every
71
- open forge issue and fix-index entry; (b) every *documented residue*
72
- the run itself generated — each feature's `known-issues.md`, the
73
- trade-offs in `decisions.md`, and every review report's
74
- proposals/trade-offs. Residue without an existing issue remains a
75
- deduplicated **proposal** in the report; the run never creates backlog
76
- to make inventory look complete. Log existing issue numbers and
77
- proposal sources separately.
78
- 2. **TRIAGE (compose `triage-issue` in-turn, equal tier).** Classify
79
- each existing inventoried issue against the CURRENT codebase. fix-now → it
80
- becomes a selectable unit; postpone / wontfix / promote-to-feature →
81
- the dated verdict is recorded on the issue and carried into the
82
- report (promotions and untracked residue become report proposals,
83
- never in-run scope or automatically-created issues).
84
- 3. **SHIP the fix-now issues** one unit at a time through the normal
85
- stages (`plan-fix` → REVIEW-PLAN → EXECUTE (`--fix`) → PR → REVIEW → AUDIT), same
86
- budget caps, floors, and merge policy as features.
87
- 6. Nothing startable → `SHIP: BLOCKED` with the **unblock map** ("merging
88
- issue `#<n>` unblocks features `05` and `07`") and the resume command.
89
-
90
- `execute-phase`'s own dependency gate stays active inside every subagent —
91
- it's the belt to this braces. **The autopilot never passes `--force`:** a
92
- gate stop inside a subagent parks the feature with the unmet chain recorded;
93
- forcing through unmet dependencies is a human-only decision, made outside
94
- the loop.