opencode-plugin-flow 4.1.12 → 4.1.14
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/CHANGELOG.md +8 -0
- package/README.md +9 -9
- package/dist/cli.js +186 -35
- package/dist/index.js +185 -34
- package/dist/index.js.map +2 -2
- package/package.json +5 -1
package/CHANGELOG.md
CHANGED
|
@@ -3,6 +3,14 @@
|
|
|
3
3
|
One short line per release. For the full rationale behind each entry, see the
|
|
4
4
|
commit history and review evidence.
|
|
5
5
|
|
|
6
|
+
## [4.1.14] - 2026-06-22
|
|
7
|
+
|
|
8
|
+
Publish Flow through npm trusted publishing so release CI no longer depends on expiring npm tokens.
|
|
9
|
+
|
|
10
|
+
## [4.1.13] - 2026-06-21
|
|
11
|
+
|
|
12
|
+
Teach Flow's parallel orchestration lore to walk a full wave while pinning hidden worker permissions to tested docs.
|
|
13
|
+
|
|
6
14
|
## [4.1.12] - 2026-06-18
|
|
7
15
|
|
|
8
16
|
Harden Flow's hidden worker lore, scriptable doctor checks, typed package smoke, and session edge contracts without widening the v4 runtime surface.
|
package/README.md
CHANGED
|
@@ -21,8 +21,8 @@ The manager still owns every Flow state change. Workers gather evidence; they do
|
|
|
21
21
|
Use OpenCode's plugin installer when your OpenCode version supports it:
|
|
22
22
|
|
|
23
23
|
```bash
|
|
24
|
-
opencode plugin opencode-plugin-flow@4.1.
|
|
25
|
-
npx -y opencode-plugin-flow@4.1.
|
|
24
|
+
opencode plugin opencode-plugin-flow@4.1.14 --global --force
|
|
25
|
+
npx -y opencode-plugin-flow@4.1.14 sync
|
|
26
26
|
```
|
|
27
27
|
|
|
28
28
|
The first command adds Flow to your global OpenCode plugin config or replaces an
|
|
@@ -48,7 +48,7 @@ OpenCode config manually instead:
|
|
|
48
48
|
|
|
49
49
|
```json
|
|
50
50
|
{
|
|
51
|
-
"plugin": ["opencode-plugin-flow@4.1.
|
|
51
|
+
"plugin": ["opencode-plugin-flow@4.1.14"]
|
|
52
52
|
}
|
|
53
53
|
```
|
|
54
54
|
|
|
@@ -59,7 +59,7 @@ duplicate entry.
|
|
|
59
59
|
Then run the same pre-start skill sync and start or restart OpenCode:
|
|
60
60
|
|
|
61
61
|
```bash
|
|
62
|
-
npx -y opencode-plugin-flow@4.1.
|
|
62
|
+
npx -y opencode-plugin-flow@4.1.14 sync
|
|
63
63
|
```
|
|
64
64
|
|
|
65
65
|
Project-local skill overrides still work through OpenCode's normal lookup:
|
|
@@ -84,15 +84,15 @@ version in your global `opencode.json`.
|
|
|
84
84
|
To inspect the installed skill set:
|
|
85
85
|
|
|
86
86
|
```bash
|
|
87
|
-
npx -y opencode-plugin-flow@4.1.
|
|
87
|
+
npx -y opencode-plugin-flow@4.1.14 doctor
|
|
88
88
|
```
|
|
89
89
|
|
|
90
90
|
For automation, keep the default human-readable `doctor` output and opt into
|
|
91
91
|
machine behavior explicitly:
|
|
92
92
|
|
|
93
93
|
```bash
|
|
94
|
-
npx -y opencode-plugin-flow@4.1.
|
|
95
|
-
npx -y opencode-plugin-flow@4.1.
|
|
94
|
+
npx -y opencode-plugin-flow@4.1.14 doctor --json
|
|
95
|
+
npx -y opencode-plugin-flow@4.1.14 doctor --check
|
|
96
96
|
```
|
|
97
97
|
|
|
98
98
|
`doctor --check` and `doctor --strict` exit nonzero when the health status is
|
|
@@ -111,7 +111,7 @@ above first. Missing, incomplete, or outdated managed skills can still be
|
|
|
111
111
|
repaired with:
|
|
112
112
|
|
|
113
113
|
```bash
|
|
114
|
-
npx -y opencode-plugin-flow@4.1.
|
|
114
|
+
npx -y opencode-plugin-flow@4.1.14 sync
|
|
115
115
|
```
|
|
116
116
|
|
|
117
117
|
Then restart OpenCode so the refreshed registry is loaded. `sync` manages all
|
|
@@ -226,7 +226,7 @@ First remove `opencode-plugin-flow` from your OpenCode plugin config so future
|
|
|
226
226
|
OpenCode startups stop loading Flow. Then remove Flow-owned synced skills:
|
|
227
227
|
|
|
228
228
|
```bash
|
|
229
|
-
npx -y opencode-plugin-flow@4.1.
|
|
229
|
+
npx -y opencode-plugin-flow@4.1.14 uninstall
|
|
230
230
|
```
|
|
231
231
|
|
|
232
232
|
Restart OpenCode after both steps. This removes Flow-owned synced skills when
|
package/dist/cli.js
CHANGED
|
@@ -1,11 +1,20 @@
|
|
|
1
1
|
#!/usr/bin/env node
|
|
2
|
-
import{createHash as De}from"node:crypto";import{mkdir as Te,readdir as oe,readFile as Pe,rm as je,writeFile as b}from"node:fs/promises";import{createRequire as Se}from"node:module";import{dirname as We,join as u,normalize as
|
|
2
|
+
import{createHash as De}from"node:crypto";import{mkdir as Te,readdir as oe,readFile as Pe,rm as je,writeFile as b}from"node:fs/promises";import{createRequire as Se}from"node:module";import{dirname as We,join as u,normalize as Oe,sep as Be}from"node:path";var S=`# Flow worker handoff contract
|
|
3
3
|
|
|
4
4
|
Flow managers merge only the worker's final response. Treat that response as the
|
|
5
5
|
worker report of record: it must include the assigned scope, what was actually
|
|
6
6
|
covered, the evidence for each useful claim, and the remaining gaps. End worker
|
|
7
7
|
prompts with "Return only this Flow handoff."
|
|
8
8
|
|
|
9
|
+
Status meanings:
|
|
10
|
+
|
|
11
|
+
- \`success\`: the assigned scope was covered, or any skipped items are explicitly
|
|
12
|
+
immaterial to the assigned question.
|
|
13
|
+
- \`partial\`: useful evidence was gathered, but material assigned scope remains
|
|
14
|
+
unchecked or unresolved.
|
|
15
|
+
- \`blocked\`: the worker cannot answer the assigned question without missing
|
|
16
|
+
access, input, dependencies, or manager clarification.
|
|
17
|
+
|
|
9
18
|
## Evidence, review, validation, or audit worker report
|
|
10
19
|
|
|
11
20
|
Use this for \`flow-evidence-worker\`, \`flow-reviewer\`,
|
|
@@ -47,7 +56,17 @@ success | partial | blocked
|
|
|
47
56
|
Validation workers must include exact command names and raw outcome summaries
|
|
48
57
|
for commands they actually ran. Audit workers must include guards checked for
|
|
49
58
|
any blocking-severity candidate. Review workers must separate blocking findings
|
|
50
|
-
from advisory notes.
|
|
59
|
+
from advisory notes. In the shared \`Findings or facts\` section, review workers
|
|
60
|
+
should prefix review items with \`blocking:\` or \`advisory:\` before the claim.
|
|
61
|
+
|
|
62
|
+
Example evidence quality:
|
|
63
|
+
|
|
64
|
+
- Good fact: \`[high] public Flow command prompts include bundled instructions;
|
|
65
|
+
evidence: src/config-shared.ts:135; corroboration: single source\`.
|
|
66
|
+
- Weak fact: \`[high] prompts look self-contained; evidence: read the config\`.
|
|
67
|
+
- Good validation: \`bun test tests/distribution-and-surface.test.ts\`, status
|
|
68
|
+
passed, summary \`surface tests passed and covered bundled command prompts\`.
|
|
69
|
+
- Weak validation: \`tests pass\`, with no command, status, or raw outcome.
|
|
51
70
|
|
|
52
71
|
## Verifier worker report
|
|
53
72
|
|
|
@@ -131,6 +150,26 @@ Read these companion references before a broad wave:
|
|
|
131
150
|
- \`verification-gates.md\` for coverage checks, handoff acceptance, verifier
|
|
132
151
|
triggers, and synthesis rules.
|
|
133
152
|
|
|
153
|
+
## Operational defaults
|
|
154
|
+
|
|
155
|
+
- Prefer serial work when the scope is small, tightly coupled, or blocked by one
|
|
156
|
+
decision that must be made before slices are meaningful.
|
|
157
|
+
- A normal first wave is two to five workers with independent slices. Use more
|
|
158
|
+
only when the coverage gate is countable and the slices remain non-overlapping.
|
|
159
|
+
- Run at most one routine follow-up wave. Extra waves need an explicit manager
|
|
160
|
+
reason, such as a high-stakes verifier check or a newly discovered bounded
|
|
161
|
+
slice.
|
|
162
|
+
- Do not fan out just to keep agents busy. Every worker should reduce a known
|
|
163
|
+
planning, validation, review, audit, or implementation uncertainty.
|
|
164
|
+
|
|
165
|
+
Skip fan-out when:
|
|
166
|
+
|
|
167
|
+
- one file, command, or design question determines the next step.
|
|
168
|
+
- slices would share the same contracts, fixtures, or edit targets.
|
|
169
|
+
- the manager can inspect the full scope faster than writing and checking
|
|
170
|
+
worker prompts.
|
|
171
|
+
- the result would still need the same manual synthesis with no time saved.
|
|
172
|
+
|
|
134
173
|
## Manager sequence
|
|
135
174
|
|
|
136
175
|
1. Call \`flow_status\` if a Flow session may already exist.
|
|
@@ -154,9 +193,10 @@ Read these companion references before a broad wave:
|
|
|
154
193
|
claims to \`flow-verifier-worker\`.
|
|
155
194
|
9. Run second waves only for material gaps, conflicts, narrowed scope, or
|
|
156
195
|
verification needs.
|
|
157
|
-
10.
|
|
158
|
-
|
|
159
|
-
|
|
196
|
+
10. Apply the manager synthesis barrier: keep only distilled, evidence-backed
|
|
197
|
+
claims and synthesize one Flow artifact, such as plan fields, completion
|
|
198
|
+
evidence, review payload, audit report, or candidate patch decision. Do not
|
|
199
|
+
paste worker handoffs as the user-facing result.
|
|
160
200
|
|
|
161
201
|
## Modes
|
|
162
202
|
|
|
@@ -174,6 +214,36 @@ carry the permission boundaries for each mode.
|
|
|
174
214
|
| \`verifier\` | \`flow-verifier-worker\` | Per-claim verdicts against cited evidence or commands | No | \`flow_status\` only if needed |
|
|
175
215
|
| \`candidate-implementation\` | \`flow-candidate-worker\` | Candidate patch summary from an isolated worktree or exact path-owned slice | Only with explicit user authorization plus isolation or exact non-overlapping path ownership | No state-changing Flow tools |
|
|
176
216
|
|
|
217
|
+
Mode examples:
|
|
218
|
+
|
|
219
|
+
- Use \`flow-evidence-worker\` when the repo shape is unclear and the output will
|
|
220
|
+
become plan requirements, decisions, targets, or validation entries.
|
|
221
|
+
- Use \`flow-reviewer\` when changed files or risk lenses can be reviewed
|
|
222
|
+
independently before the manager returns one review payload.
|
|
223
|
+
- Use \`flow-validation-worker\` when the manager needs command options or raw
|
|
224
|
+
output from an explicitly authorized command.
|
|
225
|
+
- Use \`flow-audit-worker\` when candidate findings must be refuted before they
|
|
226
|
+
can become a report or follow-up feature.
|
|
227
|
+
- Use \`flow-verifier-worker\` for atomic claims that are contested,
|
|
228
|
+
single-sourced, high-stakes, or destined for a Flow payload.
|
|
229
|
+
- Use \`flow-candidate-worker\` only after explicit user authorization and only
|
|
230
|
+
with an isolated worktree or exact non-overlapping path ownership.
|
|
231
|
+
|
|
232
|
+
## Permission contract
|
|
233
|
+
|
|
234
|
+
The plugin injects these hidden workers with the following permission values.
|
|
235
|
+
\`Flow state tools\` means the \`flow_*\` rule, while \`Flow status\` documents the
|
|
236
|
+
explicit \`flow_status\` exception.
|
|
237
|
+
|
|
238
|
+
| Worker | Edit | Bash | Task | Skill | Flow state tools | Flow status |
|
|
239
|
+
| --- | --- | --- | --- | --- | --- | --- |
|
|
240
|
+
| \`flow-reviewer\` | deny | deny | deny | deny | deny | allow |
|
|
241
|
+
| \`flow-evidence-worker\` | deny | deny | deny | deny | deny | allow |
|
|
242
|
+
| \`flow-validation-worker\` | deny | ask | deny | deny | deny | allow |
|
|
243
|
+
| \`flow-audit-worker\` | deny | ask | deny | deny | deny | allow |
|
|
244
|
+
| \`flow-candidate-worker\` | ask | ask | deny | deny | deny | allow |
|
|
245
|
+
| \`flow-verifier-worker\` | deny | ask | deny | deny | deny | allow |
|
|
246
|
+
|
|
177
247
|
Do not fan out parallel \`flow_plan_save\`, \`flow_plan_approve\`,
|
|
178
248
|
\`flow_run_start\`, \`flow_feature_complete\`, \`flow_feature_reset\`, or
|
|
179
249
|
\`flow_session_close\` calls. Runtime locking protects files, but Flow accepts only
|
|
@@ -209,7 +279,7 @@ Mode: evidence | review | validation | audit | verifier | candidate-implementati
|
|
|
209
279
|
Your exact slice: <paths, modules, command, claim ids, risk lens, or worktree>
|
|
210
280
|
Expected coverage: <count, paths, range, or complete question set>
|
|
211
281
|
Do: <bounded actions>
|
|
212
|
-
Do not: call Flow
|
|
282
|
+
Do not: call state-changing Flow tools, edit .flow/**, own sibling slices, or make the final Flow verdict.
|
|
213
283
|
Return exactly the matching handoff shape from handoff-format.md.
|
|
214
284
|
\`\`\`
|
|
215
285
|
|
|
@@ -217,6 +287,71 @@ For research or current-doc slices, require source checks for versioned or
|
|
|
217
287
|
time-sensitive facts. For implementation candidates, remind workers that other
|
|
218
288
|
work may be active and that they must not revert unrelated changes.
|
|
219
289
|
|
|
290
|
+
## Full-wave example
|
|
291
|
+
|
|
292
|
+
Goal: review whether bundled Flow command guidance is self-contained and aligned
|
|
293
|
+
with hidden worker permissions.
|
|
294
|
+
|
|
295
|
+
Serial orientation: the manager reads \`src/config-shared.ts\` enough to identify
|
|
296
|
+
five public command templates and six hidden worker configs. The manager keeps
|
|
297
|
+
\`flow-status\` local because it is one line and does not need a worker.
|
|
298
|
+
|
|
299
|
+
Coverage gate: ten countable items remain after the local check.
|
|
300
|
+
|
|
301
|
+
- Slice A: \`flow-auto\`, \`flow-plan\`, and \`flow-run\` templates, expected 3/10.
|
|
302
|
+
- Slice B: \`flow-review\` template plus \`flow-reviewer\` config, expected 2/10.
|
|
303
|
+
- Slice C: remaining hidden worker permission blocks, expected 5/10 after
|
|
304
|
+
excluding the reviewer already covered by Slice B.
|
|
305
|
+
|
|
306
|
+
Worker prompts:
|
|
307
|
+
|
|
308
|
+
\`\`\`text
|
|
309
|
+
Overall goal, context only: confirm Flow public commands are self-contained.
|
|
310
|
+
Mode: evidence
|
|
311
|
+
Your exact slice: flow-auto, flow-plan, and flow-run templates in src/config-shared.ts.
|
|
312
|
+
Expected coverage: 3/3 templates.
|
|
313
|
+
Do: report bundled sections, setup preflight coverage, and any gaps with file:line evidence.
|
|
314
|
+
Do not: call state-changing Flow tools, edit .flow/**, own sibling slices, or make the final Flow verdict.
|
|
315
|
+
Return exactly the matching handoff shape from handoff-format.md.
|
|
316
|
+
\`\`\`
|
|
317
|
+
|
|
318
|
+
\`\`\`text
|
|
319
|
+
Overall goal, context only: confirm Flow review command and hidden reviewer behavior.
|
|
320
|
+
Mode: review
|
|
321
|
+
Your exact slice: flow-review command template and flow-reviewer config in src/config-shared.ts.
|
|
322
|
+
Expected coverage: 2/2 surfaces.
|
|
323
|
+
Do: separate blocking findings from advisory notes and cite file:line evidence.
|
|
324
|
+
Do not: call state-changing Flow tools, edit .flow/**, own sibling slices, or make the final Flow verdict.
|
|
325
|
+
Return exactly the matching handoff shape from handoff-format.md.
|
|
326
|
+
\`\`\`
|
|
327
|
+
|
|
328
|
+
\`\`\`text
|
|
329
|
+
Overall goal, context only: confirm hidden worker permissions match the orchestration model.
|
|
330
|
+
Mode: audit
|
|
331
|
+
Your exact slice: flow-evidence-worker, flow-validation-worker, flow-audit-worker, flow-candidate-worker, and flow-verifier-worker permissions in src/config-shared.ts.
|
|
332
|
+
Expected coverage: 5/5 worker permission blocks.
|
|
333
|
+
Do: report edit, bash, task, skill, flow_*, and flow_status permissions with evidence.
|
|
334
|
+
Do not: call state-changing Flow tools, edit .flow/**, own sibling slices, or make the final Flow verdict.
|
|
335
|
+
Return exactly the matching handoff shape from handoff-format.md.
|
|
336
|
+
\`\`\`
|
|
337
|
+
|
|
338
|
+
Handoff checks: the manager accepts only reports with terminal status, matching
|
|
339
|
+
coverage counts, concrete file:line evidence, confidence tags, and claims inside
|
|
340
|
+
the assigned slice. A claim such as \`[high] validation workers may run commands;
|
|
341
|
+
evidence: src/config-shared.ts:281-288; corroboration: single source\` is usable.
|
|
342
|
+
A claim such as \`[high] permissions look safe; evidence: config reviewed\` is
|
|
343
|
+
dropped or retasked.
|
|
344
|
+
|
|
345
|
+
Verifier pass: the manager sends any single-source claim that will enter the
|
|
346
|
+
Flow payload to \`flow-verifier-worker\`, for example: \`C1: validation, audit,
|
|
347
|
+
candidate, and verifier workers have bash ask while evidence and review workers
|
|
348
|
+
have bash deny; sources: src/config-shared.ts worker permission blocks\`.
|
|
349
|
+
|
|
350
|
+
Final synthesis: the manager re-reads the relevant config lines, keeps only
|
|
351
|
+
verified or clearly labeled claims, and records one artifact such as a plan
|
|
352
|
+
decision, review payload, or docs patch. Raw handoffs and unverified suggestions
|
|
353
|
+
do not move into the next wave or user-facing answer.
|
|
354
|
+
|
|
220
355
|
## Where handoffs go
|
|
221
356
|
|
|
222
357
|
- Planning evidence becomes \`requirements\`, \`decisions\`, feature \`targets\`,
|
|
@@ -235,6 +370,10 @@ work may be active and that they must not revert unrelated changes.
|
|
|
235
370
|
When worker results conflict, inspect the underlying artifact directly and rerun
|
|
236
371
|
the smallest check that can settle the disagreement.
|
|
237
372
|
|
|
373
|
+
The manager synthesis barrier means raw handoffs do not move forward by default.
|
|
374
|
+
Only claims that survived coverage, evidence, confidence, and verifier checks may
|
|
375
|
+
enter the next wave, Flow payload, patch decision, or user-facing answer.
|
|
376
|
+
|
|
238
377
|
## Second waves
|
|
239
378
|
|
|
240
379
|
Start a follow-up wave when first-wave handoffs reveal:
|
|
@@ -247,7 +386,7 @@ Start a follow-up wave when first-wave handoffs reveal:
|
|
|
247
386
|
|
|
248
387
|
Do not recurse by default. If a worker says it needs another worker, the manager
|
|
249
388
|
decides whether that is a second wave and writes the next bounded prompt.
|
|
250
|
-
`;var
|
|
389
|
+
`;var O='# Recovery playbook\n\nUse this when a Flow tool returns `status: "error"`, a blocker, or a `nextAction` that conflicts with memory.\n\n## First response\n\n1. Re-anchor with `flow_status`.\n2. Read the returned `summary`, `recovery`, `lastError`, and active feature.\n3. Fix the cause, then retry the smallest valid Flow action.\n\n## Common cases\n\n- `missing_session`: start with `flow_plan_save` using the user\'s goal.\n- `missing_goal`: ask for a concrete goal before planning.\n- `Approved plans cannot be changed`: use `flow_feature_reset` when only affected features need another pass; otherwise close and start a new goal.\n- `No feature is currently running`: call `flow_run_start` before completing.\n- `already in progress`: finish, reset, or block the active feature before starting another.\n- `Completion requires recorded validation evidence`: run real validation and include at least one passing `validationRun`.\n- `Completion requires all recorded validation to pass`: fix failures and rerun. Do not relabel failed checks as passed.\n- `Non-final feature completion requires targeted validation`: use `validationScope: "targeted"` for ordinary features.\n- `Final feature completion requires broad validation`: run the project-level gate and use `validationScope: "broad"`.\n- `Completion requires a passing featureReview`: run or request a real review and include a passing `featureReview` only when there are no blocking findings.\n- `Final feature completion requires a finalReview`: perform final review and include `finalReview`.\n- `Final review depth must match the plan policy`: use `reviewDepth` equal to the approved plan\'s `finalReviewPolicy`; valid final-review values are `broad` and `detailed`.\n- `Cannot close ... unfinished features`: complete, reset, defer, or abandon honestly. Do not mark completed while work remains.\n\n## Reset guidance\n\nUse `flow_feature_reset` when the active or completed work was built on the wrong assumption, validation revealed a design issue, dependencies need to be rerun, or dependent features must be invalidated. Resetting a feature also resets its dependents.\n\n## Closure guidance\n\nUse `flow_session_close`:\n\n- `completed`: only after all planned features are complete.\n- `deferred`: the user intentionally postpones unfinished work.\n- `abandoned`: the session should be archived without claiming delivery.\n\nAfter closure, the active `.flow/session.json` is removed and the archived JSON is stored under `.flow/history/`.\n';var B=`# Verification gates
|
|
251
390
|
|
|
252
391
|
Verification is how Flow keeps parallel work from turning into parallel
|
|
253
392
|
guesswork. Worker handoffs are candidate evidence; the manager decides what can
|
|
@@ -322,7 +461,7 @@ Candidate implementation patches are not Flow evidence until the manager
|
|
|
322
461
|
inspects, merges or rejects them, and runs suitable validation in the main
|
|
323
462
|
Flow-managed workspace.
|
|
324
463
|
|
|
325
|
-
##
|
|
464
|
+
## Manager synthesis barrier
|
|
326
465
|
|
|
327
466
|
Before presenting or recording the result:
|
|
328
467
|
|
|
@@ -332,12 +471,14 @@ Before presenting or recording the result:
|
|
|
332
471
|
instead of arbitrating from summaries.
|
|
333
472
|
- Run the strongest practical local check for the deliverable.
|
|
334
473
|
- Re-read critical files or docs that will be cited in the final decision.
|
|
474
|
+
- Move only distilled, evidence-backed claims forward; raw handoffs remain
|
|
475
|
+
candidate evidence, not a plan, review, completion payload, or final answer.
|
|
335
476
|
- Record gaps honestly instead of converting missing evidence into success
|
|
336
477
|
language.
|
|
337
478
|
|
|
338
479
|
\`Status: success\` only says the worker believes its slice is done. The manager
|
|
339
480
|
still checks coverage and evidence before trusting the result.
|
|
340
|
-
`;var
|
|
481
|
+
`;var z=`---
|
|
341
482
|
name: flow
|
|
342
483
|
description: Run the end-to-end Flow loop for skills-first OpenCode work. Use when a user asks for Flow-guided planning through implementation, resumable autonomous delivery, session status, or completion with validation and review gates.
|
|
343
484
|
---
|
|
@@ -412,7 +553,7 @@ Planning and running require loaded Flow tools; do not simulate plan approval or
|
|
|
412
553
|
- Unknown runtime error: read \`summary\` and \`recovery\`; see \`references/recovery-playbook.md\` for common cases.
|
|
413
554
|
|
|
414
555
|
Never fabricate validation output, backfill review approval you did not perform, or close as \`deferred\`/\`abandoned\` merely to avoid an unfinished-work blocker.
|
|
415
|
-
`;var
|
|
556
|
+
`;var E=`---
|
|
416
557
|
name: flow-commit
|
|
417
558
|
description: Prepare safe Git commits and commit messages. Use only when the user asks to inspect, stage, validate, write a commit message, or create a commit; preserves unrelated work and never pushes, amends, rebases, or publishes without explicit authorization.
|
|
418
559
|
---
|
|
@@ -528,7 +669,7 @@ Before running \`git commit\`, report:
|
|
|
528
669
|
|
|
529
670
|
After a successful commit, report the commit hash and leave push or release
|
|
530
671
|
actions for a separate explicit request.
|
|
531
|
-
`;var
|
|
672
|
+
`;var N=`# Safe refactor workflow
|
|
532
673
|
|
|
533
674
|
Refactoring is a behavior-preserving sequence of small changes. This workflow keeps cleanup from becoming an unreviewable rewrite.
|
|
534
675
|
|
|
@@ -570,7 +711,7 @@ Weak evidence includes:
|
|
|
570
711
|
- Public contracts and compatibility shims remain intact or were explicitly planned.
|
|
571
712
|
- Deleted code is actually unreachable or obsolete.
|
|
572
713
|
- Validation can catch a realistic mistake in the refactor.
|
|
573
|
-
`;var
|
|
714
|
+
`;var G=`# Deslop smell rubric
|
|
574
715
|
|
|
575
716
|
Use this rubric to turn vague cleanup instincts into reviewable findings.
|
|
576
717
|
|
|
@@ -604,7 +745,7 @@ class; severity; location; evidence read; refutation checked; why it matters; sa
|
|
|
604
745
|
\`\`\`
|
|
605
746
|
|
|
606
747
|
Rate as blocking only when the smell materially raises defect risk, blocks planned work, hides behavior, or makes the success claim unverifiable. Style-only cleanup is advisory.
|
|
607
|
-
`;var
|
|
748
|
+
`;var V=`---
|
|
608
749
|
name: flow-deslop
|
|
609
750
|
description: Clean up and refactor code with evidence-backed code-smell analysis. Use for AI-slop removal, overengineering reduction, maintainability refactors, behavior-preserving cleanup, duplicated or bloated code, speculative abstractions, dead code, or broad cleanup/refactor review.
|
|
610
751
|
---
|
|
@@ -646,7 +787,7 @@ For each claimed smell removal, verify:
|
|
|
646
787
|
- **blast radius** — public contracts and downstream callers still work.
|
|
647
788
|
|
|
648
789
|
Never approve cleanup because it "looks cleaner" without evidence. Tests passing is necessary but not sufficient when the refactor changes structure across files.
|
|
649
|
-
`;var
|
|
790
|
+
`;var M=`# Parallel discovery
|
|
650
791
|
|
|
651
792
|
Use this only after a serial orientation pass has identified the repo shape and the likely slices. Workers are read-only evidence gatherers; the planner owns the plan.
|
|
652
793
|
|
|
@@ -679,6 +820,10 @@ For this repository, good first-wave slices are:
|
|
|
679
820
|
\`bun.lock\`, \`README.md\`, and \`CHANGELOG.md\`.
|
|
680
821
|
- Docs and operator contract: \`docs/**\`, \`README.md\`, and skill references.
|
|
681
822
|
|
|
823
|
+
Treat these as starting points, not a simultaneous coverage map. Before fan-out,
|
|
824
|
+
choose the relevant entries and de-overlap shared docs, skills, or release
|
|
825
|
+
surfaces in the coverage gate.
|
|
826
|
+
|
|
682
827
|
## Coverage gate
|
|
683
828
|
|
|
684
829
|
Before spawning workers, state the total discovery scope and one line per slice.
|
|
@@ -689,17 +834,19 @@ state the completeness rule, such as "all changed files plus callers."
|
|
|
689
834
|
## Worker prompt
|
|
690
835
|
|
|
691
836
|
\`\`\`text
|
|
692
|
-
Inspect <slice> for <goal>. Read-only. Do not edit files or call
|
|
693
|
-
Return the evidence/review/validation/audit handoff
|
|
837
|
+
Inspect <slice> for <goal>. Read-only. Do not edit files or call
|
|
838
|
+
state-changing Flow tools. Return the evidence/review/validation/audit handoff
|
|
839
|
+
shape from ../../flow/references/handoff-format.md.
|
|
694
840
|
\`\`\`
|
|
695
841
|
|
|
696
842
|
For validation-oriented discovery:
|
|
697
843
|
|
|
698
844
|
\`\`\`text
|
|
699
|
-
Inspect <slice> for validation risk. Read-only. Do not edit files or call
|
|
700
|
-
tools. You may report commands that should be run, and
|
|
701
|
-
for commands you actually ran. Return the
|
|
702
|
-
handoff shape from
|
|
845
|
+
Inspect <slice> for validation risk. Read-only. Do not edit files or call
|
|
846
|
+
state-changing Flow tools. You may report commands that should be run, and
|
|
847
|
+
include raw output only for commands you actually ran. Return the
|
|
848
|
+
evidence/review/validation/audit handoff shape from
|
|
849
|
+
../../flow/references/handoff-format.md.
|
|
703
850
|
\`\`\`
|
|
704
851
|
|
|
705
852
|
## Synthesis
|
|
@@ -712,7 +859,11 @@ Convert only evidence-backed work into plan fields:
|
|
|
712
859
|
- feature \`validation\`: checks expected to prove the feature.
|
|
713
860
|
|
|
714
861
|
If workers disagree, inspect the source artifact yourself. If a candidate finding lacks a concrete citation or refutation pass, make it a review-first deliverable rather than a fix feature.
|
|
715
|
-
|
|
862
|
+
|
|
863
|
+
Apply the manager synthesis barrier from
|
|
864
|
+
\`../../flow/references/verification-gates.md\`: only distilled, evidence-backed
|
|
865
|
+
claims become plan fields.
|
|
866
|
+
`;var L=`# Planning examples
|
|
716
867
|
|
|
717
868
|
## Rate limiting feature set
|
|
718
869
|
|
|
@@ -797,7 +948,7 @@ Better plan:
|
|
|
797
948
|
- Validation that only says "manual testing".
|
|
798
949
|
- Targets that name the entire repo.
|
|
799
950
|
- Features with hidden dependencies instead of \`dependsOn\`.
|
|
800
|
-
`;var
|
|
951
|
+
`;var $=`---
|
|
801
952
|
name: flow-plan
|
|
802
953
|
description: Plan Flow work for the v4 skills-first runtime: inspect the repo, decompose a user goal into right-sized features, save a draft with flow_plan_save, and approve it with flow_plan_approve.
|
|
803
954
|
---
|
|
@@ -1025,7 +1176,7 @@ Use \`status: "failed"\` when any blocking finding remains. Advisory findings ma
|
|
|
1025
1176
|
\`finalReview\` payload.
|
|
1026
1177
|
|
|
1027
1178
|
Never approve to unblock completion, fix findings in the review pass, or vouch for validation you did not inspect.
|
|
1028
|
-
`;var
|
|
1179
|
+
`;var Y=`# Audit findings rubric
|
|
1029
1180
|
|
|
1030
1181
|
What counts as a valid finding when the feature's deliverable is a findings report: a codebase audit, a review-first feature, or any report whose findings a later feature will fix. The commands you run are still governed by \`validation-rubric.md\`; this rubric governs the findings themselves.
|
|
1031
1182
|
|
|
@@ -1078,7 +1229,7 @@ follow-up order — correctness and persisted/user-input surfaces first
|
|
|
1078
1229
|
\`\`\`
|
|
1079
1230
|
|
|
1080
1231
|
Never: promote a hypothesis to blocking severity; cite a line you did not read in context; rate severity against a deployment model the product does not have; pad the report to look thorough — six verified findings outrank nine where three die on first contact.
|
|
1081
|
-
`;var
|
|
1232
|
+
`;var X=`# Validation evidence rubric
|
|
1082
1233
|
|
|
1083
1234
|
Use this before recording \`flow_feature_complete\`.
|
|
1084
1235
|
|
|
@@ -1135,7 +1286,7 @@ Broad validation usually means the repo's full check command, full relevant test
|
|
|
1135
1286
|
- If validation needs external access, missing credentials, or ambiguous user input, record \`status: "needs_input"\` with an honest \`outcome\`.
|
|
1136
1287
|
|
|
1137
1288
|
Never trim failing output, relabel a failed command as passed, or use "not run" as completion evidence.
|
|
1138
|
-
`;var
|
|
1289
|
+
`;var J=`---
|
|
1139
1290
|
name: flow-run
|
|
1140
1291
|
description: Execute one approved Flow feature in the v4 runtime: start a feature with flow_run_start, make scoped changes, gather real validation evidence, obtain review payloads, and complete with flow_feature_complete.
|
|
1141
1292
|
---
|
|
@@ -1478,23 +1629,23 @@ Approve only when the interface is both useful and inspectable:
|
|
|
1478
1629
|
- Screenshot/browser evidence supports the claim whenever feasible.
|
|
1479
1630
|
|
|
1480
1631
|
Never approve a UI change based only on code shape. If users will judge it visually, Flow evidence should include visual inspection.
|
|
1481
|
-
`;var y=[{name:"flow",files:[{relativePath:"SKILL.md",content:
|
|
1482
|
-
`)}async function m(e){try{return await Pe(e,"utf8")}catch(a){if(a.code==="ENOENT")return null;throw a}}function T(e){let a=new Map;if(!e)return a;for(let t of e.split(/\r?\n/)){let r=/^file=(.+) sha256=([a-f0-9]{64})$/.exec(t)??/^file=(.+)=sha256:([a-f0-9]{64})$/.exec(t);if(r?.[1]&&r[2])a.set(r[1],r[2]);let n=/^hash=sha256:([a-f0-9]{64})$/.exec(t);if(n?.[1]&&!a.has("SKILL.md"))a.set("SKILL.md",n[1])}return a}function
|
|
1632
|
+
`;var y=[{name:"flow",files:[{relativePath:"SKILL.md",content:z},{relativePath:"references/recovery-playbook.md",content:O},{relativePath:"references/parallel-orchestration.md",content:W},{relativePath:"references/handoff-format.md",content:S},{relativePath:"references/verification-gates.md",content:B}]},{name:"flow-plan",files:[{relativePath:"SKILL.md",content:$},{relativePath:"references/planning-examples.md",content:L},{relativePath:"references/parallel-discovery.md",content:M}]},{name:"flow-run",files:[{relativePath:"SKILL.md",content:J},{relativePath:"references/validation-rubric.md",content:X},{relativePath:"references/audit-rubric.md",content:Y}]},{name:"flow-test",files:[{relativePath:"SKILL.md",content:Z}]},{name:"flow-review",files:[{relativePath:"SKILL.md",content:K},{relativePath:"references/review-rubric.md",content:Q}]},{name:"flow-deslop",files:[{relativePath:"SKILL.md",content:V},{relativePath:"references/smell-rubric.md",content:G},{relativePath:"references/refactor-workflow.md",content:N}]},{name:"flow-ui-quality",files:[{relativePath:"SKILL.md",content:ae},{relativePath:"references/ui-rubric.md",content:H},{relativePath:"references/visual-verification.md",content:ee}]},{name:"flow-commit",files:[{relativePath:"SKILL.md",content:E}]}];var q=".flow-skill-version";function _(){return process.env.HOME??process.env.USERPROFILE??""}function D(e=_()){return u(e,".config","opencode","skills")}function g(e){return De("sha256").update(e).digest("hex")}function k(e,a){return[`version=${a}`,...e.files.map((t)=>`file=${t.relativePath} sha256=${g(t.content)}`),""].join(`
|
|
1633
|
+
`)}async function m(e){try{return await Pe(e,"utf8")}catch(a){if(a.code==="ENOENT")return null;throw a}}function T(e){let a=new Map;if(!e)return a;for(let t of e.split(/\r?\n/)){let r=/^file=(.+) sha256=([a-f0-9]{64})$/.exec(t)??/^file=(.+)=sha256:([a-f0-9]{64})$/.exec(t);if(r?.[1]&&r[2])a.set(r[1],r[2]);let n=/^hash=sha256:([a-f0-9]{64})$/.exec(t);if(n?.[1]&&!a.has("SKILL.md"))a.set("SKILL.md",n[1])}return a}function ze(e){if(!e)return null;for(let a of e.split(/\r?\n/)){let t=/^version=(.+)$/.exec(a);if(t?.[1])return t[1]}return null}function x(e,a){let t=Oe(u(e,...a.split("/")));if(t!==e&&t.startsWith(`${e}${Be}`))return t;throw Error(`Unsafe skill file path '${a}'.`)}async function Ee(e,a){let t=`${e}.backup.${g(a).slice(0,12)}`;for(let r=0;;r+=1){let n=r===0?t:`${t}.${r}`;try{return await b(n,a,{encoding:"utf8",flag:"wx"}),n}catch(s){if(s.code==="EEXIST")continue;throw s}}}async function Ne(e,a,t){let r=u(t,e.name),n=u(r,q),s=await m(n),d=T(s);if(await m(u(r,"SKILL.md"))!==null&&s===null)return{name:e.name,action:"skipped_foreign"};let i=!1,l=[];for(let o of e.files){let c=x(r,o.relativePath),h=await m(c);if(h===o.content)continue;i=!0;let v=d.get(o.relativePath);if(h!==null&&(v?g(h)!==v:s!==null))l.push(await Ee(c,h))}if(!i&&s===k(e,a))return{name:e.name,action:"unchanged"};if(!i)return await b(n,k(e,a),"utf8"),{name:e.name,action:"marker_updated"};let w=s!==null;for(let o of e.files){let c=x(r,o.relativePath);await Te(We(c),{recursive:!0}),await b(c,o.content,"utf8")}return await b(n,k(e,a),"utf8"),{name:e.name,action:l.length>0?"updated_with_backup":w?"updated":"installed",...l.length>0?{backupPaths:l}:{}}}function Ge(){return y.map((e)=>e.name)}function Ve(e){return`npx -y opencode-plugin-flow@${e} doctor`}function I(){if(process.env.npm_package_version)return process.env.npm_package_version;try{let e=Se(import.meta.url);for(let a of["../package.json","../../package.json"])try{let t=e(a);if(t.version)return t.version}catch{}}catch{}return"0.0.0"}async function re(e,a=_()){let t=D(a);return Promise.all(y.map((r)=>Ne(r,e,t)))}async function se(e=I(),a=_()){let t=D(a),r=new Set(Ge()),n=await Promise.all(y.map(async(o)=>{let c=u(t,o.name),h=await m(u(c,q)),v=ze(h),P=T(h);if(await m(u(c,"SKILL.md"))===null)return{name:o.name,path:c,status:"missing",markerVersion:v,missingFiles:o.files.map((f)=>f.relativePath),editedFiles:[],outdatedFiles:[]};if(h===null)return{name:o.name,path:c,status:"foreign",markerVersion:v,missingFiles:[],editedFiles:[],outdatedFiles:[]};let R=[],U=[],C=[];for(let f of o.files){let A=await m(x(c,f.relativePath));if(A===null){R.push(f.relativePath);continue}if(A===f.content)continue;let j=P.get(f.relativePath);if(j&&g(A)!==j){U.push(f.relativePath);continue}C.push(f.relativePath)}let ce=h!==k(o,e),de=R.length>0?"incomplete":U.length>0?"edited":ce||C.length>0?"outdated":"ok";return{name:o.name,path:c,status:de,markerVersion:v,missingFiles:R,editedFiles:U,outdatedFiles:C}})),s=[];try{s=await oe(t)}catch(o){if(o.code!=="ENOENT")throw o}let d=s.filter((o)=>(o==="flow"||o.startsWith("flow-"))&&!r.has(o)).map((o)=>u(t,o)),p=n.filter((o)=>["missing","incomplete","outdated"].includes(o.status)).map((o)=>o.name),i=n.filter((o)=>["foreign","edited"].includes(o.status)).map((o)=>o.name),l=i.length>0,w=p.length>0;return{status:l?"action_required":w?"sync_required":"ok",version:e,root:t,expectedSkills:[...r],skills:n,syncRequiredSkills:p,actionRequiredSkills:i,unmanagedFlowSkills:d}}function te(e,a,t){if(t.length===0)return;e.push(`- ${a}: ${t.join(", ")}`)}function ne(e){let a=["Flow doctor",`- status: ${e.status}`,`- plugin version: ${e.version}`,`- skills root: ${e.root}`,`- expected skills: ${e.expectedSkills.join(", ")}`];te(a,"startup sync can install/update",e.syncRequiredSkills),te(a,"needs user decision",e.actionRequiredSkills),a.push("","Skills:");for(let t of e.skills){if(a.push(`- ${t.name}: ${t.status} (${t.path})${t.markerVersion?` marker=${t.markerVersion}`:""}`),t.missingFiles.length>0)a.push(` missing: ${t.missingFiles.join(", ")}`);if(t.editedFiles.length>0)a.push(` edited: ${t.editedFiles.join(", ")}`);if(t.outdatedFiles.length>0)a.push(` outdated: ${t.outdatedFiles.join(", ")}`)}if(e.unmanagedFlowSkills.length>0){a.push("","Unmanaged Flow-like skill folders:");for(let t of e.unmanagedFlowSkills)a.push(`- ${t}`)}if(a.push("","Recommendation:"),e.status==="ok")a.push("- Flow skills are present and current.");else if(e.status==="sync_required")a.push("- Start or restart OpenCode with opencode-plugin-flow enabled so startup sync can install or update the listed skills. If Flow then reports restart_required, restart OpenCode once more so the refreshed skill registry is used.");else a.push("- Resolve user-owned or edited managed skill folders, then restart OpenCode. Move a folder aside to let Flow recreate it, or keep it intentionally as a local override.");return a.push(`- Details command: ${Ve(e.version)}`),`${a.join(`
|
|
1483
1634
|
`)}
|
|
1484
|
-
`}async function ie(e=_()){let a=D(e),t=[],r=[],n;try{n=await oe(a)}catch(s){if(s.code==="ENOENT")return{removed:t,kept:r};throw s}for(let s of n){if(s!=="flow"&&!s.startsWith("flow-"))continue;let d=u(a,s),
|
|
1485
|
-
`)}function
|
|
1486
|
-
`);return}process.stdout.write(ne(e))}async function
|
|
1635
|
+
`}async function ie(e=_()){let a=D(e),t=[],r=[],n;try{n=await oe(a)}catch(s){if(s.code==="ENOENT")return{removed:t,kept:r};throw s}for(let s of n){if(s!=="flow"&&!s.startsWith("flow-"))continue;let d=u(a,s),p=await m(u(d,q));if(p===null){r.push(d);continue}let i=T(p),l=!1;for(let[w,o]of i){let c=await m(x(d,w));if(c!==null&&g(c)!==o){l=!0;break}}if(l){r.push(d);continue}await je(d,{recursive:!0,force:!0}),t.push(d)}return{removed:t,kept:r}}function F(){return["usage: opencode-plugin-flow <doctor|sync|uninstall> [options]","","commands:"," doctor Inspect managed Flow skills"," sync Install or refresh managed Flow skills"," uninstall Remove pristine Flow-owned managed skills","","doctor options:"," --json Write the doctor report as JSON"," --check, --strict Exit nonzero when doctor status is not ok","","global options:"," --help Show this help"," --version Print the plugin version"].join(`
|
|
1636
|
+
`)}function Me(e,a){return e.every((t)=>a.has(t))}function Le(e,a){if(a.json){process.stdout.write(`${JSON.stringify(e,null,2)}
|
|
1637
|
+
`);return}process.stdout.write(ne(e))}async function $e(e){let a=e[2],t=e.slice(3);if(a==="--help"||a==="-h"){process.stdout.write(`${F()}
|
|
1487
1638
|
`);return}if(a==="--version"||a==="-v"){process.stdout.write(`${I()}
|
|
1488
1639
|
`);return}if(a!=="uninstall"&&a!=="doctor"&&a!=="sync"){process.stderr.write(`${F()}
|
|
1489
|
-
`),process.exitCode=2;return}if(a==="doctor"){if(!
|
|
1490
|
-
`),process.exitCode=2;return}let s=await se();if(
|
|
1491
|
-
`),process.exitCode=2;return}if(a==="sync"){let n=I(),s=await re(n),d=s.filter((i)=>["installed","updated","updated_with_backup"].includes(i.action)),
|
|
1640
|
+
`),process.exitCode=2;return}if(a==="doctor"){if(!Me(t,new Set(["--json","--check","--strict"]))){process.stderr.write(`${F()}
|
|
1641
|
+
`),process.exitCode=2;return}let s=await se();if(Le(s,{json:t.includes("--json")}),(s.status==="sync_required"||s.status==="action_required")&&(t.includes("--check")||t.includes("--strict")))process.exitCode=1;return}if(t.length>0){process.stderr.write(`${F()}
|
|
1642
|
+
`),process.exitCode=2;return}if(a==="sync"){let n=I(),s=await re(n),d=s.filter((i)=>["installed","updated","updated_with_backup"].includes(i.action)),p=s.filter((i)=>i.action==="skipped_foreign");process.stdout.write(`Flow skill sync (${n})
|
|
1492
1643
|
`);for(let i of s){process.stdout.write(`- ${i.name}: ${i.action}
|
|
1493
1644
|
`);for(let l of i.backupPaths??[])process.stdout.write(` backup: ${l}
|
|
1494
1645
|
`)}if(d.length>0)process.stdout.write(`Restart OpenCode so the refreshed skill registry is used.
|
|
1495
|
-
`);if(
|
|
1646
|
+
`);if(p.length>0)process.stdout.write(`Some managed skill folders are user-owned or edited; run doctor for repair guidance.
|
|
1496
1647
|
`);return}let r=await ie();for(let n of r.removed)process.stdout.write(`Removed Flow skill: ${n}
|
|
1497
1648
|
`);for(let n of r.kept)process.stdout.write(`Kept non-Flow or user-edited skill: ${n}
|
|
1498
1649
|
`);process.stdout.write(`Remove opencode-plugin-flow from your OpenCode plugin config and restart OpenCode.
|
|
1499
|
-
`)}
|
|
1650
|
+
`)}$e(process.argv).catch((e)=>{process.stderr.write(`${e instanceof Error?e.message:String(e)}
|
|
1500
1651
|
`),process.exitCode=1});
|