test(slack/api): preserve real exports through mock.module stub

The previous mock.module factory only returned getApp + getSlackBotToken,
so when later test files in the same Bun process imported other exports
from packages/ims/slack/client (e.g. sendChannelMessage in
packages/core/tasks/scheduler.ts and packages/core/test/web-routes.test.ts),
Bun resolved them against the stub and threw:

  SyntaxError: Export named 'sendChannelMessage' not found in module
  '/home/runner/work/ode/ode/packages/ims/slack/client.ts'

Locally test ordering hid this; CI's alphabetic ordering surfaced it as
2 failed tests / 2 unhandled errors in fast-checks.

Fix: load the real ./client first and spread its exports into the stub
before overriding getApp + getSlackBotToken. Downstream importers in the
same process now still see every real export.

Author: Kai Liu

packages/ims/slack/api.test.ts

@@ -4,10 +4,20 @@ import { afterEach, beforeEach, describe, expect, it, mock } from "bun:test";
 // the single `./client` module so we don't need a real Slack connection,
 // and expose both the raw `apiCall` surface (used by the streaming helpers)
 // and the typed `chat.postMessage` surface (used by `postSlackQuestion`).
+//
+// IMPORTANT: `mock.module(...)` is process-wide in Bun, so the stub stays in
+// place for every later test file that loads `./client`. To avoid breaking
+// downstream tests that pull other exports (e.g. `sendChannelMessage` used by
+// `packages/core/tasks/scheduler.ts` and `packages/core/test/web-routes.test.ts`),
+// we start from the real module's exports and only override `getApp` /
+// `getSlackBotToken`. This way later imports keep finding every real export.
 const apiCalls: Array<{ method: string; args: Record<string, unknown> }> = [];
 const postMessageCalls: Array<Record<string, unknown>> = [];
 
+const realClient = await import("./client");
+
 mock.module("./client", () => ({
+  ...realClient,
   getApp: () => ({
     client: {
       apiCall: async (method: string, args: Record<string, unknown>) => {
@@ -22,8 +32,8 @@ mock.module("./client", () => ({
       },
     },
   }),
-  // `postSlackQuestion` does not call getSlackBotToken; we still export it
-  // so the module load doesn't fail when ./api re-imports it.
+  // Override so the test never accidentally picks up a stale workspace token
+  // during the test run.
   getSlackBotToken: () => "xoxb-test",
 }));