opencode-swarm 7.88.2 → 7.88.3

package/.opencode/skills/phase-wrap/SKILL.md

@@ -126,14 +126,31 @@ The tool will automatically write the retrospective to \`.swarm/evidence/retro-{
    - If `auto-proceed: off` AND `nudge: false`: after the user confirms the phase transition, suggest enabling auto-proceed. Use the swarm_command tool to record the user's answer: `swarm_command({ command: "auto-proceed", args: ["on"] })` for yes, `swarm_command({ command: "auto-proceed", args: ["off"] })` for no. Either call sets nudge to true and prevents re-nudging.
    - If `auto-proceed: off` AND `nudge: true`: Ask "Ready for Phase [N+1]?" and wait for user confirmation before proceeding.
 
+5.59. **Required agent dispatch for phase_complete**: Before calling `phase_complete`, the architect MUST have dispatched each of the active swarm's standard agents at least once during this phase. By default, `phase_complete` requires these agents:
+
+| Agent | When required | Where dispatched during normal task execution |
+|---|---|---|
+| `coder` | Always | Task implementation (coder) |
+| `reviewer` | Always | Task review (reviewer) |
+| `test_engineer` | Always | Test verification (test_engineer) |
+| `docs` | When `require_docs: true` in QA gate profile | Documentation updates |
+
+If any required agent is missing, `phase_complete` returns `{ success: false, status: 'incomplete', message: 'Phase N incomplete: missing required agents: <list>', agentsMissing: [...] }` and the phase is not closed. Dispatch each agent during normal task execution (not only inside optional Phase/Final Councils in steps 5.65/5.7) so the closeout gate is satisfied.
+
+The `docs` agent is only required when `require_docs: true` in the effective QA gate profile (visible via `get_qa_gate_profile`). For most small plans and feedback cycles, `docs` is NOT required and can be skipped. For multi-task implementation plans, `docs` is typically required.
+
+The `coder` and `test_engineer` agents are required because every phase that modifies source code or tests must have at least one implementation and one test-verification delegation. For pure documentation or retrospective phases, these may be waived by the user explicitly.
+
+This is a hard enforcement mechanism, not a suggestion. `phase_complete` will not return `status: success` if any required agent is missing from `agentsDispatched`.
+
 CATASTROPHIC VIOLATION CHECK — ask yourself at EVERY phase boundary (MODE: PHASE-WRAP):
-"Have I delegated to the active swarm's reviewer agent at least once this phase?"
-If the answer is NO: you have a catastrophic process violation.
+"Have I delegated to each of the active swarm's required agents (coder, reviewer, test_engineer, plus docs if required) at least once this phase?"
+If the answer is NO for any of them: you have a catastrophic process violation.
 STOP. Do not proceed to the next phase. Inform the user:
-"⛔ PROCESS VIOLATION: Phase [N] completed with zero reviewer-agent delegations in the active swarm.
-All code changes in this phase are unreviewed. Recommend retrospective review before proceeding."
-This is not optional. Zero active-swarm reviewer calls in a phase is always a violation.
-There is no project where code ships without review.
+"⛔ PROCESS VIOLATION: Phase [N] completed with missing required-agent delegations in the active swarm: [list missing agents].
+All code changes in this phase are unreviewed/untested/undocumented. Recommend retrospective review before proceeding."
+This is not optional. Missing required-agent calls in a phase is always a violation.
+There is no project where code ships without review, tests, and required documentation.
 
 ### Blockers
 Mark [BLOCKED] in plan.md, skip to next unblocked task, inform user.

package/.opencode/skills/swarm-pr-feedback/SKILL.md

@@ -134,8 +134,38 @@ tree:
   filesystem (`Read`/`Glob`/`Grep`), and fixes must land on the PR branch — without a
   checkout you would verify and patch the base branch's code instead. Record the
   `base_ref..head_ref` range for diff-scoped inspection.
-- If no PR reference was provided (a pasted-feedback session on the current branch),
-  confirm the current branch is the intended PR branch before editing.
+ - If no PR reference was provided (a pasted-feedback session on the current branch),
+   confirm the current branch is the intended PR branch before editing.
+
+When a verification lane result includes `output_ref`, treat `output` as a
+preview and call `retrieve_lane_output` before using it to classify, resolve,
+disprove, or group feedback items. If the result is `output_degraded`,
+`transcript_incomplete`, or truncated without a usable ref, keep the affected
+ledger items as `NEEDS_MORE_EVIDENCE` or re-dispatch a narrower read-only lane.
+
+## Pre-flight: Dirty Worktree Handling
+
+Before staging any files for the PR commit, check the working tree state:
+
+**The problem:** `git add -A` stages every uncommitted change in the working tree,
+including pre-existing changes from other branches or prior work. This was hit twice
+in one session during PR #1472 review, producing a 59-file commit instead of the
+intended 2-file targeted fix.
+
+**The check:** Run `git status --porcelain` first. If output is non-empty, identify
+which files are PR-related vs pre-existing uncommitted changes.
+
+**The rule:** Stage files explicitly by path when the working tree contains files
+unrelated to the PR. For example:
+
+```bash
+git add src/foo.ts tests/foo.test.ts
+```
+
+Never use `git add -A` when the working tree has pre-existing changes from other
+branches or prior work sessions.
+
+*Reference: Caught during PR #1472 Round 1 closure.*
 
 ## Intake Surfaces
 
@@ -172,12 +202,6 @@ Missing, stale, cancelled, or failed lanes remain explicit ledger limitations.
 If `dispatch_lanes_async` is unavailable, use blocking verification and record
 that async advisory lanes were unavailable.
 
-When a verification lane result includes `output_ref`, treat `output` as a
-preview and call `retrieve_lane_output` before using it to classify, resolve,
-disprove, or group feedback items. If the result is `output_degraded`,
-`transcript_incomplete`, or truncated without a usable ref, keep the affected
-ledger items as `NEEDS_MORE_EVIDENCE` or re-dispatch a narrower read-only lane.
-
 ### CI matrix cascade check (do this before fixing)
 
 When the PR's `unit` job is a matrix across multiple OSes and downstream jobs

package/dist/cli/{guardrail-explain-qd243wrm.js → guardrail-explain-eqypvw60.js}

@@ -1,8 +1,8 @@
 // @bun
 import {
   handleGuardrailExplain
-} from "./index-09smngfp.js";
-import"./index-y72bw1wb.js";
+} from "./index-ds057q5k.js";
+import"./index-g6f4tt38.js";
 import"./index-g00qm2gf.js";
 import"./index-yhsmmv2z.js";
 import"./index-32axfg6h.js";

package/dist/cli/{index-tjr1m8wf.js → index-0rt5aamg.js}

@@ -1,7 +1,7 @@
 // @bun
 import {
   handleGuardrailExplain
-} from "./index-09smngfp.js";
+} from "./index-ds057q5k.js";
 import {
   handleGuardrailLog
 } from "./index-vjsr9bqt.js";
@@ -76,7 +76,7 @@ import {
   handleWriteRetroCommand,
   normalizeSwarmCommandInput,
   resolveCommand
-} from "./index-y72bw1wb.js";
+} from "./index-g6f4tt38.js";
 import"./index-g00qm2gf.js";
 import"./index-yhsmmv2z.js";
 import"./index-32axfg6h.js";

package/dist/cli/{index-09smngfp.js → index-ds057q5k.js}

@@ -12,7 +12,7 @@ import {
   detectPosixWrites,
   detectWindowsWrites,
   resolveWriteTargets
-} from "./index-y72bw1wb.js";
+} from "./index-g6f4tt38.js";
 import {
   checkFileAuthority,
   classifyFile,

package/dist/cli/{index-y72bw1wb.js → index-g6f4tt38.js}

@@ -899,7 +899,7 @@ var init_executor = __esm(() => {
 // package.json
 var package_default = {
   name: "opencode-swarm",
-  version: "7.88.2",
+  version: "7.88.3",
   description: "Architect-centric agentic swarm plugin for OpenCode - hub-and-spoke orchestration with SME consultation, code generation, and QA review",
   main: "dist/index.js",
   types: "dist/index.d.ts",
@@ -29934,7 +29934,7 @@ function buildDetailedHelp(commandName, entry) {
 async function handleHelpCommand(ctx) {
   const targetCommand = ctx.args.join(" ");
   if (!targetCommand) {
-    const { buildHelpText } = await import("./index-tjr1m8wf.js");
+    const { buildHelpText } = await import("./index-0rt5aamg.js");
     return buildHelpText();
   }
   const tokens = targetCommand.split(/\s+/);
@@ -29943,7 +29943,7 @@ async function handleHelpCommand(ctx) {
     return _internals45.buildDetailedHelp(resolved.key, resolved.entry);
   }
   const similar = _internals45.findSimilarCommands(targetCommand);
-  const { buildHelpText: fullHelp } = await import("./index-tjr1m8wf.js");
+  const { buildHelpText: fullHelp } = await import("./index-0rt5aamg.js");
   if (similar.length > 0) {
     return `Command '/swarm ${targetCommand}' not found.
 
