@vellumai/assistant 0.8.8 → 0.8.9-dev.202606091926.ebb2d62

package/src/daemon/tool-side-effects.ts

@@ -155,6 +155,21 @@ registerHook("app_refresh", (_name, input, _result, { ctx }) => {
   notifyAppChanged(ctx, appId, { fileChange: true });
 });
 
+// app_update compiles internally (like app_refresh) but emits no events of its
+// own, so without this hook an updated app leaves open surfaces rendering the
+// stale dist and never re-deploys or invalidates the Library. The executor owns
+// the compile; notifyAppChanged only refreshes surfaces and broadcasts.
+registerHook("app_update", (_name, input, _result, { ctx }) => {
+  const appId = input.app_id as string | undefined;
+  if (!appId) return;
+  try {
+    addAppConversationId(appId, ctx.conversationId);
+  } catch (err) {
+    log.warn({ err, appId }, "Failed to track conversation ID on app_update");
+  }
+  notifyAppChanged(ctx, appId, { fileChange: true });
+});
+
 registerHook("voice_config_update", (_name, input) => {
   const setting = input.setting as string | undefined;
   if (!setting) return;

package/src/daemon/wake-conversation-ops.ts

@@ -0,0 +1,269 @@
+/**
+ * Daemon-side side effects a wake performs against a live
+ * {@link Conversation}, used by `wakeAgentForOpportunity()` in
+ * `runtime/agent-wake.ts`.
+ *
+ * These are kept as standalone functions (rather than methods on
+ * `Conversation`) so the wake-specific concerns — event→wire translation,
+ * the wake ui_surface card, and the wake's one-shot tail persistence —
+ * stay out of the conversation class. They live in the daemon layer
+ * because they translate agent-loop events into client wire frames and
+ * touch the message store, so `runtime/agent-wake.ts` references only the
+ * erased `Conversation` type and never imports `message-protocol.ts`.
+ */
+
+import type { AgentEvent } from "../agent/loop.js";
+import {
+  addMessage,
+  getConversation,
+  provenanceFromTrustContext,
+} from "../memory/conversation-crud.js";
+import { syncMessageToDisk } from "../memory/conversation-disk-view.js";
+import { backfillMessageIdOnLogs } from "../memory/llm-request-log-store.js";
+import type { Message } from "../providers/types.js";
+import { broadcastMessage } from "../runtime/assistant-event-hub.js";
+import { getLogger } from "../util/logger.js";
+import type { Conversation } from "./conversation.js";
+import type { ServerMessage } from "./message-protocol.js";
+
+const log = getLogger("wake-conversation-ops");
+
+/**
+ * Translate a raw {@link AgentEvent} from the agent loop into the
+ * corresponding {@link ServerMessage} wire frame. The normal user-turn
+ * path does this via the full state-aware handler in
+ * `conversation-agent-loop-handlers.ts`; the wake path has no tool
+ * accounting, title generation, or activity-state tracking to worry
+ * about, so we only need the subset that produces client-visible
+ * frames. Events that have no client-visible wire shape (usage, error,
+ * preview/input-json deltas, etc.) are dropped — they produce no UI.
+ */
+function translateAgentEventToServerMessage(
+  event: AgentEvent,
+  conversationId: string,
+): ServerMessage | null {
+  switch (event.type) {
+    case "text_delta":
+      return {
+        type: "assistant_text_delta",
+        text: event.text,
+        conversationId,
+      };
+    case "thinking_delta":
+      return {
+        type: "assistant_thinking_delta",
+        thinking: event.thinking,
+        conversationId,
+      };
+    case "tool_use":
+      return {
+        type: "tool_use_start",
+        toolName: event.name,
+        input: event.input,
+        conversationId,
+        toolUseId: event.id,
+      };
+    case "tool_use_preview_start":
+      return {
+        type: "tool_use_preview_start",
+        toolUseId: event.toolUseId,
+        toolName: event.toolName,
+        conversationId,
+      };
+    case "tool_output_chunk":
+      return {
+        type: "tool_output_chunk",
+        chunk: event.chunk,
+        conversationId,
+        toolUseId: event.toolUseId,
+      };
+    case "tool_result": {
+      const imageBlocks = event.contentBlocks?.filter(
+        (b): b is Extract<typeof b, { type: "image" }> => b.type === "image",
+      );
+      const imageDataList = imageBlocks?.length
+        ? imageBlocks.map((b) => b.source.data)
+        : undefined;
+      return {
+        type: "tool_result",
+        toolName: "",
+        result: event.content,
+        isError: event.isError,
+        diff: event.diff,
+        status: event.status,
+        conversationId,
+        imageData: imageDataList?.[0],
+        imageDataList,
+        toolUseId: event.toolUseId,
+      };
+    }
+    case "server_tool_start":
+      return {
+        type: "tool_use_start",
+        toolName: event.name,
+        input: event.input,
+        conversationId,
+        toolUseId: event.toolUseId,
+      };
+    case "server_tool_complete": {
+      let resultText = "";
+      if (Array.isArray(event.content) && event.content.length > 0) {
+        resultText = (event.content as unknown[])
+          .filter(
+            (r): r is { type: string; title: string; url: string } =>
+              typeof r === "object" &&
+              r != null &&
+              (r as { type?: string }).type === "web_search_result",
+          )
+          .map((r) => `${r.title}\n${r.url}`)
+          .join("\n\n");
+      }
+      return {
+        type: "tool_result",
+        toolName: "web_search",
+        result: resultText,
+        isError: event.isError,
+        conversationId,
+        toolUseId: event.toolUseId,
+      };
+    }
+    case "message_complete":
+      return {
+        type: "message_complete",
+        conversationId,
+      };
+    case "input_json_delta":
+    case "usage":
+    case "error":
+    case "provider_error":
+    case "max_tokens_reached":
+    case "context_compacting":
+    case "compaction_circuit_open":
+    case "compaction_circuit_closed":
+    case "compaction_completed":
+    case "history_stripped":
+    case "agent_loop_exit":
+      return null;
+    case "llm_call_started":
+      // The wake path persists its assistant tail via
+      // `persistWakeTailMessage` (an `addMessage`-shaped call) rather than
+      // the main event-handler's `reserve` → `updateContent` pipeline, so
+      // there is no row to reserve here. Translation returns null and the
+      // wake path's existing end-of-turn persist continues to mint the row.
+      return null;
+  }
+}
+
+/**
+ * Translate a raw agent event to its wire frame and broadcast it to
+ * connected clients. No-op for events with no client-visible shape.
+ */
+export function emitWakeAgentEvent(
+  conversation: Conversation,
+  event: AgentEvent,
+): void {
+  const frame = translateAgentEventToServerMessage(
+    event,
+    conversation.conversationId,
+  );
+  if (!frame) return;
+  broadcastMessage(frame);
+}
+
+/**
+ * Emit the live ui_surface card announcing that a wake produced output.
+ * The `surfaceId` matches the `ui_surface` content block injected into
+ * the wake's first assistant tail message so the client renders one card,
+ * not two.
+ */
+export function broadcastWakeSurface(
+  conversation: Conversation,
+  source: string,
+  hint: string,
+  surfaceId: string,
+): void {
+  broadcastMessage({
+    type: "ui_surface_show",
+    conversationId: conversation.conversationId,
+    surfaceId,
+    surfaceType: "card",
+    data: {
+      title: "Conversation Woke",
+      body: hint,
+      metadata: [{ label: "Source", value: source }],
+    },
+    display: "inline",
+  });
+}
+
+/**
+ * Persist a single tail message produced by a wake (assistant outputs and
+ * intervening tool_result user messages). Builds the channel/interface
+ * metadata, persists via the canonical `addMessage` path, syncs the row to
+ * the disk view, and backfills the message id onto this turn's LLM request
+ * logs so wake-produced messages match the user-turn persistence path.
+ */
+export async function persistWakeTailMessage(
+  conversation: Conversation,
+  message: Message,
+): Promise<void> {
+  const turnChannelCtx = conversation.getTurnChannelContext();
+  const turnInterfaceCtx = conversation.getTurnInterfaceContext();
+  const metadata: Record<string, unknown> = {
+    ...provenanceFromTrustContext(conversation.trustContext),
+    userMessageChannel: turnChannelCtx?.userMessageChannel ?? "vellum",
+    assistantMessageChannel:
+      turnChannelCtx?.assistantMessageChannel ?? "vellum",
+    userMessageInterface: turnInterfaceCtx?.userMessageInterface ?? "web",
+    assistantMessageInterface:
+      turnInterfaceCtx?.assistantMessageInterface ?? "web",
+  };
+  const persisted = await addMessage(
+    conversation.conversationId,
+    message.role,
+    JSON.stringify(message.content),
+    { metadata },
+  );
+  if (message.role === "assistant") {
+    try {
+      backfillMessageIdOnLogs(conversation.conversationId, persisted.id);
+    } catch (err) {
+      log.warn(
+        { err, conversationId: conversation.conversationId },
+        "wake persist: backfill messageId on LLM logs failed (non-fatal)",
+      );
+    }
+  }
+  try {
+    const convRow = getConversation(conversation.conversationId);
+    if (convRow) {
+      syncMessageToDisk(
+        conversation.conversationId,
+        persisted.id,
+        convRow.createdAt,
+      );
+    }
+  } catch (err) {
+    log.warn(
+      { err, conversationId: conversation.conversationId },
+      "wake persist: syncMessageToDisk failed (non-fatal)",
+    );
+  }
+}
+
+/**
+ * Temporarily restrict the tools visible/executable during a wake by
+ * reusing the conversation's subagent allowlist slot. Returns a restore
+ * callback that reinstates the previous allowlist so the wake can release
+ * the scope before any queued user turn is drained.
+ */
+export function scopeWakeAllowedTools(
+  conversation: Conversation,
+  tools: ReadonlySet<string>,
+): () => void {
+  const previous = conversation.subagentAllowedTools;
+  conversation.setSubagentAllowedTools(new Set(tools));
+  return () => {
+    conversation.setSubagentAllowedTools(previous);
+  };
+}

package/src/embedded/plugin-api.ts

@@ -8,7 +8,7 @@
  * means plugin-api ships as part of the assistant binary's regular code
  * graph; in JIT/Docker, it loads from source.
  *
- * The loaded namespace is then installed on `globalThis` under a versioned
+ * The loaded namespace is then installed on `globalThis` under a well-known
  * symbol. The boot-time shim writer (`ensurePluginApiShim`) enumerates
  * {@link PLUGIN_API_EXPORTS} and generates a tiny ESM module at
  * `<workspaceDir>/node_modules/@vellumai/plugin-api/index.js` that
@@ -26,7 +26,7 @@
 import * as pluginApi from "../plugin-api/index.js";
 
 /** Symbol key under which the plugin-api namespace is published on globalThis. */
-export const PLUGIN_API_REGISTRY_KEY = Symbol.for("vellum.plugin-api.v1");
+export const PLUGIN_API_REGISTRY_KEY = Symbol.for("vellum.plugin-api");
 
 // Install on globalThis once at module-load time. The shim writer reads
 // `PLUGIN_API_EXPORTS` to know which bindings to re-export; the shim's

package/src/export/__tests__/transcript-formatter.test.ts

@@ -9,6 +9,7 @@ type TestMessage = {
   content: string;
   createdAt: number;
   metadata: string | null;
+  clientMessageId: string | null;
 };
 
 const parentMessages: TestMessage[] = [
@@ -19,6 +20,7 @@ const parentMessages: TestMessage[] = [
     content: JSON.stringify([{ type: "text", text: "go research foo" }]),
     createdAt: 1_700_000_000_000,
     metadata: null,
+    clientMessageId: null,
   },
   {
     id: "msg-parent-2",
@@ -34,6 +36,7 @@ const parentMessages: TestMessage[] = [
         conversationId: "child-conv-1",
       },
     }),
+    clientMessageId: null,
   },
 ];
 
