@vellumai/assistant 0.11.0 → 0.11.1-staging.2

package/src/daemon/profiler-run-store.ts

@@ -93,14 +93,18 @@ interface ProfilerSweepResult {
  */
 function computeDirBytes(dirPath: string): number {
   let total = 0;
-  if (!existsSync(dirPath)) return 0;
+  if (!existsSync(dirPath)) {
+    return 0;
+  }
 
   const names = readdirSync(dirPath);
   for (const name of names) {
     const entryPath = join(dirPath, name);
     try {
       const stat = lstatSync(entryPath);
-      if (stat.isSymbolicLink()) continue;
+      if (stat.isSymbolicLink()) {
+        continue;
+      }
       if (stat.isDirectory()) {
         total += computeDirBytes(entryPath);
       } else if (stat.isFile()) {
@@ -177,7 +181,9 @@ interface RescanRunsOptions {
 export function rescanRuns(options?: RescanRunsOptions): ProfilerRunManifest[] {
   const readOnly = options?.readOnly ?? false;
   const runsDir = getProfilerRunsDir();
-  if (!existsSync(runsDir)) return [];
+  if (!existsSync(runsDir)) {
+    return [];
+  }
 
   const activeRunId = getProfilerRunId();
   const manifests: ProfilerRunManifest[] = [];
@@ -193,7 +199,9 @@ export function rescanRuns(options?: RescanRunsOptions): ProfilerRunManifest[] {
   for (const runId of names) {
     const runDir = getProfilerRunDir(runId);
     try {
-      if (!statSync(runDir).isDirectory()) continue;
+      if (!statSync(runDir).isDirectory()) {
+        continue;
+      }
     } catch {
       continue;
     }
@@ -310,7 +318,9 @@ export function runProfilerSweep(): ProfilerSweepResult {
     const overRunCount = completedRuns.length > maxRuns;
     const overFreeSpace = getFreeDiskBytes(runsDir) < minFreeBytes;
 
-    if (!overBytesBudget && !overRunCount && !overFreeSpace) break;
+    if (!overBytesBudget && !overRunCount && !overFreeSpace) {
+      break;
+    }
 
     const oldest = completedRuns[0]!;
     const runDir = getProfilerRunDir(oldest.runId);
@@ -436,7 +446,9 @@ const PROFILER_SUMMARY_EXTENSIONS = [".md"];
  * Count profiler artifact files (raw profiles) in a run directory.
  */
 function countArtifacts(runDir: string): number {
-  if (!existsSync(runDir)) return 0;
+  if (!existsSync(runDir)) {
+    return 0;
+  }
   try {
     return readdirSync(runDir).filter((name) =>
       PROFILER_ARTIFACT_EXTENSIONS.some((ext) => name.endsWith(ext)),
@@ -451,7 +463,9 @@ function countArtifacts(runDir: string): number {
  * directory.
  */
 function hasSummaryFiles(runDir: string): boolean {
-  if (!existsSync(runDir)) return false;
+  if (!existsSync(runDir)) {
+    return false;
+  }
   try {
     return readdirSync(runDir).some((name) =>
       PROFILER_SUMMARY_EXTENSIONS.some((ext) => name.endsWith(ext)),

package/src/daemon/shutdown-handlers.ts

@@ -10,6 +10,7 @@ import {
   spawnDetachedWalCheckpoint,
 } from "../persistence/db-async-query.js";
 import { getSqlite, isDbOpen, resetDb } from "../persistence/db-connection.js";
+import { shutdownEmbeddingBackends } from "../persistence/embeddings/embedding-backend.js";
 import { stopQdrantManager } from "../persistence/embeddings/qdrant-manager.js";
 import { stopConsentRefresh } from "../platform/consent-cache.js";
 import { HOOKS } from "../plugin-api/constants.js";
@@ -70,12 +71,15 @@ async function shutdown(): Promise<void> {
   // Set this BEFORE awaiting heartbeat stop so it covers all
   // potentially-blocking async shutdown work.
   //
-  // 20s budget: 15s reserved for Meet session teardown
-  // (`MeetSessionManager.shutdownAll`), plus ~5s for the remaining
-  // daemon work (workspace commits, server drain, enrichment, telemetry,
-  // mcp, qdrant, sqlite checkpoint). Without a live Meet session the
-  // rest of the shutdown routinely completes in under a second, so this
-  // bump only changes behavior for the stuck-shutdown path.
+  // 30s budget: 15s reserved for Meet session teardown
+  // (`MeetSessionManager.shutdownAll`), up to
+  // `EMBEDDING_SHUTDOWN_BUDGET_MS` for reaping the embedding worker
+  // subprocesses, plus ~5s for the remaining daemon work (workspace commits,
+  // server drain, enrichment, telemetry, mcp, qdrant, sqlite checkpoint).
+  // Without a live Meet session the rest of the shutdown routinely completes
+  // in under a second, so this only changes behavior for the stuck-shutdown
+  // path. Forcing the exit before the embedding reap finishes would orphan the
+  // worker this shutdown exists to collect.
   const forceTimer = setTimeout(() => {
     log.warn("Graceful shutdown timed out, forcing exit");
     // A stuck shutdown may never reach the graceful WAL checkpoint below —
@@ -90,7 +94,7 @@ async function shutdown(): Promise<void> {
     }
     stopBackgroundServicesAndCleanupPidFile();
     process.exit(1);
-  }, 20_000);
+  }, 30_000);
   forceTimer.unref();
 
   await stopWorkspaceHeartbeatService();
@@ -181,6 +185,15 @@ async function shutdown(): Promise<void> {
     log.warn({ err }, "MCP server manager shutdown failed (non-fatal)");
   }
 
+  // Local embedding backends own an ONNX worker subprocess each. Nothing else
+  // reaps them, so without this they outlive the daemon holding ~570 MB apiece
+  // until the next boot happens to reclaim them (JARVIS-1125).
+  try {
+    await shutdownEmbeddingBackends();
+  } catch (err) {
+    log.warn({ err }, "Embedding backend shutdown failed (non-fatal)");
+  }
+
   await stopQdrantManager();
 
   // Optimize query planner statistics before closing so they persist for

package/src/daemon/sqlite-corruption-watchdog.ts

@@ -95,7 +95,9 @@ function reportCorruption(
   const now = Date.now();
   const last =
     lastReportAtByDatabase.get(detail.database) ?? Number.NEGATIVE_INFINITY;
-  if (now - last < REPORT_COOLDOWN_MS) return;
+  if (now - last < REPORT_COOLDOWN_MS) {
+    return;
+  }
   lastReportAtByDatabase.set(detail.database, now);
 
   log.error(detail, "SQLite corruption detected");
@@ -135,7 +137,9 @@ export function observeSqliteStatementError(
   err: unknown,
   context: StatementErrorContext,
 ): void {
-  if (!isSqliteCorruptionError(err)) return;
+  if (!isSqliteCorruptionError(err)) {
+    return;
+  }
   reportCorruption({
     database: context.database ?? "unknown",
     error: err instanceof Error ? err.message : String(err),

package/src/daemon/summarize-boundary.ts

@@ -16,7 +16,9 @@ import { UserError } from "../util/errors.js";
  * these reconstructs `turnCount` on load.
  */
 export function startsNewTurn(msg: MessageRow): boolean {
-  if (msg.role !== "user") return false;
+  if (msg.role !== "user") {
+    return false;
+  }
   return !isToolResultOnlyUserMessage(msg);
 }
 

package/src/daemon/turn-latency-tracker.ts

@@ -98,7 +98,9 @@ export class TurnLatencyTracker {
   mark(name: LatencyMark): void {
     // A retried provider call re-issues `request_sent`; drop the failed
     // attempt's stale marks first so the retry's segment measures only itself.
-    if (name === "request_sent") this.supersedeFailedAttempt();
+    if (name === "request_sent") {
+      this.supersedeFailedAttempt();
+    }
     this.marks.push({ name, at: Date.now() });
   }
 
@@ -120,20 +122,28 @@ export class TurnLatencyTracker {
     let failedRequestSent = -1;
     for (let i = this.marks.length - 1; i >= 0; i--) {
       const name = this.marks[i]!.name;
-      if (name === "call_complete") return; // prior attempt finished — nothing stale
+      if (name === "call_complete") {
+        return;
+      } // prior attempt finished — nothing stale
       if (name === "request_sent") {
         failedRequestSent = i;
         break;
       }
     }
-    if (failedRequestSent === -1) return; // no prior attempt this turn
+    if (failedRequestSent === -1) {
+      return;
+    } // no prior attempt this turn
     // Drop from the failed attempt's paired setup mark (its `tools_resolved`,
     // if any) through its trailing marks, keeping a retry `tools_resolved`
     // already stamped as the last mark.
     let start = failedRequestSent;
-    if (this.marks[start - 1]?.name === "tools_resolved") start -= 1;
+    if (this.marks[start - 1]?.name === "tools_resolved") {
+      start -= 1;
+    }
     let end = this.marks.length;
-    if (this.marks[end - 1]!.name === "tools_resolved") end -= 1;
+    if (this.marks[end - 1]!.name === "tools_resolved") {
+      end -= 1;
+    }
     this.marks.splice(start, end - start);
   }
 
@@ -158,7 +168,9 @@ export class TurnLatencyTracker {
     cursor: number;
   } {
     const end = this.marks.length;
-    if (end === 0 || cursor >= end) return { breakdown: null, cursor: end };
+    if (end === 0 || cursor >= end) {
+      return { breakdown: null, cursor: end };
+    }
 
     const phases: LatencyPhase[] = [];
     // Start at max(cursor, 1): the first mark of the whole turn has no
@@ -166,11 +178,15 @@ export class TurnLatencyTracker {
     // is measured against the previous segment's last mark (index cursor-1).
     for (let i = Math.max(cursor, 1); i < end; i++) {
       const meta = PHASE_BY_CLOSING_MARK[this.marks[i]!.name];
-      if (!meta) continue;
+      if (!meta) {
+        continue;
+      }
       const ms = this.marks[i]!.at - this.marks[i - 1]!.at;
       // A non-monotonic clock would yield a negative span; drop it rather
       // than render misleading noise.
-      if (ms < 0) continue;
+      if (ms < 0) {
+        continue;
+      }
       const phase: LatencyPhase = { key: meta.key, label: meta.label, ms };
       const subPhases = this.subSpansByPhase.get(meta.key);
       if (subPhases) {
@@ -179,7 +195,9 @@ export class TurnLatencyTracker {
       }
       phases.push(phase);
     }
-    if (phases.length === 0) return { breakdown: null, cursor: end };
+    if (phases.length === 0) {
+      return { breakdown: null, cursor: end };
+    }
 
     const breakdown: LatencyBreakdown = { phases };
     const requestSent = this.findMarkFrom("request_sent", cursor)?.at;
@@ -202,7 +220,9 @@ export class TurnLatencyTracker {
     }
     // Per-call kind, read off this segment's own `first_token` mark (a pure
     // tool-call response streams none, leaving it undefined).
-    if (firstTokenMark?.kind) breakdown.firstTokenKind = firstTokenMark.kind;
+    if (firstTokenMark?.kind) {
+      breakdown.firstTokenKind = firstTokenMark.kind;
+    }
     return { breakdown, cursor: end };
   }
 
@@ -211,7 +231,9 @@ export class TurnLatencyTracker {
     from: number,
   ): { at: number; kind?: "thinking" | "text" } | undefined {
     for (let i = Math.max(from, 0); i < this.marks.length; i++) {
-      if (this.marks[i]!.name === name) return this.marks[i]!;
+      if (this.marks[i]!.name === name) {
+        return this.marks[i]!;
+      }
     }
     return undefined;
   }

package/src/daemon/verification-session-intent.ts

@@ -74,7 +74,9 @@ function extractChannelHint(
   text: string,
 ): "phone" | "telegram" | "slack" | undefined {
   for (const { pattern, channel } of CHANNEL_HINT_PATTERNS) {
-    if (pattern.test(text)) return channel;
+    if (pattern.test(text)) {
+      return channel;
+    }
   }
   return undefined;
 }

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

@@ -176,7 +176,9 @@ export function emitWakeAgentEvent(
     event,
     conversation.conversationId,
   );
-  if (!frame) {return;}
+  if (!frame) {
+    return;
+  }
   broadcastMessage(frame);
 }
 

package/src/daemon/web-search-history.ts

@@ -47,7 +47,9 @@ export function stripHistoricalWebSearchResults(
     const transformed: ContentBlock[] = [];
 
     for (const block of msg.content) {
-      if (block.type !== "web_search_tool_result") continue;
+      if (block.type !== "web_search_tool_result") {
+        continue;
+      }
       const wsr = block as WebSearchToolResultContent;
       const query = findQueryForToolUseId(msg.content, wsr.tool_use_id);
       transformed.push(formatAsText(wsr, query));
@@ -55,7 +57,9 @@ export function stripHistoricalWebSearchResults(
       stats.blocksStripped++;
     }
 
-    if (droppedServerToolUseIds.size === 0) return msg;
+    if (droppedServerToolUseIds.size === 0) {
+      return msg;
+    }
 
     const rewritten: ContentBlock[] = [];
     let wsrIndex = 0;
@@ -86,9 +90,13 @@ function findQueryForToolUseId(
   toolUseId: string,
 ): string | null {
   for (const b of blocks) {
-    if (b.type !== "server_tool_use") continue;
+    if (b.type !== "server_tool_use") {
+      continue;
+    }
     const stu = b as ServerToolUseContent;
-    if (stu.id !== toolUseId) continue;
+    if (stu.id !== toolUseId) {
+      continue;
+    }
     const q = stu.input?.query;
     return typeof q === "string" ? q : null;
   }

package/src/documents/document-store.ts

@@ -323,7 +323,9 @@ export function findInDocument(
       /*sql*/ `SELECT content FROM documents WHERE surface_id = ?`,
       surfaceId,
     );
-    if (!row) return null;
+    if (!row) {
+      return null;
+    }
 
     const lines = row.content.split("\n");
     const matches: FindMatch[] = [];
@@ -347,7 +349,9 @@ export function findInDocument(
               matchText: m[0],
             });
           }
-          if (m[0].length === 0) re.lastIndex++;
+          if (m[0].length === 0) {
+            re.lastIndex++;
+          }
         }
       }
     } else {
@@ -358,7 +362,9 @@ export function findInDocument(
         let startPos = 0;
         while (startPos <= haystack.length) {
           const idx = haystack.indexOf(needle, startPos);
-          if (idx === -1) break;
+          if (idx === -1) {
+            break;
+          }
           uncappedTotal++;
           if (matches.length < MAX_FIND_MATCHES) {
             matches.push({
@@ -505,7 +511,9 @@ export function replaceInDocument(
       let lastIndex = 0;
       let m: RegExpExecArray | null;
       while ((m = pattern.exec(row.content)) !== null) {
-        if (count >= limit) break;
+        if (count >= limit) {
+          break;
+        }
         result += row.content.slice(lastIndex, m.index);
         const singleMatchPattern = new RegExp(
           pattern.source,
@@ -514,7 +522,9 @@ export function replaceInDocument(
         result += m[0].replace(singleMatchPattern, replace);
         lastIndex = m.index + m[0].length;
         count++;
-        if (m[0].length === 0) pattern.lastIndex++;
+        if (m[0].length === 0) {
+          pattern.lastIndex++;
+        }
       }
       result += row.content.slice(lastIndex);
       newContent = result;

package/src/followups/followup-store.ts

@@ -51,7 +51,9 @@ export function createFollowUp(input: FollowUpCreateInput): FollowUp {
 function getFollowUp(id: string): FollowUp | null {
   const db = getDb();
   const row = db.select().from(followups).where(eq(followups.id, id)).get();
-  if (!row) return null;
+  if (!row) {
+    return null;
+  }
   return parseFollowUp(row);
 }
 
@@ -94,7 +96,9 @@ export function resolveFollowUp(id: string): FollowUp {
     .from(followups)
     .where(eq(followups.id, id))
     .get();
-  if (!existing) throw new Error(`Follow-up "${id}" not found`);
+  if (!existing) {
+    throw new Error(`Follow-up "${id}" not found`);
+  }
 
   db.update(followups)
     .set({ status: "resolved", updatedAt: now })
@@ -128,7 +132,9 @@ export function resolveByConversation(
     )
     .all();
 
-  if (rows.length === 0) return [];
+  if (rows.length === 0) {
+    return [];
+  }
 
   for (const row of rows) {
     db.update(followups)

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

@@ -125,7 +125,9 @@ export function completeHeartbeatRun(
     .from(heartbeatRuns)
     .where(eq(heartbeatRuns.id, runId))
     .get();
-  if (!row) return false;
+  if (!row) {
+    return false;
+  }
 
   const durationMs = row.startedAt != null ? now - row.startedAt : null;
 

package/src/home/conversation-starter-checkpoints.ts

@@ -27,7 +27,9 @@ export function checkpointKey(base: string): string {
 }
 
 export function parseCheckpointInt(value: string | undefined): number | null {
-  if (value == null) return null;
+  if (value == null) {
+    return null;
+  }
   const n = parseInt(value, 10);
   return Number.isNaN(n) ? null : n;
 }

package/src/home/feed-source-enrichment.test.ts

@@ -34,6 +34,13 @@ describe("classifyConversationSource", () => {
     expect(classifyConversationSource("home-feed")).toBe("user");
   });
 
+  it("classifies imported conversations ('import:<provider>') as user", () => {
+    expect(classifyConversationSource("import:chatgpt")).toBe("user");
+    expect(classifyConversationSource("import:claude")).toBe("user");
+    // Bare "import" without the prefix separator is not an import source.
+    expect(classifyConversationSource("import")).toBe("other");
+  });
+
   it("falls through to 'other' for unknown / paired-delivery / empty sources", () => {
     expect(classifyConversationSource("notification")).toBe("other");
     expect(classifyConversationSource("something-else")).toBe("other");

package/src/home/feed-source-enrichment.ts

@@ -57,6 +57,11 @@ export interface FeedSourceEnrichmentDeps {
 export function classifyConversationSource(
   source: string | null | undefined,
 ): FeedItemSourceType {
+  // Imported history (`import:<provider>`) is a user conversation that
+  // happened elsewhere; present it like any other user conversation.
+  if (source?.startsWith("import:")) {
+    return "user";
+  }
   switch (source) {
     case "heartbeat":
       return "heartbeat";
@@ -115,7 +120,9 @@ export function enrichFeedItemsWithSource(
   items: FeedItem[],
   deps: FeedSourceEnrichmentDeps = {},
 ): FeedItem[] {
-  if (items.length === 0) return items;
+  if (items.length === 0) {
+    return items;
+  }
 
   const getConversationRow =
     deps.getConversationRow ?? defaultGetConversationRow;
@@ -123,7 +130,9 @@ export function enrichFeedItemsWithSource(
 
   const convCache = new Map<string, ConversationSourceRow | null>();
   const resolveConv = (id: string): ConversationSourceRow | null => {
-    if (!convCache.has(id)) convCache.set(id, getConversationRow(id));
+    if (!convCache.has(id)) {
+      convCache.set(id, getConversationRow(id));
+    }
     return convCache.get(id) ?? null;
   };
 

package/src/home/feed-writer.ts

@@ -413,9 +413,15 @@ async function runWrite(): Promise<void> {
     let count = 0;
     for (let i = 0; i < items.length; i++) {
       const current = items[i]!;
-      if (current.status === op.to) continue;
-      if (!fromSet.has(current.status)) continue;
-      if (idSet && !idSet.has(current.id)) continue;
+      if (current.status === op.to) {
+        continue;
+      }
+      if (!fromSet.has(current.status)) {
+        continue;
+      }
+      if (idSet && !idSet.has(current.id)) {
+        continue;
+      }
       items[i] = { ...current, status: op.to };
       count++;
     }
@@ -495,9 +501,13 @@ function mergeIncoming(items: FeedItem[], incoming: FeedItem): FeedItem[] {
  * expired (fail-open).
  */
 function isExpired(item: FeedItem, nowMs: number): boolean {
-  if (!item.expiresAt) return false;
+  if (!item.expiresAt) {
+    return false;
+  }
   const expiresMs = Date.parse(item.expiresAt);
-  if (Number.isNaN(expiresMs)) return false;
+  if (Number.isNaN(expiresMs)) {
+    return false;
+  }
   return expiresMs <= nowMs;
 }
 
@@ -507,12 +517,20 @@ function isExpired(item: FeedItem, nowMs: number): boolean {
  * items sort to the top of the feed.
  */
 function compareFeedItems(a: FeedItem, b: FeedItem): number {
-  if (a.priority !== b.priority) return b.priority - a.priority;
+  if (a.priority !== b.priority) {
+    return b.priority - a.priority;
+  }
   const aMs = Date.parse(a.createdAt);
   const bMs = Date.parse(b.createdAt);
-  if (Number.isNaN(aMs) && Number.isNaN(bMs)) return 0;
-  if (Number.isNaN(aMs)) return 1;
-  if (Number.isNaN(bMs)) return -1;
+  if (Number.isNaN(aMs) && Number.isNaN(bMs)) {
+    return 0;
+  }
+  if (Number.isNaN(aMs)) {
+    return 1;
+  }
+  if (Number.isNaN(bMs)) {
+    return -1;
+  }
   return bMs - aMs;
 }
 

package/src/home/home-greeting-cache.ts

@@ -35,7 +35,9 @@ const IDENTITY_FILES = ["IDENTITY.md", "SOUL.md"] as const;
 function readWorkspaceFile(name: string): string {
   try {
     const path = getWorkspacePromptPath(name);
-    if (!existsSync(path)) return "";
+    if (!existsSync(path)) {
+      return "";
+    }
     return readFileSync(path, "utf-8");
   } catch {
     return "";

package/src/home/job-handlers/conversation-starters.ts

@@ -53,7 +53,9 @@ function buildMemoryRollup(): string {
   }>;
   try {
     const db = getMemoryDb();
-    if (!db) return "";
+    if (!db) {
+      return "";
+    }
     rows = db
       .select({
         type: memoryGraphNodes.type,
@@ -70,7 +72,9 @@ function buildMemoryRollup(): string {
     return "";
   }
 
-  if (rows.length === 0) return "";
+  if (rows.length === 0) {
+    return "";
+  }
 
   const byKind = new Map<string, string[]>();
   for (const item of rows) {
@@ -96,7 +100,9 @@ function buildNewItemsDiff(): string {
   const lastGenAt =
     parseCheckpointInt(getCheckpointValue(checkpointKey(CK_LAST_GEN_AT))) ?? 0;
 
-  if (lastGenAt === 0) return ""; // No previous generation — skip diff
+  if (lastGenAt === 0) {
+    return "";
+  } // No previous generation — skip diff
 
   const newItems = rawMemoryAll<{
     kind: string;
@@ -109,7 +115,9 @@ function buildNewItemsDiff(): string {
     lastGenAt,
   );
 
-  if (newItems.length === 0) return "";
+  if (newItems.length === 0) {
+    return "";
+  }
 
   return (
     "## New since last generation\n" +
@@ -127,7 +135,9 @@ function buildNewItemsDiff(): string {
 function buildSkillsSummary(): string {
   try {
     const catalog = loadSkillCatalog();
-    if (catalog.length === 0) return "";
+    if (catalog.length === 0) {
+      return "";
+    }
 
     const lines = catalog
       .filter((s) => s.description && s.displayName)

package/src/home/post-connect-feed.ts

@@ -28,7 +28,9 @@ const EMAIL_SERVICES = new Set(["google"]);
  * the feed writer's warn-log contract absorbs persistence failures.
  */
 export async function emitPostConnectNudge(service: string): Promise<void> {
-  if (!EMAIL_SERVICES.has(service)) return;
+  if (!EMAIL_SERVICES.has(service)) {
+    return;
+  }
 
   const now = new Date();
   const expiresAt = new Date(

package/src/home/progress-formula.ts

@@ -78,9 +78,13 @@ export function computeTier(input: ProgressInput): 1 | 2 | 3 | 4 {
   ).length;
   const convs = input.conversationCount;
   // "Hitting our stride" — real working relationship.
-  if (convs >= 20 && unlocked >= 3) return 3;
+  if (convs >= 20 && unlocked >= 3) {
+    return 3;
+  }
   // "Finding my footing" — starting to understand how they work.
-  if (convs >= 5 && input.facts.length >= 3) return 2;
+  if (convs >= 5 && input.facts.length >= 3) {
+    return 2;
+  }
   // "Getting to know you" — default for fresh relationships.
   return 1;
 }