@@ -30076,7 +30076,7 @@ var COMMAND_REGISTRY = {
   },
   "guardrail explain": {
     handler: async (ctx) => {
-      const { handleGuardrailExplain } = await import("./guardrail-explain-qd243wrm.js");
+      const { handleGuardrailExplain } = await import("./guardrail-explain-eqypvw60.js");
       return handleGuardrailExplain(ctx.directory, ctx.args);
     },
     description: "Dry-run: show what the guardrails would do to a command or write target (executes nothing)",

package/dist/cli/index.js

@@ -7,7 +7,7 @@ import {
   getPluginLockFilePaths,
   package_default,
   resolveCommand
-} from "./index-y72bw1wb.js";
+} from "./index-g6f4tt38.js";
 import"./index-g00qm2gf.js";
 import"./index-yhsmmv2z.js";
 import"./index-32axfg6h.js";

package/dist/index.js

@@ -69,7 +69,7 @@ var package_default;
 var init_package = __esm(() => {
   package_default = {
     name: "opencode-swarm",
-    version: "7.88.2",
+    version: "7.88.3",
     description: "Architect-centric agentic swarm plugin for OpenCode - hub-and-spoke orchestration with SME consultation, code generation, and QA review",
     main: "dist/index.js",
     types: "dist/index.d.ts",

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "opencode-swarm",
-	"version": "7.88.2",
+	"version": "7.88.3",
 	"description": "Architect-centric agentic swarm plugin for OpenCode - hub-and-spoke orchestration with SME consultation, code generation, and QA review",
 	"main": "dist/index.js",
 	"types": "dist/index.d.ts",