@kodelyth/msteams 2026.5.42 → 2026.6.2

package/src/graph-messages.search.test.ts

@@ -1,227 +0,0 @@
-import { beforeAll, describe, expect, it } from "vitest";
-import type { KlawConfig } from "../runtime-api.js";
-import {
-  CHANNEL_TO,
-  CHAT_ID,
-  type GraphMessagesTestModule,
-  getGraphMessagesMockState,
-  installGraphMessagesMockDefaults,
-  loadGraphMessagesTestModule,
-} from "./graph-messages.test-helpers.js";
-
-const mockState = getGraphMessagesMockState();
-installGraphMessagesMockDefaults();
-let searchMessagesMSTeams: GraphMessagesTestModule["searchMessagesMSTeams"];
-
-beforeAll(async () => {
-  ({ searchMessagesMSTeams } = await loadGraphMessagesTestModule());
-});
-
-function readFirstGraphPath(): string {
-  const [call] = mockState.fetchGraphJson.mock.calls;
-  if (!call) {
-    throw new Error("Expected Graph fetch call");
-  }
-  const [request] = call;
-  if (!request || typeof request !== "object" || typeof request.path !== "string") {
-    throw new Error("Expected Graph fetch request path");
-  }
-  return request.path;
-}
-
-describe("searchMessagesMSTeams", () => {
-  it("searches chat messages with query string", async () => {
-    mockState.fetchGraphJson.mockResolvedValue({
-      value: [
-        {
-          id: "msg-1",
-          body: { content: "Meeting notes from Monday" },
-          from: { user: { id: "u1", displayName: "Alice" } },
-          createdDateTime: "2026-03-25T10:00:00Z",
-        },
-      ],
-    });
-
-    const result = await searchMessagesMSTeams({
-      cfg: {} as KlawConfig,
-      to: CHAT_ID,
-      query: "meeting notes",
-    });
-
-    expect(result.messages).toEqual([
-      {
-        id: "msg-1",
-        text: "Meeting notes from Monday",
-        from: { user: { id: "u1", displayName: "Alice" } },
-        createdAt: "2026-03-25T10:00:00Z",
-      },
-    ]);
-    const calledPath = readFirstGraphPath();
-    expect(calledPath).toContain(`/chats/${encodeURIComponent(CHAT_ID)}/messages?`);
-    expect(calledPath).toContain("$search=");
-    expect(calledPath).toContain("$top=25");
-    const decoded = decodeURIComponent(calledPath);
-    expect(decoded).toContain('$search="meeting notes"');
-  });
-
-  it("searches channel messages", async () => {
-    mockState.fetchGraphJson.mockResolvedValue({
-      value: [
-        {
-          id: "msg-2",
-          body: { content: "Sprint review" },
-          from: { user: { id: "u2", displayName: "Bob" } },
-          createdDateTime: "2026-03-25T11:00:00Z",
-        },
-      ],
-    });
-
-    const result = await searchMessagesMSTeams({
-      cfg: {} as KlawConfig,
-      to: CHANNEL_TO,
-      query: "sprint",
-    });
-
-    expect(result.messages).toHaveLength(1);
-    const calledPath = readFirstGraphPath();
-    expect(calledPath).toContain("/teams/team-id-1/channels/channel-id-1/messages?");
-  });
-
-  it("applies limit parameter", async () => {
-    mockState.fetchGraphJson.mockResolvedValue({ value: [] });
-
-    await searchMessagesMSTeams({
-      cfg: {} as KlawConfig,
-      to: CHAT_ID,
-      query: "test",
-      limit: 10,
-    });
-
-    const calledPath = readFirstGraphPath();
-    expect(calledPath).toContain("$top=10");
-  });
-
-  it("clamps limit to max 50", async () => {
-    mockState.fetchGraphJson.mockResolvedValue({ value: [] });
-
-    await searchMessagesMSTeams({
-      cfg: {} as KlawConfig,
-      to: CHAT_ID,
-      query: "test",
-      limit: 100,
-    });
-
-    const calledPath = readFirstGraphPath();
-    expect(calledPath).toContain("$top=50");
-  });
-
-  it("clamps limit to min 1", async () => {
-    mockState.fetchGraphJson.mockResolvedValue({ value: [] });
-
-    await searchMessagesMSTeams({
-      cfg: {} as KlawConfig,
-      to: CHAT_ID,
-      query: "test",
-      limit: 0,
-    });
-
-    const calledPath = readFirstGraphPath();
-    expect(calledPath).toContain("$top=1");
-  });
-
-  it("applies from filter", async () => {
-    mockState.fetchGraphJson.mockResolvedValue({ value: [] });
-
-    await searchMessagesMSTeams({
-      cfg: {} as KlawConfig,
-      to: CHAT_ID,
-      query: "budget",
-      from: "Alice",
-    });
-
-    const calledPath = readFirstGraphPath();
-    expect(calledPath).toContain("$filter=");
-    const decoded = decodeURIComponent(calledPath);
-    expect(decoded).toContain("from/user/displayName eq 'Alice'");
-  });
-
-  it("escapes single quotes in from filter", async () => {
-    mockState.fetchGraphJson.mockResolvedValue({ value: [] });
-
-    await searchMessagesMSTeams({
-      cfg: {} as KlawConfig,
-      to: CHAT_ID,
-      query: "test",
-      from: "O'Brien",
-    });
-
-    const calledPath = readFirstGraphPath();
-    const decoded = decodeURIComponent(calledPath);
-    expect(decoded).toContain("O''Brien");
-  });
-
-  it("strips double quotes from query to prevent injection", async () => {
-    mockState.fetchGraphJson.mockResolvedValue({ value: [] });
-
-    await searchMessagesMSTeams({
-      cfg: {} as KlawConfig,
-      to: CHAT_ID,
-      query: 'say "hello" world',
-    });
-
-    const calledPath = readFirstGraphPath();
-    const decoded = decodeURIComponent(calledPath);
-    expect(decoded).toContain('$search="say hello world"');
-    expect(decoded).not.toContain('""');
-  });
-
-  it("passes ConsistencyLevel: eventual header", async () => {
-    mockState.fetchGraphJson.mockResolvedValue({ value: [] });
-
-    await searchMessagesMSTeams({
-      cfg: {} as KlawConfig,
-      to: CHAT_ID,
-      query: "test",
-    });
-
-    expect(mockState.fetchGraphJson).toHaveBeenCalledWith({
-      token: "test-graph-token",
-      path: `/chats/${encodeURIComponent(CHAT_ID)}/messages?$search=${encodeURIComponent(
-        '"test"',
-      )}&$top=25`,
-      headers: { ConsistencyLevel: "eventual" },
-    });
-  });
-
-  it("returns empty array when no messages match", async () => {
-    mockState.fetchGraphJson.mockResolvedValue({ value: [] });
-
-    const result = await searchMessagesMSTeams({
-      cfg: {} as KlawConfig,
-      to: CHAT_ID,
-      query: "nonexistent",
-    });
-
-    expect(result.messages).toStrictEqual([]);
-  });
-
-  it("resolves user: target through conversation store", async () => {
-    mockState.findPreferredDmByUserId.mockResolvedValue({
-      conversationId: "a:bot-id",
-      reference: { graphChatId: "19:dm-chat@thread.tacv2" },
-    });
-    mockState.fetchGraphJson.mockResolvedValue({ value: [] });
-
-    await searchMessagesMSTeams({
-      cfg: {} as KlawConfig,
-      to: "user:aad-user-1",
-      query: "hello",
-    });
-
-    expect(mockState.findPreferredDmByUserId).toHaveBeenCalledWith("aad-user-1");
-    const calledPath = readFirstGraphPath();
-    expect(calledPath).toContain(
-      `/chats/${encodeURIComponent("19:dm-chat@thread.tacv2")}/messages?`,
-    );
-  });
-});

