@desplega.ai/agent-swarm 2.4.4 → 2.5.0

package/src/tests/fixtures/bitbucket/pullrequest-fulfilled.json

@@ -0,0 +1,134 @@
+{
+  "actor": {
+    "type": "user",
+    "display_name": "Dana Developer",
+    "nickname": "dana",
+    "account_id": "557058:11111111-1111-4111-8111-111111111111",
+    "uuid": "{11111111-2222-4333-8444-555555555555}",
+    "links": {
+      "html": {
+        "href": "https://bitbucket.org/%7B11111111-2222-4333-8444-555555555555%7D/"
+      }
+    }
+  },
+  "repository": {
+    "type": "repository",
+    "full_name": "acme-ws/widgets",
+    "name": "widgets",
+    "is_private": true,
+    "scm": "git",
+    "uuid": "{0f9b2c4e-7a1d-4e5f-9c3b-2d8e6a1f4b70}",
+    "links": {
+      "html": {
+        "href": "https://bitbucket.org/acme-ws/widgets"
+      }
+    },
+    "workspace": {
+      "type": "workspace",
+      "slug": "acme-ws",
+      "name": "Acme",
+      "uuid": "{a1a1a1a1-b2b2-4c3c-8d4d-e5e5e5e5e5e5}"
+    },
+    "project": {
+      "type": "project",
+      "key": "WID",
+      "name": "Widgets"
+    }
+  },
+  "pullrequest": {
+    "type": "pullrequest",
+    "id": 42,
+    "title": "Add retry to the widget fetcher",
+    "description": "Adds exponential backoff.",
+    "state": "MERGED",
+    "author": {
+      "type": "user",
+      "display_name": "Dana Developer",
+      "nickname": "dana",
+      "account_id": "557058:11111111-1111-4111-8111-111111111111",
+      "uuid": "{11111111-2222-4333-8444-555555555555}",
+      "links": {
+        "html": {
+          "href": "https://bitbucket.org/%7B11111111-2222-4333-8444-555555555555%7D/"
+        }
+      }
+    },
+    "source": {
+      "branch": {
+        "name": "feature/retry"
+      },
+      "commit": {
+        "hash": "1a2b3c4d5e6f"
+      },
+      "repository": {
+        "type": "repository",
+        "full_name": "acme-ws/widgets",
+        "name": "widgets",
+        "is_private": true,
+        "scm": "git",
+        "uuid": "{0f9b2c4e-7a1d-4e5f-9c3b-2d8e6a1f4b70}",
+        "links": {
+          "html": {
+            "href": "https://bitbucket.org/acme-ws/widgets"
+          }
+        },
+        "workspace": {
+          "type": "workspace",
+          "slug": "acme-ws",
+          "name": "Acme",
+          "uuid": "{a1a1a1a1-b2b2-4c3c-8d4d-e5e5e5e5e5e5}"
+        },
+        "project": {
+          "type": "project",
+          "key": "WID",
+          "name": "Widgets"
+        }
+      }
+    },
+    "destination": {
+      "branch": {
+        "name": "main"
+      },
+      "commit": {
+        "hash": "6f5e4d3c2b1a"
+      },
+      "repository": {
+        "type": "repository",
+        "full_name": "acme-ws/widgets",
+        "name": "widgets",
+        "is_private": true,
+        "scm": "git",
+        "uuid": "{0f9b2c4e-7a1d-4e5f-9c3b-2d8e6a1f4b70}",
+        "links": {
+          "html": {
+            "href": "https://bitbucket.org/acme-ws/widgets"
+          }
+        },
+        "workspace": {
+          "type": "workspace",
+          "slug": "acme-ws",
+          "name": "Acme",
+          "uuid": "{a1a1a1a1-b2b2-4c3c-8d4d-e5e5e5e5e5e5}"
+        },
+        "project": {
+          "type": "project",
+          "key": "WID",
+          "name": "Widgets"
+        }
+      }
+    },
+    "merge_commit": {
+      "hash": "abcdef123456"
+    },
+    "participants": [],
+    "reviewers": [],
+    "close_source_branch": true,
+    "created_on": "2026-10-22T09:00:00.000000+00:00",
+    "updated_on": "2026-10-22T09:00:00.000000+00:00",
+    "links": {
+      "html": {
+        "href": "https://bitbucket.org/acme-ws/widgets/pull-requests/42"
+      }
+    }
+  }
+}

