@vellumai/assistant 0.11.0 → 0.11.1-staging.2

package/src/messaging/providers/slack/api.ts

@@ -57,7 +57,9 @@ const SLACK_ERROR_CODE_MAP: Record<string, SlackErrorCategory> = {
 };
 
 function classifySlackError(errorCode: string | undefined): SlackErrorCategory {
-  if (!errorCode) return "unknown";
+  if (!errorCode) {
+    return "unknown";
+  }
   return SLACK_ERROR_CODE_MAP[errorCode] ?? "unknown";
 }
 
@@ -335,13 +337,23 @@ export async function startSlackStream(params: {
     channel: params.channel,
     thread_ts: params.threadTs,
   };
-  if (params.markdownText) body.markdown_text = params.markdownText;
-  if (params.taskDisplayMode) body.task_display_mode = params.taskDisplayMode;
+  if (params.markdownText) {
+    body.markdown_text = params.markdownText;
+  }
+  if (params.taskDisplayMode) {
+    body.task_display_mode = params.taskDisplayMode;
+  }
   // Channel streams must name the reader; DMs infer it and omit both fields.
-  if (params.recipientUserId) body.recipient_user_id = params.recipientUserId;
-  if (params.recipientTeamId) body.recipient_team_id = params.recipientTeamId;
+  if (params.recipientUserId) {
+    body.recipient_user_id = params.recipientUserId;
+  }
+  if (params.recipientTeamId) {
+    body.recipient_team_id = params.recipientTeamId;
+  }
   const chunks = toStreamChunks(params);
-  if (chunks) body.chunks = chunks;
+  if (chunks) {
+    body.chunks = chunks;
+  }
 
   const data = await callSlackApi("chat.startStream", body);
   return data.ts;
@@ -365,9 +377,13 @@ export async function appendSlackStream(params: {
     channel: params.channel,
     ts: params.streamTs,
   };
-  if (params.markdownText) body.markdown_text = params.markdownText;
+  if (params.markdownText) {
+    body.markdown_text = params.markdownText;
+  }
   const chunks = toStreamChunks(params);
-  if (chunks) body.chunks = chunks;
+  if (chunks) {
+    body.chunks = chunks;
+  }
 
   await callSlackApi("chat.appendStream", body);
 }
@@ -390,16 +406,24 @@ export async function stopSlackStream(params: {
     channel: params.channel,
     ts: params.streamTs,
   };
-  if (params.markdownText) body.markdown_text = params.markdownText;
-  if (params.blocks && params.blocks.length > 0) body.blocks = params.blocks;
+  if (params.markdownText) {
+    body.markdown_text = params.markdownText;
+  }
+  if (params.blocks && params.blocks.length > 0) {
+    body.blocks = params.blocks;
+  }
   const chunks = toStreamChunks(params);
-  if (chunks) body.chunks = chunks;
+  if (chunks) {
+    body.chunks = chunks;
+  }
 
   await callSlackApi("chat.stopStream", body);
 }
 
 function normalizeSlackString(value: unknown): string | undefined {
-  if (typeof value !== "string") return undefined;
+  if (typeof value !== "string") {
+    return undefined;
+  }
   const trimmed = value.trim();
   return trimmed || undefined;
 }
@@ -413,7 +437,9 @@ export async function getSlackConversationInfo(
   )) as SlackConversationsInfoResponse;
 
   const id = normalizeSlackString(data.channel?.id);
-  if (!id) return null;
+  if (!id) {
+    return null;
+  }
 
   const name = normalizeSlackString(data.channel?.name);
   const nameNormalized = normalizeSlackString(data.channel?.name_normalized);

package/src/messaging/providers/slack/client.ts

