@vellumai/assistant 0.12.1-dev.202609141913.5e16607 → 0.12.1-staging.2

package/src/{calls/__tests__ → __tests__}/media-stream-server-integration.test.ts

@@ -15,7 +15,7 @@ import {
 // Mock the STT resolve module (used by MediaStreamSttSession).
 // resolveStreamingTranscriber yields no transcriber, so sessions settle on
 // the batch path regardless of the calls.voice.telephonyStreaming default.
-mock.module("../../providers/speech-to-text/resolve.js", () => ({
+mock.module("../providers/speech-to-text/resolve.js", () => ({
   resolveTelephonySttCapability: jest.fn(),
   resolveBatchTranscriber: jest.fn(),
   resolveStreamingTranscriber: jest.fn(async () => null),
@@ -29,7 +29,7 @@ const mockEvents: Array<{
   data: unknown;
 }> = [];
 
-mock.module("../call-store.js", () => ({
+mock.module("../calls/call-store.js", () => ({
   getCallSession: jest.fn((id: string) => mockSessions.get(id) ?? null),
   updateCallSession: jest.fn((id: string, updates: Record<string, unknown>) => {
     const session = mockSessions.get(id);
@@ -52,7 +52,7 @@ mock.module("../call-store.js", () => ({
 }));
 
 // Mock the call state machine
-mock.module("../call-state-machine.js", () => ({
+mock.module("../calls/call-state-machine.js", () => ({
   isTerminalState: jest.fn(
     (status: string) =>
       status === "completed" || status === "failed" || status === "cancelled",
@@ -61,7 +61,7 @@ mock.module("../call-state-machine.js", () => ({
 
 // Mock the call state (controller registry)
 const mockControllers = new Map<string, unknown>();
-mock.module("../call-state.js", () => ({
+mock.module("../calls/call-state.js", () => ({
   registerCallController: jest.fn(
     (callSessionId: string, controller: unknown) => {
       mockControllers.set(callSessionId, controller);
@@ -85,12 +85,12 @@ mock.module("../call-state.js", () => ({
 }));
 
 // Mock the finalize-call module
-mock.module("../finalize-call.js", () => ({
+mock.module("../calls/finalize-call.js", () => ({
   finalizeCall: jest.fn(),
 }));
 
 // Mock the call pointer messages
-mock.module("../call-pointer-messages.js", () => ({
+mock.module("../calls/call-pointer-messages.js", () => ({
   postPointerMessageSafe: jest.fn(),
   formatDuration: jest.fn((ms: number) => `${Math.round(ms / 1000)}s`),
 }));
@@ -104,13 +104,10 @@ const mockHandleInterrupt = jest.fn();
 const mockDestroy = jest.fn();
 
 // Mirrors CallController.handleBargeIn: invokes onAccepted only when the
-// barge-in passes the controller's gate.
+// barge-in passes the speaking gate.
 const mockHandleBargeIn = jest.fn((_onAccepted?: () => void) => false);
-// What the mocked controller reports as its state; the media-stream
-// session arms the barge-in guard only while a turn is in flight.
-let mockControllerState: "idle" | "processing" | "speaking" = "idle";
 
-mock.module("../call-controller.js", () => ({
+mock.module("../calls/call-controller.js", () => ({
   CallController: jest.fn().mockImplementation(() => ({
     startInitialGreeting: mockStartInitialGreeting,
     startPostVerificationGreeting: mockStartPostVerificationGreeting,
@@ -119,7 +116,7 @@ mock.module("../call-controller.js", () => ({
     handleInterrupt: mockHandleInterrupt,
     handleBargeIn: mockHandleBargeIn,
     destroy: mockDestroy,
-    getState: jest.fn(() => mockControllerState),
+    getState: jest.fn(() => "idle"),
     setTrustContext: jest.fn(),
     getPendingConsultationQuestionId: jest.fn(),
     handleUserAnswer: jest.fn(),
@@ -128,7 +125,7 @@ mock.module("../call-controller.js", () => ({
 }));
 
 // Mock the assistant scope
-mock.module("../../runtime/assistant-scope.js", () => ({
+mock.module("../runtime/assistant-scope.js", () => ({
   DAEMON_INTERNAL_ASSISTANT_ID: "self",
 }));
 
@@ -160,7 +157,7 @@ let mockRouteSetupResult: {
 // teardown (e.g. the outbound unusable-verdict abort).
 let mockRouteSetupError: Error | null = null;
 
-mock.module("../call-setup-router.js", () => ({
+mock.module("../calls/call-setup-router.js", () => ({
   routeSetup: jest.fn(() => {
     if (mockRouteSetupError) {
       throw mockRouteSetupError;
@@ -224,7 +221,7 @@ const mockReadPhoneCallerTrust = jest.fn(
     };
   },
 );
-mock.module("../inbound-trust-reader.js", () => ({
+mock.module("../calls/inbound-trust-reader.js", () => ({
   readPhoneCallerTrust: mockReadPhoneCallerTrust,
   getPhoneCallerVerdict: (otherPartyNumber: string | undefined) =>
     mockGetInboundTrustVerdict({
@@ -234,7 +231,7 @@ mock.module("../inbound-trust-reader.js", () => ({
 }));
 
 // Mock the actor trust resolver (used by handleStart to derive trust context)
-mock.module("../../runtime/actor-trust-resolver.js", () => ({
+mock.module("../runtime/actor-trust-resolver.js", () => ({
   toTrustContext: jest.fn(() => ({
     sourceChannel: "phone",
     trustClass: "guardian",
@@ -246,19 +243,19 @@ mock.module("../../runtime/actor-trust-resolver.js", () => ({
 }));
 
 // Mock the call speech output (speakSystemPrompt used in deny/unsupported paths)
-mock.module("../call-speech-output.js", () => ({
+mock.module("../calls/call-speech-output.js", () => ({
   speakSystemPrompt: jest.fn(async () => {}),
 }));
 
 // Mock scoped approval grants (used in handleTransportClosed and early teardown)
-mock.module("../../approvals/scoped-approval-grants.js", () => ({
+mock.module("../approvals/scoped-approval-grants.js", () => ({
   revokeScopedApprovalGrantsForContext: jest.fn(),
 }));
 
 // Mock the TTS provider resolution so that the dynamic import inside
 // MediaStreamOutput.processSynthesizeItem() doesn't pull in the real
 // config/provider chain (which would hang or error in a test environment).
-mock.module("../resolve-call-tts-provider.js", () => ({
+mock.module("../calls/resolve-call-tts-provider.js", () => ({
   resolveCallTtsProvider: jest.fn(() => ({
     provider: null,
     useSynthesizedPath: false,
@@ -277,10 +274,7 @@ mock.module("../resolve-call-tts-provider.js", () => ({
 let mockTtsPlaybackDelayMs = 5;
 let mockAccessRequestPollIntervalMs = 5;
 let mockUserConsultationTimeoutMs = 500;
-import * as realCallConstants from "../call-constants.js";
-
-mock.module("../call-constants.js", () => ({
-  ...realCallConstants,
+mock.module("../calls/call-constants.js", () => ({
   isDeniedNumber: jest.fn(() => false),
   getMaxCallDurationMs: jest.fn(() => 3_600_000),
   getUserConsultationTimeoutMs: jest.fn(() => mockUserConsultationTimeoutMs),
@@ -304,7 +298,7 @@ const mockGetGuardianDelivery = jest.fn(async () => [
   { channelType: "phone", status: "active", displayName: "Alex" },
 ]);
 const mockGetGuardianDeliveryFresh = jest.fn(async () => []);
-mock.module("../../contacts/guardian-delivery-reader.js", () => ({
+mock.module("../contacts/guardian-delivery-reader.js", () => ({
   getGuardianDelivery: mockGetGuardianDelivery,
   getGuardianDeliveryFresh: mockGetGuardianDeliveryFresh,
   peekCachedGuardianDelivery: jest.fn(() => null),
@@ -315,20 +309,20 @@ mock.module("../../contacts/guardian-delivery-reader.js", () => ({
 }));
 
 // Guardian/assistant display labels (filesystem-backed).
-mock.module("../../prompts/user-reference.js", () => ({
+mock.module("../prompts/user-reference.js", () => ({
   DEFAULT_USER_REFERENCE: "my human",
   resolveGuardianName: jest.fn(
     (primed?: string | null) => primed ?? "my human",
   ),
 }));
-mock.module("../../daemon/identity-helpers.js", () => ({
+mock.module("../daemon/identity-helpers.js", () => ({
   getAssistantName: jest.fn(() => "Aria"),
   resolveUserName: jest.fn(() => null),
 }));
 
 // Conversation persistence (used by callee verification code posting).
 const mockAddMessage = jest.fn(async () => ({}));
-mock.module("../../persistence/conversation-crud.js", () => ({
+mock.module("../persistence/conversation-crud.js", () => ({
   addMessage: mockAddMessage,
 }));
 
@@ -368,7 +362,7 @@ let mockInviteResult: MockInviteResult = {
   type: "trusted_contact",
 };
 const mockAttemptInviteCodeRedemption = jest.fn(async () => mockInviteResult);
-mock.module("../call-verification.js", () => ({
+mock.module("../calls/call-verification.js", () => ({
   attemptVerificationCode: mockAttemptVerificationCode,
   attemptInviteCodeRedemption: mockAttemptInviteCodeRedemption,
   parseDigitsFromSpeech: jest.fn((text: string) => text.replace(/\D+/g, "")),
@@ -384,7 +378,7 @@ let mockNotifyResult: MockNotifyResult = {
   requestId: "req-1",
 };
 const mockNotifyGuardianOfAccessRequest = jest.fn(async () => mockNotifyResult);
-mock.module("../../runtime/access-request-helper.js", () => ({
+mock.module("../runtime/access-request-helper.js", () => ({
   notifyGuardianOfAccessRequest: mockNotifyGuardianOfAccessRequest,
 }));
 
@@ -395,8 +389,8 @@ const mockGetGuardianRequest = jest.fn(async () => mockGuardianRequest);
 // factory does not stub keep resolving (a partial factory breaks at
 // import time when the graph gains a new named import).
 const actualGatewayGuardianRequests =
-  await import("../../channels/gateway-guardian-requests.js");
-mock.module("../../channels/gateway-guardian-requests.js", () => ({
+  await import("../channels/gateway-guardian-requests.js");
+mock.module("../channels/gateway-guardian-requests.js", () => ({
   ...actualGatewayGuardianRequests,
   getGuardianRequestOrNull: mockGetGuardianRequest,
 }));
@@ -408,7 +402,7 @@ const mockEmitCallbackHandoff = jest.fn(
     notified: false,
   }),
 );
-mock.module("../access-request-wait.js", () => ({
+mock.module("../calls/access-request-wait.js", () => ({
   classifyWaitUtterance: jest.fn(() => "neutral"),
   scheduleNextHeartbeat: jest.fn(() => null),
   emitAccessRequestCallbackHandoff: mockEmitCallbackHandoff,
@@ -420,21 +414,21 @@ mock.module("../access-request-wait.js", () => ({
 
 import { GATEWAY_TUNNEL_LOST_WS_CLOSE_CODE } from "@vellumai/service-contracts/ingress";
 
-import { revokeScopedApprovalGrantsForContext } from "../../approvals/scoped-approval-grants.js";
-import { CallController } from "../call-controller.js";
-import { postPointerMessageSafe } from "../call-pointer-messages.js";
-import { routeSetup } from "../call-setup-router.js";
-import { speakSystemPrompt } from "../call-speech-output.js";
+import { revokeScopedApprovalGrantsForContext } from "../approvals/scoped-approval-grants.js";
+import { CallController } from "../calls/call-controller.js";
+import { postPointerMessageSafe } from "../calls/call-pointer-messages.js";
+import { routeSetup } from "../calls/call-setup-router.js";
+import { speakSystemPrompt } from "../calls/call-speech-output.js";
 import {
   fireCallTranscriptNotifier,
   registerCallController,
-} from "../call-state.js";
-import { recordCallEvent, updateCallSession } from "../call-store.js";
-import { finalizeCall } from "../finalize-call.js";
+} from "../calls/call-state.js";
+import { recordCallEvent, updateCallSession } from "../calls/call-store.js";
+import { finalizeCall } from "../calls/finalize-call.js";
 import {
   activeMediaStreamSessions,
   MediaStreamCallSession,
-} from "../media-stream-server.js";
+} from "../calls/media-stream-server.js";
 
 // ---------------------------------------------------------------------------
 // Mock WebSocket factory
@@ -563,7 +557,6 @@ beforeEach(() => {
   mockHandleInterrupt.mockClear();
   mockHandleBargeIn.mockClear();
   mockHandleBargeIn.mockReturnValue(false);
-  mockControllerState = "idle";
   mockDestroy.mockClear();
   (CallController as unknown as jest.Mock).mockClear();
   (registerCallController as jest.Mock).mockClear();
@@ -1315,23 +1308,28 @@ describe("media-stream setup outcome scenarios", () => {
       await session.whenSetupSettled();
       expect(mockStartInitialGreeting).toHaveBeenCalled();
 
-      // Immediate inbound audio (speech-like payloads) before the assistant
-      // has spoken. The controller is idle and nothing is playing, so there
-      // is nothing to interrupt: the guard never arms, however long the
-      // caller talks, and the controller is never consulted.
+      // Immediate inbound audio (speech-like payloads) — before the
+      // assistant has spoken. The speech detector classifies these as
+      // speech, so onSpeechStart fires and calls handleBargeIn. Since
+      // the controller mock returns false (not speaking), handleInterrupt
+      // should NOT be called.
       const speechPayload = Buffer.alloc(160, 0x00).toString("base64");
-      for (let i = 1; i <= 16; i++) {
-        session.handleMessage(makeMediaMessage(speechPayload, String(i)));
-      }
-      expect(mockHandleBargeIn).not.toHaveBeenCalled();
+      session.handleMessage(makeMediaMessage(speechPayload, "1"));
+      session.handleMessage(makeMediaMessage(speechPayload, "2"));
+      session.handleMessage(makeMediaMessage(speechPayload, "3"));
+
+      // handleBargeIn was called but returned false
+      expect(mockHandleBargeIn).toHaveBeenCalled();
       expect(mockHandleInterrupt).not.toHaveBeenCalled();
 
-      // Nothing was audible, so nothing was cleared: the queued greeting
-      // still plays.
+      // An ignored barge-in flushes only Twilio's buffered audio (to
+      // stop a completed turn's tail talking over the caller). The
+      // internal playback queue is untouched, so the queued greeting
+      // still plays — handleInterrupt must not have run.
       const clearCommands = mockWs.sent.filter(
         (s) => JSON.parse(s).event === "clear",
       );
-      expect(clearCommands.length).toBe(0);
+      expect(clearCommands.length).toBeGreaterThan(0);
 
       // voice_session_aborted should NOT appear in recorded events
       const abortEvents = mockEvents.filter(
@@ -1351,7 +1349,6 @@ describe("media-stream setup outcome scenarios", () => {
         onAccepted?.();
         return true;
       });
-      mockControllerState = "speaking";
 
       const mockWs = createMockWs();
       mockSessions.set("call-bargein-2", {
@@ -1367,17 +1364,14 @@ describe("media-stream setup outcome scenarios", () => {
       session.handleMessage(makeStartMessage());
       await session.whenSetupSettled();
 
-      // Sustained inbound speech while the assistant is speaking. A
-      // high-amplitude mu-law payload trips speech detection on every
-      // 20 ms frame; the guard fires once 250 ms have accumulated.
+      // Simulate inbound speech audio while assistant is speaking.
+      // Use a high-amplitude mu-law payload so speech detection triggers.
       const speechPayload = Buffer.alloc(160, 0x00).toString("base64");
-      for (let i = 1; i <= 13; i++) {
-        session.handleMessage(makeMediaMessage(speechPayload, String(i)));
-      }
+      session.handleMessage(makeMediaMessage(speechPayload, "1"));
 
-      // handleBargeIn was called once (returning true), and the accepted
-      // barge-in flushed outbound audio via the onAccepted hook.
-      expect(mockHandleBargeIn).toHaveBeenCalledTimes(1);
+      // handleBargeIn should have been called (returning true), and the
+      // accepted barge-in flushes outbound audio via the onAccepted hook.
+      expect(mockHandleBargeIn).toHaveBeenCalled();
       const clearCommands = mockWs.sent.filter(
         (s) => JSON.parse(s).event === "clear",
       );
@@ -1385,93 +1379,6 @@ describe("media-stream setup outcome scenarios", () => {
 
       session.destroy();
     });
-
-    test("speech from the caller's own utterance does not carry into a turn that starts mid-utterance", async () => {
-      mockHandleBargeIn.mockImplementation((onAccepted?: () => void) => {
-        onAccepted?.();
-        return true;
-      });
-
-      const mockWs = createMockWs();
-      mockSessions.set("call-bargein-4", {
-        id: "call-bargein-4",
-        conversationId: "conv-bargein-4",
-        status: "in_progress",
-        task: null,
-        startedAt: Date.now() - 5000,
-        toNumber: "+15555550198",
-      });
-
-      const session = new MediaStreamCallSession(mockWs.ws, "call-bargein-4");
-      session.handleMessage(makeStartMessage());
-      await session.whenSetupSettled();
-
-      // 240 ms of the caller's own utterance while the controller is idle.
-      const speechPayload = Buffer.alloc(160, 0x00).toString("base64");
-      let chunk = 1;
-      for (let i = 0; i < 12; i++) {
-        session.handleMessage(makeMediaMessage(speechPayload, String(chunk++)));
-      }
-      // A streaming final starts the assistant turn before the local VAD
-      // ends the utterance. One more speech frame must not cut it off.
-      mockControllerState = "processing";
-      session.handleMessage(makeMediaMessage(speechPayload, String(chunk++)));
-      expect(mockHandleBargeIn).not.toHaveBeenCalled();
-
-      // Sustained speech during the turn still does.
-      for (let i = 0; i < 12; i++) {
-        session.handleMessage(makeMediaMessage(speechPayload, String(chunk++)));
-      }
-      expect(mockHandleBargeIn).toHaveBeenCalledTimes(1);
-
-      session.destroy();
-    });
-
-    test("speech that stops short of the guard never reaches the controller", async () => {
-      mockHandleBargeIn.mockImplementation((onAccepted?: () => void) => {
-        onAccepted?.();
-        return true;
-      });
-      mockControllerState = "processing";
-
-      const mockWs = createMockWs();
-      mockSessions.set("call-bargein-3", {
-        id: "call-bargein-3",
-        conversationId: "conv-bargein-3",
-        status: "in_progress",
-        task: null,
-        startedAt: Date.now() - 5000,
-        toNumber: "+15555550199",
-      });
-
-      const session = new MediaStreamCallSession(mockWs.ws, "call-bargein-3");
-      session.handleMessage(makeStartMessage());
-      await session.whenSetupSettled();
-
-      // 100 ms of speech, then a silence gap past the tolerance: the run
-      // resets, and another 100 ms of speech is still short of the guard.
-      const speechPayload = Buffer.alloc(160, 0x00).toString("base64");
-      const silencePayload = Buffer.alloc(160, 0xff).toString("base64");
-      let chunk = 1;
-      for (let i = 0; i < 5; i++) {
-        session.handleMessage(makeMediaMessage(speechPayload, String(chunk++)));
-      }
-      for (let i = 0; i < 15; i++) {
-        session.handleMessage(
-          makeMediaMessage(silencePayload, String(chunk++)),
-        );
-      }
-      for (let i = 0; i < 5; i++) {
-        session.handleMessage(makeMediaMessage(speechPayload, String(chunk++)));
-      }
-
-      expect(mockHandleBargeIn).not.toHaveBeenCalled();
-      expect(
-        mockWs.sent.filter((s) => JSON.parse(s).event === "clear").length,
-      ).toBe(0);
-
-      session.destroy();
-    });
   });
 
   // ── E2E regression scenario ──────────────────────────────────────

package/src/{calls/__tests__ → __tests__}/media-stream-stt-session.test.ts

@@ -13,27 +13,27 @@ import type {
   StreamingTranscriber,
   SttStreamServerEvent,
   SttTranscribeRequest,
-} from "../../stt/types.js";
+} from "../stt/types.js";
 
 // ---------------------------------------------------------------------------
 // Module mocks — must be declared before the module under test is imported.
 // ---------------------------------------------------------------------------
 
 // Mock the STT resolve module
-mock.module("../../providers/speech-to-text/resolve.js", () => ({
+mock.module("../providers/speech-to-text/resolve.js", () => ({
   resolveTelephonySttCapability: jest.fn(),
   resolveBatchTranscriber: jest.fn(),
   resolveStreamingTranscriber: jest.fn(),
 }));
 
 // Now import the mocked modules and the module under test.
-import { setConfig } from "../../__tests__/helpers/set-config.js";
+import { MediaStreamSttSession } from "../calls/media-stream-stt-session.js";
 import {
   resolveBatchTranscriber,
   resolveStreamingTranscriber,
   resolveTelephonySttCapability,
-} from "../../providers/speech-to-text/resolve.js";
-import { MediaStreamSttSession } from "../media-stream-stt-session.js";
+} from "../providers/speech-to-text/resolve.js";
+import { setConfig } from "./helpers/set-config.js";
 
 /**
  * Seed the session's `calls.voice` settings in the real workspace config.

package/src/{calls/__tests__ → __tests__}/media-turn-detector.test.ts

@@ -1,6 +1,6 @@
 import { afterEach, beforeEach, describe, expect, jest, test } from "bun:test";
 
-import { MediaTurnDetector } from "../media-turn-detector.js";
+import { MediaTurnDetector } from "../calls/media-turn-detector.js";
 
 // ---------------------------------------------------------------------------
 // Helpers

package/src/__tests__/messages-after-tiebreaker.test.ts

@@ -1,14 +1,11 @@
 import { beforeEach, describe, expect, test } from "bun:test";
 
-import { eq } from "drizzle-orm";
-
 import {
   countMessagesAfter,
   getMessagesAfter,
 } from "../persistence/conversation-crud.js";
 import { getDb } from "../persistence/db-connection.js";
 import { initializeDb } from "../persistence/db-init.js";
-import { resolveMessagesAfterBound } from "../persistence/message-cursor.js";
 import { conversations, messages } from "../persistence/schema/index.js";
 
 await initializeDb();
@@ -116,91 +113,3 @@ describe("countMessagesAfter / getMessagesAfter — millisecond-collision tie-br
     expect(getMessagesAfter(CONV_ID, "")).toHaveLength(2);
   });
 });
-
-describe("countMessagesAfter / getMessagesAfter: a cursor survives its row's deletion", () => {
-  beforeEach(() => {
-    clearDb();
-    seedConversation();
-  });
-
-  test("a cursor carrying createdAt keeps bounding the read after its row is deleted", () => {
-    const ts = 1_700_000_000_000;
-    insertMessage("a", ts);
-    // The row the cursor points at: a reply the user then regenerates.
-    insertMessage("b", ts + 1);
-    insertMessage("c", ts + 2);
-    getDb().delete(messages).where(eq(messages.id, "b")).run();
-    // The regenerated reply.
-    insertMessage("d", ts + 3);
-
-    const cursor = { id: "b", createdAt: ts + 1 };
-    expect(countMessagesAfter(CONV_ID, cursor)).toBe(2);
-    expect(getMessagesAfter(CONV_ID, cursor).map((m) => m.id)).toEqual([
-      "c",
-      "d",
-    ]);
-  });
-
-  test("a cursor without createdAt keeps the conservative id-only semantics once its row is gone", () => {
-    insertMessage("a", 1_700_000_000_000);
-
-    const cursor = { id: "nonexistent", createdAt: null };
-    expect(countMessagesAfter(CONV_ID, cursor)).toBe(0);
-    expect(getMessagesAfter(CONV_ID, cursor)).toEqual([]);
-  });
-
-  test("the live row wins over the timestamp the cursor carries", () => {
-    const ts = 1_700_000_000_000;
-    insertMessage("a", ts);
-    insertMessage("b", ts + 10);
-
-    // A stale cursor timestamp must not pull processed rows back in, nor
-    // hide unprocessed ones, while the row itself can still be read.
-    expect(countMessagesAfter(CONV_ID, { id: "b", createdAt: ts - 100 })).toBe(
-      0,
-    );
-    expect(countMessagesAfter(CONV_ID, { id: "a", createdAt: ts + 999 })).toBe(
-      1,
-    );
-  });
-
-  test("a cursor with an empty id reads everything, like null and the empty string", () => {
-    insertMessage("a", 1_700_000_000_000);
-
-    expect(countMessagesAfter(CONV_ID, { id: "", createdAt: 5 })).toBe(1);
-    expect(getMessagesAfter(CONV_ID, { id: "", createdAt: 5 })).toHaveLength(1);
-  });
-});
-
-describe("resolveMessagesAfterBound", () => {
-  beforeEach(() => {
-    clearDb();
-    seedConversation();
-  });
-
-  test("resolves the unbounded, live-row, stored-timestamp, and vanished cases", () => {
-    insertMessage("a", 1_000);
-
-    expect(resolveMessagesAfterBound(null)).toEqual({ kind: "all" });
-    expect(resolveMessagesAfterBound("")).toEqual({ kind: "all" });
-    expect(resolveMessagesAfterBound({ id: "", createdAt: 3 })).toEqual({
-      kind: "all",
-    });
-    expect(resolveMessagesAfterBound("a")).toEqual({
-      kind: "after",
-      bound: { createdAt: 1_000, id: "a" },
-    });
-    expect(resolveMessagesAfterBound({ id: "a", createdAt: 99 })).toEqual({
-      kind: "after",
-      bound: { createdAt: 1_000, id: "a" },
-    });
-    expect(resolveMessagesAfterBound("gone")).toEqual({ kind: "vanished" });
-    expect(resolveMessagesAfterBound({ id: "gone", createdAt: null })).toEqual({
-      kind: "vanished",
-    });
-    expect(resolveMessagesAfterBound({ id: "gone", createdAt: 7 })).toEqual({
-      kind: "after",
-      bound: { createdAt: 7, id: "gone" },
-    });
-  });
-});

package/src/__tests__/messages-read-boundary-guard.test.ts

@@ -52,7 +52,7 @@ const BASELINE: Readonly<Record<string, number>> = {
   "src/plugins/defaults/memory/context-search/sources/conversations.ts": 1,
   "src/plugins/defaults/memory/graph/image-ref-utils.ts": 1,
   "src/plugins/defaults/memory/indexer.ts": 2,
-  "src/plugins/defaults/memory/memory-retrospective-accounting.ts": 2,
+  "src/plugins/defaults/memory/memory-retrospective-accounting.ts": 4,
   "src/plugins/defaults/memory/substrate/sweep-job.ts": 1,
   "src/plugins/defaults/memory/v1/graph/extraction.ts": 3,
   "src/plugins/defaults/memory/v1/job-handlers/backfill.ts": 1,