@openclaw/msteams 2026.5.2 → 2026.5.3-beta.1

package/src/graph-group-management.test.ts

@@ -1,318 +0,0 @@
-import { beforeEach, describe, expect, it, vi } from "vitest";
-import type { OpenClawConfig } from "../runtime-api.js";
-import {
-  addParticipantMSTeams,
-  removeParticipantMSTeams,
-  renameGroupMSTeams,
-} from "./graph-group-management.js";
-
-const mockState = vi.hoisted(() => ({
-  resolveGraphToken: vi.fn(),
-  fetchGraphJson: vi.fn(),
-  postGraphJson: vi.fn(),
-  deleteGraphRequest: vi.fn(),
-  patchGraphJson: vi.fn(),
-  findPreferredDmByUserId: vi.fn(),
-}));
-
-vi.mock("./graph.js", async (importOriginal) => {
-  const actual = await importOriginal<typeof import("./graph.js")>();
-  return {
-    ...actual,
-    resolveGraphToken: mockState.resolveGraphToken,
-    fetchGraphJson: mockState.fetchGraphJson,
-    postGraphJson: mockState.postGraphJson,
-    deleteGraphRequest: mockState.deleteGraphRequest,
-    patchGraphJson: mockState.patchGraphJson,
-  };
-});
-
-vi.mock("./conversation-store-fs.js", () => ({
-  createMSTeamsConversationStoreFs: () => ({
-    findPreferredDmByUserId: mockState.findPreferredDmByUserId,
-  }),
-}));
-
-const TOKEN = "test-graph-token";
-const CHAT_ID = "19:abc@thread.tacv2";
-const CHANNEL_TO = "team-id-1/channel-id-1";
-
-describe("addParticipantMSTeams", () => {
-  beforeEach(() => {
-    vi.clearAllMocks();
-    mockState.resolveGraphToken.mockResolvedValue(TOKEN);
-  });
-
-  it("adds member to a chat with default role", async () => {
-    mockState.postGraphJson.mockResolvedValue({});
-
-    const result = await addParticipantMSTeams({
-      cfg: {} as OpenClawConfig,
-      to: CHAT_ID,
-      userId: "user-aad-id-1",
-    });
-
-    expect(result).toEqual({ added: { userId: "user-aad-id-1", chatId: CHAT_ID } });
-    expect(mockState.postGraphJson).toHaveBeenCalledWith({
-      token: TOKEN,
-      path: `/chats/${encodeURIComponent(CHAT_ID)}/members`,
-      body: {
-        "@odata.type": "#microsoft.graph.aadUserConversationMember",
-        roles: ["member"],
-        "user@odata.bind": "https://graph.microsoft.com/v1.0/users('user-aad-id-1')",
-      },
-    });
-  });
-
-  it("adds member to a chat with owner role", async () => {
-    mockState.postGraphJson.mockResolvedValue({});
-
-    const result = await addParticipantMSTeams({
-      cfg: {} as OpenClawConfig,
-      to: CHAT_ID,
-      userId: "user-aad-id-2",
-      role: "owner",
-    });
-
-    expect(result).toEqual({ added: { userId: "user-aad-id-2", chatId: CHAT_ID } });
-    expect(mockState.postGraphJson).toHaveBeenCalledWith({
-      token: TOKEN,
-      path: `/chats/${encodeURIComponent(CHAT_ID)}/members`,
-      body: {
-        "@odata.type": "#microsoft.graph.aadUserConversationMember",
-        roles: ["owner"],
-        "user@odata.bind": "https://graph.microsoft.com/v1.0/users('user-aad-id-2')",
-      },
-    });
-  });
-
-  it("normalizes role casing and whitespace", async () => {
-    mockState.postGraphJson.mockResolvedValue({});
-
-    await addParticipantMSTeams({
-      cfg: {} as OpenClawConfig,
-      to: CHAT_ID,
-      userId: "user-aad-id-2",
-      role: " OWNER ",
-    });
-
-    expect(mockState.postGraphJson).toHaveBeenCalledWith(
-      expect.objectContaining({
-        body: expect.objectContaining({
-          roles: ["owner"],
-        }),
-      }),
-    );
-  });
-
-  it("rejects unknown roles", async () => {
-    await expect(
-      addParticipantMSTeams({
-        cfg: {} as OpenClawConfig,
-        to: CHAT_ID,
-        userId: "user-aad-id-2",
-        role: "admin",
-      }),
-    ).rejects.toThrow('role must be "member" or "owner"');
-
-    expect(mockState.postGraphJson).not.toHaveBeenCalled();
-  });
-
-  it("constructs correct user@odata.bind URL", async () => {
-    mockState.postGraphJson.mockResolvedValue({});
-
-    await addParticipantMSTeams({
-      cfg: {} as OpenClawConfig,
-      to: CHAT_ID,
-      userId: "abc-def-123",
-    });
-
-    const calledBody = mockState.postGraphJson.mock.calls[0][0].body;
-    expect(calledBody["user@odata.bind"]).toBe(
-      "https://graph.microsoft.com/v1.0/users('abc-def-123')",
-    );
-  });
-
-  it("escapes user ids before building the OData bind URL", async () => {
-    mockState.postGraphJson.mockResolvedValue({});
-
-    await addParticipantMSTeams({
-      cfg: {} as OpenClawConfig,
-      to: CHAT_ID,
-      userId: "o'hara@example.com",
-    });
-
-    const calledBody = mockState.postGraphJson.mock.calls[0][0].body;
-    expect(calledBody["user@odata.bind"]).toBe(
-      "https://graph.microsoft.com/v1.0/users('o''hara@example.com')",
-    );
-  });
-
-  it("adds member to a channel", async () => {
-    mockState.postGraphJson.mockResolvedValue({});
-
-    const result = await addParticipantMSTeams({
-      cfg: {} as OpenClawConfig,
-      to: CHANNEL_TO,
-      userId: "user-aad-id-3",
-    });
-
-    expect(result).toEqual({ added: { userId: "user-aad-id-3", chatId: CHANNEL_TO } });
-    expect(mockState.postGraphJson).toHaveBeenCalledWith({
-      token: TOKEN,
-      path: "/teams/team-id-1/channels/channel-id-1/members",
-      body: {
-        "@odata.type": "#microsoft.graph.aadUserConversationMember",
-        roles: ["member"],
-        "user@odata.bind": "https://graph.microsoft.com/v1.0/users('user-aad-id-3')",
-      },
-    });
-  });
-});
-
-describe("removeParticipantMSTeams", () => {
-  beforeEach(() => {
-    vi.clearAllMocks();
-    mockState.resolveGraphToken.mockResolvedValue(TOKEN);
-  });
-
-  it("lists members, finds match, deletes by membershipId", async () => {
-    mockState.fetchGraphJson.mockResolvedValue({
-      value: [
-        { id: "membership-1", userId: "user-aad-id-1" },
-        { id: "membership-2", userId: "user-aad-id-2" },
-      ],
-    });
-    mockState.deleteGraphRequest.mockResolvedValue(undefined);
-
-    const result = await removeParticipantMSTeams({
-      cfg: {} as OpenClawConfig,
-      to: CHAT_ID,
-      userId: "user-aad-id-2",
-    });
-
-    expect(result).toEqual({ removed: { userId: "user-aad-id-2", chatId: CHAT_ID } });
-    expect(mockState.fetchGraphJson).toHaveBeenCalledWith({
-      token: TOKEN,
-      path: `/chats/${encodeURIComponent(CHAT_ID)}/members`,
-    });
-    expect(mockState.deleteGraphRequest).toHaveBeenCalledWith({
-      token: TOKEN,
-      path: `/chats/${encodeURIComponent(CHAT_ID)}/members/membership-2`,
-    });
-  });
-
-  it("throws when user not found in member list", async () => {
-    mockState.fetchGraphJson.mockResolvedValue({
-      value: [
-        { id: "membership-1", userId: "user-aad-id-1" },
-        { id: "membership-3", userId: "user-aad-id-3" },
-      ],
-    });
-
-    await expect(
-      removeParticipantMSTeams({
-        cfg: {} as OpenClawConfig,
-        to: CHAT_ID,
-        userId: "user-not-in-list",
-      }),
-    ).rejects.toThrow("User user-not-in-list is not a member of this conversation");
-  });
-
-  it("removes member from a channel", async () => {
-    mockState.fetchGraphJson.mockResolvedValue({
-      value: [{ id: "membership-5", userId: "user-aad-id-5" }],
-    });
-    mockState.deleteGraphRequest.mockResolvedValue(undefined);
-
-    const result = await removeParticipantMSTeams({
-      cfg: {} as OpenClawConfig,
-      to: CHANNEL_TO,
-      userId: "user-aad-id-5",
-    });
-
-    expect(result).toEqual({ removed: { userId: "user-aad-id-5", chatId: CHANNEL_TO } });
-    expect(mockState.fetchGraphJson).toHaveBeenCalledWith({
-      token: TOKEN,
-      path: "/teams/team-id-1/channels/channel-id-1/members",
-    });
-    expect(mockState.deleteGraphRequest).toHaveBeenCalledWith({
-      token: TOKEN,
-      path: "/teams/team-id-1/channels/channel-id-1/members/membership-5",
-    });
-  });
-
-  it("follows member pagination before concluding the user is missing", async () => {
-    mockState.fetchGraphJson
-      .mockResolvedValueOnce({
-        value: [{ id: "membership-1", userId: "user-aad-id-1" }],
-        "@odata.nextLink":
-          "https://graph.microsoft.com/v1.0/chats/19%3Aabc%40thread.tacv2/members?$skip=2",
-      })
-      .mockResolvedValueOnce({
-        value: [{ id: "membership-9", userId: "user-aad-id-9" }],
-      });
-    mockState.deleteGraphRequest.mockResolvedValue(undefined);
-
-    const result = await removeParticipantMSTeams({
-      cfg: {} as OpenClawConfig,
-      to: CHAT_ID,
-      userId: "user-aad-id-9",
-    });
-
-    expect(result).toEqual({ removed: { userId: "user-aad-id-9", chatId: CHAT_ID } });
-    expect(mockState.fetchGraphJson).toHaveBeenNthCalledWith(1, {
-      token: TOKEN,
-      path: `/chats/${encodeURIComponent(CHAT_ID)}/members`,
-    });
-    expect(mockState.fetchGraphJson).toHaveBeenNthCalledWith(2, {
-      token: TOKEN,
-      path: `/chats/${encodeURIComponent(CHAT_ID)}/members?$skip=2`,
-    });
-    expect(mockState.deleteGraphRequest).toHaveBeenCalledWith({
-      token: TOKEN,
-      path: `/chats/${encodeURIComponent(CHAT_ID)}/members/membership-9`,
-    });
-  });
-});
-
-describe("renameGroupMSTeams", () => {
-  beforeEach(() => {
-    vi.clearAllMocks();
-    mockState.resolveGraphToken.mockResolvedValue(TOKEN);
-  });
-
-  it("renames a chat with topic", async () => {
-    mockState.patchGraphJson.mockResolvedValue(undefined);
-
-    const result = await renameGroupMSTeams({
-      cfg: {} as OpenClawConfig,
-      to: CHAT_ID,
-      name: "New Chat Name",
-    });
-
-    expect(result).toEqual({ renamed: { chatId: CHAT_ID, newName: "New Chat Name" } });
-    expect(mockState.patchGraphJson).toHaveBeenCalledWith({
-      token: TOKEN,
-      path: `/chats/${encodeURIComponent(CHAT_ID)}`,
-      body: { topic: "New Chat Name" },
-    });
-  });
-
-  it("renames a channel with displayName", async () => {
-    mockState.patchGraphJson.mockResolvedValue(undefined);
-
-    const result = await renameGroupMSTeams({
-      cfg: {} as OpenClawConfig,
-      to: CHANNEL_TO,
-      name: "New Channel Name",
-    });
-
-    expect(result).toEqual({ renamed: { chatId: CHANNEL_TO, newName: "New Channel Name" } });
-    expect(mockState.patchGraphJson).toHaveBeenCalledWith({
-      token: TOKEN,
-      path: "/teams/team-id-1/channels/channel-id-1",
-      body: { displayName: "New Channel Name" },
-    });
-  });
-});

