@kodelyth/msteams 2026.5.42 → 2026.6.2

package/src/graph-thread.ts

@@ -1,146 +0,0 @@
-import { fetchGraphJson, type GraphResponse } from "./graph.js";
-
-export type GraphThreadMessage = {
-  id?: string;
-  from?: {
-    user?: { displayName?: string; id?: string };
-    application?: { displayName?: string; id?: string };
-  };
-  body?: { content?: string; contentType?: string };
-  createdDateTime?: string;
-};
-
-// TTL cache for team ID -> group GUID mapping.
-const teamGroupIdCache = new Map<string, { groupId: string; expiresAt: number }>();
-const CACHE_TTL_MS = 10 * 60 * 1000; // 10 minutes
-
-/**
- * Strip HTML tags from Teams message content, preserving @mention display names.
- * Teams wraps mentions in <at>Name</at> tags.
- */
-export function stripHtmlFromTeamsMessage(html: string): string {
-  // Preserve mention display names by replacing <at>Name</at> with @Name.
-  let text = html.replace(/<at[^>]*>(.*?)<\/at>/gi, "@$1");
-  // Strip remaining HTML tags.
-  text = text.replace(/<[^>]*>/g, " ");
-  // Decode common HTML entities.
-  text = text
-    .replace(/&amp;/g, "&")
-    .replace(/&lt;/g, "<")
-    .replace(/&gt;/g, ">")
-    .replace(/&quot;/g, '"')
-    .replace(/&#39;/g, "'")
-    .replace(/&nbsp;/g, " ");
-  // Normalize whitespace.
-  return text.replace(/\s+/g, " ").trim();
-}
-
-/**
- * Resolve the Azure AD group GUID for a Teams conversation team ID.
- * Results are cached with a TTL to avoid repeated Graph API calls.
- */
-export async function resolveTeamGroupId(
-  token: string,
-  conversationTeamId: string,
-): Promise<string> {
-  const cached = teamGroupIdCache.get(conversationTeamId);
-  if (cached && cached.expiresAt > Date.now()) {
-    return cached.groupId;
-  }
-
-  // The team ID in channelData is typically the group ID itself for standard teams.
-  // Validate by fetching /teams/{id} and returning the confirmed id.
-  // Requires Team.ReadBasic.All permission; fall back to raw ID if missing.
-  try {
-    const path = `/teams/${encodeURIComponent(conversationTeamId)}?$select=id`;
-    const team = await fetchGraphJson<{ id?: string }>({ token, path });
-    const groupId = team.id ?? conversationTeamId;
-
-    // Only cache when the Graph lookup succeeds — caching a fallback raw ID
-    // can cause silent failures for the entire TTL if the ID is not a valid
-    // Graph team GUID (e.g. Bot Framework conversation key).
-    teamGroupIdCache.set(conversationTeamId, {
-      groupId,
-      expiresAt: Date.now() + CACHE_TTL_MS,
-    });
-
-    return groupId;
-  } catch {
-    // Fallback to raw team ID without caching so subsequent calls retry the
-    // Graph lookup instead of using a potentially invalid cached value.
-    return conversationTeamId;
-  }
-}
-
-/**
- * Fetch a single channel message (the parent/root of a thread).
- * Returns undefined on error so callers can degrade gracefully.
- */
-export async function fetchChannelMessage(
-  token: string,
-  groupId: string,
-  channelId: string,
-  messageId: string,
-): Promise<GraphThreadMessage | undefined> {
-  const path = `/teams/${encodeURIComponent(groupId)}/channels/${encodeURIComponent(channelId)}/messages/${encodeURIComponent(messageId)}?$select=id,from,body,createdDateTime`;
-  try {
-    return await fetchGraphJson<GraphThreadMessage>({ token, path });
-  } catch {
-    return undefined;
-  }
-}
-
-/**
- * Fetch thread replies for a channel message, ordered chronologically.
- *
- * **Limitation:** The Graph API replies endpoint (`/messages/{id}/replies`) does not
- * support `$orderby`, so results are always returned in ascending (oldest-first) order.
- * Combined with the `$top` cap of 50, this means only the **oldest 50 replies** are
- * returned for long threads — newer replies are silently omitted. There is currently no
- * Graph API workaround for this; pagination via `@odata.nextLink` can retrieve more
- * replies but still in ascending order only.
- */
-export async function fetchThreadReplies(
-  token: string,
-  groupId: string,
-  channelId: string,
-  messageId: string,
-  limit = 50,
-): Promise<GraphThreadMessage[]> {
-  const top = Math.min(Math.max(limit, 1), 50);
-  // NOTE: Graph replies endpoint returns oldest-first and does not support $orderby.
-  // For threads with >50 replies, only the oldest 50 are returned. The most recent
-  // replies (often the most relevant context) may be truncated.
-  const path = `/teams/${encodeURIComponent(groupId)}/channels/${encodeURIComponent(channelId)}/messages/${encodeURIComponent(messageId)}/replies?$top=${top}&$select=id,from,body,createdDateTime`;
-  const res = await fetchGraphJson<GraphResponse<GraphThreadMessage>>({ token, path });
-  return res.value ?? [];
-}
-
-/**
- * Format thread messages into a context string for the agent.
- * Skips the current message (by id) and blank messages.
- */
-export function formatThreadContext(
-  messages: GraphThreadMessage[],
-  currentMessageId?: string,
-): string {
-  const lines: string[] = [];
-  for (const msg of messages) {
-    if (msg.id && msg.id === currentMessageId) {
-      continue;
-    } // Skip the triggering message.
-    const sender = msg.from?.user?.displayName ?? msg.from?.application?.displayName ?? "unknown";
-    const contentType = msg.body?.contentType ?? "text";
-    const rawContent = msg.body?.content ?? "";
-    const content =
-      contentType === "html" ? stripHtmlFromTeamsMessage(rawContent) : rawContent.trim();
-    if (!content) {
-      continue;
-    }
-    lines.push(`${sender}: ${content}`);
-  }
-  return lines.join("\n");
-}
-
-// Exported for testing only.
-export { teamGroupIdCache as _teamGroupIdCacheForTest };

package/src/graph-upload.test.ts

@@ -1,253 +0,0 @@
-import { withFetchPreconnect } from "klaw/plugin-sdk/test-env";
-import { describe, expect, it, vi } from "vitest";
-import { buildTeamsFileInfoCard } from "./graph-chat.js";
-import { resolveGraphChatId, uploadToOneDrive, uploadToSharePoint } from "./graph-upload.js";
-
-type FetchCall = [string, { method?: string; headers?: Record<string, string> } | undefined];
-
-function requireFetchCall(fetchFn: ReturnType<typeof vi.fn>, index = 0): FetchCall {
-  const call = fetchFn.mock.calls[index] as unknown as FetchCall | undefined;
-  if (!call) {
-    throw new Error(`fetch call ${index} missing`);
-  }
-  return call;
-}
-
-function expectGraphUploadFetch(fetchFn: ReturnType<typeof vi.fn>, expectedUrl: string): void {
-  const [url, init] = requireFetchCall(fetchFn);
-  expect(url).toBe(expectedUrl);
-  expect(init?.method).toBe("PUT");
-  expect(init?.headers?.Authorization).toBe("Bearer graph-token");
-  expect(init?.headers?.["Content-Type"]).toBe("application/octet-stream");
-  expect(init?.headers?.["User-Agent"]).toMatch(/^teams\.ts\[apps\]\/.+ Klaw\/.+$/);
-}
-
-describe("graph upload helpers", () => {
-  const tokenProvider = {
-    getAccessToken: vi.fn(async () => "graph-token"),
-  };
-
-  it("uploads to OneDrive with the personal drive path", async () => {
-    const fetchFn = vi.fn(
-      async () =>
-        new Response(
-          JSON.stringify({ id: "item-1", webUrl: "https://example.com/1", name: "a.txt" }),
-          {
-            status: 200,
-            headers: { "content-type": "application/json" },
-          },
-        ),
-    );
-
-    const result = await uploadToOneDrive({
-      buffer: Buffer.from("hello"),
-      filename: "a.txt",
-      tokenProvider,
-      fetchFn: withFetchPreconnect(fetchFn),
-    });
-
-    expectGraphUploadFetch(
-      fetchFn,
-      "https://graph.microsoft.com/v1.0/me/drive/root:/KlawShared/a.txt:/content",
-    );
-    expect(result).toEqual({
-      id: "item-1",
-      webUrl: "https://example.com/1",
-      name: "a.txt",
-    });
-  });
-
-  it("uploads to SharePoint with the site drive path", async () => {
-    const fetchFn = vi.fn(
-      async () =>
-        new Response(
-          JSON.stringify({ id: "item-2", webUrl: "https://example.com/2", name: "b.txt" }),
-          {
-            status: 200,
-            headers: { "content-type": "application/json" },
-          },
-        ),
-    );
-
-    const result = await uploadToSharePoint({
-      buffer: Buffer.from("world"),
-      filename: "b.txt",
-      siteId: "site-123",
-      tokenProvider,
-      fetchFn: withFetchPreconnect(fetchFn),
-    });
-
-    expectGraphUploadFetch(
-      fetchFn,
-      "https://graph.microsoft.com/v1.0/sites/site-123/drive/root:/KlawShared/b.txt:/content",
-    );
-    expect(result).toEqual({
-      id: "item-2",
-      webUrl: "https://example.com/2",
-      name: "b.txt",
-    });
-  });
-
-  it("rejects upload responses missing required fields", async () => {
-    const fetchFn = vi.fn(
-      async () =>
-        new Response(JSON.stringify({ id: "item-3" }), {
-          status: 200,
-          headers: { "content-type": "application/json" },
-        }),
-    );
-
-    await expect(
-      uploadToSharePoint({
-        buffer: Buffer.from("world"),
-        filename: "bad.txt",
-        siteId: "site-123",
-        tokenProvider,
-        fetchFn: withFetchPreconnect(fetchFn),
-      }),
-    ).rejects.toThrow("SharePoint upload response missing required fields");
-  });
-});
-
-describe("resolveGraphChatId", () => {
-  const tokenProvider = {
-    getAccessToken: vi.fn(async () => "graph-token"),
-  };
-
-  it("returns the ID directly when it already starts with 19:", async () => {
-    const fetchFn = vi.fn();
-    const result = await resolveGraphChatId({
-      botFrameworkConversationId: "19:abc123@thread.tacv2",
-      tokenProvider,
-      fetchFn: withFetchPreconnect(fetchFn),
-    });
-    // Should short-circuit without making any API call
-    expect(fetchFn).not.toHaveBeenCalled();
-    expect(result).toBe("19:abc123@thread.tacv2");
-  });
-
-  it("resolves personal DM chat ID via Graph API using user AAD object ID", async () => {
-    const fetchFn = vi.fn(
-      async () =>
-        new Response(JSON.stringify({ value: [{ id: "19:dm-chat-id@unq.gbl.spaces" }] }), {
-          status: 200,
-          headers: { "content-type": "application/json" },
-        }),
-    );
-
-    const result = await resolveGraphChatId({
-      botFrameworkConversationId: "a:1abc_bot_framework_dm_id",
-      userAadObjectId: "user-aad-object-id-123",
-      tokenProvider,
-      fetchFn: withFetchPreconnect(fetchFn),
-    });
-
-    expect(fetchFn).toHaveBeenCalledTimes(1);
-    const [callUrlRaw, init] = requireFetchCall(fetchFn);
-    expect(init?.headers?.Authorization).toBe("Bearer graph-token");
-    expect(init?.headers?.["User-Agent"]).toMatch(/^teams\.ts\[apps\]\/.+ Klaw\/.+$/);
-    const callUrl = new URL(callUrlRaw);
-    expect(callUrl.origin).toBe("https://graph.microsoft.com");
-    expect(callUrl.pathname).toBe("/v1.0/me/chats");
-    expect(callUrl.searchParams.get("$filter")).toBe(
-      "chatType eq 'oneOnOne' and members/any(m:m/microsoft.graph.aadUserConversationMember/userId eq 'user-aad-object-id-123')",
-    );
-    expect(callUrl.searchParams.get("$select")).toBe("id");
-    expect(result).toBe("19:dm-chat-id@unq.gbl.spaces");
-  });
-
-  it("resolves personal DM chat ID without user AAD object ID (lists all 1:1 chats)", async () => {
-    const fetchFn = vi.fn(
-      async () =>
-        new Response(JSON.stringify({ value: [{ id: "19:fallback-chat@unq.gbl.spaces" }] }), {
-          status: 200,
-          headers: { "content-type": "application/json" },
-        }),
-    );
-
-    const result = await resolveGraphChatId({
-      botFrameworkConversationId: "8:orgid:user-object-id",
-      tokenProvider,
-      fetchFn: withFetchPreconnect(fetchFn),
-    });
-
-    expect(fetchFn).toHaveBeenCalledOnce();
-    expect(result).toBe("19:fallback-chat@unq.gbl.spaces");
-  });
-
-  it("returns null when Graph API returns no chats", async () => {
-    const fetchFn = vi.fn(
-      async () =>
-        new Response(JSON.stringify({ value: [] }), {
-          status: 200,
-          headers: { "content-type": "application/json" },
-        }),
-    );
-
-    const result = await resolveGraphChatId({
-      botFrameworkConversationId: "a:1unknown_dm",
-      userAadObjectId: "some-user",
-      tokenProvider,
-      fetchFn: withFetchPreconnect(fetchFn),
-    });
-
-    expect(result).toBeNull();
-  });
-
-  it("returns null when Graph API call fails", async () => {
-    const fetchFn = vi.fn(
-      async () =>
-        new Response("Unauthorized", {
-          status: 401,
-          headers: { "content-type": "text/plain" },
-        }),
-    );
-
-    const result = await resolveGraphChatId({
-      botFrameworkConversationId: "a:1some_dm_id",
-      userAadObjectId: "some-user",
-      tokenProvider,
-      fetchFn: withFetchPreconnect(fetchFn),
-    });
-
-    expect(result).toBeNull();
-  });
-});
-
-describe("buildTeamsFileInfoCard", () => {
-  it("extracts a unique id from quoted etags and lowercases file extensions", () => {
-    expect(
-      buildTeamsFileInfoCard({
-        eTag: '"{ABC-123},42"',
-        name: "Quarterly.Report.PDF",
-        webDavUrl: "https://sharepoint.example.com/file.pdf",
-      }),
-    ).toEqual({
-      contentType: "application/vnd.microsoft.teams.card.file.info",
-      contentUrl: "https://sharepoint.example.com/file.pdf",
-      name: "Quarterly.Report.PDF",
-      content: {
-        uniqueId: "ABC-123",
-        fileType: "pdf",
-      },
-    });
-  });
-
-  it("keeps the raw etag when no version suffix exists and handles extensionless files", () => {
-    expect(
-      buildTeamsFileInfoCard({
-        eTag: "plain-etag",
-        name: "README",
-        webDavUrl: "https://sharepoint.example.com/readme",
-      }),
-    ).toEqual({
-      contentType: "application/vnd.microsoft.teams.card.file.info",
-      contentUrl: "https://sharepoint.example.com/readme",
-      name: "README",
-      content: {
-        uniqueId: "plain-etag",
-        fileType: "",
-      },
-    });
-  });
-});