opencode-swarm 7.88.1 → 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/README.md

@@ -1106,7 +1106,7 @@ Control how tool outputs are summarized for LLM context.
 | `/swarm pr-feedback [<pr-url\|owner/repo#N\|N>] [instructions...]` | Ingest and close known PR feedback (review comments, CI failures, conflicts) without a fresh review |
 | `/swarm pr subscribe <pr-url\|owner/repo#N\|N>` | Subscribe current session to PR monitoring (session-scoped); requires `pr_monitor.enabled: true` |
 | `/swarm pr unsubscribe <pr-url\|owner/repo#N\|N>` | Remove session's subscription to a PR |
-| `/swarm pr status` | List active PR subscriptions for current session with relative timestamps |
+| `/swarm pr status` | List active PR subscriptions for current session with relative timestamps (the `bunx opencode-swarm run pr status` CLI has no session context, so it lists subscriptions across all sessions) |
 | `/swarm deep-dive <scope> [--profile <name>] [--max-explorers <n>]` | Read-only codebase audit with parallel explorers, dual reviewers, and critic challenge |
 | `/swarm design-docs <description> [--out <dir>] [--lang <name>] [--update]` | Generate or sync language-agnostic design docs (requires `design_docs.enabled`) |
 | `/swarm dark-matter` | Detect hidden file couplings from co-change history |

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

@@ -1,8 +1,8 @@
 // @bun
 import {
   handleGuardrailExplain
-} from "./index-tt5aehrb.js";
-import"./index-4td9ef53.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-sr7g2msm.js → index-0rt5aamg.js}

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

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

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

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

