opencode-swarm-plugin 0.22.0 → 0.23.1

package/src/streams/swarm-mail.ts

@@ -1,552 +0,0 @@
-/**
- * Swarm Mail - Embedded event-sourced implementation
- *
- * Replaces the MCP-based agent-mail with embedded PGLite storage.
- * Same API surface, but no external server dependency.
- *
- * Key features:
- * - Event sourcing for full audit trail
- * - Offset-based resumability (Durable Streams inspired)
- * - Materialized views for fast queries
- * - File reservation with conflict detection
- *
- * Effect-TS Integration:
- * - DurableMailbox for message send/receive (envelope pattern)
- * - DurableCursor for positioned inbox consumption with checkpointing
- * - DurableLock for file reservations (mutual exclusion via CAS)
- * - DurableDeferred for request/response messaging
- */
-import { createEvent } from "./events";
-import { isDatabaseHealthy, getDatabaseStats } from "./index";
-import {
-  checkConflicts,
-  getActiveReservations,
-  getInbox,
-  getMessage,
-} from "./projections";
-import { appendEvent, registerAgent, reserveFiles, sendMessage } from "./store";
-
-// ============================================================================
-// Constants
-// ============================================================================
-
-const MAX_INBOX_LIMIT = 5; // HARD CAP - context preservation
-const DEFAULT_TTL_SECONDS = 3600; // 1 hour
-
-// Agent name generation
-const ADJECTIVES = [
-  "Blue",
-  "Red",
-  "Green",
-  "Gold",
-  "Silver",
-  "Swift",
-  "Bright",
-  "Dark",
-  "Calm",
-  "Bold",
-  "Wise",
-  "Quick",
-  "Warm",
-  "Cool",
-  "Pure",
-  "Wild",
-];
-const NOUNS = [
-  "Lake",
-  "Stone",
-  "River",
-  "Mountain",
-  "Forest",
-  "Ocean",
-  "Star",
-  "Moon",
-  "Wind",
-  "Fire",
-  "Cloud",
-  "Storm",
-  "Dawn",
-  "Dusk",
-  "Hawk",
-  "Wolf",
-];
-
-function generateSwarmAgentName(): string {
-  const adj = ADJECTIVES[Math.floor(Math.random() * ADJECTIVES.length)];
-  const noun = NOUNS[Math.floor(Math.random() * NOUNS.length)];
-  return `${adj}${noun}`;
-}
-
-// ============================================================================
-// Types
-// ============================================================================
-
-export interface SwarmMailContext {
-  projectKey: string;
-  agentName: string;
-}
-
-export interface InitSwarmAgentOptions {
-  projectPath: string;
-  agentName?: string;
-  program?: string;
-  model?: string;
-  taskDescription?: string;
-}
-
-export interface SendSwarmMessageOptions {
-  projectPath: string;
-  fromAgent: string;
-  toAgents: string[];
-  subject: string;
-  body: string;
-  threadId?: string;
-  importance?: "low" | "normal" | "high" | "urgent";
-  ackRequired?: boolean;
-}
-
-export interface SendSwarmMessageResult {
-  success: boolean;
-  messageId: number;
-  threadId?: string;
-  recipientCount: number;
-}
-
-export interface GetSwarmInboxOptions {
-  projectPath: string;
-  agentName: string;
-  limit?: number;
-  urgentOnly?: boolean;
-  unreadOnly?: boolean;
-  includeBodies?: boolean;
-}
-
-export interface SwarmInboxMessage {
-  id: number;
-  from_agent: string;
-  subject: string;
-  body?: string;
-  thread_id: string | null;
-  importance: string;
-  created_at: number;
-}
-
-export interface SwarmInboxResult {
-  messages: SwarmInboxMessage[];
-  total: number;
-}
-
-export interface ReadSwarmMessageOptions {
-  projectPath: string;
-  messageId: number;
-  agentName?: string;
-  markAsRead?: boolean;
-}
-
-export interface ReserveSwarmFilesOptions {
-  projectPath: string;
-  agentName: string;
-  paths: string[];
-  reason?: string;
-  exclusive?: boolean;
-  ttlSeconds?: number;
-  force?: boolean;
-}
-
-export interface GrantedSwarmReservation {
-  id: number;
-  path_pattern: string;
-  exclusive: boolean;
-  expiresAt: number;
-}
-
-export interface SwarmReservationConflict {
-  path: string;
-  holder: string;
-  pattern: string;
-}
-
-export interface ReserveSwarmFilesResult {
-  granted: GrantedSwarmReservation[];
-  conflicts: SwarmReservationConflict[];
-}
-
-export interface ReleaseSwarmFilesOptions {
-  projectPath: string;
-  agentName: string;
-  paths?: string[];
-  reservationIds?: number[];
-}
-
-export interface ReleaseSwarmFilesResult {
-  released: number;
-  releasedAt: number;
-}
-
-export interface AcknowledgeSwarmOptions {
-  projectPath: string;
-  messageId: number;
-  agentName: string;
-}
-
-export interface AcknowledgeSwarmResult {
-  acknowledged: boolean;
-  acknowledgedAt: string | null;
-}
-
-export interface SwarmHealthResult {
-  healthy: boolean;
-  database: "connected" | "disconnected";
-  stats?: {
-    events: number;
-    agents: number;
-    messages: number;
-    reservations: number;
-  };
-}
-
-// ============================================================================
-// Agent Operations
-// ============================================================================
-
-/**
- * Initialize a swarm agent for this session
- *
- * Future: Can use DurableMailbox.create() for actor-style message consumption
- */
-export async function initSwarmAgent(
-  options: InitSwarmAgentOptions,
-): Promise<SwarmMailContext> {
-  const {
-    projectPath,
-    agentName = generateSwarmAgentName(),
-    program = "opencode",
-    model = "unknown",
-    taskDescription,
-  } = options;
-
-  // Register the agent (creates event + updates view)
-  await registerAgent(
-    projectPath, // Use projectPath as projectKey
-    agentName,
-    { program, model, taskDescription },
-    projectPath,
-  );
-
-  return {
-    projectKey: projectPath,
-    agentName,
-  };
-}
-
-// ============================================================================
-// Message Operations
-// ============================================================================
-
-/**
- * Send a message to other swarm agents
- *
- * Future: Use DurableMailbox.send() for envelope pattern with replyTo support
- */
-export async function sendSwarmMessage(
-  options: SendSwarmMessageOptions,
-): Promise<SendSwarmMessageResult> {
-  const {
-    projectPath,
-    fromAgent,
-    toAgents,
-    subject,
-    body,
-    threadId,
-    importance = "normal",
-    ackRequired = false,
-  } = options;
-
-  await sendMessage(
-    projectPath,
-    fromAgent,
-    toAgents,
-    subject,
-    body,
-    { threadId, importance, ackRequired },
-    projectPath,
-  );
-
-  // Get the message ID from the messages table (not the event ID)
-  const { getDatabase } = await import("./index");
-  const db = await getDatabase(projectPath);
-  const result = await db.query<{ id: number }>(
-    `SELECT id FROM messages 
-     WHERE project_key = $1 AND from_agent = $2 AND subject = $3
-     ORDER BY created_at DESC LIMIT 1`,
-    [projectPath, fromAgent, subject],
-  );
-
-  const messageId = result.rows[0]?.id ?? 0;
-
-  return {
-    success: true,
-    messageId,
-    threadId,
-    recipientCount: toAgents.length,
-  };
-}
-
-/**
- * Get inbox messages for a swarm agent
- *
- * Future: Use DurableCursor.consume() for positioned consumption with checkpointing
- */
-export async function getSwarmInbox(
-  options: GetSwarmInboxOptions,
-): Promise<SwarmInboxResult> {
-  const {
-    projectPath,
-    agentName,
-    limit = MAX_INBOX_LIMIT,
-    urgentOnly = false,
-    unreadOnly = false,
-    includeBodies = false,
-  } = options;
-
-  // Enforce max limit
-  const effectiveLimit = Math.min(limit, MAX_INBOX_LIMIT);
-
-  const messages = await getInbox(
-    projectPath,
-    agentName,
-    {
-      limit: effectiveLimit,
-      urgentOnly,
-      unreadOnly,
-      includeBodies,
-    },
-    projectPath,
-  );
-
-  return {
-    messages: messages.map((m) => ({
-      id: m.id,
-      from_agent: m.from_agent,
-      subject: m.subject,
-      body: includeBodies ? m.body : undefined,
-      thread_id: m.thread_id,
-      importance: m.importance,
-      created_at: m.created_at,
-    })),
-    total: messages.length,
-  };
-}
-
-/**
- * Read a single message with full body
- */
-export async function readSwarmMessage(
-  options: ReadSwarmMessageOptions,
-): Promise<SwarmInboxMessage | null> {
-  const { projectPath, messageId, agentName, markAsRead = false } = options;
-
-  const message = await getMessage(projectPath, messageId, projectPath);
-
-  if (!message) {
-    return null;
-  }
-
-  // Mark as read if requested
-  if (markAsRead && agentName) {
-    await appendEvent(
-      createEvent("message_read", {
-        project_key: projectPath,
-        message_id: messageId,
-        agent_name: agentName,
-      }),
-      projectPath,
-    );
-  }
-
-  return {
-    id: message.id,
-    from_agent: message.from_agent,
-    subject: message.subject,
-    body: message.body,
-    thread_id: message.thread_id,
-    importance: message.importance,
-    created_at: message.created_at,
-  };
-}
-
-// ============================================================================
-// Reservation Operations
-// ============================================================================
-
-/**
- * Reserve files for exclusive editing
- *
- * Always grants reservations (even with conflicts) - conflicts are warnings, not blockers.
- * This matches the test expectations and allows agents to proceed with awareness.
- *
- * Future: Use DurableLock.acquire() for distributed mutex with automatic expiry
- */
-export async function reserveSwarmFiles(
-  options: ReserveSwarmFilesOptions,
-): Promise<ReserveSwarmFilesResult> {
-  const {
-    projectPath,
-    agentName,
-    paths,
-    reason,
-    exclusive = true,
-    ttlSeconds = DEFAULT_TTL_SECONDS,
-  } = options;
-
-  // Check for conflicts first
-  const conflicts = await checkConflicts(
-    projectPath,
-    agentName,
-    paths,
-    projectPath,
-  );
-
-  // Always create reservations - conflicts are warnings, not blockers
-  await reserveFiles(
-    projectPath,
-    agentName,
-    paths,
-    { reason, exclusive, ttlSeconds },
-    projectPath,
-  );
-
-  // Query the actual reservation IDs from the database
-  const reservations = await getActiveReservations(
-    projectPath,
-    projectPath,
-    agentName,
-  );
-
-  // Filter to just the paths we reserved (most recent ones)
-  const granted: GrantedSwarmReservation[] = reservations
-    .filter((r) => paths.includes(r.path_pattern))
-    .map((r) => ({
-      id: r.id,
-      path_pattern: r.path_pattern,
-      exclusive: r.exclusive,
-      expiresAt: r.expires_at,
-    }));
-
-  return {
-    granted,
-    conflicts: conflicts.map((c) => ({
-      path: c.path,
-      holder: c.holder,
-      pattern: c.pattern,
-    })),
-  };
-}
-
-/**
- * Release file reservations
- *
- * Future: Use DurableLock.release() for automatic cleanup
- */
-export async function releaseSwarmFiles(
-  options: ReleaseSwarmFilesOptions,
-): Promise<ReleaseSwarmFilesResult> {
-  const { projectPath, agentName, paths, reservationIds } = options;
-
-  // Get current reservations to count what we're releasing
-  const currentReservations = await getActiveReservations(
-    projectPath,
-    projectPath,
-    agentName,
-  );
-
-  let releaseCount = 0;
-
-  if (paths && paths.length > 0) {
-    // Release specific paths
-    releaseCount = currentReservations.filter((r) =>
-      paths.includes(r.path_pattern),
-    ).length;
-  } else if (reservationIds && reservationIds.length > 0) {
-    // Release by ID
-    releaseCount = currentReservations.filter((r) =>
-      reservationIds.includes(r.id),
-    ).length;
-  } else {
-    // Release all
-    releaseCount = currentReservations.length;
-  }
-
-  // Create release event
-  await appendEvent(
-    createEvent("file_released", {
-      project_key: projectPath,
-      agent_name: agentName,
-      paths,
-      reservation_ids: reservationIds,
-    }),
-    projectPath,
-  );
-
-  return {
-    released: releaseCount,
-    releasedAt: Date.now(),
-  };
-}
-
-// ============================================================================
-// Acknowledgement Operations
-// ============================================================================
-
-/**
- * Acknowledge a swarm message
- */
-export async function acknowledgeSwarmMessage(
-  options: AcknowledgeSwarmOptions,
-): Promise<AcknowledgeSwarmResult> {
-  const { projectPath, messageId, agentName } = options;
-
-  const timestamp = Date.now();
-
-  await appendEvent(
-    createEvent("message_acked", {
-      project_key: projectPath,
-      message_id: messageId,
-      agent_name: agentName,
-    }),
-    projectPath,
-  );
-
-  return {
-    acknowledged: true,
-    acknowledgedAt: new Date(timestamp).toISOString(),
-  };
-}
-
-// ============================================================================
-// Health Check
-// ============================================================================
-
-/**
- * Check if the swarm mail store is healthy
- */
-export async function checkSwarmHealth(
-  projectPath?: string,
-): Promise<SwarmHealthResult> {
-  const healthy = await isDatabaseHealthy(projectPath);
-
-  if (!healthy) {
-    return {
-      healthy: false,
-      database: "disconnected",
-    };
-  }
-
-  const stats = await getDatabaseStats(projectPath);
-
-  return {
-    healthy: true,
-    database: "connected",
-    stats,
-  };
-}