@@ -47,6 +50,7 @@ const childMessages: TestMessage[] = [
     ]),
     createdAt: 1_700_000_002_000,
     metadata: null,
+    clientMessageId: null,
   },
   {
     id: "msg-child-2",
@@ -57,6 +61,7 @@ const childMessages: TestMessage[] = [
     ]),
     createdAt: 1_700_000_003_000,
     metadata: null,
+    clientMessageId: null,
   },
 ];
 

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

@@ -85,6 +85,7 @@ const stubConfig: {
     activeHoursStart: number | null;
     activeHoursEnd: number | null;
     maxConsecutiveRuns: number | null;
+    maxDailyRuns: number | null;
     disposition: string;
   };
 } = {
@@ -94,6 +95,7 @@ const stubConfig: {
     activeHoursStart: null,
     activeHoursEnd: null,
     maxConsecutiveRuns: null,
+    maxDailyRuns: null,
     disposition: "Default disposition text.",
   },
 };
@@ -195,6 +197,7 @@ mock.module("../heartbeat-run-store.js", () => ({
   markStaleRunsAsMissed: () => 0,
   markStaleRunningAsError: () => 0,
   countCompletedHeartbeatRuns: () => 10,
+  countCompletedRunsToday: () => 0,
   countRecentConsecutiveRuns: () => 0,
 }));
 

