@vellumai/assistant 0.10.2-dev.202606250318.5e7cfb0 → 0.10.2

package/src/__tests__/non-member-access-request.test.ts

@@ -61,36 +61,6 @@ mock.module("../runtime/gateway-client.js", () => ({
   },
 }));
 
-// Guardian identity for the access request resolves via the gateway delivery
-// reader, not the local contacts DB. Seed it per-test via seedGatewayGuardian.
-interface GatewayGuardian {
-  channelType: string;
-  contactId: string;
-  principalId?: string | null;
-  displayName?: string | null;
-  address: string;
-  externalChatId?: string | null;
-  status: string;
-  verifiedAt?: number | null;
-}
-let gatewayGuardians: GatewayGuardian[] = [];
-mock.module("../contacts/guardian-delivery-reader.js", () => ({
-  getGuardianDelivery: async () => gatewayGuardians,
-  guardianForChannel: (list: GatewayGuardian[], channelType: string) =>
-    list.find((g) => g.channelType === channelType && g.status === "active"),
-}));
-
-function seedGatewayGuardian(g: Partial<GatewayGuardian> & {
-  channelType: string;
-  address: string;
-}): void {
-  gatewayGuardians.push({
-    contactId: `c-${g.channelType}`,
-    status: "active",
-    ...g,
-  });
-}
-
 import {
   listCanonicalGuardianDeliveries,
   listCanonicalGuardianRequests,
@@ -125,7 +95,6 @@ function resetState(): string {
   db.run("DELETE FROM contacts");
   emitSignalCalls.length = 0;
   deliverReplyCalls.length = 0;
-  gatewayGuardians = [];
   mockEmitResult = {
     signalId: "mock-signal-id",
     deduplicated: false,
@@ -133,15 +102,8 @@ function resetState(): string {
     reason: "mock",
     deliveryResults: [],
   };
-  // Seed the vellum anchor binding in the gateway list (gateway does this at
-  // startup in production). The DB write mirrors it for any local INFO reads.
+  // Seed the vellum anchor binding (gateway does this at startup in production)
   const principalId = `vellum-principal-${crypto.randomUUID()}`;
-  seedGatewayGuardian({
-    channelType: "vellum",
-    address: principalId,
-    principalId,
-    displayName: principalId,
-  });
   createGuardianBinding({
     channel: "vellum",
     guardianExternalUserId: principalId,
@@ -210,12 +172,6 @@ describe("non-member access request notification", () => {
 
   test("guardian is notified when a non-member messages and a guardian binding exists", async () => {
     // Set up a guardian binding for this channel using the anchor principal
-    seedGatewayGuardian({
-      channelType: "telegram",
-      address: "guardian-user-789",
-      externalChatId: "guardian-chat-789",
-      principalId: anchorPrincipalId,
-    });
     createGuardianBinding({
       channel: "telegram",
       guardianExternalUserId: "guardian-user-789",
@@ -261,12 +217,6 @@ describe("non-member access request notification", () => {
   });
 
   test("no duplicate approval requests for repeated messages from same non-member", async () => {
-    seedGatewayGuardian({
-      channelType: "telegram",
-      address: "guardian-user-789",
-      externalChatId: "guardian-chat-789",
-      principalId: anchorPrincipalId,
-    });
     createGuardianBinding({
       channel: "telegram",
       guardianExternalUserId: "guardian-user-789",
@@ -339,12 +289,6 @@ describe("non-member access request notification", () => {
     // Only a voice guardian binding exists — no Telegram binding.
     // Since cross-channel fallback was removed, the access request resolves
     // to the assistant's vellum anchor identity instead.
-    seedGatewayGuardian({
-      channelType: "phone",
-      address: "guardian-voice-user",
-      externalChatId: "guardian-voice-chat",
-      principalId: anchorPrincipalId,
-    });
     createGuardianBinding({
       channel: "phone",
       guardianExternalUserId: "guardian-voice-user",
@@ -407,8 +351,8 @@ describe("access-request-helper unit tests", () => {
     anchorPrincipalId = resetState();
   });
 
-  test("notifyGuardianOfAccessRequest returns no_sender_id when actorExternalId is absent", async () => {
-    const result = await notifyGuardianOfAccessRequest({
+  test("notifyGuardianOfAccessRequest returns no_sender_id when actorExternalId is absent", () => {
+    const result = notifyGuardianOfAccessRequest({
       canonicalAssistantId: "self",
       sourceChannel: "telegram",
       conversationExternalId: "chat-123",
@@ -428,8 +372,8 @@ describe("access-request-helper unit tests", () => {
     expect(pending.length).toBe(0);
   });
 
-  test("notifyGuardianOfAccessRequest creates request with self-healed principal when no binding exists", async () => {
-    const result = await notifyGuardianOfAccessRequest({
+  test("notifyGuardianOfAccessRequest creates request with self-healed principal when no binding exists", () => {
+    const result = notifyGuardianOfAccessRequest({
       canonicalAssistantId: "self",
       sourceChannel: "telegram",
       conversationExternalId: "chat-123",
@@ -456,14 +400,8 @@ describe("access-request-helper unit tests", () => {
     expect(emitSignalCalls.length).toBe(1);
   });
 
-  test("notifyGuardianOfAccessRequest falls back to assistant-anchored vellum identity when source-channel binding is missing", async () => {
+  test("notifyGuardianOfAccessRequest falls back to assistant-anchored vellum identity when source-channel binding is missing", () => {
     // Only voice binding exists
-    seedGatewayGuardian({
-      channelType: "phone",
-      address: "guardian-voice",
-      externalChatId: "voice-chat",
-      principalId: "test-principal-id",
-    });
     createGuardianBinding({
       channel: "phone",
       guardianExternalUserId: "guardian-voice",
@@ -472,7 +410,7 @@ describe("access-request-helper unit tests", () => {
       verifiedVia: "test",
     });
 
-    const result = await notifyGuardianOfAccessRequest({
+    const result = notifyGuardianOfAccessRequest({
       canonicalAssistantId: "self",
       sourceChannel: "telegram",
       conversationExternalId: "tg-chat",
@@ -497,20 +435,8 @@ describe("access-request-helper unit tests", () => {
     expect(payload.guardianBindingChannel).toBe("vellum");
   });
 
-  test("notifyGuardianOfAccessRequest prefers source-channel binding over vellum anchor", async () => {
+  test("notifyGuardianOfAccessRequest prefers source-channel binding over vellum anchor", () => {
     // Both Telegram and voice bindings exist with the anchor principal
-    seedGatewayGuardian({
-      channelType: "telegram",
-      address: "guardian-tg",
-      externalChatId: "tg-chat",
-      principalId: anchorPrincipalId,
-    });
-    seedGatewayGuardian({
-      channelType: "phone",
-      address: "guardian-voice",
-      externalChatId: "voice-chat",
-      principalId: anchorPrincipalId,
-    });
     createGuardianBinding({
       channel: "telegram",
       guardianExternalUserId: "guardian-tg",
@@ -526,7 +452,7 @@ describe("access-request-helper unit tests", () => {
       verifiedVia: "test",
     });
 
-    const result = await notifyGuardianOfAccessRequest({
+    const result = notifyGuardianOfAccessRequest({
       canonicalAssistantId: "self",
       sourceChannel: "telegram",
       conversationExternalId: "chat-123",
@@ -551,100 +477,8 @@ describe("access-request-helper unit tests", () => {
     expect(payload.guardianBindingChannel).toBe("telegram");
   });
 
-  test("notifyGuardianOfAccessRequest falls back to local source-channel binding when gateway delivery is empty", async () => {
-    // Gateway delivery read yields nothing (restart/timeout/malformed IPC).
-    gatewayGuardians = [];
-    // Local dual-written mirror still has the vellum anchor + telegram binding.
-    createGuardianBinding({
-      channel: "telegram",
-      guardianExternalUserId: "guardian-tg",
-      guardianDeliveryChatId: "tg-chat",
-      guardianPrincipalId: anchorPrincipalId,
-      verifiedVia: "test",
-    });
-
-    const result = await notifyGuardianOfAccessRequest({
-      canonicalAssistantId: "self",
-      sourceChannel: "telegram",
-      conversationExternalId: "chat-123",
-      actorExternalId: "unknown-user",
-    });
-
-    expect(result.notified).toBe(true);
-
-    const pending = listCanonicalGuardianRequests({
-      status: "pending",
-      requesterExternalUserId: "unknown-user",
-      kind: "access_request",
-    });
-    expect(pending.length).toBe(1);
-    // Request is decidable: local read supplied the principal + source binding.
-    expect(pending[0].guardianPrincipalId).toBe(anchorPrincipalId);
-    expect(pending[0].guardianExternalUserId).toBe("guardian-tg");
-
-    const payload = emitSignalCalls[0].contextPayload as Record<
-      string,
-      unknown
-    >;
-    expect(payload.guardianBindingChannel).toBe("telegram");
-  });
-
-  test("notifyGuardianOfAccessRequest falls back to local vellum anchor when gateway delivery is empty", async () => {
-    // Gateway empty; only the local vellum anchor from resetState remains.
-    gatewayGuardians = [];
-
-    const result = await notifyGuardianOfAccessRequest({
-      canonicalAssistantId: "self",
-      sourceChannel: "telegram",
-      conversationExternalId: "chat-123",
-      actorExternalId: "unknown-user",
-    });
-
-    expect(result.notified).toBe(true);
-
-    const pending = listCanonicalGuardianRequests({
-      status: "pending",
-      requesterExternalUserId: "unknown-user",
-      kind: "access_request",
-    });
-    expect(pending.length).toBe(1);
-    // Decidable via the local vellum anchor principal.
-    expect(pending[0].guardianPrincipalId).toBe(anchorPrincipalId);
-
-    const payload = emitSignalCalls[0].contextPayload as Record<
-      string,
-      unknown
-    >;
-    expect(payload.guardianBindingChannel).toBe("vellum");
-  });
-
-  test("notifyGuardianOfAccessRequest does not create a decisionable request when both gateway and local reads are empty", async () => {
-    // Genuinely unbound assistant: gateway empty AND no local binding. Prior
-    // behavior rejects creation of a decisionable request without a principal.
-    gatewayGuardians = [];
-    const db = getDb();
-    db.run("DELETE FROM contact_channels");
-    db.run("DELETE FROM contacts");
-
-    await expect(
-      notifyGuardianOfAccessRequest({
-        canonicalAssistantId: "self",
-        sourceChannel: "telegram",
-        conversationExternalId: "chat-123",
-        actorExternalId: "unknown-user",
-      }),
-    ).rejects.toThrow();
-
-    const pending = listCanonicalGuardianRequests({
-      status: "pending",
-      requesterExternalUserId: "unknown-user",
-      kind: "access_request",
-    });
-    expect(pending.length).toBe(0);
-  });
-
-  test("notifyGuardianOfAccessRequest for voice channel includes actorDisplayName in contextPayload", async () => {
-    const result = await notifyGuardianOfAccessRequest({
+  test("notifyGuardianOfAccessRequest for voice channel includes actorDisplayName in contextPayload", () => {
+    const result = notifyGuardianOfAccessRequest({
       canonicalAssistantId: "self",
       sourceChannel: "phone",
       conversationExternalId: "+15559998888",
@@ -674,8 +508,8 @@ describe("access-request-helper unit tests", () => {
     expect(pending.length).toBe(1);
   });
 
-  test("notifyGuardianOfAccessRequest includes requestCode in contextPayload", async () => {
-    const result = await notifyGuardianOfAccessRequest({
+  test("notifyGuardianOfAccessRequest includes requestCode in contextPayload", () => {
+    const result = notifyGuardianOfAccessRequest({
       canonicalAssistantId: "self",
       sourceChannel: "telegram",
       conversationExternalId: "chat-123",
@@ -695,8 +529,8 @@ describe("access-request-helper unit tests", () => {
     expect((payload.requestCode as string).length).toBe(6);
   });
 
-  test("notifyGuardianOfAccessRequest includes previousMemberStatus in contextPayload", async () => {
-    const result = await notifyGuardianOfAccessRequest({
+  test("notifyGuardianOfAccessRequest includes previousMemberStatus in contextPayload", () => {
+    const result = notifyGuardianOfAccessRequest({
       canonicalAssistantId: "self",
       sourceChannel: "telegram",
       conversationExternalId: "chat-123",
@@ -736,7 +570,7 @@ describe("access-request-helper unit tests", () => {
       ],
     };
 
-    const result = await notifyGuardianOfAccessRequest({
+    const result = notifyGuardianOfAccessRequest({
       canonicalAssistantId: "self",
       sourceChannel: "phone",
       conversationExternalId: "+15556667777",
@@ -769,12 +603,6 @@ describe("access-request-helper unit tests", () => {
     // Set up a telegram guardian binding with the anchor principal so
     // guardianResolutionSource resolves to "source-channel-contact" and
     // sameChannelOnly is true.
-    seedGatewayGuardian({
-      channelType: "telegram",
-      address: "guardian-user-456",
-      externalChatId: "guardian-chat-456",
-      principalId: anchorPrincipalId,
-    });
     createGuardianBinding({
       channel: "telegram",
       guardianExternalUserId: "guardian-user-456",
@@ -797,7 +625,7 @@ describe("access-request-helper unit tests", () => {
       ],
     };
 
-    const result = await notifyGuardianOfAccessRequest({
+    const result = notifyGuardianOfAccessRequest({
       canonicalAssistantId: "self",
       sourceChannel: "telegram",
       conversationExternalId: "chat-123",

package/src/__tests__/notification-broadcaster.test.ts

@@ -24,10 +24,6 @@ mock.module("../util/logger.js", () => ({
     }),
 }));
 
-mock.module("../contacts/guardian-delivery-reader.js", () => ({
-  getGuardianDelivery: async () => null,
-}));
-
 // Mock destination-resolver to return a destination for every requested channel.
 // External channels (telegram, slack) include bindingContext.
 mock.module("../notifications/destination-resolver.js", () => ({

package/src/__tests__/notification-decision-recipient-context.test.ts

@@ -36,36 +36,17 @@ mock.module("../prompts/system-prompt.js", () => ({
   buildCoreIdentityContext: () => null,
 }));
 
-// ── Guardian binding + contact notes mocks ───────────────────────────
+// ── Guardian contact mock ────────────────────────────────────────────
 
-// Guardian binding (ACL) is resolved via the gateway pull; notes (INFO) are
-// joined locally by contactId. Tests drive both via mutable slots.
-let guardianDeliveryFixture: Array<{ contactId: string }> = [];
-let contactInfoFixture: Record<string, { notes: string | null } | null> = {};
-
-mock.module("../contacts/guardian-delivery-reader.js", () => ({
-  getGuardianDelivery: async () => guardianDeliveryFixture,
-  anyGuardian: (list: Array<{ contactId: string }>) => list[0],
-}));
+let mockGuardianResult: {
+  contact: { notes: string | null };
+  channels: Record<string, unknown>[];
+} | null = null;
 
 mock.module("../contacts/contact-store.js", () => ({
-  findContactInfoById: (contactId: string) =>
-    contactInfoFixture[contactId] ?? null,
+  listGuardianChannels: () => mockGuardianResult,
 }));
 
-const GUARDIAN_CONTACT_ID = "guardian-contact-1";
-
-/** Bind a guardian with the given notes (or no guardian when notes is null). */
-function setGuardian(notes: string | null | undefined): void {
-  if (notes === undefined) {
-    guardianDeliveryFixture = [];
-    contactInfoFixture = {};
-    return;
-  }
-  guardianDeliveryFixture = [{ contactId: GUARDIAN_CONTACT_ID }];
-  contactInfoFixture = { [GUARDIAN_CONTACT_ID]: { notes } };
-}
-
 // ── Provider mock with system prompt capture ──────────────────────────
 
 let configuredProvider: {
@@ -153,12 +134,15 @@ describe("recipient context in notification decision engine", () => {
   beforeEach(() => {
     configuredProvider = null;
     extractedToolUse = null;
-    setGuardian(undefined);
+    mockGuardianResult = null;
     capturedSystemPrompt = undefined;
   });
 
   test("guardian contact notes appear in system prompt as <recipient-context>", async () => {
-    setGuardian("Prefers formal tone. Address as Dr. Smith.");
+    mockGuardianResult = {
+      contact: { notes: "Prefers formal tone. Address as Dr. Smith." },
+      channels: [{ type: "vellum" }],
+    };
     setupLLMProvider();
 
     const signal = makeSignal();
@@ -173,7 +157,7 @@ describe("recipient context in notification decision engine", () => {
   });
 
   test("recipient-context is omitted when no guardian exists", async () => {
-    setGuardian(undefined);
+    mockGuardianResult = null;
     setupLLMProvider();
 
     const signal = makeSignal();
@@ -185,7 +169,10 @@ describe("recipient context in notification decision engine", () => {
   });
 
   test("recipient-context is omitted when guardian notes are null", async () => {
-    setGuardian(null);
+    mockGuardianResult = {
+      contact: { notes: null },
+      channels: [{ type: "vellum" }],
+    };
     setupLLMProvider();
 
     const signal = makeSignal();
@@ -197,7 +184,10 @@ describe("recipient context in notification decision engine", () => {
   });
 
   test("recipient-context is omitted when guardian notes are empty string", async () => {
-    setGuardian("");
+    mockGuardianResult = {
+      contact: { notes: "" },
+      channels: [{ type: "vellum" }],
+    };
     setupLLMProvider();
 
     const signal = makeSignal();
@@ -209,7 +199,10 @@ describe("recipient context in notification decision engine", () => {
   });
 
   test("large guardian notes are truncated to prevent oversized prompts", async () => {
-    setGuardian("N".repeat(3000));
+    mockGuardianResult = {
+      contact: { notes: "N".repeat(3000) },
+      channels: [{ type: "vellum" }],
+    };
     setupLLMProvider();
 
     const signal = makeSignal();
@@ -233,7 +226,10 @@ describe("recipient context in notification decision engine", () => {
   });
 
   test("fallback path works correctly without recipient context", async () => {
-    setGuardian("Prefers formal tone.");
+    mockGuardianResult = {
+      contact: { notes: "Prefers formal tone." },
+      channels: [{ type: "vellum" }],
+    };
     // null provider forces fallback path
     configuredProvider = null;
 
@@ -251,7 +247,10 @@ describe("recipient context in notification decision engine", () => {
   });
 
   test("recipient-context appears after user-preferences in prompt", async () => {
-    setGuardian("Prefers brief updates.");
+    mockGuardianResult = {
+      contact: { notes: "Prefers brief updates." },
+      channels: [{ type: "vellum" }],
+    };
     setupLLMProvider();
 
     const signal = makeSignal();

package/src/__tests__/notification-deep-link.test.ts

@@ -20,10 +20,6 @@ mock.module("../util/logger.js", () => ({
     }),
 }));
 
-mock.module("../contacts/guardian-delivery-reader.js", () => ({
-  getGuardianDelivery: async () => null,
-}));
-
 // Mock destination-resolver for broadcaster tests
 mock.module("../notifications/destination-resolver.js", () => ({
   resolveDestinations: (channels: string[]) => {
@@ -46,8 +42,6 @@ mock.module("../notifications/deliveries-store.js", () => ({
 // can be driven from tests without DB access.
 let mockExistingConversations: Record<string, { id: string }> = {};
 mock.module("../memory/conversation-crud.js", () => ({
-    setConversationProcessingStartedAt: () => {},
-    isConversationProcessing: () => false,
   getConversation: (id: string) => mockExistingConversations[id] ?? null,
 }));
 

package/src/__tests__/notification-guardian-path.test.ts

@@ -68,25 +68,6 @@ mock.module("../notifications/emit-signal.js", () => ({
   },
 }));
 
-// Guardian principalId is resolved from the gateway binding reader. Mirror the
-// seeded vellum binding so dispatch can resolve the principal.
-mock.module("../contacts/guardian-delivery-reader.js", () => ({
-  getGuardianDelivery: async () => [
-    {
-      channelType: "vellum",
-      contactId: "guardian-vellum",
-      principalId: "test-principal-id",
-      address: "local",
-      status: "active",
-    },
-  ],
-  guardianForChannel: (
-    list: Array<{ channelType: string; status: string }>,
-    channelType: string,
-  ) => list.find((g) => g.channelType === channelType && g.status === "active"),
-  anyGuardian: (list: unknown[]) => list[0],
-}));
-
 import {
   createCallSession,
   createPendingQuestion,

package/src/__tests__/openai-provider.test.ts

@@ -58,16 +58,6 @@ let lastConstructorOptions: Record<string, unknown> | null = null;
 let shouldThrow: Error | null = null;
 const DEFAULT_SDK_TIMEOUT_MS = 1_860_000;
 
-// Each provider installs a `fetch` wrapper to capture raw error bodies, so the
-// constructor options carry a `fetch` function; assert the meaningful options
-// via objectContaining and confirm `fetch` is wired.
-function expectOpenAIConstructorOptions(
-  expected: Record<string, unknown>,
-): void {
-  expect(lastConstructorOptions).toEqual(expect.objectContaining(expected));
-  expect(typeof lastConstructorOptions?.fetch).toBe("function");
-}
-
 function userMsg(text: string): Message {
   return { role: "user", content: [{ type: "text", text }] };
 }
@@ -319,7 +309,7 @@ describe("OpenAIProvider", () => {
     });
 
     expect(compatible.name).toBe("ollama");
-    expectOpenAIConstructorOptions({
+    expect(lastConstructorOptions).toEqual({
       apiKey: "sk-local",
       baseURL: "http://127.0.0.1:11434/v1",
       timeout: DEFAULT_SDK_TIMEOUT_MS,
@@ -332,7 +322,7 @@ describe("OpenAIProvider", () => {
       delete process.env.OLLAMA_BASE_URL;
       const ollama = new OllamaProvider("llama3.2");
       expect(ollama.name).toBe("ollama");
-      expectOpenAIConstructorOptions({
+      expect(lastConstructorOptions).toEqual({
         apiKey: "ollama",
         baseURL: "http://127.0.0.1:11434/v1",
         timeout: DEFAULT_SDK_TIMEOUT_MS,
@@ -352,7 +342,7 @@ describe("OpenAIProvider", () => {
       process.env.OLLAMA_BASE_URL = "   ";
       const ollama = new OllamaProvider("llama3.2");
       expect(ollama.name).toBe("ollama");
-      expectOpenAIConstructorOptions({
+      expect(lastConstructorOptions).toEqual({
         apiKey: "ollama",
         baseURL: "http://127.0.0.1:11434/v1",
         timeout: DEFAULT_SDK_TIMEOUT_MS,
@@ -1270,7 +1260,7 @@ describe("custom baseURL initialization", () => {
     });
 
     expect(managed.name).toBe("openai");
-    expectOpenAIConstructorOptions({
+    expect(lastConstructorOptions).toEqual({
       apiKey: "ast-key-123",
       baseURL: "https://platform.example.com/v1/runtime-proxy/openai",
       timeout: DEFAULT_SDK_TIMEOUT_MS,
@@ -1280,7 +1270,7 @@ describe("custom baseURL initialization", () => {
   test("OpenAIProvider without baseURL calls provider directly", () => {
     new OpenAIProvider("sk-user-key", "gpt-4o");
 
-    expectOpenAIConstructorOptions({
+    expect(lastConstructorOptions).toEqual({
       apiKey: "sk-user-key",
       baseURL: undefined,
       timeout: DEFAULT_SDK_TIMEOUT_MS,
@@ -1292,7 +1282,7 @@ describe("custom baseURL initialization", () => {
       streamTimeoutMs: 300_000,
     });
 
-    expectOpenAIConstructorOptions({
+    expect(lastConstructorOptions).toEqual({
       apiKey: "sk-user-key",
       baseURL: undefined,
       timeout: 360_000,
@@ -1309,7 +1299,7 @@ describe("custom baseURL initialization", () => {
     );
 
     expect(managed.name).toBe("fireworks");
-    expectOpenAIConstructorOptions({
+    expect(lastConstructorOptions).toEqual({
       apiKey: "ast-key-123",
       baseURL: "https://platform.example.com/v1/runtime-proxy/fireworks",
       timeout: DEFAULT_SDK_TIMEOUT_MS,
@@ -1322,7 +1312,7 @@ describe("custom baseURL initialization", () => {
       "accounts/fireworks/models/llama-v3p1-70b-instruct",
     );
 
-    expectOpenAIConstructorOptions({
+    expect(lastConstructorOptions).toEqual({
       apiKey: "fw-user-key",
       baseURL: "https://api.fireworks.ai/inference/v1",
       timeout: DEFAULT_SDK_TIMEOUT_MS,
@@ -1335,7 +1325,7 @@ describe("custom baseURL initialization", () => {
     });
 
     expect(managed.name).toBe("openrouter");
-    expectOpenAIConstructorOptions({
+    expect(lastConstructorOptions).toEqual({
       apiKey: "ast-key-123",
       baseURL: "https://platform.example.com/v1/runtime-proxy/openrouter",
       timeout: DEFAULT_SDK_TIMEOUT_MS,
@@ -1345,7 +1335,7 @@ describe("custom baseURL initialization", () => {
   test("OpenRouterProvider without custom baseURL uses default OpenRouter URL", () => {
     new OpenRouterProvider("or-user-key", "openai/gpt-4o");
 
-    expectOpenAIConstructorOptions({
+    expect(lastConstructorOptions).toEqual({
       apiKey: "or-user-key",
       baseURL: "https://openrouter.ai/api/v1",
       timeout: DEFAULT_SDK_TIMEOUT_MS,
@@ -1358,7 +1348,7 @@ describe("custom baseURL initialization", () => {
     });
 
     expect(managed.name).toBe("minimax");
-    expectOpenAIConstructorOptions({
+    expect(lastConstructorOptions).toEqual({
       apiKey: "ast-key-123",
       baseURL: "https://platform.example.com/v1/runtime-proxy/minimax",
       timeout: DEFAULT_SDK_TIMEOUT_MS,
@@ -1368,7 +1358,7 @@ describe("custom baseURL initialization", () => {
   test("MinimaxProvider without custom baseURL uses default MiniMax URL", () => {
     new MinimaxProvider("mm-user-key", "MiniMax-M2.7");
 
-    expectOpenAIConstructorOptions({
+    expect(lastConstructorOptions).toEqual({
       apiKey: "mm-user-key",
       baseURL: "https://api.minimax.io/v1",
       timeout: DEFAULT_SDK_TIMEOUT_MS,

package/src/__tests__/openai-responses-provider.test.ts

@@ -24,16 +24,6 @@ let lastConstructorOptions: Record<string, unknown> | null = null;
 let shouldThrow: Error | null = null;
 const DEFAULT_SDK_TIMEOUT_MS = 1_860_000;
 
-// Each provider installs a `fetch` wrapper to capture raw error bodies, so the
-// constructor options carry a `fetch` function; assert the meaningful options
-// via objectContaining and confirm `fetch` is wired.
-function expectOpenAIConstructorOptions(
-  expected: Record<string, unknown>,
-): void {
-  expect(lastConstructorOptions).toEqual(expect.objectContaining(expected));
-  expect(typeof lastConstructorOptions?.fetch).toBe("function");
-}
-
 // Simulate OpenAI.APIError
 class FakeAPIError extends Error {
   status: number;
@@ -228,7 +218,7 @@ describe("OpenAIResponsesProvider", () => {
       providerLabel: "Managed OpenAI",
     });
 
-    expectOpenAIConstructorOptions({
+    expect(lastConstructorOptions).toEqual({
       apiKey: "sk-custom",
       baseURL: "https://proxy.example.com/v1",
       timeout: DEFAULT_SDK_TIMEOUT_MS,
@@ -240,7 +230,7 @@ describe("OpenAIResponsesProvider", () => {
       streamTimeoutMs: 300_000,
     });
 
-    expectOpenAIConstructorOptions({
+    expect(lastConstructorOptions).toEqual({
       apiKey: "sk-custom",
       baseURL: undefined,
       timeout: 360_000,

package/src/__tests__/outbound-slack-persistence.test.ts

@@ -72,8 +72,6 @@ const persistedRows: Array<{
   metadata: string | null;
 }> = [];
 mock.module("../memory/conversation-crud.js", () => ({
-    setConversationProcessingStartedAt: () => {},
-    isConversationProcessing: () => false,
   addMessage: (
     conversationId: string,
     role: string,