opencode-swarm-plugin 0.30.7 → 0.31.0

package/src/swarm-orchestrate.ts

@@ -72,7 +72,7 @@ import {
   isToolAvailable,
   warnMissingTool,
 } from "./tool-availability";
-import { getHiveAdapter } from "./hive";
+import { getHiveAdapter, hive_sync, setHiveWorkingDirectory, getHiveWorkingDirectory } from "./hive";
 import { listSkills } from "./skills";
 import {
   canUseWorktreeIsolation,
@@ -1570,6 +1570,30 @@ This will be recorded as a negative learning signal.`;
         );
       }
 
+      // Sync cell to .hive/issues.jsonl (auto-sync on complete)
+      // This ensures the worker's completed work persists before process exits
+      let syncSuccess = false;
+      let syncError: string | undefined;
+      try {
+        // Save current working directory and set to project path
+        const previousWorkingDir = getHiveWorkingDirectory();
+        setHiveWorkingDirectory(args.project_key);
+        
+        try {
+          const syncResult = await hive_sync.execute({ auto_pull: false }, _ctx);
+          syncSuccess = !syncResult.includes("error");
+        } finally {
+          // Restore previous working directory
+          setHiveWorkingDirectory(previousWorkingDir);
+        }
+      } catch (error) {
+        // Non-fatal - log warning but don't block completion
+        syncError = error instanceof Error ? error.message : String(error);
+        console.warn(
+          `[swarm_complete] Auto-sync failed (non-fatal): ${syncError}`,
+        );
+      }
+
       // Emit SubtaskOutcomeEvent for learning system
       try {
         const epicId = args.bead_id.includes(".")
@@ -1709,6 +1733,8 @@ This will be recorded as a negative learning signal.`;
         bead_id: args.bead_id,
         closed: true,
         reservations_released: true,
+        synced: syncSuccess,
+        sync_error: syncError,
         message_sent: messageSent,
         message_error: messageError,
         agent_registration: {

package/src/swarm-prompts.ts

@@ -46,7 +46,7 @@ Agents MUST update their cell status as they work. No silent progress.
 
 ## Requirements
 
-1. **Break into 2-{max_subtasks} independent subtasks** that can run in parallel
+1. **Break into independent subtasks** that can run in parallel (as many as needed)
 2. **Assign files** - each subtask must specify which files it will modify
 3. **No file overlap** - files cannot appear in multiple subtasks (they get exclusive locks)
 4. **Order by dependency** - if subtask B needs subtask A's output, A must come first in the array
@@ -123,7 +123,7 @@ Agents MUST update their cell status as they work. No silent progress.
 
 ## Requirements
 
-1. **Break into 2-{max_subtasks} independent subtasks** that can run in parallel
+1. **Break into independent subtasks** that can run in parallel (as many as needed)
 2. **Assign files** - each subtask must specify which files it will modify
 3. **No file overlap** - files cannot appear in multiple subtasks (they get exclusive locks)
 4. **Order by dependency** - if subtask B needs subtask A's output, A must come first in the array
@@ -927,10 +927,9 @@ export const swarm_plan_prompt = tool({
     max_subtasks: tool.schema
       .number()
       .int()
-      .min(2)
-      .max(10)
-      .default(5)
-      .describe("Maximum number of subtasks (default: 5)"),
+      .min(1)
+      .optional()
+      .describe("Suggested max subtasks (optional - LLM decides if not specified)"),
     context: tool.schema
       .string()
       .optional()
@@ -943,7 +942,6 @@ export const swarm_plan_prompt = tool({
       .number()
       .int()
       .min(1)
-      .max(10)
       .optional()
       .describe("Max CASS results to include (default: 3)"),
     include_skills: tool.schema

package/src/swarm.integration.test.ts

@@ -2341,4 +2341,74 @@ describe("Contract Validation", () => {
       expect(parsed.error).toBeUndefined();
     });
   });
+
+  describe("swarm_complete auto-sync", () => {
+    it("calls hive_sync after closing cell on successful completion", async () => {
+      const testProjectPath = "/tmp/swarm-auto-sync-test-" + Date.now();
+      const { getHiveAdapter } = await import("./hive");
+      const adapter = await getHiveAdapter(testProjectPath);
+
+      // Create a task cell directly
+      const cell = await adapter.createCell(testProjectPath, {
+        title: "Test task for auto-sync",
+        type: "task",
+        priority: 2,
+      });
+
+      // Start the task
+      await adapter.updateCell(testProjectPath, cell.id, {
+        status: "in_progress",
+      });
+
+      // Complete with skip_review and skip_verification
+      const result = await swarm_complete.execute(
+        {
+          project_key: testProjectPath,
+          agent_name: "TestAgent",
+          bead_id: cell.id,
+          summary: "Done - testing auto-sync",
+          files_touched: [],
+          skip_verification: true,
+          skip_review: true,
+        },
+        mockContext,
+      );
+
+      const parsed = JSON.parse(result);
+
+      // Should complete successfully
+      expect(parsed.success).toBe(true);
+      expect(parsed.closed).toBe(true);
+
+      // Check that cell is actually closed in database
+      const closedCell = await adapter.getCell(testProjectPath, cell.id);
+      expect(closedCell?.status).toBe("closed");
+
+      // The sync should have flushed the cell to .hive/issues.jsonl
+      // We can verify the cell appears in the JSONL
+      const hivePath = `${testProjectPath}/.hive/issues.jsonl`;
+      const hiveFile = Bun.file(hivePath);
+      const exists = await hiveFile.exists();
+
+      // The file should exist after sync
+      expect(exists).toBe(true);
+
+      if (exists) {
+        const content = await hiveFile.text();
+        const lines = content.trim().split("\n");
+
+        // Should have at least one cell exported
+        expect(lines.length).toBeGreaterThan(0);
+
+        // Parse the exported cells to find our closed cell
+        const cells = lines.map((line) => JSON.parse(line));
+        const exportedCell = cells.find((c) => c.id === cell.id);
+
+        // Our cell should be in the export
+        expect(exportedCell).toBeDefined();
+        expect(exportedCell.status).toBe("closed");
+        expect(exportedCell.title).toBe("Test task for auto-sync");
+      }
+    });
+  });
 });