package/src/graph-group-management.ts

@@ -1,168 +0,0 @@
-import type { OpenClawConfig } from "../runtime-api.js";
-import { resolveConversationPath, resolveGraphConversationId } from "./graph-messages.js";
-import {
-  deleteGraphRequest,
-  escapeOData,
-  fetchGraphJson,
-  patchGraphJson,
-  postGraphJson,
-  resolveGraphToken,
-} from "./graph.js";
-
-// ---------------------------------------------------------------------------
-// Add Participant
-// ---------------------------------------------------------------------------
-
-type AddParticipantMSTeamsParams = {
-  cfg: OpenClawConfig;
-  to: string;
-  userId: string;
-  role?: string;
-};
-
-type AddParticipantMSTeamsResult = {
-  added: { userId: string; chatId: string };
-};
-
-type ConversationMemberRole = "member" | "owner";
-
-function normalizeConversationMemberRole(role: string | undefined): ConversationMemberRole {
-  const normalized = role?.trim().toLowerCase() ?? "";
-  if (!normalized) {
-    return "member";
-  }
-  if (normalized === "member" || normalized === "owner") {
-    return normalized;
-  }
-  throw new Error('MS Teams participant role must be "member" or "owner".');
-}
-
-/**
- * Add a user to a chat or channel via Graph API.
- */
-export async function addParticipantMSTeams(
-  params: AddParticipantMSTeamsParams,
-): Promise<AddParticipantMSTeamsResult> {
-  const token = await resolveGraphToken(params.cfg);
-  const conversationId = await resolveGraphConversationId(params.to);
-  const conv = resolveConversationPath(conversationId);
-
-  const body = {
-    "@odata.type": "#microsoft.graph.aadUserConversationMember",
-    roles: [normalizeConversationMemberRole(params.role)],
-    "user@odata.bind": `https://graph.microsoft.com/v1.0/users('${escapeOData(params.userId)}')`,
-  };
-
-  await postGraphJson<unknown>({
-    token,
-    path: `${conv.basePath}/members`,
-    body,
-  });
-
-  return { added: { userId: params.userId, chatId: conversationId } };
-}
-
-// ---------------------------------------------------------------------------
-// Remove Participant
-// ---------------------------------------------------------------------------
-
-type RemoveParticipantMSTeamsParams = {
-  cfg: OpenClawConfig;
-  to: string;
-  userId: string;
-};
-
-type RemoveParticipantMSTeamsResult = {
-  removed: { userId: string; chatId: string };
-};
-
-type GraphConversationMember = {
-  id?: string;
-  userId?: string;
-};
-
-type GraphConversationMemberResponse = {
-  value?: GraphConversationMember[];
-  "@odata.nextLink"?: string;
-};
-
-/**
- * Remove a user from a chat or channel via Graph API.
- * Lists members first to resolve the membership ID, then deletes.
- */
-export async function removeParticipantMSTeams(
-  params: RemoveParticipantMSTeamsParams,
-): Promise<RemoveParticipantMSTeamsResult> {
-  const token = await resolveGraphToken(params.cfg);
-  const conversationId = await resolveGraphConversationId(params.to);
-  const conv = resolveConversationPath(conversationId);
-
-  // List members to find the membership ID for the target user. Graph can
-  // paginate large chats/channels, so walk `@odata.nextLink` before concluding
-  // the user is missing.
-  const MAX_PAGES = 10;
-  let nextPath: string | undefined = `${conv.basePath}/members`;
-  let page = 0;
-  let member: GraphConversationMember | undefined;
-  while (nextPath && page < MAX_PAGES && !member) {
-    const membersRes: GraphConversationMemberResponse =
-      await fetchGraphJson<GraphConversationMemberResponse>({
-        token,
-        path: nextPath,
-      });
-    member = (membersRes.value ?? []).find(
-      (candidate: GraphConversationMember) => candidate.userId === params.userId,
-    );
-    if (member) {
-      break;
-    }
-    const nextLink: string | undefined = membersRes["@odata.nextLink"];
-    nextPath = nextLink ? nextLink.replace("https://graph.microsoft.com/v1.0", "") : undefined;
-    page++;
-  }
-  if (!member?.id) {
-    throw new Error(`User ${params.userId} is not a member of this conversation`);
-  }
-
-  await deleteGraphRequest({
-    token,
-    path: `${conv.basePath}/members/${encodeURIComponent(member.id)}`,
-  });
-
-  return { removed: { userId: params.userId, chatId: conversationId } };
-}
-
-// ---------------------------------------------------------------------------
-// Rename Group
-// ---------------------------------------------------------------------------
-
-type RenameGroupMSTeamsParams = {
-  cfg: OpenClawConfig;
-  to: string;
-  name: string;
-};
-
-type RenameGroupMSTeamsResult = {
-  renamed: { chatId: string; newName: string };
-};
-
-/**
- * Rename a chat (topic) or channel (displayName) via Graph API.
- */
-export async function renameGroupMSTeams(
-  params: RenameGroupMSTeamsParams,
-): Promise<RenameGroupMSTeamsResult> {
-  const token = await resolveGraphToken(params.cfg);
-  const conversationId = await resolveGraphConversationId(params.to);
-  const conv = resolveConversationPath(conversationId);
-
-  const body = conv.kind === "chat" ? { topic: params.name } : { displayName: params.name };
-
-  await patchGraphJson<unknown>({
-    token,
-    path: conv.basePath,
-    body,
-  });
-
-  return { renamed: { chatId: conversationId, newName: params.name } };
-}

