@openclaw/msteams 2026.1.29 → 2026.2.1

package/src/attachments/shared.ts

@@ -48,6 +48,15 @@ export const DEFAULT_MEDIA_HOST_ALLOWLIST = [
   "microsoft.com",
 ] as const;
 
+export const DEFAULT_MEDIA_AUTH_HOST_ALLOWLIST = [
+  "api.botframework.com",
+  "botframework.com",
+  "graph.microsoft.com",
+  "graph.microsoft.us",
+  "graph.microsoft.de",
+  "graph.microsoft.cn",
+] as const;
+
 export const GRAPH_ROOT = "https://graph.microsoft.com/v1.0";
 
 export function isRecord(value: unknown): value is Record<string, unknown> {
@@ -55,7 +64,9 @@ export function isRecord(value: unknown): value is Record<string, unknown> {
 }
 
 export function normalizeContentType(value: unknown): string | undefined {
-  if (typeof value !== "string") return undefined;
+  if (typeof value !== "string") {
+    return undefined;
+  }
   const trimmed = value.trim();
   return trimmed ? trimmed : undefined;
 }
@@ -78,17 +89,25 @@ export function inferPlaceholder(params: {
 export function isLikelyImageAttachment(att: MSTeamsAttachmentLike): boolean {
   const contentType = normalizeContentType(att.contentType) ?? "";
   const name = typeof att.name === "string" ? att.name : "";
-  if (contentType.startsWith("image/")) return true;
-  if (IMAGE_EXT_RE.test(name)) return true;
+  if (contentType.startsWith("image/")) {
+    return true;
+  }
+  if (IMAGE_EXT_RE.test(name)) {
+    return true;
+  }
 
   if (
     contentType === "application/vnd.microsoft.teams.file.download.info" &&
     isRecord(att.content)
   ) {
     const fileType = typeof att.content.fileType === "string" ? att.content.fileType : "";
-    if (fileType && IMAGE_EXT_RE.test(`x.${fileType}`)) return true;
+    if (fileType && IMAGE_EXT_RE.test(`x.${fileType}`)) {
+      return true;
+    }
     const fileName = typeof att.content.fileName === "string" ? att.content.fileName : "";
-    if (fileName && IMAGE_EXT_RE.test(fileName)) return true;
+    if (fileName && IMAGE_EXT_RE.test(fileName)) {
+      return true;
+    }
   }
 
   return false;
@@ -124,9 +143,15 @@ function isHtmlAttachment(att: MSTeamsAttachmentLike): boolean {
 }
 
 export function extractHtmlFromAttachment(att: MSTeamsAttachmentLike): string | undefined {
-  if (!isHtmlAttachment(att)) return undefined;
-  if (typeof att.content === "string") return att.content;
-  if (!isRecord(att.content)) return undefined;
+  if (!isHtmlAttachment(att)) {
+    return undefined;
+  }
+  if (typeof att.content === "string") {
+    return att.content;
+  }
+  if (!isRecord(att.content)) {
+    return undefined;
+  }
   const text =
     typeof att.content.text === "string"
       ? att.content.text
@@ -140,12 +165,18 @@ export function extractHtmlFromAttachment(att: MSTeamsAttachmentLike): string |
 
 function decodeDataImage(src: string): InlineImageCandidate | null {
   const match = /^data:(image\/[a-z0-9.+-]+)?(;base64)?,(.*)$/i.exec(src);
-  if (!match) return null;
+  if (!match) {
+    return null;
+  }
   const contentType = match[1]?.toLowerCase();
   const isBase64 = Boolean(match[2]);
-  if (!isBase64) return null;
+  if (!isBase64) {
+    return null;
+  }
   const payload = match[3] ?? "";
-  if (!payload) return null;
+  if (!payload) {
+    return null;
+  }
   try {
     const data = Buffer.from(payload, "base64");
     return { kind: "data", data, contentType, placeholder: "<media:image>" };
@@ -170,7 +201,9 @@ export function extractInlineImageCandidates(
   const out: InlineImageCandidate[] = [];
   for (const att of attachments) {
     const html = extractHtmlFromAttachment(att);
-    if (!html) continue;
+    if (!html) {
+      continue;
+    }
     IMG_SRC_RE.lastIndex = 0;
     let match: RegExpExecArray | null = IMG_SRC_RE.exec(html);
     while (match) {
@@ -178,7 +211,9 @@ export function extractInlineImageCandidates(
       if (src && !src.startsWith("cid:")) {
         if (src.startsWith("data:")) {
           const decoded = decodeDataImage(src);
-          if (decoded) out.push(decoded);
+          if (decoded) {
+            out.push(decoded);
+          }
         } else {
           out.push({
             kind: "url",
@@ -204,8 +239,12 @@ export function safeHostForUrl(url: string): string {
 
 function normalizeAllowHost(value: string): string {
   const trimmed = value.trim().toLowerCase();
-  if (!trimmed) return "";
-  if (trimmed === "*") return "*";
+  if (!trimmed) {
+    return "";
+  }
+  if (trimmed === "*") {
+    return "*";
+  }
   return trimmed.replace(/^\*\.?/, "");
 }
 
@@ -214,12 +253,27 @@ export function resolveAllowedHosts(input?: string[]): string[] {
     return DEFAULT_MEDIA_HOST_ALLOWLIST.slice();
   }
   const normalized = input.map(normalizeAllowHost).filter(Boolean);
-  if (normalized.includes("*")) return ["*"];
+  if (normalized.includes("*")) {
+    return ["*"];
+  }
+  return normalized;
+}
+
+export function resolveAuthAllowedHosts(input?: string[]): string[] {
+  if (!Array.isArray(input) || input.length === 0) {
+    return DEFAULT_MEDIA_AUTH_HOST_ALLOWLIST.slice();
+  }
+  const normalized = input.map(normalizeAllowHost).filter(Boolean);
+  if (normalized.includes("*")) {
+    return ["*"];
+  }
   return normalized;
 }
 
 function isHostAllowed(host: string, allowlist: string[]): boolean {
-  if (allowlist.includes("*")) return true;
+  if (allowlist.includes("*")) {
+    return true;
+  }
   const normalized = host.toLowerCase();
   return allowlist.some((entry) => normalized === entry || normalized.endsWith(`.${entry}`));
 }
@@ -227,7 +281,9 @@ function isHostAllowed(host: string, allowlist: string[]): boolean {
 export function isUrlAllowed(url: string, allowlist: string[]): boolean {
   try {
     const parsed = new URL(url);
-    if (parsed.protocol !== "https:") return false;
+    if (parsed.protocol !== "https:") {
+      return false;
+    }
     return isHostAllowed(parsed.hostname, allowlist);
   } catch {
     return false;

package/src/attachments.test.ts

@@ -1,6 +1,5 @@
-import { beforeEach, describe, expect, it, vi } from "vitest";
-
 import type { PluginRuntime } from "openclaw/plugin-sdk";
+import { beforeEach, describe, expect, it, vi } from "vitest";
 import { setMSTeamsRuntime } from "./runtime.js";
 
 const detectMimeMock = vi.fn(async () => "image/png");
@@ -242,6 +241,7 @@ describe("msteams attachments", () => {
         maxBytes: 1024 * 1024,
         tokenProvider: { getAccessToken: vi.fn(async () => "token") },
         allowHosts: ["x"],
+        authAllowHosts: ["x"],
         fetchFn: fetchMock as unknown as typeof fetch,
       });
 
@@ -250,6 +250,41 @@ describe("msteams attachments", () => {
       expect(fetchMock).toHaveBeenCalledTimes(2);
     });
 
+    it("skips auth retries when the host is not in auth allowlist", async () => {
+      const { downloadMSTeamsAttachments } = await load();
+      const tokenProvider = { getAccessToken: vi.fn(async () => "token") };
+      const fetchMock = vi.fn(async (_url: string, opts?: RequestInit) => {
+        const hasAuth = Boolean(
+          opts &&
+          typeof opts === "object" &&
+          "headers" in opts &&
+          (opts.headers as Record<string, string>)?.Authorization,
+        );
+        if (!hasAuth) {
+          return new Response("forbidden", { status: 403 });
+        }
+        return new Response(Buffer.from("png"), {
+          status: 200,
+          headers: { "content-type": "image/png" },
+        });
+      });
+
+      const media = await downloadMSTeamsAttachments({
+        attachments: [
+          { contentType: "image/png", contentUrl: "https://attacker.azureedge.net/img" },
+        ],
+        maxBytes: 1024 * 1024,
+        tokenProvider,
+        allowHosts: ["azureedge.net"],
+        authAllowHosts: ["graph.microsoft.com"],
+        fetchFn: fetchMock as unknown as typeof fetch,
+      });
+
+      expect(media).toHaveLength(0);
+      expect(fetchMock).toHaveBeenCalledTimes(1);
+      expect(tokenProvider.getAccessToken).not.toHaveBeenCalled();
+    });
+
     it("skips urls outside the allowlist", async () => {
       const { downloadMSTeamsAttachments } = await load();
       const fetchMock = vi.fn();

package/src/channel.directory.test.ts

@@ -1,7 +1,5 @@
-import { describe, expect, it } from "vitest";
-
 import type { OpenClawConfig } from "openclaw/plugin-sdk";
-
+import { describe, expect, it } from "vitest";
 import { msteamsPlugin } from "./channel.js";
 
 describe("msteams directory", () => {
@@ -27,7 +25,9 @@ describe("msteams directory", () => {
     expect(msteamsPlugin.directory?.listPeers).toBeTruthy();
     expect(msteamsPlugin.directory?.listGroups).toBeTruthy();
 
-    await expect(msteamsPlugin.directory!.listPeers({ cfg, query: undefined, limit: undefined })).resolves.toEqual(
+    await expect(
+      msteamsPlugin.directory!.listPeers({ cfg, query: undefined, limit: undefined }),
+    ).resolves.toEqual(
       expect.arrayContaining([
         { kind: "user", id: "user:alice" },
         { kind: "user", id: "user:Bob" },
@@ -36,7 +36,9 @@ describe("msteams directory", () => {
       ]),
     );
 
-    await expect(msteamsPlugin.directory!.listGroups({ cfg, query: undefined, limit: undefined })).resolves.toEqual(
+    await expect(
+      msteamsPlugin.directory!.listGroups({ cfg, query: undefined, limit: undefined }),
+    ).resolves.toEqual(
       expect.arrayContaining([
         { kind: "group", id: "conversation:chan1" },
         { kind: "group", id: "conversation:chan2" },

package/src/channel.ts

@@ -5,11 +5,11 @@ import {
   MSTeamsConfigSchema,
   PAIRING_APPROVED_MESSAGE,
 } from "openclaw/plugin-sdk";
-
+import { listMSTeamsDirectoryGroupsLive, listMSTeamsDirectoryPeersLive } from "./directory-live.js";
 import { msteamsOnboardingAdapter } from "./onboarding.js";
 import { msteamsOutbound } from "./outbound.js";
-import { probeMSTeams } from "./probe.js";
 import { resolveMSTeamsGroupToolPolicy } from "./policy.js";
+import { probeMSTeams } from "./probe.js";
 import {
   normalizeMSTeamsMessagingTarget,
   normalizeMSTeamsUserInput,
@@ -20,10 +20,6 @@ import {
 } from "./resolve-allowlist.js";
 import { sendAdaptiveCardMSTeams, sendMessageMSTeams } from "./send.js";
 import { resolveMSTeamsCredentials } from "./token.js";
-import {
-  listMSTeamsDirectoryGroupsLive,
-  listMSTeamsDirectoryPeersLive,
-} from "./directory-live.js";
 
 type ResolvedMSTeamsAccount = {
   accountId: string;
@@ -129,7 +125,9 @@ export const msteamsPlugin: ChannelPlugin<ResolvedMSTeamsAccount> = {
     collectWarnings: ({ cfg }) => {
       const defaultGroupPolicy = cfg.channels?.defaults?.groupPolicy;
       const groupPolicy = cfg.channels?.msteams?.groupPolicy ?? defaultGroupPolicy ?? "allowlist";
-      if (groupPolicy !== "open") return [];
+      if (groupPolicy !== "open") {
+        return [];
+      }
       return [
         `- MS Teams groups: groupPolicy="open" allows any member to trigger (mention-gated). Set channels.msteams.groupPolicy="allowlist" + channels.msteams.groupAllowFrom to restrict senders.`,
       ];
@@ -153,8 +151,12 @@ export const msteamsPlugin: ChannelPlugin<ResolvedMSTeamsAccount> = {
     targetResolver: {
       looksLikeId: (raw) => {
         const trimmed = raw.trim();
-        if (!trimmed) return false;
-        if (/^conversation:/i.test(trimmed)) return true;
+        if (!trimmed) {
+          return false;
+        }
+        if (/^conversation:/i.test(trimmed)) {
+          return true;
+        }
         if (/^user:/i.test(trimmed)) {
           // Only treat as ID if the value after user: looks like a UUID
           const id = trimmed.slice("user:".length).trim();
@@ -172,11 +174,15 @@ export const msteamsPlugin: ChannelPlugin<ResolvedMSTeamsAccount> = {
       const ids = new Set<string>();
       for (const entry of cfg.channels?.msteams?.allowFrom ?? []) {
         const trimmed = String(entry).trim();
-        if (trimmed && trimmed !== "*") ids.add(trimmed);
+        if (trimmed && trimmed !== "*") {
+          ids.add(trimmed);
+        }
       }
       for (const userId of Object.keys(cfg.channels?.msteams?.dms ?? {})) {
         const trimmed = userId.trim();
-        if (trimmed) ids.add(trimmed);
+        if (trimmed) {
+          ids.add(trimmed);
+        }
       }
       return Array.from(ids)
         .map((raw) => raw.trim())
@@ -184,8 +190,12 @@ export const msteamsPlugin: ChannelPlugin<ResolvedMSTeamsAccount> = {
         .map((raw) => normalizeMSTeamsMessagingTarget(raw) ?? raw)
         .map((raw) => {
           const lowered = raw.toLowerCase();
-          if (lowered.startsWith("user:")) return raw;
-          if (lowered.startsWith("conversation:")) return raw;
+          if (lowered.startsWith("user:")) {
+            return raw;
+          }
+          if (lowered.startsWith("conversation:")) {
+            return raw;
+          }
           return `user:${raw}`;
         })
         .filter((id) => (q ? id.toLowerCase().includes(q) : true))
@@ -198,7 +208,9 @@ export const msteamsPlugin: ChannelPlugin<ResolvedMSTeamsAccount> = {
       for (const team of Object.values(cfg.channels?.msteams?.teams ?? {})) {
         for (const channelId of Object.keys(team.channels ?? {})) {
           const trimmed = channelId.trim();
-          if (trimmed && trimmed !== "*") ids.add(trimmed);
+          if (trimmed && trimmed !== "*") {
+            ids.add(trimmed);
+          }
         }
       }
       return Array.from(ids)
@@ -225,8 +237,7 @@ export const msteamsPlugin: ChannelPlugin<ResolvedMSTeamsAccount> = {
         note: undefined as string | undefined,
       }));
 
-      const stripPrefix = (value: string) =>
-        normalizeMSTeamsUserInput(value);
+      const stripPrefix = (value: string) => normalizeMSTeamsUserInput(value);
 
       if (kind === "user") {
         const pending: Array<{ input: string; query: string; index: number }> = [];
@@ -253,7 +264,9 @@ export const msteamsPlugin: ChannelPlugin<ResolvedMSTeamsAccount> = {
             });
             resolved.forEach((entry, idx) => {
               const target = results[pending[idx]?.index ?? -1];
-              if (!target) return;
+              if (!target) {
+                return;
+              }
               target.resolved = entry.resolved;
               target.id = entry.id;
               target.name = entry.name;
@@ -263,7 +276,9 @@ export const msteamsPlugin: ChannelPlugin<ResolvedMSTeamsAccount> = {
             runtime.error?.(`msteams resolve failed: ${String(err)}`);
             pending.forEach(({ index }) => {
               const entry = results[index];
-              if (entry) entry.note = "lookup failed";
+              if (entry) {
+                entry.note = "lookup failed";
+              }
             });
           }
         }
@@ -302,7 +317,9 @@ export const msteamsPlugin: ChannelPlugin<ResolvedMSTeamsAccount> = {
           });
           resolved.forEach((entry, idx) => {
             const target = results[pending[idx]?.index ?? -1];
-            if (!target) return;
+            if (!target) {
+              return;
+            }
             if (!entry.resolved || !entry.teamId) {
               target.resolved = false;
               target.note = entry.note;
@@ -314,19 +331,23 @@ export const msteamsPlugin: ChannelPlugin<ResolvedMSTeamsAccount> = {
               target.name =
                 entry.channelName && entry.teamName
                   ? `${entry.teamName}/${entry.channelName}`
-                  : entry.channelName ?? entry.teamName;
+                  : (entry.channelName ?? entry.teamName);
             } else {
               target.id = entry.teamId;
               target.name = entry.teamName;
               target.note = "team id";
             }
-            if (entry.note) target.note = entry.note;
+            if (entry.note) {
+              target.note = entry.note;
+            }
           });
         } catch (err) {
           runtime.error?.(`msteams resolve failed: ${String(err)}`);
           pending.forEach(({ index }) => {
             const entry = results[index];
-            if (entry) entry.note = "lookup failed";
+            if (entry) {
+              entry.note = "lookup failed";
+            }
           });
         }
       }
@@ -339,7 +360,9 @@ export const msteamsPlugin: ChannelPlugin<ResolvedMSTeamsAccount> = {
       const enabled =
         cfg.channels?.msteams?.enabled !== false &&
         Boolean(resolveMSTeamsCredentials(cfg.channels?.msteams));
-      if (!enabled) return [];
+      if (!enabled) {
+        return [];
+      }
       return ["poll"] satisfies ChannelMessageActionName[];
     },
     supportsCards: ({ cfg }) => {

package/src/conversation-store-fs.test.ts

@@ -1,10 +1,8 @@
+import type { PluginRuntime } from "openclaw/plugin-sdk";
 import fs from "node:fs";
 import os from "node:os";
 import path from "node:path";
-
 import { beforeEach, describe, expect, it } from "vitest";
-
-import type { PluginRuntime } from "openclaw/plugin-sdk";
 import type { StoredConversationReference } from "./conversation-store.js";
 import { createMSTeamsConversationStoreFs } from "./conversation-store-fs.js";
 import { setMSTeamsRuntime } from "./runtime.js";
@@ -12,9 +10,10 @@ import { setMSTeamsRuntime } from "./runtime.js";
 const runtimeStub = {
   state: {
     resolveStateDir: (env: NodeJS.ProcessEnv = process.env, homedir?: () => string) => {
-      const override =
-        env.OPENCLAW_STATE_DIR?.trim() || env.OPENCLAW_STATE_DIR?.trim();
-      if (override) return override;
+      const override = env.OPENCLAW_STATE_DIR?.trim() || env.OPENCLAW_STATE_DIR?.trim();
+      if (override) {
+        return override;
+      }
       const resolvedHome = homedir ? homedir() : os.homedir();
       return path.join(resolvedHome, ".openclaw");
     },
@@ -67,7 +66,7 @@ describe("msteams conversation store (fs)", () => {
     await fs.promises.writeFile(filePath, `${JSON.stringify(json, null, 2)}\n`);
 
     const list = await store.list();
-    const ids = list.map((e) => e.conversationId).sort();
+    const ids = list.map((e) => e.conversationId).toSorted();
     expect(ids).toEqual(["19:active@thread.tacv2", "19:legacy@thread.tacv2"]);
 
     expect(await store.get("19:old@thread.tacv2")).toBeNull();
@@ -80,7 +79,7 @@ describe("msteams conversation store (fs)", () => {
 
     const rawAfter = await fs.promises.readFile(filePath, "utf-8");
     const jsonAfter = JSON.parse(rawAfter) as typeof json;
-    expect(Object.keys(jsonAfter.conversations).sort()).toEqual([
+    expect(Object.keys(jsonAfter.conversations).toSorted()).toEqual([
       "19:active@thread.tacv2",
       "19:legacy@thread.tacv2",
       "19:new@thread.tacv2",

package/src/conversation-store-fs.ts

@@ -16,9 +16,13 @@ const MAX_CONVERSATIONS = 1000;
 const CONVERSATION_TTL_MS = 365 * 24 * 60 * 60 * 1000;
 
 function parseTimestamp(value: string | undefined): number | null {
-  if (!value) return null;
+  if (!value) {
+    return null;
+  }
   const parsed = Date.parse(value);
-  if (!Number.isFinite(parsed)) return null;
+  if (!Number.isFinite(parsed)) {
+    return null;
+  }
   return parsed;
 }
 
@@ -26,7 +30,9 @@ function pruneToLimit(
   conversations: Record<string, StoredConversationReference & { lastSeenAt?: string }>,
 ) {
   const entries = Object.entries(conversations);
-  if (entries.length <= MAX_CONVERSATIONS) return conversations;
+  if (entries.length <= MAX_CONVERSATIONS) {
+    return conversations;
+  }
 
   entries.sort((a, b) => {
     const aTs = parseTimestamp(a[1].lastSeenAt) ?? 0;
@@ -109,7 +115,9 @@ export function createMSTeamsConversationStoreFs(params?: {
 
   const findByUserId = async (id: string): Promise<MSTeamsConversationStoreEntry | null> => {
     const target = id.trim();
-    if (!target) return null;
+    if (!target) {
+      return null;
+    }
     for (const entry of await list()) {
       const { conversationId, reference } = entry;
       if (reference.user?.aadObjectId === target) {
@@ -144,7 +152,9 @@ export function createMSTeamsConversationStoreFs(params?: {
     const normalizedId = normalizeConversationId(conversationId);
     return await withFileLock(filePath, empty, async () => {
       const store = await readStore();
-      if (!(normalizedId in store.conversations)) return false;
+      if (!(normalizedId in store.conversations)) {
+        return false;
+      }
       delete store.conversations[normalizedId];
       await writeJsonFile(filePath, store);
       return true;

package/src/conversation-store-memory.ts

@@ -30,7 +30,9 @@ export function createMSTeamsConversationStoreMemory(
     },
     findByUserId: async (id) => {
       const target = id.trim();
-      if (!target) return null;
+      if (!target) {
+        return null;
+      }
       for (const [conversationId, reference] of map.entries()) {
         if (reference.user?.aadObjectId === target) {
           return { conversationId, reference };

package/src/directory-live.ts

@@ -1,5 +1,4 @@
 import type { ChannelDirectoryEntry } from "openclaw/plugin-sdk";
-
 import { GRAPH_ROOT } from "./attachments/shared.js";
 import { loadMSTeamsSdkWithAuth } from "./sdk.js";
 import { resolveMSTeamsCredentials } from "./token.js";
@@ -24,7 +23,9 @@ type GraphChannel = {
 type GraphResponse<T> = { value?: T[] };
 
 function readAccessToken(value: unknown): string | null {
-  if (typeof value === "string") return value;
+  if (typeof value === "string") {
+    return value;
+  }
   if (value && typeof value === "object") {
     const token =
       (value as { accessToken?: unknown }).accessToken ?? (value as { token?: unknown }).token;
@@ -49,7 +50,7 @@ async function fetchGraphJson<T>(params: {
   const res = await fetch(`${GRAPH_ROOT}${params.path}`, {
     headers: {
       Authorization: `Bearer ${params.token}`,
-      ...(params.headers ?? {}),
+      ...params.headers,
     },
   });
   if (!res.ok) {
@@ -60,13 +61,19 @@ async function fetchGraphJson<T>(params: {
 }
 
 async function resolveGraphToken(cfg: unknown): Promise<string> {
-  const creds = resolveMSTeamsCredentials((cfg as { channels?: { msteams?: unknown } })?.channels?.msteams);
-  if (!creds) throw new Error("MS Teams credentials missing");
+  const creds = resolveMSTeamsCredentials(
+    (cfg as { channels?: { msteams?: unknown } })?.channels?.msteams,
+  );
+  if (!creds) {
+    throw new Error("MS Teams credentials missing");
+  }
   const { sdk, authConfig } = await loadMSTeamsSdkWithAuth(creds);
   const tokenProvider = new sdk.MsalTokenProvider(authConfig);
   const token = await tokenProvider.getAccessToken("https://graph.microsoft.com");
   const accessToken = readAccessToken(token);
-  if (!accessToken) throw new Error("MS Teams graph token unavailable");
+  if (!accessToken) {
+    throw new Error("MS Teams graph token unavailable");
+  }
   return accessToken;
 }
 
@@ -90,7 +97,9 @@ export async function listMSTeamsDirectoryPeersLive(params: {
   limit?: number | null;
 }): Promise<ChannelDirectoryEntry[]> {
   const query = normalizeQuery(params.query);
-  if (!query) return [];
+  if (!query) {
+    return [];
+  }
   const token = await resolveGraphToken(params.cfg);
   const limit = typeof params.limit === "number" && params.limit > 0 ? params.limit : 20;
 
@@ -114,7 +123,9 @@ export async function listMSTeamsDirectoryPeersLive(params: {
   return users
     .map((user) => {
       const id = user.id?.trim();
-      if (!id) return null;
+      if (!id) {
+        return null;
+      }
       const name = user.displayName?.trim();
       const handle = user.userPrincipalName?.trim() || user.mail?.trim();
       return {
@@ -134,11 +145,16 @@ export async function listMSTeamsDirectoryGroupsLive(params: {
   limit?: number | null;
 }): Promise<ChannelDirectoryEntry[]> {
   const rawQuery = normalizeQuery(params.query);
-  if (!rawQuery) return [];
+  if (!rawQuery) {
+    return [];
+  }
   const token = await resolveGraphToken(params.cfg);
   const limit = typeof params.limit === "number" && params.limit > 0 ? params.limit : 20;
   const [teamQuery, channelQuery] = rawQuery.includes("/")
-    ? rawQuery.split("/", 2).map((part) => part.trim()).filter(Boolean)
+    ? rawQuery
+        .split("/", 2)
+        .map((part) => part.trim())
+        .filter(Boolean)
     : [rawQuery, null];
 
   const teams = await listTeamsByName(token, teamQuery);
@@ -146,7 +162,9 @@ export async function listMSTeamsDirectoryGroupsLive(params: {
 
   for (const team of teams) {
     const teamId = team.id?.trim();
-    if (!teamId) continue;
+    if (!teamId) {
+      continue;
+    }
     const teamName = team.displayName?.trim() || teamQuery;
     if (!channelQuery) {
       results.push({
@@ -156,14 +174,20 @@ export async function listMSTeamsDirectoryGroupsLive(params: {
         handle: teamName ? `#${teamName}` : undefined,
         raw: team,
       });
-      if (results.length >= limit) return results;
+      if (results.length >= limit) {
+        return results;
+      }
       continue;
     }
     const channels = await listChannelsForTeam(token, teamId);
     for (const channel of channels) {
       const name = channel.displayName?.trim();
-      if (!name) continue;
-      if (!name.toLowerCase().includes(channelQuery.toLowerCase())) continue;
+      if (!name) {
+        continue;
+      }
+      if (!name.toLowerCase().includes(channelQuery.toLowerCase())) {
+        continue;
+      }
       results.push({
         kind: "group",
         id: `conversation:${channel.id}`,
@@ -171,7 +195,9 @@ export async function listMSTeamsDirectoryGroupsLive(params: {
         handle: `#${name}`,
         raw: channel,
       });
-      if (results.length >= limit) return results;
+      if (results.length >= limit) {
+        return results;
+      }
     }
   }
 

package/src/errors.test.ts

@@ -1,5 +1,4 @@
 import { describe, expect, it } from "vitest";
-
 import {
   classifyMSTeamsSendError,
   formatMSTeamsSendErrorHint,