flow-workflows 0.69.0 → 0.71.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 +4 -2
- package/adapters/codex/CORE.md +84 -12
- package/adapters/codex/skills/flow-bug-fix/SKILL.md +33 -3
- package/adapters/codex/skills/flow-bug-investigate/SKILL.md +1 -1
- package/adapters/codex/skills/flow-bug-postmortem/SKILL.md +1 -1
- package/adapters/codex/skills/flow-bug-review/SKILL.md +1 -1
- package/adapters/codex/skills/flow-bug-ship/SKILL.md +3 -3
- package/adapters/codex/skills/flow-bug-start/SKILL.md +2 -2
- package/adapters/codex/skills/flow-bug-validate/SKILL.md +1 -1
- package/adapters/codex/skills/flow-doctor/SKILL.md +1 -0
- package/adapters/codex/skills/flow-feat-build/SKILL.md +38 -6
- package/adapters/codex/skills/flow-feat-design/SKILL.md +1 -1
- package/adapters/codex/skills/flow-feat-plan/SKILL.md +2 -2
- package/adapters/codex/skills/flow-feat-review/SKILL.md +1 -1
- package/adapters/codex/skills/flow-feat-ship/SKILL.md +3 -3
- package/adapters/codex/skills/flow-feat-start/SKILL.md +4 -4
- package/adapters/codex/skills/flow-feat-validate/SKILL.md +1 -1
- package/adapters/codex/skills/flow-next/SKILL.md +1 -1
- package/adapters/codex/skills/flow-work-README/SKILL.md +9 -7
- package/adapters/codex/skills/flow-work-abandon/SKILL.md +2 -2
- package/adapters/codex/skills/flow-work-clean/SKILL.md +1 -1
- package/adapters/codex/skills/flow-work-daily/SKILL.md +1 -1
- package/adapters/codex/skills/flow-work-green/SKILL.md +1 -1
- package/adapters/codex/skills/flow-work-query/SKILL.md +1 -1
- package/adapters/codex/skills/flow-work-respond/SKILL.md +1 -1
- package/adapters/codex/skills/flow-work-resume/SKILL.md +3 -3
- package/adapters/codex/skills/flow-work-status/SKILL.md +1 -1
- package/adapters/codex/skills/flow-work-try/SKILL.md +1 -1
- package/adapters/codex/skills/flow-work-watch/SKILL.md +3 -3
- package/adapters/gemini/CORE.md +84 -12
- package/adapters/gemini/commands/flow/bug/fix.toml +33 -3
- package/adapters/gemini/commands/flow/bug/investigate.toml +1 -1
- package/adapters/gemini/commands/flow/bug/postmortem.toml +1 -1
- package/adapters/gemini/commands/flow/bug/review.toml +1 -1
- package/adapters/gemini/commands/flow/bug/ship.toml +3 -3
- package/adapters/gemini/commands/flow/bug/start.toml +2 -2
- package/adapters/gemini/commands/flow/bug/validate.toml +1 -1
- package/adapters/gemini/commands/flow/doctor.toml +1 -0
- package/adapters/gemini/commands/flow/feat/build.toml +38 -6
- package/adapters/gemini/commands/flow/feat/design.toml +1 -1
- package/adapters/gemini/commands/flow/feat/plan.toml +2 -2
- package/adapters/gemini/commands/flow/feat/review.toml +1 -1
- package/adapters/gemini/commands/flow/feat/ship.toml +3 -3
- package/adapters/gemini/commands/flow/feat/start.toml +4 -4
- package/adapters/gemini/commands/flow/feat/validate.toml +1 -1
- package/adapters/gemini/commands/flow/next.toml +1 -1
- package/adapters/gemini/commands/flow/work/README.toml +9 -7
- package/adapters/gemini/commands/flow/work/abandon.toml +2 -2
- package/adapters/gemini/commands/flow/work/clean.toml +1 -1
- package/adapters/gemini/commands/flow/work/daily.toml +1 -1
- package/adapters/gemini/commands/flow/work/green.toml +1 -1
- package/adapters/gemini/commands/flow/work/query.toml +1 -1
- package/adapters/gemini/commands/flow/work/respond.toml +1 -1
- package/adapters/gemini/commands/flow/work/resume.toml +3 -3
- package/adapters/gemini/commands/flow/work/status.toml +1 -1
- package/adapters/gemini/commands/flow/work/try.toml +1 -1
- package/adapters/gemini/commands/flow/work/watch.toml +3 -3
- package/adapters/hermes/CORE.md +84 -12
- package/adapters/hermes/skills/flow-bug-fix/SKILL.md +33 -3
- package/adapters/hermes/skills/flow-bug-investigate/SKILL.md +1 -1
- package/adapters/hermes/skills/flow-bug-postmortem/SKILL.md +1 -1
- package/adapters/hermes/skills/flow-bug-review/SKILL.md +1 -1
- package/adapters/hermes/skills/flow-bug-ship/SKILL.md +3 -3
- package/adapters/hermes/skills/flow-bug-start/SKILL.md +2 -2
- package/adapters/hermes/skills/flow-bug-validate/SKILL.md +1 -1
- package/adapters/hermes/skills/flow-doctor/SKILL.md +1 -0
- package/adapters/hermes/skills/flow-feat-build/SKILL.md +38 -6
- package/adapters/hermes/skills/flow-feat-design/SKILL.md +1 -1
- package/adapters/hermes/skills/flow-feat-plan/SKILL.md +2 -2
- package/adapters/hermes/skills/flow-feat-review/SKILL.md +1 -1
- package/adapters/hermes/skills/flow-feat-ship/SKILL.md +3 -3
- package/adapters/hermes/skills/flow-feat-start/SKILL.md +4 -4
- package/adapters/hermes/skills/flow-feat-validate/SKILL.md +1 -1
- package/adapters/hermes/skills/flow-next/SKILL.md +1 -1
- package/adapters/hermes/skills/flow-work-README/SKILL.md +9 -7
- package/adapters/hermes/skills/flow-work-abandon/SKILL.md +2 -2
- package/adapters/hermes/skills/flow-work-clean/SKILL.md +1 -1
- package/adapters/hermes/skills/flow-work-daily/SKILL.md +1 -1
- package/adapters/hermes/skills/flow-work-green/SKILL.md +1 -1
- package/adapters/hermes/skills/flow-work-query/SKILL.md +1 -1
- package/adapters/hermes/skills/flow-work-respond/SKILL.md +1 -1
- package/adapters/hermes/skills/flow-work-resume/SKILL.md +3 -3
- package/adapters/hermes/skills/flow-work-status/SKILL.md +1 -1
- package/adapters/hermes/skills/flow-work-try/SKILL.md +1 -1
- package/adapters/hermes/skills/flow-work-watch/SKILL.md +3 -3
- package/adapters/opencode/CORE.md +84 -12
- package/adapters/opencode/commands/flow-bug-fix.md +33 -3
- package/adapters/opencode/commands/flow-bug-investigate.md +1 -1
- package/adapters/opencode/commands/flow-bug-postmortem.md +1 -1
- package/adapters/opencode/commands/flow-bug-review.md +1 -1
- package/adapters/opencode/commands/flow-bug-ship.md +3 -3
- package/adapters/opencode/commands/flow-bug-start.md +2 -2
- package/adapters/opencode/commands/flow-bug-validate.md +1 -1
- package/adapters/opencode/commands/flow-doctor.md +1 -0
- package/adapters/opencode/commands/flow-feat-build.md +38 -6
- package/adapters/opencode/commands/flow-feat-design.md +1 -1
- package/adapters/opencode/commands/flow-feat-plan.md +2 -2
- package/adapters/opencode/commands/flow-feat-review.md +1 -1
- package/adapters/opencode/commands/flow-feat-ship.md +3 -3
- package/adapters/opencode/commands/flow-feat-start.md +4 -4
- package/adapters/opencode/commands/flow-feat-validate.md +1 -1
- package/adapters/opencode/commands/flow-next.md +1 -1
- package/adapters/opencode/commands/flow-work-README.md +9 -7
- package/adapters/opencode/commands/flow-work-abandon.md +2 -2
- package/adapters/opencode/commands/flow-work-clean.md +1 -1
- package/adapters/opencode/commands/flow-work-daily.md +1 -1
- package/adapters/opencode/commands/flow-work-green.md +1 -1
- package/adapters/opencode/commands/flow-work-query.md +1 -1
- package/adapters/opencode/commands/flow-work-respond.md +1 -1
- package/adapters/opencode/commands/flow-work-resume.md +3 -3
- package/adapters/opencode/commands/flow-work-status.md +1 -1
- package/adapters/opencode/commands/flow-work-try.md +1 -1
- package/adapters/opencode/commands/flow-work-watch.md +3 -3
- package/package.json +1 -1
- package/plugins/flow/.claude-plugin/plugin.json +1 -1
- package/plugins/flow/CHANGELOG.md +60 -0
|
@@ -18,7 +18,7 @@ description: "Monitor the observability platform after a deploy and alert on err
|
|
|
18
18
|
> - `knowledge.*` roles → whatever tools `FLOW.md` names there; an MCP tool keeps its name, its server is declared under `mcp_servers` in `~/.hermes/config.yaml` (see `config.snippet.yaml`).
|
|
19
19
|
> - `$ARGUMENTS` → what the user typed after the skill name: Hermes takes everything from the first non-skill token on as the instruction.
|
|
20
20
|
|
|
21
|
-
Read `~/.claude/flow/CORE.hermes.md` first (shared rules: `FLOW.md` step 0, models, autonomy modes and hard gates, how a stop reads,
|
|
21
|
+
Read `~/.claude/flow/CORE.hermes.md` first (shared rules: `FLOW.md` step 0, models, autonomy modes and hard gates, how a stop reads, the live panel, `00-summary.md`) — skip if you already read it in this session. **Models: this command runs with the model it was launched with; the wait it delegates takes `models.supervisors`.**
|
|
22
22
|
|
|
23
23
|
**Autopiloted post-deploy monitoring**: signals **scoped to the change** over a window (default 30 min), against a baseline; alert on errors or performance regressions from the deploy.
|
|
24
24
|
|
|
@@ -129,7 +129,7 @@ Over `[last cycle, now]`, scoped to the surface. **Default thresholds** (tunable
|
|
|
129
129
|
|
|
130
130
|
After each cycle: update `monitor.md` (accumulated state — no repeated alerts, feeds the final summary) and **reschedule with `ScheduleWakeup`** (~270-300s, or the chosen interval) passing the same `/flow-work-watch {PREFIX}XXXXX` until `T_end`. Platform fails or slow → retry next cycle, never break.
|
|
131
131
|
|
|
132
|
-
**Refresh the live panel every cycle** (only with a `<work-dir>` from §1)
|
|
132
|
+
**Refresh the live panel every cycle** (only with a `<work-dir>` from §1), by the transport flow-core §4 selects — the panel tools where the terminal has them, `.claude/work/<work>/panel.json` overwritten **whole** where it does not:
|
|
133
133
|
|
|
134
134
|
```json
|
|
135
135
|
{
|
|
@@ -170,7 +170,7 @@ Write the summary to `<work-dir>/monitor.md` (§1) and present it:
|
|
|
170
170
|
- **Evidence strength**: low traffic (§4) → say it ("green, but the flow barely executed during the window: weak evidence"). Never sell a zero-traffic green as a guarantee.
|
|
171
171
|
- **Honest limits**: no slow leaks, no regressions needing input not exercised in the window. A first-hour safety net.
|
|
172
172
|
|
|
173
|
-
Final verdict to
|
|
173
|
+
Final verdict to the panel too — `mark: "info"` plus `style: ok|warn|error` for 🟢/🟡/🔴 — and a `Now` line reading `nothing — the watch window is over`. On 🔴, a `Decision` line marked `wait` pointing at `/flow-bug-start`.
|
|
174
174
|
|
|
175
175
|
`knowledge.stage` is set → `knowledge.stage` relevant findings (measured baselines, low-traffic signals, error patterns) for this branch's staging.
|
|
176
176
|
|
|
@@ -16,7 +16,7 @@ Every `/flow-*` command assumes these rules. They are stated once, here, so a co
|
|
|
16
16
|
carries what is specific to its phase. Read this once per session; a command that says "load
|
|
17
17
|
`flow-core`" means this file.
|
|
18
18
|
|
|
19
|
-
**This file belongs to flow `0.
|
|
19
|
+
**This file belongs to flow `0.71.0`.** Compare it once, at the start of the session, against
|
|
20
20
|
`version` in `~/.claude/flow/.claude-plugin/plugin.json`. The two differing means the session
|
|
21
21
|
is running a **mixture** — the commands from one copy of the plugin, these shared rules from another
|
|
22
22
|
— which is exactly what happens when a branch or a release candidate is loaded over an installed
|
|
@@ -235,7 +235,7 @@ terminal by construction, the turn is over before you notice.
|
|
|
235
235
|
- **The continuing call goes first.** When a step closes and the next one runs anyway
|
|
236
236
|
(`guided`/`auto` chaining, §2), the turn opens with the call that starts it, not with prose about
|
|
237
237
|
the one that ended. **A report is never the last thing in a turn you meant to continue.**
|
|
238
|
-
- **Progress belongs to the panel, not to a turn.** Someone watching reads
|
|
238
|
+
- **Progress belongs to the panel, not to a turn.** Someone watching reads the live panel (§4):
|
|
239
239
|
`Now`, `Next`. Writing it costs no turn and no stop; saying it in chat costs both.
|
|
240
240
|
- **Stopping? Then say so.** `I need:` names the decision or the action you are waiting on. It
|
|
241
241
|
never announces that you are continuing — if you were, you would not be writing this header.
|
|
@@ -259,11 +259,19 @@ I need: <one line — the decision or action you are waiting on>
|
|
|
259
259
|
- **Zero-context rule.** First mention of an identifier carries 4–6 words of what it is. Never cite a section number without naming what it is. No jargon the user has not used first.
|
|
260
260
|
- **If it is a question, it is `AskUserQuestion`.** Never end a message with a question in prose. If it does not deserve the menu, it is a decision you take and record.
|
|
261
261
|
|
|
262
|
-
## 4. Live panel — `panel.json`
|
|
262
|
+
## 4. Live panel — over the terminal's MCP, or `panel.json`
|
|
263
263
|
|
|
264
|
-
Whenever the state a panel would show changes,
|
|
265
|
-
**
|
|
266
|
-
|
|
264
|
+
Whenever the state a panel would show changes, publish the panel from `meta.json` plus what you know
|
|
265
|
+
right now. **Any progress line you were about to type into the chat is a panel update instead**
|
|
266
|
+
(§3).
|
|
267
|
+
|
|
268
|
+
**Two transports, one document.** When the tools `panel_set` / `panel_patch` / `panel_get` are
|
|
269
|
+
available, this console is a pane of agent-terminal: publish the panel through them (§4.2) and
|
|
270
|
+
**stop writing `.claude/work/<work>/panel.json`** — while a pane is claimed, its file is not read at
|
|
271
|
+
all. When the tools are not there, write the file exactly as before (§4.3). No work folder
|
|
272
|
+
(lightweight `respond`/`green`) → nothing to publish, either way.
|
|
273
|
+
|
|
274
|
+
### 4.1 The document — the same whichever transport carries it
|
|
267
275
|
|
|
268
276
|
```json
|
|
269
277
|
{
|
|
@@ -289,11 +297,75 @@ about to type into the chat is a panel write instead** (§3):
|
|
|
289
297
|
- **`mark` says what a line is**; the reader draws it. `done` · `current` (at most one) · `pending` · `wait` (waiting on someone else: an open MR/PR, a user decision) · `block` · `info`. Marked lines form an aligned column: symbol, `ref`, text, link pinned right. Do not set `style` on a marked line — except `mark: "info"` + `style: ok|warn|error` when the colour *is* the information (a monitoring verdict).
|
|
290
298
|
- **`ref` need not be a number**: `#1`, `#3–#6`, `Now`, `Next`, `Decision`. Column width is per block; blocks are separated by blank lines — use them so the MR/PR train and the labels below do not drag each other wide.
|
|
291
299
|
- **`link` is a field, never a URL inside `text`.**
|
|
292
|
-
- **Order:** (1) work title (`style: title`, no mark); (2) the MR/PR train, one entry per `meta.json.mrs[]` with `ref` `#n`, short title, real-state `mark`, `link` when there is a URL — not-started entries collapse into one `#a–#z` `pending` line; omit the block without `mrs`; (3) `Now` — what is running, the one fact `meta.json` cannot hold; (4) `Next`; (5) `Decision`, `mark: wait`, **only** when parked on the user; (6) blockers, `mark: block` (a sibling repo with `contract_handoff: pending`, a red pipeline, an unmerged dependency).
|
|
293
|
-
- **
|
|
294
|
-
|
|
300
|
+
- **Order:** (1) work title (`style: title`, no mark); (2) the MR/PR train, one entry per `meta.json.mrs[]` with `ref` `#n`, short title, real-state `mark`, `link` when there is a URL — not-started entries collapse into one `#a–#z` `pending` line; omit the block without `mrs`; (3) `Now` — what is running, the one fact `meta.json` cannot hold; (4) `Next`; (5) `Decision`, `mark: wait`, **only** when parked on the user — over the tools it travels with `attention: "wait"` (§4.2), and both are cleared together; (6) blockers, `mark: block` (a sibling repo with `contract_handoff: pending`, a red pipeline, an unmerged dependency).
|
|
301
|
+
- **Rules:** `phase` is the phase you are **running now** (not `meta.json.phase`, which advances only at Close). `header: true` means ticket, type, phase and age are drawn by the reader — do not repeat them in `lines`. Keep it under ~14 lines. Every fact from `meta.json` and the artifacts — an invented MR/PR state is worse than a blank panel. Write in the language of the work's artifacts.
|
|
302
|
+
|
|
303
|
+
### 4.2 Over the terminal's MCP — `panel_set` once, `panel_patch` from then on
|
|
304
|
+
|
|
305
|
+
**The first publication of the session — `panel_set`.** `source: "flow"`. It claims the pane for as
|
|
306
|
+
long as this session lives, and a second producer is refused by name. The document is the one §4.1
|
|
307
|
+
specifies, plus two additions:
|
|
308
|
+
|
|
309
|
+
- **`attention`, at the top level**, beside `phase`: `wait` when parked on the user, `block` when
|
|
310
|
+
stuck on somebody else (red pipeline, sibling repo's contract), `done` when the cycle closed and
|
|
311
|
+
nobody has looked. Absent while simply working. The app never deduces it: a `mark: "wait"` inside
|
|
312
|
+
`lines` is body text and claims nothing. The tab's card shows the strongest claim among its panes,
|
|
313
|
+
so a `wait` you forget to clear flags that tab for ever — clear it with
|
|
314
|
+
`{"op": "set", "field": "attention", "value": null}`.
|
|
315
|
+
- **A `key` on every line you may want to change later** — `title`, `mr-1`, `now`, `next`,
|
|
316
|
+
`decision`, `block-1`. It is never painted, and it must be unique: a key that names two lines
|
|
317
|
+
refuses every patch addressing it.
|
|
318
|
+
|
|
319
|
+
**Everything after that is `panel_patch`:**
|
|
320
|
+
|
|
321
|
+
```json
|
|
322
|
+
{"ops": [
|
|
323
|
+
{"op": "replace", "key": "now",
|
|
324
|
+
"line": {"key": "now", "ref": "Now", "text": "unit suite over #2", "mark": "info"}},
|
|
325
|
+
{"op": "set", "field": "phase", "value": "validate"},
|
|
326
|
+
{"op": "set", "field": "attention", "value": "wait"}]}
|
|
327
|
+
```
|
|
328
|
+
|
|
329
|
+
- Ops: `replace`, `before`, `after`, `delete` (by `key`), `first`, `last`, and `set` for `phase`,
|
|
330
|
+
`attention`, `header` or `stale_after_minutes`. They see each other, in the order written.
|
|
331
|
+
- **Either the whole list applies or none of it does**, so a patch is safe to send without reading
|
|
332
|
+
first: the worst case is that nothing happened and you were told why.
|
|
333
|
+
- **A patch whose answer never arrived may have been applied — send it again.** The resend is
|
|
334
|
+
refused rather than doubling the line.
|
|
335
|
+
- At most 64 ops per call. `actions` are not patchable: change buttons with a whole `panel_set`.
|
|
336
|
+
- Republish with `panel_set` only when the *shape* changes (a new MR/PR train, a different body).
|
|
337
|
+
|
|
338
|
+
**Refusals are information, not noise.** `no line of this panel carries the key "…"` → your keys and
|
|
339
|
+
the panel disagree: `panel_get` and republish. `publish with panel_set before patching` → this pane
|
|
340
|
+
is still on its file. `this panel is held by "…"` → another producer owns it; do not fight it, say
|
|
341
|
+
so at the next stop.
|
|
342
|
+
|
|
343
|
+
**After a compaction, a restart or a resumed work**, call `panel_get` **before publishing**. It
|
|
344
|
+
returns the document on screen — yours or one that arrived by file — and publishing it back leaves
|
|
345
|
+
the panel exactly as it is. What comes back is the document as the app understood it: unknown fields
|
|
346
|
+
are gone and defaults come back absent.
|
|
347
|
+
|
|
348
|
+
**Optional — buttons at a stop.** At a stop parked on the user, `actions` (up to 8) paint a button
|
|
349
|
+
whose command is shown literally and is written into this pane's program on one click — which here
|
|
350
|
+
is this very session's prompt. So `{"label": "Approve the design", "send": "approve"}` answers you
|
|
351
|
+
without the user typing. Plain text, no line breaks, never a credential: it is painted on screen and
|
|
352
|
+
lands in the shell history.
|
|
353
|
+
|
|
354
|
+
### 4.3 Without those tools — `.claude/work/<work>/panel.json`
|
|
355
|
+
|
|
356
|
+
Overwrite the file **whole** (never patch) with the §4.1 document. `updated_at` from the real clock
|
|
357
|
+
(`date -Iseconds`), never carried over — it is how the reader flags a stale snapshot. `attention`
|
|
358
|
+
and `key` buy nothing here: the file panel is drawn from `mark` and `ref` alone.
|
|
359
|
+
|
|
360
|
+
### 4.4 When to publish — whichever transport is in play
|
|
361
|
+
|
|
362
|
+
(a) In pre-flight, as soon as `meta.json` is loaded; (b) immediately **before** every stop header;
|
|
363
|
+
(c) **before** any long stretch (fan-out, full suite, CI poll) — never after, so a step that dies
|
|
364
|
+
halfway is not shown as finished. When the stretch will outlast the ~30 min staleness warning, set
|
|
365
|
+
`stale_after_minutes` to what it will really take; (d) wherever `## Close` updates `meta.json`. A
|
|
366
|
+
held panel still goes stale: the app stamps each delivery's arrival.
|
|
295
367
|
|
|
296
|
-
### `meta.json.cost[]` — what a phase measured, when it could
|
|
368
|
+
### 4.5 `meta.json.cost[]` — what a phase measured, when it could
|
|
297
369
|
|
|
298
370
|
A phase that can measure what it spent appends one entry: `{ "phase": "review", "at": "<ISO>",
|
|
299
371
|
"reviewers": 8, "usd": 0.0124, "source": "harness" }`. `flow review --record` writes it; `flow cost`
|
|
@@ -321,7 +393,7 @@ single largest token cost of a work, so every work carries a short handoff:
|
|
|
321
393
|
- **Name the phase, never the command.** An artifact outlives the session that wrote it and may be
|
|
322
394
|
resumed from a different agent, where the same phase is invoked under another name. Write the
|
|
323
395
|
phase — `validate`, `review`, `ship #2` — not the invocation. Applies to `00-summary.md`,
|
|
324
|
-
|
|
396
|
+
the panel's `Next` line and every numbered artifact; the prose you *say* at a stop is exempt.
|
|
325
397
|
|
|
326
398
|
## 6. Delegating — what every brief owes the parent
|
|
327
399
|
|
|
@@ -444,7 +516,7 @@ and the evidence; the verdict stays here.
|
|
|
444
516
|
bills and saves nothing. What wakes you is whatever the harness offers — a background task that
|
|
445
517
|
re-invokes you when it finishes, a scheduled wake-up, a message back to this thread; where it offers
|
|
446
518
|
none, the supervisor's evidence file is read at your next stop, and **you say that** rather than
|
|
447
|
-
implying something is being watched. Refresh
|
|
519
|
+
implying something is being watched. Refresh the panel before you go (§4): the line carries
|
|
448
520
|
`mark: "wait"` and `stale_after_minutes` set to the deadline you gave, because a thread that goes
|
|
449
521
|
quiet with `current` on screen reads as a session that died.
|
|
450
522
|
|
|
@@ -20,7 +20,7 @@ Apply the fix. **Minimum viable**: no refactor of adjacent areas. Note other pro
|
|
|
20
20
|
|
|
21
21
|
## 1. Pre-flight
|
|
22
22
|
|
|
23
|
-
Read `~/.claude/flow/CORE.opencode.md` first (shared rules: `FLOW.md` step 0, models, autonomy modes and hard gates, how a stop reads,
|
|
23
|
+
Read `~/.claude/flow/CORE.opencode.md` first (shared rules: `FLOW.md` step 0, models, autonomy modes and hard gates, how a stop reads, the live panel, `00-summary.md`) — skip if you already read it in this session. **Models: the subagents it launches take `models.agents`.**
|
|
24
24
|
|
|
25
25
|
- Read `meta.json` and `00-summary.md`; open in full only `03-investigation.md` (root cause, constraints for the fix) (minimal reproduction, root cause, constraints for the fix). (flow-core §5)
|
|
26
26
|
- `size` `XS`: may start without `investigate`, but require a 2-3 line description of the fix.
|
|
@@ -46,7 +46,7 @@ What is NOT touched:
|
|
|
46
46
|
|
|
47
47
|
**Ask with `AskUserQuestion`** whether this reflects the expected fix — a hard gate, **in every mode, `auto` included**: widening the fix is the main source of collateral regressions, and this is the last point to settle scope before there is a diff.
|
|
48
48
|
|
|
49
|
-
In `
|
|
49
|
+
In `auto` this is one of the only two stops of the whole flow (this one and `ship`; `guided` adds the plan of §2.0bis on a fix of more than two steps): open with the **full stop header** (flow-core §3), then the brief. Options:
|
|
50
50
|
- **Yes, go ahead** → apply the fix.
|
|
51
51
|
- **No, something is missing or wrong** → adjust the brief, ask again. No code until confirmed.
|
|
52
52
|
|
|
@@ -58,11 +58,38 @@ blockers lived in the half a third option let in.
|
|
|
58
58
|
|
|
59
59
|
Save the brief at the top of `04-fix.md`. Any "also fix X while we're at it" during implementation → §2.3.
|
|
60
60
|
|
|
61
|
+
## 2.0bis Plan the fix (more than 2 steps)
|
|
62
|
+
|
|
63
|
+
A fix of one or two steps is its own plan, and a planning round on it is pure friction. Past that the steps stop fitting in the brief — which says what stops happening, never in which file — and the fix starts being discovered while it is typed. **Trigger: more than 2 fix steps.** Two or fewer → skip, record one line in `04-fix.md` (`plan: skipped — 2 steps`) and go to §2.1.
|
|
64
|
+
|
|
65
|
+
The steps come from the root cause in `03-investigation.md`, not from the repo you are about to open: a step that cause never implied is either the investigation being wrong (§4.1) or scope that belongs in §2.3. **No code yet.** Under 200 words:
|
|
66
|
+
|
|
67
|
+
```markdown
|
|
68
|
+
## Plan
|
|
69
|
+
1. <step> — `<file or path>` — <the test that proves it, or "covered by step M">
|
|
70
|
+
2. …
|
|
71
|
+
|
|
72
|
+
- **Point of no return**: <the migration, contract or published event after which rollback stops being a revert — or "none">.
|
|
73
|
+
- **Out of this plan**: <the adjacent thing that will be tempting once these files are open>.
|
|
74
|
+
```
|
|
75
|
+
|
|
76
|
+
Rules, and the first one is the bug flow's own:
|
|
77
|
+
- **The regression test is a step, and it comes first when it can fail before the fix.** `/flow-bug-validate` is what proves the fix, and `/flow-bug-review` refuses to run without it on size ≥ S (`/flow-bug-start §2`). A plan that leaves the test until last has the one step that can still say "the cause was not this" arriving after everything is already built on the answer.
|
|
78
|
+
- **A step is a thing on disk**, not a phase of thought: "add the idempotency key on the retry path", not "look at the retry logic".
|
|
79
|
+
- **A step that widens the blast radius is not part of the minimal fix** (§2.1) — it goes to §2.3 before it is written down here, not after it is built.
|
|
80
|
+
- A fix whose plan needs more than about five steps is usually two tickets or a feature in disguise: check §4.2 before planning further.
|
|
81
|
+
|
|
82
|
+
**Present it, then behave by mode:**
|
|
83
|
+
- `manual` / `guided` → **stop**: full stop header (flow-core §3), the plan, one `AskUserQuestion` — **Approve and fix** · **Change the order or the steps**. On a bug the order is the decision: it is what settles whether the proof comes before the repair.
|
|
84
|
+
- `auto` → **do not stop**. Print the numbered steps and the point of no return, record the plan as accepted, and open the same message with the call that starts §2.1 (flow-core §3: a report is never the last thing in a turn you meant to continue).
|
|
85
|
+
|
|
86
|
+
Save it in `04-fix.md` under `## Plan`, right after the brief. **A fix that was going to be one step and turns into a third**: come back here and write it then — the trigger is the shape of the work, not the moment it was first guessed.
|
|
87
|
+
|
|
61
88
|
## 2.1 Work
|
|
62
89
|
|
|
63
90
|
- Apply the minimal fix targeting the finding of `03-investigation.md` (or the diagnosis if investigate was skipped).
|
|
64
91
|
- Sensitive area (authentication, payments, sensitive data) → consult the `agents.architecture` agent from FLOW.md to confirm the correct layer; empty → check against `conventions` in FLOW.md.
|
|
65
|
-
- More than 2 fix steps →
|
|
92
|
+
- More than 2 fix steps → the §2.0bis plan is already written: track its numbered steps with `TaskCreate` where the harness offers one, otherwise tick them off in `04-fix.md` where they stand. One task per step, same order, marked `in_progress` and `completed` as they go — not batched.
|
|
66
93
|
- **Comment discipline**: comment only a *why* the code cannot say (a non-obvious constraint, the reason for a workaround, a subtle invariant); match the surrounding file's comment density. **Never write the ticket ID or "fix for #N" into a code comment** — that lives in the commit/branch/MR-PR.
|
|
67
94
|
- Keep the log updated while editing.
|
|
68
95
|
|
|
@@ -116,6 +143,9 @@ A **product decision** the fix turns out to need is neither of these — not sco
|
|
|
116
143
|
**What is NOT touched**:
|
|
117
144
|
- <adjacent areas out of scope>
|
|
118
145
|
|
|
146
|
+
## Plan
|
|
147
|
+
<the numbered steps of §2.0bis, its point of no return and its "out of this plan" line — or one line saying the step was skipped and why (`plan: skipped — 2 steps`). What §2.1 tracks with `TaskCreate`. Steps are annotated as they are reworked, never deleted: a plan edited to match what happened records nothing.>
|
|
148
|
+
|
|
119
149
|
## Fix description
|
|
120
150
|
<one sentence: "The fix consists of …">
|
|
121
151
|
|
|
@@ -20,7 +20,7 @@ Investigation phase: reproduce the failure, then find **why it happened** — no
|
|
|
20
20
|
|
|
21
21
|
## 1. Pre-flight
|
|
22
22
|
|
|
23
|
-
Read `~/.claude/flow/CORE.opencode.md` first (shared rules: `FLOW.md` step 0, models, autonomy modes and hard gates, how a stop reads,
|
|
23
|
+
Read `~/.claude/flow/CORE.opencode.md` first (shared rules: `FLOW.md` step 0, models, autonomy modes and hard gates, how a stop reads, the live panel, `00-summary.md`) — skip if you already read it in this session. **Models: the subagents it launches take `models.agents`; its parallel rounds take `models.workers`.**
|
|
24
24
|
|
|
25
25
|
- Read `meta.json` and `00-summary.md`; open in full only `01-context.md`. (flow-core §5)
|
|
26
26
|
- `type` is not `bug` → refuse. `size` is `XS` → suggest `/flow-bug-fix` and stop.
|
|
@@ -22,7 +22,7 @@ Goal: capture lessons that prevent recurrence — not to blame anyone.
|
|
|
22
22
|
|
|
23
23
|
## 1. Pre-flight
|
|
24
24
|
|
|
25
|
-
Read `~/.claude/flow/CORE.opencode.md` first (shared rules: `FLOW.md` step 0, models, autonomy modes and hard gates, how a stop reads,
|
|
25
|
+
Read `~/.claude/flow/CORE.opencode.md` first (shared rules: `FLOW.md` step 0, models, autonomy modes and hard gates, how a stop reads, the live panel, `00-summary.md`) — skip if you already read it in this session. **Models: this command runs with the model it was launched with (no `models` key).**
|
|
26
26
|
|
|
27
27
|
- Read `meta.json` and `00-summary.md`; open in full every previous artifact (`01-context.md` … `06-review.md`) — §2 needs the whole history. (flow-core §5)
|
|
28
28
|
- Require `review` in `phases_done`.
|
|
@@ -20,7 +20,7 @@ Mandatory code review of the fix.
|
|
|
20
20
|
|
|
21
21
|
## 1. Pre-flight
|
|
22
22
|
|
|
23
|
-
Read `~/.claude/flow/CORE.opencode.md` first (shared rules: `FLOW.md` step 0, models, autonomy modes and hard gates, how a stop reads,
|
|
23
|
+
Read `~/.claude/flow/CORE.opencode.md` first (shared rules: `FLOW.md` step 0, models, autonomy modes and hard gates, how a stop reads, the live panel, `00-summary.md`) — skip if you already read it in this session. **Models: the subagents it launches take `models.agents`; its parallel rounds take `models.workers`.**
|
|
24
24
|
|
|
25
25
|
- Read `meta.json` and `00-summary.md`; open in full only `03-investigation.md` and `04-fix.md` (reviewer context, §2.1) and `05-validation.md` (regression test). (flow-core §5)
|
|
26
26
|
- **A premise left open by `fix` §2.2 is an input to this review, not a note.** Every row of "Premises the fix depends on" whose verdict is `unsettled` enters §5 as an **ambiguous finding** and is named in §7 whether or not that gate opened — in a bug it is usually the claim that the root cause was the whole cause, which is exactly what the regression test cannot prove.
|
|
@@ -23,7 +23,7 @@ Close the bug flow: commit, push, MR/PR. Same mechanics as `/flow-feat-ship`, wi
|
|
|
23
23
|
|
|
24
24
|
## 0. Pre-flight
|
|
25
25
|
|
|
26
|
-
Read `~/.claude/flow/CORE.opencode.md` first (shared rules: `FLOW.md` step 0, models, autonomy modes and hard gates, how a stop reads,
|
|
26
|
+
Read `~/.claude/flow/CORE.opencode.md` first (shared rules: `FLOW.md` step 0, models, autonomy modes and hard gates, how a stop reads, the live panel, `00-summary.md`) — skip if you already read it in this session. **Models: this command runs with the model it was launched with; the wait it delegates takes `models.supervisors`.**
|
|
27
27
|
|
|
28
28
|
- Read `meta.json` and `00-summary.md`; open in full only `04-fix.md` (the Brief), `06-review.md` (the verdict), `05-validation.md`, the §"Root cause identified" of `03-investigation.md`, and `99-postmortem.md` if it exists. (flow-core §5)
|
|
29
29
|
- Require `review` in `phases_done`; also `validate` if `size` ≥ S, and `postmortem` if `size` is M or L (the same sizes `/flow-bug-review §8` routes to postmortem — one rule). Missing → refuse and redirect to the missing step.
|
|
@@ -156,7 +156,7 @@ Only here — with the content approved in §2 — invoke `Skill commit-commands
|
|
|
156
156
|
|
|
157
157
|
Assign to `git.assignee` from FLOW.md (empty → unassigned). Squash per `git.squash`.
|
|
158
158
|
|
|
159
|
-
**Record the URL the moment it exists**: write it into `meta.json` (the `mrs` entry if there is one) and refresh
|
|
159
|
+
**Record the URL the moment it exists**: write it into `meta.json` (the `mrs` entry if there is one) and refresh the panel (flow-core §4) right here, before §3.2 and before anything else can fail.
|
|
160
160
|
|
|
161
161
|
**The pipeline the push starts is not watched from this thread — `/flow-feat-ship` §4.1, in full and unchanged**: close with CI reported as *running*, or delegate the wait to a supervisor subagent on `models.supervisors` (flow-core §6.5) with a deadline, taking back status and failing-job evidence only. A red pipeline is `/flow-work-green`'s to triage, never this command's.
|
|
162
162
|
|
|
@@ -175,7 +175,7 @@ nothing was measured. A slowness bug is the case where this comment *is* the pro
|
|
|
175
175
|
|
|
176
176
|
- **A worktree does not own the tree its tools wrote into — `/flow-feat-ship` §5.5, in full and unchanged.** First, before anything below: `meta.json.worktree` not null → read the main checkout's `git status --porcelain`, name any paths it holds in one line as what they are (in the main checkout, not in the branch just pushed, and not necessarily put there by this run), and offer once to carry them over. A fix consolidates its knowledge one phase earlier, in `/flow-bug-postmortem` §4, so by the time this runs those writes have already happened and the branch has already been pushed — which is exactly the window this check exists to close.
|
|
177
177
|
- Update `meta.json`: add `ship` to `phases_done`, update `updated_at`. **`phase` becomes `done` only once the MR/PR is confirmed merged** — creating it is not finishing it. Right after §3.1 it is open, so ask once with `AskUserQuestion` whether it has already been merged: **no** (the normal answer) → leave `phase = "ship"`; **yes** → `phase = "done"`. `/flow-work-green` and `/flow-work-respond` work on the open MR/PR; `/flow-work-clean` only sweeps what merged.
|
|
178
|
-
- Refresh
|
|
178
|
+
- Refresh the panel: while the MR/PR is open its line is `wait` with the `link`; once merged it is `done`, and only then say there is nothing left here (`mark: "info"`, `style: "ok"`). Drop the `Decision` line either way.
|
|
179
179
|
- Overwrite `00-summary.md` whole (≤15 lines, flow-core §5).
|
|
180
180
|
- **Tracker: move to done.** **Only when this ship set `phase = "done"`** (merge confirmed just above) and `tracker.tool` is not `none`/empty, `tracker.done_cmd` is set, and `meta.json.ticket` is a **real tracker id**: run `tracker.done_cmd` with `{TICKET}` = `meta.json.ticket`. Never on an open MR/PR. If the answer was "not merged", skip silently — the transition belongs to whoever confirms the merge (`/flow-work-clean` reports the branch as merged, and re-running this close is legitimate). Same contract as `/flow-bug-start §4.5`: **best-effort, idempotent, gated** (ask once in `autonomy.mode: manual`; automatic in `guided`/`auto`); failure or already-done ticket → warn and continue, never block. **Leave `tracker.done_cmd` empty on GitHub/GitLab** — `Closes #N` already auto-closes on merge; this is for Jira/Linear-style trackers.
|
|
181
181
|
- Summarize: ticket, MR/PR URL, regression test added.
|
|
@@ -23,7 +23,7 @@ Start a bug. `$ARGUMENTS` is **optional**:
|
|
|
23
23
|
|
|
24
24
|
## 0. Pre-flight
|
|
25
25
|
|
|
26
|
-
Read `~/.claude/flow/CORE.opencode.md` first (shared rules: `FLOW.md` step 0, models, autonomy modes and hard gates, how a stop reads,
|
|
26
|
+
Read `~/.claude/flow/CORE.opencode.md` first (shared rules: `FLOW.md` step 0, models, autonomy modes and hard gates, how a stop reads, the live panel, `00-summary.md`) — skip if you already read it in this session. **Models: this command runs with the model it was launched with (no `models` key).**
|
|
27
27
|
|
|
28
28
|
- Verify you are in the correct repo.
|
|
29
29
|
- **Determine the mode** from `$ARGUMENTS`: non-empty → *ticket mode* (identifier = `$ARGUMENTS`); empty → *ticket-less mode* (identifier = the slug resolved in §1.5).
|
|
@@ -160,7 +160,7 @@ Create the work directory per §0: `.claude/work/<TICKET>-<slug>/` in ticket mod
|
|
|
160
160
|
}
|
|
161
161
|
```
|
|
162
162
|
|
|
163
|
-
|
|
163
|
+
**Live panel**: publish it now (flow-core §4 — the panel tools, or `<work-dir>/panel.json`). A bug has no MR/PR train, so it is short: the title (the symptom, `style: title`), a `Now` line for what is starting, a `Next` line for the phase this size routes to, and any sibling repo from §1.6 as a `block` line. Every later phase updates it.
|
|
164
164
|
|
|
165
165
|
Populate `related_repos` from §1.6 — one `{ "repo": "<name>", "scope": "<one line>", "status": "pending", "contract_handoff": "pending" | "none" }` per *other* repo the fix touches; `[]` for a single-repo fix. `pending` only when the fix **changes a surface that sibling consumes** (a payload key, an error code, a route, an event shape); otherwise `none`.
|
|
166
166
|
|
|
@@ -20,7 +20,7 @@ Validate that the fix works and that the bug does not return.
|
|
|
20
20
|
|
|
21
21
|
## 1. Pre-flight
|
|
22
22
|
|
|
23
|
-
Read `~/.claude/flow/CORE.opencode.md` first (shared rules: `FLOW.md` step 0, models, autonomy modes and hard gates, how a stop reads,
|
|
23
|
+
Read `~/.claude/flow/CORE.opencode.md` first (shared rules: `FLOW.md` step 0, models, autonomy modes and hard gates, how a stop reads, the live panel, `00-summary.md`) — skip if you already read it in this session. **Models: the subagents it launches take `models.agents`; the wait it delegates takes `models.supervisors`.**
|
|
24
24
|
|
|
25
25
|
- Read `meta.json` and `00-summary.md`; open in full only `03-investigation.md` (§3 areas with similar risk, and the minimal reproduction) — the testing agent reads it and `04-fix.md` itself. (flow-core §5)
|
|
26
26
|
- Require `fix` in `phases_done`.
|
|
@@ -197,6 +197,7 @@ refuse) · **degraded** (it runs, quietly worse than the config promises), plus
|
|
|
197
197
|
`.agents/agents`, or a plugin. Report each missing name and that it falls back to `general-purpose`
|
|
198
198
|
(a skill: skipped) — a panel missing agents reviews less and still reports a clean pass.
|
|
199
199
|
- Harness cannot set a model per subagent → note that every `models.*` value degrades to inheritance.
|
|
200
|
+
- **Where the live panel will go this session** (flow-core §4): `panel_set` / `panel_patch` exposed → the terminal draws the panel and no `panel.json` is written; absent → the panel is that file, as before. Neither is a fault — one line saying which one this session gets.
|
|
200
201
|
|
|
201
202
|
### 2.3 Knowledge sources
|
|
202
203
|
|
|
@@ -16,7 +16,7 @@ description: Implement the feature following the approved design and keep a runn
|
|
|
16
16
|
> - `/model <value>` → opencode's model picker (`/models`).
|
|
17
17
|
> - `knowledge.*` roles → whatever tools `FLOW.md` names there; an MCP tool keeps its name, its server is declared in `opencode.json` (see this adapter's `opencode.json` for the domain-memory example).
|
|
18
18
|
|
|
19
|
-
Read `~/.claude/flow/CORE.opencode.md` first (shared rules: `FLOW.md` step 0, models, autonomy modes and hard gates, how a stop reads,
|
|
19
|
+
Read `~/.claude/flow/CORE.opencode.md` first (shared rules: `FLOW.md` step 0, models, autonomy modes and hard gates, how a stop reads, the live panel, `00-summary.md`) — skip if you already read it in this session. **Models: the subagents it launches take `models.agents`.**
|
|
20
20
|
|
|
21
21
|
Implementation phase. Code is written here.
|
|
22
22
|
|
|
@@ -57,7 +57,7 @@ Rules for writing it:
|
|
|
57
57
|
|
|
58
58
|
**Ask the user with `AskUserQuestion`** whether the brief reflects what they expect — **hard gate, in every autonomy mode, `auto` included**: it is the last point where scope can be corrected before there is a diff to argue with.
|
|
59
59
|
|
|
60
|
-
In `
|
|
60
|
+
In `auto` this is one of the only two stops per MR/PR (`guided` adds the plan of §2.0ter on a big one), so it carries the **full stop header** (flow-core §3) — ticket, size, phase, `MR #<n> of <N>`, plan state — followed by the brief. Options:
|
|
61
61
|
- **Yes, proceed** → start building.
|
|
62
62
|
- **No, something is extra or missing** → the user clarifies, adjust the brief, ask again. **Do not touch code** until the brief is confirmed.
|
|
63
63
|
|
|
@@ -82,7 +82,36 @@ Hard rules:
|
|
|
82
82
|
- **If there is a "Pattern deviation" section**: copy it too — it reminds you at coding time not to mimic the repo pattern.
|
|
83
83
|
- **If the design says "none"** (no external surfaces), skip this step and record: "## Contracts to respect — none declared in design".
|
|
84
84
|
|
|
85
|
-
Without this copy, do not proceed to §2.
|
|
85
|
+
Without this copy, do not proceed to §2.0ter.
|
|
86
|
+
|
|
87
|
+
## 2.0ter Plan this MR/PR (size-gated)
|
|
88
|
+
|
|
89
|
+
`03-design.md` §"Implementation plan (order)" is the **feature's** order, written before `/flow-feat-plan` split anything — it does not know which of its steps belong to this MR/PR. The brief above says what this MR/PR delivers in business terms; neither says which files are touched in which order. On a MR/PR big enough to lose its thread, that plan is written here.
|
|
90
|
+
|
|
91
|
+
**Gate — run this step only when the current `mrs[]` entry is big enough**: `lines_est > 150` **or** `files_est > 5`. No estimates in the entry (an older plan) → fall back to its `size`: `M`/`L` run it, `XS`/`S` skip. No `mrs[]` at all (XS/S, single MR/PR) → skip; the brief and the contracts are the whole plan, and a 30-line change does not earn a planning round. Skipped → record one line in `05-implementation.md` — the estimates that let it through (`plan: skipped — 40 lines / 2 files`), or `plan: skipped — single MR/PR` when there is no entry to read — and go to §2.1.
|
|
92
|
+
|
|
93
|
+
Write it against `03-design.md` and the contracts just copied — **no code yet, no exploratory edits**. Under 250 words, as numbered steps:
|
|
94
|
+
|
|
95
|
+
```markdown
|
|
96
|
+
## Plan MR/PR #N
|
|
97
|
+
1. <step> — `<file or path>` — <test that proves it, or "covered by step M">
|
|
98
|
+
2. …
|
|
99
|
+
|
|
100
|
+
- **Point of no return**: <the migration, contract, published event or flag after which rollback stops being a revert — or "none">.
|
|
101
|
+
- **Out of this plan**: <what belongs to a later MR/PR and will be tempting while inside these files>.
|
|
102
|
+
```
|
|
103
|
+
|
|
104
|
+
Rules for the steps:
|
|
105
|
+
- **A step is a thing on disk**, not a phase of thought: "add the nullable column + migration", not "analyse the schema". A step nobody can tell is finished is two steps or none.
|
|
106
|
+
- **Order by what unblocks what**, and put the point of no return as late as the design allows: everything reversible lands before it.
|
|
107
|
+
- **Every step names its test** or the step that covers it. A step that proves no acceptance criterion and guards no contract is a candidate to cut — same bar as `03-design.md` §"Planned tests".
|
|
108
|
+
- Steps come from the **design**, not from the repo you are about to open. A step the design never mentions is either a deviation to log (§3 "Deviations from design") or scope that belongs in §2.4.
|
|
109
|
+
|
|
110
|
+
**Present it, then behave by mode:**
|
|
111
|
+
- `manual` / `guided` → **stop**: full stop header (flow-core §3), the plan, and one `AskUserQuestion` — **Approve and build** · **Change the order or the steps** (the user edits, rewrite it, ask again). This is a genuine decision point: the order of an irreversible step is exactly what is expensive to argue with once there is a diff.
|
|
112
|
+
- `auto` → **do not stop**. Print the numbered steps and the point of no return, record the plan as accepted, and open the same message with the call that starts §2.1 (flow-core §3: a report is never the last thing in a turn you meant to continue).
|
|
113
|
+
|
|
114
|
+
Save it in `05-implementation.md` under `## Plan MR/PR #N`, right after the brief. It is what seeds `TaskCreate` in §2.1, and what a §2.3 hot cut splits: the steps already done stay in this MR/PR, the ones still pending move to the new entry.
|
|
86
115
|
|
|
87
116
|
## 2.1 Work
|
|
88
117
|
|
|
@@ -113,7 +142,7 @@ Execution mode:
|
|
|
113
142
|
- **Give it a bounded first step.** A piece with no natural first move ("do the whole module") produces nothing for hours; split it by file or by numbered step until it has one.
|
|
114
143
|
- **Check the round at every stop of this command**, not only at the end: an agent past `agents.stall_after_minutes` (empty → 25) with nothing written to its path is stopped, split in two and relaunched.
|
|
115
144
|
|
|
116
|
-
Use `TaskCreate` to track the steps from the
|
|
145
|
+
Use `TaskCreate` to track the steps of the **§2.0ter plan for this MR/PR** — one task per numbered step, same order. Step skipped there (small MR/PR) → seed from `03-design.md` §"Implementation plan (order)", taking only the steps this MR/PR covers per `04-mr-plan.md`; the feature's plan predates the split and carries steps that belong to siblings. Mark each step `in_progress` when starting and `completed` when done — do not batch.
|
|
117
146
|
|
|
118
147
|
### 2.1bis The premise this change rests on (any size, XS included)
|
|
119
148
|
|
|
@@ -219,7 +248,7 @@ Warning thresholds:
|
|
|
219
248
|
|
|
220
249
|
If either is exceeded, **pause** and ask the user with `AskUserQuestion` (options, in this order):
|
|
221
250
|
|
|
222
|
-
1. **Cut here (recommended if the current piece is coherent)**. What is built so far stays as this MR/PR; what remains
|
|
251
|
+
1. **Cut here (recommended if the current piece is coherent)**. What is built so far stays as this MR/PR; what remains — the §2.0ter steps still pending, or what is left of `04-mr-plan.md` where that step was skipped — goes into a new one inserted in `meta.json.mrs` right after. Zero code wasted.
|
|
223
252
|
2. **Continue and record the overrun**. When the cut would be artificial. Note the deviation in `05-implementation.md` to calibrate `/flow-feat-plan` on future tickets.
|
|
224
253
|
3. **Reopen plan**. Return to `/flow-feat-plan` to rethink the entire split. Only if the overrun shows the plan is wrong at a deeper level, not just that this MR/PR is slightly underestimated.
|
|
225
254
|
|
|
@@ -267,6 +296,9 @@ Keep `.claude/work/<TICKET>/05-implementation.md` updated as you work (not at th
|
|
|
267
296
|
**This MR/PR does NOT include**:
|
|
268
297
|
- <pieces that are out of scope>
|
|
269
298
|
|
|
299
|
+
## Plan MR/PR #N
|
|
300
|
+
<the numbered steps of §2.0ter, its point of no return and its "out of this plan" line — or one line saying the step was skipped and why (`plan: skipped — 40 lines / 2 files`). What §2.1 seeds `TaskCreate` from and what a §2.3 cut splits. Steps are struck through or annotated as they are reworked, never deleted: a plan edited to match what happened records nothing.>
|
|
301
|
+
|
|
270
302
|
## Changes per file
|
|
271
303
|
- <file> — what changed and why (1 line each)
|
|
272
304
|
|
|
@@ -358,5 +390,5 @@ If there were no copied contracts (design said "none"), skip this step and recor
|
|
|
358
390
|
- Update `meta.json`: `phase = "build"`, add to `phases_done`.
|
|
359
391
|
- Multi-MR/PR build: leave the current MR/PR `in_progress` in `meta.json.mrs` (it becomes `merged` when `/flow-feat-ship` confirms the merge). **Also add `build` to that MR/PR's own `phases_done`** (its `mrs[]` entry) — the per-MR/PR marker the downstream gates read.
|
|
360
392
|
- Overwrite `00-summary.md` whole (≤15 lines, flow-core §5).
|
|
361
|
-
- Report **following the stop header** (flow-core §3) **only when this is a stop** — `manual`, or a blocker in any mode. Then in bullets: files touched (high level), pending items, **result of §4.2 (contracts verified)**, and **any premise left unsettled by §2.1bis** — one line, in the language of what breaks if it is false. An open premise the user never hears about is an assumption they were not asked about. In `guided`/`auto` `/flow-feat-review` runs in this same turn, so there is no report here: those same points go to the phase artifact,
|
|
393
|
+
- Report **following the stop header** (flow-core §3) **only when this is a stop** — `manual`, or a blocker in any mode. Then in bullets: files touched (high level), pending items, **result of §4.2 (contracts verified)**, and **any premise left unsettled by §2.1bis** — one line, in the language of what breaks if it is false. An open premise the user never hears about is an assumption they were not asked about. In `guided`/`auto` `/flow-feat-review` runs in this same turn, so there is no report here: those same points go to the phase artifact, the panel carries `Now`/`Next`, and the turn opens with the call that chains.
|
|
362
394
|
- **Autonomy handoff.** The summary is a report, not the end of the flow. `manual`: stop and propose `/flow-feat-review` with a single `AskUserQuestion` (recommended option by default); invoke it only on confirmation, never make the user type it. `guided`/`auto`: **chain into `/flow-feat-review` automatically** in this same turn, without asking — never end the turn with the next command as a suggestion.
|
|
@@ -16,7 +16,7 @@ description: Design the technical solution (architecture, DB, APIs, risks) befor
|
|
|
16
16
|
> - `/model <value>` → opencode's model picker (`/models`).
|
|
17
17
|
> - `knowledge.*` roles → whatever tools `FLOW.md` names there; an MCP tool keeps its name, its server is declared in `opencode.json` (see this adapter's `opencode.json` for the domain-memory example).
|
|
18
18
|
|
|
19
|
-
Read `~/.claude/flow/CORE.opencode.md` first (shared rules: `FLOW.md` step 0, models, autonomy modes and hard gates, how a stop reads,
|
|
19
|
+
Read `~/.claude/flow/CORE.opencode.md` first (shared rules: `FLOW.md` step 0, models, autonomy modes and hard gates, how a stop reads, the live panel, `00-summary.md`) — skip if you already read it in this session. **Models: the subagents it launches take `models.agents`; its parallel rounds take `models.workers`.**
|
|
20
20
|
|
|
21
21
|
Technical design phase. **Still no production code is written.** Opens the option space first (M/L), then designs the approach that was chosen. Output: a plan the next step executes.
|
|
22
22
|
|
|
@@ -16,7 +16,7 @@ description: Split the work into small, independently mergeable MRs/PRs before i
|
|
|
16
16
|
> - `/model <value>` → opencode's model picker (`/models`).
|
|
17
17
|
> - `knowledge.*` roles → whatever tools `FLOW.md` names there; an MCP tool keeps its name, its server is declared in `opencode.json` (see this adapter's `opencode.json` for the domain-memory example).
|
|
18
18
|
|
|
19
|
-
Read `~/.claude/flow/CORE.opencode.md` first (shared rules: `FLOW.md` step 0, models, autonomy modes and hard gates, how a stop reads,
|
|
19
|
+
Read `~/.claude/flow/CORE.opencode.md` first (shared rules: `FLOW.md` step 0, models, autonomy modes and hard gates, how a stop reads, the live panel, `00-summary.md`) — skip if you already read it in this session. **Models: the subagents it launches take `models.agents`.**
|
|
20
20
|
|
|
21
21
|
Delivery planning phase. **No code is written.** Splits the feature into MRs/PRs that can each live on the main branch even if the subsequent ones never land.
|
|
22
22
|
|
|
@@ -150,7 +150,7 @@ A slice landing in **another repo** is not one of *this* repo's `mrs`: record it
|
|
|
150
150
|
|
|
151
151
|
- Update `meta.json`: `phase = "plan"`, add `plan` to `phases_done`.
|
|
152
152
|
- Overwrite `00-summary.md` whole (≤15 lines, flow-core §5).
|
|
153
|
-
- Refresh
|
|
153
|
+
- Refresh the panel — a new train is a new *shape*, so republish it whole (flow-core §4): from here on it carries one line per `mrs[]` entry. Nothing is merged or open yet, so every entry carries `mark: "pending"`, the not-yet-started ones collapsed into a single `#a–#z` line, and **no heading over the train** — `mark` states each entry's real state (schema in `/flow-work-README`).
|
|
154
154
|
- **Show the plan the way it will be executed**, not just as a list: print the wave line first, then the table:
|
|
155
155
|
|
|
156
156
|
```
|
|
@@ -16,7 +16,7 @@ description: Mandatory multi-agent code review before shipping
|
|
|
16
16
|
> - `/model <value>` → opencode's model picker (`/models`).
|
|
17
17
|
> - `knowledge.*` roles → whatever tools `FLOW.md` names there; an MCP tool keeps its name, its server is declared in `opencode.json` (see this adapter's `opencode.json` for the domain-memory example).
|
|
18
18
|
|
|
19
|
-
Read `~/.claude/flow/CORE.opencode.md` first (shared rules: `FLOW.md` step 0, models, autonomy modes and hard gates, how a stop reads,
|
|
19
|
+
Read `~/.claude/flow/CORE.opencode.md` first (shared rules: `FLOW.md` step 0, models, autonomy modes and hard gates, how a stop reads, the live panel, `00-summary.md`) — skip if you already read it in this session. **Models: the subagents it launches take `models.agents`; its parallel rounds take `models.workers`.**
|
|
20
20
|
|
|
21
21
|
Mandatory review phase. **`/flow-feat-ship` cannot run without passing through here and resolving blockers.**
|
|
22
22
|
|
|
@@ -16,7 +16,7 @@ description: Commit, push, MR/PR, and offer to save domain knowledge
|
|
|
16
16
|
> - `/model <value>` → opencode's model picker (`/models`).
|
|
17
17
|
> - `knowledge.*` roles → whatever tools `FLOW.md` names there; an MCP tool keeps its name, its server is declared in `opencode.json` (see this adapter's `opencode.json` for the domain-memory example).
|
|
18
18
|
|
|
19
|
-
Read `~/.claude/flow/CORE.opencode.md` first (shared rules: `FLOW.md` step 0, models, autonomy modes and hard gates, how a stop reads,
|
|
19
|
+
Read `~/.claude/flow/CORE.opencode.md` first (shared rules: `FLOW.md` step 0, models, autonomy modes and hard gates, how a stop reads, the live panel, `00-summary.md`) — skip if you already read it in this session. **Models: this command runs with the model it was launched with; the wait it delegates takes `models.supervisors`.**
|
|
20
20
|
|
|
21
21
|
Closes the feature: commit, push, MR/PR (assigned per `git.assignee`, squash per `git.squash`, sections per `git.request_sections`) and an optional offer to consolidate knowledge.
|
|
22
22
|
|
|
@@ -177,7 +177,7 @@ Only here — with the content approved in §3 — invoke `Skill commit-commands
|
|
|
177
177
|
- `git.assignee` not empty → assign to that user. `git.squash` `true` → enable squash-before-merge.
|
|
178
178
|
- `commit-push-pr` skill unavailable → commit and push manually and create the MR/PR with the `git.cli` CLI from `FLOW.md` — always with the content confirmed in §3.
|
|
179
179
|
|
|
180
|
-
**Record the URL the moment it exists**: write it into this MR/PR's `meta.json.mrs` entry and refresh
|
|
180
|
+
**Record the URL the moment it exists**: write it into this MR/PR's `meta.json.mrs` entry and refresh the panel (flow-core §4) right here, before §4.2 and before anything else can fail.
|
|
181
181
|
|
|
182
182
|
**The push just started a pipeline, and this thread does not sit on it.** Its result belongs to `/flow-work-green`, not here, so either close the ship with CI reported as *running* — the honest state — or, when the user wants to hear how it ended, delegate the wait as any other (flow-core §6.5): one supervisor subagent on `models.supervisors`, a deadline, and back with status plus failing-job evidence. Whatever it returns, a red pipeline is triaged by `/flow-work-green`; this command never fixes CI and never closes reporting a green it did not see.
|
|
183
183
|
|
|
@@ -270,7 +270,7 @@ In every scenario:
|
|
|
270
270
|
```
|
|
271
271
|
|
|
272
272
|
Nothing back → the content is **not** on the base whatever the forge says (see §6.2.1). Do not mark the entry `merged`: leave it `in_progress` with a `note`, tell the user in one line, and recover it with a fresh MR/PR against `git.default_base` before the train moves on.
|
|
273
|
-
- Refresh
|
|
273
|
+
- Refresh the panel from the updated `meta.json`. A shipped-but-open MR/PR is a `wait` line carrying its `link`; it becomes `done` only when confirmed merged. When this ship sets `phase = "done"`, say so in plain words (`mark: "info"` with `style: "ok"`: nothing left here) and drop the `Decision` line.
|
|
274
274
|
- Overwrite `00-summary.md` whole (≤15 lines, flow-core §5).
|
|
275
275
|
|
|
276
276
|
### 6.1.1 Tracker: move to done
|
|
@@ -16,7 +16,7 @@ description: Start a new feature (read the tracker, classify size, create branch
|
|
|
16
16
|
> - `/model <value>` → opencode's model picker (`/models`).
|
|
17
17
|
> - `knowledge.*` roles → whatever tools `FLOW.md` names there; an MCP tool keeps its name, its server is declared in `opencode.json` (see this adapter's `opencode.json` for the domain-memory example).
|
|
18
18
|
|
|
19
|
-
Read `~/.claude/flow/CORE.opencode.md` first (shared rules: `FLOW.md` step 0, models, autonomy modes and hard gates, how a stop reads,
|
|
19
|
+
Read `~/.claude/flow/CORE.opencode.md` first (shared rules: `FLOW.md` step 0, models, autonomy modes and hard gates, how a stop reads, the live panel, `00-summary.md`) — skip if you already read it in this session. **Models: this command runs with the model it was launched with (no `models` key).**
|
|
20
20
|
|
|
21
21
|
`$ARGUMENTS` is **optional**:
|
|
22
22
|
|
|
@@ -233,9 +233,9 @@ Create the work directory per §1: `.claude/work/<TICKET>-<slug>/` (ticket mode)
|
|
|
233
233
|
- `related_repos` from §3.5 — one `{ "repo": "<name>", "scope": "<one line>", "status": "pending", "contract_handoff": "pending" | "none" }` per *other* repo; `[]` for a single-repo task.
|
|
234
234
|
- **This work *is* an accepted follow-up** (the ticket came from another work's `followups[]`, flow-core §7) → set `origin: { "work": "<originating work dir>", "followup": "F<n>" }`, carry that entry's recorded `why` into `01-context.md` verbatim instead of re-deriving it, and set the originating entry to `in_progress` with `work` naming this folder. Not a follow-up → omit `origin` entirely.
|
|
235
235
|
|
|
236
|
-
###
|
|
236
|
+
### Live panel
|
|
237
237
|
|
|
238
|
-
|
|
238
|
+
Publish it now, in the flow-core §4 shape and by the transport that section selects. No train yet (`plan` has not run), so: the title (`style: title`), a `Now` line for what is starting, a `Next` line for the phase this size routes to, any §3.5 sibling repo as a `block` line. Every later phase updates it.
|
|
239
239
|
|
|
240
240
|
### `00-summary.md`
|
|
241
241
|
|
|
@@ -283,5 +283,5 @@ Run `tracker.start_cmd` with `{TICKET}` = `meta.json.ticket`, `{ASSIGNEE}` = `tr
|
|
|
283
283
|
## 7. Close
|
|
284
284
|
|
|
285
285
|
- Overwrite `00-summary.md` whole (≤15 lines, flow-core §5) — first written in §6; refresh it if §6.5 changed anything.
|
|
286
|
-
- Report **following the stop header** (flow-core §3), branch added, body 2-3 lines: ticket, size, branch; recommended next command per the §4 table — in `guided`/`auto` you run it in this same turn, so there is no report and no question here: update
|
|
286
|
+
- Report **following the stop header** (flow-core §3), branch added, body 2-3 lines: ticket, size, branch; recommended next command per the §4 table — in `guided`/`auto` you run it in this same turn, so there is no report and no question here: update the panel and chain (flow-core §3).
|
|
287
287
|
- Apply `autonomy.mode`: `manual` → stop and let the user invoke the recommended command; `guided`/`auto` → chain into it, subject to the hard gates (the §5 branch-base ambiguity already stops on its own).
|
|
@@ -16,7 +16,7 @@ description: Validate tests, edge cases, and integrity before shipping
|
|
|
16
16
|
> - `/model <value>` → opencode's model picker (`/models`).
|
|
17
17
|
> - `knowledge.*` roles → whatever tools `FLOW.md` names there; an MCP tool keeps its name, its server is declared in `opencode.json` (see this adapter's `opencode.json` for the domain-memory example).
|
|
18
18
|
|
|
19
|
-
Read `~/.claude/flow/CORE.opencode.md` first (shared rules: `FLOW.md` step 0, models, autonomy modes and hard gates, how a stop reads,
|
|
19
|
+
Read `~/.claude/flow/CORE.opencode.md` first (shared rules: `FLOW.md` step 0, models, autonomy modes and hard gates, how a stop reads, the live panel, `00-summary.md`) — skip if you already read it in this session. **Models: the subagents it launches take `models.agents`; the wait it delegates takes `models.supervisors`.**
|
|
20
20
|
|
|
21
21
|
Verify the feature is complete: test coverage, edge cases, performance, regressions.
|
|
22
22
|
|
|
@@ -50,4 +50,4 @@ runs a phase on its own and never advances `meta.json`.
|
|
|
50
50
|
## 3. Hand over
|
|
51
51
|
|
|
52
52
|
Invoke the confirmed command with the arguments the user chose. Nothing to write, no
|
|
53
|
-
artifact, no
|
|
53
|
+
artifact, no panel: the command it hands to owns the rest.
|