@vellumai/assistant 0.11.0 → 0.11.1-staging.2

package/src/contacts/contact-store.ts

@@ -128,11 +128,15 @@ export function generateUserFileSlug(displayName: string): string {
   const taken = new Set(rows.map((r) => r.userFile?.toLowerCase()));
 
   const base = `${slug}.md`;
-  if (!taken.has(base)) return base;
+  if (!taken.has(base)) {
+    return base;
+  }
 
   for (let i = 2; ; i++) {
     const candidate = `${slug}-${i}.md`;
-    if (!taken.has(candidate)) return candidate;
+    if (!taken.has(candidate)) {
+      return candidate;
+    }
   }
 }
 
@@ -199,7 +203,9 @@ interface SyncChannelData {
 export function getContact(id: string): ContactWithChannels | null {
   const db = getDb();
   const row = db.select().from(contacts).where(eq(contacts.id, id)).get();
-  if (!row) return null;
+  if (!row) {
+    return null;
+  }
   return withChannels(parseContact(row));
 }
 
@@ -217,7 +223,9 @@ export interface ContactInfo {
  */
 export function findContactInfoById(contactId: string): ContactInfo | null {
   const contact = getContact(contactId);
-  if (!contact) return null;
+  if (!contact) {
+    return null;
+  }
   return {
     notes: contact.notes,
   };
@@ -280,10 +288,15 @@ export function upsertContact(params: {
         displayName: params.displayName,
         updatedAt: now,
       };
-      if (params.notes !== undefined) updateSet.notes = params.notes;
-      if (params.contactType !== undefined)
+      if (params.notes !== undefined) {
+        updateSet.notes = params.notes;
+      }
+      if (params.contactType !== undefined) {
         updateSet.contactType = params.contactType;
-      if (params.userFile !== undefined) updateSet.userFile = params.userFile;
+      }
+      if (params.userFile !== undefined) {
+        updateSet.userFile = params.userFile;
+      }
 
       db.update(contacts)
         .set(updateSet)
@@ -315,10 +328,15 @@ export function upsertContact(params: {
           displayName: params.displayName,
           updatedAt: now,
         };
-        if (params.notes !== undefined) updateSet.notes = params.notes;
-        if (params.contactType !== undefined)
+        if (params.notes !== undefined) {
+          updateSet.notes = params.notes;
+        }
+        if (params.contactType !== undefined) {
           updateSet.contactType = params.contactType;
-        if (params.userFile !== undefined) updateSet.userFile = params.userFile;
+        }
+        if (params.userFile !== undefined) {
+          updateSet.userFile = params.userFile;
+        }
 
         db.update(contacts)
           .set(updateSet)
@@ -405,7 +423,9 @@ function syncChannels(
         const crossContact = byId.contactId !== contactId;
         // Never steal a channel the gateway left under another contact unless
         // the caller opts into reassignment (mirrors the address-conflict path).
-        if (crossContact && !reassignConflicting) continue;
+        if (crossContact && !reassignConflicting) {
+          continue;
+        }
 
         const updateSet: Record<string, unknown> = { updatedAt: now };
         if (byId.address !== ch.address) {
@@ -428,10 +448,15 @@ function syncChannels(
             updateSet.address = ch.address;
           }
         }
-        if (crossContact) updateSet.contactId = contactId;
-        if (ch.isPrimary !== undefined) updateSet.isPrimary = ch.isPrimary;
-        if (ch.externalChatId !== undefined)
+        if (crossContact) {
+          updateSet.contactId = contactId;
+        }
+        if (ch.isPrimary !== undefined) {
+          updateSet.isPrimary = ch.isPrimary;
+        }
+        if (ch.externalChatId !== undefined) {
           updateSet.externalChatId = ch.externalChatId;
+        }
 
         db.update(contactChannels)
           .set(updateSet)
@@ -458,10 +483,15 @@ function syncChannels(
     if (existing) {
       const updateSet: Record<string, unknown> = {};
       // Self-heal legacy lowercased addresses to canonical form.
-      if (existing.address !== ch.address) updateSet.address = ch.address;
-      if (ch.isPrimary !== undefined) updateSet.isPrimary = ch.isPrimary;
-      if (ch.externalChatId !== undefined)
+      if (existing.address !== ch.address) {
+        updateSet.address = ch.address;
+      }
+      if (ch.isPrimary !== undefined) {
+        updateSet.isPrimary = ch.isPrimary;
+      }
+      if (ch.externalChatId !== undefined) {
         updateSet.externalChatId = ch.externalChatId;
+      }
 
       if (Object.keys(updateSet).length > 0) {
         updateSet.updatedAt = now;
@@ -484,9 +514,12 @@ function syncChannels(
           contactId,
           updatedAt: now,
         };
-        if (ch.externalChatId !== undefined)
+        if (ch.externalChatId !== undefined) {
           reassignSet.externalChatId = ch.externalChatId;
-        if (ch.isPrimary !== undefined) reassignSet.isPrimary = ch.isPrimary;
+        }
+        if (ch.isPrimary !== undefined) {
+          reassignSet.isPrimary = ch.isPrimary;
+        }
 
         db.update(contactChannels)
           .set(reassignSet)
@@ -526,7 +559,9 @@ export function searchContacts(params: {
   // Search by channel address first (exact or partial match)
   if (params.channelAddress) {
     const escapedAddress = escapeLike(params.channelAddress);
-    if (!escapedAddress) return [];
+    if (!escapedAddress) {
+      return [];
+    }
     const channelRows = db
       .select({ contactId: contactChannels.contactId })
       .from(contactChannels)
@@ -542,7 +577,9 @@ export function searchContacts(params: {
       .all();
 
     const contactIds = [...new Set(channelRows.map((r) => r.contactId))];
-    if (contactIds.length === 0) return [];
+    if (contactIds.length === 0) {
+      return [];
+    }
 
     // Pre-compute the sanitized query for display-name filtering so the
     // loop body stays cheap.
@@ -552,7 +589,9 @@ export function searchContacts(params: {
 
     const results: ContactWithChannels[] = [];
     for (const id of contactIds) {
-      if (results.length >= limit) break;
+      if (results.length >= limit) {
+        break;
+      }
       const contact = getContact(id);
       if (
         contact &&
@@ -577,11 +616,15 @@ export function searchContacts(params: {
       .all();
 
     const contactIds = [...new Set(channelRows.map((r) => r.contactId))];
-    if (contactIds.length === 0) return [];
+    if (contactIds.length === 0) {
+      return [];
+    }
 
     const results: ContactWithChannels[] = [];
     for (const id of contactIds) {
-      if (results.length >= limit) break;
+      if (results.length >= limit) {
+        break;
+      }
       const contact = getContact(id);
       if (
         contact &&
@@ -597,7 +640,9 @@ export function searchContacts(params: {
   const conditions = [];
   if (params.query) {
     const sanitized = escapeLike(params.query);
-    if (!sanitized && !params.contactType) return [];
+    if (!sanitized && !params.contactType) {
+      return [];
+    }
     if (sanitized) {
       conditions.push(like(contacts.displayName, `%${sanitized}%`));
     }
@@ -624,11 +669,15 @@ export function searchContacts(params: {
       .all();
 
     const contactIds = [...new Set(rows.map((r) => r.contactId))];
-    if (contactIds.length === 0) return [];
+    if (contactIds.length === 0) {
+      return [];
+    }
 
     const results: ContactWithChannels[] = [];
     for (const id of contactIds) {
-      if (results.length >= limit) break;
+      if (results.length >= limit) {
+        break;
+      }
       const contact = getContact(id);
       if (contact) {
         results.push(contact);
@@ -688,7 +737,9 @@ export function mergeContactMirror(params: {
       .from(contacts)
       .where(eq(contacts.id, params.mergeContactId))
       .get();
-    if (!donor) return false;
+    if (!donor) {
+      return false;
+    }
 
     const now = Date.now();
     const survivor = tx
@@ -725,7 +776,9 @@ export function mergeContactMirror(params: {
     return true;
   });
 
-  if (applied) notifyContactsChanged();
+  if (applied) {
+    notifyContactsChanged();
+  }
 }
 
 /**
@@ -775,7 +828,9 @@ export function upsertContactMirrorFull(params: {
       if (params.displayName !== undefined) {
         updateSet.displayName = params.displayName;
       }
-      if (params.notes !== undefined) updateSet.notes = params.notes;
+      if (params.notes !== undefined) {
+        updateSet.notes = params.notes;
+      }
       if (params.contactType !== undefined) {
         updateSet.contactType = params.contactType;
       }
@@ -848,7 +903,9 @@ export function upsertContactMirrorFull(params: {
       }
 
       // Address owned by ANOTHER contact is never stolen.
-      if (findConflictingChannel(tx, ch.type, ch.address)) continue;
+      if (findConflictingChannel(tx, ch.type, ch.address)) {
+        continue;
+      }
 
       tx.insert(contactChannels)
         .values({
@@ -891,7 +948,9 @@ export function findContactByAddress(
     )
     .get();
 
-  if (!channel) return null;
+  if (!channel) {
+    return null;
+  }
   return getContact(channel.contactId);
 }
 
@@ -917,7 +976,9 @@ function findContactByChannelExternalChatId(
     )
     .orderBy(desc(contactChannels.updatedAt), desc(contactChannels.createdAt))
     .get();
-  if (!channel) return null;
+  if (!channel) {
+    return null;
+  }
   return getContact(channel.contactId);
 }
 
@@ -943,7 +1004,9 @@ export function findContactChannel(params: {
           c.type === params.channelType &&
           c.address.toLowerCase() === canonical.toLowerCase(),
       );
-      if (ch) return { contact, channel: ch };
+      if (ch) {
+        return { contact, channel: ch };
+      }
     }
   }
   if (params.externalChatId) {
@@ -957,7 +1020,9 @@ export function findContactChannel(params: {
           c.type === params.channelType &&
           c.externalChatId === params.externalChatId,
       );
-      if (ch) return { contact, channel: ch };
+      if (ch) {
+        return { contact, channel: ch };
+      }
     }
   }
   return null;
@@ -985,7 +1050,9 @@ export function repairChannelAddress(
     .from(contactChannels)
     .where(eq(contactChannels.id, channelId))
     .get();
-  if (!channel) return false;
+  if (!channel) {
+    return false;
+  }
 
   // Guard: check if another channel row holds this canonical identity.
   const conflicting = findConflictingChannel(db, channel.type, newAddress);
@@ -1030,6 +1097,8 @@ export function getAssistantContactMetadata(
     .where(eq(assistantContactMetadata.contactId, contactId))
     .get();
 
-  if (!row) return null;
+  if (!row) {
+    return null;
+  }
   return parseAssistantMetadata(row);
 }

package/src/contacts/guardian-contact-reader.ts

@@ -26,7 +26,9 @@ let cache: { ids: Set<string>; expiresAt: number } | null = null;
  * returns an empty set on IPC error (never throws).
  */
 export async function getGuardianContactIds(): Promise<Set<string>> {
-  if (cache && cache.expiresAt > Date.now()) return cache.ids;
+  if (cache && cache.expiresAt > Date.now()) {
+    return cache.ids;
+  }
 
   try {
     const result = await ipcCallPersistent("get_guardian_contact", {});

package/src/contacts/guardian-delivery-reader.ts

@@ -55,7 +55,9 @@ const inFlight = new Map<
 let cacheGeneration = 0;
 
 function cacheKey(channelTypes?: string[]): string {
-  if (!channelTypes || channelTypes.length === 0) return "ALL";
+  if (!channelTypes || channelTypes.length === 0) {
+    return "ALL";
+  }
   return [...channelTypes].sort().join(",");
 }
 
@@ -68,7 +70,9 @@ async function fetchGuardianDelivery(input: {
       input,
       GUARDIAN_DELIVERY_IPC_TIMEOUT_MS,
     );
-    if (!result) return null;
+    if (!result) {
+      return null;
+    }
 
     const parsed = ResolveGuardianDeliveryResponseSchema.safeParse(result);
     return parsed.success ? parsed.data.guardians : null;
@@ -100,7 +104,9 @@ async function readGuardianDelivery(input: {
   // only coalesce with another force fetch — never with a non-force fetch that
   // could have started before a gateway-side binding write and resolve stale.
   const pending = inFlight.get(key);
-  if (pending && (!input.forceRefresh || pending.fresh)) return pending.promise;
+  if (pending && (!input.forceRefresh || pending.fresh)) {
+    return pending.promise;
+  }
 
   const startGen = cacheGeneration;
   const promise = fetchGuardianDelivery({ channelTypes: input.channelTypes })
@@ -116,7 +122,9 @@ async function readGuardianDelivery(input: {
     .finally(() => {
       // Only clear the slot if it still holds this fetch — a concurrent force
       // read may have replaced a non-force entry (or vice versa).
-      if (inFlight.get(key)?.promise === promise) inFlight.delete(key);
+      if (inFlight.get(key)?.promise === promise) {
+        inFlight.delete(key);
+      }
     });
 
   inFlight.set(key, { promise, fresh: !!input.forceRefresh });
@@ -150,7 +158,9 @@ export function peekCachedGuardianDelivery(input?: {
   channelTypes?: string[];
 }): GuardianDelivery[] | undefined {
   const cached = cache.get(cacheKey(input?.channelTypes));
-  if (!cached) return undefined;
+  if (!cached) {
+    return undefined;
+  }
   if (Date.now() - cached.fetchedAt >= GUARDIAN_DELIVERY_CACHE_TTL_MS) {
     return undefined;
   }

package/src/context/compactor.ts

@@ -83,7 +83,9 @@ function recordCompactionRequestLog(
   response: ProviderResponse,
   provider: Provider,
 ): string | null {
-  if (!response.rawRequest || !response.rawResponse) return null;
+  if (!response.rawRequest || !response.rawResponse) {
+    return null;
+  }
   try {
     // Inserted unlinked (no message id) — user-initiated compaction flows
     // (/compact, summarize-up-to) link the row to their result card after
@@ -376,7 +378,9 @@ export function parseCompactionResult(
   opts: { requireTailStart?: boolean } = {},
 ): ParsedCompactionResult | null {
   const openIdx = raw.indexOf(RESULT_TAG_OPEN);
-  if (openIdx < 0) return null;
+  if (openIdx < 0) {
+    return null;
+  }
   const closeIdx = raw.lastIndexOf(RESULT_TAG_CLOSE);
   const inner =
     closeIdx > openIdx
@@ -384,7 +388,9 @@ export function parseCompactionResult(
       : raw.slice(openIdx + RESULT_TAG_OPEN.length);
 
   const summary = extractTagContent(inner, "summary")?.trim() ?? "";
-  if (summary.length === 0) return null;
+  if (summary.length === 0) {
+    return null;
+  }
 
   const keyState = extractTagContent(inner, "key_state")?.trim() ?? "";
 
@@ -411,9 +417,13 @@ function extractTagContent(haystack: string, tag: string): string | null {
   const open = `<${tag}>`;
   const close = `</${tag}>`;
   const openIdx = haystack.indexOf(open);
-  if (openIdx < 0) return null;
+  if (openIdx < 0) {
+    return null;
+  }
   const closeIdx = haystack.indexOf(close, openIdx + open.length);
-  if (closeIdx < 0) return null;
+  if (closeIdx < 0) {
+    return null;
+  }
   return haystack.slice(openIdx + open.length, closeIdx);
 }
 
@@ -425,7 +435,9 @@ function extractTailStart(
   const tagMatch = inner.match(
     /<tail_start\b([\s\S]*?)(?:\/>|<\/tail_start>)/i,
   );
-  if (!tagMatch) return null;
+  if (!tagMatch) {
+    return null;
+  }
   const attrs = tagMatch[1];
   const timestamp = extractAttr(attrs, "timestamp") ?? "";
   const preview = extractAttr(attrs, "preview") ?? "";
@@ -440,14 +452,18 @@ function extractAttr(attrs: string, name: string): string | null {
 
 function extractRetainedImages(inner: string): string[] {
   const block = extractTagContent(inner, "retained_images");
-  if (block == null) return [];
+  if (block == null) {
+    return [];
+  }
   const out: string[] = [];
   const seen = new Set<string>();
   const re = /<image\b[^>]*\bfile\s*=\s*"([^"]+)"[^>]*\/?>/gi;
   let m: RegExpExecArray | null;
   while ((m = re.exec(block)) !== null) {
     const name = m[1].trim();
-    if (name.length === 0 || seen.has(name)) continue;
+    if (name.length === 0 || seen.has(name)) {
+      continue;
+    }
     seen.add(name);
     out.push(name);
   }
@@ -498,7 +514,9 @@ export function collectImageManifest(
   for (const row of rows) {
     const atts = getAttachmentMetadataForMessage(row.id);
     for (const att of atts) {
-      if (att.kind !== "image") continue;
+      if (att.kind !== "image") {
+        continue;
+      }
       entries.push({
         filename: att.originalFilename,
         attachmentId: att.id,
@@ -511,7 +529,9 @@ export function collectImageManifest(
 }
 
 export function renderImageManifest(entries: ManifestEntry[]): string {
-  if (entries.length === 0) return "(no images in this conversation)";
+  if (entries.length === 0) {
+    return "(no images in this conversation)";
+  }
   return entries
     .map((e) => {
       const ts = new Date(e.timestamp).toISOString();
@@ -531,16 +551,24 @@ export function renderImageManifest(entries: ManifestEntry[]): string {
  * `2026-04-02 (Thursday) 01:52:33 -05:00 (America/Chicago)`).
  */
 export function extractTurnContextTimestamp(message: Message): string | null {
-  if (message.role !== "user") return null;
+  if (message.role !== "user") {
+    return null;
+  }
   for (const block of message.content) {
-    if (block.type !== "text") continue;
+    if (block.type !== "text") {
+      continue;
+    }
     const text = block.text;
     const idx = text.indexOf("<turn_context>");
-    if (idx < 0) continue;
+    if (idx < 0) {
+      continue;
+    }
     const end = text.indexOf("</turn_context>", idx);
     const slice = end > 0 ? text.slice(idx, end) : text.slice(idx);
     const m = slice.match(/current_time:\s*([^\n]+)/);
-    if (m) return m[1].trim();
+    if (m) {
+      return m[1].trim();
+    }
   }
   return null;
 }
@@ -559,16 +587,22 @@ function buildTimestampIndex(messages: Message[]): (string | null)[] {
 
 function extractFirstTextPreview(message: Message, maxChars = 120): string {
   for (const block of message.content) {
-    if (block.type !== "text") continue;
+    if (block.type !== "text") {
+      continue;
+    }
     let text = block.text;
     // Skip injected blocks (`<turn_context>`, `<memory>`, `<workspace>`, ...) —
     // they're not what the model means by "first 60 chars of that message".
     while (text.startsWith("<") && text.includes("</")) {
       const closeMatch = text.match(/<\/[a-zA-Z_][\w-]*>\s*\n?/);
-      if (!closeMatch || closeMatch.index === undefined) break;
+      if (!closeMatch || closeMatch.index === undefined) {
+        break;
+      }
       text = text.slice(closeMatch.index + closeMatch[0].length).trimStart();
     }
-    if (text.length === 0) continue;
+    if (text.length === 0) {
+      continue;
+    }
     return text.slice(0, maxChars);
   }
   return "";
@@ -610,7 +644,9 @@ function describeFixedBoundary(message: Message): string {
  */
 export function canonicalDateTimeKey(ts: string): string | null {
   const m = ts.match(/(\d{4}-\d{2}-\d{2})\D+(\d{2}:\d{2}:\d{2})/);
-  if (!m) return null;
+  if (!m) {
+    return null;
+  }
   return `${m[1]}T${m[2]}`;
 }
 
@@ -631,18 +667,26 @@ function resolveTailStartIndex(
   const wantedTs = parsed.tailStartTimestamp.trim();
   if (wantedTs.length > 0) {
     for (let i = 0; i < timestamps.length; i++) {
-      if (timestamps[i] === wantedTs) return i;
+      if (timestamps[i] === wantedTs) {
+        return i;
+      }
     }
     for (let i = 0; i < timestamps.length; i++) {
       const ts = timestamps[i];
-      if (ts && (ts.includes(wantedTs) || wantedTs.includes(ts))) return i;
+      if (ts && (ts.includes(wantedTs) || wantedTs.includes(ts))) {
+        return i;
+      }
     }
     const wantedKey = canonicalDateTimeKey(wantedTs);
     if (wantedKey) {
       for (let i = 0; i < timestamps.length; i++) {
         const ts = timestamps[i];
-        if (!ts) continue;
-        if (canonicalDateTimeKey(ts) === wantedKey) return i;
+        if (!ts) {
+          continue;
+        }
+        if (canonicalDateTimeKey(ts) === wantedKey) {
+          return i;
+        }
       }
     }
   }
@@ -651,9 +695,13 @@ function resolveTailStartIndex(
     const previewHead = wantedPreview.slice(0, 40);
     for (let i = 0; i < messages.length; i++) {
       const m = messages[i];
-      if (m.role !== "user") continue;
+      if (m.role !== "user") {
+        continue;
+      }
       const head = extractFirstTextPreview(m);
-      if (head.length > 0 && head.startsWith(previewHead)) return i;
+      if (head.length > 0 && head.startsWith(previewHead)) {
+        return i;
+      }
     }
   }
   return null;
@@ -707,7 +755,9 @@ export function adjustTailIndexForToolPairing(
  */
 function isForwardCutBoundary(messages: Message[], index: number): boolean {
   const m = messages[index];
-  if (m == null || m.role !== "user") return false;
+  if (m == null || m.role !== "user") {
+    return false;
+  }
   // guard:allow-tool-result-only — server-side web_search_tool_result is
   // self-paired inside its assistant message and never spans user turns.
   return !m.content.some((block) => block.type === "tool_result");
@@ -766,7 +816,9 @@ function advanceTailForBudget(args: {
   let chosen = startIndex;
   let fits = false;
   for (let i = startIndex + 1; i <= floorIndex; i++) {
-    if (!isForwardCutBoundary(messages, i)) continue;
+    if (!isForwardCutBoundary(messages, i)) {
+      continue;
+    }
     chosen = i;
     const estimate = estimateTail(messages.slice(i));
     if (estimate <= targetTokens) {
@@ -797,9 +849,13 @@ function resolveTailFloorIndex(messages: Message[], tailIndex: number): number {
       break;
     }
   }
-  if (lastAssistant < 0) return tailIndex;
+  if (lastAssistant < 0) {
+    return tailIndex;
+  }
   for (let i = lastAssistant - 1; i > tailIndex; i--) {
-    if (isForwardCutBoundary(messages, i)) return i;
+    if (isForwardCutBoundary(messages, i)) {
+      return i;
+    }
   }
   return tailIndex;
 }
@@ -956,7 +1012,9 @@ export function buildSummaryMemoryText(
 ): string {
   const trimmedSummary = summary.trim();
   const trimmedKey = keyState.trim();
-  if (trimmedKey.length === 0) return trimmedSummary;
+  if (trimmedKey.length === 0) {
+    return trimmedSummary;
+  }
   return `${trimmedSummary}\n\n## Pending State\n${trimmedKey}`;
 }
 
@@ -1355,7 +1413,9 @@ export async function runAssistantDrivenCompaction(
     // budget and land short of the real low-watermark.
     const toolTokenBudget = args.tools ? estimateToolsTokens(args.tools) : 0;
     const fixedPrefix: Message[] = [summaryMessage];
-    if (retainedImageMessage) fixedPrefix.push(retainedImageMessage);
+    if (retainedImageMessage) {
+      fixedPrefix.push(retainedImageMessage);
+    }
     const estimateRebuilt = (tail: Message[]): number =>
       estimatePromptTokens(
         [...fixedPrefix, ...stripInjectionsForCompaction(tail)],
@@ -1560,9 +1620,13 @@ Structured list of:
 function findLastToolPairStart(messages: Message[]): number | null {
   for (let i = messages.length - 1; i >= 0; i--) {
     const msg = messages[i];
-    if (msg.role !== "assistant") continue;
+    if (msg.role !== "assistant") {
+      continue;
+    }
     const hasToolUse = msg.content.some((b) => b.type === "tool_use");
-    if (hasToolUse) return i;
+    if (hasToolUse) {
+      return i;
+    }
   }
   return null;
 }