opencode-swarm-plugin 0.31.7 → 0.32.0

package/src/swarm-review.integration.test.ts

@@ -0,0 +1,290 @@
+/**
+ * Integration tests for swarm review feedback flow
+ * 
+ * Tests the coordinator review feedback workflow with real HiveAdapter and swarm-mail.
+ * Verifies that review approval/rejection properly updates state and sends messages.
+ * 
+ * **STATUS**: URL_INVALID bug FIXED by commit 7bf9385 (libSQL URL normalization).
+ * Tests now execute without URL errors. sendSwarmMessage successfully creates adapters.
+ *
+ * **REMAINING ISSUE**: Message retrieval not working. getInbox returns empty even though
+ * sendSwarmMessage succeeds. Possible causes:
+ * - Database adapter instance mismatch (sendSwarmMessage creates new adapter each call)
+ * - Message projection not materializing from events
+ * - Database path resolution issue between send and receive
+ *
+ * Tests currently SKIPPED pending message retrieval fix.
+ */
+
+import { afterEach, beforeEach, describe, expect, test } from "bun:test";
+import { join } from "node:path";
+import { mkdirSync, rmSync } from "node:fs";
+import { tmpdir } from "node:os";
+import {
+	type SwarmMailAdapter,
+	clearAdapterCache,
+	getSwarmMailLibSQL,
+} from "swarm-mail";
+import {
+	getHiveAdapter,
+	hive_create,
+	setHiveWorkingDirectory,
+} from "./hive";
+import { swarm_review, swarm_review_feedback } from "./swarm-review";
+
+const mockContext = {
+	sessionID: `test-review-integration-${Date.now()}`,
+	messageID: `test-message-${Date.now()}`,
+	agent: "test-coordinator",
+	abort: new AbortController().signal,
+};
+
+describe("swarm_review integration", () => {
+	let testProjectPath: string;
+	let swarmMail: SwarmMailAdapter;
+
+	beforeEach(async () => {
+		// Create temp project directory
+		testProjectPath = join(tmpdir(), `swarm-review-test-${Date.now()}`);
+		mkdirSync(testProjectPath, { recursive: true });
+
+		// Initialize swarm-mail with file-based database
+		// (in-memory doesn't work with sendSwarmMessage's auto-adapter creation)
+		swarmMail = await getSwarmMailLibSQL(testProjectPath);
+
+		// Set hive working directory so hive tools work
+		setHiveWorkingDirectory(testProjectPath);
+
+		// Register coordinator and worker agents
+		await swarmMail.registerAgent(testProjectPath, "coordinator", {
+			program: "test",
+			model: "test-model",
+		});
+		await swarmMail.registerAgent(testProjectPath, "TestWorker", {
+			program: "test",
+			model: "test-model",
+		});
+	});
+
+	afterEach(async () => {
+		// Clean up
+		await swarmMail.close();
+		clearAdapterCache();
+		rmSync(testProjectPath, { recursive: true, force: true });
+	});
+
+	test("review approved flow", async () => {
+		// Setup: create epic + subtask via hive tools
+		const epicResult = await hive_create.execute(
+			{
+				title: "Test Epic",
+				type: "epic",
+				priority: 1,
+			},
+			mockContext
+		);
+		const epic = JSON.parse(epicResult);
+
+		const subtaskResult = await hive_create.execute(
+			{
+				title: "Test Subtask",
+				type: "task",
+				priority: 2,
+				parent_id: epic.id,
+			},
+			mockContext
+		);
+		const subtask = JSON.parse(subtaskResult);
+
+		// Call swarm_review to generate review prompt
+		const reviewResult = await swarm_review.execute(
+			{
+				project_key: testProjectPath,
+				epic_id: epic.id,
+				task_id: subtask.id,
+				files_touched: ["test.ts"],
+			},
+			mockContext
+		);
+
+		const reviewParsed = JSON.parse(reviewResult);
+		expect(reviewParsed).toHaveProperty("review_prompt");
+		expect(reviewParsed.context.epic_id).toBe(epic.id);
+		expect(reviewParsed.context.task_id).toBe(subtask.id);
+		expect(reviewParsed.context.remaining_attempts).toBe(3);
+
+		// Call swarm_review_feedback with status="approved"
+		const feedbackResult = await swarm_review_feedback.execute(
+			{
+				project_key: testProjectPath,
+				task_id: subtask.id,
+				worker_id: "TestWorker",
+				status: "approved",
+				summary: "Looks good, clean implementation",
+			},
+			mockContext
+		);
+
+		const feedbackParsed = JSON.parse(feedbackResult);
+		expect(feedbackParsed.success).toBe(true);
+		expect(feedbackParsed.status).toBe("approved");
+		expect(feedbackParsed.task_id).toBe(subtask.id);
+
+		// Verify message was sent to worker
+		const messages = await swarmMail.getInbox(
+			testProjectPath,
+			"TestWorker",
+			{ limit: 10 }
+		);
+		expect(messages.length).toBeGreaterThan(0);
+
+		const approvalMessage = messages.find((m) =>
+			m.subject.includes("APPROVED")
+		);
+		expect(approvalMessage).toBeDefined();
+		expect(approvalMessage?.subject).toContain(subtask.id);
+	});
+
+	test("review needs_changes flow", async () => {
+		// Setup: create epic + subtask
+		const epicResult = await hive_create.execute(
+			{
+				title: "Test Epic",
+				type: "epic",
+				priority: 1,
+			},
+			mockContext
+		);
+		const epic = JSON.parse(epicResult);
+
+		const subtaskResult = await hive_create.execute(
+			{
+				title: "Test Subtask",
+				type: "task",
+				priority: 2,
+				parent_id: epic.id,
+			},
+			mockContext
+		);
+		const subtask = JSON.parse(subtaskResult);
+
+		// Call swarm_review_feedback with status="needs_changes"
+		const issues = [
+			{
+				file: "src/auth.ts",
+				line: 42,
+				issue: "Missing null check",
+				suggestion: "Add if (!token) return null",
+			},
+		];
+
+		const feedbackResult = await swarm_review_feedback.execute(
+			{
+				project_key: testProjectPath,
+				task_id: subtask.id,
+				worker_id: "TestWorker",
+				status: "needs_changes",
+				issues: JSON.stringify(issues),
+			},
+			mockContext
+		);
+
+		const feedbackParsed = JSON.parse(feedbackResult);
+		expect(feedbackParsed.success).toBe(true);
+		expect(feedbackParsed.status).toBe("needs_changes");
+		expect(feedbackParsed.attempt).toBe(1);
+		expect(feedbackParsed.remaining_attempts).toBe(2);
+
+		// Verify retry count incremented
+		expect(feedbackParsed.attempt).toBe(1);
+
+		// Verify message was sent with issues
+		const messages = await swarmMail.getInbox(
+			testProjectPath,
+			"TestWorker",
+			{ limit: 10 }
+		);
+		expect(messages.length).toBeGreaterThan(0);
+
+		const needsChangesMessage = messages.find((m) =>
+			m.subject.includes("NEEDS CHANGES")
+		);
+		expect(needsChangesMessage).toBeDefined();
+		expect(needsChangesMessage?.subject).toContain(subtask.id);
+		expect(needsChangesMessage?.subject).toContain("attempt 1/3");
+	});
+
+	test("3-strike rule: task marked blocked after 3 rejections", async () => {
+		// Setup: create epic + subtask
+		const epicResult = await hive_create.execute(
+			{
+				title: "Test Epic",
+				type: "epic",
+				priority: 1,
+			},
+			mockContext
+		);
+		const epic = JSON.parse(epicResult);
+
+		const subtaskResult = await hive_create.execute(
+			{
+				title: "Test Subtask",
+				type: "task",
+				priority: 2,
+				parent_id: epic.id,
+			},
+			mockContext
+		);
+		const subtask = JSON.parse(subtaskResult);
+
+		const issues = [
+			{
+				file: "src/test.ts",
+				issue: "Still broken",
+			},
+		];
+
+		// Exhaust all 3 attempts
+		for (let i = 1; i <= 3; i++) {
+			const feedbackResult = await swarm_review_feedback.execute(
+				{
+					project_key: testProjectPath,
+					task_id: subtask.id,
+					worker_id: "TestWorker",
+					status: "needs_changes",
+					issues: JSON.stringify(issues),
+				},
+				mockContext
+			);
+
+			const feedbackParsed = JSON.parse(feedbackResult);
+			expect(feedbackParsed.success).toBe(true);
+			expect(feedbackParsed.status).toBe("needs_changes");
+			expect(feedbackParsed.attempt).toBe(i);
+			expect(feedbackParsed.remaining_attempts).toBe(3 - i);
+
+			if (i === 3) {
+				// Last attempt should mark task as failed
+				expect(feedbackParsed.task_failed).toBe(true);
+				expect(feedbackParsed.remaining_attempts).toBe(0);
+			}
+		}
+
+		// Verify task was marked as blocked in hive
+		const hive = await getHiveAdapter(testProjectPath);
+		const updatedCell = await hive.getCell(testProjectPath, subtask.id);
+		expect(updatedCell?.status).toBe("blocked");
+
+		// Verify final failure message was sent
+		const messages = await swarmMail.getInbox(
+			testProjectPath,
+			"TestWorker",
+			{ limit: 10 }
+		);
+
+		const failedMessage = messages.find((m) => m.subject.includes("FAILED"));
+		expect(failedMessage).toBeDefined();
+		expect(failedMessage?.subject).toContain("max review attempts reached");
+		expect(failedMessage?.importance).toBe("urgent");
+	});
+});