package/test-bug-fixes.ts

@@ -1,86 +0,0 @@
-/**
- * Quick test to verify bug fixes in store.ts
- *
- * Bug 1 (xcavl.5): Transaction rollback error propagation
- * Bug 2 (xcavl.6): File reservation idempotency
- */
-
-import { getDatabase, resetDatabase } from "./src/streams/index";
-import { reserveFiles } from "./src/streams/store";
-
-async function testBug1_RollbackErrorPropagation() {
-  console.log("\n=== Testing Bug 1: Rollback Error Propagation ===");
-  console.log(
-    "Note: This is hard to test in isolation without forcing connection loss.",
-  );
-  console.log(
-    "The fix ensures composite errors are thrown when both transaction AND rollback fail.",
-  );
-  console.log(
-    "✅ Code review confirms fix is in place (see lines 141-166 in store.ts)",
-  );
-}
-
-async function testBug2_ReservationIdempotency() {
-  console.log("\n=== Testing Bug 2: File Reservation Idempotency ===");
-
-  await resetDatabase(); // Clean slate
-  const db = await getDatabase();
-
-  const projectKey = "test-project";
-  const agentName = "TestWorker";
-  const paths = ["src/file.ts", "src/other.ts"];
-
-  // First reservation
-  await reserveFiles(projectKey, agentName, paths, {
-    reason: "Initial reservation",
-    exclusive: true,
-    ttlSeconds: 3600,
-  });
-
-  // Query reservations
-  const result1 = await db.query<{ count: string }>(
-    `SELECT COUNT(*) as count FROM reservations 
-     WHERE project_key = $1 AND agent_name = $2 AND released_at IS NULL`,
-    [projectKey, agentName],
-  );
-  const count1 = parseInt(result1.rows[0]?.count || "0");
-  console.log(`After first reservation: ${count1} active reservations`);
-
-  // RETRY the same reservation (simulating network timeout + retry)
-  await reserveFiles(projectKey, agentName, paths, {
-    reason: "Retry after timeout",
-    exclusive: true,
-    ttlSeconds: 3600,
-  });
-
-  // Query again - should still be 2 (idempotent)
-  const result2 = await db.query<{ count: string }>(
-    `SELECT COUNT(*) as count FROM reservations 
-     WHERE project_key = $1 AND agent_name = $2 AND released_at IS NULL`,
-    [projectKey, agentName],
-  );
-  const count2 = parseInt(result2.rows[0]?.count || "0");
-  console.log(`After retry: ${count2} active reservations`);
-
-  if (count1 === count2 && count1 === 2) {
-    console.log("✅ PASS: Reservation is idempotent (no duplicates created)");
-  } else {
-    console.log(
-      `❌ FAIL: Expected 2 reservations both times, got ${count1} then ${count2}`,
-    );
-  }
-}
-
-async function main() {
-  try {
-    await testBug1_RollbackErrorPropagation();
-    await testBug2_ReservationIdempotency();
-    console.log("\n=== All tests complete ===\n");
-  } catch (error) {
-    console.error("Test failed:", error);
-    process.exit(1);
-  }
-}
-
-main();

