@vellumai/assistant 0.10.8 → 0.10.9-dev.202607152225.ec6fdbf

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

@@ -17,13 +17,6 @@ import type { PairingOptions } from "../notifications/conversation-pairing.js";
 
 // -- Mocks (must be declared before importing modules that depend on them) ----
 
-mock.module("../util/logger.js", () => ({
-  getLogger: () =>
-    new Proxy({} as Record<string, unknown>, {
-      get: () => () => {},
-    }),
-}));
-
 mock.module("../contacts/guardian-delivery-reader.js", () => ({
   getGuardianDelivery: async () => null,
 }));
@@ -63,8 +56,8 @@ mock.module("../notifications/deliveries-store.js", () => ({
 // Captures call arguments so tests can inspect what was passed in.
 // Set `nextPairingResult` to override the return value for a single call.
 let nextPairingResult:
-  | import("../notifications/conversation-pairing.js").PairingResult
-  | null = null;
+  import("../notifications/conversation-pairing.js").PairingResult | null =
+  null;
 let pairingCallCount = 0;
 
 interface PairingCall {
@@ -346,9 +339,9 @@ describe("notification broadcaster", () => {
     const vellumAdapter = new MockAdapter("vellum");
     const broadcaster = new NotificationBroadcaster([vellumAdapter]);
     const conversationCreatedCalls: ConversationCreatedInfo[] = [];
-    broadcaster.setOnConversationCreated((info) =>
-      conversationCreatedCalls.push(info),
-    );
+    broadcaster.setOnConversationCreated((info) => {
+      conversationCreatedCalls.push(info);
+    });
 
     const signal = makeSignal();
     // No conversationActions means default start_new behavior
@@ -370,7 +363,9 @@ describe("notification broadcaster", () => {
     const decision = makeDecision();
 
     await broadcaster.broadcastDecision(signal, decision, {
-      onConversationCreated: (info) => dispatchCalls.push(info),
+      onConversationCreated: (info) => {
+        dispatchCalls.push(info);
+      },
     });
 
     expect(dispatchCalls).toHaveLength(1);
@@ -381,7 +376,9 @@ describe("notification broadcaster", () => {
     const broadcaster = new NotificationBroadcaster([vellumAdapter]);
     const eventCalls: ConversationCreatedInfo[] = [];
     const dispatchCalls: ConversationCreatedInfo[] = [];
-    broadcaster.setOnConversationCreated((info) => eventCalls.push(info));
+    broadcaster.setOnConversationCreated((info) => {
+      eventCalls.push(info);
+    });
 
     // Simulate a successful reuse by injecting a pairing result with
     // createdNewConversation=false. This bypasses the real conversation
@@ -406,7 +403,9 @@ describe("notification broadcaster", () => {
     });
 
     await broadcaster.broadcastDecision(signal, decision, {
-      onConversationCreated: (info) => dispatchCalls.push(info),
+      onConversationCreated: (info) => {
+        dispatchCalls.push(info);
+      },
     });
 
     // The class-level event callback should NOT fire because
@@ -496,7 +495,9 @@ describe("notification broadcaster", () => {
     const vellumAdapter = new MockAdapter("vellum");
     const broadcaster = new NotificationBroadcaster([vellumAdapter]);
     const eventCalls: ConversationCreatedInfo[] = [];
-    broadcaster.setOnConversationCreated((info) => eventCalls.push(info));
+    broadcaster.setOnConversationCreated((info) => {
+      eventCalls.push(info);
+    });
 
     // Simulate binding-key continuation: pairing reuses an existing bound
     // conversation (createdNewConversation=false, strategy=continue_existing_conversation)
@@ -523,7 +524,9 @@ describe("notification broadcaster", () => {
     const vellumAdapter = new MockAdapter("vellum");
     const broadcaster = new NotificationBroadcaster([vellumAdapter]);
     const eventCalls: ConversationCreatedInfo[] = [];
-    broadcaster.setOnConversationCreated((info) => eventCalls.push(info));
+    broadcaster.setOnConversationCreated((info) => {
+      eventCalls.push(info);
+    });
 
     // First delivery to a new destination: creates a fresh conversation but
     // the strategy is continue_existing_conversation (not start_new_conversation),
@@ -565,7 +568,9 @@ describe("notification broadcaster", () => {
     const decision = makeDecision();
 
     await broadcaster.broadcastDecision(signal, decision, {
-      onConversationCreated: (info) => dispatchCalls.push(info),
+      onConversationCreated: (info) => {
+        dispatchCalls.push(info);
+      },
     });
 
     // The per-dispatch callback SHOULD fire regardless of reuse
@@ -594,7 +599,9 @@ describe("notification broadcaster", () => {
     const vellumAdapter = new MockAdapter("vellum");
     const broadcaster = new NotificationBroadcaster([vellumAdapter]);
     const createdCalls: ConversationCreatedInfo[] = [];
-    broadcaster.setOnConversationCreated((info) => createdCalls.push(info));
+    broadcaster.setOnConversationCreated((info) => {
+      createdCalls.push(info);
+    });
 
     const signal = makeSignal({
       sourceEventName: "schedule.notify",
@@ -618,7 +625,9 @@ describe("notification broadcaster", () => {
     const vellumAdapter = new MockAdapter("vellum");
     const broadcaster = new NotificationBroadcaster([vellumAdapter]);
     const createdCalls: ConversationCreatedInfo[] = [];
-    broadcaster.setOnConversationCreated((info) => createdCalls.push(info));
+    broadcaster.setOnConversationCreated((info) => {
+      createdCalls.push(info);
+    });
 
     const signal = makeSignal(); // no conversationMetadata
     const decision = makeDecision();
@@ -645,7 +654,9 @@ describe("notification broadcaster", () => {
     const decision = makeDecision();
 
     await broadcaster.broadcastDecision(signal, decision, {
-      onConversationCreated: (info) => dispatchCalls.push(info),
+      onConversationCreated: (info) => {
+        dispatchCalls.push(info);
+      },
     });
 
     expect(dispatchCalls).toHaveLength(1);

package/src/__tests__/notification-candidate-guardian-context.test.ts

@@ -3,7 +3,7 @@
  * candidates.
  *
  * `buildConversationCandidates` counts each candidate's pending guardian
- * requests through the canonical store's conversation-scope query, which
+ * requests through the gateway client's conversation-scope query, which
  * matches both a request's source conversation and any conversation its card
  * was delivered to. These tests lock that wiring — in particular that a
  * request delivered to a conversation different from its synthetic source is
@@ -12,23 +12,23 @@
 
 import { beforeEach, describe, expect, mock, test } from "bun:test";
 
-mock.module("../util/logger.js", () => ({
-  getLogger: () =>
-    new Proxy({} as Record<string, unknown>, {
-      get: () => () => {},
-    }),
-}));
-
+// The recorder and candidate builder go through the gateway client; serve
+// that surface from the in-memory sim the tests seed.
 import {
-  createCanonicalGuardianDelivery,
-  createCanonicalGuardianRequest,
-  resolveCanonicalGuardianRequest,
-} from "../contacts/canonical-guardian-store.js";
-import { recordGuardianRequestDeliveries } from "../notifications/canonical-delivery-recorder.js";
+  bridgeState,
+  gatewayGuardianRequestsStoreBridge,
+} from "./helpers/gateway-guardian-requests-store-bridge.js";
+
+mock.module(
+  "../channels/gateway-guardian-requests.js",
+  () => gatewayGuardianRequestsStoreBridge,
+);
+
 import { buildConversationCandidates } from "../notifications/conversation-candidates.js";
 import { createDecision } from "../notifications/decisions-store.js";
 import { createDelivery } from "../notifications/deliveries-store.js";
 import { createEvent } from "../notifications/events-store.js";
+import { recordGuardianRequestDeliveries } from "../notifications/guardian-delivery-recorder.js";
 import type { NotificationChannel } from "../notifications/types.js";
 import { getDb } from "../persistence/db-connection.js";
 import { initializeDb } from "../persistence/db-init.js";
@@ -41,8 +41,7 @@ const CHANNEL = "vellum" as NotificationChannel;
 
 function resetTables(): void {
   const db = getDb();
-  db.run("DELETE FROM canonical_guardian_deliveries");
-  db.run("DELETE FROM canonical_guardian_requests");
+  bridgeState.reset();
   db.run("DELETE FROM notification_deliveries");
   db.run("DELETE FROM notification_decisions");
   db.run("DELETE FROM notification_events");
@@ -104,11 +103,11 @@ function seedCandidateConversation(
   });
 }
 
-function candidateFor(
+async function candidateFor(
   conversationId: string,
   channel: NotificationChannel = CHANNEL,
 ) {
-  const set = buildConversationCandidates([channel]);
+  const set = await buildConversationCandidates([channel]);
   return set[channel]?.find((c) => c.conversationId === conversationId);
 }
 
@@ -117,57 +116,60 @@ describe("buildConversationCandidates guardian enrichment", () => {
     resetTables();
   });
 
-  test("counts pending requests via both source and delivery conversation scope", () => {
+  test("counts pending requests via both source and delivery conversation scope", async () => {
     const convId = "conv-guardian";
     seedCandidateConversation(convId);
 
     // (1) pending request whose SOURCE conversation is the candidate.
-    createCanonicalGuardianRequest({
+    bridgeState.seedRequest({
       kind: "tool_approval",
       sourceType: "channel",
-      conversationId: convId,
+      sourceConversationId: convId,
       guardianPrincipalId: TEST_PRINCIPAL,
     });
 
     // (2) pending request with a synthetic source id whose card was DELIVERED
     // to the candidate conversation (a different conversation than its source).
-    const delivered = createCanonicalGuardianRequest({
+    const delivered = bridgeState.seedRequest({
       kind: "access_request",
       sourceType: "channel",
-      conversationId: "access-req-xyz",
+      sourceConversationId: "access-req-xyz",
       guardianPrincipalId: TEST_PRINCIPAL,
     });
-    createCanonicalGuardianDelivery({
+    bridgeState.seedDelivery({
       requestId: delivered.id,
       destinationChannel: CHANNEL,
       destinationConversationId: convId,
     });
 
     expect(
-      candidateFor(convId)?.guardianContext?.pendingUnresolvedRequestCount,
+      (await candidateFor(convId))?.guardianContext
+        ?.pendingUnresolvedRequestCount,
     ).toBe(2);
   });
 
-  test("omits guardian context when the conversation's only request is resolved", () => {
+  test("omits guardian context when the conversation's only request is resolved", async () => {
     const convId = "conv-resolved";
     seedCandidateConversation(convId);
 
-    const req = createCanonicalGuardianRequest({
+    const req = bridgeState.seedRequest({
       kind: "tool_approval",
       sourceType: "channel",
-      conversationId: convId,
+      sourceConversationId: convId,
       guardianPrincipalId: TEST_PRINCIPAL,
     });
-    resolveCanonicalGuardianRequest(req.id, "pending", { status: "approved" });
+    await bridgeState.module.updateGuardianRequest(req.id, {
+      status: "approved",
+    });
 
     // The conversation still surfaces as a candidate (it had a delivery), but
     // with no guardian context since nothing is pending.
-    const candidate = candidateFor(convId);
+    const candidate = await candidateFor(convId);
     expect(candidate).toBeDefined();
     expect(candidate?.guardianContext).toBeUndefined();
   });
 
-  test("counts a Slack channel card recorded through the real recorder", () => {
+  test("counts a Slack channel card recorded through the real recorder", async () => {
     const SLACK = "slack" as NotificationChannel;
     const convId = "conv-slack-card";
     seedCandidateConversation(convId, SLACK);
@@ -175,14 +177,14 @@ describe("buildConversationCandidates guardian enrichment", () => {
     // Channel-only access request: a synthetic source conversation, card
     // delivered to a Slack chat. The recorder records the card's internal
     // conversation, so the Slack candidate counts it.
-    const req = createCanonicalGuardianRequest({
+    const req = bridgeState.seedRequest({
       kind: "access_request",
       sourceType: "channel",
       sourceChannel: "slack",
-      conversationId: "access-req-synthetic",
+      sourceConversationId: "access-req-synthetic",
       guardianPrincipalId: TEST_PRINCIPAL,
     });
-    recordGuardianRequestDeliveries({
+    await recordGuardianRequestDeliveries({
       requestId: req.id,
       deliveryResults: [
         {
@@ -196,7 +198,7 @@ describe("buildConversationCandidates guardian enrichment", () => {
     });
 
     expect(
-      candidateFor(convId, SLACK)?.guardianContext
+      (await candidateFor(convId, SLACK))?.guardianContext
         ?.pendingUnresolvedRequestCount,
     ).toBe(1);
   });

package/src/__tests__/notification-decision-fallback.test.ts

@@ -41,13 +41,6 @@ mock.module("../providers/provider-send-message.js", () => ({
   userMessage: (text: string) => ({ role: "user", content: text }),
 }));
 
-mock.module("../util/logger.js", () => ({
-  getLogger: () =>
-    new Proxy({} as Record<string, unknown>, {
-      get: () => () => {},
-    }),
-}));
-
 import { evaluateSignal } from "../notifications/decision-engine.js";
 import type { NotificationSignal } from "../notifications/signal.js";
 import type { NotificationChannel } from "../notifications/types.js";

package/src/__tests__/notification-decision-identity.test.ts

@@ -56,13 +56,6 @@ mock.module("../providers/provider-send-message.js", () => ({
   userMessage: (text: string) => ({ role: "user", content: text }),
 }));
 
-mock.module("../util/logger.js", () => ({
-  getLogger: () =>
-    new Proxy({} as Record<string, unknown>, {
-      get: () => () => {},
-    }),
-}));
-
 // ── Imports (after all mocks) ─────────────────────────────────────────
 
 import { evaluateSignal } from "../notifications/decision-engine.js";

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

@@ -84,13 +84,6 @@ mock.module("../providers/provider-send-message.js", () => ({
   userMessage: (text: string) => ({ role: "user", content: text }),
 }));
 
-mock.module("../util/logger.js", () => ({
-  getLogger: () =>
-    new Proxy({} as Record<string, unknown>, {
-      get: () => () => {},
-    }),
-}));
-
 // ── Imports (after all mocks) ─────────────────────────────────────────
 
 import { evaluateSignal } from "../notifications/decision-engine.js";

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

@@ -15,13 +15,6 @@ import type { PairingOptions } from "../notifications/conversation-pairing.js";
 
 // -- Mocks (must be declared before importing modules that depend on them) ----
 
-mock.module("../util/logger.js", () => ({
-  getLogger: () =>
-    new Proxy({} as Record<string, unknown>, {
-      get: () => () => {},
-    }),
-}));
-
 mock.module("../contacts/guardian-delivery-reader.js", () => ({
   getGuardianDelivery: async () => null,
 }));

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

@@ -1,5 +1,5 @@
 /**
- * Regression tests for the ASK_GUARDIAN canonical notification path.
+ * Regression tests for the ASK_GUARDIAN notification path.
  *
  * Validates that guardian dispatch relies on the generic notification
  * pipeline (including conversation-created callbacks) without a custom dispatch path.
@@ -10,27 +10,10 @@ import { beforeEach, describe, expect, mock, test } from "bun:test";
 import type { ConversationCreatedInfo } from "../notifications/broadcaster.js";
 import type { NotificationDeliveryResult } from "../notifications/types.js";
 
-mock.module("../util/logger.js", () => ({
-  getLogger: () =>
-    new Proxy({} as Record<string, unknown>, {
-      get: () => () => {},
-    }),
-}));
-
 // Note: stale mock for channel-guardian-store.js removed — the barrel was
 // deleted and none of the functions it mocked (getActiveBinding, createBinding,
 // listActiveBindingsByAssistant) existed in the barrel.
 
-mock.module("../config/loader.js", () => ({
-  getConfig: () => ({
-    ui: {},
-
-    calls: {
-      userConsultTimeoutSeconds: 120,
-    },
-  }),
-}));
-
 const emitCalls: unknown[] = [];
 let mockConversationCreated: ConversationCreatedInfo | null = null;
 let mockEmitResult: {
@@ -87,6 +70,18 @@ mock.module("../contacts/guardian-delivery-reader.js", () => ({
   anyGuardian: (list: unknown[]) => list[0],
 }));
 
+// Guardian requests/deliveries are created through the gateway client; serve
+// that surface from the in-memory sim the assertions read.
+import {
+  bridgeState,
+  gatewayGuardianRequestsStoreBridge,
+} from "./helpers/gateway-guardian-requests-store-bridge.js";
+
+mock.module(
+  "../channels/gateway-guardian-requests.js",
+  () => gatewayGuardianRequestsStoreBridge,
+);
+
 import {
   createCallSession,
   createPendingQuestion,
@@ -112,12 +107,15 @@ function ensureConversation(id: string): void {
     .run();
 }
 
+function requestForCallSession(callSessionId: string) {
+  return [...bridgeState.requests.values()]
+    .filter((r) => r.callSessionId === callSessionId)
+    .sort((a, b) => a.createdAt - b.createdAt)[0];
+}
+
 function resetTables(): void {
   const db = getDb();
-  db.run("DELETE FROM canonical_guardian_deliveries");
-  db.run("DELETE FROM canonical_guardian_requests");
-  db.run("DELETE FROM guardian_action_deliveries");
-  db.run("DELETE FROM guardian_action_requests");
+  bridgeState.reset();
   db.run("DELETE FROM call_pending_questions");
   db.run("DELETE FROM call_events");
   db.run("DELETE FROM call_sessions");
@@ -152,7 +150,7 @@ function resetTables(): void {
   };
 }
 
-describe("ASK_GUARDIAN canonical notification path", () => {
+describe("ASK_GUARDIAN notification path", () => {
   beforeEach(() => {
     resetTables();
   });
@@ -279,35 +277,21 @@ describe("ASK_GUARDIAN canonical notification path", () => {
       pendingQuestion: pq,
     });
 
-    const db = getDb();
-    const raw = (db as unknown as { $client: import("bun:sqlite").Database })
-      .$client;
-    const request = raw
-      .query(
-        "SELECT * FROM canonical_guardian_requests WHERE call_session_id = ?",
-      )
-      .get(session.id) as { id: string } | undefined;
-    const deliveries = raw
-      .query(
-        "SELECT destination_channel, destination_conversation_id, destination_chat_id, status FROM canonical_guardian_deliveries WHERE request_id = ? ORDER BY destination_channel ASC",
-      )
-      .all(request!.id) as Array<{
-      destination_channel: string;
-      destination_conversation_id: string | null;
-      destination_chat_id: string | null;
-      status: string;
-    }>;
+    const request = requestForCallSession(session.id);
+    const deliveries = bridgeState.deliveries.filter(
+      (d) => d.requestId === request!.id,
+    );
 
     expect(deliveries).toHaveLength(2);
     const telegram = deliveries.find(
-      (d) => d.destination_channel === "telegram",
+      (d) => d.destinationChannel === "telegram",
     );
-    const vellum = deliveries.find((d) => d.destination_channel === "vellum");
+    const vellum = deliveries.find((d) => d.destinationChannel === "vellum");
     expect(telegram).toBeDefined();
-    expect(telegram!.destination_chat_id).toBe("tg-chat-abc");
+    expect(telegram!.destinationChatId).toBe("tg-chat-abc");
     expect(telegram!.status).toBe("sent");
     expect(vellum).toBeDefined();
-    expect(vellum!.destination_conversation_id).toBe("conv-guardian-vellum");
+    expect(vellum!.destinationConversationId).toBe("conv-guardian-vellum");
     expect(vellum!.status).toBe("sent");
   });
 
@@ -340,19 +324,10 @@ describe("ASK_GUARDIAN canonical notification path", () => {
       }),
     ).resolves.toBeUndefined();
 
-    const db = getDb();
-    const raw = (db as unknown as { $client: import("bun:sqlite").Database })
-      .$client;
-    const request = raw
-      .query(
-        "SELECT * FROM canonical_guardian_requests WHERE call_session_id = ?",
-      )
-      .get(session.id) as { id: string } | undefined;
-    const vellumDelivery = raw
-      .query(
-        "SELECT status FROM canonical_guardian_deliveries WHERE request_id = ? AND destination_channel = ?",
-      )
-      .get(request!.id, "vellum") as { status: string } | undefined;
+    const request = requestForCallSession(session.id);
+    const vellumDelivery = bridgeState.deliveries.find(
+      (d) => d.requestId === request!.id && d.destinationChannel === "vellum",
+    );
 
     expect(vellumDelivery).toBeDefined();
     expect(vellumDelivery!.status).toBe("failed");
@@ -453,34 +428,23 @@ describe("ASK_GUARDIAN canonical notification path", () => {
       pendingQuestion: pq2,
     });
 
-    // Verify: two distinct canonical_guardian_requests exist
-    const db = getDb();
-    const raw = (db as unknown as { $client: import("bun:sqlite").Database })
-      .$client;
-    const requests = raw
-      .query(
-        "SELECT id, question_text FROM canonical_guardian_requests WHERE call_session_id = ? ORDER BY created_at ASC",
-      )
-      .all(session.id) as Array<{ id: string; question_text: string }>;
+    // Verify: two distinct requests exist
+    const requests = [...bridgeState.requests.values()]
+      .filter((r) => r.callSessionId === session.id)
+      .sort((a, b) => a.createdAt - b.createdAt);
     expect(requests).toHaveLength(2);
-    expect(requests[0].question_text).toBe(
+    expect(requests[0].questionText).toBe(
       "Can they enter through the side gate?",
     );
-    expect(requests[1].question_text).toBe("What about the back door?");
+    expect(requests[1].questionText).toBe("What about the back door?");
 
     // Verify: each request has its own delivery row pointing to the shared conversation
-    const deliveries = raw
-      .query(
-        "SELECT request_id, destination_conversation_id, status FROM canonical_guardian_deliveries WHERE destination_conversation_id = ? ORDER BY created_at ASC",
-      )
-      .all(sharedConvId) as Array<{
-      request_id: string;
-      destination_conversation_id: string;
-      status: string;
-    }>;
+    const deliveries = bridgeState.deliveries.filter(
+      (d) => d.destinationConversationId === sharedConvId,
+    );
     expect(deliveries).toHaveLength(2);
-    expect(deliveries[0].request_id).toBe(requests[0].id);
-    expect(deliveries[1].request_id).toBe(requests[1].id);
+    expect(deliveries[0].requestId).toBe(requests[0].id);
+    expect(deliveries[1].requestId).toBe(requests[1].id);
     expect(deliveries[0].status).toBe("sent");
     expect(deliveries[1].status).toBe("sent");
   });
@@ -514,21 +478,12 @@ describe("ASK_GUARDIAN canonical notification path", () => {
     });
 
     // The dispatch should still create a failed fallback delivery row
-    const db = getDb();
-    const raw = (db as unknown as { $client: import("bun:sqlite").Database })
-      .$client;
-    const request = raw
-      .query(
-        "SELECT id FROM canonical_guardian_requests WHERE call_session_id = ?",
-      )
-      .get(session.id) as { id: string } | undefined;
+    const request = requestForCallSession(session.id);
     expect(request).toBeDefined();
 
-    const delivery = raw
-      .query(
-        "SELECT status FROM canonical_guardian_deliveries WHERE request_id = ? AND destination_channel = ?",
-      )
-      .get(request!.id, "vellum") as { status: string } | undefined;
+    const delivery = bridgeState.deliveries.find(
+      (d) => d.requestId === request!.id && d.destinationChannel === "vellum",
+    );
     expect(delivery).toBeDefined();
     expect(delivery!.status).toBe("failed");
   });

package/src/__tests__/notification-platform-adapter.test.ts

@@ -2,13 +2,6 @@ import { beforeEach, describe, expect, mock, test } from "bun:test";
 
 // ── Mock logger ──────────────────────────────────────────────────────────────
 
-mock.module("../util/logger.js", () => ({
-  getLogger: () =>
-    new Proxy({} as Record<string, unknown>, {
-      get: () => () => {},
-    }),
-}));
-
 // ── Mock sleep so retry tests don't slow down the suite ──────────────────────
 
 mock.module("../util/retry.js", () => ({

package/src/__tests__/notification-routing-intent.test.ts

@@ -6,15 +6,7 @@
  * persisted on the reminder at create time.
  */
 
-import { describe, expect, mock, test } from "bun:test";
-
-mock.module("../util/logger.js", () => ({
-  getLogger: () =>
-    new Proxy({} as Record<string, unknown>, {
-      get: () => () => {},
-    }),
-  truncateForLog: (v: string) => v,
-}));
+import { describe, expect, test } from "bun:test";
 
 import { enforceRoutingIntent } from "../notifications/decision-engine.js";
 import type {

package/src/__tests__/notification-schedule-notify-dedup.test.ts

@@ -9,15 +9,7 @@
  * `schedule:notify:<id>`.
  */
 
-import { beforeEach, describe, expect, mock, test } from "bun:test";
-
-mock.module("../util/logger.js", () => ({
-  getLogger: () =>
-    new Proxy({} as Record<string, unknown>, {
-      get: () => () => {},
-    }),
-  truncateForLog: (value: string) => value,
-}));
+import { beforeEach, describe, expect, test } from "bun:test";
 
 import { runDeterministicChecks } from "../notifications/deterministic-checks.js";
 import { createEvent } from "../notifications/events-store.js";

package/src/__tests__/notification-telegram-adapter.test.ts

@@ -36,13 +36,6 @@ mock.module("../messaging/providers/telegram-bot/send.js", () => ({
   sendTelegramTypingIndicator: async () => true,
 }));
 
-mock.module("../util/logger.js", () => ({
-  getLogger: () =>
-    new Proxy({} as Record<string, unknown>, {
-      get: () => () => {},
-    }),
-}));
-
 import { TelegramAdapter } from "../notifications/adapters/telegram.js";
 import type {
   ChannelDeliveryPayload,