msteams: fix User-Agent gaps from review feedback

SidU · claude · SidU · commit 7ba1f97938ab · 2026-03-21T04:38:01.000-07:00
- Add User-Agent header to SharePoint fetchImpl callback (was the only
  remaining outbound Graph call without it)
- Guard CloudAdapter User-Agent injection so caller-supplied headers
  are not clobbered by propagation.override()

Co-Authored-By: Claude Opus 4.6 (1M context) &lt;noreply@anthropic.com&gt;
diff --git a/extensions/msteams/src/attachments/graph.ts b/extensions/msteams/src/attachments/graph.ts
@@ -282,6 +282,7 @@ export async function downloadMSTeamsGraphMedia(params: {
             fetchImpl: async (input, init) => {
               const requestUrl = resolveRequestUrl(input);
               const headers = new Headers(init?.headers);
+              headers.set("User-Agent", buildUserAgent());
               headers.set("Authorization", `Bearer ${accessToken}`);
               return await safeFetch({
                 url: requestUrl,
diff --git a/extensions/msteams/src/sdk.ts b/extensions/msteams/src/sdk.ts
@@ -37,7 +37,9 @@ export function createMSTeamsAdapter(
     ) {
       const [serviceUrl, scope, identity, headers] = args;
       const propagation = headers ?? new HeaderPropagation({});
-      propagation.override({ "User-Agent": buildUserAgent() });
+      if (!propagation.get("User-Agent")) {
+        propagation.override({ "User-Agent": buildUserAgent() });
+      }
       return super.createConnectorClient(serviceUrl, scope, identity, propagation);
     }
 
@@ -46,7 +48,9 @@ export function createMSTeamsAdapter(
     ) {
       const [identity, activity, headers] = args;
       const propagation = headers ?? new HeaderPropagation({});
-      propagation.override({ "User-Agent": buildUserAgent() });
+      if (!propagation.get("User-Agent")) {
+        propagation.override({ "User-Agent": buildUserAgent() });
+      }
       return super.createConnectorClientWithIdentity(identity, activity, propagation);
     }
   }