package/src/tests/fixtures/bitbucket/pullrequest-rejected.json

@@ -0,0 +1,133 @@
+{
+  "actor": {
+    "type": "user",
+    "display_name": "Dana Developer",
+    "nickname": "dana",
+    "account_id": "557058:11111111-1111-4111-8111-111111111111",
+    "uuid": "{11111111-2222-4333-8444-555555555555}",
+    "links": {
+      "html": {
+        "href": "https://bitbucket.org/%7B11111111-2222-4333-8444-555555555555%7D/"
+      }
+    }
+  },
+  "repository": {
+    "type": "repository",
+    "full_name": "acme-ws/widgets",
+    "name": "widgets",
+    "is_private": true,
+    "scm": "git",
+    "uuid": "{0f9b2c4e-7a1d-4e5f-9c3b-2d8e6a1f4b70}",
+    "links": {
+      "html": {
+        "href": "https://bitbucket.org/acme-ws/widgets"
+      }
+    },
+    "workspace": {
+      "type": "workspace",
+      "slug": "acme-ws",
+      "name": "Acme",
+      "uuid": "{a1a1a1a1-b2b2-4c3c-8d4d-e5e5e5e5e5e5}"
+    },
+    "project": {
+      "type": "project",
+      "key": "WID",
+      "name": "Widgets"
+    }
+  },
+  "pullrequest": {
+    "type": "pullrequest",
+    "id": 42,
+    "title": "Add retry to the widget fetcher",
+    "description": "Adds exponential backoff.",
+    "state": "DECLINED",
+    "author": {
+      "type": "user",
+      "display_name": "Dana Developer",
+      "nickname": "dana",
+      "account_id": "557058:11111111-1111-4111-8111-111111111111",
+      "uuid": "{11111111-2222-4333-8444-555555555555}",
+      "links": {
+        "html": {
+          "href": "https://bitbucket.org/%7B11111111-2222-4333-8444-555555555555%7D/"
+        }
+      }
+    },
+    "source": {
+      "branch": {
+        "name": "feature/retry"
+      },
+      "commit": {
+        "hash": "1a2b3c4d5e6f"
+      },
+      "repository": {
+        "type": "repository",
+        "full_name": "acme-ws/widgets",
+        "name": "widgets",
+        "is_private": true,
+        "scm": "git",
+        "uuid": "{0f9b2c4e-7a1d-4e5f-9c3b-2d8e6a1f4b70}",
+        "links": {
+          "html": {
+            "href": "https://bitbucket.org/acme-ws/widgets"
+          }
+        },
+        "workspace": {
+          "type": "workspace",
+          "slug": "acme-ws",
+          "name": "Acme",
+          "uuid": "{a1a1a1a1-b2b2-4c3c-8d4d-e5e5e5e5e5e5}"
+        },
+        "project": {
+          "type": "project",
+          "key": "WID",
+          "name": "Widgets"
+        }
+      }
+    },
+    "destination": {
+      "branch": {
+        "name": "main"
+      },
+      "commit": {
+        "hash": "6f5e4d3c2b1a"
+      },
+      "repository": {
+        "type": "repository",
+        "full_name": "acme-ws/widgets",
+        "name": "widgets",
+        "is_private": true,
+        "scm": "git",
+        "uuid": "{0f9b2c4e-7a1d-4e5f-9c3b-2d8e6a1f4b70}",
+        "links": {
+          "html": {
+            "href": "https://bitbucket.org/acme-ws/widgets"
+          }
+        },
+        "workspace": {
+          "type": "workspace",
+          "slug": "acme-ws",
+          "name": "Acme",
+          "uuid": "{a1a1a1a1-b2b2-4c3c-8d4d-e5e5e5e5e5e5}"
+        },
+        "project": {
+          "type": "project",
+          "key": "WID",
+          "name": "Widgets"
+        }
+      }
+    },
+    "merge_commit": null,
+    "participants": [],
+    "reviewers": [],
+    "close_source_branch": true,
+    "created_on": "2026-10-22T09:00:00.000000+00:00",
+    "updated_on": "2026-10-22T09:00:00.000000+00:00",
+    "links": {
+      "html": {
+        "href": "https://bitbucket.org/acme-ws/widgets/pull-requests/42"
+      }
+    },
+    "reason": "Superseded by #43"
+  }
+}