package/src/heartbeat/heartbeat-run-store.ts

@@ -24,7 +24,8 @@ export type HeartbeatSkipReason =
   | "outside_active_hours"
   | "overlap"
   | "pre_first_user_message"
-  | "max_consecutive_runs";
+  | "max_consecutive_runs"
+  | "max_daily_runs";
 
 export interface HeartbeatRunRecord {
   id: string;
@@ -217,6 +218,27 @@ export function countCompletedHeartbeatRuns(): number {
   return row?.count ?? 0;
 }
 
+/**
+ * Count heartbeat runs that completed with status `ok` today (local midnight).
+ */
+export function countCompletedRunsToday(): number {
+  const db = getDb();
+  const now = new Date();
+  const startOfDay = new Date(
+    now.getFullYear(),
+    now.getMonth(),
+    now.getDate(),
+  ).getTime();
+  const row = db
+    .select({ count: sql<number>`count(*)` })
+    .from(heartbeatRuns)
+    .where(
+      sql`${heartbeatRuns.status} = 'ok' AND ${heartbeatRuns.scheduledFor} >= ${startOfDay}`,
+    )
+    .get();
+  return row?.count ?? 0;
+}
+
 /**
  * Count the most recent consecutive heartbeat runs with status `ok`,
  * walking backwards from newest. Stops at the first non-`ok` status

package/src/heartbeat/heartbeat-service.ts

@@ -25,6 +25,7 @@ import { stripCommentLines } from "../util/strip-comment-lines.js";
 import {
   completeHeartbeatRun,
   countCompletedHeartbeatRuns,
+  countCompletedRunsToday,
   countRecentConsecutiveRuns,
   insertPendingHeartbeatRun,
   markStaleRunningAsError,
@@ -187,6 +188,13 @@ export class HeartbeatService {
     );
   }
 
+  /** Whether the daily run cap has been reached. */
+  get isDailyCapReached(): boolean {
+    const config = getConfig().heartbeat;
+    if (config.maxDailyRuns == null) return false;
+    return countCompletedRunsToday() >= config.maxDailyRuns;
+  }
+
   async runManagedWakeIfDue(
     options: ManagedWakeHeartbeatRunOptions = {},
   ): Promise<ManagedWakeHeartbeatRunResult> {
@@ -551,6 +559,24 @@ export class HeartbeatService {
       return false;
     }
 
+    // Daily run cap — stop burning tokens when the daily budget is exhausted.
+    // Force runs bypass the cap.
+    if (
+      !force &&
+      config.maxDailyRuns != null &&
+      countCompletedRunsToday() >= config.maxDailyRuns
+    ) {
+      log.debug(
+        { maxDailyRuns: config.maxDailyRuns },
+        "Daily run cap reached, skipping",
+      );
+      if (runId) skipHeartbeatRun(runId, "max_daily_runs");
+      if (!this.cronMode) {
+        this.scheduleNextRun(config.intervalMs);
+      }
+      return false;
+    }
+
     // Overlap prevention
     if (this.activeRun) {
       log.debug("Previous heartbeat run still active, skipping");

package/src/ipc/__tests__/browser-ipc.test.ts

@@ -55,7 +55,7 @@ mock.module("../../browser/operations.js", () => ({
   },
 }));
 
-mock.module("../../daemon/conversation-store.js", () => ({
+mock.module("../../daemon/conversation-registry.js", () => ({
   findConversation: (conversationId: string) => {
     mockFindConversationCalls.push(conversationId);
     return mockConversation ?? undefined;

package/src/ipc/__tests__/ui-request-route.test.ts

@@ -32,9 +32,9 @@ let mockSurfaceResult: InteractiveUiResult = {
 let mockSurfaceThrows: Error | null = null;
 
 // Re-export the real module and override only findConversation.
-const realStore = await import("../../daemon/conversation-store.js");
-mock.module("../../daemon/conversation-store.js", () => ({
-  ...realStore,
+const realRegistry = await import("../../daemon/conversation-registry.js");
+mock.module("../../daemon/conversation-registry.js", () => ({
+  ...realRegistry,
   findConversation: (_conversationId: string) => mockConversation ?? undefined,
 }));
 

package/src/ipc/skill-routes/__tests__/memory.test.ts

@@ -137,6 +137,21 @@ describe("host.memory.addMessage", () => {
       }),
     ).rejects.toThrow();
   });
+
+  test("rejects the non-renderable system role", async () => {
+    // GIVEN the messages store is UI-facing (ConversationMessage), so only
+    // renderable turns may be persisted via this facet
+    // WHEN a skill attempts to add a system row
+    // THEN the route rejects it instead of persisting agent-context scaffolding
+    await expect(
+      memoryAddMessageRoute.handler({
+        conversationId: "c",
+        role: "system",
+        content: "agent-context scaffolding",
+      }),
+    ).rejects.toThrow();
+    expect(addMessageSpy).not.toHaveBeenCalled();
+  });
 });
 
 describe("host.memory.wakeAgentForOpportunity", () => {

package/src/ipc/skill-routes/memory.ts

@@ -17,7 +17,9 @@ import type { SkillIpcRoute } from "../skill-ipc-types.js";
 
 /**
  * IPC params for `addMessage()`. `role` is constrained to the
- * `MessageRole` union. `metadata` is a free-form record (validated
+ * renderable conversation roles (`user`/`assistant`): the messages store
+ * is UI-facing (`ConversationMessage`), so agent-context `system` rows are
+ * not accepted here. `metadata` is a free-form record (validated
  * downstream by `messageMetadataSchema` with a warn-and-store fallback).
  * `skipIndexing` and `clientMessageId` mirror `AddMessageOptions`.
  *
@@ -28,7 +30,7 @@ import type { SkillIpcRoute } from "../skill-ipc-types.js";
  */
 const MemoryAddMessageParams = z.object({
   conversationId: z.string().min(1),
-  role: z.enum(["user", "assistant", "system"]),
+  role: z.enum(["user", "assistant"]),
   content: z.string(),
   metadata: z.record(z.string(), z.unknown()).optional(),
   skipIndexing: z.boolean().optional(),

package/src/memory/__tests__/activation-session-store.test.ts

@@ -0,0 +1,41 @@
+import { beforeEach, describe, expect, mock, test } from "bun:test";
+
+mock.module("../../util/logger.js", () => ({
+  getLogger: () =>
+    new Proxy({} as Record<string, unknown>, {
+      get: () => () => {},
+    }),
+}));
+
+import {
+  isActivationSession,
+  markActivationSession,
+} from "../activation-session-store.js";
+import { getDb } from "../db-connection.js";
+import { initializeDb } from "../db-init.js";
+import { activationSessions } from "../schema.js";
+
+initializeDb();
+
+describe("activation-session-store", () => {
+  beforeEach(() => {
+    getDb().delete(activationSessions).run();
+  });
+
+  test("marks a conversation and reports it as an activation session", () => {
+    markActivationSession("c1");
+    expect(isActivationSession("c1")).toBe(true);
+  });
+
+  test("unmarked conversations are not activation sessions", () => {
+    expect(isActivationSession("nope")).toBe(false);
+  });
+
+  test("marking the same conversation twice is idempotent", () => {
+    markActivationSession("c1");
+    markActivationSession("c1");
+    expect(isActivationSession("c1")).toBe(true);
+    const rows = getDb().select().from(activationSessions).all();
+    expect(rows.length).toBe(1);
+  });
+});

package/src/memory/__tests__/jobs-worker-v2-schedule.test.ts

@@ -151,6 +151,7 @@ describe("maybeEnqueueGraphMaintenanceJobs — memory v2 consolidation", () => {
 
   test("enqueues consolidate when v2 is on and no checkpoint exists", () => {
     const config = buildConfig({ v2Enabled: true, intervalHours: 1 });
+    writeBuffer(15);
 
     maybeEnqueueGraphMaintenanceJobs(config, Date.now());
 
@@ -177,6 +178,7 @@ describe("maybeEnqueueGraphMaintenanceJobs — memory v2 consolidation", () => {
 
   test("enqueues consolidate again once the interval elapses", () => {
     const config = buildConfig({ v2Enabled: true, intervalHours: 1 });
+    writeBuffer(15);
 
     const now = Date.now();
     // Stamp checkpoint to >1h ago.
@@ -193,6 +195,7 @@ describe("maybeEnqueueGraphMaintenanceJobs — memory v2 consolidation", () => {
 
   test("respects a custom consolidation_interval_hours value", () => {
     const config = buildConfig({ v2Enabled: true, intervalHours: 6 });
+    writeBuffer(15);
 
     const now = Date.now();
     // 4h elapsed — under the configured 6h interval.
@@ -216,6 +219,7 @@ describe("maybeEnqueueGraphMaintenanceJobs — memory v2 consolidation", () => {
 
   test("v1 maintenance entries are suppressed when v2 is active", () => {
     const config = buildConfig({ v2Enabled: true, intervalHours: 1 });
+    writeBuffer(15);
 
     // No checkpoints set — every entry would be due if it were scheduled.
     deleteMemoryCheckpoint("graph_maintenance:decay:last_run");
@@ -426,3 +430,86 @@ describe("maybeEnqueueGraphMaintenanceJobs — buffer-size trigger", () => {
     expect(countPendingJobs("memory_v2_consolidate")).toBe(0);
   });
 });
+
+describe("maybeEnqueueGraphMaintenanceJobs — min buffer lines noop", () => {
+  test("skips scheduled consolidation when buffer is under 10 lines", () => {
+    // GIVEN v2 consolidation is enabled and the interval has elapsed
+    const config = buildConfig({ v2Enabled: true, intervalHours: 1 });
+    const now = Date.now();
+    setMemoryCheckpoint(
+      CONSOLIDATE_CHECKPOINT_KEY,
+      String(now - 2 * 60 * 60 * 1000),
+    );
+    // AND the buffer has fewer than 10 lines
+    writeBuffer(5);
+
+    // WHEN the schedule runs
+    maybeEnqueueGraphMaintenanceJobs(config, now);
+
+    // THEN no consolidation job is enqueued
+    expect(countPendingJobs("memory_v2_consolidate")).toBe(0);
+    // AND the checkpoint advances so the skip doesn't re-fire next tick
+    expect(getMemoryCheckpoint(CONSOLIDATE_CHECKPOINT_KEY)).toBe(String(now));
+  });
+
+  test("allows scheduled consolidation when buffer has exactly 10 lines", () => {
+    // GIVEN v2 consolidation is enabled and the interval has elapsed
+    const config = buildConfig({ v2Enabled: true, intervalHours: 1 });
+    const now = Date.now();
+    setMemoryCheckpoint(
+      CONSOLIDATE_CHECKPOINT_KEY,
+      String(now - 2 * 60 * 60 * 1000),
+    );
+    // AND the buffer has exactly 10 lines
+    writeBuffer(10);
+
+    // WHEN the schedule runs
+    maybeEnqueueGraphMaintenanceJobs(config, now);
+
+    // THEN consolidation is enqueued
+    expect(countPendingJobs("memory_v2_consolidate")).toBe(1);
+  });
+
+  test("skips scheduled consolidation when buffer file is missing", () => {
+    // GIVEN v2 consolidation is enabled and the interval has elapsed
+    const config = buildConfig({ v2Enabled: true, intervalHours: 1 });
+    const now = Date.now();
+    setMemoryCheckpoint(
+      CONSOLIDATE_CHECKPOINT_KEY,
+      String(now - 2 * 60 * 60 * 1000),
+    );
+    // AND no buffer file exists (0 lines)
+    removeBuffer();
+
+    // WHEN the schedule runs
+    maybeEnqueueGraphMaintenanceJobs(config, now);
+
+    // THEN no consolidation job is enqueued
+    expect(countPendingJobs("memory_v2_consolidate")).toBe(0);
+    // AND the checkpoint advances
+    expect(getMemoryCheckpoint(CONSOLIDATE_CHECKPOINT_KEY)).toBe(String(now));
+  });
+
+  test("size-based trigger is independent of the min-lines noop", () => {
+    // GIVEN the time-based interval has elapsed but the buffer has fewer
+    // than 10 lines, while exceeding the configured maxBufferLines threshold
+    const config = buildConfig({
+      v2Enabled: true,
+      intervalHours: 1,
+      maxBufferLines: 5,
+    });
+    const now = Date.now();
+    setMemoryCheckpoint(
+      CONSOLIDATE_CHECKPOINT_KEY,
+      String(now - 2 * 60 * 60 * 1000),
+    );
+    writeBuffer(8);
+
+    // WHEN the schedule runs
+    maybeEnqueueGraphMaintenanceJobs(config, now);
+
+    // THEN consolidation is enqueued via the size trigger despite the
+    // time-based schedule being nooped (8 < 10 min lines, but 8 >= 5 max)
+    expect(countPendingJobs("memory_v2_consolidate")).toBe(1);
+  });
+});