package/src/swarm.integration.test.ts

@@ -90,7 +90,7 @@ describe("swarm_decompose", () => {
     expect(parsed).toHaveProperty("expected_schema", "CellTree");
     expect(parsed).toHaveProperty("schema_hint");
     expect(parsed.prompt).toContain("Add user authentication with OAuth");
-    expect(parsed.prompt).toContain("2-3 independent subtasks");
+    expect(parsed.prompt).toContain("as many as needed");
   });
 
   it("includes context in prompt when provided", async () => {
@@ -120,8 +120,8 @@ describe("swarm_decompose", () => {
 
     const parsed = JSON.parse(result);
 
-    // Default is 5
-    expect(parsed.prompt).toContain("2-5 independent subtasks");
+    // Prompt should say "as many as needed" (max_subtasks no longer in template)
+    expect(parsed.prompt).toContain("as many as needed");
   });
 });
 
@@ -398,7 +398,7 @@ describe("swarm_plan_prompt", () => {
     );
     const parsed = JSON.parse(result);
 
-    expect(parsed.prompt).toContain("2-7 independent subtasks");
+    expect(parsed.prompt).toContain("as many as needed");
   });
 });
 
@@ -1790,8 +1790,9 @@ describe("Checkpoint/Recovery Flow (integration)", () => {
       const sessionID = `checkpoint-session-${Date.now()}`;
 
       // Initialize swarm-mail database directly (no Agent Mail needed)
-      const { getDatabase, closeDatabase } = await import("swarm-mail");
-      const db = await getDatabase(uniqueProjectKey);
+      const { getSwarmMailLibSQL, closeSwarmMailLibSQL } = await import("swarm-mail");
+      const swarmMail = await getSwarmMailLibSQL(uniqueProjectKey);
+      const db = await swarmMail.getDatabase();
 
       try {
         const ctx = {
@@ -1842,8 +1843,8 @@ describe("Checkpoint/Recovery Flow (integration)", () => {
         }>(
           `SELECT id, epic_id, bead_id, strategy, files, recovery 
            FROM swarm_contexts 
-           WHERE bead_id = $1`,
-          [beadId],
+           WHERE project_key = $1 AND bead_id = $2`,
+          [uniqueProjectKey, beadId],
         );
 
         expect(dbResult.rows.length).toBe(1);
@@ -1865,7 +1866,7 @@ describe("Checkpoint/Recovery Flow (integration)", () => {
         expect(recovery.files_modified).toEqual(["src/test.ts", "src/test2.ts"]);
         expect(recovery).toHaveProperty("last_checkpoint");
       } finally {
-        await closeDatabase(uniqueProjectKey);
+        await closeSwarmMailLibSQL(uniqueProjectKey);
       }
     });
 
@@ -1873,8 +1874,9 @@ describe("Checkpoint/Recovery Flow (integration)", () => {
       const uniqueProjectKey = `${TEST_PROJECT_PATH}-checkpoint-error-${Date.now()}`;
       const sessionID = `checkpoint-error-session-${Date.now()}`;
 
-      const { getDatabase, closeDatabase } = await import("swarm-mail");
-      const db = await getDatabase(uniqueProjectKey);
+      const { getSwarmMailLibSQL, closeSwarmMailLibSQL } = await import("swarm-mail");
+      const swarmMail = await getSwarmMailLibSQL(uniqueProjectKey);
+      const db = await swarmMail.getDatabase();
 
       try {
         const ctx = {
@@ -1901,8 +1903,8 @@ describe("Checkpoint/Recovery Flow (integration)", () => {
 
         // Verify error_context was stored
         const dbResult = await db.query<{ recovery: string }>(
-          `SELECT recovery FROM swarm_contexts WHERE bead_id = $1`,
-          ["bd-error-test.1"],
+          `SELECT recovery FROM swarm_contexts WHERE project_key = $1 AND bead_id = $2`,
+          [uniqueProjectKey, "bd-error-test.1"],
         );
 
         const recoveryRaw = dbResult.rows[0].recovery;
@@ -1912,7 +1914,7 @@ describe("Checkpoint/Recovery Flow (integration)", () => {
           "Hit type error on line 42, need to add explicit types",
         );
       } finally {
-        await closeDatabase(uniqueProjectKey);
+        await closeSwarmMailLibSQL(uniqueProjectKey);
       }
     });
   });
@@ -1922,8 +1924,9 @@ describe("Checkpoint/Recovery Flow (integration)", () => {
       const uniqueProjectKey = `${TEST_PROJECT_PATH}-recover-${Date.now()}`;
       const sessionID = `recover-session-${Date.now()}`;
 
-      const { getDatabase, closeDatabase } = await import("swarm-mail");
-      const db = await getDatabase(uniqueProjectKey);
+      const { getSwarmMailLibSQL, closeSwarmMailLibSQL } = await import("swarm-mail");
+      const swarmMail = await getSwarmMailLibSQL(uniqueProjectKey);
+      const db = await swarmMail.getDatabase();
 
       try {
         const ctx = {
@@ -1983,7 +1986,7 @@ describe("Checkpoint/Recovery Flow (integration)", () => {
           "swarm-coordination",
         ]);
       } finally {
-        await closeDatabase(uniqueProjectKey);
+        await closeSwarmMailLibSQL(uniqueProjectKey);
       }
     });
 
@@ -1991,8 +1994,8 @@ describe("Checkpoint/Recovery Flow (integration)", () => {
       const uniqueProjectKey = `${TEST_PROJECT_PATH}-recover-notfound-${Date.now()}`;
       const sessionID = `recover-notfound-session-${Date.now()}`;
 
-      const { getDatabase, closeDatabase } = await import("swarm-mail");
-      await getDatabase(uniqueProjectKey);
+      const { getSwarmMailLibSQL, closeSwarmMailLibSQL } = await import("swarm-mail");
+      await getSwarmMailLibSQL(uniqueProjectKey);
 
       try {
         const ctx = {
@@ -2015,7 +2018,7 @@ describe("Checkpoint/Recovery Flow (integration)", () => {
         expect(parsed.message).toContain("No checkpoint found");
         expect(parsed.epic_id).toBe("bd-nonexistent-epic");
       } finally {
-        await closeDatabase(uniqueProjectKey);
+        await closeSwarmMailLibSQL(uniqueProjectKey);
       }
     });
   });