@vellumai/assistant 0.11.0 → 0.11.1-staging.2

package/src/messaging/providers/gmail/adapter.ts

@@ -44,7 +44,9 @@ function extractHeader(msg: GmailMessage, name: string): string {
 }
 
 function extractPlainTextBody(msg: GmailMessage): string {
-  if (!msg.payload) return "";
+  if (!msg.payload) {
+    return "";
+  }
 
   function walkParts(part: GmailMessagePart): string | null {
     if (part.mimeType === "text/plain" && part.body?.data) {
@@ -53,7 +55,9 @@ function extractPlainTextBody(msg: GmailMessage): string {
     if (part.parts) {
       for (const child of part.parts) {
         const result = walkParts(child);
-        if (result) return result;
+        if (result) {
+          return result;
+        }
       }
     }
     return null;
@@ -183,7 +187,9 @@ export const gmailMessagingProvider: MessagingProvider = {
       [conversationId],
     );
 
-    if (!listResult.messages?.length) return [];
+    if (!listResult.messages?.length) {
+      return [];
+    }
 
     const messages = await gmail.batchGetMessages(
       conn,
@@ -256,7 +262,9 @@ export const gmailMessagingProvider: MessagingProvider = {
     const conn = requireConnection(connection);
     const thread = await gmail.getThread(conn, threadId, "full");
     const messages = thread.messages ?? [];
-    if (!messages.length) return [];
+    if (!messages.length) {
+      return [];
+    }
     const limit = options?.limit ?? 50;
     return messages.slice(0, limit).map(mapGmailMessage);
   },
@@ -267,7 +275,9 @@ export const gmailMessagingProvider: MessagingProvider = {
     messageId?: string,
   ): Promise<void> {
     const conn = requireConnection(connection);
-    if (!messageId) return;
+    if (!messageId) {
+      return;
+    }
     await gmail.modifyMessage(conn, messageId, {
       removeLabelIds: ["UNREAD"],
     });
@@ -304,7 +314,9 @@ export const gmailMessagingProvider: MessagingProvider = {
         pageToken,
       );
       const ids = (listResp.messages ?? []).map((m) => m.id);
-      if (ids.length === 0) break;
+      if (ids.length === 0) {
+        break;
+      }
       allMessageIds.push(...ids);
       fetchPromises.push(
         gmail.batchGetMessages(
@@ -316,7 +328,9 @@ export const gmailMessagingProvider: MessagingProvider = {
         ),
       );
       pageToken = listResp.nextPageToken ?? undefined;
-      if (!pageToken) break;
+      if (!pageToken) {
+        break;
+      }
     }
 
     // If we stopped because we hit the cap but there were still more pages, flag truncation
@@ -360,7 +374,9 @@ export const gmailMessagingProvider: MessagingProvider = {
       const displayName = match
         ? match[1].replace(/^["']|["']$/g, "").trim()
         : "";
-      if (!email) continue;
+      if (!email) {
+        continue;
+      }
 
       let agg = senderMap.get(email);
       if (!agg) {
@@ -379,8 +395,12 @@ export const gmailMessagingProvider: MessagingProvider = {
       }
 
       agg.messageCount++;
-      if (listUnsub) agg.hasUnsubscribe = true;
-      if (!agg.displayName && displayName) agg.displayName = displayName;
+      if (listUnsub) {
+        agg.hasUnsubscribe = true;
+      }
+      if (!agg.displayName && displayName) {
+        agg.displayName = displayName;
+      }
 
       if (agg.messageIds.length < maxIdsPerSender) {
         agg.messageIds.push(msg.id);
@@ -442,10 +462,14 @@ export const gmailMessagingProvider: MessagingProvider = {
         pageToken,
       );
       const ids = (listResp.messages ?? []).map((m) => m.id);
-      if (ids.length === 0) break;
+      if (ids.length === 0) {
+        break;
+      }
       allMessageIds.push(...ids);
       pageToken = listResp.nextPageToken ?? undefined;
-      if (!pageToken) break;
+      if (!pageToken) {
+        break;
+      }
     }
 
     if (allMessageIds.length >= maxMessages && pageToken) {

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

@@ -105,20 +105,28 @@ interface GmailRequestOptions extends RequestInit {
 function extractNonAuthHeaders(
   options?: GmailRequestOptions,
 ): Record<string, string> | undefined {
-  if (!options?.headers) return undefined;
+  if (!options?.headers) {
+    return undefined;
+  }
   const raw = options.headers;
   const result: Record<string, string> = {};
   if (raw instanceof Headers) {
     raw.forEach((v, k) => {
-      if (k.toLowerCase() !== "authorization") result[k] = v;
+      if (k.toLowerCase() !== "authorization") {
+        result[k] = v;
+      }
     });
   } else if (Array.isArray(raw)) {
     for (const [k, v] of raw) {
-      if (k.toLowerCase() !== "authorization") result[k] = v;
+      if (k.toLowerCase() !== "authorization") {
+        result[k] = v;
+      }
     }
   } else {
     for (const [k, v] of Object.entries(raw)) {
-      if (k.toLowerCase() !== "authorization" && v !== undefined) result[k] = v;
+      if (k.toLowerCase() !== "authorization" && v !== undefined) {
+        result[k] = v;
+      }
     }
   }
   return Object.keys(result).length > 0 ? result : undefined;
@@ -142,7 +150,9 @@ function paramsToQuery(
  * Extract the JSON body from request options for use with OAuthConnection.
  */
 function extractBody(options?: GmailRequestOptions): unknown | undefined {
-  if (!options?.body) return undefined;
+  if (!options?.body) {
+    return undefined;
+  }
   if (typeof options.body === "string") {
     try {
       return JSON.parse(options.body);
@@ -238,11 +248,17 @@ export async function listMessages(
   signal?: AbortSignal,
 ): Promise<GmailMessageListResponse> {
   const params = new URLSearchParams();
-  if (query) params.set("q", query);
+  if (query) {
+    params.set("q", query);
+  }
   params.set("maxResults", String(maxResults));
-  if (pageToken) params.set("pageToken", pageToken);
+  if (pageToken) {
+    params.set("pageToken", pageToken);
+  }
   if (labelIds) {
-    for (const id of labelIds) params.append("labelIds", id);
+    for (const id of labelIds) {
+      params.append("labelIds", id);
+    }
   }
   return request<GmailMessageListResponse>(
     connection,
@@ -264,9 +280,13 @@ async function getMessage(
 ): Promise<GmailMessage> {
   const params = new URLSearchParams({ format });
   if (format === "metadata" && metadataHeaders) {
-    for (const h of metadataHeaders) params.append("metadataHeaders", h);
+    for (const h of metadataHeaders) {
+      params.append("metadataHeaders", h);
+    }
+  }
+  if (fields) {
+    params.set("fields", fields);
   }
-  if (fields) params.set("fields", fields);
   return request<GmailMessage>(
     connection,
     `/messages/${messageId}`,
@@ -285,7 +305,9 @@ export async function getThread(
 ): Promise<GmailThread> {
   const params = new URLSearchParams({ format });
   if (format === "metadata" && metadataHeaders) {
-    for (const h of metadataHeaders) params.append("metadataHeaders", h);
+    for (const h of metadataHeaders) {
+      params.append("metadataHeaders", h);
+    }
   }
   return request<GmailThread>(
     connection,
@@ -303,12 +325,16 @@ function parseSubResponse(
   part: string,
 ): { index: number; status: number; json: string | null } | null {
   const idMatch = part.match(/Content-ID:\s*<response-(\d+)>/i);
-  if (!idMatch) return null;
+  if (!idMatch) {
+    return null;
+  }
   const index = parseInt(idMatch[1], 10);
 
   // Split MIME headers from the embedded HTTP response (separated by blank line)
   const mimeEnd = part.search(/\r?\n\r?\n/);
-  if (mimeEnd === -1) return null;
+  if (mimeEnd === -1) {
+    return null;
+  }
   const httpResponse = part.slice(mimeEnd).replace(/^(\r?\n){2}/, "");
 
   const statusMatch = httpResponse.match(/^HTTP\/[\d.]+ (\d+)/);
@@ -316,7 +342,9 @@ function parseSubResponse(
 
   // Split HTTP headers from body (separated by blank line)
   const bodyStart = httpResponse.search(/\r?\n\r?\n/);
-  if (bodyStart === -1) return { index, status, json: null };
+  if (bodyStart === -1) {
+    return { index, status, json: null };
+  }
   const json = httpResponse
     .slice(bodyStart)
     .replace(/^(\r?\n){2}/, "")
@@ -347,9 +375,13 @@ async function executeBatchCall(
   // Build query string once (shared by all sub-requests)
   const params = new URLSearchParams({ format });
   if (format === "metadata" && metadataHeaders) {
-    for (const h of metadataHeaders) params.append("metadataHeaders", h);
+    for (const h of metadataHeaders) {
+      params.append("metadataHeaders", h);
+    }
+  }
+  if (fields) {
+    params.set("fields", fields);
   }
-  if (fields) params.set("fields", fields);
   const qs = params.toString();
 
   // Build multipart request body
@@ -399,8 +431,9 @@ async function executeBatchCall(
         /boundary=(?:"([^"]+)"|([^\s;]+))/,
       );
       const respBoundary = boundaryMatch?.[1] ?? boundaryMatch?.[2];
-      if (!respBoundary)
+      if (!respBoundary) {
         throw new Error("Missing boundary in Gmail batch response");
+      }
 
       const respParts = responseText.split(`--${respBoundary}`);
       const messages: Array<{ index: number; msg: GmailMessage }> = [];
@@ -408,7 +441,9 @@ async function executeBatchCall(
 
       for (const rp of respParts) {
         const parsed = parseSubResponse(rp);
-        if (!parsed) continue;
+        if (!parsed) {
+          continue;
+        }
 
         if (parsed.status >= 200 && parsed.status < 300 && parsed.json) {
           try {
@@ -498,7 +533,9 @@ export async function batchGetMessages(
   fields?: string,
   signal?: AbortSignal,
 ): Promise<GmailMessage[]> {
-  if (messageIds.length === 0) return [];
+  if (messageIds.length === 0) {
+    return [];
+  }
 
   // Single message -- just use getMessage directly
   if (messageIds.length === 1) {
@@ -639,8 +676,12 @@ export async function createDraft(
     `Subject: ${subject}`,
     "Content-Type: text/plain; charset=utf-8",
   ];
-  if (cc) headers.push(`Cc: ${cc}`);
-  if (bcc) headers.push(`Bcc: ${bcc}`);
+  if (cc) {
+    headers.push(`Cc: ${cc}`);
+  }
+  if (bcc) {
+    headers.push(`Bcc: ${bcc}`);
+  }
   if (inReplyTo) {
     headers.push(`In-Reply-To: ${inReplyTo}`);
     headers.push(`References: ${inReplyTo}`);
@@ -651,7 +692,9 @@ export async function createDraft(
     .replace(/\//g, "_")
     .replace(/=+$/, "");
   const message: Record<string, unknown> = { raw };
-  if (threadId) message.threadId = threadId;
+  if (threadId) {
+    message.threadId = threadId;
+  }
   return request<GmailDraft>(connection, "/drafts", {
     method: "POST",
     body: JSON.stringify({ message }),
@@ -665,7 +708,9 @@ export async function createDraftRaw(
   threadId?: string,
 ): Promise<GmailDraft> {
   const message: Record<string, unknown> = { raw };
-  if (threadId) message.threadId = threadId;
+  if (threadId) {
+    message.threadId = threadId;
+  }
   return request<GmailDraft>(connection, "/drafts", {
     method: "POST",
     body: JSON.stringify({ message }),
@@ -696,7 +741,9 @@ export async function sendMessage(
     .replace(/\//g, "_")
     .replace(/=+$/, "");
   const payload: Record<string, unknown> = { raw };
-  if (threadId) payload.threadId = threadId;
+  if (threadId) {
+    payload.threadId = threadId;
+  }
   return request<GmailMessage>(connection, "/messages/send", {
     method: "POST",
     body: JSON.stringify(payload),

package/src/messaging/providers/gmail/mime-builder.ts

@@ -55,8 +55,12 @@ export function buildMultipartMime(options: MimeMessageOptions): string {
     "MIME-Version: 1.0",
     `Content-Type: multipart/mixed; boundary="${boundary}"`,
   ];
-  if (sanitizedCc) headers.push(`Cc: ${sanitizedCc}`);
-  if (sanitizedBcc) headers.push(`Bcc: ${sanitizedBcc}`);
+  if (sanitizedCc) {
+    headers.push(`Cc: ${sanitizedCc}`);
+  }
+  if (sanitizedBcc) {
+    headers.push(`Bcc: ${sanitizedBcc}`);
+  }
   if (sanitizedInReplyTo) {
     headers.push(`In-Reply-To: ${sanitizedInReplyTo}`);
     headers.push(`References: ${sanitizedInReplyTo}`);

package/src/messaging/providers/gmail/people-client.ts

@@ -51,7 +51,9 @@ export async function listContacts(
     personFields: PERSON_FIELDS,
     pageSize: String(pageSize),
   };
-  if (pageToken) query.pageToken = pageToken;
+  if (pageToken) {
+    query.pageToken = pageToken;
+  }
   return request<PeopleConnectionsResponse>(
     connection,
     "/people/me/connections",

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

@@ -149,11 +149,21 @@ export async function listMessages(
     : "/v1.0/me/messages";
 
   const query: Record<string, string> = {};
-  if (options?.top !== undefined) query["$top"] = String(options.top);
-  if (options?.skip !== undefined) query["$skip"] = String(options.skip);
-  if (options?.filter) query["$filter"] = options.filter;
-  if (options?.orderby) query["$orderby"] = options.orderby;
-  if (options?.select) query["$select"] = options.select;
+  if (options?.top !== undefined) {
+    query["$top"] = String(options.top);
+  }
+  if (options?.skip !== undefined) {
+    query["$skip"] = String(options.skip);
+  }
+  if (options?.filter) {
+    query["$filter"] = options.filter;
+  }
+  if (options?.orderby) {
+    query["$orderby"] = options.orderby;
+  }
+  if (options?.select) {
+    query["$select"] = options.select;
+  }
 
   return request<OutlookMessageListResponse>(
     connection,
@@ -176,8 +186,12 @@ export async function searchMessages(
     $search: `"${searchQuery.replace(/\\/g, "\\\\").replace(/"/g, '\\"')}"`,
     $count: "true",
   };
-  if (options?.top !== undefined) query["$top"] = String(options.top);
-  if (options?.skip !== undefined) query["$skip"] = String(options.skip);
+  if (options?.top !== undefined) {
+    query["$top"] = String(options.top);
+  }
+  if (options?.skip !== undefined) {
+    query["$skip"] = String(options.skip);
+  }
 
   return request<OutlookMessageListResponse>(
     connection,
@@ -240,7 +254,9 @@ export async function listMailFolders(
       undefined,
       nextQuery,
     );
-    if (resp.value) allFolders.push(...resp.value);
+    if (resp.value) {
+      allFolders.push(...resp.value);
+    }
     if (resp["@odata.nextLink"]) {
       const nextUrl = new URL(resp["@odata.nextLink"]);
       nextQuery = {};
@@ -335,8 +351,12 @@ export async function createForwardDraft(
   comment?: string,
 ): Promise<OutlookMessage> {
   const body: Record<string, unknown> = {};
-  if (toRecipients !== undefined) body.toRecipients = toRecipients;
-  if (comment !== undefined) body.comment = comment;
+  if (toRecipients !== undefined) {
+    body.toRecipients = toRecipients;
+  }
+  if (comment !== undefined) {
+    body.comment = comment;
+  }
 
   return request<OutlookMessage>(
     connection,

package/src/messaging/providers/slack/__tests__/adapter-token-routing.test.ts

@@ -150,7 +150,9 @@ describe("Slack adapter token routing", () => {
     });
     getSecureKeyAsyncMock.mockReset();
     getSecureKeyAsyncMock.mockImplementation(async (key: string) => {
-      if (key === credentialKey("slack_channel", "bot_token")) return BOT_TOKEN;
+      if (key === credentialKey("slack_channel", "bot_token")) {
+        return BOT_TOKEN;
+      }
       return null;
     });
     installFetchStub();
@@ -187,9 +189,12 @@ describe("Slack adapter token routing", () => {
     // adapter can see channels the user is in but the bot isn't. Writes
     // MUST stay on the bot token so posts come from the bot identity.
     getSecureKeyAsyncMock.mockImplementation(async (key: string) => {
-      if (key === credentialKey("slack_channel", "bot_token")) return BOT_TOKEN;
-      if (key === credentialKey("slack_channel", "user_token"))
+      if (key === credentialKey("slack_channel", "bot_token")) {
+        return BOT_TOKEN;
+      }
+      if (key === credentialKey("slack_channel", "user_token")) {
         return USER_TOKEN;
+      }
       return null;
     });
 
@@ -235,9 +240,12 @@ describe("Slack adapter token routing", () => {
     // with the bot token and reset the read cache so subsequent reads in the
     // same session don't re-hit the failure path.
     getSecureKeyAsyncMock.mockImplementation(async (key: string) => {
-      if (key === credentialKey("slack_channel", "bot_token")) return BOT_TOKEN;
-      if (key === credentialKey("slack_channel", "user_token"))
+      if (key === credentialKey("slack_channel", "bot_token")) {
+        return BOT_TOKEN;
+      }
+      if (key === credentialKey("slack_channel", "user_token")) {
         return USER_TOKEN;
+      }
       return null;
     });
 
@@ -307,8 +315,9 @@ describe("Slack adapter token routing", () => {
     // documents current behavior — user-token-only without an OAuth
     // connection is not a supported install configuration.
     getSecureKeyAsyncMock.mockImplementation(async (key: string) => {
-      if (key === credentialKey("slack_channel", "user_token"))
+      if (key === credentialKey("slack_channel", "user_token")) {
         return USER_TOKEN;
+      }
       return null;
     });
 

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

@@ -96,9 +96,15 @@ const botUserIdByBotIdCache = new Map<string, string>();
  * getReadAuth() / getWriteAuth() directly.
  */
 function getSlackAuth(connection?: OAuthConnection): OAuthConnection | string {
-  if (connection) return connection;
-  if (_cachedSlackWriteAuth) return _cachedSlackWriteAuth;
-  if (_cachedSlackReadAuth) return _cachedSlackReadAuth;
+  if (connection) {
+    return connection;
+  }
+  if (_cachedSlackWriteAuth) {
+    return _cachedSlackWriteAuth;
+  }
+  if (_cachedSlackReadAuth) {
+    return _cachedSlackReadAuth;
+  }
   throw new Error(
     "Slack: no connection or cached token available. Was resolveConnection() called?",
   );
@@ -109,8 +115,12 @@ function getSlackAuth(connection?: OAuthConnection): OAuthConnection | string {
  * conversation replies, search, users.info lookups).
  */
 function getReadAuth(connection?: OAuthConnection): OAuthConnection | string {
-  if (connection) return connection;
-  if (_cachedSlackReadAuth) return _cachedSlackReadAuth;
+  if (connection) {
+    return connection;
+  }
+  if (_cachedSlackReadAuth) {
+    return _cachedSlackReadAuth;
+  }
   return getSlackAuth(connection);
 }
 
@@ -124,8 +134,12 @@ function getReadAuth(connection?: OAuthConnection): OAuthConnection | string {
  * future reactions, joins, leaves, updates, or deletes).
  */
 function getWriteAuth(connection?: OAuthConnection): OAuthConnection | string {
-  if (connection) return connection;
-  if (_cachedSlackWriteAuth) return _cachedSlackWriteAuth;
+  if (connection) {
+    return connection;
+  }
+  if (_cachedSlackWriteAuth) {
+    return _cachedSlackWriteAuth;
+  }
   return getSlackAuth(connection);
 }
 
@@ -148,8 +162,12 @@ export async function withSlackBotToken<T>(
   // directly instead of accepting any previously cached workspace token.
   const resolvedAuth = await slackProvider.resolveConnection?.(account);
   const auth = resolvedAuth ?? _cachedSlackWriteAuth;
-  if (!auth) return null;
-  if (typeof auth === "string") return fn(auth);
+  if (!auth) {
+    return null;
+  }
+  if (typeof auth === "string") {
+    return fn(auth);
+  }
   return auth.withToken(fn);
 }
 
@@ -158,7 +176,9 @@ export async function resolveSlackBotUserId(
   botId: string,
 ): Promise<string | null> {
   const trimmedBotId = botId.trim();
-  if (!trimmedBotId) return null;
+  if (!trimmedBotId) {
+    return null;
+  }
 
   const cacheKey = account ? `${account}:${trimmedBotId}` : null;
   if (cacheKey && botUserIdByBotIdCache.has(cacheKey)) {
@@ -190,7 +210,9 @@ async function runReadWithFallback<T>(
   connection: OAuthConnection | undefined,
   call: (auth: OAuthConnection | string) => Promise<T>,
 ): Promise<T> {
-  if (connection) return call(connection);
+  if (connection) {
+    return call(connection);
+  }
   const auth = getReadAuth(undefined);
   const usingUserToken =
     _cachedSlackWriteAuth !== null &&
@@ -222,10 +244,14 @@ async function resolveUserInfo(
   auth: OAuthConnection | string,
   userId: string,
 ): Promise<NormalizedSlackUserInfo> {
-  if (!userId) return { displayName: "unknown" };
+  if (!userId) {
+    return { displayName: "unknown" };
+  }
   const cacheKey = slackUserInfoCacheKey(auth, userId);
   const cached = userInfoCache.get(cacheKey);
-  if (cached) return (await cached).info;
+  if (cached) {
+    return (await cached).info;
+  }
 
   const resolved = resolveUserInfoUncached(auth, userId).then(
     (result) => {
@@ -313,7 +339,9 @@ function normalizeSlackUserInfo(
 }
 
 function trimNonEmpty(value: unknown): string | undefined {
-  if (typeof value !== "string") return undefined;
+  if (typeof value !== "string") {
+    return undefined;
+  }
   const trimmed = value.trim();
   return trimmed.length > 0 ? trimmed : undefined;
 }
@@ -325,7 +353,9 @@ export function __resetSlackUserInfoCacheForTests(): void {
 function slackUserInfoMetadata(
   userInfo: NormalizedSlackUserInfo | undefined,
 ): Record<string, unknown> {
-  if (!userInfo) return {};
+  if (!userInfo) {
+    return {};
+  }
   return {
     ...(userInfo.timezone ? { actorTimezone: userInfo.timezone } : {}),
     ...(userInfo.timezoneLabel
@@ -369,7 +399,9 @@ function mapSlackFiles(files: SlackMessage["files"]):
       urlPrivate?: string;
     }>
   | undefined {
-  if (!files || files.length === 0) return undefined;
+  if (!files || files.length === 0) {
+    return undefined;
+  }
   const mapped = files
     .map((file) => ({
       ...(file.id ? { id: file.id } : {}),
@@ -504,7 +536,9 @@ export const slackProvider: MessagingProvider = {
     const botToken = await getSecureKeyAsync(
       credentialKey("slack_channel", "bot_token"),
     );
-    if (botToken) return true;
+    if (botToken) {
+      return true;
+    }
     // Preserve existing OAuth path for backwards compat.
     return isProviderConnected("slack");
   },