fix(msteams): block provider promise until abort signal fires

Anandesh-Sharma · claude · Anandesh-Sharma · commit 83caf25d28ae · 2026-02-27T03:57:24.000+05:30
The msteams provider returned immediately after starting the Express server. The channel manager in server-channels.ts treats promise resolution as "provider stopped" and triggers auto-restart with exponential backoff, causing an infinite restart loop. Add the same blocking pattern used by the Slack provider: await a promise that only resolves when the abort signal fires. Also add { once: true } to the existing abort listener to prevent leaks. Closes #27885 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
diff --git a/extensions/msteams/src/monitor.ts b/extensions/msteams/src/monitor.ts
@@ -296,10 +296,24 @@ export async function monitorMSTeamsProvider(
 
   // Handle abort signal
   if (opts.abortSignal) {
-    opts.abortSignal.addEventListener("abort", () => {
-      void shutdown();
-    });
+    opts.abortSignal.addEventListener(
+      "abort",
+      () => {
+        void shutdown();
+      },
+      { once: true },
+    );
   }
 
+  // Block until abort signal fires — the channel manager treats promise
+  // resolution as "provider stopped" and triggers auto-restart.
+  await new Promise<void>((resolve) => {
+    if (opts.abortSignal?.aborted) {
+      resolve();
+      return;
+    }
+    opts.abortSignal?.addEventListener("abort", () => resolve(), { once: true });
+  });
+
   return { app: expressApp, shutdown };
 }
