@vellumai/assistant 0.8.7-dev.202606052232.2ddc989 → 0.8.8

package/src/__tests__/conversation-workspace-cache-state.test.ts

@@ -198,7 +198,6 @@ mock.module("../agent/loop.js", () => ({
 }));
 
 import { Conversation } from "../daemon/conversation.js";
-import { refreshWorkspaceTopLevelContextIfNeeded } from "../daemon/conversation-workspace.js";
 
 // ---------------------------------------------------------------------------
 // Helpers
@@ -250,7 +249,7 @@ describe("Conversation workspace cache state", () => {
   });
 
   test("refreshWorkspaceTopLevelContextIfNeeded populates context and clears dirty", () => {
-    refreshWorkspaceTopLevelContextIfNeeded(conversation);
+    conversation.refreshWorkspaceTopLevelContextIfNeeded();
 
     expect(conversation.isWorkspaceTopLevelDirty()).toBe(false);
     expect(conversation.getWorkspaceTopLevelContext()).not.toBeNull();
@@ -266,10 +265,10 @@ describe("Conversation workspace cache state", () => {
   });
 
   test("refreshWorkspaceTopLevelContextIfNeeded no-ops when not dirty and cache exists", () => {
-    refreshWorkspaceTopLevelContextIfNeeded(conversation);
+    conversation.refreshWorkspaceTopLevelContextIfNeeded();
     const first = conversation.getWorkspaceTopLevelContext();
 
-    refreshWorkspaceTopLevelContextIfNeeded(conversation);
+    conversation.refreshWorkspaceTopLevelContextIfNeeded();
     const second = conversation.getWorkspaceTopLevelContext();
 
     // Same reference — no recomputation
@@ -277,7 +276,7 @@ describe("Conversation workspace cache state", () => {
   });
 
   test("markWorkspaceTopLevelDirty sets dirty flag", () => {
-    refreshWorkspaceTopLevelContextIfNeeded(conversation);
+    conversation.refreshWorkspaceTopLevelContextIfNeeded();
     expect(conversation.isWorkspaceTopLevelDirty()).toBe(false);
 
     conversation.markWorkspaceTopLevelDirty();
@@ -285,10 +284,10 @@ describe("Conversation workspace cache state", () => {
   });
 
   test("refresh after marking dirty produces fresh context", () => {
-    refreshWorkspaceTopLevelContextIfNeeded(conversation);
+    conversation.refreshWorkspaceTopLevelContextIfNeeded();
 
     conversation.markWorkspaceTopLevelDirty();
-    refreshWorkspaceTopLevelContextIfNeeded(conversation);
+    conversation.refreshWorkspaceTopLevelContextIfNeeded();
 
     expect(conversation.getWorkspaceTopLevelContext()).not.toBeNull();
     expect(conversation.getWorkspaceTopLevelContext()!).toContain(
@@ -300,7 +299,7 @@ describe("Conversation workspace cache state", () => {
   test("renders client-reported host env when set on the conversation", () => {
     conversation.hostHomeDir = "/Users/alice";
     conversation.hostUsername = "alice";
-    refreshWorkspaceTopLevelContextIfNeeded(conversation);
+    conversation.refreshWorkspaceTopLevelContextIfNeeded();
 
     const block = conversation.getWorkspaceTopLevelContext();
     expect(block).not.toBeNull();
@@ -310,7 +309,7 @@ describe("Conversation workspace cache state", () => {
 
   test("falls back to daemon os info when client host env is absent", async () => {
     const { homedir, userInfo } = await import("node:os");
-    refreshWorkspaceTopLevelContextIfNeeded(conversation);
+    conversation.refreshWorkspaceTopLevelContextIfNeeded();
 
     const block = conversation.getWorkspaceTopLevelContext();
     expect(block).not.toBeNull();
@@ -321,7 +320,7 @@ describe("Conversation workspace cache state", () => {
   test("re-renders with updated host env after marking dirty", () => {
     conversation.hostHomeDir = "/Users/alice";
     conversation.hostUsername = "alice";
-    refreshWorkspaceTopLevelContextIfNeeded(conversation);
+    conversation.refreshWorkspaceTopLevelContextIfNeeded();
     expect(conversation.getWorkspaceTopLevelContext()!).toContain(
       "Host home directory: /Users/alice",
     );
@@ -329,7 +328,7 @@ describe("Conversation workspace cache state", () => {
     conversation.hostHomeDir = "/Users/bob";
     conversation.hostUsername = "bob";
     conversation.markWorkspaceTopLevelDirty();
-    refreshWorkspaceTopLevelContextIfNeeded(conversation);
+    conversation.refreshWorkspaceTopLevelContextIfNeeded();
 
     const block = conversation.getWorkspaceTopLevelContext();
     expect(block).not.toBeNull();
@@ -345,7 +344,7 @@ describe("Conversation workspace cache state", () => {
     // Simulate a macOS turn populating host env.
     conversation.hostHomeDir = "/Users/alice";
     conversation.hostUsername = "alice";
-    refreshWorkspaceTopLevelContextIfNeeded(conversation);
+    conversation.refreshWorkspaceTopLevelContextIfNeeded();
     expect(conversation.getWorkspaceTopLevelContext()!).toContain(
       "Host home directory: /Users/alice",
     );
@@ -356,7 +355,7 @@ describe("Conversation workspace cache state", () => {
     conversation.hostHomeDir = undefined;
     conversation.hostUsername = undefined;
     conversation.markWorkspaceTopLevelDirty();
-    refreshWorkspaceTopLevelContextIfNeeded(conversation);
+    conversation.refreshWorkspaceTopLevelContextIfNeeded();
 
     const block = conversation.getWorkspaceTopLevelContext();
     expect(block).not.toBeNull();
@@ -382,7 +381,7 @@ describe("Conversation workspace cache state", () => {
     expect(conversation.hostUsername).toBe("alice");
     expect(conversation.isWorkspaceTopLevelDirty()).toBe(true);
 
-    refreshWorkspaceTopLevelContextIfNeeded(conversation);
+    conversation.refreshWorkspaceTopLevelContextIfNeeded();
     const block = conversation.getWorkspaceTopLevelContext();
     expect(block!).toContain("Host home directory: /Users/alice");
     expect(block!).toContain("Host username: alice");
@@ -456,7 +455,7 @@ describe("Conversation workspace cache state", () => {
       hostUsername: "alice",
     });
     // Render once so the dirty flag clears.
-    refreshWorkspaceTopLevelContextIfNeeded(conversation);
+    conversation.refreshWorkspaceTopLevelContextIfNeeded();
     expect(conversation.isWorkspaceTopLevelDirty()).toBe(false);
 
     // Re-apply the same values — dirty flag should remain false so we don't
@@ -477,7 +476,7 @@ describe("Conversation workspace cache state", () => {
       hostHomeDir: "/Users/alice",
       hostUsername: "alice",
     });
-    refreshWorkspaceTopLevelContextIfNeeded(conversation);
+    conversation.refreshWorkspaceTopLevelContextIfNeeded();
     expect(conversation.isWorkspaceTopLevelDirty()).toBe(false);
 
     // New values — should mark dirty so the next render picks them up.
@@ -501,7 +500,7 @@ describe("Conversation workspace cache state", () => {
 
     try {
       const tempConversation = makeConversation(workspaceRoot);
-      refreshWorkspaceTopLevelContextIfNeeded(tempConversation);
+      tempConversation.refreshWorkspaceTopLevelContextIfNeeded();
 
       expect(tempConversation.getWorkspaceTopLevelContext()!).toContain(
         `Current conversation attachments: conversations/${legacyDirName}/attachments/`,

package/src/__tests__/conversation-workspace-injection.test.ts

@@ -295,14 +295,6 @@ mock.module("../memory/canonical-guardian-store.js", () => ({
 }));
 
 import { Conversation } from "../daemon/conversation.js";
-import {
-  clearConversations,
-  findConversation,
-  removeSubagentConversation,
-  setConversation,
-  setSubagentConversation,
-} from "../daemon/conversation-registry.js";
-import { resolveWorkspaceTopLevelContext } from "../daemon/conversation-workspace.js";
 import { resetPluginRegistryAndRegisterDefaults } from "../plugins/defaults/index.js";
 
 function makeConversation(): Conversation {
@@ -317,7 +309,7 @@ function makeConversation(): Conversation {
       };
     },
   };
-  const conversation = new Conversation(
+  return new Conversation(
     "conv-1",
     provider,
     "system prompt",
@@ -325,10 +317,6 @@ function makeConversation(): Conversation {
     "/tmp",
     { maxTokens: 4096 },
   );
-  // Mirror production: top-level conversations are registered in the store, so
-  // the workspace injector can resolve them by id via the conversation registry.
-  setConversation(conversation.conversationId, conversation);
-  return conversation;
 }
 
 function messageText(message: Message): string {
@@ -347,7 +335,6 @@ describe("Conversation workspace injection", () => {
     runCalls = [];
     agentLoopScript = () => {};
     scanCallCount = 0;
-    clearConversations();
     resetPluginRegistryAndRegisterDefaults();
   });
 
@@ -430,58 +417,11 @@ describe("Conversation workspace injection", () => {
   });
 });
 
-describe("Conversation workspace injection — subagents", () => {
-  beforeEach(() => {
-    scanCallCount = 0;
-    clearConversations();
-    resetPluginRegistryAndRegisterDefaults();
-  });
-
-  test("workspace context resolves for subagent conversations not in the store", () => {
-    const provider = {
-      name: "mock",
-      async sendMessage(): Promise<ProviderResponse> {
-        return {
-          content: [],
-          model: "mock",
-          usage: { inputTokens: 0, outputTokens: 0 },
-          stopReason: "end_turn",
-        };
-      },
-    };
-    const subagentId = "subagent-conv-1";
-    const conversation = new Conversation(
-      subagentId,
-      provider,
-      "system prompt",
-      () => {},
-      "/tmp",
-      { maxTokens: 4096 },
-    );
-    setSubagentConversation(subagentId, conversation);
-
-    try {
-      // Subagents live only in the manager's index, never in the
-      // eviction-managed store, so the store lookup must not see them.
-      expect(findConversation(subagentId)).toBeUndefined();
-
-      // The per-conversation workspace lookup still resolves them, so subagent
-      // turns retain workspace grounding.
-      const context = resolveWorkspaceTopLevelContext(subagentId);
-      expect(context).not.toBeNull();
-      expect(context).toContain("Root: /tmp");
-    } finally {
-      removeSubagentConversation(subagentId, conversation);
-    }
-  });
-});
-
 describe("Conversation workspace dirty-refresh E2E", () => {
   beforeEach(() => {
     runCalls = [];
     agentLoopScript = () => {};
     scanCallCount = 0;
-    clearConversations();
     resetPluginRegistryAndRegisterDefaults();
   });
 

package/src/__tests__/conversation-workspace-tool-tracking.test.ts

@@ -273,7 +273,6 @@ mock.module("../memory/canonical-guardian-store.js", () => ({
 }));
 
 import { Conversation } from "../daemon/conversation.js";
-import { refreshWorkspaceTopLevelContextIfNeeded } from "../daemon/conversation-workspace.js";
 
 function makeConversation(): Conversation {
   const provider = {
@@ -311,7 +310,7 @@ describe("Conversation workspace dirty on file mutations", () => {
     await conversation.loadFromDb();
 
     // Prime the cache so dirty=false
-    refreshWorkspaceTopLevelContextIfNeeded(conversation);
+    conversation.refreshWorkspaceTopLevelContextIfNeeded();
     expect(conversation.isWorkspaceTopLevelDirty()).toBe(false);
 
     agentLoopScript = (onEvent) => {
@@ -340,7 +339,7 @@ describe("Conversation workspace dirty on file mutations", () => {
     const conversation = makeConversation();
     await conversation.loadFromDb();
 
-    refreshWorkspaceTopLevelContextIfNeeded(conversation);
+    conversation.refreshWorkspaceTopLevelContextIfNeeded();
     expect(conversation.isWorkspaceTopLevelDirty()).toBe(false);
 
     agentLoopScript = (onEvent) => {
@@ -371,7 +370,7 @@ describe("Conversation workspace dirty on file mutations", () => {
     const conversation = makeConversation();
     await conversation.loadFromDb();
 
-    refreshWorkspaceTopLevelContextIfNeeded(conversation);
+    conversation.refreshWorkspaceTopLevelContextIfNeeded();
     expect(conversation.isWorkspaceTopLevelDirty()).toBe(false);
 
     agentLoopScript = (onEvent) => {
@@ -400,7 +399,7 @@ describe("Conversation workspace dirty on file mutations", () => {
     const conversation = makeConversation();
     await conversation.loadFromDb();
 
-    refreshWorkspaceTopLevelContextIfNeeded(conversation);
+    conversation.refreshWorkspaceTopLevelContextIfNeeded();
     expect(conversation.isWorkspaceTopLevelDirty()).toBe(false);
 
     agentLoopScript = (onEvent) => {
@@ -429,7 +428,7 @@ describe("Conversation workspace dirty on file mutations", () => {
     const conversation = makeConversation();
     await conversation.loadFromDb();
 
-    refreshWorkspaceTopLevelContextIfNeeded(conversation);
+    conversation.refreshWorkspaceTopLevelContextIfNeeded();
     expect(conversation.isWorkspaceTopLevelDirty()).toBe(false);
 
     agentLoopScript = (onEvent) => {
@@ -458,7 +457,7 @@ describe("Conversation workspace dirty on file mutations", () => {
     const conversation = makeConversation();
     await conversation.loadFromDb();
 
-    refreshWorkspaceTopLevelContextIfNeeded(conversation);
+    conversation.refreshWorkspaceTopLevelContextIfNeeded();
     expect(conversation.isWorkspaceTopLevelDirty()).toBe(false);
 
     agentLoopScript = (onEvent) => {

package/src/__tests__/db-acp-history.test.ts

@@ -5,7 +5,6 @@ import { drizzle } from "drizzle-orm/bun-sqlite";
 
 import { getSqliteFrom } from "../memory/db-connection.js";
 import { migrate230AcpSessionHistory } from "../memory/migrations/230-acp-session-history.js";
-import { migrateAcpSessionHistoryCwd } from "../memory/migrations/272-acp-session-history-cwd.js";
 import * as schema from "../memory/schema.js";
 
 function createTestDb() {
@@ -283,103 +282,3 @@ describe("acp_session_history migration", () => {
     expect(row?.id).toBe("history-rerun");
   });
 });
-
-describe("acp_session_history cwd migration (272)", () => {
-  function readColumns(raw: Database): Map<string, ColumnRow> {
-    const columns = raw
-      .query(`PRAGMA table_info(acp_session_history)`)
-      .all() as ColumnRow[];
-    return new Map(columns.map((c) => [c.name, c]));
-  }
-
-  test("adds a nullable cwd TEXT column to an upgraded table", () => {
-    const db = createTestDb();
-    const raw = getSqliteFrom(db);
-    bootstrapCheckpointsTable(raw);
-
-    migrate230AcpSessionHistory(db);
-    expect(readColumns(raw).has("cwd")).toBe(false);
-
-    migrateAcpSessionHistoryCwd(db);
-
-    const cwd = readColumns(raw).get("cwd");
-    expect(cwd).toBeDefined();
-    expect(cwd?.type).toBe("TEXT");
-    expect(cwd?.notnull).toBe(0);
-  });
-
-  test("re-running is a no-op, including with a cleared checkpoint", () => {
-    const db = createTestDb();
-    const raw = getSqliteFrom(db);
-    bootstrapCheckpointsTable(raw);
-
-    migrate230AcpSessionHistory(db);
-    migrateAcpSessionHistoryCwd(db);
-
-    // Second run short-circuits on the completed checkpoint.
-    expect(() => migrateAcpSessionHistoryCwd(db)).not.toThrow();
-
-    // Even with the checkpoint cleared (simulating crash recovery), the
-    // column-existence guard makes the ALTER a no-op.
-    raw
-      .query(`DELETE FROM memory_checkpoints WHERE key = ?`)
-      .run("migration_acp_session_history_cwd_v1");
-    expect(() => migrateAcpSessionHistoryCwd(db)).not.toThrow();
-
-    const columns = raw
-      .query(`PRAGMA table_info(acp_session_history)`)
-      .all() as ColumnRow[];
-    expect(columns.filter((c) => c.name === "cwd")).toHaveLength(1);
-  });
-
-  test("legacy rows read back with cwd null; new rows round-trip cwd", () => {
-    const db = createTestDb();
-    const raw = getSqliteFrom(db);
-    bootstrapCheckpointsTable(raw);
-
-    migrate230AcpSessionHistory(db);
-
-    // Legacy row inserted before the cwd column existed.
-    raw
-      .query(
-        /*sql*/ `
-        INSERT INTO acp_session_history (
-          id, agent_id, acp_session_id, parent_conversation_id,
-          started_at, status
-        ) VALUES (?, ?, ?, ?, ?, ?)
-      `,
-      )
-      .run("legacy-1", "agent-old", "acp-old", "conv-old", 100, "completed");
-
-    migrateAcpSessionHistoryCwd(db);
-
-    const legacy = raw
-      .query(`SELECT cwd FROM acp_session_history WHERE id = 'legacy-1'`)
-      .get() as { cwd: string | null } | null;
-    expect(legacy?.cwd).toBeNull();
-
-    raw
-      .query(
-        /*sql*/ `
-        INSERT INTO acp_session_history (
-          id, agent_id, acp_session_id, parent_conversation_id,
-          started_at, status, cwd
-        ) VALUES (?, ?, ?, ?, ?, ?, ?)
-      `,
-      )
-      .run(
-        "new-1",
-        "agent-new",
-        "acp-new",
-        "conv-new",
-        200,
-        "completed",
-        "/Users/me/project",
-      );
-
-    const fresh = raw
-      .query(`SELECT cwd FROM acp_session_history WHERE id = 'new-1'`)
-      .get() as { cwd: string | null } | null;
-    expect(fresh?.cwd).toBe("/Users/me/project");
-  });
-});

package/src/__tests__/dynamic-page-surface.test.ts

@@ -94,37 +94,6 @@ describe("ui_show dynamic_page app substitute guard", () => {
     expect(proxied).toBe(false);
   });
 
-  test("rejects dynamic_page with a clean title but substantial interactive html", async () => {
-    let proxied = false;
-
-    const result = await uiShowTool.execute(
-      {
-        surface_type: "dynamic_page",
-        title: "Labor Market Stats",
-        data: {
-          html:
-            "<div id='root'></div><script>" +
-            "const data=[/*...*/];".padEnd(2100, "/") +
-            "new Chart(document.getElementById('root'), {});</script>",
-          preview: { title: "Labor Market Stats" },
-        },
-      },
-      {
-        conversationId: "conversation-123",
-        workingDir: "/tmp",
-        trustClass: "guardian",
-        proxyToolResolver: async () => {
-          proxied = true;
-          return { content: "proxied", isError: false };
-        },
-      },
-    );
-
-    expect(result.isError).toBe(true);
-    expect(result.content).toContain('skill: "app-builder"');
-    expect(proxied).toBe(false);
-  });
-
   test("allows transient non-app dynamic_page surfaces", async () => {
     let proxied = false;
 

package/src/__tests__/file-write-tool.test.ts

@@ -260,66 +260,3 @@ describe("file_write tool PKB re-index hook", () => {
     expect(existsSync(join(workingDir, "pkb", "oops.md"))).toBe(true);
   });
 });
-
-describe("file_write artifact-HTML guard", () => {
-  test("rejects a self-contained interactive HTML visualization", async () => {
-    const dir = makeTempDir();
-    const html =
-      "<!doctype html><html><head><title>Food Market</title></head>" +
-      "<body><canvas id='c'></canvas><script>" +
-      ("const data=[{x:1,y:2}];").padEnd(4000, "/") +
-      "new Chart(document.getElementById('c'), {data});</script></body></html>";
-
-    const result = await fileWriteTool.execute(
-      { path: "food-market-stats.html", content: html },
-      makeContext(dir),
-    );
-
-    expect(result.isError).toBe(true);
-    expect(result.content).toContain('skill: "app-builder"');
-    expect(existsSync(join(dir, "food-market-stats.html"))).toBe(false);
-  });
-
-  test("allows app-builder's thin shell index.html (external module script)", async () => {
-    const dir = makeTempDir();
-    const shell =
-      "<!doctype html><html><head>" +
-      "<link rel='stylesheet' href='/src/styles.css'>".padEnd(3200, " ") +
-      "</head><body><div id='app'></div>" +
-      "<script type='module' src='/src/main.tsx'></script></body></html>";
-
-    const result = await fileWriteTool.execute(
-      { path: "src/index.html", content: shell },
-      makeContext(dir),
-    );
-
-    expect(result.isError).toBe(false);
-    expect(existsSync(join(dir, "src", "index.html"))).toBe(true);
-  });
-
-  test("allows a small/static HTML snippet", async () => {
-    const dir = makeTempDir();
-    const result = await fileWriteTool.execute(
-      { path: "note.html", content: "<html><body><h1>Hi</h1></body></html>" },
-      makeContext(dir),
-    );
-
-    expect(result.isError).toBe(false);
-    expect(existsSync(join(dir, "note.html"))).toBe(true);
-  });
-
-  test("allows a non-HTML file even with inline script-like content", async () => {
-    const dir = makeTempDir();
-    const result = await fileWriteTool.execute(
-      {
-        path: "notes.md",
-        content:
-          "<script>" + "x".padEnd(2000, "x") + "</script>" + "\n# notes\n",
-      },
-      makeContext(dir),
-    );
-
-    expect(result.isError).toBe(false);
-    expect(existsSync(join(dir, "notes.md"))).toBe(true);
-  });
-});

package/src/__tests__/gateway-only-guard.test.ts

@@ -27,7 +27,6 @@ const ALLOWLIST = new Set([
   "clients/shared/App/Auth/PlatformOAuthService.swift", // comment explaining runtimeUrl vs platformUrl
   "clients/macos/vellum-assistant/App/AppDelegate.swift",
   "clients/macos/vellum-assistant/Features/Settings/SettingsConnectTab.swift",
-  "apps/macos/src/main/bundle-flow.ts", // Electron main calls the local gateway (gatewayPort) with a Guardian token to scan bundles
   ".claude/skills/update/SKILL.md", // daemon health check script
 
   // --- Test fixtures that poll the daemon directly (gateway may require auth) ---
@@ -85,21 +84,12 @@ function isGatewayInternal(filePath: string): boolean {
   return filePath.startsWith("gateway/");
 }
 
-/** Additional files allowed for the interpolated-port check only (use gateway port, not runtime). */
-const INTERPOLATED_PORT_ALLOWLIST = new Set([
-  "apps/macos/src/main/bundle-flow.ts",
-]);
-
 /** Shared violation filter: exempt test files, gateway internals, and allowlisted paths. */
-function filterViolations(
-  files: string[],
-  extraAllowlist?: Set<string>,
-): string[] {
+function filterViolations(files: string[]): string[] {
   return files.filter((f) => {
     if (isTestFile(f)) return false;
     if (isGatewayInternal(f)) return false;
     if (ALLOWLIST.has(f)) return false;
-    if (extraAllowlist?.has(f)) return false;
     return true;
   });
 }
@@ -191,7 +181,7 @@ describe("gateway-only API consumption guard", () => {
     }
 
     const files = grepOutput.split("\n").filter((f) => f.length > 0);
-    const violations = filterViolations(files, INTERPOLATED_PORT_ALLOWLIST);
+    const violations = filterViolations(files);
 
     if (violations.length > 0) {
       const message = [

package/src/__tests__/guardian-grant-minting.test.ts

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

package/src/__tests__/guardian-routing-invariants.test.ts

@@ -21,7 +21,7 @@ import { beforeEach, describe, expect, mock, test } from "bun:test";
 mock.module("../config/env.js", () => ({ isHttpAuthDisabled: () => true }));
 
 const _conversationMocks = new Map<string, unknown>();
-mock.module("../daemon/conversation-registry.js", () => ({
+mock.module("../daemon/conversation-store.js", () => ({
   findConversation: (id: string) => _conversationMocks.get(id),
 }));
 
@@ -51,7 +51,9 @@ import {
   routeGuardianReply,
 } from "../runtime/guardian-reply-router.js";
 import * as pendingInteractions from "../runtime/pending-interactions.js";
-import { listGuardianDecisionPrompts } from "../runtime/routes/guardian-action-routes.js";
+import {
+  listGuardianDecisionPrompts,
+} from "../runtime/routes/guardian-action-routes.js";
 
 initializeDb();
 

package/src/__tests__/handlers-user-message-approval-consumption.test.ts

@@ -143,7 +143,7 @@ mock.module("../util/logger.js", () => ({
 import {
   clearConversations,
   setConversation,
-} from "../daemon/conversation-registry.js";
+} from "../daemon/conversation-store.js";
 import { handleConfirmationResponse } from "../daemon/handlers/conversations.js";
 
 describe("handleConfirmationResponse canonical status sync", () => {

package/src/__tests__/heartbeat-disk-pressure.test.ts

@@ -56,7 +56,6 @@ mock.module("../heartbeat/heartbeat-run-store.js", () => ({
   markStaleRunsAsMissed: mockMarkStaleRunsAsMissed,
   markStaleRunningAsError: mockMarkStaleRunningAsError,
   countCompletedHeartbeatRuns: mock(() => 10),
-  countCompletedRunsToday: mock(() => 0),
   countRecentConsecutiveRuns: mock(() => 0),
 }));
 

package/src/__tests__/heartbeat-service.test.ts

@@ -32,7 +32,6 @@ mock.module("../heartbeat/heartbeat-run-store.js", () => ({
   markStaleRunningAsError: mockMarkStaleRunningAsError,
   listHeartbeatRuns: mockListHeartbeatRuns,
   countCompletedHeartbeatRuns: mockCountCompletedHeartbeatRuns,
-  countCompletedRunsToday: mock(() => 0),
   countRecentConsecutiveRuns: mock(() => 0),
 }));
 

package/src/__tests__/host-app-control-routes.test.ts

@@ -47,7 +47,7 @@ interface FakeConversation {
 
 const conversations = new Map<string, FakeConversation>();
 
-mock.module("../daemon/conversation-registry.js", () => ({
+mock.module("../daemon/conversation-store.js", () => ({
   findConversation: (id: string) => conversations.get(id),
 }));
 

package/src/__tests__/host-cu-routes-targeted.test.ts

@@ -8,9 +8,9 @@
  *  4. Untargeted (no targetClientId, no header) → 200 accepted (regression)
  *
  * Resolution goes through conversation.hostCuProxy?.resolve(...). The
- * conversation registry is mocked to return a controlled conversation object.
+ * conversation store is mocked to return a controlled conversation object.
  *
- * Note: host-cu-routes.ts has a deep import chain (conversation-registry →
+ * Note: host-cu-routes.ts has a deep import chain (conversation-store →
  * conversation.ts → ces-client → service-contracts) that requires mocking
  * before the module loads. We use dynamic imports to ensure all mocks are
  * registered before the route module is evaluated.
@@ -75,7 +75,7 @@ const conversationStore = new Map<
   { hostCuProxy?: { processObservation: (...args: unknown[]) => void } }
 >();
 
-mock.module("../daemon/conversation-registry.js", () => ({
+mock.module("../daemon/conversation-store.js", () => ({
   findConversation: (conversationId: string) =>
     conversationStore.get(conversationId),
 }));

package/src/__tests__/injector-background-turn.test.ts

@@ -23,7 +23,7 @@ mock.module("../config/loader.js", () => ({
 import {
   DEFAULT_INJECTOR_ORDER,
   defaultInjectors,
-} from "../plugins/defaults/memory-retrieval/injectors.js";
+} from "../plugins/defaults/injectors/register.js";
 import type { Injector, TurnContext } from "../plugins/types.js";
 
 function findInjector(name: string): Injector {