@desplega.ai/agent-swarm 2.4.5 → 2.5.1

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

@@ -13,7 +13,7 @@ import {
   initDb,
 } from "../be/db";
 import { findUserByExternalId, linkIdentity } from "../be/users";
-import { GITLAB_BOT_NAME } from "../gitlab/auth";
+import { getGitLabBotName } from "../gitlab/auth";
 import { handleIssue, handleMergeRequest, handleNote, handlePipeline } from "../gitlab/handlers";
 import type { IssueEvent, MergeRequestEvent, NoteEvent, PipelineEvent } from "../gitlab/types";
 
@@ -37,7 +37,7 @@ function makeMREvent(overrides: Partial<MergeRequestEvent> = {}): MergeRequestEv
       id: 100,
       iid: 1,
       title: "Test MR",
-      description: `@${GITLAB_BOT_NAME} please review`,
+      description: `@${getGitLabBotName()} please review`,
       state: "opened",
       action: "open",
       source_branch: "feature",
@@ -66,7 +66,7 @@ function makeIssueEvent(overrides: Partial<IssueEvent> = {}): IssueEvent {
       id: 200,
       iid: 10,
       title: "Test Issue",
-      description: `@${GITLAB_BOT_NAME} fix this`,
+      description: `@${getGitLabBotName()} fix this`,
       state: "opened",
       action: "open",
       url: "https://gitlab.com/group/project/-/issues/10",
@@ -90,7 +90,7 @@ function makeNoteEvent(overrides: Partial<NoteEvent> = {}): NoteEvent {
     },
     object_attributes: {
       id: 300,
-      note: `@${GITLAB_BOT_NAME} can you help?`,
+      note: `@${getGitLabBotName()} can you help?`,
       noteable_type: "MergeRequest",
       noteable_id: 100,
       url: "https://gitlab.com/group/project/-/merge_requests/1#note_300",
@@ -181,7 +181,7 @@ describe("handleMergeRequest", () => {
         id: 101,
         iid: 50,
         title: "MR with mention",
-        description: `@${GITLAB_BOT_NAME} review this please`,
+        description: `@${getGitLabBotName()} review this please`,
         state: "opened",
         action: "open",
         source_branch: "feat-a",
@@ -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,18 +297,212 @@ 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();
   });
 
+  test("a merge cancels every active task for the MR, not only the newest", async () => {
+    const mrTask = await createTaskExtended("[GitLab MR #62] Test", {
+      source: "gitlab",
+      vcsProvider: "gitlab",
+      vcsRepo: "group/project",
+      vcsEventType: "merge_request",
+      vcsNumber: 62,
+      agentId: "lead-gl-001",
+    });
+    const noteTask = await createTaskExtended("[GitLab MR #62] Comment", {
+      source: "gitlab",
+      vcsProvider: "gitlab",
+      vcsRepo: "group/project",
+      vcsEventType: "note_on_mr",
+      vcsNumber: 62,
+      agentId: "lead-gl-001",
+      parentTaskId: mrTask.id,
+    });
+
+    const event = makeMREvent({
+      object_attributes: {
+        id: 105,
+        iid: 62,
+        title: "MR to merge",
+        description: null,
+        state: "merged",
+        action: "merge",
+        source_branch: "feat-e",
+        target_branch: "main",
+        url: "https://gitlab.com/group/project/-/merge_requests/62",
+        last_commit: null,
+        author_id: 1,
+      },
+    });
+    await handleMergeRequest(event);
+
+    for (const id of [mrTask.id, noteTask.id]) {
+      const task = await getTaskById(id);
+      expect(task?.status).toBe("failed");
+      expect(task?.failureReason).toBe("MR was merged");
+    }
+  });
+
+  test("closing MR !5 leaves issue #5 and its notes active", async () => {
+    // GitLab issue and MR iids are separate sequences: issue #5 and MR !5 coexist.
+    const issue = await handleIssue(
+      makeIssueEvent({
+        object_attributes: {
+          id: 205,
+          iid: 5,
+          title: "Issue five",
+          description: `@${getGitLabBotName()} fix this`,
+          state: "opened",
+          action: "open",
+          url: "https://gitlab.com/group/project/-/issues/5",
+          author_id: 1,
+        },
+      }),
+    );
+    const issueNote = await handleNote(
+      makeNoteEvent({
+        object_attributes: {
+          id: 395,
+          note: `@${getGitLabBotName()} any update?`,
+          noteable_type: "Issue",
+          noteable_id: 205,
+          url: "https://gitlab.com/group/project/-/issues/5#note_395",
+          author_id: 1,
+          type: null,
+        },
+        merge_request: undefined,
+        issue: {
+          id: 205,
+          iid: 5,
+          title: "Issue five",
+          description: null,
+          state: "opened",
+          action: "open",
+          url: "https://gitlab.com/group/project/-/issues/5",
+          author_id: 1,
+        },
+      }),
+    );
+    const mr = await handleMergeRequest(
+      makeMREvent({
+        object_attributes: {
+          id: 155,
+          iid: 5,
+          title: "MR five",
+          description: `@${getGitLabBotName()} please review`,
+          state: "opened",
+          action: "open",
+          source_branch: "feat-five",
+          target_branch: "main",
+          url: "https://gitlab.com/group/project/-/merge_requests/5",
+          last_commit: null,
+          author_id: 1,
+        },
+      }),
+    );
+    const mrNote = await handleNote(
+      makeNoteEvent({
+        object_attributes: {
+          id: 396,
+          note: `@${getGitLabBotName()} looks good?`,
+          noteable_type: "MergeRequest",
+          noteable_id: 155,
+          url: "https://gitlab.com/group/project/-/merge_requests/5#note_396",
+          author_id: 1,
+          type: null,
+        },
+        merge_request: {
+          id: 155,
+          iid: 5,
+          title: "MR five",
+          description: null,
+          state: "opened",
+          action: "open",
+          source_branch: "feat-five",
+          target_branch: "main",
+          url: "https://gitlab.com/group/project/-/merge_requests/5",
+          last_commit: null,
+          author_id: 1,
+        },
+      }),
+    );
+    // A pre-contextKey issue row must not be swept up either.
+    const legacyIssue = await createTaskExtended("[GitLab Issue #5] Legacy", {
+      source: "gitlab",
+      vcsProvider: "gitlab",
+      vcsRepo: "group/project",
+      vcsEventType: "issue",
+      vcsNumber: 5,
+      agentId: "lead-gl-001",
+    });
+    for (const r of [issue, issueNote, mr, mrNote]) expect(r.created).toBe(true);
+
+    await handleMergeRequest(
+      makeMREvent({
+        object_attributes: {
+          id: 155,
+          iid: 5,
+          title: "MR five",
+          description: null,
+          state: "closed",
+          action: "close",
+          source_branch: "feat-five",
+          target_branch: "main",
+          url: "https://gitlab.com/group/project/-/merge_requests/5",
+          last_commit: null,
+          author_id: 1,
+        },
+      }),
+    );
+
+    for (const id of [mr.taskId!, mrNote.taskId!]) {
+      const task = await getTaskById(id);
+      expect(task?.status).toBe("failed");
+      expect(task?.failureReason).toBe("MR was closed");
+    }
+    for (const id of [issue.taskId!, issueNote.taskId!, legacyIssue.id]) {
+      const task = await getTaskById(id);
+      expect(task?.status).not.toBe("failed");
+    }
+  });
+
+  test("reads GITLAB_BOT_NAME at match time, so a config reload applies", async () => {
+    const mrEvent = (iid: number, description: string) =>
+      makeMREvent({
+        object_attributes: {
+          id: 1000 + iid,
+          iid,
+          title: "MR for renamed bot",
+          description,
+          state: "opened",
+          action: "open",
+          source_branch: "feat-renamed",
+          target_branch: "main",
+          url: `https://gitlab.com/group/project/-/merge_requests/${iid}`,
+          last_commit: null,
+          author_id: 1,
+        },
+      });
+    process.env.GITLAB_BOT_NAME = "renamed-bot";
+    try {
+      expect((await handleMergeRequest(mrEvent(63, "@renamed-bot review"))).created).toBe(true);
+      expect((await handleMergeRequest(mrEvent(64, "@agent-swarm-bot review"))).created).toBe(
+        false,
+      );
+    } finally {
+      delete process.env.GITLAB_BOT_NAME;
+    }
+  });
+
   test("ignores update action when no active task", async () => {
     const event = makeMREvent({
       object_attributes: {
         id: 105,
         iid: 62,
         title: "Updated MR",
-        description: `@${GITLAB_BOT_NAME} review`,
+        description: `@${getGitLabBotName()} review`,
         state: "opened",
         action: "update",
         source_branch: "feat-e",
@@ -329,7 +523,7 @@ describe("handleMergeRequest", () => {
         id: 106,
         iid: 70,
         title: "Dedup MR",
-        description: `@${GITLAB_BOT_NAME} review`,
+        description: `@${getGitLabBotName()} review`,
         state: "opened",
         action: "open",
         source_branch: "feat-dedup",
@@ -360,7 +554,7 @@ describe("handleIssue", () => {
         id: 201,
         iid: 20,
         title: "Issue with mention",
-        description: `@${GITLAB_BOT_NAME} implement this feature`,
+        description: `@${getGitLabBotName()} implement this feature`,
         state: "opened",
         action: "open",
         url: "https://gitlab.com/group/project/-/issues/20",
@@ -391,7 +585,7 @@ describe("handleIssue", () => {
         url: "https://gitlab.com/group/project/-/issues/21",
         author_id: 1,
       },
-      assignees: [{ id: 99, name: "Bot", username: GITLAB_BOT_NAME, avatar_url: "" }],
+      assignees: [{ id: 99, name: "Bot", username: getGitLabBotName(), avatar_url: "" }],
     });
 
     const result = await handleIssue(event);
@@ -444,9 +638,63 @@ 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();
   });
+
+  test("closing issue #6 fails the issue task, not the newer MR !6 task", async () => {
+    const issue = await handleIssue(
+      makeIssueEvent({
+        object_attributes: {
+          id: 206,
+          iid: 6,
+          title: "Issue six",
+          description: `@${getGitLabBotName()} fix this`,
+          state: "opened",
+          action: "open",
+          url: "https://gitlab.com/group/project/-/issues/6",
+          author_id: 1,
+        },
+      }),
+    );
+    // Created second, so it is the newest row for vcsNumber 6.
+    const mr = await handleMergeRequest(
+      makeMREvent({
+        object_attributes: {
+          id: 156,
+          iid: 6,
+          title: "MR six",
+          description: `@${getGitLabBotName()} please review`,
+          state: "opened",
+          action: "open",
+          source_branch: "feat-six",
+          target_branch: "main",
+          url: "https://gitlab.com/group/project/-/merge_requests/6",
+          last_commit: null,
+          author_id: 1,
+        },
+      }),
+    );
+    expect(issue.created && mr.created).toBe(true);
+
+    await handleIssue(
+      makeIssueEvent({
+        object_attributes: {
+          id: 206,
+          iid: 6,
+          title: "Issue six",
+          description: null,
+          state: "closed",
+          action: "close",
+          url: "https://gitlab.com/group/project/-/issues/6",
+          author_id: 1,
+        },
+      }),
+    );
+
+    expect((await getTaskById(issue.taskId!))?.failureReason).toBe("Issue was closed");
+    expect((await getTaskById(mr.taskId!))?.status).not.toBe("failed");
+  });
 });
 
 // ═══════════════════════════════════════════════════════
@@ -458,7 +706,7 @@ describe("handleNote", () => {
     const event = makeNoteEvent({
       object_attributes: {
         id: 301,
-        note: `@${GITLAB_BOT_NAME} please check this code`,
+        note: `@${getGitLabBotName()} please check this code`,
         noteable_type: "MergeRequest",
         noteable_id: 100,
         url: "https://gitlab.com/group/project/-/merge_requests/1#note_301",
@@ -493,7 +741,7 @@ describe("handleNote", () => {
     const event = makeNoteEvent({
       object_attributes: {
         id: 302,
-        note: `@${GITLAB_BOT_NAME} what do you think?`,
+        note: `@${getGitLabBotName()} what do you think?`,
         noteable_type: "Issue",
         noteable_id: 200,
         url: "https://gitlab.com/group/project/-/issues/40#note_302",
@@ -542,7 +790,7 @@ describe("handleNote", () => {
     const event = makeNoteEvent({
       object_attributes: {
         id: 304,
-        note: `@${GITLAB_BOT_NAME} review this commit`,
+        note: `@${getGitLabBotName()} review this commit`,
         noteable_type: "Commit",
         noteable_id: 999,
         url: "https://gitlab.com/group/project/-/commit/abc#note_304",
@@ -572,7 +820,7 @@ describe("handleNote", () => {
     const event = makeNoteEvent({
       object_attributes: {
         id: 305,
-        note: `@${GITLAB_BOT_NAME} also fix the linting`,
+        note: `@${getGitLabBotName()} also fix the linting`,
         noteable_type: "MergeRequest",
         noteable_id: 100,
         url: "https://gitlab.com/group/project/-/merge_requests/85#note_305",
@@ -742,7 +990,7 @@ describe("identity resolution — MR handler", () => {
         id: 901,
         iid: 901,
         title: "Known MR",
-        description: `@${GITLAB_BOT_NAME} review`,
+        description: `@${getGitLabBotName()} review`,
         state: "opened",
         action: "open",
         source_branch: "feat-known",
@@ -779,7 +1027,7 @@ describe("identity resolution — MR handler", () => {
         id: 902,
         iid: 902,
         title: "Inline-email MR",
-        description: `@${GITLAB_BOT_NAME} please`,
+        description: `@${getGitLabBotName()} please`,
         state: "opened",
         action: "open",
         source_branch: "feat-inline",
@@ -816,7 +1064,7 @@ describe("identity resolution — MR handler", () => {
         id: 903,
         iid: 903,
         title: "Ghost MR",
-        description: `@${GITLAB_BOT_NAME} ping`,
+        description: `@${getGitLabBotName()} ping`,
         state: "opened",
         action: "open",
         source_branch: "feat-ghost",
@@ -850,7 +1098,7 @@ describe("identity resolution — MR handler", () => {
           id: 910 + i,
           iid: 910 + i,
           title: `Ghost MR ${i}`,
-          description: `@${GITLAB_BOT_NAME} ping`,
+          description: `@${getGitLabBotName()} ping`,
           state: "opened",
           action: "open",
           source_branch: `feat-g-${i}`,
@@ -879,7 +1127,7 @@ describe("identity resolution — MR handler", () => {
         id: 920,
         iid: 920,
         title: "Empty email MR",
-        description: `@${GITLAB_BOT_NAME} review`,
+        description: `@${getGitLabBotName()} review`,
         state: "opened",
         action: "open",
         source_branch: "feat-empty",
@@ -914,7 +1162,7 @@ describe("identity resolution — Issue handler", () => {
         id: 950,
         iid: 950,
         title: "Ghost issue",
-        description: `@${GITLAB_BOT_NAME} fix`,
+        description: `@${getGitLabBotName()} fix`,
         state: "opened",
         action: "open",
         url: "https://gitlab.com/group/project/-/issues/950",
@@ -942,7 +1190,7 @@ describe("identity resolution — Note handler", () => {
   });
 
   test("unknown user WITHOUT email → unmapped entry tagged 'note'", async () => {
-    const noteBody = `@${GITLAB_BOT_NAME} can you look at this comment that is somewhat longer than usual to verify the slice(0,100) truncation downstream`;
+    const noteBody = `@${getGitLabBotName()} can you look at this comment that is somewhat longer than usual to verify the slice(0,100) truncation downstream`;
     const event = makeNoteEvent({
       user: { id: 31, name: "Note Ghost", username: "noteghost", avatar_url: "" },
       object_attributes: {
@@ -976,7 +1224,7 @@ describe("identity resolution — Note handler", () => {
     expect(meta).not.toBeNull();
     expect(meta?.sampleEventType).toBe("note");
     expect((meta?.sampleContext as string).length).toBeLessThanOrEqual(100);
-    expect((meta?.sampleContext as string).startsWith(`@${GITLAB_BOT_NAME}`)).toBe(true);
+    expect((meta?.sampleContext as string).startsWith(`@${getGitLabBotName()}`)).toBe(true);
     expect(await getUnmappedCount("noteghost")).toBe(1);
 
     // The dead-coded `_requestedByUserId` is now wired: undefined for an
@@ -996,7 +1244,7 @@ describe("identity resolution — Note handler", () => {
       user: { id: 32, name: "Known Commenter", username: "knowncommenter", avatar_url: "" },
       object_attributes: {
         id: 961,
-        note: `@${GITLAB_BOT_NAME} please take a look`,
+        note: `@${getGitLabBotName()} please take a look`,
         noteable_type: "MergeRequest",
         noteable_id: 101,
         url: "https://gitlab.com/group/project/-/merge_requests/2#note_961",

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/integration-disable-flags.test.ts

@@ -12,6 +12,8 @@
  */
 import { afterEach, beforeEach, describe, expect, test } from "bun:test";
 import { initAgentMail, isAgentMailEnabled, resetAgentMail } from "../agentmail/app";
+import { validateConfigValue } from "../be/swarm-config-guard";
+import { isBitbucketEnabled, resetBitbucket } from "../bitbucket/auth";
 import { initGitHub, isGitHubEnabled, resetGitHub } from "../github/app";
 import { isJiraEnabled, resetJira } from "../jira/app";
 import { isLinearEnabled, resetLinear } from "../linear/app";
@@ -28,6 +30,8 @@ const ENV_KEYS = [
   "LINEAR_ENABLED",
   "AGENTMAIL_DISABLE",
   "AGENTMAIL_WEBHOOK_SECRET",
+  "BITBUCKET_DISABLE",
+  "BITBUCKET_WEBHOOK_SECRET",
 ] as const;
 const originalEnv = new Map(ENV_KEYS.map((key) => [key, process.env[key]]));
 
@@ -45,6 +49,7 @@ afterEach(() => {
   resetAgentMail();
   resetJira();
   resetLinear();
+  resetBitbucket();
   for (const key of ENV_KEYS) {
     const value = originalEnv.get(key);
     if (value === undefined) delete process.env[key];
@@ -185,6 +190,37 @@ describe("AgentMail AGENTMAIL_DISABLE", () => {
   });
 });
 
+describe("Bitbucket BITBUCKET_DISABLE", () => {
+  test("every truthy spelling disables the integration", () => {
+    process.env.BITBUCKET_WEBHOOK_SECRET = "secret";
+    for (const spelling of TRUTHY_SPELLINGS) {
+      process.env.BITBUCKET_DISABLE = spelling;
+      expect(isBitbucketEnabled()).toBe(false);
+    }
+  });
+
+  test("non-disabling values keep the integration enabled", () => {
+    process.env.BITBUCKET_WEBHOOK_SECRET = "secret";
+    for (const value of NON_DISABLING) {
+      if (value === undefined) delete process.env.BITBUCKET_DISABLE;
+      else process.env.BITBUCKET_DISABLE = value;
+      expect(isBitbucketEnabled()).toBe(true);
+    }
+  });
+
+  test("stays disabled without a webhook secret", () => {
+    expect(isBitbucketEnabled()).toBe(false);
+  });
+
+  test("the config guard validates BITBUCKET_DISABLE as a boolean like GITLAB_DISABLE", () => {
+    for (const key of ["BITBUCKET_DISABLE", "GITLAB_DISABLE"]) {
+      expect(validateConfigValue(key, "true")).toBeNull();
+      expect(validateConfigValue(key, "false")).toBeNull();
+      expect(validateConfigValue(key, "maybe")).not.toBeNull();
+    }
+  });
+});
+
 describe("Slack SLACK_DISABLE", () => {
   test("every truthy spelling reports disabled", () => {
     for (const spelling of TRUTHY_SPELLINGS) {

package/src/tests/mcp-tools.test.ts

@@ -52,7 +52,7 @@ describe("script MCP tools", () => {
       "script-upsert":
         'Typecheck and persist a TypeScript script to the swarm catalog under your agent scope (or global if you\'re a lead). Import `ScriptContext` from "swarm-sdk" for a real context type. Other agents and workflow nodes will be able to find and run it. For local-only scripts, use code-mode `save`.',
       "script-delete":
-        "Remove a swarm-shared script from the catalog. Versions table preserves history.",
+        "Permanently remove a swarm-shared script and all its versions from the catalog. This cannot be undone.",
       "script-query-types":
         // Updated for the frozen SwarmToolResult contract refactor: `name` is
         // now optional — omit it to get the swarm-wide type surface instead

package/src/tests/onboarding.test.ts

@@ -34,6 +34,7 @@ const ENV_KEYS = [
   "EMBEDDING_MODEL",
   "GITHUB_TOKEN",
   "GITLAB_TOKEN",
+  "BITBUCKET_TOKEN",
   "SLACK_BOT_TOKEN",
   "SLACK_APP_TOKEN",
   "SLACK_MODE",
@@ -380,6 +381,17 @@ describe("onboarding state", () => {
     expect(body.state.steps.integrations).toMatchObject({ status: "done", method: "github" });
   });
 
+  test("GET derives the Bitbucket integration from BITBUCKET_TOKEN", async () => {
+    process.env.BITBUCKET_TOKEN = "example-bitbucket-token";
+    const body = (await (await request("/api/onboarding")).json()) as {
+      state: OnboardingState;
+      signals: { integrations: Record<string, boolean> };
+    };
+
+    expect(body.signals.integrations.bitbucket).toBe(true);
+    expect(body.state.steps.integrations).toMatchObject({ status: "done", method: "bitbucket" });
+  });
+
   test("the selected first task completes onboarding after that task completes", async () => {
     // Onboarding starts before the first task exists (the task alone would mark an existing install).
     await request("/api/onboarding");

package/src/tests/pi-codemode.test.ts

@@ -340,8 +340,14 @@ describe("bounded codemode", () => {
         undefined,
         fakeCtx().ctx,
       );
-      expect(Date.now() - started).toBeLessThan(5_000);
-      expect(textOf(result)).toContain("Codemode worker limit reached");
+      const elapsed = Date.now() - started;
+      expect(elapsed).toBeGreaterThanOrEqual(250);
+      expect(elapsed).toBeLessThan(5_000);
+      // pi 1.4 queues the script for a worker; our deadline ends the wait.
+      expect(textOf(result)).toContain(
+        "Script aborted: Codemode script exceeded its 300 ms deadline",
+      );
+      expect(textOf(result)).toContain("The script did not start.");
     } finally {
       holders.abort(new Error("test done"));
       await Promise.all(holding);

package/src/tests/preload-env.test.ts

@@ -0,0 +1,57 @@
+import { expect, test } from "bun:test";
+import { mkdtemp, rm } from "node:fs/promises";
+import { tmpdir } from "node:os";
+import { join } from "node:path";
+import { CHILD_PROCESS_TEST_BUDGET_MS, expectChildOk, runChild } from "./test-proc";
+
+test(
+  "preload clears inherited worker configuration before test files load",
+  async () => {
+    const directory = await mkdtemp(join(tmpdir(), "swarm-preload-env-"));
+    const fixture = join(directory, "worker-env.test.ts");
+    const keys = [
+      "SLACK_RENDER_V2",
+      "SLACK_STATUS_ICON_COMPLETED",
+      "SLACK_FUTURE_SETTING",
+      "COMB_ENABLED",
+      "APP_URL",
+      "AGENT_FS_API_URL",
+      "AGENT_FS_LIVE_URL",
+      "AGENT_FS_DEFAULT_ORG_ID",
+      "AGENT_FS_DEFAULT_DRIVE_ID",
+      "SWARM_ORG_NAME",
+      "SWARM_ORG_WEBSITE",
+    ];
+    try {
+      await Bun.write(
+        fixture,
+        `import { expect, test } from "bun:test";
+const inherited = ${JSON.stringify(keys)}.filter(key => process.env[key] !== undefined);
+test("worker defaults are cleared before module evaluation", () => {
+  expect(inherited).toEqual([]);
+  expect(process.env.PRELOAD_ENV_CONTROL).toBe("preserved");
+  process.env.SLACK_STATUS_ICON_COMPLETED = "test-icon";
+});
+test("suite configuration survives between tests", () => {
+  expect(process.env.SLACK_STATUS_ICON_COMPLETED).toBe("test-icon");
+});
+`,
+      );
+      const result = expectChildOk(
+        await runChild([process.execPath, "test", fixture], {
+          cwd: join(import.meta.dir, "../.."),
+          env: {
+            ...process.env,
+            ...Object.fromEntries(keys.map((key) => [key, "inherited-worker-setting"])),
+            PRELOAD_ENV_CONTROL: "preserved",
+          },
+        }),
+        "test runner with inherited worker env",
+      );
+      expect(result.stderr).toContain("2 pass");
+    } finally {
+      await rm(directory, { recursive: true, force: true });
+    }
+  },
+  CHILD_PROCESS_TEST_BUDGET_MS,
+);