package/src/tests/generate-mcp-docs.test.ts

@@ -0,0 +1,101 @@
+import { expect, test } from "bun:test";
+import { mkdir, mkdtemp, rm, symlink } from "node:fs/promises";
+import { tmpdir } from "node:os";
+import path from "node:path";
+import { CHILD_PROCESS_TEST_BUDGET_MS, expectChildOk, runChild } from "./test-proc";
+
+test(
+  "MCP docs retain all app-row helper registrations with descriptions and parameters",
+  async () => {
+    const root = path.resolve(import.meta.dir, "../..");
+    const temp = await mkdtemp(path.join(tmpdir(), "mcp-docs-"));
+    try {
+      await mkdir(path.join(temp, "scripts"));
+      await symlink(path.join(root, "src"), path.join(temp, "src"));
+      await symlink(path.join(root, "node_modules"), path.join(temp, "node_modules"));
+      const script = path.join(temp, "scripts/generate-mcp-docs.ts");
+      await Bun.write(script, Bun.file(path.join(root, "scripts/generate-mcp-docs.ts")));
+      const result = expectChildOk(
+        await runChild([process.execPath, script]),
+        "MCP docs generation",
+      );
+      const markdown = await Bun.file(path.join(temp, "MCP.md")).text();
+      expect(result.stderr).not.toContain('No info found for tool "app-row"');
+      expect(markdown).not.toContain("### app-row\n");
+      expect(markdown).not.toContain("[app-row](#app-row)");
+      // Direct subdirectory imports, barrel imports and multiple registrars
+      // sharing one file must keep their individual tools without duplicates.
+      for (const name of ["list-model-profiles", "get-config", "room-get", "room-reset"]) {
+        expect(markdown.split(`### ${name}\n`)).toHaveLength(2);
+      }
+      const tools = {
+        "app-rows-list": [
+          "List app rows with equality filters, sorting, and a limit.",
+          "equals",
+          "sort",
+          "limit",
+        ],
+        "app-row-get": ["Get an app row.", "rowId"],
+        "app-row-create": ["Create an app row.", "values"],
+        "app-rows-create": ["Create up to 500 app rows within 10 MiB.", "rows"],
+        "app-row-patch": ["Patch an app row.", "rowId", "values"],
+        "app-row-delete": ["Delete an app row.", "rowId"],
+        "app-rows-delete": [
+          "Delete matching app rows. Supply at least one equality filter.",
+          "equals",
+          "limit",
+        ],
+      };
+      for (const [name, [description, ...params]] of Object.entries(tools)) {
+        expect(markdown).toContain(`[${name}](#${name})`);
+        expect(markdown.split(`### ${name}\n`)).toHaveLength(2);
+        const section = markdown
+          .split(`### ${name}\n`)[1]
+          ?.split(/\n## /)[0]
+          ?.split(/\n### /)[0];
+        expect(section).toBeDefined();
+        expect(section).toContain(`\n${description}\n`);
+        for (const param of ["appId", "model", ...params]) {
+          expect(section).toContain(`| \`${param}\` |`);
+        }
+      }
+    } finally {
+      await rm(temp, { recursive: true, force: true });
+    }
+  },
+  CHILD_PROCESS_TEST_BUDGET_MS,
+);
+
+for (const name of ["chat-reply", "chat-post", "model-catalog-overlay-upsert"]) {
+  test(
+    `MCP docs preserve shared field metadata for ${name}`,
+    async () => {
+      const root = path.resolve(import.meta.dir, "../..");
+      const temp = await mkdtemp(path.join(tmpdir(), "mcp-docs-fields-"));
+      try {
+        await mkdir(path.join(temp, "scripts"));
+        await symlink(path.join(root, "src"), path.join(temp, "src"));
+        await symlink(path.join(root, "node_modules"), path.join(temp, "node_modules"));
+        const script = path.join(temp, "scripts/generate-mcp-docs.ts");
+        await Bun.write(script, Bun.file(path.join(root, "scripts/generate-mcp-docs.ts")));
+        expectChildOk(await runChild([process.execPath, script]), "MCP docs generation");
+        const markdown = await Bun.file(path.join(temp, "MCP.md")).text();
+        const section = markdown.split(`### ${name}\n`)[1]?.split(/\n#{2,3} /)[0];
+        expect(section).toBeDefined();
+        if (name.startsWith("chat-")) {
+          expect(section).toContain(
+            "| `attachments` | `string[]` | No | - | Files to send after the text, up to 10:",
+          );
+          expect(section).toContain("A file over the platform upload limit");
+        } else {
+          const provider = section?.split("\n").find((line) => line.startsWith("| `provider` |"));
+          expect(provider).toContain("opencode-go, nan, helmcode");
+          expect(provider).not.toContain("${");
+        }
+      } finally {
+        await rm(temp, { recursive: true, force: true });
+      }
+    },
+    CHILD_PROCESS_TEST_BUDGET_MS,
+  );
+}

package/src/tests/gitlab-handlers.test.ts

@@ -240,7 +240,7 @@ describe("handleMergeRequest", () => {
       agentId: "lead-gl-001",
     });
 
-    const existing = await findTaskByVcs("group/project", 60);
+    const existing = await findTaskByVcs("group/project", 60, "gitlab");
     expect(existing).not.toBeNull();
 
     const event = makeMREvent({
@@ -297,7 +297,7 @@ describe("handleMergeRequest", () => {
     const result = await handleMergeRequest(event);
     expect(result.created).toBe(false);
 
-    const existing = await findTaskByVcs("group/project", 61);
+    const existing = await findTaskByVcs("group/project", 61, "gitlab");
     // Should no longer be active (findTaskByVcs filters out completed/failed)
     expect(existing).toBeNull();
   });
@@ -444,7 +444,7 @@ describe("handleIssue", () => {
     expect(result.created).toBe(false);
 
     // Task should no longer be active
-    const task = await findTaskByVcs("group/project", 30);
+    const task = await findTaskByVcs("group/project", 30, "gitlab");
     expect(task).toBeNull();
   });
 });

package/src/tests/gitlab-vcs-db.test.ts

@@ -100,7 +100,7 @@ describe("findTaskByVcs", () => {
       agentId: "vcs-lead-001",
     });
 
-    const found = await findTaskByVcs("findme/gh-repo", 100);
+    const found = await findTaskByVcs("findme/gh-repo", 100, "github");
     expect(found).not.toBeNull();
     expect(found?.vcsProvider).toBe("github");
   });
@@ -114,7 +114,7 @@ describe("findTaskByVcs", () => {
       agentId: "vcs-lead-001",
     });
 
-    const found = await findTaskByVcs("findme/gl-project", 200);
+    const found = await findTaskByVcs("findme/gl-project", 200, "gitlab");
     expect(found).not.toBeNull();
     expect(found?.vcsProvider).toBe("gitlab");
   });
