opencode-swarm 7.29.2 → 7.29.4

package/dist/cli/index.js

@@ -34,7 +34,7 @@ var package_default;
 var init_package = __esm(() => {
   package_default = {
     name: "opencode-swarm",
-    version: "7.29.2",
+    version: "7.29.4",
     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",
@@ -505,9 +505,7 @@ function bunSpawn(cmd, options) {
       return proc.exitCode;
     },
     kill(signal) {
-      try {
-        killChild(signal);
-      } catch {}
+      killChild(signal);
     }
   };
 }
@@ -19859,15 +19857,36 @@ function validateProjectRoot(directory) {
     throw new Error(`Cannot verify project root for "${directory}" \u2014 directory may not exist or is inaccessible`);
   }
   let current = resolved;
+  let depth = 0;
   while (true) {
+    if (depth >= MAX_DEPTH)
+      break;
+    depth++;
     const parent = path7.dirname(current);
     if (parent === current)
       break;
     const parentSwarm = path7.join(parent, ".swarm");
     try {
       if (statSync4(parentSwarm).isDirectory()) {
-        warn(`[evidence] Rejecting write to subdirectory "${resolved}" \u2014 parent "${parent}" already contains .swarm/`);
-        throw new Error(`Cannot write evidence in "${resolved}" \u2014 parent directory "${parent}" already contains a .swarm/ folder. Evidence must be written to the project root.`);
+        let hasProjectIndicator = false;
+        for (const indicator of PROJECT_INDICATORS) {
+          try {
+            const indicatorStat = statSync4(path7.join(parent, indicator));
+            if (indicatorStat.isFile() || indicatorStat.isDirectory()) {
+              hasProjectIndicator = true;
+              break;
+            }
+          } catch (error49) {
+            if (error49 instanceof Error && "code" in error49 && error49.code === "ENOENT") {} else {
+              hasProjectIndicator = true;
+              break;
+            }
+          }
+        }
+        if (hasProjectIndicator) {
+          warn(`[evidence] Rejecting write to subdirectory "${resolved}" \u2014 parent "${parent}" already contains .swarm/`);
+          throw new Error(`Cannot write evidence in "${resolved}" \u2014 parent directory "${parent}" already contains a .swarm/ folder. Evidence must be written to the project root.`);
+        }
       }
     } catch (error49) {
       if (error49 instanceof Error && error49.message.startsWith("Cannot write evidence")) {
@@ -20110,7 +20129,7 @@ async function archiveEvidence(directory, maxAgeDays, maxBundles) {
   }
   return archived;
 }
-var VALID_EVIDENCE_TYPES, sanitizeTaskId2, LEGACY_TASK_COMPLEXITY_MAP, _internals5;
+var VALID_EVIDENCE_TYPES, sanitizeTaskId2, MAX_DEPTH = 20, PROJECT_INDICATORS, LEGACY_TASK_COMPLEXITY_MAP, _internals5;
 var init_manager2 = __esm(() => {
   init_zod();
   init_evidence_schema();
@@ -20135,6 +20154,19 @@ var init_manager2 = __esm(() => {
     "secretscan"
   ];
   sanitizeTaskId2 = sanitizeTaskId;
+  PROJECT_INDICATORS = [
+    "package.json",
+    ".git",
+    ".opencode",
+    "Cargo.toml",
+    "go.mod",
+    "pyproject.toml",
+    "Gemfile",
+    "composer.json",
+    "pom.xml",
+    "build.gradle",
+    "CMakeLists.txt"
+  ];
   LEGACY_TASK_COMPLEXITY_MAP = {
     low: "simple",
     medium: "moderate",
@@ -40695,10 +40727,10 @@ function detectStraySwarmDirs(projectRoot) {
     "__pycache__",
     ".tox"
   ]);
-  const MAX_DEPTH = 10;
+  const MAX_DEPTH2 = 10;
   const MAX_CONTENTS_ENTRIES = 20;
   function walk(dir, depth) {
-    if (depth > MAX_DEPTH)
+    if (depth > MAX_DEPTH2)
       return;
     let entries;
     try {
@@ -46282,7 +46314,7 @@ function defaultBuildTestCommand(profile, framework, files, dir = ".", opts = {}
   const coverage = opts.coverage ?? false;
   switch (framework) {
     case "bun": {
-      const args = ["bun", "--smol", "test"];
+      const args = ["bun", "test"];
       if (coverage)
         args.push("--coverage");
       if (scope !== "all" && files.length > 0)
@@ -48715,7 +48747,7 @@ function getTargetedExecutionUnsupportedReason(framework) {
 function buildTestCommand2(framework, scope, files, coverage, baseDir) {
   switch (framework) {
     case "bun": {
-      const args = ["bun", "--smol", "test"];
+      const args = ["bun", "test"];
       if (coverage)
         args.push("--coverage");
       if (scope !== "all" && files.length > 0) {
@@ -49252,24 +49284,17 @@ async function runTests(framework, scope, files, coverage, timeout_ms, cwd) {
     const proc = bunSpawn(command, {
       stdout: "pipe",
       stderr: "pipe",
-      stdin: "ignore",
-      cwd,
-      killProcessTree: true
-    });
-    let timeoutHandle;
-    const timeoutPromise = new Promise((resolve14) => {
-      timeoutHandle = setTimeout(() => {
-        proc.kill();
-        resolve14(-1);
-      }, timeout_ms);
+      cwd
     });
+    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) {
@@ -49572,6 +49597,7 @@ 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) {
@@ -49645,8 +49671,7 @@ var init_test_runner = __esm(() => {
       }
       const scope = args.scope || "all";
       if (scope === "all") {
-        const fullSuiteAllowed = process.env.SWARM_ALLOW_FULL_SUITE === "1" || process.env.SWARM_ALLOW_FULL_SUITE === "true";
-        if (!fullSuiteAllowed) {
+        if (!process.env.SWARM_ALLOW_FULL_SUITE) {
           const errorResult = {
             success: false,
             framework: "none",

package/dist/evidence/manager.d.ts

@@ -38,11 +38,17 @@ export declare function isQualityBudgetEvidence(evidence: Evidence): evidence is
 export declare function isSecretscanEvidence(evidence: Evidence): evidence is SecretscanEvidence;
 import { sanitizeTaskId as _sanitizeTaskId } from '../validation/task-id';
 export declare const sanitizeTaskId: typeof _sanitizeTaskId;
+/** Maximum depth to walk up the directory tree before stopping (fail-open). */
+export declare const MAX_DEPTH = 20;
+/** File/directory names that indicate a real project root. */
+export declare const PROJECT_INDICATORS: readonly ["package.json", ".git", ".opencode", "Cargo.toml", "go.mod", "pyproject.toml", "Gemfile", "composer.json", "pom.xml", "build.gradle", "CMakeLists.txt"];
 /**
  * Defense-in-depth: verify that `directory` is the project root and not a subdirectory
  * of a project that already has a .swarm/ at its root.
- * Walks up the directory tree to filesystem root looking for a parent .swarm/ directory.
- * @throws Error if a parent directory contains .swarm/
+ * Walks up the directory tree (bounded by MAX_DEPTH) looking for a parent .swarm/ directory.
+ * When .swarm/ is found, checks for at least one PROJECT_INDICATORS entry to distinguish
+ * real projects from stray artifacts (e.g. `C:\.swarm`).
+ * @throws Error if a parent directory contains both .swarm/ and a project indicator
  */
 export declare function validateProjectRoot(directory: string): void;
 /**

package/dist/index.js

@@ -48,7 +48,7 @@ var package_default;
 var init_package = __esm(() => {
   package_default = {
     name: "opencode-swarm",
-    version: "7.29.2",
+    version: "7.29.4",
     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",
@@ -16715,9 +16715,7 @@ function bunSpawn(cmd, options) {
       return proc.exitCode;
     },
     kill(signal) {
-      try {
-        killChild(signal);
-      } catch {}
+      killChild(signal);
     }
   };
 }
@@ -16969,6 +16967,13 @@ var init_telemetry = __esm(() => {
     gatePassed(sessionId, gate, taskId) {
       _internals4.emit("gate_passed", { sessionId, gate, taskId });
     },
+    gateParseError(taskId, error49) {
+      _internals4.emit("gate_parse_error", {
+        taskId,
+        errorName: error49.name,
+        errorMessage: error49.message.slice(0, 200)
+      });
+    },
     gateFailed(sessionId, gate, taskId, reason) {
       _internals4.emit("gate_failed", { sessionId, gate, taskId, reason });
     },
@@ -20687,15 +20692,36 @@ function validateProjectRoot(directory) {
     throw new Error(`Cannot verify project root for "${directory}" — directory may not exist or is inaccessible`);
   }
   let current = resolved;
+  let depth = 0;
   while (true) {
+    if (depth >= MAX_DEPTH)
+      break;
+    depth++;
     const parent = path8.dirname(current);
     if (parent === current)
       break;
     const parentSwarm = path8.join(parent, ".swarm");
     try {
       if (statSync5(parentSwarm).isDirectory()) {
-        warn(`[evidence] Rejecting write to subdirectory "${resolved}" — parent "${parent}" already contains .swarm/`);
-        throw new Error(`Cannot write evidence in "${resolved}" — parent directory "${parent}" already contains a .swarm/ folder. Evidence must be written to the project root.`);
+        let hasProjectIndicator = false;
+        for (const indicator of PROJECT_INDICATORS) {
+          try {
+            const indicatorStat = statSync5(path8.join(parent, indicator));
+            if (indicatorStat.isFile() || indicatorStat.isDirectory()) {
+              hasProjectIndicator = true;
+              break;
+            }
+          } catch (error49) {
+            if (error49 instanceof Error && "code" in error49 && error49.code === "ENOENT") {} else {
+              hasProjectIndicator = true;
+              break;
+            }
+          }
+        }
+        if (hasProjectIndicator) {
+          warn(`[evidence] Rejecting write to subdirectory "${resolved}" — parent "${parent}" already contains .swarm/`);
+          throw new Error(`Cannot write evidence in "${resolved}" — parent directory "${parent}" already contains a .swarm/ folder. Evidence must be written to the project root.`);
+        }
       }
     } catch (error49) {
       if (error49 instanceof Error && error49.message.startsWith("Cannot write evidence")) {
@@ -20938,7 +20964,7 @@ async function archiveEvidence(directory, maxAgeDays, maxBundles) {
   }
   return archived;
 }
-var VALID_EVIDENCE_TYPES, sanitizeTaskId2, LEGACY_TASK_COMPLEXITY_MAP, _internals8;
+var VALID_EVIDENCE_TYPES, sanitizeTaskId2, MAX_DEPTH = 20, PROJECT_INDICATORS, LEGACY_TASK_COMPLEXITY_MAP, _internals8;
 var init_manager2 = __esm(() => {
   init_zod();
   init_evidence_schema();
@@ -20963,6 +20989,19 @@ var init_manager2 = __esm(() => {
     "secretscan"
   ];
   sanitizeTaskId2 = sanitizeTaskId;
+  PROJECT_INDICATORS = [
+    "package.json",
+    ".git",
+    ".opencode",
+    "Cargo.toml",
+    "go.mod",
+    "pyproject.toml",
+    "Gemfile",
+    "composer.json",
+    "pom.xml",
+    "build.gradle",
+    "CMakeLists.txt"
+  ];
   LEGACY_TASK_COMPLEXITY_MAP = {
     low: "simple",
     medium: "moderate",
@@ -38741,12 +38780,15 @@ function getEvidencePath(directory, taskId) {
   assertValidTaskId(taskId);
   return path12.join(getEvidenceDir(directory), `${taskId}.json`);
 }
-function readExisting(evidencePath) {
+function readExisting(evidencePath, taskId) {
   try {
     const raw = readFileSync5(evidencePath, "utf-8");
     return TaskEvidenceSchema.parse(JSON.parse(raw));
-  } catch {
-    return null;
+  } catch (error49) {
+    if (error49.code === "ENOENT")
+      return null;
+    telemetry.gateParseError(taskId, error49);
+    throw error49;
   }
 }
 async function atomicWrite2(targetPath, content) {
@@ -38767,7 +38809,13 @@ async function recordGateEvidence(directory, taskId, gate, sessionId, turbo) {
   const lockRelPath = path12.join("evidence", `${taskId}.json`);
   await withEvidenceLock(directory, lockRelPath, gate, taskId, async () => {
     const evidencePath = getEvidencePath(directory, taskId);
-    const existing = readExisting(evidencePath);
+    let existing = null;
+    try {
+      existing = readExisting(evidencePath, taskId);
+    } catch (error49) {
+      telemetry.gateParseError(taskId, error49);
+      throw error49;
+    }
     const requiredGates = existing ? expandRequiredGates(existing.required_gates, gate) : deriveRequiredGates(gate);
     const updated = {
       taskId,
@@ -38793,7 +38841,13 @@ async function recordAgentDispatch(directory, taskId, agentType, turbo) {
   const lockRelPath = path12.join("evidence", `${taskId}.json`);
   await withEvidenceLock(directory, lockRelPath, agentType, taskId, async () => {
     const evidencePath = getEvidencePath(directory, taskId);
-    const existing = readExisting(evidencePath);
+    let existing = null;
+    try {
+      existing = readExisting(evidencePath, taskId);
+    } catch (error49) {
+      telemetry.gateParseError(taskId, error49);
+      throw error49;
+    }
     const requiredGates = existing ? expandRequiredGates(existing.required_gates, agentType) : deriveRequiredGates(agentType);
     const updated = {
       taskId,
@@ -38807,7 +38861,7 @@ async function recordAgentDispatch(directory, taskId, agentType, turbo) {
 async function readTaskEvidence(directory, taskId) {
   try {
     assertValidTaskId(taskId);
-    return readExisting(getEvidencePath(directory, taskId));
+    return readExisting(getEvidencePath(directory, taskId), taskId);
   } catch {
     return null;
   }
@@ -39003,46 +39057,7 @@ async function buildParallelExecutionGuidance(directory, sessionID, session) {
   if (eligible.length === 0) {
     return `[PARALLEL EXECUTION PROFILE] parallelization_enabled=true max_concurrent_tasks=${maxConcurrent}; no dependency-ready pending tasks are available for a new coder slot. Continue the current task/gate.`;
   }
-  return `[PARALLEL EXECUTION PROFILE] parallelization_enabled=true max_concurrent_tasks=${maxConcurrent}; ${occupied.size} slot(s) occupied. Eligible now: ${eligible.join(", ")}. [NEXT] dispatch up to ${availableSlots} eligible coder task(s) before waiting; for each dispatched task, call update_task_status(in_progress), call declare_scope, then send the coder Task. Preserve ONE atomic task per coder Task call.`;
-}
-async function buildPlanContinuationGuidance(directory) {
-  if (!directory)
-    return null;
-  const plan = await loadPlanJsonOnly(directory);
-  const currentTaskId = getPlanContinuationTaskId(plan);
-  if (!currentTaskId)
-    return null;
-  const sanitizedTaskId = sanitizeGuidanceValue(currentTaskId, 32);
-  return `[NEXT] Continue plan task ${sanitizedTaskId}: if it is not already in progress, call update_task_status with task_id="${sanitizedTaskId}" and status="in_progress"; ` + `then call declare_scope for the task files and dispatch coder Task call(s) according to the execution profile. ` + `Preserve ONE atomic task per coder Task call; when parallel execution is enabled, use available coder slots instead of forcing a single coder.`;
-}
-function sanitizeGuidanceValue(value, maxLength) {
-  return value.replace(/</g, "&lt;").replace(/>/g, "&gt;").replace(/\[ \]/g, "()").replace(/\[/g, "(").replace(/\]/g, ")").replace(/[\r\n]/g, " ").slice(0, maxLength);
-}
-function getPlanContinuationTaskId(plan) {
-  if (!plan)
-    return;
-  const currentPhase = plan.current_phase ?? 1;
-  const phase = plan.phases.find((p) => p.id === currentPhase);
-  if (!phase)
-    return;
-  const sortedTasks = [...phase.tasks].sort((a, b) => comparePlanTaskIds(a.id, b.id));
-  const inProgress = sortedTasks.find((task) => task.status === "in_progress");
-  if (inProgress)
-    return inProgress.id;
-  const incomplete = sortedTasks.find((task) => task.status !== "completed" && task.status !== "closed");
-  return incomplete?.id;
-}
-function comparePlanTaskIds(a, b) {
-  const partsA = a.split(".").map((part) => Number.parseInt(part, 10));
-  const partsB = b.split(".").map((part) => Number.parseInt(part, 10));
-  const maxLength = Math.max(partsA.length, partsB.length);
-  for (let i2 = 0;i2 < maxLength; i2++) {
-    const numA = partsA[i2] ?? 0;
-    const numB = partsB[i2] ?? 0;
-    if (numA !== numB)
-      return numA - numB;
-  }
-  return 0;
+  return `[PARALLEL EXECUTION PROFILE] parallelization_enabled=true max_concurrent_tasks=${maxConcurrent}; ${occupied.size} slot(s) occupied. Eligible now: ${eligible.join(", ")}. [NEXT] dispatch up to ${availableSlots} eligible coder task(s) before waiting; preserve ONE task per coder call and call declare_scope for each task.`;
 }
 function isParallelGuidancePhaseComplete(phase) {
   return phase.status === "complete" || phase.status === "completed" || phase.status === "closed";
@@ -39620,7 +39635,6 @@ ${trimComment}${after}`;
             const deliberationSession = ensureAgentSession(deliberationSessionID);
             const lastGate = deliberationSession.lastGateOutcome;
             const parallelGuidance = await buildParallelExecutionGuidance(directory, deliberationSessionID, deliberationSession);
-            const planContinuationGuidance = parallelGuidance === null ? await buildPlanContinuationGuidance(directory) : null;
             const taskAwaitingCompletion = await findTaskAwaitingCompletion(directory, deliberationSession);
             let guidance;
             if (taskAwaitingCompletion) {
@@ -39628,12 +39642,12 @@ ${trimComment}${after}`;
 [NEXT] Print the task completion checklist, then call update_task_status with task_id="${taskAwaitingCompletion}" and status="completed" before declare_scope or starting another task.`;
             } else if (lastGate?.taskId) {
               const gateResult = lastGate.passed ? "PASSED" : "FAILED";
-              const sanitizedGate = sanitizeGuidanceValue(lastGate.gate, 64);
-              const sanitizedTaskId = sanitizeGuidanceValue(lastGate.taskId, 32);
+              const sanitizedGate = lastGate.gate.replace(/</g, "&lt;").replace(/>/g, "&gt;").replace(/\[ \]/g, "()").replace(/\[/g, "(").replace(/\]/g, ")").replace(/[\r\n]/g, " ").slice(0, 64);
+              const sanitizedTaskId = lastGate.taskId.replace(/</g, "&lt;").replace(/>/g, "&gt;").replace(/\[/g, "(").replace(/\]/g, ")").replace(/[\r\n]/g, " ").slice(0, 32);
               guidance = `[Last gate: ${sanitizedGate} ${gateResult} for task ${sanitizedTaskId}]
 ${parallelGuidance ?? "[NEXT] Execute the next gate for the current task."}`;
             } else {
-              guidance = parallelGuidance ?? planContinuationGuidance ?? "[NEXT] Begin the first plan task and run gates sequentially.";
+              guidance = parallelGuidance ?? "[NEXT] Begin the first plan task and run gates sequentially.";
             }
             const systemMsgIdx = messages.findIndex((m) => m && m.info?.role === "system");
             const insertIdx = systemMsgIdx >= 0 ? systemMsgIdx + 1 : 0;
@@ -61639,10 +61653,10 @@ function detectStraySwarmDirs(projectRoot) {
     "__pycache__",
     ".tox"
   ]);
-  const MAX_DEPTH = 10;
+  const MAX_DEPTH2 = 10;
   const MAX_CONTENTS_ENTRIES = 20;
   function walk(dir, depth) {
-    if (depth > MAX_DEPTH)
+    if (depth > MAX_DEPTH2)
       return;
     let entries;
     try {
@@ -67415,7 +67429,7 @@ function defaultBuildTestCommand(profile, framework, files, dir = ".", opts = {}
   const coverage = opts.coverage ?? false;
   switch (framework) {
     case "bun": {
-      const args2 = ["bun", "--smol", "test"];
+      const args2 = ["bun", "test"];
       if (coverage)
         args2.push("--coverage");
       if (scope !== "all" && files.length > 0)
@@ -69848,7 +69862,7 @@ function getTargetedExecutionUnsupportedReason(framework) {
 function buildTestCommand2(framework, scope, files, coverage, baseDir) {
   switch (framework) {
     case "bun": {
-      const args2 = ["bun", "--smol", "test"];
+      const args2 = ["bun", "test"];
       if (coverage)
         args2.push("--coverage");
       if (scope !== "all" && files.length > 0) {
@@ -70385,24 +70399,17 @@ async function runTests(framework, scope, files, coverage, timeout_ms, cwd) {
     const proc = bunSpawn(command, {
       stdout: "pipe",
       stderr: "pipe",
-      stdin: "ignore",
-      cwd,
-      killProcessTree: true
-    });
-    let timeoutHandle;
-    const timeoutPromise = new Promise((resolve16) => {
-      timeoutHandle = setTimeout(() => {
-        proc.kill();
-        resolve16(-1);
-      }, timeout_ms);
+      cwd
     });
+    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) {
@@ -70705,6 +70712,7 @@ 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) {
@@ -70778,8 +70786,7 @@ var init_test_runner = __esm(() => {
       }
       const scope = args2.scope || "all";
       if (scope === "all") {
-        const fullSuiteAllowed = process.env.SWARM_ALLOW_FULL_SUITE === "1" || process.env.SWARM_ALLOW_FULL_SUITE === "true";
-        if (!fullSuiteAllowed) {
+        if (!process.env.SWARM_ALLOW_FULL_SUITE) {
           const errorResult = {
             success: false,
             framework: "none",
@@ -75971,8 +75978,6 @@ SECURITY_KEYWORDS: password, secret, token, credential, auth, login, encryption,
 {{AGENT_PREFIX}}test_engineer - Test generation AND execution (writes tests, runs them, reports PASS/FAIL)
 {{AGENT_PREFIX}}critic - Plan review gate (reviews plan BEFORE implementation)
 {{AGENT_PREFIX}}critic_sounding_board - Pre-escalation pushback (honest engineer review before user contact)
-{{AGENT_PREFIX}}skill_improver - Low-frequency skill / knowledge / prompt improvement adviser
-{{AGENT_PREFIX}}spec_writer - .swarm/spec.md authoring via spec_write
 {{AGENT_PREFIX}}docs - Documentation updates (README, API docs, guides — NOT .swarm/ files)
 {{AGENT_PREFIX}}designer - UI/UX design specs (scaffold generation for UI components — runs BEFORE coder on UI tasks)
 
@@ -76023,20 +76028,30 @@ For every applicable directive in the block:
 
 You may also call the \`knowledge_ack\` tool to record an outcome explicitly when chat-text markers would be ambiguous (e.g. inside structured tool args).
 
-## SKILL IMPROVER
+## SKILL IMPROVER (low-frequency, expensive-model adviser)
 
-\`{{AGENT_PREFIX}}skill_improver\` / \`skill_improve\`: rare, quota-bounded,
-disabled by default, proposal-only. Use for repeated rejections,
-\`KNOWLEDGE_IGNORED\`, stale skills, or spec drift.
+The \`skill_improver\` agent and the \`skill_improve\` tool exist for rare, deep
+review of accumulated knowledge / skills / spec / architect prompt. They are
+quota-bounded (default 10 calls/day) and disabled by default. Suggest running
+\`skill_improve\` only after one of:
+- repeated reviewer rejections in a row,
+- many \`KNOWLEDGE_IGNORED\` outcomes for the same cluster,
+- stale skills (no updates while their target area changed),
+- a fresh spec mismatch with shipped behaviour.
 
 When \`skill_improver.require_user_approval\` is true (default), ASK the user
 before running. Default outputs are proposals only — they never modify source.
 
 ## SPEC WRITER
 
-For substantial spec authoring/revision, prefer \`{{AGENT_PREFIX}}spec_writer\`;
-it writes via \`spec_write\`. Use for major specs or non-trivial
-decomposition. Handle small touch-ups.
+For substantial spec authoring or revision, prefer delegating to the
+\`spec_writer\` agent (independent model from architect). It writes only via
+the safe \`spec_write\` tool. Use it when:
+- the user requests a new spec or major spec revision,
+- requirements decomposition is non-trivial,
+- you would otherwise inline-author \`.swarm/spec.md\` yourself.
+
+Continue handling small touch-ups (typos, cross-references) inline.
 
 ### ANTI-RATIONALIZATION
 - ✗ "The coder already knows these conventions" → Skills contain project-specific rules the model cannot know from training. Always pass.
@@ -76217,12 +76232,11 @@ MODE: BRAINSTORM runs seven phases in strict order. Do not skip phases. Do not c
 - Exit with a design outline the user can skim in under two minutes.
 
 **Phase 5: SPEC WRITE + SELF-REVIEW (architect + reviewer).**
-- Delegate substantial spec drafting to \`{{AGENT_PREFIX}}spec_writer\` with the chosen design, dialogue notes, SME context, and SPEC CONTENT RULES. The spec writer must persist \`.swarm/spec.md\` through \`spec_write\`.
-- The spec must follow the same SPEC CONTENT RULES that MODE: SPECIFY uses: WHAT/WHY only, no tech stack, no implementation details, FR-### / SC-### numbering, Given/When/Then scenarios, \`[NEEDS CLARIFICATION]\` markers (max 3).
+- Generate \`.swarm/spec.md\` following the same SPEC CONTENT RULES that MODE: SPECIFY uses: WHAT/WHY only, no tech stack, no implementation details, FR-### / SC-### numbering, Given/When/Then scenarios, \`[NEEDS CLARIFICATION]\` markers (max 3).
 - Cross-reference design sections by name where relevant context helps (but keep HOW out of the spec).
 - Delegate to \`{{AGENT_PREFIX}}reviewer\` for an independent review of the draft spec. Reviewer must flag: requirements that encode HOW, untestable requirements, missing edge cases, silent assumptions.
 - Apply reviewer feedback. If reviewer rejects, iterate once and re-review. After two rounds, surface remaining disagreements to the user.
-- Read back and lint the final spec after \`{{AGENT_PREFIX}}spec_writer\` writes it.
+- Write the final spec to \`.swarm/spec.md\`.
 - Exit when reviewer signs off (or user explicitly accepts remaining disagreements).
 
 **Phase 6: QA GATE SELECTION (architect, dialogue only).**
@@ -76294,7 +76308,7 @@ Activates when: user asks to "specify", "define requirements", "write a spec", o
 1b. Run CODEBASE REALITY CHECK for any codebase references mentioned by the user or implied by the feature. Skip if work is purely greenfield (no existing codebase to check). Report discrepancies before proceeding to explorer.
 2. Delegate to \`{{AGENT_PREFIX}}explorer\` to scan the codebase for relevant context (existing patterns, related code, affected areas).
 3. Delegate to \`{{AGENT_PREFIX}}sme\` for domain research on the feature area to surface known constraints, best practices, and integration concerns.
-4. Delegate substantial spec drafting to \`{{AGENT_PREFIX}}spec_writer\`. Include the user requirements, explorer findings, SME constraints, and these required contents:
+4. Generate \`.swarm/spec.md\` capturing:
    - First line must be: \`# Specification: <feature-name>\`
    - Feature description: WHAT users need and WHY — never HOW to implement
    - User scenarios with acceptance criteria (Given/When/Then format)
@@ -76303,7 +76317,7 @@ Activates when: user asks to "specify", "define requirements", "write a spec", o
    - Key entities if data is involved (no schema or field definitions — entity names only)
    - Edge cases and known failure modes
    - \`[NEEDS CLARIFICATION]\` markers (max 3) for items where uncertainty could change scope, security, or core behavior; prefer informed defaults over asking
-5. Require \`{{AGENT_PREFIX}}spec_writer\` to write the spec via \`spec_write\`, then read back and lint \`.swarm/spec.md\`.
+5. Write the spec to \`.swarm/spec.md\`.
 5b. **QA GATE SELECTION (dialogue only).**
 {{QA_GATE_DIALOGUE_SPECIFY}}
 
@@ -76458,7 +76472,6 @@ If .swarm/plan.md exists:
      - Update context.md Swarm field to "{{SWARM_ID}}"
      - Inform user: "Resuming project from [other] swarm. Cleared stale context. Ready to continue."
      - Resume at current task
-Resume execution rule: after identifying the current task, do not restart broad discovery. Move into EXECUTE: call \`update_task_status\` if the task is not already in progress, call \`declare_scope\` for the task files, then dispatch coder Task call(s) according to \`execution_profile\`. Preserve ONE atomic task per coder Task call; parallel profiles may dispatch up to the available coder slots.
 If .swarm/plan.md does not exist → New project, proceed to MODE: CLARIFY
 If new project: Run \`complexity_hotspots\` tool (90 days) to generate a risk map. Note modules with recommendation "security_review" or "full_gates" in context.md for stricter QA gates during Phase 5. Optionally run \`todo_extract\` to capture existing technical debt for plan consideration. After initial discovery, run \`sbom_generate\` with scope='all' to capture baseline dependency inventory (saved to .swarm/evidence/sbom/).
 
@@ -79507,24 +79520,8 @@ function createSwarmAgents(swarmId, swarmConfig, isDefault, pluginConfig, projec
     const swarmIdentity = isDefault ? "default" : swarmId;
     const agentPrefix = prefix;
     architect.config.prompt = architect.config.prompt?.replace(/\{\{SWARM_ID\}\}/g, swarmIdentity).replace(/\{\{AGENT_PREFIX\}\}/g, agentPrefix).replace(/\{\{QA_RETRY_LIMIT\}\}/g, String(qaRetryLimit)).replace(/\{\{PROJECT_LANGUAGE\}\}/g, projectContext.PROJECT_LANGUAGE).replace(/\{\{PROJECT_FRAMEWORK\}\}/g, projectContext.PROJECT_FRAMEWORK).replace(/\{\{BUILD_CMD\}\}/g, projectContext.BUILD_CMD).replace(/\{\{TEST_CMD\}\}/g, projectContext.TEST_CMD).replace(/\{\{LINT_CMD\}\}/g, projectContext.LINT_CMD).replace(/\{\{ENTRY_POINTS\}\}/g, projectContext.ENTRY_POINTS).replace(/\{\{CODER_CONSTRAINTS\}\}/g, projectContext.CODER_CONSTRAINTS).replace(/\{\{TEST_CONSTRAINTS\}\}/g, projectContext.TEST_CONSTRAINTS).replace(/\{\{REVIEWER_CHECKLIST\}\}/g, projectContext.REVIEWER_CHECKLIST).replace(/\{\{PROJECT_CONTEXT_SECONDARY_LANGUAGES\}\}/g, projectContext.PROJECT_CONTEXT_SECONDARY_LANGUAGES);
-    const skillImproverEnabled = !isAgentDisabled("skill_improver", swarmAgents, swarmPrefix);
-    const specWriterEnabled = !isAgentDisabled("spec_writer", swarmAgents, swarmPrefix);
-    if (!skillImproverEnabled) {
-      architect.config.prompt = architect.config.prompt?.replace(`, ${agentPrefix}skill_improver`, "").replace(`
-${agentPrefix}skill_improver - Low-frequency skill / knowledge / prompt improvement adviser`, "").replace(/\n## SKILL IMPROVER[\s\S]*?(?=\n\n## SPEC WRITER)/, "");
-    }
-    if (!specWriterEnabled) {
-      const escapedAgentPrefix = agentPrefix.replace(/[.*+?^${}()|[\]\\]/g, "\\$&");
-      architect.config.prompt = architect.config.prompt?.replace(`, ${agentPrefix}spec_writer`, "").replace(`
-${agentPrefix}spec_writer - .swarm/spec.md authoring via spec_write`, "").replace(/\n## SPEC WRITER[\s\S]*?(?=\n\n### ANTI-RATIONALIZATION)/, "").replace(new RegExp(`- Delegate substantial spec drafting to \`${escapedAgentPrefix}spec_writer\`[^\\n]*\\n`, "g"), "- spec_writer is disabled. Ask the user to enable the spec_writer agent before creating or revising `.swarm/spec.md`.\n").replace(new RegExp(`4\\. Delegate substantial spec drafting to \`${escapedAgentPrefix}spec_writer\`[^\\n]*`), "4. spec_writer is disabled. Ask the user to enable the spec_writer agent before creating or revising `.swarm/spec.md`.").replace(new RegExp(`5\\. Require \`${escapedAgentPrefix}spec_writer\` to write the spec via \`spec_write\`, then read back and lint \`\\.swarm/spec\\.md\`\\.`), "5. Do not continue SPECIFY until spec_writer is available.").replace(new RegExp(`- Read back and lint the final spec after \`${escapedAgentPrefix}spec_writer\` writes it\\.`), "- Do not continue BRAINSTORM spec writing until spec_writer is available.");
-    }
     if (!isDefault) {
       architect.description = `[${swarmName}] ${architect.description}`;
-      const optionalAgentLines = [
-        skillImproverEnabled ? `- @${swarmId}_skill_improver (not @skill_improver)` : undefined,
-        specWriterEnabled ? `- @${swarmId}_spec_writer (not @spec_writer)` : undefined
-      ].filter((line) => Boolean(line)).join(`
-`);
       const swarmHeader = `## ⚠️ YOU ARE THE ${swarmName.toUpperCase()} SWARM ARCHITECT
 
 Your swarm ID is "${swarmId}". ALL your agents have the "${swarmId}_" prefix:
@@ -79532,8 +79529,8 @@ Your swarm ID is "${swarmId}". ALL your agents have the "${swarmId}_" prefix:
 - @${swarmId}_coder (not @coder)
 - @${swarmId}_sme (not @sme)
 - @${swarmId}_reviewer (not @reviewer)
-${optionalAgentLines ? `${optionalAgentLines}
-` : ""}- etc.
+- @${swarmId}_spec_writer (not @spec_writer)
+- etc.
 
 CRITICAL: Agents without the "${swarmId}_" prefix DO NOT EXIST or belong to a DIFFERENT swarm.
 If you call @coder instead of @${swarmId}_coder, the call will FAIL or go to the wrong swarm.

package/dist/telemetry.d.ts

@@ -1,4 +1,4 @@
-export type TelemetryEvent = 'session_started' | 'session_ended' | 'agent_activated' | 'delegation_begin' | 'delegation_end' | 'task_state_changed' | 'gate_passed' | 'gate_failed' | 'phase_changed' | 'budget_updated' | 'model_fallback' | 'hard_limit_hit' | 'revision_limit_hit' | 'loop_detected' | 'scope_violation' | 'qa_skip_violation' | 'heartbeat' | 'turbo_mode_changed' | 'auto_oversight_escalation' | 'environment_detected' | 'evidence_lock_acquired' | 'evidence_lock_contended' | 'evidence_lock_stale_recovered' | 'plan_ledger_cas_retry' | 'plan_md_write_failed' | 'prm_pattern_detected' | 'prm_course_correction_injected' | 'prm_escalation_triggered' | 'prm_hard_stop';
+export type TelemetryEvent = 'session_started' | 'session_ended' | 'agent_activated' | 'delegation_begin' | 'delegation_end' | 'task_state_changed' | 'gate_passed' | 'gate_failed' | 'gate_parse_error' | 'phase_changed' | 'budget_updated' | 'model_fallback' | 'hard_limit_hit' | 'revision_limit_hit' | 'loop_detected' | 'scope_violation' | 'qa_skip_violation' | 'heartbeat' | 'turbo_mode_changed' | 'auto_oversight_escalation' | 'environment_detected' | 'evidence_lock_acquired' | 'evidence_lock_contended' | 'evidence_lock_stale_recovered' | 'plan_ledger_cas_retry' | 'plan_md_write_failed' | 'prm_pattern_detected' | 'prm_course_correction_injected' | 'prm_escalation_triggered' | 'prm_hard_stop';
 export type TelemetryListener = (event: TelemetryEvent, data: Record<string, unknown>) => void;
 /** @internal - For testing only */
 export declare function resetTelemetryForTesting(): void;
@@ -39,6 +39,7 @@ export declare const telemetry: {
     delegationEnd(sessionId: string, agentName: string, taskId: string, result: string): void;
     taskStateChanged(sessionId: string, taskId: string, newState: string, oldState?: string): void;
     gatePassed(sessionId: string, gate: string, taskId: string): void;
+    gateParseError(taskId: string, error: Error): void;
     gateFailed(sessionId: string, gate: string, taskId: string, reason: string): void;
     phaseChanged(sessionId: string, oldPhase: number, newPhase: number): void;
     budgetUpdated(sessionId: string, budgetPct: number, agentName: string): void;

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

@@ -23,6 +23,7 @@ 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/package.json

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