@vellumai/assistant 0.8.0 → 0.8.2

package/src/daemon/memory-v2-startup.ts

@@ -35,6 +35,55 @@ export function maybeSeedMemoryV2Skills(config: AssistantConfig): void {
     );
 }
 
+/**
+ * Build the v2 BM25 corpus stats (per-token document frequencies + avg doc
+ * length), then re-seed the v2 skill entries so any skills written during
+ * cold start with the legacy TF encoder get rewritten with stemmed BM25
+ * vectors. The cold-start window exists because the very first
+ * `maybeSeedMemoryV2Skills` call can race ahead of the corpus-stats build —
+ * `skill-store.runSeedOnce` falls back to `generateSparseEmbedding` while
+ * `getConceptPageCorpusStats()` is still `null`, leaving stored skill
+ * sparse vectors in a different hash space than the BM25 query vectors
+ * callers issue (see `simBatch`, `activation.selectCandidates`). Reseeding
+ * here closes that gap without operator intervention.
+ *
+ * Fire-and-forget by design — startup must not block on either step. The
+ * reseed depends on the corpus-stats build, so a corpus-stats failure
+ * short-circuits and skips the reseed (the BM25 vectors it would produce
+ * would be wrong without fresh stats). Both steps log and swallow their own
+ * errors so neither blocks startup.
+ */
+export async function rebuildBm25CorpusStatsAndReseedSkills(
+  config: AssistantConfig,
+): Promise<void> {
+  try {
+    const { rebuildConceptPageCorpusStats } =
+      await import("../memory/v2/sparse-bm25.js");
+    await rebuildConceptPageCorpusStats(getWorkspaceDir());
+    log.info("Memory v2 BM25 corpus stats built");
+  } catch (err) {
+    log.warn(
+      { err },
+      "BM25 corpus-stats rebuild failed — sparse channel will fall back to TF-only until next rebuild",
+    );
+    return;
+  }
+
+  if (!config.memory.v2.enabled) return;
+  try {
+    const { seedV2SkillEntries } = await import("../memory/v2/skill-store.js");
+    await seedV2SkillEntries({ throwOnError: true });
+    log.info(
+      "Memory v2 skill embeddings re-seeded with BM25 vectors after corpus-stats build",
+    );
+  } catch (err) {
+    log.warn(
+      { err },
+      "Failed to re-seed v2 skill entries after BM25 corpus-stats build — skills seeded during cold start may keep TF-only sparse vectors until next reseed",
+    );
+  }
+}
+
 /**
  * Reconcile the v2 concept-page Qdrant collection with the expected schema
  * and enqueue `memory_v2_reembed` when the collection is missing data.
@@ -55,8 +104,11 @@ export async function maybeRebuildMemoryV2Concepts(
   if (!config.memory.v2.enabled) return;
 
   try {
-    const { ensureConceptPageCollection, countConceptPagePoints } =
-      await import("../memory/v2/qdrant.js");
+    const {
+      ensureConceptPageCollection,
+      countConceptPagePoints,
+      clearReembedSentinel,
+    } = await import("../memory/v2/qdrant.js");
     const { hasConceptPages } = await import("../memory/v2/page-store.js");
     const { enqueueMemoryJob } = await import("../memory/jobs-store.js");
 
@@ -76,6 +128,10 @@ export async function maybeRebuildMemoryV2Concepts(
         { jobId, collectionMigrated: migrated },
         "Memory v2 collection rebuild required — enqueued reembed job",
       );
+      // Clear the on-disk sentinel that the qdrant ensure-path writes before
+      // delete: now that reembed is queued, the cross-call signal can retire.
+      // If the sentinel never existed this is a no-op.
+      await clearReembedSentinel();
     }
   } catch (err) {
     log.warn(

package/src/daemon/message-protocol.ts

@@ -15,6 +15,7 @@
 // Re-export domain modules (all individual types remain importable)
 export * from "./message-types/acp.js";
 export * from "./message-types/apps.js";
+export * from "./message-types/bookmarks.js";
 export * from "./message-types/browser.js";
 export * from "./message-types/computer-use.js";
 export * from "./message-types/contacts.js";
@@ -42,6 +43,7 @@ export * from "./message-types/shared.js";
 export * from "./message-types/skills.js";
 export * from "./message-types/subagents.js";
 export * from "./message-types/surfaces.js";
+export * from "./message-types/sync.js";
 export * from "./message-types/upgrades.js";
 export * from "./message-types/work-items.js";
 export * from "./message-types/workspace.js";
@@ -52,6 +54,7 @@ import type {
   _AppsClientMessages,
   _AppsServerMessages,
 } from "./message-types/apps.js";
+import type { _BookmarksServerMessages } from "./message-types/bookmarks.js";
 import type {
   _BrowserClientMessages,
   _BrowserServerMessages,
@@ -129,6 +132,7 @@ import type {
   _SurfacesClientMessages,
   _SurfacesServerMessages,
 } from "./message-types/surfaces.js";
+import type { _SyncInvalidationServerMessages } from "./message-types/sync.js";
 import type { _UpgradesServerMessages } from "./message-types/upgrades.js";
 import type {
   _WorkItemsClientMessages,
@@ -145,6 +149,7 @@ import type {
 export interface SubagentEvent {
   type: "subagent_event";
   subagentId: string;
+  conversationId: string;
   event: ServerMessage;
 }
 
@@ -188,6 +193,7 @@ export type ServerMessage =
   | _SubagentsServerMessages
   | _DocumentsServerMessages
   | _GuardianActionsServerMessages
+  | _SyncInvalidationServerMessages
   | _HomeServerMessages
   | _HostAppControlServerMessages
   | _HostBashServerMessages
@@ -205,6 +211,7 @@ export type ServerMessage =
   | _NotificationsServerMessages
   | _UpgradesServerMessages
   | _AcpServerMessages
+  | _BookmarksServerMessages
   | _DiskPressureServerMessages
   | SubagentEvent;
 

package/src/daemon/message-types/bookmarks.ts

@@ -0,0 +1,18 @@
+// Bookmark events. Surfaced over SSE so a `BookmarkStore` instance in any
+// connected client can stay in sync when another window mutates the list.
+
+import type { BookmarkSummary } from "../../memory/bookmark-crud.js";
+
+export interface BookmarkCreated {
+  type: "bookmark.created";
+  bookmark: BookmarkSummary;
+}
+
+export interface BookmarkDeleted {
+  type: "bookmark.deleted";
+  messageId: string;
+}
+
+// --- Domain-level union aliases (consumed by the barrel file) ---
+
+export type _BookmarksServerMessages = BookmarkCreated | BookmarkDeleted;

package/src/daemon/message-types/conversations.ts

@@ -138,12 +138,6 @@ export interface ModelGetRequest {
   type: "model_get";
 }
 
-export interface ModelSetRequest {
-  type: "model_set";
-  model: string;
-  provider?: string;
-}
-
 export interface ImageGenModelSetRequest {
   type: "image_gen_model_set";
   model: string;
@@ -230,9 +224,18 @@ export interface ConversationTitleUpdated {
 interface ChannelBinding {
   sourceChannel: ChannelId;
   externalChatId: string;
+  externalThreadId?: string | null;
   externalUserId?: string | null;
   displayName?: string | null;
   username?: string | null;
+  slackThread?: {
+    channelId: string;
+    threadTs: string;
+    link?: {
+      appUrl?: string;
+      webUrl?: string;
+    };
+  };
 }
 
 /** Attention state metadata for a conversation's latest assistant message. */
