fix(msteams): add fetch timeout to Microsoft Graph API calls

hclsys · claude · hclsys · commit 51409b845825 · 2026-03-16T11:53:20.000+08:00
fetchGraphJson() had no timeout, so a hanging Microsoft Graph
endpoint would stall the entire MS Teams extension indefinitely.

Add a 30-second AbortSignal.timeout and wrap the fetch in a
try/catch that preserves the cause chain.

Co-Authored-By: Claude Opus 4.6 &lt;noreply@anthropic.com&gt;
Signed-off-by: HCL &lt;chenglunhu@gmail.com&gt;
diff --git a/extensions/msteams/src/graph.ts b/extensions/msteams/src/graph.ts
@@ -36,12 +36,21 @@ export async function fetchGraphJson<T>(params: {
   path: string;
   headers?: Record<string, string>;
 }): Promise<T> {
-  const res = await fetch(`${GRAPH_ROOT}${params.path}`, {
-    headers: {
-      Authorization: `Bearer ${params.token}`,
-      ...params.headers,
-    },
-  });
+  let res: Response;
+  try {
+    res = await fetch(`${GRAPH_ROOT}${params.path}`, {
+      headers: {
+        Authorization: `Bearer ${params.token}`,
+        ...params.headers,
+      },
+      signal: AbortSignal.timeout(30_000),
+    });
+  } catch (err) {
+    throw new Error(
+      `Graph ${params.path} request failed: ${err instanceof Error ? err.message : String(err)}`,
+      { cause: err },
+    );
+  }
   if (!res.ok) {
     const text = await res.text().catch(() => "");
     throw new Error(`Graph ${params.path} failed (${res.status}): ${text || "unknown error"}`);