package/src/graph-members.test.ts

@@ -1,89 +0,0 @@
-import { beforeEach, describe, expect, it, vi } from "vitest";
-import type { OpenClawConfig } from "../runtime-api.js";
-import { getMemberInfoMSTeams } from "./graph-members.js";
-
-const mockState = vi.hoisted(() => ({
-  resolveGraphToken: vi.fn(),
-  fetchGraphJson: vi.fn(),
-}));
-
-vi.mock("./graph.js", () => {
-  return {
-    resolveGraphToken: mockState.resolveGraphToken,
-    fetchGraphJson: mockState.fetchGraphJson,
-  };
-});
-
-const TOKEN = "test-graph-token";
-
-describe("getMemberInfoMSTeams", () => {
-  beforeEach(() => {
-    vi.clearAllMocks();
-    mockState.resolveGraphToken.mockResolvedValue(TOKEN);
-  });
-
-  it("fetches user profile and maps all fields", async () => {
-    mockState.fetchGraphJson.mockResolvedValue({
-      id: "user-123",
-      displayName: "Alice Smith",
-      mail: "alice@contoso.com",
-      jobTitle: "Engineer",
-      userPrincipalName: "alice@contoso.com",
-      officeLocation: "Building 1",
-    });
-
-    const result = await getMemberInfoMSTeams({
-      cfg: {} as OpenClawConfig,
-      userId: "user-123",
-    });
-
-    expect(result).toEqual({
-      user: {
-        id: "user-123",
-        displayName: "Alice Smith",
-        mail: "alice@contoso.com",
-        jobTitle: "Engineer",
-        userPrincipalName: "alice@contoso.com",
-        officeLocation: "Building 1",
-      },
-    });
-    expect(mockState.fetchGraphJson).toHaveBeenCalledWith({
-      token: TOKEN,
-      path: `/users/${encodeURIComponent("user-123")}?$select=id,displayName,mail,jobTitle,userPrincipalName,officeLocation`,
-    });
-  });
-
-  it("handles sparse data with some fields undefined", async () => {
-    mockState.fetchGraphJson.mockResolvedValue({
-      id: "user-456",
-      displayName: "Bob",
-    });
-
-    const result = await getMemberInfoMSTeams({
-      cfg: {} as OpenClawConfig,
-      userId: "user-456",
-    });
-
-    expect(result).toEqual({
-      user: {
-        id: "user-456",
-        displayName: "Bob",
-        mail: undefined,
-        jobTitle: undefined,
-        userPrincipalName: undefined,
-        officeLocation: undefined,
-      },
-    });
-  });
-
-  it("propagates Graph API errors", async () => {
-    mockState.fetchGraphJson.mockRejectedValue(new Error("Graph API 404: user not found"));
-
-    await expect(
-      getMemberInfoMSTeams({
-        cfg: {} as OpenClawConfig,
-        userId: "nonexistent-user",
-      }),
-    ).rejects.toThrow("Graph API 404: user not found");
-  });
-});