package/src/graph-messages.test-helpers.ts

@@ -1,50 +0,0 @@
-import { beforeEach, vi } from "vitest";
-
-const graphMessagesMockState = vi.hoisted(() => ({
-  resolveGraphToken: vi.fn(),
-  fetchGraphJson: vi.fn(),
-  fetchGraphAbsoluteUrl: vi.fn(),
-  postGraphJson: vi.fn(),
-  postGraphBetaJson: vi.fn(),
-  deleteGraphRequest: vi.fn(),
-  findPreferredDmByUserId: vi.fn(),
-}));
-
-vi.mock("./graph.js", () => {
-  return {
-    resolveGraphToken: graphMessagesMockState.resolveGraphToken,
-    fetchGraphJson: graphMessagesMockState.fetchGraphJson,
-    fetchGraphAbsoluteUrl: graphMessagesMockState.fetchGraphAbsoluteUrl,
-    postGraphJson: graphMessagesMockState.postGraphJson,
-    postGraphBetaJson: graphMessagesMockState.postGraphBetaJson,
-    deleteGraphRequest: graphMessagesMockState.deleteGraphRequest,
-    escapeOData: vi.fn((value: string) => value.replaceAll("'", "''")),
-  };
-});
-
-vi.mock("./conversation-store-fs.js", () => ({
-  createMSTeamsConversationStoreFs: () => ({
-    findPreferredDmByUserId: graphMessagesMockState.findPreferredDmByUserId,
-  }),
-}));
-
-export const TOKEN = "test-graph-token";
-export const CHAT_ID = "19:abc@thread.tacv2";
-export const CHANNEL_TO = "team-id-1/channel-id-1";
-
-export function getGraphMessagesMockState(): typeof graphMessagesMockState {
-  return graphMessagesMockState;
-}
-
-export type GraphMessagesTestModule = typeof import("./graph-messages.js");
-
-export function loadGraphMessagesTestModule(): Promise<GraphMessagesTestModule> {
-  return import("./graph-messages.js");
-}
-
-export function installGraphMessagesMockDefaults(): void {
-  beforeEach(() => {
-    vi.clearAllMocks();
-    graphMessagesMockState.resolveGraphToken.mockResolvedValue(TOKEN);
-  });
-}