@@ -133,12 +133,12 @@ describe("findTaskByVcs", () => {
     await db.startTask(task.id);
     await db.completeTask(task.id, "Done");
 
-    const found = await findTaskByVcs("findme/completed", 300);
+    const found = await findTaskByVcs("findme/completed", 300, "gitlab");
     expect(found).toBeNull();
   });
 
   test("returns null for non-existent repo/number", async () => {
-    const found = await findTaskByVcs("nonexistent/repo", 9999);
+    const found = await findTaskByVcs("nonexistent/repo", 9999, "github");
     expect(found).toBeNull();
   });
 });

package/src/tests/heartbeat-checklist.test.ts

@@ -532,6 +532,39 @@ describe("Heartbeat Checklist", () => {
         expect(calls.map((call) => call.scriptName)).toEqual(["heartbeat-jev-gate"]);
       });
 
+      test("unset script with a Cloudflare token and account id runs heartbeat-jev-gate", async () => {
+        await setGlobal("CLOUDFLARE_API_TOKEN", "cf-test-token");
+        await setGlobal("CLOUDFLARE_ACCOUNT_ID", "0123456789abcdef0123456789abcdef");
+        useGate({ quiet: true });
+        await seedLead();
+
+        await checkHeartbeatChecklist();
+
+        expect(calls.map((call) => call.scriptName)).toEqual(["heartbeat-jev-gate"]);
+      });
+
+      test("CLOUDFLARE_API_KEY counts as the Cloudflare token", async () => {
+        await setGlobal("CLOUDFLARE_API_KEY", "cf-test-key");
+        await setGlobal("CLOUDFLARE_ACCOUNT_ID", "0123456789abcdef0123456789abcdef");
+        process.env.HEARTBEAT_JEV_GATE_PROVIDER = "cloudflare";
+        useGate({ quiet: true });
+        await seedLead();
+
+        await checkHeartbeatChecklist();
+
+        expect(calls.map((call) => call.scriptName)).toEqual(["heartbeat-jev-gate"]);
+      });
+
+      test("a Cloudflare token without an account id leaves the gate off", async () => {
+        await setGlobal("CLOUDFLARE_API_TOKEN", "cf-test-token");
+        useGate({ quiet: true });
+        await seedLead();
+
+        await checkHeartbeatChecklist();
+
+        expect(calls).toHaveLength(0);
+      });
+
       test("unset script and no Jev key: gate stays off", async () => {
         await setGlobal("ANTHROPIC_API_KEY", "unrelated");
         useGate({ quiet: true });

package/src/tests/heartbeat-jev-gate-credentials.test.ts

@@ -165,4 +165,48 @@ describe("system-one-decide keeps the key on the server", () => {
     ]);
     expect(JSON.stringify(result)).not.toContain(TYPESAFE_KEY);
   });