package/src/graph-members.ts

@@ -1,48 +0,0 @@
-import type { OpenClawConfig } from "../runtime-api.js";
-import { fetchGraphJson, resolveGraphToken } from "./graph.js";
-
-type GraphUserProfile = {
-  id?: string;
-  displayName?: string;
-  mail?: string;
-  jobTitle?: string;
-  userPrincipalName?: string;
-  officeLocation?: string;
-};
-
-type GetMemberInfoMSTeamsParams = {
-  cfg: OpenClawConfig;
-  userId: string;
-};
-
-type GetMemberInfoMSTeamsResult = {
-  user: {
-    id: string | undefined;
-    displayName: string | undefined;
-    mail: string | undefined;
-    jobTitle: string | undefined;
-    userPrincipalName: string | undefined;
-    officeLocation: string | undefined;
-  };
-};
-
-/**
- * Fetch a user profile from Microsoft Graph by user ID.
- */
-export async function getMemberInfoMSTeams(
-  params: GetMemberInfoMSTeamsParams,
-): Promise<GetMemberInfoMSTeamsResult> {
-  const token = await resolveGraphToken(params.cfg);
-  const path = `/users/${encodeURIComponent(params.userId)}?$select=id,displayName,mail,jobTitle,userPrincipalName,officeLocation`;
-  const user = await fetchGraphJson<GraphUserProfile>({ token, path });
-  return {
-    user: {
-      id: user.id,
-      displayName: user.displayName,
-      mail: user.mail,
-      jobTitle: user.jobTitle,
-      userPrincipalName: user.userPrincipalName,
-      officeLocation: user.officeLocation,
-    },
-  };
-}