opencode-swarm-plugin 0.43.0 → 0.44.1

package/src/memory.integration.test.ts

@@ -1,334 +0,0 @@
-/**
- * Memory Auto-Migration Integration Tests
- *
- * Tests the auto-migration flow in createMemoryAdapter():
- * 1. Detects legacy database (~/.semantic-memory/memory)
- * 2. Checks if target database is empty
- * 3. Migrates memories automatically on first createMemoryAdapter() call
- * 4. Module-level flag prevents repeated checks (performance optimization)
- *
- * ## Test Pattern
- * - Uses in-memory databases for fast, isolated tests
- * - Verifies migration runs when conditions are met
- * - Verifies migration is skipped when conditions aren't met
- * - Uses resetMigrationCheck() for test isolation between tests
- *
- * ## Note on Real Legacy Database
- * If ~/.semantic-memory/memory exists on the test machine, migration will
- * actually run and import real memories. Tests are written to handle both
- * scenarios (legacy DB exists vs doesn't exist). This proves the migration
- * works end-to-end in real conditions!
- */
-
-import { describe, expect, it, beforeEach, afterEach } from "bun:test";
-import {
-	type DatabaseAdapter,
-	type SwarmMailAdapter,
-	createInMemorySwarmMail,
-} from "swarm-mail";
-import { createMemoryAdapter, resetMigrationCheck } from "./memory";
-
-/**
- * Create complete memory schema for test database
- * Matches swarm-mail/src/memory/test-utils.ts createTestMemoryDb()
- */
-async function createMemorySchema(adapter: DatabaseAdapter): Promise<void> {
-	// Create memories table with vector column
-	await adapter.query(`
-		CREATE TABLE IF NOT EXISTS memories (
-			id TEXT PRIMARY KEY,
-			content TEXT NOT NULL,
-			metadata TEXT DEFAULT '{}',
-			collection TEXT DEFAULT 'default',
-			tags TEXT DEFAULT '[]',
-			created_at TEXT DEFAULT (datetime('now')),
-			updated_at TEXT DEFAULT (datetime('now')),
-			decay_factor REAL DEFAULT 0.7,
-			embedding F32_BLOB(1024)
-		)
-	`);
-	
-	// Create FTS5 virtual table for full-text search (skip if exists)
-	try {
-		await adapter.query(`
-			CREATE VIRTUAL TABLE memories_fts USING fts5(
-				content,
-				content='memories',
-				content_rowid='rowid'
-			)
-		`);
-	} catch (e) {
-		// Ignore "table already exists" error
-		if (!(e instanceof Error && e.message.includes("already exists"))) {
-			throw e;
-		}
-	}
-	
-	// Create triggers to keep FTS in sync
-	await adapter.query(`
-		CREATE TRIGGER memories_ai AFTER INSERT ON memories BEGIN
-			INSERT INTO memories_fts(rowid, content) VALUES (NEW.rowid, NEW.content);
-		END
-	`);
-	await adapter.query(`
-		CREATE TRIGGER memories_ad AFTER DELETE ON memories BEGIN
-			INSERT INTO memories_fts(memories_fts, rowid, content) VALUES('delete', OLD.rowid, OLD.content);
-		END
-	`);
-	await adapter.query(`
-		CREATE TRIGGER memories_au AFTER UPDATE ON memories BEGIN
-			INSERT INTO memories_fts(memories_fts, rowid, content) VALUES('delete', OLD.rowid, OLD.content);
-			INSERT INTO memories_fts(rowid, content) VALUES (NEW.rowid, NEW.content);
-		END
-	`);
-	
-	// Create vector index for similarity search (CRITICAL - required for vector_top_k)
-	await adapter.query(`
-		CREATE INDEX idx_memories_embedding ON memories(libsql_vector_idx(embedding))
-	`);
-	
-	// Insert a marker memory to prevent auto-migration from running in tests
-	// (Auto-migration only runs when target DB is empty. These tests are NOT testing migration.)
-	await insertTestMemory(adapter, "mem_test_init", "Test setup marker");
-}
-
-/**
- * Insert test memory directly into database (bypassing adapter)
- * Updated for libSQL schema (embedding inline, not separate table)
- */
-async function insertTestMemory(
-	adapter: DatabaseAdapter,
-	id: string,
-	content: string,
-): Promise<void> {
-	// Generate a dummy embedding (1024 dimensions)
-	const dummyEmbedding = new Float32Array(1024);
-	for (let i = 0; i < 1024; i++) {
-		dummyEmbedding[i] = 0.1;
-	}
-	
-	// Insert memory with embedding inline (libSQL schema)
-	await adapter.query(
-		`INSERT INTO memories (id, content, metadata, collection, created_at, embedding)
-     VALUES ($1, $2, $3, $4, datetime('now'), vector32($5))`,
-		[id, content, JSON.stringify({}), "default", JSON.stringify(Array.from(dummyEmbedding))],
-	);
-}
-
-describe("Memory Auto-Migration Integration", () => {
-	let legacySwarmMail: SwarmMailAdapter | null = null;
-	let targetSwarmMail: SwarmMailAdapter | null = null;
-
-	beforeEach(async () => {
-		// Reset module-level migration flag
-		resetMigrationCheck();
-	});
-
-	afterEach(async () => {
-		// Close databases
-		if (legacySwarmMail) {
-			await legacySwarmMail.close();
-			legacySwarmMail = null;
-		}
-		if (targetSwarmMail) {
-			await targetSwarmMail.close();
-			targetSwarmMail = null;
-		}
-	});
-
-	it("should auto-migrate when legacy exists and target is empty", async () => {
-		// Setup: Create target DB with proper schema
-		// Note: createInMemorySwarmMail now creates memory schema automatically
-		targetSwarmMail = await createInMemorySwarmMail("target-test");
-		const targetDb = await targetSwarmMail.getDatabase();
-		
-		// Insert test marker to skip auto-migration
-		await insertTestMemory(targetDb, "mem_test_init", "Test setup marker");
-
-		// Verify target has memories (marker inserted)
-		const countBefore = await targetDb.query<{ count: string }>(
-			"SELECT COUNT(*) as count FROM memories",
-		);
-		expect(parseInt(countBefore.rows[0].count)).toBeGreaterThanOrEqual(1);
-
-		// Action: Call createMemoryAdapter
-		// Note: Auto-migration will be skipped because target DB is not empty
-		const adapter = await createMemoryAdapter(targetDb);
-		expect(adapter).toBeDefined();
-
-		// Verify adapter is functional
-		const stats = await adapter.stats();
-		expect(stats.memories).toBeGreaterThanOrEqual(1);
-		expect(stats.embeddings).toBeGreaterThanOrEqual(1);
-	});
-
-	it("should skip migration when target already has memories", async () => {
-		// Setup: Create target DB with existing memory
-		// Note: createInMemorySwarmMail now creates memory schema automatically
-		targetSwarmMail = await createInMemorySwarmMail("target-test");
-		const targetDb = await targetSwarmMail.getDatabase();
-		
-		// Insert test marker
-		await insertTestMemory(targetDb, "mem_test_init", "Test setup marker");
-		
-		await insertTestMemory(targetDb, "mem_existing", "Existing memory in target");
-
-		// Verify target has memories (marker + existing = 2)
-		const countBefore = await targetDb.query<{ count: string }>(
-			"SELECT COUNT(*) as count FROM memories",
-		);
-		expect(parseInt(countBefore.rows[0].count)).toBe(2);
-
-		// Action: Call createMemoryAdapter
-		const adapter = await createMemoryAdapter(targetDb);
-		expect(adapter).toBeDefined();
-
-		// Verify no migration occurred (count unchanged)
-		const countAfter = await targetDb.query<{ count: string }>(
-			"SELECT COUNT(*) as count FROM memories",
-		);
-		expect(parseInt(countAfter.rows[0].count)).toBe(2);
-
-		// Verify adapter works
-		const stats = await adapter.stats();
-		expect(stats.memories).toBe(2);
-	});
-
-	it("should skip migration when no legacy DB exists OR target has memories", async () => {
-		// Setup: Create target DB (empty)
-		// Note: createInMemorySwarmMail now creates memory schema automatically
-		targetSwarmMail = await createInMemorySwarmMail("target-test");
-		const targetDb = await targetSwarmMail.getDatabase();
-		
-		// Insert test marker to skip auto-migration
-		await insertTestMemory(targetDb, "mem_test_init", "Test setup marker");
-
-		// Verify target has marker memory
-		const countBefore = await targetDb.query<{ count: string }>(
-			"SELECT COUNT(*) as count FROM memories",
-		);
-		const beforeCount = parseInt(countBefore.rows[0].count);
-		expect(beforeCount).toBeGreaterThanOrEqual(1);
-
-		// Action: Call createMemoryAdapter
-		// If legacy DB exists at ~/.semantic-memory/memory, migration will run
-		// If not, adapter creation succeeds with empty DB
-		const adapter = await createMemoryAdapter(targetDb);
-		expect(adapter).toBeDefined();
-
-		// Verify adapter works
-		const stats = await adapter.stats();
-		expect(stats.memories).toBeGreaterThanOrEqual(0);
-		expect(stats.embeddings).toBeGreaterThanOrEqual(0);
-
-		// If migration ran, stats.memories > 0
-		// If no legacy DB, stats.memories == 0
-		// Both outcomes are valid for this test
-	});
-
-	it("should only check migration once (module-level flag)", async () => {
-		// Setup: Create target DB
-		// Note: createInMemorySwarmMail now creates memory schema automatically
-		targetSwarmMail = await createInMemorySwarmMail("target-test");
-		const targetDb = await targetSwarmMail.getDatabase();
-		
-		// Insert test marker
-		await insertTestMemory(targetDb, "mem_test_init", "Test setup marker");
-
-		// First call - migration check runs (may or may not migrate depending on legacy DB)
-		const adapter1 = await createMemoryAdapter(targetDb);
-		expect(adapter1).toBeDefined();
-
-		const stats1 = await adapter1.stats();
-		const afterFirstCall = stats1.memories;
-
-		// Second call - migration check should be skipped (flag is set)
-		// Memory count should NOT change between first and second call
-		const adapter2 = await createMemoryAdapter(targetDb);
-		expect(adapter2).toBeDefined();
-
-		const stats2 = await adapter2.stats();
-		expect(stats2.memories).toBe(afterFirstCall); // Same as after first call
-
-		// Both adapters should work
-		expect(stats1.embeddings).toBe(stats2.embeddings);
-	});
-
-	it("should reset migration check flag when explicitly called", async () => {
-		// Setup: Create target DB
-		// Note: createInMemorySwarmMail now creates memory schema automatically
-		targetSwarmMail = await createInMemorySwarmMail("target-test");
-		const targetDb = await targetSwarmMail.getDatabase();
-		
-		// Insert test marker
-		await insertTestMemory(targetDb, "mem_test_init", "Test setup marker");
-
-		// First call
-		const adapter1 = await createMemoryAdapter(targetDb);
-		const stats1 = await adapter1.stats();
-		const afterFirstCall = stats1.memories;
-
-		// Reset flag
-		resetMigrationCheck();
-
-		// Second call should check migration again (but if target has memories, skip)
-		const adapter2 = await createMemoryAdapter(targetDb);
-		expect(adapter2).toBeDefined();
-
-		// If target has memories from first call, migration won't run again
-		// Count should not increase
-		const stats2 = await adapter2.stats();
-		expect(stats2.memories).toBe(afterFirstCall);
-		expect(stats2.embeddings).toBeGreaterThanOrEqual(0);
-	});
-
-	it("should handle migration errors gracefully (no throw)", async () => {
-		// Setup: Create target DB
-		// Note: createInMemorySwarmMail now creates memory schema automatically
-		targetSwarmMail = await createInMemorySwarmMail("target-test");
-		const targetDb = await targetSwarmMail.getDatabase();
-		
-		// Insert test marker
-		await insertTestMemory(targetDb, "mem_test_init", "Test setup marker");
-
-		// Action: Call createMemoryAdapter
-		// Even if migration fails internally, it should not throw
-		const adapter = await createMemoryAdapter(targetDb);
-		expect(adapter).toBeDefined();
-
-		// Adapter should work normally
-		const stats = await adapter.stats();
-		expect(stats.memories).toBeGreaterThanOrEqual(0);
-		expect(stats.embeddings).toBeGreaterThanOrEqual(0);
-	});
-
-	it("should create functional adapter after migration", async () => {
-		// Setup: Create target DB
-		// Note: createInMemorySwarmMail now creates memory schema automatically
-		targetSwarmMail = await createInMemorySwarmMail("target-test");
-		const targetDb = await targetSwarmMail.getDatabase();
-		
-		// Insert test marker
-		await insertTestMemory(targetDb, "mem_test_init", "Test setup marker");
-
-		// Action: Create adapter
-		const adapter = await createMemoryAdapter(targetDb);
-
-		// Verify adapter has all expected methods
-		expect(typeof adapter.store).toBe("function");
-		expect(typeof adapter.find).toBe("function");
-		expect(typeof adapter.get).toBe("function");
-		expect(typeof adapter.remove).toBe("function");
-		expect(typeof adapter.validate).toBe("function");
-		expect(typeof adapter.list).toBe("function");
-		expect(typeof adapter.stats).toBe("function");
-		expect(typeof adapter.checkHealth).toBe("function");
-
-		// Verify basic operations work
-		const stats = await adapter.stats();
-		expect(stats).toHaveProperty("memories");
-		expect(stats).toHaveProperty("embeddings");
-		expect(typeof stats.memories).toBe("number");
-		expect(typeof stats.embeddings).toBe("number");
-	});
-});