@@ -220,7 +220,9 @@ async function requestViaToken<T extends SlackApiResponse>(
     if (params) {
       const searchParams = new URLSearchParams();
       for (const [k, v] of Object.entries(params)) {
-        if (v !== undefined) searchParams.set(k, v);
+        if (v !== undefined) {
+          searchParams.set(k, v);
+        }
       }
       url += `?${searchParams}`;
     }
@@ -420,8 +422,12 @@ export async function postMessage(
       ? { threadTs: optionsOrThreadTs }
       : (optionsOrThreadTs ?? {});
   const body: Record<string, unknown> = { channel, text };
-  if (opts.threadTs) body.thread_ts = opts.threadTs;
-  if (opts.blocks) body.blocks = opts.blocks;
+  if (opts.threadTs) {
+    body.thread_ts = opts.threadTs;
+  }
+  if (opts.blocks) {
+    body.blocks = opts.blocks;
+  }
   return request<SlackPostMessageResponse>(
     connectionOrToken,
     "chat.postMessage",

package/src/messaging/providers/slack/deep-link.ts

@@ -13,7 +13,9 @@ export function buildSlackAppMessageUrl(params: {
   messageTs: string;
 }): string | undefined {
   const teamId = params.teamId?.trim();
-  if (!teamId) return undefined;
+  if (!teamId) {
+    return undefined;
+  }
 
   const search = new URLSearchParams({
     team: teamId,
@@ -25,11 +27,15 @@ export function buildSlackAppMessageUrl(params: {
 
 function normalizeSlackTeamUrl(teamUrl?: string | null): string | undefined {
   const trimmed = teamUrl?.trim();
-  if (!trimmed) return undefined;
+  if (!trimmed) {
+    return undefined;
+  }
 
   try {
     const parsed = new URL(trimmed);
-    if (parsed.protocol !== "https:") return undefined;
+    if (parsed.protocol !== "https:") {
+      return undefined;
+    }
     return parsed.toString().replace(/\/+$/, "");
   } catch {
     return undefined;
@@ -45,7 +51,9 @@ function buildArchivesMessageUrl(
   const baseUrl = `${teamUrl}/archives/${encodeURIComponent(
     channelId,
   )}/p${formatSlackPermalinkTimestamp(messageTs)}`;
-  if (!threadTs) return baseUrl;
+  if (!threadTs) {
+    return baseUrl;
+  }
 
   const search = new URLSearchParams({
     thread_ts: threadTs,
@@ -61,7 +69,9 @@ export function buildSlackWebMessageUrl(params: {
   threadTs?: string;
 }): string | undefined {
   const teamUrl = normalizeSlackTeamUrl(params.teamUrl);
-  if (!teamUrl) return undefined;
+  if (!teamUrl) {
+    return undefined;
+  }
 
   return buildArchivesMessageUrl(
     teamUrl,

package/src/messaging/providers/slack/message-metadata.ts

@@ -137,7 +137,9 @@ function compactStoredSlackTimezoneLabel(
   label: string | null | undefined,
 ): string | null {
   const trimmed = label?.trim();
-  if (!trimmed) return null;
+  if (!trimmed) {
+    return null;
+  }
   return (
     COMPACT_SLACK_TIMEZONE_LABEL_BY_NAME.get(trimmed.toUpperCase()) ?? trimmed
   );
@@ -174,12 +176,18 @@ export function formatSlackTimezoneLabel(
   opts: { persistedLabel?: string | null; nowMs?: number } = {},
 ): string | null {
   const persisted = compactStoredSlackTimezoneLabel(opts.persistedLabel);
-  if (persisted) return persisted;
+  if (persisted) {
+    return persisted;
+  }
 
   const trimmedTimezone = timeZone?.trim();
-  if (!trimmedTimezone) return null;
+  if (!trimmedTimezone) {
+    return null;
+  }
   const mapped = COMMON_SLACK_TIMEZONE_LABEL_BY_IANA.get(trimmedTimezone);
-  if (mapped) return mapped;
+  if (mapped) {
+    return mapped;
+  }
   return (
     extractSlackShortTimeZoneName(trimmedTimezone, opts.nowMs ?? Date.now()) ??
     trimmedTimezone
@@ -252,7 +260,9 @@ export function readSlackMetadataFromMessageMetadata(
   metadata: string | null | undefined,
   opts?: { allowFlatLegacy?: boolean },
 ): SlackMessageMetadata | null {
-  if (!metadata) return null;
+  if (!metadata) {
+    return null;
+  }
 
   let parent: Record<string, unknown> | null = null;
   try {
@@ -263,12 +273,16 @@ export function readSlackMetadataFromMessageMetadata(
   } catch {
     return null;
   }
-  if (!parent) return null;
+  if (!parent) {
+    return null;
+  }
 
   const nested = parent.slackMeta;
   if (typeof nested === "string") {
     const parsedNested = readSlackMetadata(nested);
-    if (parsedNested) return parsedNested;
+    if (parsedNested) {
+      return parsedNested;
+    }
   }
 
   return opts?.allowFlatLegacy ? readSlackMetadata(metadata) : null;

package/src/messaging/providers/slack/render-transcript.ts

@@ -135,7 +135,9 @@ export function parentAlias(channelTs: string): string {
  */
 function formatSlackTs(channelTs: string): string {
   const seconds = Number.parseFloat(channelTs);
-  if (!Number.isFinite(seconds)) return "??/??/?? ??:??";
+  if (!Number.isFinite(seconds)) {
+    return "??/??/?? ??:??";
+  }
   return formatEpochMs(seconds * 1000);
 }
 
@@ -143,7 +145,9 @@ function formatSlackTs(channelTs: string): string {
  * Format an epoch millisecond timestamp as `MM/DD/YY HH:MM` (UTC).
  */
 function formatEpochMs(ms: number): string {
-  if (!Number.isFinite(ms)) return "??/??/?? ??:??";
+  if (!Number.isFinite(ms)) {
+    return "??/??/?? ??:??";
+  }
   const d = new Date(ms);
   const mo = String(d.getUTCMonth() + 1).padStart(2, "0");
   const da = String(d.getUTCDate()).padStart(2, "0");
@@ -208,7 +212,9 @@ function formatCompactEpochMs(
     }`;
   }
   const parts = compactDateTimeParts(ms, timeZone);
-  if (!parts) return formatEpochMs(ms);
+  if (!parts) {
+    return formatEpochMs(ms);
+  }
   const label =
     formatSlackTimezoneLabel(timeZone, {
       persistedLabel: timezoneLabel,
@@ -255,14 +261,18 @@ function speakerLabel(
       : formatSlackTimezoneLabel(undefined, {
           persistedLabel: meta.speakerTimezoneLabel,
         });
-  if (!speaker) return "";
+  if (!speaker) {
+    return "";
+  }
   return suffix ? `${speaker} (${suffix})` : speaker;
 }
 
 function renderSlackFileMarkers(
   files: SlackMessageMetadata["slackFiles"],
 ): string {
-  if (!files || files.length === 0) return "";
+  if (!files || files.length === 0) {
+    return "";
+  }
   return files
     .map((file) => {
       const name = file.name.replace(/\s+/g, " ").trim();
@@ -279,7 +289,9 @@ function appendSlackFileMarkers(
   files: SlackMessageMetadata["slackFiles"],
 ): string {
   const markers = renderSlackFileMarkers(files);
-  if (!markers) return content;
+  if (!markers) {
+    return content;
+  }
   return content.length > 0 ? `${content} ${markers}` : markers;
 }
 
@@ -293,7 +305,9 @@ function appendSlackFileMarkers(
 function sortKey(msg: RenderableSlackMessage): number {
   if (msg.metadata) {
     const n = Number.parseFloat(msg.metadata.channelTs);
-    if (Number.isFinite(n)) return n;
+    if (Number.isFinite(n)) {
+      return n;
+    }
   }
   // createdAt is epoch ms; convert to seconds for like-with-like comparison.
   return msg.createdAt / 1000;
@@ -313,8 +327,12 @@ export function isSlackTsAfter(ts: string, watermarkTs: string): boolean {
 }
 
 function maxNullableSlackTs(a: string | null, b: string | null): string | null {
-  if (a === null) return b;
-  if (b === null) return a;
+  if (a === null) {
+    return b;
+  }
+  if (b === null) {
+    return a;
+  }
   return compareSlackTs(a, b) >= 0 ? a : b;
 }
 
@@ -349,7 +367,9 @@ function renderMessage(msg: RenderableSlackMessage): string {
   const meta = msg.metadata;
 
   if (msg.role === "assistant") {
-    if (msg.metadata?.deletedAt !== undefined) return "[deleted]";
+    if (msg.metadata?.deletedAt !== undefined) {
+      return "[deleted]";
+    }
     return appendSlackFileMarkers(msg.content, msg.metadata?.slackFiles);
   }
 
@@ -455,7 +475,9 @@ function renderModelBody(msg: RenderableSlackMessage, body: string): string {
  */
 function renderReaction(msg: RenderableSlackMessage): string | null {
   const meta = msg.metadata;
-  if (!meta || meta.eventKind !== "reaction" || !meta.reaction) return null;
+  if (!meta || meta.eventKind !== "reaction" || !meta.reaction) {
+    return null;
+  }
   const time = hasTimestampTimezone(meta)
     ? formatCompactSlackTs(
         meta.channelTs,
@@ -599,7 +621,9 @@ export function renderSlackTranscriptWithProvenance(
   // Stable sort: decorate-sort-undecorate so equal keys preserve input order.
   const indexed = messages.map((m, i) => ({ m, i, k: sortKey(m) }));
   indexed.sort((a, b) => {
-    if (a.k !== b.k) return a.k - b.k;
+    if (a.k !== b.k) {
+      return a.k - b.k;
+    }
     return a.i - b.i;
   });
   const sorted = indexed.map((x) => x.m);
@@ -644,7 +668,9 @@ export function renderSlackTranscriptWithProvenance(
     keepTarget: string | null,
   ) => {
     for (const target of Array.from(overflowAccumulator.keys())) {
-      if (target === keepTarget) continue;
+      if (target === keepTarget) {
+        continue;
+      }
       const acc = overflowAccumulator.get(target)!;
       out.push(trailerMessage(target, acc));
       overflowAccumulator.delete(target);
@@ -695,7 +721,9 @@ export function renderSlackTranscriptWithProvenance(
     flushOverflowExcept(out, null);
     const tagLine = renderMessage(m);
     const blocks = buildMessageContentBlocks(m, tagLine);
-    if (blocks.length === 0) continue;
+    if (blocks.length === 0) {
+      continue;
+    }
     const hasRenderedText = blocks.some((block) => block.type === "text");
     out.push({
       message: {
@@ -749,9 +777,14 @@ function filterOrphanToolPairs(
   const consumed = new Set<string>();
   for (const { message: msg } of entries) {
     for (const b of msg.content) {
-      if (b.type === "tool_use") produced.add(b.id);
-      else if (b.type === "tool_result" || b.type === "web_search_tool_result")
+      if (b.type === "tool_use") {
+        produced.add(b.id);
+      } else if (
+        b.type === "tool_result" ||
+        b.type === "web_search_tool_result"
+      ) {
         consumed.add(b.tool_use_id);
+      }
     }
   }
   const out: RenderedSlackTranscriptMessage[] = [];
@@ -759,12 +792,15 @@ function filterOrphanToolPairs(
     const msg = entry.message;
     const kept: ContentBlock[] = [];
     for (const b of msg.content) {
-      if (b.type === "tool_use" && !consumed.has(b.id)) continue;
+      if (b.type === "tool_use" && !consumed.has(b.id)) {
+        continue;
+      }
       if (
         (b.type === "tool_result" || b.type === "web_search_tool_result") &&
         !produced.has(b.tool_use_id)
-      )
+      ) {
         continue;
+      }
       kept.push(b);
     }
     if (kept.length > 0) {

package/src/messaging/providers/slack/render.ts

@@ -84,7 +84,9 @@ const HEADING_FORMATTING_TYPES = new Set([
  * the input is empty so callers can skip the `blocks` field entirely.
  */
 export function renderSlackBlocks(text: string): KnownBlock[] | undefined {
-  if (!text || text.trim().length === 0) return undefined;
+  if (!text || text.trim().length === 0) {
+    return undefined;
+  }
   const blocks = renderSlack(parseMarkdown(text), text);
   return blocks.length > 0 ? blocks : undefined;
 }
@@ -101,7 +103,9 @@ export function renderSlack(tree: Root, source: string): KnownBlock[] {
 
   const flushRun = (): void => {
     const md = sliceRun(run, source);
-    if (md.length > 0) blocks.push({ type: "markdown", text: md });
+    if (md.length > 0) {
+      blocks.push({ type: "markdown", text: md });
+    }
     run = [];
   };
 
@@ -120,7 +124,9 @@ export function renderSlack(tree: Root, source: string): KnownBlock[] {
         // as its raw table markdown so it still shows, just not as a rendered
         // table.
         const md = sliceNode(node, source).trim();
-        if (md.length > 0) blocks.push({ type: "markdown", text: md });
+        if (md.length > 0) {
+          blocks.push({ type: "markdown", text: md });
+        }
       }
     } else if (node.type === "thematicBreak") {
       flushRun();
@@ -150,7 +156,9 @@ export function renderSlack(tree: Root, source: string): KnownBlock[] {
 function sliceNode(node: RootContent, source: string): string {
   const start = node.position?.start.offset;
   const end = node.position?.end.offset;
-  if (start === undefined || end === undefined) return "";
+  if (start === undefined || end === undefined) {
+    return "";
+  }
   return source.slice(start, end);
 }
 
@@ -161,10 +169,14 @@ function sliceNode(node: RootContent, source: string): string {
  * — is preserved verbatim. Returns "" for an empty run.
  */
 function sliceRun(run: RootContent[], source: string): string {
-  if (run.length === 0) return "";
+  if (run.length === 0) {
+    return "";
+  }
   const start = run[0]?.position?.start.offset;
   const end = run[run.length - 1]?.position?.end.offset;
-  if (start === undefined || end === undefined) return "";
+  if (start === undefined || end === undefined) {
+    return "";
+  }
   return source.slice(start, end).trim();
 }
 
@@ -210,8 +222,12 @@ function imageBlocksFromParagraph(node: Paragraph): ImageBlock[] | null {
       return null;
     }
   }
-  if (images.length === 0) return null;
-  if (!images.every((img) => isHostableImageUrl(img.url))) return null;
+  if (images.length === 0) {
+    return null;
+  }
+  if (!images.every((img) => isHostableImageUrl(img.url))) {
+    return null;
+  }
   return images.map((img) => ({
     type: "image",
     image_url: img.url,
@@ -251,8 +267,12 @@ function tryTableBlock(
 ): { block: TableBlock; cellChars: number } | null {
   const rows = node.children;
   const columnCount = Math.max(0, ...rows.map((row) => row.children.length));
-  if (columnCount === 0 || columnCount > SLACK_TABLE_MAX_COLUMNS) return null;
-  if (rows.length > SLACK_TABLE_MAX_ROWS) return null;
+  if (columnCount === 0 || columnCount > SLACK_TABLE_MAX_COLUMNS) {
+    return null;
+  }
+  if (rows.length > SLACK_TABLE_MAX_ROWS) {
+    return null;
+  }
 
   const cellNodesAt = (row: TableRow, c: number): PhrasingContent[] =>
     row.children[c]?.children ?? [];
@@ -270,8 +290,12 @@ function tryTableBlock(
       sum + row.reduce((rowSum, cell) => rowSum + cellTextLength(cell), 0),
     0,
   );
-  if (cellChars > SLACK_TABLE_MAX_TOTAL_CHARS) return null;
-  if (charsUsed + cellChars > SLACK_TABLE_MAX_TOTAL_CHARS) return null;
+  if (cellChars > SLACK_TABLE_MAX_TOTAL_CHARS) {
+    return null;
+  }
+  if (charsUsed + cellChars > SLACK_TABLE_MAX_TOTAL_CHARS) {
+    return null;
+  }
 
   const columnSettings = columnSettingsFor(node, columnCount);
   const block: TableBlock = columnSettings
@@ -309,13 +333,20 @@ function cellBlock(nodes: PhrasingContent[]): TableCell {
  * empty-alt images is the URL), so the estimate never undercounts the payload.
  */
 function cellTextLength(cell: TableCell): number {
-  if (cell.type === "raw_text") return cell.text.length;
+  if (cell.type === "raw_text") {
+    return cell.text.length;
+  }
   let total = 0;
   for (const section of cell.elements) {
-    if (section.type !== "rich_text_section") continue;
+    if (section.type !== "rich_text_section") {
+      continue;
+    }
     for (const el of section.elements) {
-      if (el.type === "text") total += el.text.length;
-      else if (el.type === "link") total += (el.text ?? el.url).length;
+      if (el.type === "text") {
+        total += el.text.length;
+      } else if (el.type === "link") {
+        total += (el.text ?? el.url).length;
+      }
     }
   }
   return total;
@@ -349,7 +380,9 @@ function hasInlineFormatting(nodes: PhrasingContent[]): boolean {
     ) {
       return true;
     }
-    if ("children" in node && hasInlineFormatting(node.children)) return true;
+    if ("children" in node && hasInlineFormatting(node.children)) {
+      return true;
+    }
   }
   return false;
 }

package/src/messaging/providers/slack/transport.ts

@@ -56,7 +56,9 @@ export const slackTransport: ChannelTransport = {
 
   async sendReaction(_ctx, payload) {
     const reaction = payload.reaction;
-    if (!reaction) return { ok: true };
+    if (!reaction) {
+      return { ok: true };
+    }
     await sendSlackReaction(
       payload.chatId,
       reaction.name,
@@ -68,7 +70,9 @@ export const slackTransport: ChannelTransport = {
 
   async setThreadStatus(_ctx, payload) {
     const status = payload.assistantThreadStatus;
-    if (!status) return { ok: true };
+    if (!status) {
+      return { ok: true };
+    }
     await sendSlackAssistantThreadStatus(
       status.channel,
       status.threadTs,
@@ -80,7 +84,9 @@ export const slackTransport: ChannelTransport = {
 
   async streamReply(_ctx, payload) {
     const op = payload.slackStream;
-    if (!op) return { ok: true };
+    if (!op) {
+      return { ok: true };
+    }
     return sendSlackStreamOp(payload.chatId, op);
   },
 };

package/src/messaging/providers/telegram-bot/adapter.ts

@@ -37,9 +37,13 @@ export const telegramBotMessagingProvider: MessagingProvider = {
 
   async isConnected(): Promise<boolean> {
     const conn = getConnectionByProvider("telegram");
-    if (!(conn && conn.status === "active")) return false;
+    if (!(conn && conn.status === "active")) {
+      return false;
+    }
     const botToken = await getBotToken();
-    if (!botToken) return false;
+    if (!botToken) {
+      return false;
+    }
     const webhookSecret = await getSecureKeyAsync(
       credentialKey("telegram", "webhook_secret"),
     );

package/src/messaging/providers/telegram-bot/api.ts

@@ -88,7 +88,9 @@ function computeDelay(
     const targetTime = new Date(retryAfterHeader).getTime();
     if (Number.isFinite(targetTime)) {
       const delayMs = targetTime - Date.now();
-      if (delayMs > 0) return Math.min(delayMs, 2_147_483_647);
+      if (delayMs > 0) {
+        return Math.min(delayMs, 2_147_483_647);
+      }
     }
   }
   const exponential = initialBackoffMs * Math.pow(2, attempt - 1);