@vellumai/assistant 0.8.8 → 0.8.9-staging.1

package/src/__tests__/agent-wake-disk-pressure-callsite.test.ts

@@ -22,7 +22,6 @@ import type { Message } from "../providers/types.js";
 
 mock.module("../memory/conversation-crud.js", () => ({
   getConversationOverrideProfile: () => undefined,
-  reserveMessage: mock(async () => ({ id: "msg-reserve" })),
 }));
 
 mock.module("../config/loader.js", () => ({
@@ -62,36 +61,37 @@ mock.module("../daemon/disk-pressure-guard.js", () => ({
   }),
 }));
 
+import type { Conversation } from "../daemon/conversation.js";
 import {
   __resetWakeChainForTests,
   wakeAgentForOpportunity,
-  type WakeTarget,
 } from "../runtime/agent-wake.js";
 
-function makeTarget(): WakeTarget {
-  const history: Message[] = [];
+function makeTarget(): Conversation {
+  const messages: Message[] = [];
   let processing = false;
-  return {
+  const target = {
     conversationId: "conv-wake-callsite",
     agentLoop: {
-      run: (async (messages: Message[]) => ({
+      run: async ({ messages }: { messages: Message[] }) => ({
         history: messages,
         exitReason: null,
         appendedNewMessages: false,
         newMessages: [],
-      })) as WakeTarget["agentLoop"]["run"],
+      }),
     },
-    getMessages: () => history,
-    pushMessage: (msg) => {
-      history.push(msg);
-    },
-    emitAgentEvent: () => {},
+    messages,
+    getMessages: () => messages,
     isProcessing: () => processing,
-    markProcessing: (on) => {
+    setProcessing: (on: boolean) => {
       processing = on;
     },
-    persistTailMessage: async () => {},
+    setTrustContext: () => {},
+    getTurnChannelContext: () => null,
+    getTurnInterfaceContext: () => null,
+    drainQueue: async () => {},
   };
+  return target as unknown as Conversation;
 }
 
 beforeEach(() => {

package/src/__tests__/agent-wake-override-profile.test.ts

@@ -48,8 +48,9 @@ mock.module("../config/loader.js", () => ({
   }),
 }));
 
-import type { AgentEvent, AgentLoopRunOptions } from "../agent/loop.js";
+import type { AgentLoopRunOptions } from "../agent/loop.js";
 import { LLMSchema } from "../config/schemas/llm.js";
+import type { Conversation } from "../daemon/conversation.js";
 import { RetryProvider } from "../providers/retry.js";
 import type {
   Message,
@@ -60,7 +61,6 @@ import type {
 import {
   __resetWakeChainForTests,
   wakeAgentForOpportunity,
-  type WakeTarget,
 } from "../runtime/agent-wake.js";
 
 interface RunArgs {
@@ -69,44 +69,40 @@ interface RunArgs {
 }
 
 function makeTarget(): {
-  target: WakeTarget;
+  target: Conversation;
   runArgs: RunArgs[];
 } {
   const runArgs: RunArgs[] = [];
-  const history: Message[] = [
+  const messages: Message[] = [
     { role: "user", content: [{ type: "text", text: "hi" }] },
   ];
   let processing = false;
 
-  const target: WakeTarget = {
+  const target = {
     conversationId: "conv-wake-override",
     agentLoop: {
-      run: (async (
-        messages: Message[],
-        _onEvent: (event: AgentEvent) => void | Promise<void>,
-        options?: AgentLoopRunOptions,
-      ) => {
+      run: async (options: AgentLoopRunOptions) => {
         runArgs.push({
-          messages: [...messages],
+          messages: [...options.messages],
           options,
         });
         // Return the input verbatim → silent no-op (no assistant tail).
         // Wake never yields at a checkpoint, so the pause-reason is null.
-        return { history: messages, exitReason: null };
-      }) as WakeTarget["agentLoop"]["run"],
-    },
-    getMessages: () => history,
-    pushMessage: (msg) => {
-      history.push(msg);
+        return { history: options.messages, exitReason: null };
+      },
     },
-    emitAgentEvent: () => {},
+    messages,
+    getMessages: () => messages,
     isProcessing: () => processing,
-    markProcessing: (on) => {
+    setProcessing: (on: boolean) => {
       processing = on;
     },
-    persistTailMessage: async () => {},
+    setTrustContext: () => {},
+    getTurnChannelContext: () => null,
+    getTurnInterfaceContext: () => null,
+    drainQueue: async () => {},
   };
-  return { target, runArgs };
+  return { target: target as unknown as Conversation, runArgs };
 }
 
 beforeEach(() => {

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

@@ -102,7 +102,7 @@ mock.module("@anthropic-ai/sdk", () => ({
 }));
 
 // Import after mocking
-import { cachedTextBlock } from "../memory/v3/provider-blocks.js";
+import { cachedTextBlock } from "../plugins/defaults/memory-v3-shadow/provider-blocks.js";
 import { AnthropicProvider } from "../providers/anthropic/client.js";
 import {
   isPlaceholderSentinelText,

package/src/__tests__/app-control-flow.test.ts

@@ -103,7 +103,7 @@ const { ROUTES } = await import("../runtime/routes/host-app-control-routes.js");
 const { surfaceProxyResolver } =
   await import("../daemon/conversation-surfaces.js");
 const { setConversation, clearConversations } =
-  await import("../daemon/conversation-store.js");
+  await import("../daemon/conversation-registry.js");
 type SurfaceConversationContext =
   import("../daemon/conversation-surfaces.js").SurfaceConversationContext;
 

package/src/__tests__/app-dir-path-guard.test.ts

@@ -19,6 +19,7 @@ const ALLOWLIST = new Set([
   "assistant/src/memory/app-store.ts", // defines getAppsDir
   "assistant/src/memory/app-git-service.ts", // uses getAppsDir for git repo root, not per-app paths
   "assistant/src/daemon/app-source-watcher.ts", // uses getAppsDir for recursive fs.watch root, not per-app paths
+  "assistant/src/tools/filesystem/write.ts", // uses getAppsDir as an exemption root for the artifact-HTML guard, not per-app paths
 ]);
 
 function isTestFile(filePath: string): boolean {

package/src/__tests__/approval-cascade.test.ts

@@ -176,6 +176,7 @@ mock.module("../memory/retriever.js", () => ({
 mock.module("../context/window-manager.js", () => ({
   ContextWindowManager: class {
     constructor() {}
+    updateConfig() {}
     shouldCompact() {
       return { needed: false, estimatedTokens: 0 };
     }
@@ -208,10 +209,10 @@ mock.module("../agent/loop.js", () => ({
     getActiveModel() {
       return undefined;
     }
-    async run(
-      _messages: Message[],
-      _onEvent: (event: AgentEvent) => void,
-    ): Promise<Message[]> {
+    async run(_options: {
+      messages: Message[];
+      onEvent: (event: AgentEvent) => void;
+    }): Promise<Message[]> {
       return [];
     }
   },

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

@@ -119,12 +119,15 @@ mock.module("../permissions/trust-store.js", () => ({
 // ---------------------------------------------------------------------------
 let _conversationFactory: (() => Conversation) | undefined;
 
-mock.module("../daemon/conversation-store.js", () => ({
+mock.module("../daemon/conversation-registry.js", () => ({
   findConversation: () => {
     // Return the current test session for any conversation ID lookup.
     if (!_conversationFactory) return undefined;
     return _conversationFactory();
   },
+}));
+
+mock.module("../daemon/conversation-store.js", () => ({
   getOrCreateConversation: async () => {
     if (!_conversationFactory)
       throw new Error("_conversationFactory not set in test");

package/src/__tests__/background-workers-disk-pressure.test.ts

@@ -117,7 +117,7 @@ mock.module("../memory/conversation-crud.js", () => ({
   findMostRecentRetrospectiveFor: mock(() => null),
   forkConversation: mock(() => ({ id: "conv-fork" })),
   getConversationOverrideProfile: () => undefined,
-  getConversationOverrideProfileFromRow: () => undefined,
+  resolveOverrideProfile: () => undefined,
   getConversationMemoryScopeId: () => "default",
   getConversationOriginChannel: () => null,
   getConversationOriginInterface: () => null,

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

@@ -16,7 +16,7 @@ mock.module("../util/logger.js", () => ({
 }));
 
 const _conversationMocks = new Map<string, unknown>();
-mock.module("../daemon/conversation-store.js", () => ({
+mock.module("../daemon/conversation-registry.js", () => ({
   findConversation: (id: string) => _conversationMocks.get(id),
 }));
 

package/src/__tests__/channel-approvals.test.ts

@@ -11,7 +11,7 @@ mock.module("../util/logger.js", () => ({
 
 // Map conversationId → mock session so findConversation returns the right mock.
 const conversationMocks = new Map<string, unknown>();
-mock.module("../daemon/conversation-store.js", () => ({
+mock.module("../daemon/conversation-registry.js", () => ({
   findConversation: (id: string) => conversationMocks.get(id),
 }));
 

package/src/__tests__/compaction-circuit.test.ts

@@ -0,0 +1,258 @@
+/**
+ * Tests for the `CompactionCircuit` class.
+ *
+ * The breaker logic lives as direct methods on the per-conversation
+ * `CompactionCircuit` state container (`agent/compaction-circuit.ts`). These
+ * tests assert the threshold (3 consecutive failures) and cooldown (1 hour)
+ * exactly match the user-visible behavior:
+ *   (a) counter increments on each failure outcome
+ *   (b) circuit opens after exactly 3 consecutive failures
+ *   (c) successful compaction resets counter and clears the circuit
+ *   (d) isOpen() reflects state and cooldown expiry
+ *   (e) circuit re-opens after cooldown expiry when 3 more failures
+ *       accumulate (guards the stale-timestamp regression)
+ *   (f) isOpen() is query-only and never mutates the counter
+ *   (g) open→closed transition emits `compaction_circuit_closed` exactly once
+ *   (h) closed→closed transition emits nothing
+ */
+
+import { afterEach, beforeEach, describe, expect, test } from "bun:test";
+
+import {
+  COMPACTION_CIRCUIT_COOLDOWN_MS,
+  COMPACTION_CIRCUIT_FAILURE_THRESHOLD,
+  CompactionCircuit,
+} from "../agent/compaction-circuit.js";
+import type { CompactionCircuitEvent } from "../plugins/types.js";
+
+// ─── Fixtures ───────────────────────────────────────────────────────────────
+
+const CONVERSATION_ID = "conv-breaker-test";
+
+function collectEvents(): {
+  events: CompactionCircuitEvent[];
+  onEvent: (msg: CompactionCircuitEvent) => void;
+} {
+  const events: CompactionCircuitEvent[] = [];
+  return { events, onEvent: (msg) => events.push(msg) };
+}
+
+describe("CompactionCircuit", () => {
+  let originalDateNow: () => number;
+
+  beforeEach(() => {
+    originalDateNow = Date.now;
+  });
+
+  afterEach(() => {
+    Date.now = originalDateNow;
+  });
+
+  test("threshold and cooldown constants match the user-visible contract", () => {
+    expect(COMPACTION_CIRCUIT_FAILURE_THRESHOLD).toBe(3);
+    expect(COMPACTION_CIRCUIT_COOLDOWN_MS).toBe(60 * 60 * 1000);
+  });
+
+  test("(a) counter increments on each failure outcome", async () => {
+    const circuit = new CompactionCircuit(CONVERSATION_ID);
+    const { onEvent, events } = collectEvents();
+
+    await circuit.recordOutcome(true, onEvent);
+    expect(circuit.consecutiveCompactionFailures).toBe(1);
+    expect(circuit.compactionCircuitOpenUntil).toBeNull();
+    expect(events).toHaveLength(0);
+
+    await circuit.recordOutcome(true, onEvent);
+    expect(circuit.consecutiveCompactionFailures).toBe(2);
+    expect(circuit.compactionCircuitOpenUntil).toBeNull();
+    expect(events).toHaveLength(0);
+  });
+
+  test("(b) circuit opens after exactly 3 consecutive failures", async () => {
+    const fixedNow = 1_700_000_000_000;
+    Date.now = () => fixedNow;
+
+    const circuit = new CompactionCircuit(CONVERSATION_ID);
+    const { onEvent, events } = collectEvents();
+
+    await circuit.recordOutcome(true, onEvent);
+    await circuit.recordOutcome(true, onEvent);
+    // Two failures — circuit still closed.
+    expect(circuit.compactionCircuitOpenUntil).toBeNull();
+    expect(events).toHaveLength(0);
+
+    await circuit.recordOutcome(true, onEvent);
+    // Third failure — circuit trips and fires the event exactly once.
+    expect(circuit.consecutiveCompactionFailures).toBe(3);
+    expect(circuit.compactionCircuitOpenUntil).toBe(fixedNow + 60 * 60 * 1000);
+    expect(await circuit.isOpen()).toBe(true);
+    expect(events).toHaveLength(1);
+    expect(events[0]).toEqual({
+      type: "compaction_circuit_open",
+      conversationId: CONVERSATION_ID,
+      reason: "3_consecutive_failures",
+      openUntil: fixedNow + 60 * 60 * 1000,
+    });
+
+    // Further failures do not re-fire the event while the circuit is open.
+    await circuit.recordOutcome(true, onEvent);
+    expect(circuit.consecutiveCompactionFailures).toBe(4);
+    expect(events).toHaveLength(1);
+  });
+
+  test("(c) successful outcome resets counter and clears circuit", async () => {
+    const fixedNow = 1_700_000_000_000;
+    Date.now = () => fixedNow;
+
+    const circuit = new CompactionCircuit(CONVERSATION_ID);
+    const { onEvent } = collectEvents();
+
+    // Trip the breaker.
+    await circuit.recordOutcome(true, onEvent);
+    await circuit.recordOutcome(true, onEvent);
+    await circuit.recordOutcome(true, onEvent);
+    expect(circuit.compactionCircuitOpenUntil).not.toBeNull();
+
+    // Success resets state.
+    await circuit.recordOutcome(false, onEvent);
+    expect(circuit.consecutiveCompactionFailures).toBe(0);
+    expect(circuit.compactionCircuitOpenUntil).toBeNull();
+  });
+
+  test("(d) isOpen() reflects state and expiry", async () => {
+    const fixedNow = 1_700_000_000_000;
+    Date.now = () => fixedNow;
+
+    const circuit = new CompactionCircuit(CONVERSATION_ID);
+    const { onEvent } = collectEvents();
+
+    // Fresh state: closed.
+    expect(await circuit.isOpen()).toBe(false);
+
+    // Trip the breaker.
+    await circuit.recordOutcome(true, onEvent);
+    await circuit.recordOutcome(true, onEvent);
+    await circuit.recordOutcome(true, onEvent);
+
+    // While open.
+    expect(await circuit.isOpen()).toBe(true);
+
+    // After cooldown expires the breaker reports closed again, even without
+    // an explicit reset — the open-until timestamp is the only source of
+    // truth for the gate.
+    Date.now = () => fixedNow + 60 * 60 * 1000 + 1;
+    expect(await circuit.isOpen()).toBe(false);
+  });
+
+  test("(e) circuit re-opens after cooldown expiry when 3 more failures accumulate", async () => {
+    // Regression: opening the breaker a second time must not require
+    // `compactionCircuitOpenUntil === null`. Once a cooldown expires, the
+    // gate correctly reports "closed" but a stale past-timestamp stays on the
+    // state, so the next 3-strike window must still trip a new cooldown. Any
+    // expired timestamp is treated the same as null.
+    const t0 = 1_700_000_000_000;
+    Date.now = () => t0;
+
+    const circuit = new CompactionCircuit(CONVERSATION_ID);
+    const { onEvent, events } = collectEvents();
+
+    // Trip the breaker the first time.
+    await circuit.recordOutcome(true, onEvent);
+    await circuit.recordOutcome(true, onEvent);
+    await circuit.recordOutcome(true, onEvent);
+    expect(circuit.compactionCircuitOpenUntil).toBe(t0 + 60 * 60 * 1000);
+    expect(events).toHaveLength(1);
+
+    // Advance past the cooldown window. Reset the counter — in production this
+    // happens when a subsequent `maybeCompact` call succeeds after the
+    // cooldown elapses, but the bug manifests even when the counter is reset:
+    // the stale `compactionCircuitOpenUntil` is what breaks re-opening.
+    const t1 = t0 + 60 * 60 * 1000 + 1;
+    Date.now = () => t1;
+    expect(await circuit.isOpen()).toBe(false);
+    circuit.consecutiveCompactionFailures = 0;
+    // `compactionCircuitOpenUntil` is deliberately left as the old timestamp
+    // to reproduce the bug condition.
+    expect(circuit.compactionCircuitOpenUntil).toBe(t0 + 60 * 60 * 1000);
+
+    // Three more failures must trip a fresh cooldown even though the old
+    // timestamp is still set.
+    await circuit.recordOutcome(true, onEvent);
+    await circuit.recordOutcome(true, onEvent);
+    await circuit.recordOutcome(true, onEvent);
+    expect(circuit.consecutiveCompactionFailures).toBe(3);
+    expect(circuit.compactionCircuitOpenUntil).toBe(t1 + 60 * 60 * 1000);
+    expect(events).toHaveLength(2);
+    expect(events[1]).toEqual({
+      type: "compaction_circuit_open",
+      conversationId: CONVERSATION_ID,
+      reason: "3_consecutive_failures",
+      openUntil: t1 + 60 * 60 * 1000,
+    });
+  });
+
+  test("(f) isOpen() is query-only and never mutates the counter", async () => {
+    // `maybeCompact()` early-return paths skip `recordOutcome` entirely and
+    // only gate on `isOpen()`. The query must never touch the 3-strike
+    // counter so those early returns can't silently reset it.
+    const circuit = new CompactionCircuit(CONVERSATION_ID);
+    const { onEvent } = collectEvents();
+
+    await circuit.recordOutcome(true, onEvent);
+    await circuit.recordOutcome(true, onEvent);
+    expect(circuit.consecutiveCompactionFailures).toBe(2);
+
+    // Query-only — must NOT touch the counter.
+    expect(await circuit.isOpen()).toBe(false);
+    expect(circuit.consecutiveCompactionFailures).toBe(2);
+
+    // A third real failure then trips the breaker as expected.
+    await circuit.recordOutcome(true, onEvent);
+    expect(circuit.consecutiveCompactionFailures).toBe(3);
+    expect(circuit.compactionCircuitOpenUntil).not.toBeNull();
+  });
+
+  test("(g) open→closed transition emits compaction_circuit_closed exactly once", async () => {
+    // Regression: the reset branch must notify the client on open→closed.
+    // Otherwise the Swift banner set from `compaction_circuit_open` stays
+    // visible until the original `openUntil` deadline (up to 1h),
+    // misrepresenting the live state.
+    const fixedNow = 1_700_000_000_000;
+    Date.now = () => fixedNow;
+
+    const circuit = new CompactionCircuit(CONVERSATION_ID);
+    const { onEvent, events } = collectEvents();
+
+    // Force the circuit into the open state directly — the emitted-event
+    // transition logic is what we're testing, not the tripping path.
+    circuit.compactionCircuitOpenUntil = fixedNow + 60 * 60 * 1000;
+    circuit.consecutiveCompactionFailures = 3;
+
+    await circuit.recordOutcome(false, onEvent);
+
+    expect(circuit.consecutiveCompactionFailures).toBe(0);
+    expect(circuit.compactionCircuitOpenUntil).toBeNull();
+    expect(events).toHaveLength(1);
+    expect(events[0]).toEqual({
+      type: "compaction_circuit_closed",
+      conversationId: CONVERSATION_ID,
+    });
+  });
+
+  test("(h) successful outcome against an already-closed circuit emits no event", async () => {
+    // Emitting `compaction_circuit_closed` on every successful compaction
+    // would spam the client (the breaker is closed in the common case).
+    // Only the open→closed transition is meaningful.
+    const circuit = new CompactionCircuit(CONVERSATION_ID);
+    const { onEvent, events } = collectEvents();
+
+    expect(circuit.compactionCircuitOpenUntil).toBeNull();
+    await circuit.recordOutcome(false, onEvent);
+    expect(circuit.compactionCircuitOpenUntil).toBeNull();
+    expect(events).toHaveLength(0);
+
+    // A second successful outcome while still closed — still no event.
+    await circuit.recordOutcome(false, onEvent);
+    expect(events).toHaveLength(0);
+  });
+});

package/src/__tests__/compaction-direct.test.ts

@@ -0,0 +1,132 @@
+/**
+ * Tests for the default compaction plugin (`defaultCompact`).
+ *
+ * The agent loop calls {@link defaultCompact} directly with a
+ * {@link CompactionContext} rather than routing through a middleware pipeline.
+ * These tests assert that the default implementation delegates to the
+ * supplied {@link ContextWindowManager} and forwards the request's
+ * conversational options verbatim. The orchestrator integration path
+ * (conversation-agent-loop) is exercised by
+ * `conversation-agent-loop-overflow.test.ts`.
+ */
+
+import { describe, expect, test } from "bun:test";
+
+import {
+  type CompactionContext,
+  defaultCompact,
+} from "../plugins/defaults/compaction/compact.js";
+
+type ContextWindowResultShape = {
+  compacted: boolean;
+  summaryText: string;
+  messages: unknown[];
+  previousEstimatedInputTokens: number;
+  estimatedInputTokens: number;
+  maxInputTokens: number;
+  thresholdTokens: number;
+  compactedMessages: number;
+  compactedPersistedMessages: number;
+  summaryCalls: number;
+  summaryInputTokens: number;
+  summaryOutputTokens: number;
+  summaryModel: string;
+  reason?: string;
+};
+
+function makeResult(
+  overrides: Partial<ContextWindowResultShape> = {},
+): ContextWindowResultShape {
+  return {
+    compacted: true,
+    summaryText: "default-summary",
+    messages: [],
+    previousEstimatedInputTokens: 1000,
+    estimatedInputTokens: 100,
+    maxInputTokens: 100000,
+    thresholdTokens: 80000,
+    compactedMessages: 3,
+    compactedPersistedMessages: 3,
+    summaryCalls: 1,
+    summaryInputTokens: 500,
+    summaryOutputTokens: 120,
+    summaryModel: "default-model",
+    ...overrides,
+  };
+}
+
+function makeManager(result: ContextWindowResultShape) {
+  const observed: {
+    messages: unknown;
+    signal: unknown;
+    options: unknown;
+  }[] = [];
+  const manager = {
+    maybeCompact: async (
+      messages: unknown,
+      signal: unknown,
+      options: unknown,
+    ) => {
+      observed.push({ messages, signal, options });
+      return result;
+    },
+  } as unknown as CompactionContext["manager"];
+  return { manager, observed };
+}
+
+describe("defaultCompact", () => {
+  test("delegates to the manager and returns its result unchanged", async () => {
+    // GIVEN a manager whose maybeCompact records its arguments
+    const expected = makeResult({
+      summaryText: "manager-summary",
+      compactedMessages: 7,
+    });
+    const { manager, observed } = makeManager(expected);
+    const messages = [{ role: "user", content: "hi" }] as never;
+    const signal = new AbortController().signal;
+
+    // WHEN defaultCompact runs with those messages and a signal
+    const result = (await defaultCompact({
+      manager,
+      messages,
+      signal,
+    })) as unknown as ContextWindowResultShape;
+
+    // THEN the manager saw the same messages and signal
+    expect(observed).toHaveLength(1);
+    expect(observed[0]!.messages).toBe(messages);
+    expect(observed[0]!.signal).toBe(signal);
+
+    // AND the returned result is the manager's object, unmodified
+    expect(result).toBe(expected);
+    expect(result.summaryText).toBe("manager-summary");
+    expect(result.compactedMessages).toBe(7);
+  });
+
+  test("forwards the request's compaction options to the manager", async () => {
+    // GIVEN a manager and a fully-populated compaction context
+    const { manager, observed } = makeManager(makeResult());
+
+    // WHEN defaultCompact runs with force/profile/trust options
+    await defaultCompact({
+      manager,
+      messages: [] as never,
+      force: true,
+      overrideProfile: "fast-profile",
+      precomputedEstimate: 1234,
+      minKeepRecentUserTurns: 0,
+      actorTrustClass: "guardian",
+    });
+
+    // THEN the manager received exactly those options, and the manager,
+    // messages, and signal are not leaked into the options bag
+    expect(observed).toHaveLength(1);
+    expect(observed[0]!.options).toEqual({
+      force: true,
+      overrideProfile: "fast-profile",
+      precomputedEstimate: 1234,
+      minKeepRecentUserTurns: 0,
+      actorTrustClass: "guardian",
+    });
+  });
+});

package/src/__tests__/compaction-events.test.ts

@@ -190,8 +190,8 @@ let mockCompactResult: ContextWindowResult = {
 mock.module("../context/window-manager.js", () => ({
   ContextWindowManager: class {
     nonPersistedPrefixCount = 0;
-    summaryIsInjected = false;
     constructor() {}
+    updateConfig() {}
     shouldCompact() {
       return { needed: false, estimatedTokens: 0 };
     }
@@ -228,10 +228,10 @@ mock.module("../agent/loop.js", () => ({
     getActiveModel() {
       return undefined;
     }
-    async run(
-      _messages: Message[],
-      _onEvent: (event: AgentEvent) => void,
-    ): Promise<Message[]> {
+    async run(_options: {
+      messages: Message[];
+      onEvent: (event: AgentEvent) => void;
+    }): Promise<Message[]> {
       return [];
     }
   },

package/src/__tests__/conversation-abort-tool-results.test.ts

@@ -170,6 +170,7 @@ mock.module("../memory/retriever.js", () => ({
 mock.module("../context/window-manager.js", () => ({
   ContextWindowManager: class {
     constructor() {}
+    updateConfig() {}
     shouldCompact() {
       return { needed: false, estimatedTokens: 0 };
     }
@@ -198,10 +199,11 @@ mock.module("../agent/loop.js", () => ({
     getActiveModel() {
       return undefined;
     }
-    async run(
-      messages: Message[],
-      onEvent: (event: AgentEvent) => void,
-    ): Promise<AgentLoopRunResult> {
+    async run(options: {
+      messages: Message[];
+      onEvent: (event: AgentEvent) => void;
+    }): Promise<AgentLoopRunResult> {
+      const { messages, onEvent } = options;
       // Prime the assistant row anchor — production code emits this from
       // `AgentLoop.run` just before `provider.sendMessage`.
       await onEvent({ type: "llm_call_started" });