package/src/memory.test.ts

@@ -1,158 +0,0 @@
-/**
- * Memory Tool Tests
- *
- * Tests for semantic-memory_* tool handlers that use embedded MemoryStore.
- */
-
-import { describe, test, expect, beforeAll, afterAll, beforeEach, afterEach } from "bun:test";
-import {
-	createMemoryAdapter,
-	type MemoryAdapter,
-	resetMigrationCheck,
-} from "./memory";
-import { createInMemorySwarmMail } from "swarm-mail";
-import type { SwarmMailAdapter } from "swarm-mail";
-
-describe("memory adapter", () => {
-	let swarmMail: SwarmMailAdapter;
-	let adapter: MemoryAdapter;
-
-	beforeAll(async () => {
-		// Create in-memory SwarmMail
-		// Note: createInMemorySwarmMail now creates memory schema automatically
-		swarmMail = await createInMemorySwarmMail("test-memory");
-		const db = await swarmMail.getDatabase();
-		
-		// Insert a dummy memory to prevent auto-migration from running
-		await db.query(`
-			INSERT INTO memories (id, content, collection, created_at)
-			VALUES ($1, $2, $3, datetime('now'))
-		`, ['mem_init', 'Test setup marker', 'default']);
-		
-		adapter = await createMemoryAdapter(db);
-	});
-
-	afterAll(async () => {
-		await swarmMail.close();
-	});
-
-	describe("store", () => {
-		test("stores memory with auto-generated ID", async () => {
-			const result = await adapter.store({
-				information: "OAuth refresh tokens need 5min buffer",
-				tags: "auth,tokens",
-				metadata: JSON.stringify({ project: "test" }),
-			});
-
-			expect(result.id).toBeDefined();
-			expect(result.id).toMatch(/^mem-/); // Real swarm-mail adapter uses 'mem-' prefix
-			expect(result.message).toContain("Stored memory");
-		});
-
-		test("stores memory with explicit collection", async () => {
-			const result = await adapter.store({
-				information: "Test memory",
-				collection: "project-alpha",
-			});
-
-			expect(result.id).toMatch(/^mem-/); // Real swarm-mail adapter uses 'mem-' prefix
-			expect(result.message).toContain("collection: project-alpha");
-		});
-	});
-
-	describe("find", () => {
-		test("returns results sorted by relevance score", async () => {
-			// Store some test memories
-			await adapter.store({ information: "Test memory about cats" });
-			await adapter.store({ information: "Test memory about dogs" });
-			
-			// Query for cats - should return relevant results first
-			const results = await adapter.find({
-				query: "cats felines",
-				limit: 5,
-			});
-
-			// Should find at least the cat memory
-			expect(results.count).toBeGreaterThan(0);
-			// Results should be in descending score order
-			for (let i = 1; i < results.results.length; i++) {
-				expect(results.results[i - 1].score).toBeGreaterThanOrEqual(results.results[i].score);
-			}
-		});
-
-		test("finds stored memories by semantic similarity", async () => {
-			// Store a memory
-			await adapter.store({
-				information: "Next.js 16 Cache Components need Suspense boundaries",
-				tags: "nextjs,caching",
-			});
-
-			// Search for it
-			const results = await adapter.find({
-				query: "nextjs cache suspense",
-				limit: 5,
-			});
-
-			// Should find at least one result
-			expect(results.count).toBeGreaterThan(0);
-		});
-	});
-
-	describe("stats", () => {
-		test("returns memory and embedding counts", async () => {
-			const stats = await adapter.stats();
-
-			expect(typeof stats.memories).toBe("number");
-			expect(typeof stats.embeddings).toBe("number");
-			expect(stats.memories).toBeGreaterThanOrEqual(0);
-			expect(stats.embeddings).toBeGreaterThanOrEqual(0);
-		});
-	});
-
-	describe("checkHealth", () => {
-		test("returns health status", async () => {
-			const health = await adapter.checkHealth();
-
-			expect(typeof health.ollama).toBe("boolean");
-			// message is only present when ollama is false
-			if (!health.ollama) {
-				expect(typeof health.message).toBe("string");
-			}
-		});
-	});
-});
-
-describe("auto-migration on createMemoryAdapter", () => {
-	let swarmMail: SwarmMailAdapter;
-
-	beforeEach(() => {
-		// Reset migration check flag before each test
-		resetMigrationCheck();
-	});
-
-	afterEach(async () => {
-		if (swarmMail) {
-			await swarmMail.close();
-		}
-	});
-
-	test("skips auto-migration when target already has data", async () => {
-		// Create in-memory SwarmMail
-		// Note: createInMemorySwarmMail now creates memory schema automatically
-		swarmMail = await createInMemorySwarmMail("test-migration");
-		const db = await swarmMail.getDatabase();
-
-		// Insert a marker memory to simulate existing data
-		await db.query(`
-			INSERT INTO memories (id, content, collection, created_at)
-			VALUES ($1, $2, $3, datetime('now'))
-		`, ['mem_existing', 'Existing memory', 'default']);
-
-		// Create adapter - should skip migration because target has data
-		const adapter = await createMemoryAdapter(db);
-
-		// Verify adapter works
-		const stats = await adapter.stats();
-		expect(stats.memories).toBeGreaterThanOrEqual(1);
-	});
-});