@vellumai/assistant 0.11.0 → 0.11.1-staging.2

package/src/plugins/defaults/surface-completion-nudge/hooks/post-model-call.ts

@@ -175,7 +175,9 @@ interface SurfaceState {
 }
 
 function isNonTerminal(state: SurfaceState): boolean {
-  if (state.dismissed) return false;
+  if (state.dismissed) {
+    return false;
+  }
   return state.status === undefined || !TERMINAL_STATUSES.has(state.status);
 }
 
@@ -198,41 +200,59 @@ function hasDanglingProgressSurface(messages: ReadonlyArray<Message>): boolean {
   for (const message of currentCycleMessages(messages)) {
     if (message.role === "assistant") {
       for (const block of message.content) {
-        if (block.type !== "tool_use") continue;
+        if (block.type !== "tool_use") {
+          continue;
+        }
         if (block.name === "ui_show") {
           const info = progressShowInfo(block.input);
-          if (info.isProgress) pendingShows.set(block.id, info.status);
+          if (info.isProgress) {
+            pendingShows.set(block.id, info.status);
+          }
         } else if (block.name === "ui_update") {
           const id = surfaceIdOf(block.input);
           const status = extractStatus(block.input);
           if (id && status !== undefined) {
             const existing = surfaces.get(id);
-            if (existing) existing.status = status;
-            else surfaces.set(id, { status, dismissed: false });
+            if (existing) {
+              existing.status = status;
+            } else {
+              surfaces.set(id, { status, dismissed: false });
+            }
           }
         } else if (block.name === "ui_dismiss") {
           const id = surfaceIdOf(block.input);
           if (id) {
             const existing = surfaces.get(id);
-            if (existing) existing.dismissed = true;
-            else surfaces.set(id, { status: undefined, dismissed: true });
+            if (existing) {
+              existing.dismissed = true;
+            } else {
+              surfaces.set(id, { status: undefined, dismissed: true });
+            }
           }
         }
       }
       continue;
     }
-    if (message.role !== "user") continue;
+    if (message.role !== "user") {
+      continue;
+    }
     for (const block of message.content) {
       // guard:allow-tool-result-only — only the local tool executor's
       // `tool_result` carries a `ui_show` `surfaceId` to correlate. A
       // `web_search_tool_result` comes from a `server_tool_use`, never a
       // `ui_show`, so it can never match a pending show and is correctly skipped.
-      if (block.type !== "tool_result") continue;
-      if (!pendingShows.has(block.tool_use_id)) continue;
+      if (block.type !== "tool_result") {
+        continue;
+      }
+      if (!pendingShows.has(block.tool_use_id)) {
+        continue;
+      }
       const initialStatus = pendingShows.get(block.tool_use_id);
       pendingShows.delete(block.tool_use_id);
       const id = parseSurfaceId(block.content);
-      if (!id) continue;
+      if (!id) {
+        continue;
+      }
       const existing = surfaces.get(id);
       // A later update/dismiss can register the id before its show result is
       // scanned only if history was reordered; guard so we never clobber a
@@ -248,7 +268,9 @@ function hasDanglingProgressSurface(messages: ReadonlyArray<Message>): boolean {
   }
 
   for (const state of surfaces.values()) {
-    if (isNonTerminal(state)) return true;
+    if (isNonTerminal(state)) {
+      return true;
+    }
   }
   return false;
 }
@@ -256,17 +278,27 @@ function hasDanglingProgressSurface(messages: ReadonlyArray<Message>): boolean {
 const postModelCall: HookFunction<PostModelCallContext> = async (ctx) => {
   // A provider rejection carries no turn content to assess (a recovery hook
   // owns the rejection).
-  if (ctx.error) return;
+  if (ctx.error) {
+    return;
+  }
   // A tool-bearing turn continues mid-run — the loop runs the tools and the
   // model gets another chance to close the surface — so leave it alone.
-  if (hasToolUse(ctx.content)) return;
+  if (hasToolUse(ctx.content)) {
+    return;
+  }
   // Only nudge the user-facing reply: background call sites have no live user
   // watching a spinner.
-  if (ctx.callSite !== "mainAgent") return;
+  if (ctx.callSite !== "mainAgent") {
+    return;
+  }
   // One nudge per run; the sibling `stop` hook clears the mark on terminal stop.
-  if (isSurfaceCompletionNudged(ctx.conversationId)) return;
+  if (isSurfaceCompletionNudged(ctx.conversationId)) {
+    return;
+  }
 
-  if (!hasDanglingProgressSurface(ctx.messages)) return;
+  if (!hasDanglingProgressSurface(ctx.messages)) {
+    return;
+  }
 
   markSurfaceCompletionNudged(ctx.conversationId);
   ctx.messages.push({

package/src/plugins/defaults/task-progress-nudge/hooks/post-tool-use.ts

@@ -101,9 +101,13 @@ export function resetTaskProgressNudgeStateForTests(): void {
  * server-side normalization tolerance.
  */
 function isTaskProgressShowInput(input: unknown): boolean {
-  if (input === null || typeof input !== "object") return false;
+  if (input === null || typeof input !== "object") {
+    return false;
+  }
   const record = input as Record<string, unknown>;
-  if (record.template === "task_progress") return true;
+  if (record.template === "task_progress") {
+    return true;
+  }
   const data = record.data;
   return (
     data !== null &&
@@ -132,19 +136,27 @@ function scanTurn(messages: ReadonlyArray<Message>): {
           block.type === "tool_result" ||
           block.type === "web_search_tool_result",
       );
-      if (!carriesToolResult) break; // genuine user prompt — turn boundary
+      if (!carriesToolResult) {
+        break;
+      } // genuine user prompt — turn boundary
+      continue;
+    }
+    if (message.role !== "assistant") {
       continue;
     }
-    if (message.role !== "assistant") continue;
     let hasToolUse = false;
     for (const block of message.content) {
-      if (block.type !== "tool_use") continue;
+      if (block.type !== "tool_use") {
+        continue;
+      }
       hasToolUse = true;
       if (block.name === "ui_show" && isTaskProgressShowInput(block.input)) {
         taskProgressShown = true;
       }
     }
-    if (hasToolUse) rounds++;
+    if (hasToolUse) {
+      rounds++;
+    }
   }
   return { rounds, taskProgressShown };
 }
@@ -155,9 +167,15 @@ const postToolUse: HookFunction<PostToolUseContext> = async (ctx) => {
   // `subagentSpawn`, background work under its own call sites), on a client
   // that can actually render the card, driven by a model family this nudge
   // targets.
-  if (ctx.callSite !== "mainAgent") return;
-  if (!ctx.supportsDynamicUi) return;
-  if (!NUDGE_TARGET_MODEL_PATTERN.test(ctx.model)) return;
+  if (ctx.callSite !== "mainAgent") {
+    return;
+  }
+  if (!ctx.supportsDynamicUi) {
+    return;
+  }
+  if (!NUDGE_TARGET_MODEL_PATTERN.test(ctx.model)) {
+    return;
+  }
 
   const { rounds, taskProgressShown } = scanTurn(ctx.messages);
 
@@ -176,8 +194,12 @@ const postToolUse: HookFunction<PostToolUseContext> = async (ctx) => {
     return;
   }
 
-  if (rounds < TASK_PROGRESS_NUDGE_ROUND_THRESHOLD) return;
-  if (lastNudged !== 0) return; // already nudged this turn
+  if (rounds < TASK_PROGRESS_NUDGE_ROUND_THRESHOLD) {
+    return;
+  }
+  if (lastNudged !== 0) {
+    return;
+  } // already nudged this turn
 
   lastNudgedRoundsByConversation.set(ctx.conversationId, rounds);
   ctx.logger.info(

package/src/plugins/defaults/title-generate/hooks/stop.ts

@@ -48,7 +48,9 @@ function isToolResultMessage(message: Message): boolean {
 function countUserTurns(messages: ReadonlyArray<Message>): number {
   let turns = 0;
   for (const message of messages) {
-    if (message.role === "user" && !isToolResultMessage(message)) turns++;
+    if (message.role === "user" && !isToolResultMessage(message)) {
+      turns++;
+    }
   }
   return turns;
 }
@@ -57,7 +59,9 @@ function shouldRetryFallbackTitle(conversation: {
   title: string | null;
   isAutoTitle: number;
 }): boolean {
-  if (!conversation.isAutoTitle) return false;
+  if (!conversation.isAutoTitle) {
+    return false;
+  }
   return (
     isReplaceableTitle(conversation.title) ||
     conversation.isAutoTitle === AUTO_TITLE_DETERMINISTIC
@@ -68,10 +72,14 @@ const stop: HookFunction<StopContext> = async (ctx) => {
   // Re-title only at a genuine successful turn end (the model returned a reply
   // with no tool calls). Any other terminal — a provider rejection, abort, or
   // an output-limit cutoff — produced no new topic to re-title from.
-  if (ctx.exitReason !== "no_tool_calls") return;
+  if (ctx.exitReason !== "no_tool_calls") {
+    return;
+  }
 
   const userTurns = countUserTurns(ctx.messages);
-  if (userTurns === 0) return;
+  if (userTurns === 0) {
+    return;
+  }
 
   // System conversations (background/scheduled) keep their deterministic
   // bootstrap title — multi-prompt background jobs can reach three user-role
@@ -81,7 +89,9 @@ const stop: HookFunction<StopContext> = async (ctx) => {
   let retryFallbackTitle = false;
   try {
     const conversation = getConversation(ctx.conversationId);
-    if (conversation && conversation.conversationType !== "standard") return;
+    if (conversation && conversation.conversationType !== "standard") {
+      return;
+    }
     retryFallbackTitle = conversation
       ? shouldRetryFallbackTitle(conversation)
       : false;

package/src/plugins/defaults/title-generate/hooks/user-prompt-submit.ts

@@ -22,7 +22,9 @@ const userPromptSubmit: HookFunction<UserPromptSubmitContext> = async (ctx) => {
   // Hidden machine signals (e.g. the channel-setup wizard-close marker) are
   // not user speech — a title minted from one would surface invisible
   // scaffolding text in the sidebar, and the LLM call is wasted.
-  if (ctx.isHiddenPrompt) return;
+  if (ctx.isHiddenPrompt) {
+    return;
+  }
 
   // System conversations (background/scheduled) carry a deterministic title
   // from bootstrap. Their own job prompts arrive as non-interactive turns and
@@ -33,7 +35,9 @@ const userPromptSubmit: HookFunction<UserPromptSubmitContext> = async (ctx) => {
   if (ctx.isNonInteractive) {
     try {
       const conversation = getConversation(ctx.conversationId);
-      if (conversation && conversation.conversationType !== "standard") return;
+      if (conversation && conversation.conversationType !== "standard") {
+        return;
+      }
     } catch {
       // Fall through to queueing.
     }

package/src/plugins/defaults/tool-error/hooks/post-tool-use.ts

@@ -50,9 +50,13 @@ const MAX_CONSECUTIVE_ERROR_NUDGES = 3;
 function toolNamesById(messages: ReadonlyArray<Message>): Map<string, string> {
   const names = new Map<string, string>();
   for (const message of messages) {
-    if (message.role !== "assistant") continue;
+    if (message.role !== "assistant") {
+      continue;
+    }
     for (const block of message.content) {
-      if (block.type === "tool_use") names.set(block.id, block.name);
+      if (block.type === "tool_use") {
+        names.set(block.id, block.name);
+      }
     }
   }
   return names;
@@ -71,24 +75,34 @@ function priorConsecutiveErrors(
 ): number {
   const isErrorByOrder: boolean[] = [];
   for (const message of messages) {
-    if (message.role !== "user") continue;
+    if (message.role !== "user") {
+      continue;
+    }
     for (const block of message.content) {
-      if (block.type !== "tool_result") continue;
-      if (namesById.get(block.tool_use_id) !== toolName) continue;
+      if (block.type !== "tool_result") {
+        continue;
+      }
+      if (namesById.get(block.tool_use_id) !== toolName) {
+        continue;
+      }
       isErrorByOrder.push(block.is_error === true);
     }
   }
 
   let streak = 0;
   for (let i = isErrorByOrder.length - 1; i >= 0; i--) {
-    if (!isErrorByOrder[i]) break;
+    if (!isErrorByOrder[i]) {
+      break;
+    }
     streak++;
   }
   return streak;
 }
 
 const postToolUse: HookFunction<PostToolUseContext> = async (ctx) => {
-  if (ctx.toolResponse.is_error !== true) return;
+  if (ctx.toolResponse.is_error !== true) {
+    return;
+  }
 
   const namesById = toolNamesById(ctx.messages);
   const toolName = namesById.get(ctx.toolResponse.tool_use_id);

package/src/plugins/defaults/tool-result-truncate/terminal.ts

@@ -44,7 +44,9 @@ function safeStringSlice(
     const firstCode = str.charCodeAt(safeStart);
     if (isLowSurrogate(firstCode)) {
       safeStart++;
-      if (safeStart > safeEnd) safeEnd = safeStart;
+      if (safeStart > safeEnd) {
+        safeEnd = safeStart;
+      }
     }
   }
 

package/src/plugins/defaults/turn-context/injectors.ts

@@ -28,7 +28,9 @@ const unifiedTurnContextInjector: Injector = {
   order: DEFAULT_INJECTOR_ORDER.unifiedTurnContext,
   async produce(ctx: TurnContext): Promise<InjectionBlock | null> {
     const { timestamp } = ctx;
-    if (!timestamp) return null;
+    if (!timestamp) {
+      return null;
+    }
     const text = buildUnifiedTurnContextBlock({
       timestamp,
       interfaceName: ctx.interfaceName,

package/src/plugins/defaults/workspace/injectors.ts

@@ -57,7 +57,9 @@ const diskPressureWarningInjector: Injector = {
   order: DEFAULT_INJECTOR_ORDER.diskPressureWarning,
   async produce(ctx: TurnContext): Promise<InjectionBlock | null> {
     const conversation = findConversationOrSubagent(ctx.conversationId);
-    if (!conversation?.diskPressureCleanupModeActive) return null;
+    if (!conversation?.diskPressureCleanupModeActive) {
+      return null;
+    }
     return {
       id: "disk-pressure-warning",
       text: DISK_PRESSURE_WARNING_PROMPT,
@@ -106,11 +108,16 @@ const workspaceContextInjector: Injector = {
     runMessages?: Message[],
   ): Promise<InjectionBlock | null> {
     const mode = ctx.mode ?? "full";
-    if (mode !== "full") return null;
+    if (mode !== "full") {
+      return null;
+    }
     const text = resolveWorkspaceTopLevelContext(ctx.conversationId);
-    if (!text) return null;
-    if (hasInjectedUserTextBlock(runMessages, WORKSPACE_BLOCK_MATCHERS))
+    if (!text) {
       return null;
+    }
+    if (hasInjectedUserTextBlock(runMessages, WORKSPACE_BLOCK_MATCHERS)) {
+      return null;
+    }
     return {
       id: "workspace-context",
       text,
@@ -140,10 +147,14 @@ interface ConfigQuarantineNotice {
  */
 function readConfigQuarantineNotice(): ConfigQuarantineNotice | null {
   const noticePath = getConfigQuarantineNoticePath();
-  if (!existsSync(noticePath)) return null;
+  if (!existsSync(noticePath)) {
+    return null;
+  }
   try {
     const parsed: unknown = JSON.parse(readFileSync(noticePath, "utf-8"));
-    if (parsed == null || typeof parsed !== "object") return null;
+    if (parsed == null || typeof parsed !== "object") {
+      return null;
+    }
     const { quarantinedAt, quarantinePath, originalPath } = parsed as Record<
       string,
       unknown
@@ -188,10 +199,14 @@ const configQuarantineNoticeInjector: Injector = {
   name: "config-quarantine-notice",
   order: DEFAULT_INJECTOR_ORDER.configQuarantineNotice,
   async produce(ctx: TurnContext): Promise<InjectionBlock | null> {
-    if (ctx.trust.trustClass !== "guardian") return null;
+    if (ctx.trust.trustClass !== "guardian") {
+      return null;
+    }
 
     const notice = readConfigQuarantineNotice();
-    if (!notice) return null;
+    if (!notice) {
+      return null;
+    }
 
     const quarantinedAtMs = Date.parse(notice.quarantinedAt);
     const ageMs = Number.isNaN(quarantinedAtMs)
@@ -248,12 +263,18 @@ interface ConfigValidationResetNotice {
  */
 function readConfigValidationResetNotice(): ConfigValidationResetNotice | null {
   const noticePath = getConfigValidationResetNoticePath();
-  if (!existsSync(noticePath)) return null;
+  if (!existsSync(noticePath)) {
+    return null;
+  }
   try {
     const parsed: unknown = JSON.parse(readFileSync(noticePath, "utf-8"));
-    if (parsed == null || typeof parsed !== "object") return null;
+    if (parsed == null || typeof parsed !== "object") {
+      return null;
+    }
     const { resetAt, invalidPaths } = parsed as Record<string, unknown>;
-    if (typeof resetAt !== "string") return null;
+    if (typeof resetAt !== "string") {
+      return null;
+    }
     const paths = Array.isArray(invalidPaths)
       ? invalidPaths.filter((p): p is string => typeof p === "string")
       : [];
@@ -288,10 +309,14 @@ const configValidationResetNoticeInjector: Injector = {
   name: "config-validation-reset-notice",
   order: DEFAULT_INJECTOR_ORDER.configValidationResetNotice,
   async produce(ctx: TurnContext): Promise<InjectionBlock | null> {
-    if (ctx.trust.trustClass !== "guardian") return null;
+    if (ctx.trust.trustClass !== "guardian") {
+      return null;
+    }
 
     const notice = readConfigValidationResetNotice();
-    if (!notice) return null;
+    if (!notice) {
+      return null;
+    }
 
     const resetAtMs = Date.parse(notice.resetAt);
     const ageMs = Number.isNaN(resetAtMs)
@@ -349,8 +374,12 @@ const NOW_MD_BLOCK_PREFIXES = [
  * so the `now-md` injector owns its input rather than having it threaded in.
  */
 function readGatedNowScratchpad(trust: TrustContext): string | null {
-  if (!isPersonalMemoryAllowed(trust)) return null;
-  if (!getConfig().memory.retrieval.scratchpadInjection.enabled) return null;
+  if (!isPersonalMemoryAllowed(trust)) {
+    return null;
+  }
+  if (!getConfig().memory.retrieval.scratchpadInjection.enabled) {
+    return null;
+  }
   return readNowScratchpad();
 }
 
@@ -382,11 +411,16 @@ const nowMdInjector: Injector = {
     runMessages?: Message[],
   ): Promise<InjectionBlock | null> {
     const mode = ctx.mode ?? "full";
-    if (mode !== "full") return null;
+    if (mode !== "full") {
+      return null;
+    }
     const content = readGatedNowScratchpad(ctx.trust);
-    if (!content) return null;
-    if (hasInjectedUserTextBlock(runMessages, NOW_MD_BLOCK_PREFIXES))
+    if (!content) {
+      return null;
+    }
+    if (hasInjectedUserTextBlock(runMessages, NOW_MD_BLOCK_PREFIXES)) {
       return null;
+    }
     const text = `<NOW.md Always keep this up to date; keep under 10 lines>\n${content}\n</NOW.md>`;
     return {
       id: "now-md",

package/src/plugins/external-plugin-loader.ts

@@ -197,7 +197,9 @@ export interface SurfaceFile {
  * subdirectory recursion today, so this stays flat on purpose.
  */
 export function listSurfaceDir(dir: string): SurfaceFile[] {
-  if (!existsSync(dir) || !statSync(dir).isDirectory()) return [];
+  if (!existsSync(dir) || !statSync(dir).isDirectory()) {
+    return [];
+  }
   const entries = readdirSync(dir);
   const byBase = new Map<string, string>();
   for (const entry of entries) {
@@ -205,12 +207,16 @@ export function listSurfaceDir(dir: string): SurfaceFile[] {
     // alongside compiled `.js`. They have no default-exported runtime
     // function and would crash `importDefault`, so the walker filters
     // them out before the `.js`/`.ts` extension check.
-    if (entry.endsWith(".d.ts")) continue;
+    if (entry.endsWith(".d.ts")) {
+      continue;
+    }
     const base =
       entry.endsWith(".js") || entry.endsWith(".ts")
         ? entry.slice(0, -3)
         : null;
-    if (base === null) continue;
+    if (base === null) {
+      continue;
+    }
     const existing = byBase.get(base);
     if (
       existing === undefined ||
@@ -236,7 +242,9 @@ async function loadHooks(
   pluginName: string,
 ): Promise<PluginHooks | undefined> {
   const files = listSurfaceDir(join(pluginDir, "hooks"));
-  if (files.length === 0) return undefined;
+  if (files.length === 0) {
+    return undefined;
+  }
   const hooks: PluginHooks = {};
   for (const { name, path } of files) {
     const fn = await importDefault<HookFunction>(path);
@@ -337,7 +345,9 @@ async function buildPluginFromDir(pluginDir: string): Promise<Plugin> {
   const plugin: Plugin = { manifest };
 
   const hooks = await loadHooks(pluginDir, name);
-  if (hooks !== undefined) plugin.hooks = hooks;
+  if (hooks !== undefined) {
+    plugin.hooks = hooks;
+  }
 
   const tools: Tool[] = [];
   for (const { name: toolName, path: toolPath } of listSurfaceDir(
@@ -351,7 +361,9 @@ async function buildPluginFromDir(pluginDir: string): Promise<Plugin> {
     }
     tools.push(finalizeTool(tool, deriveToolName(toolName)));
   }
-  if (tools.length > 0) plugin.tools = tools;
+  if (tools.length > 0) {
+    plugin.tools = tools;
+  }
 
   return plugin;
 }
@@ -405,7 +417,9 @@ export async function buildExternalPlugin(
     );
     return undefined;
   } finally {
-    if (timeoutHandle !== undefined) clearTimeout(timeoutHandle);
+    if (timeoutHandle !== undefined) {
+      clearTimeout(timeoutHandle);
+    }
   }
 }
 

package/src/plugins/injector-registry.ts

@@ -58,7 +58,9 @@ export function registerPluginInjectors(
     }
     seenInContribution.add(injector.name);
     for (const [owner, existing] of injectorsByPlugin) {
-      if (owner === pluginName) continue;
+      if (owner === pluginName) {
+        continue;
+      }
       if (existing.some((e) => e.name === injector.name)) {
         throw new Error(
           `Injector "${injector.name}" contributed by plugin "${pluginName}" is already registered by plugin "${owner}"`,
@@ -106,7 +108,9 @@ export function getRegisteredInjectors(): Injector[] {
   if (cachedChain === null) {
     const pairs: Array<{ plugin: string; injector: Injector }> = [];
     for (const [plugin, injectors] of injectorsByPlugin) {
-      for (const injector of injectors) pairs.push({ plugin, injector });
+      for (const injector of injectors) {
+        pairs.push({ plugin, injector });
+      }
     }
     pairs.sort((a, b) => a.injector.order - b.injector.order);
     cachedChain = pairs;

package/src/prompts/__tests__/system-prompt.test.ts

@@ -119,8 +119,11 @@ describe("buildSystemPrompt — default persona trust-class guardrail", () => {
   });
 
   afterEach(() => {
-    if (priorAuthEnv === undefined) delete process.env.DISABLE_HTTP_AUTH;
-    else process.env.DISABLE_HTTP_AUTH = priorAuthEnv;
+    if (priorAuthEnv === undefined) {
+      delete process.env.DISABLE_HTTP_AUTH;
+    } else {
+      process.env.DISABLE_HTTP_AUTH = priorAuthEnv;
+    }
   });
 
   test("stranger (unknown) sees the guardrail and the stranger greeting", () => {

package/src/prompts/bootstrap-cleanup.ts

@@ -12,7 +12,9 @@ export function cleanupBootstrapFiles(reason: string): boolean {
 
   for (const file of BOOTSTRAP_FILES) {
     const path = getWorkspacePromptPath(file);
-    if (!existsSync(path)) continue;
+    if (!existsSync(path)) {
+      continue;
+    }
 
     try {
       unlinkSync(path);

package/src/prompts/normalize-onboarding.ts

@@ -50,7 +50,9 @@ export const TASK_DISPLAY_LABELS: Record<string, string> = {
  * Capitalize the first letter of a string (fallback for unknown IDs).
  */
 function capitalizeFirst(s: string): string {
-  if (!s) return s;
+  if (!s) {
+    return s;
+  }
   return s.charAt(0).toUpperCase() + s.slice(1);
 }
 
@@ -108,12 +110,16 @@ const SCOPE_SERVICE_MAP: Record<string, string> = {
 };
 
 export function deriveGoogleServices(scopes?: string[]): string[] {
-  if (!scopes?.length) return ["Gmail", "Calendar", "Drive"];
+  if (!scopes?.length) {
+    return ["Gmail", "Calendar", "Drive"];
+  }
   const services = new Set<string>();
   for (const scope of scopes) {
     const suffix = scope.replace("https://www.googleapis.com/auth/", "");
     const service = SCOPE_SERVICE_MAP[suffix];
-    if (service) services.add(service);
+    if (service) {
+      services.add(service);
+    }
   }
   return services.size > 0 ? [...services] : ["Gmail", "Calendar", "Drive"];
 }