package/vitest.integration.config.ts

@@ -1,19 +0,0 @@
-import { defineConfig } from "vitest/config";
-
-export default defineConfig({
-  test: {
-    include: ["src/**/*.integration.test.ts"],
-    testTimeout: 30000, // Integration tests may be slower
-    hookTimeout: 30000,
-    // Run serially to avoid race conditions with shared services
-    sequence: {
-      concurrent: false,
-    },
-    env: {
-      // Enable test-specific collections to isolate test data from production
-      TEST_MEMORY_COLLECTIONS: "true",
-    },
-    // Global setup/teardown hooks
-    globalSetup: "./vitest.integration.setup.ts",
-  },
-});

package/vitest.integration.setup.ts

@@ -1,48 +0,0 @@
-/**
- * Global setup/teardown for integration tests
- *
- * Ensures test-specific semantic-memory collections are cleaned up
- * after all integration tests complete.
- */
-
-export async function setup() {
-  console.log("[vitest] Integration test setup: TEST_MEMORY_COLLECTIONS=true");
-  // Setup runs before tests - environment variables are already set via vitest config
-}
-
-export async function teardown() {
-  console.log(
-    "[vitest] Integration test teardown: cleaning up test collections",
-  );
-
-  // Clean up test collections
-  const testCollections = [
-    "swarm-feedback-test",
-    "swarm-patterns-test",
-    "swarm-maturity-test",
-    "swarm-maturity-test-feedback",
-  ];
-
-  for (const collection of testCollections) {
-    try {
-      // Attempt to remove test collection data
-      // Note: semantic-memory doesn't have a built-in "delete collection" command,
-      // so we'll use the remove command with a wildcard or rely on TTL/manual cleanup
-      console.log(`[vitest] Attempting to clean collection: ${collection}`);
-
-      // List items and remove them (semantic-memory may not support bulk delete)
-      // This is a best-effort cleanup - some backends may require manual cleanup
-      // For now, we'll just log that cleanup should happen
-      console.log(
-        `[vitest] Note: Collection "${collection}" may need manual cleanup via semantic-memory CLI`,
-      );
-    } catch (error) {
-      console.warn(
-        `[vitest] Failed to clean collection ${collection}:`,
-        error instanceof Error ? error.message : String(error),
-      );
-    }
-  }
-
-  console.log("[vitest] Integration test teardown complete");
-}