@@ -899,7 +899,7 @@ var init_executor = __esm(() => {
 // package.json
 var package_default = {
   name: "opencode-swarm",
-  version: "7.88.1",
+  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",
@@ -22294,31 +22294,35 @@ var _internals34 = {
   formatRelativeTime,
   listActive
 };
-async function handlePrMonitorStatusCommand(directory, _args, sessionID) {
+async function handlePrMonitorStatusCommand(directory, _args, sessionID, source) {
   const allActive = await _internals34.listActive(directory);
-  const sessionSubs = allActive.filter((record) => record.sessionID === sessionID);
-  if (sessionSubs.length === 0) {
-    return "No active PR subscriptions for this session.";
+  const allSessions = source === "cli";
+  const subs = allSessions ? allActive : allActive.filter((record) => record.sessionID === sessionID);
+  if (subs.length === 0) {
+    return allSessions ? "No active PR subscriptions." : "No active PR subscriptions for this session.";
   }
   const lines = [];
-  lines.push(`PR Monitor Status \u2014 Session: ${sessionID}`);
+  lines.push(allSessions ? "PR Monitor Status \u2014 all sessions" : `PR Monitor Status \u2014 Session: ${sessionID}`);
   lines.push("");
   const totalActive = allActive.length;
-  lines.push(`Active subscriptions (${sessionSubs.length}):`);
-  for (let i = 0;i < sessionSubs.length; i++) {
-    const sub = sessionSubs[i];
+  lines.push(`Active subscriptions (${subs.length}):`);
+  for (let i = 0;i < subs.length; i++) {
+    const sub = subs[i];
     const index = i + 1;
     lines.push(`  ${index}. ${sub.repoFullName}#${sub.prNumber}`);
     lines.push(`     URL: ${sub.prUrl}`);
+    if (allSessions) {
+      lines.push(`     Session: ${sub.sessionID}`);
+    }
     lines.push(`     Last checked: ${formatRelativeTime(sub.lastCheckedAt)}`);
     lines.push(`     Watching: ${sub.isWatching ? "yes" : "no"}`);
     lines.push(`     Errors: ${sub.errorCount}`);
-    if (i < sessionSubs.length - 1) {
+    if (i < subs.length - 1) {
       lines.push("");
     }
   }
   lines.push("");
-  if (totalActive !== sessionSubs.length) {
+  if (!allSessions && totalActive !== subs.length) {
     lines.push(`Total active across all sessions: ${totalActive}`);
   }
   return lines.join(`
@@ -22415,6 +22419,15 @@ async function handlePrSubscribeCommand(directory, args, sessionID) {
       "Expected: full GitHub URL, owner/repo#N shorthand,",
       "or a bare PR number (resolved against origin)."
     ].join(`
+`);
+  }
+  if (!sessionID || sessionID.trim() === "") {
+    return [
+      "Error: Cannot subscribe \u2014 no active session.",
+      "",
+      "PR subscriptions are session-scoped and require a live OpenCode session.",
+      "Subscribe from inside OpenCode; the bunx CLI has no session context."
+    ].join(`
 `);
   }
   const repoFullName = `${prInfo.owner}/${prInfo.repo}`;
@@ -29921,7 +29934,7 @@ function buildDetailedHelp(commandName, entry) {
 async function handleHelpCommand(ctx) {
   const targetCommand = ctx.args.join(" ");
   if (!targetCommand) {
-    const { buildHelpText } = await import("./index-sr7g2msm.js");
+    const { buildHelpText } = await import("./index-0rt5aamg.js");
     return buildHelpText();
   }
   const tokens = targetCommand.split(/\s+/);
@@ -29930,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-sr7g2msm.js");
+  const { buildHelpText: fullHelp } = await import("./index-0rt5aamg.js");
   if (similar.length > 0) {
     return `Command '/swarm ${targetCommand}' not found.
 
@@ -30063,7 +30076,7 @@ var COMMAND_REGISTRY = {
   },
   "guardrail explain": {
     handler: async (ctx) => {
-      const { handleGuardrailExplain } = await import("./guardrail-explain-995zavv8.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)",
@@ -30436,7 +30449,7 @@ Subcommands:
     deprecated: true
   },
   "pr status": {
-    handler: (ctx) => handlePrMonitorStatusCommand(ctx.directory, ctx.args, ctx.sessionID),
+    handler: (ctx) => handlePrMonitorStatusCommand(ctx.directory, ctx.args, ctx.sessionID, ctx.source),
     description: "Show PR monitor subscription status for the current session",
     args: "",
     details: "Displays all active PR subscriptions for the current session. Shows PR URL, last checked time, watching status, and error count per subscription. Also shows total active subscriptions across all sessions.",
@@ -30445,7 +30458,7 @@ Subcommands:
     toolNoArgs: true
   },
   "pr-status": {
-    handler: (ctx) => handlePrMonitorStatusCommand(ctx.directory, ctx.args, ctx.sessionID),
+    handler: (ctx) => handlePrMonitorStatusCommand(ctx.directory, ctx.args, ctx.sessionID, ctx.source),
     description: "Show PR monitor subscription status for the current session",
     aliasOf: "pr status",
     deprecated: true

package/dist/cli/index.js

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

package/dist/commands/pr-monitor-status.d.ts

@@ -32,5 +32,5 @@ export declare const _internals: {
  * time, watching status, and error count. Appends a cross-session
  * total at the end.
  */
-export declare function handlePrMonitorStatusCommand(directory: string, _args: string[], sessionID: string): Promise<string>;
+export declare function handlePrMonitorStatusCommand(directory: string, _args: string[], sessionID: string, source?: 'cli' | 'chat'): Promise<string>;
 export {};

package/dist/index.js

@@ -69,7 +69,7 @@ var package_default;
 var init_package = __esm(() => {
   package_default = {
     name: "opencode-swarm",
-    version: "7.88.1",
+    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",
@@ -87852,31 +87852,35 @@ function formatRelativeTime(epochMs) {
   const diffDays = Math.floor(diffHours / 24);
   return `${diffDays} day${diffDays === 1 ? "" : "s"} ago`;
 }
-async function handlePrMonitorStatusCommand(directory, _args, sessionID) {
+async function handlePrMonitorStatusCommand(directory, _args, sessionID, source) {
   const allActive = await _internals55.listActive(directory);
-  const sessionSubs = allActive.filter((record3) => record3.sessionID === sessionID);
-  if (sessionSubs.length === 0) {
-    return "No active PR subscriptions for this session.";
+  const allSessions = source === "cli";
+  const subs = allSessions ? allActive : allActive.filter((record3) => record3.sessionID === sessionID);
+  if (subs.length === 0) {
+    return allSessions ? "No active PR subscriptions." : "No active PR subscriptions for this session.";
   }
   const lines = [];
-  lines.push(`PR Monitor Status — Session: ${sessionID}`);
+  lines.push(allSessions ? "PR Monitor Status — all sessions" : `PR Monitor Status — Session: ${sessionID}`);
   lines.push("");
   const totalActive = allActive.length;
-  lines.push(`Active subscriptions (${sessionSubs.length}):`);
-  for (let i2 = 0;i2 < sessionSubs.length; i2++) {
-    const sub = sessionSubs[i2];
+  lines.push(`Active subscriptions (${subs.length}):`);
+  for (let i2 = 0;i2 < subs.length; i2++) {
+    const sub = subs[i2];
     const index = i2 + 1;
     lines.push(`  ${index}. ${sub.repoFullName}#${sub.prNumber}`);
     lines.push(`     URL: ${sub.prUrl}`);
+    if (allSessions) {
+      lines.push(`     Session: ${sub.sessionID}`);
+    }
     lines.push(`     Last checked: ${formatRelativeTime(sub.lastCheckedAt)}`);
     lines.push(`     Watching: ${sub.isWatching ? "yes" : "no"}`);
     lines.push(`     Errors: ${sub.errorCount}`);
-    if (i2 < sessionSubs.length - 1) {
+    if (i2 < subs.length - 1) {
       lines.push("");
     }
   }
   lines.push("");
-  if (totalActive !== sessionSubs.length) {
+  if (!allSessions && totalActive !== subs.length) {
     lines.push(`Total active across all sessions: ${totalActive}`);
   }
   return lines.join(`
@@ -87985,6 +87989,15 @@ async function handlePrSubscribeCommand(directory, args2, sessionID) {
       "Expected: full GitHub URL, owner/repo#N shorthand,",
       "or a bare PR number (resolved against origin)."
     ].join(`
+`);
+  }
+  if (!sessionID || sessionID.trim() === "") {
+    return [
+      "Error: Cannot subscribe — no active session.",
+      "",
+      "PR subscriptions are session-scoped and require a live OpenCode session.",
+      "Subscribe from inside OpenCode; the bunx CLI has no session context."
+    ].join(`
 `);
   }
   const repoFullName = `${prInfo.owner}/${prInfo.repo}`;
@@ -99394,7 +99407,7 @@ Subcommands:
       deprecated: true
     },
     "pr status": {
-      handler: (ctx) => handlePrMonitorStatusCommand(ctx.directory, ctx.args, ctx.sessionID),
+      handler: (ctx) => handlePrMonitorStatusCommand(ctx.directory, ctx.args, ctx.sessionID, ctx.source),
       description: "Show PR monitor subscription status for the current session",
       args: "",
       details: "Displays all active PR subscriptions for the current session. Shows PR URL, last checked time, watching status, and error count per subscription. Also shows total active subscriptions across all sessions.",
@@ -99403,7 +99416,7 @@ Subcommands:
       toolNoArgs: true
     },
     "pr-status": {
-      handler: (ctx) => handlePrMonitorStatusCommand(ctx.directory, ctx.args, ctx.sessionID),
+      handler: (ctx) => handlePrMonitorStatusCommand(ctx.directory, ctx.args, ctx.sessionID, ctx.source),
       description: "Show PR monitor subscription status for the current session",
       aliasOf: "pr status",
       deprecated: true

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "opencode-swarm",
-	"version": "7.88.1",
+	"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",