opencode-swarm-plugin 0.22.0 → 0.23.0

package/src/learning.integration.test.ts

@@ -4,10 +4,46 @@
  * Tests for confidence decay, feedback scoring, outcome tracking,
  * anti-patterns, pattern maturity, and swarm tool integrations.
  *
- * These tests don't require external services - they test the learning
- * algorithms and their integration with swarm tools.
+ * ## Test Isolation Pattern
+ *
+ * This file uses TEST_SEMANTIC_MEMORY_COLLECTION to isolate test data from
+ * production semantic-memory collections. Each test run gets a unique suffix,
+ * preventing pollution of the default collections.
+ *
+ * **Cleanup**: Test collections are NOT automatically deleted after test runs.
+ * This is intentional - semantic-memory doesn't provide bulk delete APIs.
+ * To clean up test artifacts, use:
+ *
+ * ```bash
+ * # Manual cleanup (use scripts/cleanup-test-memories.ts for automation)
+ * semantic-memory list --collection swarm-feedback-test-* --json | jq -r '.[].id' | xargs -I {} semantic-memory remove {}
+ * ```
+ *
+ * The unique suffix prevents cross-test interference even without cleanup.
+ */
+import {
+  describe,
+  it,
+  expect,
+  beforeEach,
+  afterEach,
+  beforeAll,
+  afterAll,
+  vi,
+} from "vitest";
+import { getTestCollectionName } from "./storage";
+
+// ============================================================================
+// Test Isolation Setup
+// ============================================================================
+
+/**
+ * Set unique collection suffix for this test run
+ *
+ * CRITICAL: This MUST be set before any storage instances are created.
+ * The env var is read during getCollectionNames() which happens at storage init.
  */
-import { describe, it, expect, beforeEach, afterEach, vi } from "vitest";
+process.env.TEST_SEMANTIC_MEMORY_COLLECTION = getTestCollectionName();
 
 // Learning module
 import {
@@ -73,6 +109,30 @@ const mockContext = {
   abort: new AbortController().signal,
 };
 
+/**
+ * Global test lifecycle hooks
+ *
+ * These document the test isolation pattern but don't actively clean up.
+ * Cleanup is manual via scripts/cleanup-test-memories.ts
+ */
+beforeAll(() => {
+  console.log(
+    `[test] TEST_SEMANTIC_MEMORY_COLLECTION = ${process.env.TEST_SEMANTIC_MEMORY_COLLECTION}`,
+  );
+  console.log(
+    `[test] Test collections will be prefixed with: swarm-*-${process.env.TEST_SEMANTIC_MEMORY_COLLECTION}`,
+  );
+});
+
+afterAll(() => {
+  console.log(
+    `[test] Test complete. Collections NOT auto-deleted (use scripts/cleanup-test-memories.ts for cleanup)`,
+  );
+  console.log(
+    `[test] Test collection suffix was: ${process.env.TEST_SEMANTIC_MEMORY_COLLECTION}`,
+  );
+});
+
 /**
  * Create a feedback event for testing
  */
@@ -1309,14 +1369,9 @@ describe("Storage Module", () => {
     beforeEach(async () => {
       isAvailable = await isSemanticMemoryAvailable();
       if (isAvailable) {
-        // Use unique collections per test run to ensure isolation
-        storage = new SemanticMemoryStorage({
-          collections: {
-            feedback: `test-feedback-learning-${Date.now()}`,
-            patterns: `test-patterns-learning-${Date.now()}`,
-            maturity: `test-maturity-learning-${Date.now()}`,
-          },
-        });
+        // Use default collections (which include TEST_SEMANTIC_MEMORY_COLLECTION suffix)
+        // This ensures all tests use the same isolated collections for this test run
+        storage = new SemanticMemoryStorage();
       }
     });
 

package/src/mandate-storage.test.ts

@@ -177,7 +177,7 @@ describe("InMemoryMandateStorage", () => {
     it("should throw when updating non-existent mandate", async () => {
       await expect(
         storage.update("non-existent", { content: "Updated" }),
-      ).rejects.toThrow("Mandate non-existent not found");
+      ).rejects.toThrow("Mandate 'non-existent' not found");
     });
   });
 
@@ -235,7 +235,7 @@ describe("InMemoryMandateStorage", () => {
       };
 
       await expect(storage.vote(vote2)).rejects.toThrow(
-        "Agent GreenRiver has already voted on mandate mandate-1",
+        "Agent 'GreenRiver' has already voted on mandate 'mandate-1'",
       );
     });
 
@@ -572,7 +572,7 @@ describe("InMemoryMandateStorage", () => {
         createMandateStorage({
           backend: "unknown" as "semantic-memory" | "memory",
         }),
-      ).toThrow("Unknown storage backend: unknown");
+      ).toThrow("Unknown storage backend: 'unknown'");
     });
   });
 });

