opencode-swarm 7.28.1 → 7.28.3

package/dist/cli/index.js

@@ -34,7 +34,7 @@ var package_default;
 var init_package = __esm(() => {
   package_default = {
     name: "opencode-swarm",
-    version: "7.28.1",
+    version: "7.28.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",
@@ -308,6 +308,27 @@ function bunHash(input) {
   }
   return hash;
 }
+function killProcessTreeImpl(pid, signal, directKill, wasDetached) {
+  if (typeof pid !== "number" || pid <= 0) {
+    directKill();
+    return;
+  }
+  if (process.platform === "win32") {
+    try {
+      nodeSpawnSync("taskkill", ["/PID", String(pid), "/T", "/F"]);
+    } catch {
+      directKill();
+    }
+    return;
+  }
+  if (wasDetached) {
+    try {
+      process.kill(-pid, signal ?? "SIGKILL");
+      return;
+    } catch {}
+  }
+  directKill();
+}
 function streamFromNode(pipe) {
   const collected = new Promise((resolve) => {
     if (!pipe) {
@@ -430,20 +451,34 @@ function bunSpawn(cmd, options) {
         return proc2.exitCode;
       },
       kill(sig) {
-        proc2.kill(sig);
+        if (options?.killProcessTree) {
+          killProcessTreeImpl(proc2.pid, sig, () => proc2.kill(sig), false);
+        } else {
+          proc2.kill(sig);
+        }
       }
     };
   }
   const [file, ...args] = cmd;
+  const detached = options?.killProcessTree === true;
   const proc = nodeSpawn(file, args, {
     cwd: options?.cwd,
     env: options?.env,
+    detached,
+    windowsHide: true,
     stdio: [
       mapStdio(options?.stdin),
       mapStdio(options?.stdout),
       mapStdio(options?.stderr)
     ]
   });
+  const killChild = (signal) => {
+    if (detached) {
+      killProcessTreeImpl(proc.pid, signal, () => proc.kill(signal), true);
+    } else {
+      proc.kill(signal);
+    }
+  };
   let timeoutHandle;
   const exited = new Promise((resolve) => {
     proc.on("exit", (code) => resolve(code ?? 0));
@@ -451,7 +486,7 @@ function bunSpawn(cmd, options) {
     if (options?.timeout && options.timeout > 0) {
       timeoutHandle = setTimeout(() => {
         try {
-          proc.kill("SIGKILL");
+          killChild("SIGKILL");
         } catch {}
       }, options.timeout);
       if (typeof timeoutHandle.unref === "function") {
@@ -471,7 +506,7 @@ function bunSpawn(cmd, options) {
     },
     kill(signal) {
       try {
-        proc.kill(signal);
+        killChild(signal);
       } catch {}
     }
   };
@@ -46255,7 +46290,7 @@ function defaultBuildTestCommand(profile, framework, files, dir = ".", opts = {}
   const coverage = opts.coverage ?? false;
   switch (framework) {
     case "bun": {
-      const args = ["bun", "test"];
+      const args = ["bun", "--smol", "test"];
       if (coverage)
         args.push("--coverage");
       if (scope !== "all" && files.length > 0)
@@ -48688,7 +48723,7 @@ function getTargetedExecutionUnsupportedReason(framework) {
 function buildTestCommand2(framework, scope, files, coverage, baseDir) {
   switch (framework) {
     case "bun": {
-      const args = ["bun", "test"];
+      const args = ["bun", "--smol", "test"];
       if (coverage)
         args.push("--coverage");
       if (scope !== "all" && files.length > 0) {
@@ -49225,17 +49260,24 @@ async function runTests(framework, scope, files, coverage, timeout_ms, cwd) {
     const proc = bunSpawn(command, {
       stdout: "pipe",
       stderr: "pipe",
-      cwd
+      stdin: "ignore",
+      cwd,
+      killProcessTree: true
+    });
+    let timeoutHandle;
+    const timeoutPromise = new Promise((resolve14) => {
+      timeoutHandle = setTimeout(() => {
+        proc.kill();
+        resolve14(-1);
+      }, timeout_ms);
     });
-    const timeoutPromise = new Promise((resolve14) => setTimeout(() => {
-      proc.kill();
-      resolve14(-1);
-    }, timeout_ms));
     const [exitCode, stdoutResult, stderrResult] = await Promise.all([
       Promise.race([proc.exited, timeoutPromise]),
       readBoundedStream(proc.stdout, MAX_OUTPUT_BYTES3),
       readBoundedStream(proc.stderr, MAX_OUTPUT_BYTES3)
     ]);
+    if (timeoutHandle !== undefined)
+      clearTimeout(timeoutHandle);
     const duration_ms = Date.now() - startTime;
     let output = stdoutResult.text;
     if (stderrResult.text) {
@@ -49538,7 +49580,6 @@ var init_test_runner = __esm(() => {
       files: exports_external.array(exports_external.string()).optional().describe('Specific files to test. For "convention", pass source files or direct test files. For "graph" and "impact", pass source files only.'),
       coverage: exports_external.boolean().optional().describe("Enable coverage reporting if supported"),
       timeout_ms: exports_external.number().optional().describe("Timeout in milliseconds (default 60000, max 300000)"),
-      allow_full_suite: exports_external.boolean().optional().describe('Explicit opt-in for scope "all". Required because full-suite output can destabilize SSE streaming.'),
       working_directory: exports_external.string().optional().describe("Explicit project root directory. When provided, tests run relative to this path instead of the plugin context directory. Use this when CWD differs from the actual project root.")
     },
     async execute(args, directory) {
@@ -49612,7 +49653,8 @@ var init_test_runner = __esm(() => {
       }
       const scope = args.scope || "all";
       if (scope === "all") {
-        if (!args.allow_full_suite) {
+        const fullSuiteAllowed = process.env.SWARM_ALLOW_FULL_SUITE === "1" || process.env.SWARM_ALLOW_FULL_SUITE === "true";
+        if (!fullSuiteAllowed) {
           const errorResult = {
             success: false,
             framework: "none",

package/dist/hooks/delegation-gate.d.ts

@@ -5,6 +5,7 @@
  * Uses experimental.chat.messages.transform to provide non-blocking guidance.
  */
 import type { PluginConfig } from '../config';
+import type { AgentSessionState } from '../state';
 import type { DelegationEnvelope, EnvelopeValidationResult } from '../types/delegation.js';
 /**
  * v6.33.1 CRIT-1: Fallback map for declared coder scope by taskId.
@@ -53,6 +54,26 @@ interface MessageWithParts {
     info: MessageInfo;
     parts: MessagePart[];
 }
+declare function resolveDelegatedPlanTaskId(args: Record<string, unknown>, knownPlanTaskIds?: ReadonlySet<string>): string | null;
+/**
+ * Resolves the correct task ID for evidence recording by chaining:
+ * 1. Explicit task_id in direct args (structured field)
+ * 2. Prompt-text extraction via resolveDelegatedPlanTaskId (plan-aware)
+ * 3. Session-state fallback via getEvidenceTaskId
+ *
+ * This fixes parallel evidence recording where multiple reviewer/test_engineer
+ * agents are dispatched for different tasks from the same architect session.
+ * Issue #970.
+ */
+declare function resolveEvidenceTaskId(args: Record<string, unknown> | undefined, session: AgentSessionState, directory: string): Promise<string | null>;
+/**
+ * _internals export for testing — do not use in production code.
+ * Exposes resolveEvidenceTaskId and resolveDelegatedPlanTaskId for unit testing.
+ */
+export declare const _internals: {
+    resolveEvidenceTaskId: typeof resolveEvidenceTaskId;
+    resolveDelegatedPlanTaskId: typeof resolveDelegatedPlanTaskId;
+};
 /**
  * Creates the experimental.chat.messages.transform hook for delegation gating.
  * Inspects coder delegations and warns when tasks are oversized or batched.

package/dist/index.js

@@ -48,7 +48,7 @@ var package_default;
 var init_package = __esm(() => {
   package_default = {
     name: "opencode-swarm",
-    version: "7.28.1",
+    version: "7.28.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",
@@ -16518,6 +16518,27 @@ function bunHash(input) {
   }
   return hash2;
 }
+function killProcessTreeImpl(pid, signal, directKill, wasDetached) {
+  if (typeof pid !== "number" || pid <= 0) {
+    directKill();
+    return;
+  }
+  if (process.platform === "win32") {
+    try {
+      nodeSpawnSync("taskkill", ["/PID", String(pid), "/T", "/F"]);
+    } catch {
+      directKill();
+    }
+    return;
+  }
+  if (wasDetached) {
+    try {
+      process.kill(-pid, signal ?? "SIGKILL");
+      return;
+    } catch {}
+  }
+  directKill();
+}
 function streamFromNode(pipe2) {
   const collected = new Promise((resolve) => {
     if (!pipe2) {
@@ -16640,20 +16661,34 @@ function bunSpawn(cmd, options) {
         return proc2.exitCode;
       },
       kill(sig) {
-        proc2.kill(sig);
+        if (options?.killProcessTree) {
+          killProcessTreeImpl(proc2.pid, sig, () => proc2.kill(sig), false);
+        } else {
+          proc2.kill(sig);
+        }
       }
     };
   }
   const [file2, ...args2] = cmd;
+  const detached = options?.killProcessTree === true;
   const proc = nodeSpawn(file2, args2, {
     cwd: options?.cwd,
     env: options?.env,
+    detached,
+    windowsHide: true,
     stdio: [
       mapStdio(options?.stdin),
       mapStdio(options?.stdout),
       mapStdio(options?.stderr)
     ]
   });
+  const killChild = (signal) => {
+    if (detached) {
+      killProcessTreeImpl(proc.pid, signal, () => proc.kill(signal), true);
+    } else {
+      proc.kill(signal);
+    }
+  };
   let timeoutHandle;
   const exited = new Promise((resolve) => {
     proc.on("exit", (code) => resolve(code ?? 0));
@@ -16661,7 +16696,7 @@ function bunSpawn(cmd, options) {
     if (options?.timeout && options.timeout > 0) {
       timeoutHandle = setTimeout(() => {
         try {
-          proc.kill("SIGKILL");
+          killChild("SIGKILL");
         } catch {}
       }, options.timeout);
       if (typeof timeoutHandle.unref === "function") {
@@ -16681,7 +16716,7 @@ function bunSpawn(cmd, options) {
     },
     kill(signal) {
       try {
-        proc.kill(signal);
+        killChild(signal);
       } catch {}
     }
   };
@@ -39012,6 +39047,24 @@ async function getEvidenceTaskId(session, directory) {
   }
   return null;
 }
+async function resolveEvidenceTaskId(args2, session, directory) {
+  const rawTaskId = args2?.task_id;
+  if (typeof rawTaskId === "string" && rawTaskId.length <= 20 && isStrictTaskId(rawTaskId.trim())) {
+    return rawTaskId.trim();
+  }
+  if (args2) {
+    try {
+      const plan = await loadPlanJsonOnly(directory);
+      if (plan) {
+        const planTaskIds = new Set(plan.phases.flatMap((p) => p.tasks.map((t) => t.id)));
+        const promptTaskId = resolveDelegatedPlanTaskId(args2, planTaskIds);
+        if (promptTaskId)
+          return promptTaskId;
+      }
+    } catch {}
+  }
+  return getEvidenceTaskId(session, directory);
+}
 function createDelegationGateHook(config2, directory) {
   const enabled = config2.hooks?.delegation_gate !== false;
   const delegationMaxChars = config2.hooks?.delegation_max_chars ?? 4000;
@@ -39253,8 +39306,8 @@ function createDelegationGateHook(config2, directory) {
       }
       if (typeof subagentType === "string") {
         try {
-          const rawTaskId = directArgs?.task_id;
-          const evidenceTaskId = typeof rawTaskId === "string" && rawTaskId.length <= 20 && isStrictTaskId(rawTaskId.trim()) ? rawTaskId.trim() : await getEvidenceTaskId(session, directory);
+          const mergedArgs = { ...storedArgs ?? {}, ...directArgs };
+          const evidenceTaskId = await resolveEvidenceTaskId(mergedArgs, session, directory);
           if (evidenceTaskId) {
             const turbo = hasActiveTurboMode(input.sessionID);
             const gateAgents = [
@@ -39376,8 +39429,7 @@ function createDelegationGateHook(config2, directory) {
           }
         }
         try {
-          const rawTaskId = directArgs?.task_id;
-          const evidenceTaskId = typeof rawTaskId === "string" && rawTaskId.length <= 20 && isStrictTaskId(rawTaskId.trim()) ? rawTaskId.trim() : await getEvidenceTaskId(session, directory);
+          const evidenceTaskId = await resolveEvidenceTaskId(directArgs, session, directory);
           if (evidenceTaskId) {
             const turbo = hasActiveTurboMode(input.sessionID);
             if (hasReviewer) {
@@ -67331,7 +67383,7 @@ function defaultBuildTestCommand(profile, framework, files, dir = ".", opts = {}
   const coverage = opts.coverage ?? false;
   switch (framework) {
     case "bun": {
-      const args2 = ["bun", "test"];
+      const args2 = ["bun", "--smol", "test"];
       if (coverage)
         args2.push("--coverage");
       if (scope !== "all" && files.length > 0)
@@ -69764,7 +69816,7 @@ function getTargetedExecutionUnsupportedReason(framework) {
 function buildTestCommand2(framework, scope, files, coverage, baseDir) {
   switch (framework) {
     case "bun": {
-      const args2 = ["bun", "test"];
+      const args2 = ["bun", "--smol", "test"];
       if (coverage)
         args2.push("--coverage");
       if (scope !== "all" && files.length > 0) {
@@ -70301,17 +70353,24 @@ async function runTests(framework, scope, files, coverage, timeout_ms, cwd) {
     const proc = bunSpawn(command, {
       stdout: "pipe",
       stderr: "pipe",
-      cwd
+      stdin: "ignore",
+      cwd,
+      killProcessTree: true
+    });
+    let timeoutHandle;
+    const timeoutPromise = new Promise((resolve16) => {
+      timeoutHandle = setTimeout(() => {
+        proc.kill();
+        resolve16(-1);
+      }, timeout_ms);
     });
-    const timeoutPromise = new Promise((resolve16) => setTimeout(() => {
-      proc.kill();
-      resolve16(-1);
-    }, timeout_ms));
     const [exitCode, stdoutResult, stderrResult] = await Promise.all([
       Promise.race([proc.exited, timeoutPromise]),
       readBoundedStream(proc.stdout, MAX_OUTPUT_BYTES3),
       readBoundedStream(proc.stderr, MAX_OUTPUT_BYTES3)
     ]);
+    if (timeoutHandle !== undefined)
+      clearTimeout(timeoutHandle);
     const duration_ms = Date.now() - startTime;
     let output = stdoutResult.text;
     if (stderrResult.text) {
@@ -70614,7 +70673,6 @@ var init_test_runner = __esm(() => {
       files: exports_external.array(exports_external.string()).optional().describe('Specific files to test. For "convention", pass source files or direct test files. For "graph" and "impact", pass source files only.'),
       coverage: exports_external.boolean().optional().describe("Enable coverage reporting if supported"),
       timeout_ms: exports_external.number().optional().describe("Timeout in milliseconds (default 60000, max 300000)"),
-      allow_full_suite: exports_external.boolean().optional().describe('Explicit opt-in for scope "all". Required because full-suite output can destabilize SSE streaming.'),
       working_directory: exports_external.string().optional().describe("Explicit project root directory. When provided, tests run relative to this path instead of the plugin context directory. Use this when CWD differs from the actual project root.")
     },
     async execute(args2, directory) {
@@ -70688,7 +70746,8 @@ var init_test_runner = __esm(() => {
       }
       const scope = args2.scope || "all";
       if (scope === "all") {
-        if (!args2.allow_full_suite) {
+        const fullSuiteAllowed = process.env.SWARM_ALLOW_FULL_SUITE === "1" || process.env.SWARM_ALLOW_FULL_SUITE === "true";
+        if (!fullSuiteAllowed) {
           const errorResult = {
             success: false,
             framework: "none",

package/dist/tools/test-runner.d.ts

@@ -23,7 +23,6 @@ export interface TestRunnerArgs {
     files?: string[];
     coverage?: boolean;
     timeout_ms?: number;
-    allow_full_suite?: boolean;
 }
 export type RegressionOutcome = 'pass' | 'skip' | 'regression' | 'scope_exceeded' | 'error';
 export interface TestTotals {

package/dist/utils/bun-compat.d.ts

@@ -71,6 +71,16 @@ export interface BunCompatSpawnOptions {
     stdout?: 'inherit' | 'ignore' | 'pipe';
     stderr?: 'inherit' | 'ignore' | 'pipe';
     timeout?: number;
+    /**
+     * When true, spawn the child as its own process-group leader (Node path:
+     * `detached`) and kill the entire descendant tree on `kill()`/timeout
+     * rather than only the direct child. A test runner that forks worker
+     * processes (jest/vitest, or a runaway suite) can otherwise outlive a
+     * `proc.kill()` of the parent and keep consuming memory after the timeout.
+     * Opt-in because the default single-child kill is correct for the many
+     * short-lived `bunSpawn` callers (git, lint, version checks).
+     */
+    killProcessTree?: boolean;
 }
 export interface BunCompatStream {
     text(): Promise<string>;

package/package.json

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