@@ -310,8 +313,14 @@ export interface GenerationHandoff {
   queuedCount: number;
   attachments?: UserMessageAttachment[];
   attachmentWarnings?: string[];
-  /** Database ID of the persisted assistant message, if any. */
+  /** Database ID of the final persisted assistant row, if any. */
   messageId?: string;
+  /**
+   * Database ID used by clients for the rendered assistant bubble. Tool turns
+   * may persist multiple assistant rows; this matches the history row that
+   * survives query-time merging.
+   */
+  displayMessageId?: string;
 }
 
 export interface ModelInfo {
@@ -387,7 +396,10 @@ export interface HistoryResponse {
   type: "history_response";
   conversationId: string;
   messages: Array<{
-    id?: string; // Database message ID (for matching surfaces)
+    /** Database ID used by clients for the rendered message bubble. */
+    id?: string;
+    /** Concrete persisted row ID for row-scoped actions such as TTS/fork. */
+    daemonMessageId?: string;
     role: string;
     text: string;
     timestamp: number;
@@ -408,6 +420,7 @@ export interface HistoryResponse {
       status: "running" | "completed" | "failed" | "aborted";
       error?: string;
       conversationId?: string;
+      objective?: string;
     };
     /** True when text or tool result content was truncated due to maxTextChars/maxToolResultChars. */
     wasTruncated?: boolean;
@@ -527,10 +540,12 @@ export type ConversationErrorCode =
   | "MANAGED_USAGE_LIMIT"
   | "PROVIDER_OVERLOADED"
   | "PROVIDER_API"
+  | "IMAGE_TOO_LARGE"
   | "PROVIDER_BILLING"
   | "PROVIDER_ORDERING"
   | "PROVIDER_WEB_SEARCH"
   | "PROVIDER_NOT_CONFIGURED"
+  | "PROVIDER_INVALID_KEY"
   | "MANAGED_KEY_INVALID"
   | "CONTEXT_TOO_LARGE"
   | "CONVERSATION_ABORTED"
@@ -548,6 +563,20 @@ export interface ConversationErrorMessage {
   debugDetails?: string;
   /** Machine-readable error category for log report metadata and triage. */
   errorCategory?: string;
+  /**
+   * Name of the `provider_connections` row in play when the error occurred.
+   * Surfaced by the macOS chat banner so users know which connection to fix
+   * (e.g. an invalid API key on `my-anthropic`). Optional because some
+   * errors fire before a connection is resolved.
+   */
+  connectionName?: string;
+  /**
+   * Name of the resolved profile (`llm.activeProfile` or per-call override)
+   * in play when the error occurred. Lets the macOS chat banner point
+   * users at the right profile even when the connection name is generic.
+   * Optional because some errors fire before a profile is resolved.
+   */
+  profileName?: string;
 }
 
 /** Reason the conversation list was invalidated. */
@@ -597,7 +626,6 @@ export type _ConversationsClientMessages =
   | CancelRequest
   | DeleteQueuedMessage
   | ModelGetRequest
-  | ModelSetRequest
   | ImageGenModelSetRequest
   | UndoRequest
   | UsageRequest

package/src/daemon/message-types/messages.ts

@@ -235,13 +235,79 @@ export interface SecretRequest {
   allowOneTimeSend?: boolean;
 }
 
+export interface QuestionOption {
+  id: string;
+  label: string;
+  description?: string;
+}
+
+/**
+ * One entry in a batched ask-question request.
+ *
+ * `id` is daemon-assigned (e.g. `q1`, `q2`...) — the LLM neither sees nor
+ * supplies it. It exists so the client has a stable handle to post the
+ * user's answer back against. See `QuestionRequest` for the batching
+ * contract.
+ */
+export interface QuestionEntry {
+  id: string;
+  question: string;
+  description?: string;
+  /** LLM-supplied options, capped at 4. The client always renders a fixed
+   *  5th "Type something else" slot wired to a free-text response — so this
+   *  array never represents the full choice set the user sees. */
+  options: QuestionOption[];
+  /** Optional placeholder shown in the free-text input. */
+  freeTextPlaceholder?: string;
+}
+
+/**
+ * A single broadcast that carries either one or a small batch (≤5) of
+ * clarifying questions. The whole batch is one card lifecycle on the client:
+ * one render, one state machine, one response submission.
+ *
+ * Wire-compat plan: both shapes are populated on every broadcast.
+ *  - `questions[]` is the canonical shape new clients should consume.
+ *  - The flat `question` / `description` / `options` / `freeTextPlaceholder`
+ *    fields mirror `questions[0]` for backwards compat with the existing
+ *    web client, which keys off the flat fields. Once that client adopts
+ *    `questions[]`, the flat fields can be dropped (separate cleanup).
+ *
+ * Daemon callers that don't supply a batch get a one-element `questions`
+ * array synthesized from the flat fields.
+ */
+export interface QuestionRequest {
+  type: "question_request";
+  requestId: string;
+  /** Batched-question payload. Always populated (single questions are sent
+   *  as a one-element array). Each entry's `id` is daemon-assigned. */
+  questions: QuestionEntry[];
+  /** Legacy: mirrors `questions[0].question`. Kept populated for clients
+   *  that haven't adopted the batched `questions[]` shape yet. */
+  question: string;
+  /** Legacy: mirrors `questions[0].description`. */
+  description?: string;
+  /** Legacy: mirrors `questions[0].options`. */
+  options: QuestionOption[];
+  /** Legacy: mirrors `questions[0].freeTextPlaceholder`. */
+  freeTextPlaceholder?: string;
+  conversationId?: string;
+  toolUseId?: string;
+}
+
 export interface MessageComplete {
   type: "message_complete";
   conversationId?: string;
   attachments?: UserMessageAttachment[];
   attachmentWarnings?: string[];
-  /** Database ID of the persisted assistant message, if any. */
+  /** Database ID of the final persisted assistant row, if any. */
   messageId?: string;
+  /**
+   * Database ID used by clients for the rendered assistant bubble. Tool turns
+   * may persist multiple assistant rows; this matches the history row that
+   * survives query-time merging.
+   */
+  displayMessageId?: string;
   /**
    * Distinguishes a real main-turn completion from auxiliary events such as
    * call transcripts, call summaries, and watch notifier outputs. Clients
@@ -370,6 +436,8 @@ export interface ConversationInferenceProfileUpdated {
   type: "conversation_inference_profile_updated";
   conversationId: string;
   profile: string | null;
+  sessionId?: string | null;
+  expiresAt?: number | null;
 }
 
 export type TraceEventKind =
@@ -422,6 +490,7 @@ export type _MessagesServerMessages =
   | ToolResult
   | ConfirmationRequest
   | SecretRequest
+  | QuestionRequest
   | MessageComplete
   | ErrorMessage
   | MessageQueued

package/src/daemon/message-types/subagents.ts

@@ -26,6 +26,7 @@ export interface SubagentDetailResponse {
   type: "subagent_detail_response";
   subagentId: string;
   objective?: string;
+  usage?: UsageStats;
   events: Array<{
     type: string;
     content: string;

package/src/daemon/message-types/sync.ts

@@ -0,0 +1,61 @@
+import { z } from "zod";
+
+export const SYNC_TAGS = {
+  assistantAvatar: "assistant:self:avatar",
+  assistantIdentity: "assistant:self:identity",
+  assistantConfig: "assistant:self:config",
+  assistantSounds: "assistant:self:sounds",
+  assistantSchedules: "assistant:self:schedules",
+  conversationsList: "conversations:list",
+} as const;
+
+export type KnownSyncInvalidationTag =
+  (typeof SYNC_TAGS)[keyof typeof SYNC_TAGS];
+
+export type ConversationSyncInvalidationTag =
+  | `conversation:${string}:messages`
+  | `conversation:${string}:metadata`;
+
+export type SyncInvalidationTag =
+  | KnownSyncInvalidationTag
+  | ConversationSyncInvalidationTag
+  | (string & {});
+
+export interface SyncChangedMessage {
+  type: "sync_changed";
+  tags: SyncInvalidationTag[];
+}
+
+export const SyncInvalidationTagSchema = z.string().min(1);
+
+export const SyncChangedMessageSchema = z
+  .object({
+    type: z.literal("sync_changed"),
+    tags: z.array(SyncInvalidationTagSchema).min(1),
+  })
+  .strict();
+
+export function conversationMessagesSyncTag(
+  conversationId: string,
+): ConversationSyncInvalidationTag {
+  return `conversation:${conversationId}:messages`;
+}
+
+export function conversationMetadataSyncTag(
+  conversationId: string,
+): ConversationSyncInvalidationTag {
+  return `conversation:${conversationId}:metadata`;
+}
+
+export function buildSyncChangedMessage(
+  tags: SyncInvalidationTag[],
+): SyncChangedMessage {
+  const dedupedTags = Array.from(new Set(tags));
+  const parsed = SyncChangedMessageSchema.parse({
+    type: "sync_changed",
+    tags: dedupedTags,
+  });
+  return parsed as SyncChangedMessage;
+}
+
+export type _SyncInvalidationServerMessages = SyncChangedMessage;

package/src/daemon/pkb-reminder-builder.test.ts

@@ -2,26 +2,33 @@ import { describe, expect, test } from "bun:test";
 
 import { buildPkbReminder } from "./pkb-reminder-builder.js";
 
-// Byte-for-byte fixture of the base PKB reminder.
-const BASE_REMINDER =
+// Byte-for-byte fixture of the default (non-relaxed) PKB reminder. Asserted
+// verbatim so that any future edit to the BODY text is caught by tests.
+const BASE_REMINDER_DEFAULT =
   "<system_reminder>" +
-  "\nCall `remember` this turn for anything concrete the user said — facts, preferences, plans, names, dates, decisions, corrections, felt moments. Default to remembering; skip only obvious noise. This should be your most frequently used tool." +
+  "\n**CRITICAL:** Call `remember` this turn for anything concrete the user said — facts, preferences, plans, names, dates, decisions, corrections, felt moments. Default to remembering; skip only obvious noise. This should be your most frequently used tool." +
   "\nIf you're unsure about something that may live in the workspace — past decisions, prior conversations, files — use `recall` before asking or guessing." +
-  "\nRead any unread workspace files that look even partially relevant." +
   "\n</system_reminder>";
 
-describe("buildPkbReminder", () => {
+// Byte-for-byte fixture of the relaxed PKB reminder used when the
+// `memory-retrospective` feature flag is on.
+const BASE_REMINDER_RELAXED =
+  "<system_reminder>" +
+  "\nStay present in this conversation. Use `remember` when something feels worth pausing to mark — corrections (highest priority), plans, decisions, felt moments. You don't have to capture everything in the moment — a retrospective pass reviews this conversation in the background and saves what you didn't capture." +
+  "\nIf you're unsure about something that may live in the workspace — past decisions, prior conversations, files — use `recall` before asking or guessing." +
+  "\n</system_reminder>";
+
+describe("buildPkbReminder — default body (relaxed=false)", () => {
   test("empty hints returns exact base reminder byte-for-byte", () => {
-    expect(buildPkbReminder([])).toBe(BASE_REMINDER);
+    expect(buildPkbReminder([], false)).toBe(BASE_REMINDER_DEFAULT);
   });
 
   test("single hint renders one bullet with no duplicates or trailing blank line", () => {
-    const out = buildPkbReminder(["projects/alpha.md"]);
+    const out = buildPkbReminder(["projects/alpha.md"], false);
     const expected =
       "<system_reminder>" +
-      "\nCall `remember` this turn for anything concrete the user said — facts, preferences, plans, names, dates, decisions, corrections, felt moments. Default to remembering; skip only obvious noise. This should be your most frequently used tool." +
+      "\n**CRITICAL:** Call `remember` this turn for anything concrete the user said — facts, preferences, plans, names, dates, decisions, corrections, felt moments. Default to remembering; skip only obvious noise. This should be your most frequently used tool." +
       "\nIf you're unsure about something that may live in the workspace — past decisions, prior conversations, files — use `recall` before asking or guessing." +
-      "\nRead any unread workspace files that look even partially relevant." +
       "\nBased on the current context, these files look especially relevant:" +
       "\n- projects/alpha.md" +
       "\n</system_reminder>";
@@ -37,12 +44,11 @@ describe("buildPkbReminder", () => {
 
   test("three hints render all three in order", () => {
     const hints = ["a.md", "sub/b.md", "c/d/e.md"];
-    const out = buildPkbReminder(hints);
+    const out = buildPkbReminder(hints, false);
     const expected =
       "<system_reminder>" +
-      "\nCall `remember` this turn for anything concrete the user said — facts, preferences, plans, names, dates, decisions, corrections, felt moments. Default to remembering; skip only obvious noise. This should be your most frequently used tool." +
+      "\n**CRITICAL:** Call `remember` this turn for anything concrete the user said — facts, preferences, plans, names, dates, decisions, corrections, felt moments. Default to remembering; skip only obvious noise. This should be your most frequently used tool." +
       "\nIf you're unsure about something that may live in the workspace — past decisions, prior conversations, files — use `recall` before asking or guessing." +
-      "\nRead any unread workspace files that look even partially relevant." +
       "\nBased on the current context, these files look especially relevant:" +
       "\n- a.md" +
       "\n- sub/b.md" +
@@ -61,7 +67,7 @@ describe("buildPkbReminder", () => {
 
   test("hints with special chars (< and &) are emitted verbatim (no escaping)", () => {
     const hints = ["weird<name>.md", "foo&bar.md"];
-    const out = buildPkbReminder(hints);
+    const out = buildPkbReminder(hints, false);
     expect(out).toContain("- weird<name>.md");
     expect(out).toContain("- foo&bar.md");
     // Ensure no HTML-style escaping happened.
@@ -69,3 +75,38 @@ describe("buildPkbReminder", () => {
     expect(out).not.toContain("&amp;");
   });
 });
+
+describe("buildPkbReminder — relaxed body (relaxed=true)", () => {
+  test("empty hints returns the relaxed base reminder byte-for-byte", () => {
+    expect(buildPkbReminder([], true)).toBe(BASE_REMINDER_RELAXED);
+  });
+
+  test("relaxed BODY does NOT contain the default high-pressure phrasing", () => {
+    const out = buildPkbReminder([], true);
+    expect(out).not.toContain("**CRITICAL:**");
+    expect(out).not.toContain("most frequently used tool");
+    expect(out).not.toContain("Default to remembering");
+  });
+
+  test("relaxed BODY mentions the retrospective backstop framing", () => {
+    const out = buildPkbReminder([], true);
+    expect(out).toContain("Stay present");
+    expect(out).toContain("retrospective pass");
+  });
+
+  test("hints render below the relaxed body in the same shape as default", () => {
+    const out = buildPkbReminder(["projects/alpha.md"], true);
+    expect(out).toContain("- projects/alpha.md");
+    expect(out).toContain(
+      "Based on the current context, these files look especially relevant:",
+    );
+    // Should still close cleanly with no double newline before the tag.
+    expect(out.includes("\n\n</system_reminder>")).toBe(false);
+  });
+});
+
+describe("buildPkbReminder — relaxed vs default differ", () => {
+  test("the two BODY variants are NOT byte-identical", () => {
+    expect(buildPkbReminder([], false)).not.toBe(buildPkbReminder([], true));
+  });
+});

package/src/daemon/pkb-reminder-builder.ts

@@ -1,7 +1,10 @@
-const BODY =
-  "\nCall `remember` this turn for anything concrete the user said — facts, preferences, plans, names, dates, decisions, corrections, felt moments. Default to remembering; skip only obvious noise. This should be your most frequently used tool." +
-  "\nIf you're unsure about something that may live in the workspace — past decisions, prior conversations, files — use `recall` before asking or guessing." +
-  "\nRead any unread workspace files that look even partially relevant.";
+const BODY_DEFAULT =
+  "\n**CRITICAL:** Call `remember` this turn for anything concrete the user said — facts, preferences, plans, names, dates, decisions, corrections, felt moments. Default to remembering; skip only obvious noise. This should be your most frequently used tool." +
+  "\nIf you're unsure about something that may live in the workspace — past decisions, prior conversations, files — use `recall` before asking or guessing.";
+
+const BODY_RELAXED =
+  "\nStay present in this conversation. Use `remember` when something feels worth pausing to mark — corrections (highest priority), plans, decisions, felt moments. You don't have to capture everything in the moment — a retrospective pass reviews this conversation in the background and saves what you didn't capture." +
+  "\nIf you're unsure about something that may live in the workspace — past decisions, prior conversations, files — use `recall` before asking or guessing.";
 
 /**
  * Render the PKB system_reminder text, optionally with a bulleted list of
@@ -12,12 +15,23 @@ const BODY =
  * hint. Hints are emitted verbatim — they are trusted internal paths, not
  * user input, so no escaping is performed.
  *
+ * The `relaxed` flag selects between the default high-pressure body and the
+ * relaxed "judgment framing" body used when the `memory-retrospective`
+ * feature flag is on. With the flag on, the in-conversation remember pressure
+ * eases because the retrospective is the backstop. Callers must pass
+ * `relaxed` explicitly — no default — so the contract is visible at every
+ * call site.
+ *
  * Caller is responsible for capping the hints array at 3 entries.
  */
-export function buildPkbReminder(hints: ReadonlyArray<string>): string {
+export function buildPkbReminder(
+  hints: ReadonlyArray<string>,
+  relaxed: boolean,
+): string {
+  const body = relaxed ? BODY_RELAXED : BODY_DEFAULT;
   if (hints.length === 0) {
-    return `<system_reminder>${BODY}\n</system_reminder>`;
+    return `<system_reminder>${body}\n</system_reminder>`;
   }
   const bullets = hints.map((h) => `- ${h}`).join("\n");
-  return `<system_reminder>${BODY}\nBased on the current context, these files look especially relevant:\n${bullets}\n</system_reminder>`;
+  return `<system_reminder>${body}\nBased on the current context, these files look especially relevant:\n${bullets}\n</system_reminder>`;
 }