package/src/storage.ts

@@ -141,11 +141,51 @@ export interface StorageConfig {
   useSemanticSearch: boolean;
 }
 
+/**
+ * Generate unique test collection name
+ *
+ * Creates a timestamp-based suffix for test collections to ensure complete isolation.
+ * Each test run gets its own collections that don't pollute production semantic-memory.
+ *
+ * @returns Unique suffix like "test-1734567890123"
+ *
+ * @example
+ * ```typescript
+ * // In test setup:
+ * process.env.TEST_SEMANTIC_MEMORY_COLLECTION = getTestCollectionName();
+ * // Results in collections like: swarm-feedback-test-1734567890123
+ * ```
+ */
+export function getTestCollectionName(): string {
+  return `test-${Date.now()}`;
+}
+
 /**
  * Get collection names with optional test suffix
  *
- * When TEST_MEMORY_COLLECTIONS=true, appends "-test" to all collection names
- * to isolate test data from production semantic-memory storage.
+ * Supports two test isolation modes:
+ * 1. TEST_MEMORY_COLLECTIONS=true - appends "-test" (shared across test run)
+ * 2. TEST_SEMANTIC_MEMORY_COLLECTION=<suffix> - appends custom suffix (unique per test run)
+ *
+ * Mode 2 is preferred for full isolation - prevents test pollution of production
+ * semantic-memory collections.
+ *
+ * @example
+ * ```typescript
+ * // Production
+ * getCollectionNames()
+ * // => { feedback: "swarm-feedback", patterns: "swarm-patterns", maturity: "swarm-maturity" }
+ *
+ * // Test mode 1 (legacy)
+ * process.env.TEST_MEMORY_COLLECTIONS = "true"
+ * getCollectionNames()
+ * // => { feedback: "swarm-feedback-test", patterns: "swarm-patterns-test", ... }
+ *
+ * // Test mode 2 (preferred - full isolation)
+ * process.env.TEST_SEMANTIC_MEMORY_COLLECTION = "test-1734567890123"
+ * getCollectionNames()
+ * // => { feedback: "swarm-feedback-test-1734567890123", patterns: "swarm-patterns-test-1734567890123", ... }
+ * ```
  */
 function getCollectionNames(): StorageCollections {
   const base = {
@@ -154,7 +194,17 @@ function getCollectionNames(): StorageCollections {
     maturity: "swarm-maturity",
   };
 
-  // Test isolation: suffix collections with "-test" when in test mode
+  // Test isolation mode 2 (preferred): unique suffix per test run
+  const testSuffix = process.env.TEST_SEMANTIC_MEMORY_COLLECTION;
+  if (testSuffix) {
+    return {
+      feedback: `${base.feedback}-${testSuffix}`,
+      patterns: `${base.patterns}-${testSuffix}`,
+      maturity: `${base.maturity}-${testSuffix}`,
+    };
+  }
+
+  // Test isolation mode 1 (legacy): shared "-test" suffix
   if (process.env.TEST_MEMORY_COLLECTIONS === "true") {
     return {
       feedback: `${base.feedback}-test`,
@@ -166,11 +216,27 @@ function getCollectionNames(): StorageCollections {
   return base;
 }
 
-export const DEFAULT_STORAGE_CONFIG: StorageConfig = {
-  backend: "semantic-memory",
-  collections: getCollectionNames(),
-  useSemanticSearch: true,
-};
+/**
+ * Get default storage configuration
+ *
+ * Returns a fresh config object on each call to ensure env vars (like
+ * TEST_SEMANTIC_MEMORY_COLLECTION) are read at runtime, not module load time.
+ *
+ * @returns Default storage configuration
+ */
+export function getDefaultStorageConfig(): StorageConfig {
+  return {
+    backend: "semantic-memory",
+    collections: getCollectionNames(),
+    useSemanticSearch: true,
+  };
+}
+
+/**
+ * @deprecated Use getDefaultStorageConfig() instead. This static export
+ * captures collections at module load time, breaking test isolation.
+ */
+export const DEFAULT_STORAGE_CONFIG: StorageConfig = getDefaultStorageConfig();
 
 // ============================================================================
 // Unified Storage Interface
@@ -261,7 +327,8 @@ export class SemanticMemoryStorage implements LearningStorage {
   private config: StorageConfig;
 
   constructor(config: Partial<StorageConfig> = {}) {
-    this.config = { ...DEFAULT_STORAGE_CONFIG, ...config };
+    // Use getDefaultStorageConfig() to ensure env vars are read at runtime
+    this.config = { ...getDefaultStorageConfig(), ...config };
     console.log(
       `[storage] SemanticMemoryStorage initialized with collections:`,
       this.config.collections,
@@ -723,7 +790,8 @@ export class InMemoryStorage implements LearningStorage {
 export function createStorage(
   config: Partial<StorageConfig> = {},
 ): LearningStorage {
-  const fullConfig = { ...DEFAULT_STORAGE_CONFIG, ...config };
+  // Use getDefaultStorageConfig() to ensure env vars are read at runtime
+  const fullConfig = { ...getDefaultStorageConfig(), ...config };
 
   switch (fullConfig.backend) {
     case "semantic-memory":

package/src/swarm-mail.ts

@@ -26,9 +26,9 @@ import {
   releaseSwarmFiles,
   acknowledgeSwarmMessage,
   checkSwarmHealth,
-} from "./streams/swarm-mail";
-import { getActiveReservations } from "./streams/projections";
-import type { MailSessionState } from "./streams/events";
+  getActiveReservations,
+  type MailSessionState,
+} from "swarm-mail";
 import {
   existsSync,
   mkdirSync,

package/src/swarm-orchestrate.ts

@@ -36,10 +36,10 @@ import {
   getSwarmInbox,
   releaseSwarmFiles,
   sendSwarmMessage,
-} from "./streams/swarm-mail";
-import { getAgent } from "./streams/projections";
-import { createEvent } from "./streams/events";
-import { appendEvent } from "./streams/store";
+  getAgent,
+  createEvent,
+  appendEvent,
+} from "swarm-mail";
 import {
   addStrike,
   clearStrikes,
@@ -872,7 +872,7 @@ export const swarm_progress = tool({
           await appendEvent(event, args.project_key);
 
           // Update swarm_contexts table
-          const { getDatabase } = await import("./streams/index");
+          const { getDatabase } = await import("swarm-mail");
           const db = await getDatabase(args.project_key);
           const now = Date.now();
           await db.query(
@@ -2062,7 +2062,7 @@ export const swarm_checkpoint = tool({
       await appendEvent(event, args.project_key);
 
       // Update swarm_contexts table for fast recovery
-      const { getDatabase } = await import("./streams/index");
+      const { getDatabase } = await import("swarm-mail");
       const db = await getDatabase(args.project_key);
 
       const now = Date.now();
@@ -2138,7 +2138,7 @@ export const swarm_recover = tool({
   },
   async execute(args) {
     try {
-      const { getDatabase } = await import("./streams/index");
+      const { getDatabase } = await import("swarm-mail");
       const db = await getDatabase(args.project_key);
 
       // Query most recent checkpoint for this epic

package/src/tool-availability.ts

@@ -13,7 +13,7 @@
  * - agent-mail: DEPRECATED - Legacy MCP server (use swarm-mail instead)
  */
 
-import { checkSwarmHealth } from "./streams/swarm-mail";
+import { checkSwarmHealth } from "swarm-mail";
 
 /** Default timeout for URL reachability checks in milliseconds */
 const DEFAULT_URL_TIMEOUT_MS = 2000;

package/tsconfig.json

@@ -7,13 +7,13 @@
     "types": ["bun-types"],
     "strict": true,
     "skipLibCheck": true,
-    "noEmit": true,
     "esModuleInterop": true,
     "allowSyntheticDefaultImports": true,
     "forceConsistentCasingInFileNames": true,
     "resolveJsonModule": true,
     "declaration": true,
     "declarationMap": true,
+    "emitDeclarationOnly": true,
     "sourceMap": true,
     "outDir": "./dist",
     "rootDir": "./src",

package/.beads/.local_version

@@ -1 +0,0 @@
-0.29.0

package/.beads/README.md

@@ -1,81 +0,0 @@
-# Beads - AI-Native Issue Tracking
-
-Welcome to Beads! This repository uses **Beads** for issue tracking - a modern, AI-native tool designed to live directly in your codebase alongside your code.
-
-## What is Beads?
-
-Beads is issue tracking that lives in your repo, making it perfect for AI coding agents and developers who want their issues close to their code. No web UI required - everything works through the CLI and integrates seamlessly with git.
-
-**Learn more:** [github.com/steveyegge/beads](https://github.com/steveyegge/beads)
-
-## Quick Start
-
-### Essential Commands
-
-```bash
-# Create new issues
-bd create "Add user authentication"
-
-# View all issues
-bd list
-
-# View issue details
-bd show <issue-id>
-
-# Update issue status
-bd update <issue-id> --status in_progress
-bd update <issue-id> --status done
-
-# Sync with git remote
-bd sync
-```
-
-### Working with Issues
-
-Issues in Beads are:
-- **Git-native**: Stored in `.beads/issues.jsonl` and synced like code
-- **AI-friendly**: CLI-first design works perfectly with AI coding agents
-- **Branch-aware**: Issues can follow your branch workflow
-- **Always in sync**: Auto-syncs with your commits
-
-## Why Beads?
-
-✨ **AI-Native Design**
-- Built specifically for AI-assisted development workflows
-- CLI-first interface works seamlessly with AI coding agents
-- No context switching to web UIs
-
-🚀 **Developer Focused**
-- Issues live in your repo, right next to your code
-- Works offline, syncs when you push
-- Fast, lightweight, and stays out of your way
-
-🔧 **Git Integration**
-- Automatic sync with git commits
-- Branch-aware issue tracking
-- Intelligent JSONL merge resolution
-
-## Get Started with Beads
-
-Try Beads in your own projects:
-
-```bash
-# Install Beads
-curl -sSL https://raw.githubusercontent.com/steveyegge/beads/main/scripts/install.sh | bash
-
-# Initialize in your repo
-bd init
-
-# Create your first issue
-bd create "Try out Beads"
-```
-
-## Learn More
-
-- **Documentation**: [github.com/steveyegge/beads/docs](https://github.com/steveyegge/beads/tree/main/docs)
-- **Quick Start Guide**: Run `bd quickstart`
-- **Examples**: [github.com/steveyegge/beads/examples](https://github.com/steveyegge/beads/tree/main/examples)
-
----
-
-*Beads: Issue tracking that moves at the speed of thought* ⚡