+
+  test("with no provider it uses SYSTEM_ONE_DEFAULT_PROVIDER, typesafe when unset", async () => {
+    const { provider: _omit, ...noProvider } = request("typesafe");
+    const unset = recordAuthorization();
+    expect((await tool().handler(noProvider, extra())).isError).toBeFalsy();
+    expect(unset.map((s) => s.url)).toEqual(["https://api.typesafe.ai/v1/systemone"]);
+
+    const account = "0123456789abcdef0123456789abcdef";
+    await upsertSwarmConfig({
+      scope: "global",
+      key: "SYSTEM_ONE_DEFAULT_PROVIDER",
+      value: "cloudflare",
+    });
+    await upsertSwarmConfig({ scope: "global", key: "CLOUDFLARE_ACCOUNT_ID", value: account });
+    await upsertSwarmConfig({
+      scope: "global",
+      key: "CLOUDFLARE_API_TOKEN",
+      value: "cf-default-provider-token",
+      isSecret: true,
+    });
+    const sent: string[] = [];
+    globalThis.fetch = (async (input: RequestInfo | URL) => {
+      sent.push(input instanceof Request ? input.url : String(input));
+      return Response.json({
+        result: {
+          model: "clef",
+          answers: { act: { type: "noul", noul: 0.7 } },
+          usage: { input_tokens: 12, output_tokens: 0 },
+        },
+        success: true,
+        errors: [],
+        messages: [],
+      });
+    }) as typeof fetch;
+    const result = await tool().handler(noProvider, extra());
+    expect(result.isError).toBeFalsy();
+    expect(result.structuredContent).toMatchObject({
+      model: "clef",
+      answers: { act: { noul: 0.7 } },
+    });
+    expect(sent).toEqual([
+      `https://api.cloudflare.com/client/v4/accounts/${account}/ai/run/@cf/cloudflare/clef`,
+    ]);
+  });
 });