opencode-swarm 7.17.0 → 7.17.1

package/dist/cli/index.js

@@ -34,7 +34,7 @@ var package_default;
 var init_package = __esm(() => {
   package_default = {
     name: "opencode-swarm",
-    version: "7.17.0",
+    version: "7.17.1",
     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",
@@ -15935,7 +15935,6 @@ async function handleAcknowledgeSpecDriftCommand(directory, _args) {
     return "Spec staleness file was corrupted. It has been removed.";
   }
   const { planTitle, phase } = stalenessData;
-  await fsPromises3.unlink(specStalenessPath);
   let currentHash = null;
   let planUpdateSkipped = false;
   try {
@@ -15949,6 +15948,9 @@ async function handleAcknowledgeSpecDriftCommand(directory, _args) {
     console.error("[acknowledge-spec-drift] Failed to update plan specHash:", planError instanceof Error ? planError.message : String(planError));
     planUpdateSkipped = true;
   }
+  if (!planUpdateSkipped) {
+    await fsPromises3.unlink(specStalenessPath);
+  }
   const eventsPath = validateSwarmPath(directory, "events.jsonl");
   const acknowledgmentEvent = {
     type: "spec_drift_acknowledged",
@@ -35412,6 +35414,8 @@ async function quarantineEntry(directory, entryId, reason, reportedBy) {
     const remaining = entries.filter((e) => e.id !== entryId);
     const quarantined = {
       ...entry,
+      status: "quarantined",
+      original_status: entry.status,
       quarantine_reason: sanitizedReason,
       quarantined_at: new Date().toISOString(),
       reported_by: reportedBy
@@ -35470,7 +35474,24 @@ async function restoreEntry(directory, entryId) {
       return;
     }
     const remaining = quarantinedEntries.filter((e) => e.id !== entryId);
-    const { quarantine_reason, quarantined_at, reported_by, ...original } = entryToRestore;
+    const {
+      quarantine_reason,
+      quarantined_at,
+      reported_by,
+      original_status,
+      status: _quarantineStatus,
+      ...rest
+    } = entryToRestore;
+    const original = { ...rest, status: original_status ?? "candidate" };
+    const validation = validateLesson(original.lesson, [], {
+      category: original.category,
+      scope: original.scope,
+      confidence: original.confidence
+    });
+    if (!validation.valid) {
+      warn(`[knowledge-validator] restoreEntry: entry ${entryId} failed re-validation: ${validation.reason}`);
+      return;
+    }
     const jsonlContent = remaining.length > 0 ? `${remaining.map((e) => JSON.stringify(e)).join(`
 `)}
 ` : "";
@@ -46325,7 +46346,7 @@ async function selectEntryPoints2(dir) {
 `)) {
         const m = line.match(/=\s*['"]([^'":]+)/);
         if (m) {
-          const modPath = m[1].replace(/\./g, "/") + ".py";
+          const modPath = `${m[1].replace(/\./g, "/")}.py`;
           points.add(modPath);
         }
       }
@@ -47508,7 +47529,7 @@ function clearDispatchCache() {
   manifestRootCache.clear();
   insertCounter = 0;
 }
-var _internals22, cache, insertCounter = 0, MANIFEST_FILES, MANIFEST_SET, manifestRootCache;
+var _internals22, cache, insertCounter = 0, MANIFEST_FILES, _MANIFEST_SET, manifestRootCache;
 var init_dispatch = __esm(() => {
   init_backends();
   init_detector();
@@ -47540,7 +47561,7 @@ var init_dispatch = __esm(() => {
     "Gemfile",
     "composer.json"
   ];
-  MANIFEST_SET = new Set(MANIFEST_FILES);
+  _MANIFEST_SET = new Set(MANIFEST_FILES);
   manifestRootCache = new Map;
 });
 

package/dist/hooks/knowledge-injector.d.ts

@@ -9,7 +9,8 @@ import type { KnowledgeConfig, MessageWithParts } from './knowledge-types.js';
 /**
  * Creates a knowledge injection hook that injects relevant knowledge into the
  * architect's message context at phase start. Supports caching for re-injection
- * after compaction.
+ * after compaction. Cache is per-instance (bound to the returned hook closure),
+ * ensuring no cross-test pollution in Bun's shared test-runner process.
  *
  * @param directory - The project directory containing .swarm/
  * @param config - Knowledge system configuration

package/dist/hooks/knowledge-types.d.ts

@@ -81,7 +81,7 @@ export interface KnowledgeEntryBase extends ActionableDirectiveFields {
     tags: string[];
     scope: string;
     confidence: number;
-    status: 'candidate' | 'established' | 'promoted' | 'archived';
+    status: 'candidate' | 'established' | 'promoted' | 'archived' | 'quarantined';
     confirmed_by: PhaseConfirmationRecord[] | ProjectConfirmationRecord[];
     retrieval_outcomes: RetrievalOutcome;
     schema_version: number;

package/dist/hooks/knowledge-validator.d.ts

@@ -31,6 +31,7 @@ export declare function validateSkillPath(p: unknown): boolean;
 export declare function validateActionableFields(fields: ActionableDirectiveFields | undefined): ActionableValidationResult;
 export type { ActionableDirectiveFields, DirectivePriority };
 export interface QuarantinedEntry extends KnowledgeEntryBase {
+    original_status: string;
     quarantine_reason: string;
     quarantined_at: string;
     reported_by: 'architect' | 'user' | 'auto';

package/dist/index.js

@@ -33,7 +33,7 @@ var package_default;
 var init_package = __esm(() => {
   package_default = {
     name: "opencode-swarm",
-    version: "7.17.0",
+    version: "7.17.1",
     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",
@@ -18445,7 +18445,6 @@ async function handleAcknowledgeSpecDriftCommand(directory, _args) {
     return "Spec staleness file was corrupted. It has been removed.";
   }
   const { planTitle, phase } = stalenessData;
-  await fsPromises4.unlink(specStalenessPath);
   let currentHash = null;
   let planUpdateSkipped = false;
   try {
@@ -18459,6 +18458,9 @@ async function handleAcknowledgeSpecDriftCommand(directory, _args) {
     console.error("[acknowledge-spec-drift] Failed to update plan specHash:", planError instanceof Error ? planError.message : String(planError));
     planUpdateSkipped = true;
   }
+  if (!planUpdateSkipped) {
+    await fsPromises4.unlink(specStalenessPath);
+  }
   const eventsPath = validateSwarmPath(directory, "events.jsonl");
   const acknowledgmentEvent = {
     type: "spec_drift_acknowledged",
@@ -42470,6 +42472,8 @@ async function quarantineEntry(directory, entryId, reason, reportedBy) {
     const remaining = entries.filter((e) => e.id !== entryId);
     const quarantined = {
       ...entry,
+      status: "quarantined",
+      original_status: entry.status,
       quarantine_reason: sanitizedReason,
       quarantined_at: new Date().toISOString(),
       reported_by: reportedBy
@@ -42528,7 +42532,24 @@ async function restoreEntry(directory, entryId) {
       return;
     }
     const remaining = quarantinedEntries.filter((e) => e.id !== entryId);
-    const { quarantine_reason, quarantined_at, reported_by, ...original } = entryToRestore;
+    const {
+      quarantine_reason,
+      quarantined_at,
+      reported_by,
+      original_status,
+      status: _quarantineStatus,
+      ...rest
+    } = entryToRestore;
+    const original = { ...rest, status: original_status ?? "candidate" };
+    const validation = validateLesson(original.lesson, [], {
+      category: original.category,
+      scope: original.scope,
+      confidence: original.confidence
+    });
+    if (!validation.valid) {
+      warn(`[knowledge-validator] restoreEntry: entry ${entryId} failed re-validation: ${validation.reason}`);
+      return;
+    }
     const jsonlContent = remaining.length > 0 ? `${remaining.map((e) => JSON.stringify(e)).join(`
 `)}
 ` : "";
@@ -44359,7 +44380,9 @@ async function readMergedKnowledge(directory, config3, context) {
   const maxInject = config3.max_inject_count ?? 5;
   const topN = ranked.slice(0, maxInject);
   if (topN.length > 0 && context?.currentPhase) {
-    recordLessonsShown(directory, topN.map((e) => e.id), context.currentPhase).catch(() => {});
+    recordLessonsShown(directory, topN.map((e) => e.id), context.currentPhase).catch((err2) => {
+      warn("[knowledge-reader] recordLessonsShown unexpected rejection:", err2);
+    });
   }
   return topN;
 }
@@ -54848,7 +54871,7 @@ async function selectEntryPoints2(dir) {
 `)) {
         const m = line.match(/=\s*['"]([^'":]+)/);
         if (m) {
-          const modPath = m[1].replace(/\./g, "/") + ".py";
+          const modPath = `${m[1].replace(/\./g, "/")}.py`;
           points.add(modPath);
         }
       }
@@ -56031,7 +56054,7 @@ function clearDispatchCache() {
   manifestRootCache.clear();
   insertCounter = 0;
 }
-var _internals28, cache, insertCounter = 0, MANIFEST_FILES, MANIFEST_SET, manifestRootCache;
+var _internals28, cache, insertCounter = 0, MANIFEST_FILES, _MANIFEST_SET, manifestRootCache;
 var init_dispatch = __esm(() => {
   init_backends();
   init_detector();
@@ -56063,7 +56086,7 @@ var init_dispatch = __esm(() => {
     "Gemfile",
     "composer.json"
   ];
-  MANIFEST_SET = new Set(MANIFEST_FILES);
+  _MANIFEST_SET = new Set(MANIFEST_FILES);
   manifestRootCache = new Map;
 });
 
@@ -71420,7 +71443,7 @@ function decisionFromVerdict(verdict, escalationNeeded) {
   if (verdict === "APPROVED" || verdict === "ANSWER")
     return "allow";
   if (verdict === "BLOCKED")
-    return "pause";
+    return "deny";
   if (verdict === "PENDING")
     return "pending";
   return "deny";
@@ -80646,7 +80669,7 @@ function createFullAutoPermissionHook(options) {
           phase: effectivePhase,
           taskID: taskId ?? undefined,
           planID: runState.planID,
-          architectOutput: undefined,
+          architectOutput: output.args ? JSON.stringify(output.args) : undefined,
           actionContext: {
             tool: toolName,
             ...decision.context ?? {}
@@ -81352,6 +81375,7 @@ init_extractors();
 init_knowledge_reader();
 init_knowledge_store();
 init_utils2();
+var INJECTION_SENTINEL = "‌[[KNOWLEDGE-INJECTED]]";
 function buildKnowledgeBlock(entries, charBudget, cfg, currentProject) {
   if (entries.length === 0)
     return null;
@@ -81444,7 +81468,7 @@ function isOrchestratorAgent(agentName) {
 function injectKnowledgeMessage(output, text) {
   if (!output.messages)
     return;
-  const alreadyInjected = output.messages.some((m) => m.parts?.some((p) => p.text?.includes("\uD83D\uDCDA Lessons:") || p.text?.includes("<drift_report>") || p.text?.includes("<curator_briefing>")));
+  const alreadyInjected = output.messages.some((m) => m.parts?.some((p) => p.text?.includes(INJECTION_SENTINEL)));
   if (alreadyInjected)
     return;
   let insertIdx = output.messages.length - 1;
@@ -81456,14 +81480,11 @@ function injectKnowledgeMessage(output, text) {
   }
   const knowledgeMessage = {
     info: { role: "system" },
-    parts: [{ type: "text", text }]
+    parts: [{ type: "text", text: `${INJECTION_SENTINEL}${text}` }]
   };
   output.messages.splice(insertIdx, 0, knowledgeMessage);
 }
 function createKnowledgeInjectorHook(directory, config3) {
-  let lastSeenCacheKey = null;
-  let cachedInjectionText = null;
-  let cachedShownIds = [];
   function buildContextCacheKey(phase, ctx) {
     const parts2 = [
       String(phase),
@@ -81475,6 +81496,9 @@ function createKnowledgeInjectorHook(directory, config3) {
     ].join("|");
     return createHash7("sha1").update(parts2).digest("hex").slice(0, 16);
   }
+  let lastSeenCacheKey = null;
+  let cachedInjectionText = null;
+  let cachedShownIds = [];
   return safeHook(async (_input, output) => {
     if (!output.messages || output.messages.length === 0)
       return;
@@ -87862,7 +87886,9 @@ var imports = createSwarmTool({
 });
 // src/tools/knowledge-ack.ts
 init_zod();
+import { randomUUID as randomUUID7 } from "node:crypto";
 init_state();
+init_logger();
 init_create_tool();
 var knowledge_ack = createSwarmTool({
   description: "Record an acknowledgment outcome (applied/ignored/violated) for a previously-injected knowledge directive. Updates retrieval-outcome counters and appends a record to .swarm/knowledge-application.jsonl.",
@@ -87890,7 +87916,11 @@ var knowledge_ack = createSwarmTool({
       result: a.result,
       reason: a.reason
     };
-    const sessionId = ctx?.sessionID ?? "unknown";
+    let sessionId = ctx?.sessionID;
+    if (!sessionId) {
+      warn("[knowledge-ack] No sessionID in tool context — dedup disabled for this acknowledgment");
+      sessionId = randomUUID7();
+    }
     const dedupKey = buildAckDedupKey(sessionId, a.id, a.result);
     if (swarmState.knowledgeAckDedup.has(dedupKey)) {
       return JSON.stringify({
@@ -87919,7 +87949,7 @@ init_knowledge_store();
 init_knowledge_validator();
 init_manager();
 init_create_tool();
-import { randomUUID as randomUUID7 } from "node:crypto";
+import { randomUUID as randomUUID8 } from "node:crypto";
 var VALID_CATEGORIES2 = [
   "process",
   "architecture",
@@ -87993,7 +88023,7 @@ var knowledge_add = createSwarmTool({
       project_name = plan?.title ?? "";
     } catch {}
     const entry = {
-      id: randomUUID7(),
+      id: randomUUID8(),
       tier: "swarm",
       lesson,
       category,
@@ -99561,7 +99591,7 @@ ${fileList}
   async _withStateLock(fn) {
     const timeoutMs = 1e4;
     let timeoutId;
-    const withTimeout2 = new Promise((resolve47, reject) => {
+    const withTimeout2 = new Promise((_resolve, reject) => {
       timeoutId = setTimeout(() => {
         reject(new Error(`_withStateLock timed out after ${timeoutMs}ms — state update will not block subsequent operations`));
       }, timeoutMs);

package/package.json

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