@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.
- package/README.md +3 -9
- package/dist/conductor/closeout.d.ts +18 -0
- package/dist/conductor/closeout.js +24 -0
- package/dist/conductor/decide.d.ts +25 -0
- package/dist/conductor/decide.js +104 -0
- package/dist/conductor/index.d.ts +16 -0
- package/dist/conductor/index.js +15 -0
- package/dist/conductor/invoke.d.ts +15 -0
- package/dist/conductor/invoke.js +59 -0
- package/dist/conductor/loop.d.ts +25 -0
- package/dist/conductor/loop.js +236 -0
- package/dist/conductor/sensor.d.ts +12 -0
- package/dist/conductor/sensor.js +150 -0
- package/dist/conductor/snapshot.d.ts +13 -0
- package/dist/conductor/snapshot.js +60 -0
- package/dist/conductor/types.d.ts +248 -0
- package/dist/conductor/types.js +47 -0
- package/dist/conductor/urgency.d.ts +16 -0
- package/dist/conductor/urgency.js +70 -0
- package/dist/config/merge.js +21 -1
- package/dist/config/schema.js +45 -2
- package/dist/config/types.d.ts +22 -0
- package/dist/config/types.js +7 -0
- package/dist/extension/conductor-command.d.ts +23 -0
- package/dist/extension/conductor-command.js +93 -0
- package/dist/extension/factory.js +6 -1
- package/dist/routing/types.d.ts +2 -0
- package/dist/routing/types.js +2 -0
- package/dist/runtime.d.ts +45 -0
- package/dist/runtime.js +62 -0
- package/package.json +4 -2
- package/skills/audit-docs/SKILL.md +6 -5
- package/skills/audit-pr/SKILL.md +1 -1
- package/skills/audit-pr/references/PORTABILITY.md +1 -1
- package/skills/execute-phase/SKILL.md +207 -116
- package/skills/execute-phase/references/DESCOPE.md +22 -18
- package/skills/execute-phase/references/EXECUTION_CONTRACT.md +69 -77
- package/skills/execute-phase/references/FOLDING.md +34 -47
- package/skills/execute-phase/references/PREFLIGHT.md +103 -114
- package/skills/execute-phase/references/PRE_EXECUTION_GATE.md +53 -60
- package/skills/init-workspace/SKILL.md +35 -5
- package/skills/init-workspace/references/BOOTSTRAP_DISCOVERY.md +5 -5
- package/skills/init-workspace/references/BOOTSTRAP_WRITE.md +15 -8
- package/skills/init-workspace/references/PORTABILITY.md +1 -1
- package/skills/init-workspace/references/UPGRADE.md +17 -7
- package/skills/log-session/SKILL.md +4 -3
- package/skills/orchestration-envelope/references/TURN_CONTRACT.md +1 -1
- package/skills/pre-execution-review/references/LEDGERS.md +39 -39
- package/skills/pre-execution-review/references/POLICY.md +1 -0
- package/skills/review-change/SKILL.md +1 -1
- package/skills/review-change/references/PORTABILITY.md +1 -1
- package/skills/unit-lane/SKILL.md +154 -0
- package/skills/unit-lane/references/DESIGN.md +35 -0
- package/skills/unit-lane/references/DOCS.md +36 -0
- package/skills/unit-lane/references/EVIDENCE.md +34 -0
- package/skills/unit-lane/references/IMPLEMENT.md +45 -0
- package/skills/unit-lane/references/PLAN.md +38 -0
- package/skills/unit-lane/references/RELEASE.md +45 -0
- package/skills/unit-lane/references/RESEARCH.md +37 -0
- package/skills/unit-lane/references/REVIEW.md +45 -0
- package/skills/unit-lane/references/STEPS.md +23 -0
- package/skills/unit-lane/references/TESTS.md +32 -0
- package/skills/workflow-status/references/ENVELOPE_FIELDS.md +5 -6
- package/skills/workflow-status/references/SENSOR_CORE.md +16 -20
- package/skills/workflow-status/references/SENSOR_SIGNALS.md +5 -7
- package/skills/design-feature/SKILL.md +0 -231
- package/skills/design-feature/references/INTERVIEW.md +0 -74
- package/skills/design-feature/references/PORTABILITY.md +0 -15
- package/skills/design-feature/references/REPAIR.md +0 -88
- package/skills/design-feature/references/UPSERT_EXAMPLE.md +0 -8
- package/skills/design-feature/references/WRITE_AND_UPSERT.md +0 -115
- package/skills/evidence-grounding/SKILL.md +0 -174
- package/skills/evidence-grounding/references/DELEGATION.md +0 -131
- package/skills/evidence-grounding/references/READINESS.md +0 -130
- package/skills/evidence-grounding/references/ROWS.md +0 -93
- package/skills/execute-phase/references/BATCH_AND_PORTABILITY.md +0 -57
- package/skills/execute-phase/references/CLOSEOUT.md +0 -77
- package/skills/execute-phase/references/HANDOFF.md +0 -21
- package/skills/execute-phase/references/OPPORTUNISTIC_FINDING.md +0 -45
- package/skills/execute-phase/references/UNIT_LOOP.md +0 -88
- package/skills/execute-phase/references/WORKFLOWS_FEATURE.md +0 -37
- package/skills/execute-phase/references/WORKFLOWS_FIX.md +0 -43
- package/skills/execute-phase/references/WORKFLOWS_LEGACY.md +0 -26
- package/skills/execute-phase/references/WORKFLOWS_SMALL_PHASED.md +0 -38
- package/skills/generate-docs/SKILL.md +0 -145
- package/skills/generate-docs/references/ADAPTERS.md +0 -15
- package/skills/generate-docs/references/ADAPTER_DISCOVERY.md +0 -33
- package/skills/generate-docs/references/GENERATION_PROCESS.md +0 -90
- package/skills/implementation-discovery/SKILL.md +0 -209
- package/skills/plan-feature/SKILL.md +0 -208
- package/skills/plan-feature/references/ROUTING.md +0 -155
- package/skills/plan-feature-from-issue/SKILL.md +0 -162
- package/skills/plan-feature-scaffold/SKILL.md +0 -112
- package/skills/plan-feature-scaffold/references/SCAFFOLD_PROCESS.md +0 -122
- package/skills/plan-fix/SKILL.md +0 -161
- package/skills/plan-fix/references/PLANNING_PROCESS.md +0 -148
- package/skills/plan-fix/references/SPEC_CONTRACT.md +0 -22
- package/skills/planning-preflight/SKILL.md +0 -130
- package/skills/replan-findings/SKILL.md +0 -131
- package/skills/replan-findings/references/PHASE_APPEND.md +0 -89
- package/skills/review-plan/SKILL.md +0 -154
- package/skills/review-plan/references/CHECKS.md +0 -111
- package/skills/review-plan/references/ENG-CHECKS.md +0 -30
- package/skills/review-plan/references/OUTPUT.md +0 -166
- package/skills/review-spec/SKILL.md +0 -145
- package/skills/review-spec/references/CHECKS.md +0 -108
- package/skills/review-spec/references/OUTPUT.md +0 -160
- package/skills/ship-roadmap/SKILL.md +0 -120
- package/skills/ship-roadmap/references/ADVANCE.md +0 -117
- package/skills/ship-roadmap/references/AUDIT_AND_MERGE.md +0 -64
- package/skills/ship-roadmap/references/CLOSEOUT_AND_LOG.md +0 -27
- package/skills/ship-roadmap/references/FOUNDING.md +0 -101
- package/skills/ship-roadmap/references/GUARDRAILS.md +0 -56
- package/skills/ship-roadmap/references/MODEL_ROUTING.md +0 -16
- package/skills/ship-roadmap/references/PORTABILITY.md +0 -34
- package/skills/ship-roadmap/references/RECOVERY_AND_SELECTION.md +0 -94
- package/skills/ship-roadmap/references/STOP_CONDITIONS.md +0 -9
- 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.
|