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

package/src/__tests__/call-domain.test.ts

@@ -9,19 +9,10 @@
  */
 import { beforeEach, describe, expect, mock, test } from "bun:test";
 
-mock.module("../util/logger.js", () => ({
-  getLogger: () =>
-    new Proxy({} as Record<string, unknown>, {
-      get: () => () => {},
-    }),
-}));
-
 // Track whether the Twilio provider's initiateCall should succeed or throw
 let twilioInitiateCallBehavior: "success" | "error" = "success";
 let twilioInitiateCallCount = 0;
 let twilioInitiateCallArgs: Array<Record<string, unknown>> = [];
-let mockIngressEnabled = true;
-let mockIngressPublicBaseUrl = "https://test.example.com";
 
 mock.module("../calls/twilio-config.js", () => ({
   getTwilioConfig: (assistantId?: string) => ({
@@ -57,33 +48,6 @@ mock.module("../security/secure-keys.js", () => ({
   getSecureKeyAsync: async () => null,
 }));
 
-mock.module("../config/loader.js", () => ({
-  loadConfig: () => ({
-    calls: {
-      enabled: true,
-      provider: "twilio",
-      callerIdentity: { allowPerCallOverride: true },
-    },
-    ingress: {
-      enabled: mockIngressEnabled,
-      publicBaseUrl: mockIngressPublicBaseUrl,
-    },
-    memory: { enabled: false },
-  }),
-  getConfig: () => ({
-    calls: {
-      enabled: true,
-      provider: "twilio",
-      callerIdentity: { allowPerCallOverride: true },
-    },
-    ingress: {
-      enabled: mockIngressEnabled,
-      publicBaseUrl: mockIngressPublicBaseUrl,
-    },
-    memory: { enabled: false },
-  }),
-}));
-
 mock.module("../inbound/platform-callback-registration.js", () => ({
   resolveCallbackUrl: async (fn: () => string) => fn(),
 }));
@@ -126,6 +90,18 @@ import { getMessages } from "../persistence/conversation-crud.js";
 import { getDb } from "../persistence/db-connection.js";
 import { initializeDb } from "../persistence/db-init.js";
 import { conversations } from "../persistence/schema/index.js";
+import { setConfig } from "./helpers/set-config.js";
+
+/** Seed the ingress block startCall's preflight reads for real. */
+function seedIngress(enabled: boolean): void {
+  setConfig("ingress", {
+    enabled,
+    publicBaseUrl: "https://test.example.com",
+  });
+}
+
+// Disable memory so pointer-message writes skip background indexing.
+setConfig("memory", { enabled: false });
 
 await initializeDb();
 
@@ -135,8 +111,7 @@ beforeEach(() => {
   twilioInitiateCallBehavior = "success";
   twilioInitiateCallCount = 0;
   twilioInitiateCallArgs = [];
-  mockIngressEnabled = true;
-  mockIngressPublicBaseUrl = "https://test.example.com";
+  seedIngress(true);
   mockCredentialReadiness = { status: "ready" };
 });
 
@@ -171,23 +146,13 @@ function getLatestAssistantText(conversationId: string): string | null {
   );
   if (msgs.length === 0) return null;
   const latest = msgs[msgs.length - 1];
-  try {
-    const parsed = JSON.parse(latest.content) as unknown;
-    if (Array.isArray(parsed)) {
-      return parsed
-        .filter(
-          (b): b is { type: string; text?: string } =>
-            typeof b === "object" && b != null,
-        )
-        .filter((b) => b.type === "text")
-        .map((b) => b.text ?? "")
-        .join("");
-    }
-    if (typeof parsed === "string") return parsed;
-  } catch {
-    /* fall through */
-  }
-  return latest.content;
+  return latest.content
+    .filter(
+      (b): b is { type: "text"; text: string } =>
+        b.type === "text" && typeof (b as { text?: unknown }).text === "string",
+    )
+    .map((b) => b.text)
+    .join("");
 }
 
 function makeConfig(
@@ -351,7 +316,7 @@ describe("startCall — pointer message regression", () => {
   test("fails fast when ingress is disabled and never reaches Twilio dialing", async () => {
     const convId = "conv-domain-ingress-disabled";
     ensureConversation(convId);
-    mockIngressEnabled = false;
+    seedIngress(false);
 
     const result = await startCall({
       phoneNumber: "+15559876543",

package/src/__tests__/call-pointer-message-composer.test.ts

@@ -1,12 +1,4 @@
 import { describe, expect, test } from "bun:test";
-import { mock } from "bun:test";
-
-mock.module("../util/logger.js", () => ({
-  getLogger: () =>
-    new Proxy({} as Record<string, unknown>, {
-      get: () => () => {},
-    }),
-}));
 
 import {
   buildPointerInstruction,

package/src/__tests__/call-pointer-messages.test.ts

@@ -1,12 +1,5 @@
 import { afterEach, beforeEach, describe, expect, mock, test } from "bun:test";
 
-mock.module("../util/logger.js", () => ({
-  getLogger: () =>
-    new Proxy({} as Record<string, unknown>, {
-      get: () => () => {},
-    }),
-}));
-
 // Stand in for the daemon conversation-turn path so these tests can drive its
 // behavior (success / throw) without pulling in the real conversation pipeline.
 // `runPointerTurnImpl` is swapped per-test via setProcessor/resetProcessor.
@@ -80,7 +73,7 @@ function getLatestAssistantText(conversationId: string): string {
   );
   expect(rows.length).toBeGreaterThan(0);
   const latest = rows[rows.length - 1];
-  const parsed = JSON.parse(latest.content) as Array<{
+  const parsed = latest.content as unknown as Array<{
     type: string;
     text?: string;
   }>;

package/src/__tests__/call-recovery.test.ts

@@ -1,11 +1,4 @@
-import { beforeEach, describe, expect, mock, test } from "bun:test";
-
-mock.module("../util/logger.js", () => ({
-  getLogger: () =>
-    new Proxy({} as Record<string, unknown>, {
-      get: () => () => {},
-    }),
-}));
+import { beforeEach, describe, expect, test } from "bun:test";
 
 import {
   clearActiveCallLeases,
@@ -35,7 +28,9 @@ await initializeDb();
 /** Ensure a conversation row exists for the given ID so FK constraints pass. */
 let ensuredConvIds = new Set<string>();
 function ensureConversation(id: string): void {
-  if (ensuredConvIds.has(id)) return;
+  if (ensuredConvIds.has(id)) {
+    return;
+  }
   const db = getDb();
   const now = Date.now();
   db.insert(conversations)
@@ -51,8 +46,6 @@ function ensureConversation(id: string): void {
 
 function resetTables() {
   const db = getDb();
-  db.run("DELETE FROM guardian_action_deliveries");
-  db.run("DELETE FROM guardian_action_requests");
   db.run("DELETE FROM call_pending_questions");
   db.run("DELETE FROM call_events");
   db.run("DELETE FROM call_sessions");

package/src/__tests__/call-routes-http.test.ts

@@ -9,50 +9,6 @@ import { beforeEach, describe, expect, mock, test } from "bun:test";
 
 mock.module("../config/env.js", () => ({ isHttpAuthDisabled: () => true }));
 
-mock.module("../util/logger.js", () => ({
-  getLogger: () =>
-    new Proxy({} as Record<string, unknown>, {
-      get: () => () => {},
-    }),
-}));
-
-const mockCallsConfig = {
-  enabled: true,
-  provider: "twilio",
-  maxDurationSeconds: 3600,
-  userConsultTimeoutSeconds: 120,
-  disclosure: { enabled: false, text: "" },
-  safety: { denyCategories: [] },
-  callerIdentity: {
-    allowPerCallOverride: true,
-  },
-};
-
-mock.module("../config/loader.js", () => ({
-  getConfig: () => ({
-    ui: {},
-
-    model: "test",
-    provider: "test",
-    memory: { enabled: false },
-    rateLimit: { maxRequestsPerMinute: 0 },
-    secretDetection: { enabled: false },
-    calls: mockCallsConfig,
-  }),
-  loadConfig: () => ({
-    model: "test",
-    provider: "test",
-    memory: { enabled: false },
-    rateLimit: { maxRequestsPerMinute: 0 },
-    secretDetection: { enabled: false },
-    calls: mockCallsConfig,
-    ingress: {
-      enabled: true,
-      publicBaseUrl: "https://test.example.com",
-    },
-  }),
-}));
-
 // Mock Twilio provider to avoid real API calls
 class MockTwilioVoiceProvider {
   static getAuthToken() {
@@ -108,9 +64,18 @@ import { getDb } from "../persistence/db-connection.js";
 import { initializeDb } from "../persistence/db-init.js";
 import { conversations } from "../persistence/schema/index.js";
 import { RuntimeHttpServer } from "../runtime/http-server.js";
+import { setConfig } from "./helpers/set-config.js";
 
 import "../calls/call-state.js";
 
+// Disable memory (skip background indexing of test messages) and seed the
+// ingress base URL that outbound-call URL building reads.
+setConfig("memory", { enabled: false });
+setConfig("ingress", {
+  enabled: true,
+  publicBaseUrl: "https://test.example.com",
+});
+
 await initializeDb();
 
 // ---------------------------------------------------------------------------
@@ -123,7 +88,9 @@ const AUTH_HEADERS = { Authorization: `Bearer ${TEST_TOKEN}` };
 let ensuredConvIds = new Set<string>();
 
 function ensureConversation(id: string): void {
-  if (ensuredConvIds.has(id)) return;
+  if (ensuredConvIds.has(id)) {
+    return;
+  }
   const db = getDb();
   const now = Date.now();
   db.insert(conversations)
@@ -139,8 +106,6 @@ function ensureConversation(id: string): void {
 
 function resetTables() {
   const db = getDb();
-  db.run("DELETE FROM guardian_action_deliveries");
-  db.run("DELETE FROM guardian_action_requests");
   db.run("DELETE FROM call_pending_questions");
   db.run("DELETE FROM call_events");
   db.run("DELETE FROM call_sessions");

package/src/__tests__/call-site-routing-provider.test.ts

@@ -15,21 +15,8 @@
  * Call-site profiles without a connection throw `ConnectionResolutionError`.
  */
 
-import { beforeEach, describe, expect, mock, test } from "bun:test";
+import { beforeEach, describe, expect, test } from "bun:test";
 
-mock.module("../util/logger.js", () => ({
-  getLogger: () =>
-    new Proxy({} as Record<string, unknown>, { get: () => () => {} }),
-}));
-
-// Mutable LLM config consumed by the resolver via `getConfig()`.
-let mockLlmConfig: Record<string, unknown> = {};
-
-mock.module("../config/loader.js", () => ({
-  getConfig: () => ({ llm: mockLlmConfig }),
-}));
-
-import { LLMSchema } from "../config/schemas/llm.js";
 import { CallSiteRoutingProvider } from "../providers/call-site-routing.js";
 import { ConnectionResolutionError } from "../providers/connection-resolution.js";
 import type {
@@ -38,6 +25,7 @@ import type {
   ProviderResponse,
   SendMessageOptions,
 } from "../providers/types.js";
+import { setConfig } from "./helpers/set-config.js";
 
 const DUMMY_MESSAGES: Message[] = [
   { role: "user", content: [{ type: "text", text: "hi" }] },
@@ -65,8 +53,9 @@ function makeProvider(
   };
 }
 
+/** Seed the `llm` config block for real; the schema fills in defaults. */
 function setLlmConfig(raw: unknown): void {
-  mockLlmConfig = LLMSchema.parse(raw) as Record<string, unknown>;
+  setConfig("llm", raw);
 }
 
 /**
@@ -85,7 +74,7 @@ function makeConnectionHook(
 }
 
 beforeEach(() => {
-  mockLlmConfig = LLMSchema.parse({}) as Record<string, unknown>;
+  setLlmConfig({});
 });
 
 describe("CallSiteRoutingProvider", () => {

package/src/__tests__/call-start-guardian-guard.test.ts

@@ -10,21 +10,12 @@ import {
 
 import type { ToolContext } from "../tools/types.js";
 
-let callsEnabled = true;
 const startCallInputs: Array<Record<string, unknown>> = [];
 let activeVoiceSession: {
   destinationAddress: string | null;
   expectedPhoneE164: string | null;
 } | null = null;
 
-mock.module("../config/loader.js", () => ({
-  getConfig: () => ({
-    ui: {},
-
-    calls: { enabled: callsEnabled },
-  }),
-}));
-
 mock.module("../calls/call-domain.js", () => ({
   startCall: async (input: Record<string, unknown>) => {
     startCallInputs.push(input);
@@ -76,7 +67,6 @@ describe("call_start guardian verification guard", () => {
   });
 
   beforeEach(() => {
-    callsEnabled = true;
     startCallInputs.length = 0;
     activeVoiceSession = null;
   });

package/src/__tests__/call-state.test.ts

@@ -1,11 +1,4 @@
-import { beforeEach, describe, expect, mock, test } from "bun:test";
-
-mock.module("../util/logger.js", () => ({
-  getLogger: () =>
-    new Proxy({} as Record<string, unknown>, {
-      get: () => () => {},
-    }),
-}));
+import { beforeEach, describe, expect, test } from "bun:test";
 
 import type { CallController } from "../calls/call-controller.js";
 import {

package/src/__tests__/call-store.test.ts

@@ -1,11 +1,4 @@
-import { beforeEach, describe, expect, mock, test } from "bun:test";
-
-mock.module("../util/logger.js", () => ({
-  getLogger: () =>
-    new Proxy({} as Record<string, unknown>, {
-      get: () => () => {},
-    }),
-}));
+import { beforeEach, describe, expect, test } from "bun:test";
 
 import {
   answerPendingQuestion,
@@ -32,7 +25,9 @@ await initializeDb();
 /** Ensure a conversation row exists for the given ID so FK constraints pass. */
 let ensuredConvIds = new Set<string>();
 function ensureConversation(id: string): void {
-  if (ensuredConvIds.has(id)) return;
+  if (ensuredConvIds.has(id)) {
+    return;
+  }
   const db = getDb();
   const now = Date.now();
   db.insert(conversations)
@@ -48,8 +43,6 @@ function ensureConversation(id: string): void {
 
 function resetTables() {
   const db = getDb();
-  db.run("DELETE FROM guardian_action_deliveries");
-  db.run("DELETE FROM guardian_action_requests");
   db.run("DELETE FROM call_pending_questions");
   db.run("DELETE FROM call_events");
   db.run("DELETE FROM call_sessions");

package/src/__tests__/cancel-resolves-conversation-key.test.ts

@@ -10,38 +10,6 @@ import { describe, expect, mock, test } from "bun:test";
 
 mock.module("../config/env.js", () => ({ isHttpAuthDisabled: () => true }));
 
-mock.module("../util/logger.js", () => ({
-  getLogger: () =>
-    new Proxy({} as Record<string, unknown>, {
-      get: () => () => {},
-    }),
-}));
-
-mock.module("../config/loader.js", () => ({
-  getConfig: () => ({
-    ui: {},
-    model: "test",
-    provider: "test",
-    memory: { enabled: false },
-    rateLimit: { maxRequestsPerMinute: 0 },
-    secretDetection: { enabled: false },
-    contextWindow: { maxInputTokens: 200000 },
-    services: {
-      inference: {
-        mode: "your-own",
-        provider: "anthropic",
-        model: "claude-opus-4-6",
-      },
-      "image-generation": {
-        mode: "your-own",
-        provider: "gemini",
-        model: "gemini-3.1-flash-image-preview",
-      },
-      "web-search": { mode: "your-own", provider: "inference-provider-native" },
-    },
-  }),
-}));
-
 let cancelledId: string | undefined;
 mock.module("../daemon/handlers/conversations.js", () => ({
   cancelGeneration: (id: string) => {

package/src/__tests__/catalog-cache.test.ts

@@ -13,14 +13,6 @@ import type { CatalogSkill } from "../skills/catalog-install.js";
 // Mocks — must be defined before importing the module under test
 // ---------------------------------------------------------------------------
 
-// Suppress logger output
-mock.module("../util/logger.js", () => ({
-  getLogger: () =>
-    new Proxy({} as Record<string, unknown>, {
-      get: () => () => {},
-    }),
-}));
-
 let mockRepoSkillsDir: string | undefined = undefined;
 let mockLocalCatalog: CatalogSkill[] = [];
 let mockFetchCatalogResult: CatalogSkill[] = [];

package/src/__tests__/catalog-files.test.ts

@@ -32,14 +32,6 @@ import type { CatalogSkill } from "../skills/catalog-install.js";
 // Mocks — must be declared before importing the module under test
 // ---------------------------------------------------------------------------
 
-// Suppress logger output
-mock.module("../util/logger.js", () => ({
-  getLogger: () =>
-    new Proxy({} as Record<string, unknown>, {
-      get: () => () => {},
-    }),
-}));
-
 let mockCatalog: CatalogSkill[] = [];
 let mockRepoSkillsDir: string | undefined = undefined;
 

package/src/__tests__/catalog-install-normalize.test.ts

@@ -14,13 +14,6 @@ import { tmpdir } from "node:os";
 import { join } from "node:path";
 import { afterEach, describe, expect, mock, test } from "bun:test";
 
-mock.module("../util/logger.js", () => ({
-  getLogger: () =>
-    new Proxy({} as Record<string, unknown>, {
-      get: () => () => {},
-    }),
-}));
-
 mock.module("../config/env.js", () => ({
   getPlatformBaseUrl: () => "https://platform.example.com",
 }));

package/src/__tests__/ces-rpc-credential-backend.test.ts

@@ -16,13 +16,6 @@ const isReadyFn = mock((): boolean => true);
 // Mock modules — before importing module under test
 // ---------------------------------------------------------------------------
 
-mock.module("../util/logger.js", () => ({
-  getLogger: () =>
-    new Proxy({} as Record<string, unknown>, {
-      get: () => () => {},
-    }),
-}));
-
 // Import after mocking
 import type { CesClient } from "../credential-execution/client.js";
 import { CesRpcCredentialBackend } from "../security/ces-rpc-credential-backend.js";

package/src/__tests__/channel-approval-routes.test.ts

@@ -8,13 +8,6 @@ import {
   test,
 } from "bun:test";
 
-mock.module("../util/logger.js", () => ({
-  getLogger: () =>
-    new Proxy({} as Record<string, unknown>, {
-      get: () => () => {},
-    }),
-}));
-
 const _conversationMocks = new Map<string, unknown>();
 mock.module("../daemon/conversation-registry.js", () => ({
   findConversation: (id: string) => _conversationMocks.get(id),
@@ -48,7 +41,9 @@ mock.module("../daemon/process-message.js", () => ({
   resolveTurnInterface: () => "telegram",
   prepareConversationForMessage: async () => ({}),
   processMessage: (...args: unknown[]) => {
-    if (_testProcessMessage) return _testProcessMessage(...args);
+    if (_testProcessMessage) {
+      return _testProcessMessage(...args);
+    }
     return Promise.resolve({ messageId: "mock-msg-1" });
   },
   processMessageInBackground: async () => ({ messageId: "mock-bg" }),
@@ -69,11 +64,6 @@ mock.module("../daemon/approval-generators.js", () => ({
   createApprovalConversationGenerator: () => _testApprovalConversationGenerator,
 }));
 
-import {
-  createCanonicalGuardianDelivery,
-  createCanonicalGuardianRequest,
-  getCanonicalGuardianRequest,
-} from "../contacts/canonical-guardian-store.js";
 import type { Conversation } from "../daemon/conversation.js";
 import { getDb } from "../persistence/db-connection.js";
 import { initializeDb } from "../persistence/db-init.js";
@@ -91,6 +81,7 @@ import {
 } from "./helpers/channel-test-adapter.js";
 import { createGuardianBinding } from "./helpers/create-guardian-binding.js";
 import { resetGatewayAclStore } from "./helpers/gateway-acl-store.js";
+import { bridgeState } from "./helpers/gateway-guardian-requests-store-bridge.js";
 
 await initializeDb();
 initAuthSigningKey(Buffer.from("test-signing-key-at-least-32-bytes-long"));
@@ -119,10 +110,9 @@ function ensureConversation(conversationId: string): void {
 }
 
 function resetTables(): void {
+  bridgeState.reset();
   resetTestTables(
     "scoped_approval_grants",
-    "canonical_guardian_deliveries",
-    "canonical_guardian_requests",
     "conversation_keys",
     "message_runs",
     "channel_inbound_events",
@@ -1621,8 +1611,7 @@ describe("background channel processing approval prompts", () => {
 
     expect(deliverPromptSpy).toHaveBeenCalled();
     const approvalMeta = deliverPromptSpy.mock.calls[0]?.[3] as
-      | { requestId?: string }
-      | undefined;
+      { requestId?: string } | undefined;
     expect(approvalMeta?.requestId).toBe("req-bg-1");
 
     deliverPromptSpy.mockRestore();
@@ -1789,10 +1778,10 @@ describe("background channel processing approval prompts", () => {
 });
 
 // ═══════════════════════════════════════════════════════════════════════════
-// NL approval routing via destination-scoped canonical requests
+// NL approval routing via destination-scoped guardian requests
 // ═══════════════════════════════════════════════════════════════════════════
 
-describe("NL approval routing via destination-scoped canonical requests", () => {
+describe("NL approval routing via destination-scoped guardian requests", () => {
   beforeEach(() => {
     resetTables();
     noopProcessMessage.mockClear();
@@ -1815,13 +1804,13 @@ describe("NL approval routing via destination-scoped canonical requests", () =>
       guardianPrincipalId: guardianUserId,
     });
 
-    // Create canonical tool_approval request WITHOUT guardianExternalUserId
+    // Create guardian tool_approval request WITHOUT guardianExternalUserId
     // but WITH a conversationId (required by the tool_approval resolver)
-    const canonicalReq = createCanonicalGuardianRequest({
+    const guardianReq = bridgeState.seedRequest({
       kind: "tool_approval",
       sourceType: "voice",
       sourceChannel: "twilio",
-      conversationId: "conv-voice-nl-1",
+      sourceConversationId: "conv-voice-nl-1",
       toolName: "shell",
       guardianPrincipalId: "test-principal-id",
       expiresAt: Date.now() + 60_000,
@@ -1829,11 +1818,11 @@ describe("NL approval routing via destination-scoped canonical requests", () =>
     });
 
     // Register pending interaction so resolver can find it
-    registerPendingInteraction(canonicalReq.id, "conv-voice-nl-1", "shell");
+    registerPendingInteraction(guardianReq.id, "conv-voice-nl-1", "shell");
 
-    // Create canonical delivery row targeting guardian chat
-    createCanonicalGuardianDelivery({
-      requestId: canonicalReq.id,
+    // Create guardian delivery row targeting guardian chat
+    bridgeState.seedDelivery({
+      requestId: guardianReq.id,
       destinationChannel: "telegram",
       destinationChatId: guardianChatId,
     });
@@ -1853,12 +1842,12 @@ describe("NL approval routing via destination-scoped canonical requests", () =>
     expect(body.canonicalRouter).toBe("canonical_decision_stale");
 
     // Verify the request remains pending (identity-bound fail-closed).
-    const resolved = getCanonicalGuardianRequest(canonicalReq.id);
+    const resolved = bridgeState.getRequest(guardianReq.id);
     expect(resolved).not.toBeNull();
     expect(resolved!.status).toBe("pending");
   });
 
-  test("inbound from different chat ID does not auto-match delivery-scoped canonical request", async () => {
+  test("inbound from different chat ID does not auto-match delivery-scoped guardian request", async () => {
     const guardianChatId = "guardian-chat-nl-2";
     const guardianUserId = "guardian-user-nl-2";
     const differentChatId = "different-chat-999";
@@ -1872,8 +1861,8 @@ describe("NL approval routing via destination-scoped canonical requests", () =>
       guardianPrincipalId: guardianUserId,
     });
 
-    // Create canonical pending_question WITHOUT guardianExternalUserId
-    const canonicalReq = createCanonicalGuardianRequest({
+    // Create guardian pending_question WITHOUT guardianExternalUserId
+    const guardianReq = bridgeState.seedRequest({
       kind: "tool_approval",
       sourceType: "voice",
       sourceChannel: "twilio",
@@ -1883,8 +1872,8 @@ describe("NL approval routing via destination-scoped canonical requests", () =>
     });
 
     // Delivery targets the original guardian chat, NOT the different chat
-    createCanonicalGuardianDelivery({
-      requestId: canonicalReq.id,
+    bridgeState.seedDelivery({
+      requestId: guardianReq.id,
       destinationChannel: "telegram",
       destinationChatId: guardianChatId,
     });
@@ -1907,7 +1896,7 @@ describe("NL approval routing via destination-scoped canonical requests", () =>
     expect(body.canonicalRouter).toBeUndefined();
 
     // Request should remain pending
-    const unchanged = getCanonicalGuardianRequest(canonicalReq.id);
+    const unchanged = bridgeState.getRequest(guardianReq.id);
     expect(unchanged).not.toBeNull();
     expect(unchanged!.status).toBe("pending");
   });
@@ -1956,7 +1945,7 @@ describe("trusted-contact self-approval blocked before guardian approval row exi
     const conversationId = events[0]?.conversation_id;
     ensureConversation(conversationId!);
 
-    // Register a pending interaction — but do NOT create a canonical guardian
+    // Register a pending interaction — but do NOT create a guardian
     // request row. This simulates the window between the pending confirmation
     // being created (isInteractive=true) and the guardian approval prompt being
     // delivered.