flow-workflows 0.76.0 → 0.78.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/adapters/codex/CORE.md +68 -7
- package/adapters/codex/skills/flow-bug-fix/SKILL.md +1 -1
- package/adapters/codex/skills/flow-bug-ship/SKILL.md +1 -0
- package/adapters/codex/skills/flow-doctor/SKILL.md +3 -3
- package/adapters/codex/skills/flow-feat-build/SKILL.md +6 -4
- package/adapters/codex/skills/flow-feat-design/SKILL.md +1 -1
- package/adapters/codex/skills/flow-feat-ship/SKILL.md +23 -0
- package/adapters/codex/skills/flow-work-README/SKILL.md +7 -0
- package/adapters/gemini/CORE.md +68 -7
- package/adapters/gemini/commands/flow/bug/fix.toml +1 -1
- package/adapters/gemini/commands/flow/bug/ship.toml +1 -0
- package/adapters/gemini/commands/flow/doctor.toml +3 -3
- package/adapters/gemini/commands/flow/feat/build.toml +6 -4
- package/adapters/gemini/commands/flow/feat/design.toml +1 -1
- package/adapters/gemini/commands/flow/feat/ship.toml +23 -0
- package/adapters/gemini/commands/flow/work/README.toml +7 -0
- package/adapters/hermes/CORE.md +68 -7
- package/adapters/hermes/skills/flow-bug-fix/SKILL.md +1 -1
- package/adapters/hermes/skills/flow-bug-ship/SKILL.md +1 -0
- package/adapters/hermes/skills/flow-doctor/SKILL.md +3 -3
- package/adapters/hermes/skills/flow-feat-build/SKILL.md +6 -4
- package/adapters/hermes/skills/flow-feat-design/SKILL.md +1 -1
- package/adapters/hermes/skills/flow-feat-ship/SKILL.md +23 -0
- package/adapters/hermes/skills/flow-work-README/SKILL.md +7 -0
- package/adapters/opencode/CORE.md +68 -7
- package/adapters/opencode/commands/flow-bug-fix.md +1 -1
- package/adapters/opencode/commands/flow-bug-ship.md +1 -0
- package/adapters/opencode/commands/flow-doctor.md +3 -3
- package/adapters/opencode/commands/flow-feat-build.md +6 -4
- package/adapters/opencode/commands/flow-feat-design.md +1 -1
- package/adapters/opencode/commands/flow-feat-ship.md +23 -0
- package/adapters/opencode/commands/flow-work-README.md +7 -0
- package/package.json +1 -1
- package/plugins/flow/.claude-plugin/plugin.json +1 -1
- package/plugins/flow/CHANGELOG.md +36 -0
- package/plugins/flow/examples/FLOW.template.md +10 -2
package/adapters/codex/CORE.md
CHANGED
|
@@ -17,7 +17,7 @@ Every `$flow-*` command assumes these rules. They are stated once, here, so a co
|
|
|
17
17
|
carries what is specific to its phase. Read this once per session; a command that says "load
|
|
18
18
|
`flow-core`" means this file.
|
|
19
19
|
|
|
20
|
-
**This file belongs to flow `0.
|
|
20
|
+
**This file belongs to flow `0.78.0`.** Compare it once, at the start of the session, against
|
|
21
21
|
`version` in `~/.claude/flow/.claude-plugin/plugin.json`. The two differing means the session
|
|
22
22
|
is running a **mixture** — the commands from one copy of the plugin, these shared rules from another
|
|
23
23
|
— which is exactly what happens when a branch or a release candidate is loaded over an installed
|
|
@@ -98,8 +98,7 @@ a command relies on is simply absent in that session, and nothing else will repo
|
|
|
98
98
|
"general-purpose", "phase": "review" }`. Asking here would break the never-a-question contract
|
|
99
99
|
(§2); recording keeps the decision readable afterwards, and `$flow-doctor` aggregates it.
|
|
100
100
|
|
|
101
|
-
**The list, and it is short on purpose**: `
|
|
102
|
-
role on an M/L or sensitive diff · `quality.review_depth` after a review's cost line went over
|
|
101
|
+
**The list, and it is short on purpose**: `quality.review_depth` after a review's cost line went over
|
|
103
102
|
`agents.budget_max` · `data.volumes` or `data.explain_cmd` when the query duel came back
|
|
104
103
|
schema-only · `quality.functional_check` when `validate` found a runnable app and no way to drive
|
|
105
104
|
it · `tracker.start_cmd`/`done_cmd`/`abandon_cmd` the first time a ticket could have been
|
|
@@ -157,8 +156,8 @@ improvises, `models.workers` for the parallel fan-out rounds, and `models.superv
|
|
|
157
156
|
bounded waits of §6.5 — the last two falling back to `models.agents`. Empty, absent, or no section
|
|
158
157
|
→ everything runs on the model you were launched with, and you say nothing.
|
|
159
158
|
|
|
160
|
-
- Pass the resolved value to every subagent you launch, **except** an agent named in `agents.<role
|
|
161
|
-
which keeps the model its own definition sets.
|
|
159
|
+
- Pass the resolved value to every subagent you launch, **except** an agent named in `agents.<role>`
|
|
160
|
+
or picked from the installed ones (§6.6), which keeps the model its own definition sets.
|
|
162
161
|
- **You cannot switch your own model**, so no key names the phases the main agent performs itself.
|
|
163
162
|
A phase that wants another model says it in one line at the handoff (`this step reads better on
|
|
164
163
|
<value>, you are on <current>` → `/model <value>`), records it in the artifact, and continues.
|
|
@@ -296,6 +295,9 @@ prose describing a file is otherwise obeyed to the letter.
|
|
|
296
295
|
"",
|
|
297
296
|
{"ref": "#1", "text": "batch read sources", "mark": "wait", "link": "https://gitlab.com/…/merge_requests/9977"},
|
|
298
297
|
{"ref": "#2", "text": "per-event and per-recipient counters", "mark": "current"},
|
|
298
|
+
{"ref": "1–2", "text": "counter columns + migration · repository query", "mark": "done", "indent": 4},
|
|
299
|
+
{"ref": "3", "text": "per-recipient aggregation in the use case", "mark": "current", "indent": 4},
|
|
300
|
+
{"ref": "4", "text": "expose both counters on the endpoint", "mark": "pending", "indent": 4},
|
|
299
301
|
{"ref": "#3–#6", "text": "channel map · use case · document detail · route", "mark": "pending"},
|
|
300
302
|
"",
|
|
301
303
|
{"ref": "Now", "text": "unit suite and the test agent over #2", "mark": "info"},
|
|
@@ -307,10 +309,10 @@ prose describing a file is otherwise obeyed to the letter.
|
|
|
307
309
|
}
|
|
308
310
|
```
|
|
309
311
|
|
|
310
|
-
- **`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).
|
|
312
|
+
- **`mark` says what a line is**; the reader draws it. `done` · `current` (at most one per block) · `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).
|
|
311
313
|
- **`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.
|
|
312
314
|
- **`link` is a field, never a URL inside `text`** — and **every line that names an MR/PR carries it**: the train entries, a blocker about somebody else's MR/PR, a `Now` or `Next` about the open one. The URL is in `meta.json.mrs[].url` or in whatever recorded that blocker; a bare `!10327` typed into `text` is a number the reader cannot make clickable, which is the whole point of the field. No URL recorded anywhere → the number in `text` and nothing else, and go and record it.
|
|
313
|
-
- **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).
|
|
315
|
+
- **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`; (2b) **the plan steps of the MR/PR being built**, right under its train entry — only while `build` or `fix` is working through a written plan (`$flow-feat-build §2.0ter`, `$flow-bug-fix §2.0bis`); one line per step, `ref` its number (`1`, `2`…), `indent: 4`, `key` `step-<n>`, `mark` `done` · `current` · `pending` from the plan's ticks in the phase artifact — never ahead of them; past three finished steps they collapse into one `done` line (`1–4`, `key` `steps-done`), and past three pending ones the tail collapses the same way (`key` `steps-pending`), so the block stays at five lines or fewer; drop the block when the MR/PR's build ends. This is the step list the user watches: a harness's own task list, where there is one, is a bonus on top — the panel is what every harness can feed; (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).
|
|
314
316
|
- **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.
|
|
315
317
|
|
|
316
318
|
### 4.2 Over the terminal's MCP — `panel_set` once, `panel_patch` from then on
|
|
@@ -385,6 +387,10 @@ that never asked, because they answered it and the screen says it did not land;
|
|
|
385
387
|
nobody clears keeps that tab claiming them for ever. The same applies to a blocker that lifted and
|
|
386
388
|
to a `wait` line whose MR/PR merged.
|
|
387
389
|
|
|
390
|
+
**(f) Each plan step that lands** (the (2b) block): one patch, the moment its tick goes into the
|
|
391
|
+
phase artifact — that step to `done`, the next to `current`, `Now` moved on. Not batched at the end
|
|
392
|
+
of the MR/PR: a list that jumps from nothing to finished told the user nothing while it mattered.
|
|
393
|
+
|
|
388
394
|
### 4.5 `meta.json.cost[]` — what a phase measured, when it could
|
|
389
395
|
|
|
390
396
|
A phase that can measure what it spent appends one entry: `{ "phase": "review", "at": "<ISO>",
|
|
@@ -549,6 +555,61 @@ moving.
|
|
|
549
555
|
than to run — the brief, the round trip and the report are the whole of the saving. And anything you
|
|
550
556
|
must *react to* mid-wait is not a wait at all: it is the work, and it stays here.
|
|
551
557
|
|
|
558
|
+
### 6.6 — Which agent a role resolves to
|
|
559
|
+
|
|
560
|
+
**The harness already tells you which agents are installed**, each with a line on what it is for,
|
|
561
|
+
and choosing the one that fits a piece of work is something you do well without being told. A rule
|
|
562
|
+
that answers every empty role with `general-purpose` throws that away: a repo with a frontend
|
|
563
|
+
specialist installed and no `agents.frontend` got a generalist told to act like one. So wherever a
|
|
564
|
+
command says *"`agents.<role>`, empty → `Agent general-purpose` with the role"*, it means this:
|
|
565
|
+
|
|
566
|
+
1. **`agents.<role>` names an agent** → that agent, as always.
|
|
567
|
+
2. **Empty** → the installed agent whose description fits the role **and this work's stack** best —
|
|
568
|
+
read the list your harness gives you, never a guess at a name. None fits clearly →
|
|
569
|
+
`general-purpose` with the role in the prompt, as before. An agent that fits the role but not the
|
|
570
|
+
stack (a PHP reviewer on a React diff) does not fit.
|
|
571
|
+
|
|
572
|
+
**`agents.selection`** decides how far your judgement reaches:
|
|
573
|
+
|
|
574
|
+
- **`configured`** (empty → this) — the two steps above. A role you named is never second-guessed.
|
|
575
|
+
- **`open`** — a named role is a default, not an order: for a piece clearly outside what it
|
|
576
|
+
covers, or where an installed agent fits this piece plainly better, take that one. And a piece
|
|
577
|
+
**no role covers** — a UI component in a work whose roles are all backend — may go to the
|
|
578
|
+
specialist that does cover it. Every substitution is **one line in the phase artifact**: role,
|
|
579
|
+
the agent configured, the agent used, and why. Unrecorded, a panel that ran with other agents
|
|
580
|
+
reads exactly like the one you configured.
|
|
581
|
+
|
|
582
|
+
**Nothing installed fits → create the specialist for this work.** An installed agent is, in the
|
|
583
|
+
end, a brief with a name; what made it good is the brief. So instead of *"act as frontend"*, write
|
|
584
|
+
the one an expert would carry — the stack and its versions as this repo uses them, what to check
|
|
585
|
+
in this piece, the mistakes typical of it, and the report contract of this section — and launch
|
|
586
|
+
`general-purpose` with it. Nothing is written to the harness's agent folders: the specialist lives
|
|
587
|
+
for the brief, and it works on every harness, because every one has a general-purpose agent.
|
|
588
|
+
Record it in `meta.json.specialists[]`:
|
|
589
|
+
|
|
590
|
+
`{ "name": "react-form-reviewer", "role": "frontend", "phase": "review", "brief": "<the full text
|
|
591
|
+
you sent>", "useful": null, "status": null }`
|
|
592
|
+
|
|
593
|
+
Once its result is in, set `useful`: `true` when something it returned was kept — a finding
|
|
594
|
+
accepted, a piece merged into the diff — `false` otherwise. Only a useful one is ever offered for
|
|
595
|
+
saving, and only **when the work ends** (`$flow-feat-ship` §5.1), never mid-phase: by then the
|
|
596
|
+
MR/PR is open and it is plain which of them earned a place among your agents.
|
|
597
|
+
|
|
598
|
+
Three things this never touches, in either mode:
|
|
599
|
+
|
|
600
|
+
- **The review roster** (`quality.review_skill` / `quality.reviewers`) runs as defined, with no
|
|
601
|
+
substitutions: its value is being the same panel every time, so a result can be compared.
|
|
602
|
+
- **The agents that are `general-purpose` by design** — challengers, skeptics, the completeness
|
|
603
|
+
critic, the blinded contract check. They are generalists *on purpose*: a specialist brings the
|
|
604
|
+
assumptions the blinding is there to strip. A step that writes `general-purpose` with no role
|
|
605
|
+
behind it means exactly that.
|
|
606
|
+
- **The budget.** A picked agent counts against `agents.budget_max` like any other. Like a named
|
|
607
|
+
agent it keeps its own model (§1), and `models.agents` reaches it only when it has none.
|
|
608
|
+
|
|
609
|
+
Record what you picked where a default would be recorded (§0): *"`agents.frontend` empty → used
|
|
610
|
+
`react-expert` (installed)"*. That line is also the evidence for writing the role down when the
|
|
611
|
+
next stop offers it.
|
|
612
|
+
|
|
552
613
|
## 7. Deferred work — `followups[]`
|
|
553
614
|
|
|
554
615
|
Every chain in this plugin reaches moments where the right call is **not to do it here**: a build
|
|
@@ -95,7 +95,7 @@ Save it in `04-fix.md` under `## Plan`, right after the brief. **A fix that was
|
|
|
95
95
|
|
|
96
96
|
- Apply the minimal fix targeting the finding of `03-investigation.md` (or the diagnosis if investigate was skipped).
|
|
97
97
|
- 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.
|
|
98
|
-
- More than 2 fix steps → the §2.0bis plan is already written:
|
|
98
|
+
- More than 2 fix steps → the §2.0bis plan is already written: tick its numbered steps off in `04-fix.md` where they stand, and show them in the live panel's step block (flow-core §4.1 (2b), patched per §4.4 (f)); a harness task list, if there is one, goes on top as `$flow-feat-build §2.1` says. One task per step, same order, marked `in_progress` and `completed` as they go — not batched.
|
|
99
99
|
- **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.
|
|
100
100
|
- Keep the log updated while editing.
|
|
101
101
|
|
|
@@ -186,6 +186,7 @@ nothing was measured. A slowness bug is the case where this comment *is* the pro
|
|
|
186
186
|
- **Cross-repo reminder**: for each `meta.json.related_repos` entry not `done`: *"you've shipped the `<this-repo>` part; `<repo>` still needs: `<scope>` → go there and run `$flow-bug-start <TICKET>` (or `$flow-feat-start`)"*. flow only reminds; this is not a hard gate.
|
|
187
187
|
- **Cross-repo contract handoff**: for each such entry with `contract_handoff: "pending"` — the fix changed a surface that sibling consumes — hand the new shape over as `$flow-feat-ship` §6.3 (publish the **literal** shape to the tracker ticket, mandatory preview in every autonomy mode, then record `published → <location>`; versioned-file fallback when there is no tracker). A bug flow has no §"External contracts": take the literal from the fix itself — the actual key, code, route or event the code now emits — read it off the code, never from memory.
|
|
188
188
|
- **Follow-up triage — `$flow-feat-ship` §6.4, in full and unchanged.** Read `meta.json.followups[]`; nothing `proposed` → skip silently. Otherwise §6.4 step 0 first — **3 or more candidates go to a skeptic** whose job is to refute that each deserves a ticket, and only the strongest `tracker.followup_ask_max` (empty → **2**) reach the user (flow-core §7) — then one batched `AskUserQuestion` (**Do it** / **Not worth it** / **Later**; `decision` entries asked as their question, `tooling` and everything `log`ged never asked), the accepted ones get their tracker issue created (outward-facing → **asks in every mode**), the logged ones nothing but `status: "logged"` in `meta.json`, then one offer to start the first. A bug flow is where the gate earns its keep: the prevention actions `$flow-bug-postmortem` wrote, the neighbouring bugs `$flow-bug-validate` found and the similar-risk patterns `$flow-bug-fix` flagged all land here — half a dozen of them, from one fix, and «similar risk elsewhere» that no skeptic ever read is exactly the entry that turns a closed bug into six open questions. Repeat for anything still `proposed` on the re-run that confirms the merge.
|
|
189
|
+
- **Specialists created — `$flow-feat-ship` §5.1, in full and unchanged.** Read `meta.json.specialists[]` (flow-core §6.6); nothing `useful` and pending → skip silently. Otherwise one batched offer to save each as an agent, after the MR/PR and after the knowledge `$flow-bug-postmortem` §4 already consolidated — asked in every mode, never overwriting an agent that exists.
|
|
189
190
|
- **Conventions learned — `$flow-feat-ship` §6.4bis, in full and unchanged.** Read `meta.json.conventions_candidates[]` (flow-core §8); nothing pending → skip silently. Otherwise: drop what `FLOW.md` already says, then one batched `AskUserQuestion` (**Add to FLOW.md** / **Not a rule** / **Later**, or **Replace** when it contradicts a line that is there). Asked in every mode — it edits a file in the tree. A fix is where the corrections land hardest: *«never mock the repository in that layer»*, *«the regression test goes next to the unit test, not in the functional suite»* — said once inside `bug:fix`, and proposed again in the next incident unless they reach the file.
|
|
190
191
|
- **Close the follow-up that started this work.** `meta.json.origin` set and this ship moved `phase` to `done` → open the originating work's `meta.json` (live folder or `_archive/`) and set that `followups[]` entry to `status: "done"`. Best-effort: the originating folder is gone or unreadable → warn in one line and continue, never block. Without this the entry stays `in_progress` forever and `status`/`daily` keep reporting work that is finished.
|
|
191
192
|
- **Only once `phase` is `done`**: delete the candidate anchor `$flow-bug-review §1.5` left behind — `git update-ref -d refs/flow/candidate/<TICKET>`. No question: local, never pushed, and the gate that needed it has run.
|
|
@@ -196,8 +196,8 @@ refuse) · **degraded** (it runs, quietly worse than the config promises), plus
|
|
|
196
196
|
|
|
197
197
|
- Every **role** set in `agents.*` (not `fanout_max` / `fanout_tool` — not agents), plus
|
|
198
198
|
`quality.reviewers` and `quality.review_skill`: discoverable in `~/.claude/agents`, the repo's
|
|
199
|
-
`.agents/agents`, or a plugin. Report each missing name and that it falls back
|
|
200
|
-
(a skill: skipped) — a panel missing agents reviews less and still reports a clean pass.
|
|
199
|
+
`.agents/agents`, or a plugin. Report each missing name and that it falls back per flow-core §6.6 — the installed agent
|
|
200
|
+
that fits, else `general-purpose` (a skill: skipped) — a panel missing agents reviews less and still reports a clean pass.
|
|
201
201
|
- Harness cannot set a model per subagent → note that every `models.*` value degrades to inheritance.
|
|
202
202
|
- **Where the live panel will go this session** (flow-core §4): a tool whose name ends in `panel_set` / `panel_patch` — namespaced by the harness (`mcp__panel__panel_set` on Claude Code) and possibly **deferred**, which reads as absent until the list of deferred tools is checked → the terminal draws the panel and no `panel.json` is written; genuinely absent → the panel is that file, as before. Neither is a fault — one line naming the tools as they are actually called here, or saying the file is the transport.
|
|
203
203
|
|
|
@@ -235,7 +235,7 @@ refuse) · **degraded** (it runs, quietly worse than the config promises), plus
|
|
|
235
235
|
a non-negative integer (`0` = no ceiling; else the default `12` applies). Report both effective
|
|
236
236
|
numbers with the count of panel members `quality.review_skill`/`quality.reviewers` defines, so
|
|
237
237
|
the widest command's cost is visible before it is paid, not after.
|
|
238
|
-
- **A task-list tool** (`TaskCreate` on Claude Code, possibly deferred)
|
|
238
|
+
- **A task-list tool** (`TaskCreate` on Claude Code, possibly deferred) — informational only. The plan steps are shown in the live panel and recorded in the phase artifact; a harness task list is an extra. Absent on Claude Code → one line saying it is withheld from newer models unless `"CLAUDE_CODE_ENABLE_TODO_TOOLS": "true"` is set under `env`, for whoever wants it without a panel.
|
|
239
239
|
- **The `flow-workflows` CLI, which `review` reaches for before it composes a panel**:
|
|
240
240
|
`npx flow-workflows@<version> --version` answers, with `<version>` read from `version` in
|
|
241
241
|
`~/.claude/flow/.claude-plugin/plugin.json` — the same pin `review` uses, so the number that
|
|
@@ -139,8 +139,8 @@ Take the shape from the **Access paths** table in `03-design.md`; a query matchi
|
|
|
139
139
|
|
|
140
140
|
Execution mode:
|
|
141
141
|
|
|
142
|
-
- **Single-thread (XS/S/M)**: implement yourself, step by step
|
|
143
|
-
- **Partial delegation (M/L with clear pieces)**: `Agent` for isolated endpoints, plus `agents.testing` from `FLOW.md` in parallel to prepare the test suite (
|
|
142
|
+
- **Single-thread (XS/S/M)**: implement yourself, step by step. Delegate a piece when a specialist would do it clearly better than you — a UI component to a frontend agent in a mostly backend work, a migration to a persistence agent — not because it is tedious; consult one when blocked — `agents.architecture` from `FLOW.md` for layer questions, `agents.persistence` for query/mapping questions. Which agent each role resolves to, and whether you may pick past a named one, is flow-core §6.6. A delegated piece follows the brief rules of the next bullet and the one-writer rule of flow-core §6.
|
|
143
|
+
- **Partial delegation (M/L with clear pieces)**: `Agent` for isolated endpoints, plus `agents.testing` from `FLOW.md` in parallel to prepare the test suite (empty → flow-core §6.6). Pass the full `03-design.md` in the prompt so agents do not invent things.
|
|
144
144
|
|
|
145
145
|
A brief here states the **output** as explicitly as the input (flow-core §6):
|
|
146
146
|
- **Name the path the agent writes**, and require it to **save after each finished piece, not at the end**. Code, tests and generated content are the deliverable; the report is not. Then **verify the file** instead of believing the report — a piece half-written on disk is recoverable, what only exists in the agent's context is lost the moment it is stopped.
|
|
@@ -150,7 +150,9 @@ Execution mode:
|
|
|
150
150
|
|
|
151
151
|
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.
|
|
152
152
|
|
|
153
|
-
**
|
|
153
|
+
**The steps go to the live panel** (flow-core §4.1 (2b), §4.4 (f)) — that is the list the user watches, whatever the harness. Publish the block when the first step starts, patch it as each one lands.
|
|
154
|
+
|
|
155
|
+
**A harness task list is optional on top.** `TaskCreate` is the Claude Code name; a harness may call it otherwise (`TodoWrite` on older builds), list it **deferred** — by name only, uncallable until its schema is loaded — or not offer it at all (Claude Code withholds it from newer models unless `CLAUDE_CODE_ENABLE_TODO_TOOLS=true`). There → seed it too; not there → nothing is missing and nothing is said. Either way the plan in `05-implementation.md` is the record: tick each step there (`- [x]`) as it lands — the panel and any task list follow the ticks, never the other way round.
|
|
154
156
|
|
|
155
157
|
### 2.1bis The premise this change rests on (any size, XS included)
|
|
156
158
|
|
|
@@ -201,7 +203,7 @@ The step's changes are **always reported before anything is recorded**. Who deci
|
|
|
201
203
|
|
|
202
204
|
**After completing each plan step**:
|
|
203
205
|
|
|
204
|
-
1. **
|
|
206
|
+
1. **Tick the step** in `05-implementation.md`, patch the panel's step block (flow-core §4.4 (f)), and mark it `completed` in the harness task list if there is one (§2.1).
|
|
205
207
|
2. **Report to the user** a step summary (≤ 5 lines):
|
|
206
208
|
```
|
|
207
209
|
Step N done: <description>
|
|
@@ -175,7 +175,7 @@ Save the result at the top of `03-design.md` under "## What already exists". Des
|
|
|
175
175
|
## 4. Work
|
|
176
176
|
|
|
177
177
|
1. Load the project's skills (`FLOW.md` section `conventions`).
|
|
178
|
-
2. Launch **in parallel** by feature/project type. Agent = `agents.<role>` from `FLOW.md`; empty → `Agent general-purpose` with the role in the prompt.
|
|
178
|
+
2. Launch **in parallel** by feature/project type. Agent = `agents.<role>` from `FLOW.md`; empty → the installed agent that fits, else `Agent general-purpose` with the role in the prompt (flow-core §6.6).
|
|
179
179
|
|
|
180
180
|
| When | Agent | Task |
|
|
181
181
|
|---|---|---|
|
|
@@ -234,6 +234,29 @@ Only if `knowledge.read_staging` or `knowledge.save` is set; neither → skip wi
|
|
|
234
234
|
3. **Combine staging + new findings** into a short list. Empty or only code-derivable → do not insist.
|
|
235
235
|
4. 1+ relevant findings → ask the user whether to consolidate. Yes → run the consolidation of flow-core §0 (`read_staging`, then one decision and one `knowledge.save` per finding). No → do not insist.
|
|
236
236
|
|
|
237
|
+
## 5.1 Specialists created (offer)
|
|
238
|
+
|
|
239
|
+
Read `meta.json.specialists[]` (flow-core §6.6). **No entry with `useful: true` and `status` unset
|
|
240
|
+
or `"later"` → skip this section and say nothing.** It comes after §5 on purpose: the MR/PR is open,
|
|
241
|
+
the knowledge is saved, and what is left is whether an agent this work improvised deserves a name.
|
|
242
|
+
|
|
243
|
+
1. **A harness with no file-defined agents** → the section does not exist; say so in one line if
|
|
244
|
+
there were useful specialists, so the brief is not mistaken for lost (it stays in `meta.json`).
|
|
245
|
+
2. **One `AskUserQuestion`**, at most 4 entries, most useful first: its `name`, the role, the phase,
|
|
246
|
+
and in one line what it caught. Options: **Save as agent** · **Save and use it for `<role>`** ·
|
|
247
|
+
**No** · **Later**. The second option exists only when `agents.<role>` is empty.
|
|
248
|
+
3. **Asked in every mode**, `auto` included: it writes to your agent configuration, outside the
|
|
249
|
+
tree, and that is never flow mechanics.
|
|
250
|
+
4. **Save** → write the agent where and in the format your harness reads agents from (on Claude
|
|
251
|
+
Code, `~/.claude/agents/<name>.md`: frontmatter `name` and a one-line `description` of when to
|
|
252
|
+
use it, then the brief as the body — drop what only made sense for this ticket, keep the stack,
|
|
253
|
+
the checks and the typical mistakes). A file with that name already there → never overwrite:
|
|
254
|
+
suffix the name and say so. **Save and use it** → also write `agents.<role>: <name>` to the
|
|
255
|
+
active overlay (a harness-owned agent name, flow-core §0). Then `status: "saved"`, with the path.
|
|
256
|
+
**No** → `"declined"`, never offered again. **Later** → `"later"`, offered by the re-run of this
|
|
257
|
+
close.
|
|
258
|
+
5. **One line on screen**: how many agents were saved and where.
|
|
259
|
+
|
|
237
260
|
## 5.5 A worktree does not own the tree its tools wrote into
|
|
238
261
|
|
|
239
262
|
`meta.json.worktree` null → skip silently; there is nothing here that can go wrong.
|
|
@@ -136,6 +136,12 @@ This system **orchestrates** the project's existing sub-agents and skills (it do
|
|
|
136
136
|
"evidence": "the stop it came from",
|
|
137
137
|
"status": "proposed" | "added" | "declined", "note": "" }
|
|
138
138
|
],
|
|
139
|
+
"specialists": [
|
|
140
|
+
{ "name": "react-form-reviewer", "role": "frontend", "phase": "review",
|
|
141
|
+
"brief": "the full expert brief it was launched with",
|
|
142
|
+
"useful": true | false | null,
|
|
143
|
+
"status": null | "saved" | "declined" | "later", "path": null }
|
|
144
|
+
],
|
|
139
145
|
"origin": { "work": "{PREFIX}XXXXX", "followup": "F1" },
|
|
140
146
|
"started_at": "2026-05-11T10:00:00Z",
|
|
141
147
|
"updated_at": "2026-05-11T11:30:00Z",
|
|
@@ -152,6 +158,7 @@ This system **orchestrates** the project's existing sub-agents and skills (it do
|
|
|
152
158
|
- **`followups`**: work a phase deliberately did **not** do here — a neighbouring defect found while building, a piece put out of scope while planning, a risk recorded but not mitigated, an edge case left unchecked, a prevention action from a postmortem. Written by the phase that parks it (asks nothing) when it clears the bar of flow-core §7, triaged **once** at `ship`'s Close behind a skeptic and a ceiling on how many may become questions, published in the MR/PR description while still open, and surfaced by `status`/`daily`/`next` — including from `_archive/` — until `declined` or `done`. Shape and rules: flow-core §7. `[]` when the work deferred nothing.
|
|
153
159
|
- **`defaults_used`**: an empty `FLOW.md` key a phase resolved with its default instead of asking — written in `guided`/`auto`, where asking would breach the never-a-question contract (flow-core §0). Not an error and not deferred work: a record, aggregated by `$flow-doctor` across every work including `_archive/`, so a key that keeps defaulting can be pinned once instead of decided every time. `[]` when nothing defaulted.
|
|
154
160
|
- **`conventions_candidates`**: rules the user taught **this repo's flow** while correcting a phase — *«not a listener, a message handler»*, *«the test command is `make test-unit`»*. Written by the phase that heard one, asking nothing, only when it would apply to an unrelated ticket in this repo; read by `review`'s idiom audit in the same work; offered once at `ship`'s Close (**Add to FLOW.md** / **Not a rule** / **Later**, or **Replace**), in every mode, because it edits a file in the tree. Never domain knowledge — that filter is unchanged. Shape and rules: flow-core §8. `[]` when the work taught nothing.
|
|
161
|
+
- **`specialists`**: agents a phase **created for this work** because nothing installed fit a role, under `agents.selection: open` (flow-core §6.6) — the brief it wrote and ran on `general-purpose`, and whether anything it returned was kept. Nothing is written to the harness's agent folders while the work runs; `$flow-feat-ship` §5.1 offers the useful ones for saving as real agents once the MR/PR is open and the knowledge is saved, and records where each went.
|
|
155
162
|
- **`origin`**: set when this work *is* a follow-up someone accepted — which work raised it and which entry. Lets `start` carry the recorded "why" into `01-context.md` instead of re-deriving it, and lets that entry move to `done` when this work does. Absent for a work that came straight from a ticket.
|
|
156
163
|
- Ticket format follows `tracker.prefix`; empty → free-form.
|
|
157
164
|
|
package/adapters/gemini/CORE.md
CHANGED
|
@@ -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.78.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
|
|
@@ -97,8 +97,7 @@ a command relies on is simply absent in that session, and nothing else will repo
|
|
|
97
97
|
"general-purpose", "phase": "review" }`. Asking here would break the never-a-question contract
|
|
98
98
|
(§2); recording keeps the decision readable afterwards, and `/flow:doctor` aggregates it.
|
|
99
99
|
|
|
100
|
-
**The list, and it is short on purpose**: `
|
|
101
|
-
role on an M/L or sensitive diff · `quality.review_depth` after a review's cost line went over
|
|
100
|
+
**The list, and it is short on purpose**: `quality.review_depth` after a review's cost line went over
|
|
102
101
|
`agents.budget_max` · `data.volumes` or `data.explain_cmd` when the query duel came back
|
|
103
102
|
schema-only · `quality.functional_check` when `validate` found a runnable app and no way to drive
|
|
104
103
|
it · `tracker.start_cmd`/`done_cmd`/`abandon_cmd` the first time a ticket could have been
|
|
@@ -156,8 +155,8 @@ improvises, `models.workers` for the parallel fan-out rounds, and `models.superv
|
|
|
156
155
|
bounded waits of §6.5 — the last two falling back to `models.agents`. Empty, absent, or no section
|
|
157
156
|
→ everything runs on the model you were launched with, and you say nothing.
|
|
158
157
|
|
|
159
|
-
- Pass the resolved value to every subagent you launch, **except** an agent named in `agents.<role
|
|
160
|
-
which keeps the model its own definition sets.
|
|
158
|
+
- Pass the resolved value to every subagent you launch, **except** an agent named in `agents.<role>`
|
|
159
|
+
or picked from the installed ones (§6.6), which keeps the model its own definition sets.
|
|
161
160
|
- **You cannot switch your own model**, so no key names the phases the main agent performs itself.
|
|
162
161
|
A phase that wants another model says it in one line at the handoff (`this step reads better on
|
|
163
162
|
<value>, you are on <current>` → `/model <value>`), records it in the artifact, and continues.
|
|
@@ -295,6 +294,9 @@ prose describing a file is otherwise obeyed to the letter.
|
|
|
295
294
|
"",
|
|
296
295
|
{"ref": "#1", "text": "batch read sources", "mark": "wait", "link": "https://gitlab.com/…/merge_requests/9977"},
|
|
297
296
|
{"ref": "#2", "text": "per-event and per-recipient counters", "mark": "current"},
|
|
297
|
+
{"ref": "1–2", "text": "counter columns + migration · repository query", "mark": "done", "indent": 4},
|
|
298
|
+
{"ref": "3", "text": "per-recipient aggregation in the use case", "mark": "current", "indent": 4},
|
|
299
|
+
{"ref": "4", "text": "expose both counters on the endpoint", "mark": "pending", "indent": 4},
|
|
298
300
|
{"ref": "#3–#6", "text": "channel map · use case · document detail · route", "mark": "pending"},
|
|
299
301
|
"",
|
|
300
302
|
{"ref": "Now", "text": "unit suite and the test agent over #2", "mark": "info"},
|
|
@@ -306,10 +308,10 @@ prose describing a file is otherwise obeyed to the letter.
|
|
|
306
308
|
}
|
|
307
309
|
```
|
|
308
310
|
|
|
309
|
-
- **`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).
|
|
311
|
+
- **`mark` says what a line is**; the reader draws it. `done` · `current` (at most one per block) · `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).
|
|
310
312
|
- **`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.
|
|
311
313
|
- **`link` is a field, never a URL inside `text`** — and **every line that names an MR/PR carries it**: the train entries, a blocker about somebody else's MR/PR, a `Now` or `Next` about the open one. The URL is in `meta.json.mrs[].url` or in whatever recorded that blocker; a bare `!10327` typed into `text` is a number the reader cannot make clickable, which is the whole point of the field. No URL recorded anywhere → the number in `text` and nothing else, and go and record it.
|
|
312
|
-
- **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).
|
|
314
|
+
- **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`; (2b) **the plan steps of the MR/PR being built**, right under its train entry — only while `build` or `fix` is working through a written plan (`/flow:feat:build §2.0ter`, `/flow:bug:fix §2.0bis`); one line per step, `ref` its number (`1`, `2`…), `indent: 4`, `key` `step-<n>`, `mark` `done` · `current` · `pending` from the plan's ticks in the phase artifact — never ahead of them; past three finished steps they collapse into one `done` line (`1–4`, `key` `steps-done`), and past three pending ones the tail collapses the same way (`key` `steps-pending`), so the block stays at five lines or fewer; drop the block when the MR/PR's build ends. This is the step list the user watches: a harness's own task list, where there is one, is a bonus on top — the panel is what every harness can feed; (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).
|
|
313
315
|
- **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.
|
|
314
316
|
|
|
315
317
|
### 4.2 Over the terminal's MCP — `panel_set` once, `panel_patch` from then on
|
|
@@ -384,6 +386,10 @@ that never asked, because they answered it and the screen says it did not land;
|
|
|
384
386
|
nobody clears keeps that tab claiming them for ever. The same applies to a blocker that lifted and
|
|
385
387
|
to a `wait` line whose MR/PR merged.
|
|
386
388
|
|
|
389
|
+
**(f) Each plan step that lands** (the (2b) block): one patch, the moment its tick goes into the
|
|
390
|
+
phase artifact — that step to `done`, the next to `current`, `Now` moved on. Not batched at the end
|
|
391
|
+
of the MR/PR: a list that jumps from nothing to finished told the user nothing while it mattered.
|
|
392
|
+
|
|
387
393
|
### 4.5 `meta.json.cost[]` — what a phase measured, when it could
|
|
388
394
|
|
|
389
395
|
A phase that can measure what it spent appends one entry: `{ "phase": "review", "at": "<ISO>",
|
|
@@ -548,6 +554,61 @@ moving.
|
|
|
548
554
|
than to run — the brief, the round trip and the report are the whole of the saving. And anything you
|
|
549
555
|
must *react to* mid-wait is not a wait at all: it is the work, and it stays here.
|
|
550
556
|
|
|
557
|
+
### 6.6 — Which agent a role resolves to
|
|
558
|
+
|
|
559
|
+
**The harness already tells you which agents are installed**, each with a line on what it is for,
|
|
560
|
+
and choosing the one that fits a piece of work is something you do well without being told. A rule
|
|
561
|
+
that answers every empty role with `general-purpose` throws that away: a repo with a frontend
|
|
562
|
+
specialist installed and no `agents.frontend` got a generalist told to act like one. So wherever a
|
|
563
|
+
command says *"`agents.<role>`, empty → `Agent general-purpose` with the role"*, it means this:
|
|
564
|
+
|
|
565
|
+
1. **`agents.<role>` names an agent** → that agent, as always.
|
|
566
|
+
2. **Empty** → the installed agent whose description fits the role **and this work's stack** best —
|
|
567
|
+
read the list your harness gives you, never a guess at a name. None fits clearly →
|
|
568
|
+
`general-purpose` with the role in the prompt, as before. An agent that fits the role but not the
|
|
569
|
+
stack (a PHP reviewer on a React diff) does not fit.
|
|
570
|
+
|
|
571
|
+
**`agents.selection`** decides how far your judgement reaches:
|
|
572
|
+
|
|
573
|
+
- **`configured`** (empty → this) — the two steps above. A role you named is never second-guessed.
|
|
574
|
+
- **`open`** — a named role is a default, not an order: for a piece clearly outside what it
|
|
575
|
+
covers, or where an installed agent fits this piece plainly better, take that one. And a piece
|
|
576
|
+
**no role covers** — a UI component in a work whose roles are all backend — may go to the
|
|
577
|
+
specialist that does cover it. Every substitution is **one line in the phase artifact**: role,
|
|
578
|
+
the agent configured, the agent used, and why. Unrecorded, a panel that ran with other agents
|
|
579
|
+
reads exactly like the one you configured.
|
|
580
|
+
|
|
581
|
+
**Nothing installed fits → create the specialist for this work.** An installed agent is, in the
|
|
582
|
+
end, a brief with a name; what made it good is the brief. So instead of *"act as frontend"*, write
|
|
583
|
+
the one an expert would carry — the stack and its versions as this repo uses them, what to check
|
|
584
|
+
in this piece, the mistakes typical of it, and the report contract of this section — and launch
|
|
585
|
+
`general-purpose` with it. Nothing is written to the harness's agent folders: the specialist lives
|
|
586
|
+
for the brief, and it works on every harness, because every one has a general-purpose agent.
|
|
587
|
+
Record it in `meta.json.specialists[]`:
|
|
588
|
+
|
|
589
|
+
`{ "name": "react-form-reviewer", "role": "frontend", "phase": "review", "brief": "<the full text
|
|
590
|
+
you sent>", "useful": null, "status": null }`
|
|
591
|
+
|
|
592
|
+
Once its result is in, set `useful`: `true` when something it returned was kept — a finding
|
|
593
|
+
accepted, a piece merged into the diff — `false` otherwise. Only a useful one is ever offered for
|
|
594
|
+
saving, and only **when the work ends** (`/flow:feat:ship` §5.1), never mid-phase: by then the
|
|
595
|
+
MR/PR is open and it is plain which of them earned a place among your agents.
|
|
596
|
+
|
|
597
|
+
Three things this never touches, in either mode:
|
|
598
|
+
|
|
599
|
+
- **The review roster** (`quality.review_skill` / `quality.reviewers`) runs as defined, with no
|
|
600
|
+
substitutions: its value is being the same panel every time, so a result can be compared.
|
|
601
|
+
- **The agents that are `general-purpose` by design** — challengers, skeptics, the completeness
|
|
602
|
+
critic, the blinded contract check. They are generalists *on purpose*: a specialist brings the
|
|
603
|
+
assumptions the blinding is there to strip. A step that writes `general-purpose` with no role
|
|
604
|
+
behind it means exactly that.
|
|
605
|
+
- **The budget.** A picked agent counts against `agents.budget_max` like any other. Like a named
|
|
606
|
+
agent it keeps its own model (§1), and `models.agents` reaches it only when it has none.
|
|
607
|
+
|
|
608
|
+
Record what you picked where a default would be recorded (§0): *"`agents.frontend` empty → used
|
|
609
|
+
`react-expert` (installed)"*. That line is also the evidence for writing the role down when the
|
|
610
|
+
next stop offers it.
|
|
611
|
+
|
|
551
612
|
## 7. Deferred work — `followups[]`
|
|
552
613
|
|
|
553
614
|
Every chain in this plugin reaches moments where the right call is **not to do it here**: a build
|
|
@@ -91,7 +91,7 @@ Save it in `04-fix.md` under `## Plan`, right after the brief. **A fix that was
|
|
|
91
91
|
|
|
92
92
|
- Apply the minimal fix targeting the finding of `03-investigation.md` (or the diagnosis if investigate was skipped).
|
|
93
93
|
- 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.
|
|
94
|
-
- More than 2 fix steps → the §2.0bis plan is already written:
|
|
94
|
+
- More than 2 fix steps → the §2.0bis plan is already written: tick its numbered steps off in `04-fix.md` where they stand, and show them in the live panel's step block (flow-core §4.1 (2b), patched per §4.4 (f)); a harness task list, if there is one, goes on top as `/flow:feat:build §2.1` says. One task per step, same order, marked `in_progress` and `completed` as they go — not batched.
|
|
95
95
|
- **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.
|
|
96
96
|
- Keep the log updated while editing.
|
|
97
97
|
|
|
@@ -182,6 +182,7 @@ nothing was measured. A slowness bug is the case where this comment *is* the pro
|
|
|
182
182
|
- **Cross-repo reminder**: for each `meta.json.related_repos` entry not `done`: *"you've shipped the `<this-repo>` part; `<repo>` still needs: `<scope>` → go there and run `/flow:bug:start <TICKET>` (or `/flow:feat:start`)"*. flow only reminds; this is not a hard gate.
|
|
183
183
|
- **Cross-repo contract handoff**: for each such entry with `contract_handoff: "pending"` — the fix changed a surface that sibling consumes — hand the new shape over as `/flow:feat:ship` §6.3 (publish the **literal** shape to the tracker ticket, mandatory preview in every autonomy mode, then record `published → <location>`; versioned-file fallback when there is no tracker). A bug flow has no §"External contracts": take the literal from the fix itself — the actual key, code, route or event the code now emits — read it off the code, never from memory.
|
|
184
184
|
- **Follow-up triage — `/flow:feat:ship` §6.4, in full and unchanged.** Read `meta.json.followups[]`; nothing `proposed` → skip silently. Otherwise §6.4 step 0 first — **3 or more candidates go to a skeptic** whose job is to refute that each deserves a ticket, and only the strongest `tracker.followup_ask_max` (empty → **2**) reach the user (flow-core §7) — then one batched `AskUserQuestion` (**Do it** / **Not worth it** / **Later**; `decision` entries asked as their question, `tooling` and everything `log`ged never asked), the accepted ones get their tracker issue created (outward-facing → **asks in every mode**), the logged ones nothing but `status: "logged"` in `meta.json`, then one offer to start the first. A bug flow is where the gate earns its keep: the prevention actions `/flow:bug:postmortem` wrote, the neighbouring bugs `/flow:bug:validate` found and the similar-risk patterns `/flow:bug:fix` flagged all land here — half a dozen of them, from one fix, and «similar risk elsewhere» that no skeptic ever read is exactly the entry that turns a closed bug into six open questions. Repeat for anything still `proposed` on the re-run that confirms the merge.
|
|
185
|
+
- **Specialists created — `/flow:feat:ship` §5.1, in full and unchanged.** Read `meta.json.specialists[]` (flow-core §6.6); nothing `useful` and pending → skip silently. Otherwise one batched offer to save each as an agent, after the MR/PR and after the knowledge `/flow:bug:postmortem` §4 already consolidated — asked in every mode, never overwriting an agent that exists.
|
|
185
186
|
- **Conventions learned — `/flow:feat:ship` §6.4bis, in full and unchanged.** Read `meta.json.conventions_candidates[]` (flow-core §8); nothing pending → skip silently. Otherwise: drop what `FLOW.md` already says, then one batched `AskUserQuestion` (**Add to FLOW.md** / **Not a rule** / **Later**, or **Replace** when it contradicts a line that is there). Asked in every mode — it edits a file in the tree. A fix is where the corrections land hardest: *«never mock the repository in that layer»*, *«the regression test goes next to the unit test, not in the functional suite»* — said once inside `bug:fix`, and proposed again in the next incident unless they reach the file.
|
|
186
187
|
- **Close the follow-up that started this work.** `meta.json.origin` set and this ship moved `phase` to `done` → open the originating work's `meta.json` (live folder or `_archive/`) and set that `followups[]` entry to `status: "done"`. Best-effort: the originating folder is gone or unreadable → warn in one line and continue, never block. Without this the entry stays `in_progress` forever and `status`/`daily` keep reporting work that is finished.
|
|
187
188
|
- **Only once `phase` is `done`**: delete the candidate anchor `/flow:bug:review §1.5` left behind — `git update-ref -d refs/flow/candidate/<TICKET>`. No question: local, never pushed, and the gate that needed it has run.
|
|
@@ -192,8 +192,8 @@ refuse) · **degraded** (it runs, quietly worse than the config promises), plus
|
|
|
192
192
|
|
|
193
193
|
- Every **role** set in `agents.*` (not `fanout_max` / `fanout_tool` — not agents), plus
|
|
194
194
|
`quality.reviewers` and `quality.review_skill`: discoverable in `~/.claude/agents`, the repo's
|
|
195
|
-
`.agents/agents`, or a plugin. Report each missing name and that it falls back
|
|
196
|
-
(a skill: skipped) — a panel missing agents reviews less and still reports a clean pass.
|
|
195
|
+
`.agents/agents`, or a plugin. Report each missing name and that it falls back per flow-core §6.6 — the installed agent
|
|
196
|
+
that fits, else `general-purpose` (a skill: skipped) — a panel missing agents reviews less and still reports a clean pass.
|
|
197
197
|
- Harness cannot set a model per subagent → note that every `models.*` value degrades to inheritance.
|
|
198
198
|
- **Where the live panel will go this session** (flow-core §4): a tool whose name ends in `panel_set` / `panel_patch` — namespaced by the harness (`mcp__panel__panel_set` on Claude Code) and possibly **deferred**, which reads as absent until the list of deferred tools is checked → the terminal draws the panel and no `panel.json` is written; genuinely absent → the panel is that file, as before. Neither is a fault — one line naming the tools as they are actually called here, or saying the file is the transport.
|
|
199
199
|
|
|
@@ -231,7 +231,7 @@ refuse) · **degraded** (it runs, quietly worse than the config promises), plus
|
|
|
231
231
|
a non-negative integer (`0` = no ceiling; else the default `12` applies). Report both effective
|
|
232
232
|
numbers with the count of panel members `quality.review_skill`/`quality.reviewers` defines, so
|
|
233
233
|
the widest command's cost is visible before it is paid, not after.
|
|
234
|
-
- **A task-list tool** (`TaskCreate` on Claude Code, possibly deferred)
|
|
234
|
+
- **A task-list tool** (`TaskCreate` on Claude Code, possibly deferred) — informational only. The plan steps are shown in the live panel and recorded in the phase artifact; a harness task list is an extra. Absent on Claude Code → one line saying it is withheld from newer models unless `"CLAUDE_CODE_ENABLE_TODO_TOOLS": "true"` is set under `env`, for whoever wants it without a panel.
|
|
235
235
|
- **The `flow-workflows` CLI, which `review` reaches for before it composes a panel**:
|
|
236
236
|
`npx flow-workflows@<version> --version` answers, with `<version>` read from `version` in
|
|
237
237
|
`~/.claude/flow/.claude-plugin/plugin.json` — the same pin `review` uses, so the number that
|
|
@@ -135,8 +135,8 @@ Take the shape from the **Access paths** table in `03-design.md`; a query matchi
|
|
|
135
135
|
|
|
136
136
|
Execution mode:
|
|
137
137
|
|
|
138
|
-
- **Single-thread (XS/S/M)**: implement yourself, step by step
|
|
139
|
-
- **Partial delegation (M/L with clear pieces)**: `Agent` for isolated endpoints, plus `agents.testing` from `FLOW.md` in parallel to prepare the test suite (
|
|
138
|
+
- **Single-thread (XS/S/M)**: implement yourself, step by step. Delegate a piece when a specialist would do it clearly better than you — a UI component to a frontend agent in a mostly backend work, a migration to a persistence agent — not because it is tedious; consult one when blocked — `agents.architecture` from `FLOW.md` for layer questions, `agents.persistence` for query/mapping questions. Which agent each role resolves to, and whether you may pick past a named one, is flow-core §6.6. A delegated piece follows the brief rules of the next bullet and the one-writer rule of flow-core §6.
|
|
139
|
+
- **Partial delegation (M/L with clear pieces)**: `Agent` for isolated endpoints, plus `agents.testing` from `FLOW.md` in parallel to prepare the test suite (empty → flow-core §6.6). Pass the full `03-design.md` in the prompt so agents do not invent things.
|
|
140
140
|
|
|
141
141
|
A brief here states the **output** as explicitly as the input (flow-core §6):
|
|
142
142
|
- **Name the path the agent writes**, and require it to **save after each finished piece, not at the end**. Code, tests and generated content are the deliverable; the report is not. Then **verify the file** instead of believing the report — a piece half-written on disk is recoverable, what only exists in the agent's context is lost the moment it is stopped.
|
|
@@ -146,7 +146,9 @@ Execution mode:
|
|
|
146
146
|
|
|
147
147
|
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.
|
|
148
148
|
|
|
149
|
-
**
|
|
149
|
+
**The steps go to the live panel** (flow-core §4.1 (2b), §4.4 (f)) — that is the list the user watches, whatever the harness. Publish the block when the first step starts, patch it as each one lands.
|
|
150
|
+
|
|
151
|
+
**A harness task list is optional on top.** `TaskCreate` is the Claude Code name; a harness may call it otherwise (`TodoWrite` on older builds), list it **deferred** — by name only, uncallable until its schema is loaded — or not offer it at all (Claude Code withholds it from newer models unless `CLAUDE_CODE_ENABLE_TODO_TOOLS=true`). There → seed it too; not there → nothing is missing and nothing is said. Either way the plan in `05-implementation.md` is the record: tick each step there (`- [x]`) as it lands — the panel and any task list follow the ticks, never the other way round.
|
|
150
152
|
|
|
151
153
|
### 2.1bis The premise this change rests on (any size, XS included)
|
|
152
154
|
|
|
@@ -197,7 +199,7 @@ The step's changes are **always reported before anything is recorded**. Who deci
|
|
|
197
199
|
|
|
198
200
|
**After completing each plan step**:
|
|
199
201
|
|
|
200
|
-
1. **
|
|
202
|
+
1. **Tick the step** in `05-implementation.md`, patch the panel's step block (flow-core §4.4 (f)), and mark it `completed` in the harness task list if there is one (§2.1).
|
|
201
203
|
2. **Report to the user** a step summary (≤ 5 lines):
|
|
202
204
|
```
|
|
203
205
|
Step N done: <description>
|
|
@@ -171,7 +171,7 @@ Save the result at the top of `03-design.md` under "## What already exists". Des
|
|
|
171
171
|
## 4. Work
|
|
172
172
|
|
|
173
173
|
1. Load the project's skills (`FLOW.md` section `conventions`).
|
|
174
|
-
2. Launch **in parallel** by feature/project type. Agent = `agents.<role>` from `FLOW.md`; empty → `Agent general-purpose` with the role in the prompt.
|
|
174
|
+
2. Launch **in parallel** by feature/project type. Agent = `agents.<role>` from `FLOW.md`; empty → the installed agent that fits, else `Agent general-purpose` with the role in the prompt (flow-core §6.6).
|
|
175
175
|
|
|
176
176
|
| When | Agent | Task |
|
|
177
177
|
|---|---|---|
|
|
@@ -230,6 +230,29 @@ Only if `knowledge.read_staging` or `knowledge.save` is set; neither → skip wi
|
|
|
230
230
|
3. **Combine staging + new findings** into a short list. Empty or only code-derivable → do not insist.
|
|
231
231
|
4. 1+ relevant findings → ask the user whether to consolidate. Yes → run the consolidation of flow-core §0 (`read_staging`, then one decision and one `knowledge.save` per finding). No → do not insist.
|
|
232
232
|
|
|
233
|
+
## 5.1 Specialists created (offer)
|
|
234
|
+
|
|
235
|
+
Read `meta.json.specialists[]` (flow-core §6.6). **No entry with `useful: true` and `status` unset
|
|
236
|
+
or `"later"` → skip this section and say nothing.** It comes after §5 on purpose: the MR/PR is open,
|
|
237
|
+
the knowledge is saved, and what is left is whether an agent this work improvised deserves a name.
|
|
238
|
+
|
|
239
|
+
1. **A harness with no file-defined agents** → the section does not exist; say so in one line if
|
|
240
|
+
there were useful specialists, so the brief is not mistaken for lost (it stays in `meta.json`).
|
|
241
|
+
2. **One `AskUserQuestion`**, at most 4 entries, most useful first: its `name`, the role, the phase,
|
|
242
|
+
and in one line what it caught. Options: **Save as agent** · **Save and use it for `<role>`** ·
|
|
243
|
+
**No** · **Later**. The second option exists only when `agents.<role>` is empty.
|
|
244
|
+
3. **Asked in every mode**, `auto` included: it writes to your agent configuration, outside the
|
|
245
|
+
tree, and that is never flow mechanics.
|
|
246
|
+
4. **Save** → write the agent where and in the format your harness reads agents from (on Claude
|
|
247
|
+
Code, `~/.claude/agents/<name>.md`: frontmatter `name` and a one-line `description` of when to
|
|
248
|
+
use it, then the brief as the body — drop what only made sense for this ticket, keep the stack,
|
|
249
|
+
the checks and the typical mistakes). A file with that name already there → never overwrite:
|
|
250
|
+
suffix the name and say so. **Save and use it** → also write `agents.<role>: <name>` to the
|
|
251
|
+
active overlay (a harness-owned agent name, flow-core §0). Then `status: "saved"`, with the path.
|
|
252
|
+
**No** → `"declined"`, never offered again. **Later** → `"later"`, offered by the re-run of this
|
|
253
|
+
close.
|
|
254
|
+
5. **One line on screen**: how many agents were saved and where.
|
|
255
|
+
|
|
233
256
|
## 5.5 A worktree does not own the tree its tools wrote into
|
|
234
257
|
|
|
235
258
|
`meta.json.worktree` null → skip silently; there is nothing here that can go wrong.
|
|
@@ -132,6 +132,12 @@ This system **orchestrates** the project's existing sub-agents and skills (it do
|
|
|
132
132
|
"evidence": "the stop it came from",
|
|
133
133
|
"status": "proposed" | "added" | "declined", "note": "" }
|
|
134
134
|
],
|
|
135
|
+
"specialists": [
|
|
136
|
+
{ "name": "react-form-reviewer", "role": "frontend", "phase": "review",
|
|
137
|
+
"brief": "the full expert brief it was launched with",
|
|
138
|
+
"useful": true | false | null,
|
|
139
|
+
"status": null | "saved" | "declined" | "later", "path": null }
|
|
140
|
+
],
|
|
135
141
|
"origin": { "work": "{PREFIX}XXXXX", "followup": "F1" },
|
|
136
142
|
"started_at": "2026-05-11T10:00:00Z",
|
|
137
143
|
"updated_at": "2026-05-11T11:30:00Z",
|
|
@@ -148,6 +154,7 @@ This system **orchestrates** the project's existing sub-agents and skills (it do
|
|
|
148
154
|
- **`followups`**: work a phase deliberately did **not** do here — a neighbouring defect found while building, a piece put out of scope while planning, a risk recorded but not mitigated, an edge case left unchecked, a prevention action from a postmortem. Written by the phase that parks it (asks nothing) when it clears the bar of flow-core §7, triaged **once** at `ship`'s Close behind a skeptic and a ceiling on how many may become questions, published in the MR/PR description while still open, and surfaced by `status`/`daily`/`next` — including from `_archive/` — until `declined` or `done`. Shape and rules: flow-core §7. `[]` when the work deferred nothing.
|
|
149
155
|
- **`defaults_used`**: an empty `FLOW.md` key a phase resolved with its default instead of asking — written in `guided`/`auto`, where asking would breach the never-a-question contract (flow-core §0). Not an error and not deferred work: a record, aggregated by `/flow:doctor` across every work including `_archive/`, so a key that keeps defaulting can be pinned once instead of decided every time. `[]` when nothing defaulted.
|
|
150
156
|
- **`conventions_candidates`**: rules the user taught **this repo's flow** while correcting a phase — *«not a listener, a message handler»*, *«the test command is `make test-unit`»*. Written by the phase that heard one, asking nothing, only when it would apply to an unrelated ticket in this repo; read by `review`'s idiom audit in the same work; offered once at `ship`'s Close (**Add to FLOW.md** / **Not a rule** / **Later**, or **Replace**), in every mode, because it edits a file in the tree. Never domain knowledge — that filter is unchanged. Shape and rules: flow-core §8. `[]` when the work taught nothing.
|
|
157
|
+
- **`specialists`**: agents a phase **created for this work** because nothing installed fit a role, under `agents.selection: open` (flow-core §6.6) — the brief it wrote and ran on `general-purpose`, and whether anything it returned was kept. Nothing is written to the harness's agent folders while the work runs; `/flow:feat:ship` §5.1 offers the useful ones for saving as real agents once the MR/PR is open and the knowledge is saved, and records where each went.
|
|
151
158
|
- **`origin`**: set when this work *is* a follow-up someone accepted — which work raised it and which entry. Lets `start` carry the recorded "why" into `01-context.md` instead of re-deriving it, and lets that entry move to `done` when this work does. Absent for a work that came straight from a ticket.
|
|
152
159
|
- Ticket format follows `tracker.prefix`; empty → free-form.
|
|
153
160
|
|