@kodelyth/msteams 2026.5.42 → 2026.6.2

package/src/monitor-handler/access.ts

@@ -1,136 +0,0 @@
-import {
-  channelIngressRoutes,
-  resolveStableChannelMessageIngress,
-  type StableChannelIngressIdentityParams,
-} from "klaw/plugin-sdk/channel-ingress-runtime";
-import { normalizeOptionalLowercaseString } from "klaw/plugin-sdk/string-coerce-runtime";
-import {
-  DEFAULT_ACCOUNT_ID,
-  createChannelPairingController,
-  isDangerousNameMatchingEnabled,
-  resolveDefaultGroupPolicy,
-  type KlawConfig,
-} from "../../runtime-api.js";
-import { normalizeMSTeamsConversationId } from "../inbound.js";
-import { resolveMSTeamsRouteConfig } from "../policy.js";
-import { getMSTeamsRuntime } from "../runtime.js";
-import type { MSTeamsTurnContext } from "../sdk-types.js";
-
-const MSTEAMS_SENDER_NAME_KIND = "plugin:msteams-sender-name" as const;
-const msteamsIngressIdentity = {
-  key: "sender-id",
-  normalize: normalizeIngressValue,
-  aliases: [
-    {
-      key: "sender-name",
-      kind: MSTEAMS_SENDER_NAME_KIND,
-      normalizeEntry: normalizeIngressValue,
-      normalizeSubject: normalizeIngressValue,
-      dangerous: true,
-    },
-  ],
-  isWildcardEntry: (entry) => normalizeIngressValue(entry) === "*",
-  resolveEntryId: ({ entryIndex, fieldKey }) =>
-    `msteams-entry-${entryIndex + 1}:${fieldKey === "sender-name" ? "name" : "id"}`,
-} satisfies StableChannelIngressIdentityParams;
-
-function normalizeIngressValue(value?: string | null): string | null {
-  return normalizeOptionalLowercaseString(value) ?? null;
-}
-
-export async function resolveMSTeamsSenderAccess(params: {
-  cfg: KlawConfig;
-  activity: MSTeamsTurnContext["activity"];
-  hasControlCommand?: boolean;
-}) {
-  const activity = params.activity;
-  const msteamsCfg = params.cfg.channels?.msteams;
-  const conversationId = normalizeMSTeamsConversationId(activity.conversation?.id ?? "unknown");
-  const convType = normalizeOptionalLowercaseString(activity.conversation?.conversationType);
-  const isDirectMessage = convType === "personal" || (!convType && !activity.conversation?.isGroup);
-  const senderId = activity.from?.aadObjectId ?? activity.from?.id ?? "unknown";
-  const senderName = activity.from?.name ?? activity.from?.id ?? senderId;
-
-  const core = getMSTeamsRuntime();
-  const pairing = createChannelPairingController({
-    core,
-    channel: "msteams",
-    accountId: DEFAULT_ACCOUNT_ID,
-  });
-  const dmPolicy = msteamsCfg?.dmPolicy ?? "pairing";
-  const configuredDmAllowFrom = msteamsCfg?.allowFrom ?? [];
-  const groupAllowFrom = msteamsCfg?.groupAllowFrom;
-  const defaultGroupPolicy = resolveDefaultGroupPolicy(params.cfg);
-  const groupPolicy =
-    !isDirectMessage && msteamsCfg
-      ? (msteamsCfg.groupPolicy ?? defaultGroupPolicy ?? "allowlist")
-      : "disabled";
-  const allowNameMatching = isDangerousNameMatchingEnabled(msteamsCfg);
-  const channelGate = resolveMSTeamsRouteConfig({
-    cfg: msteamsCfg,
-    teamId: activity.channelData?.team?.id,
-    teamName: activity.channelData?.team?.name,
-    conversationId,
-    channelName: activity.channelData?.channel?.name,
-    allowNameMatching,
-  });
-
-  const resolved = await resolveStableChannelMessageIngress({
-    channelId: "msteams",
-    accountId: pairing.accountId,
-    identity: msteamsIngressIdentity,
-    cfg: params.cfg,
-    readStoreAllowFrom: pairing.readAllowFromStore,
-    subject: {
-      stableId: senderId,
-      aliases: { "sender-name": senderName },
-    },
-    conversation: {
-      kind: isDirectMessage ? "direct" : convType === "channel" ? "channel" : "group",
-      id: conversationId,
-      parentId: activity.channelData?.team?.id,
-    },
-    route: channelIngressRoutes(
-      !isDirectMessage &&
-        channelGate.allowlistConfigured && {
-          id: "msteams:team-channel",
-          kind: "nestedAllowlist",
-          allowed: channelGate.allowed,
-          precedence: 0,
-          matchId: "msteams-route",
-          ...(channelGate.allowed && groupPolicy === "allowlist"
-            ? {
-                senderPolicy: "deny-when-empty" as const,
-                senderAllowFromSource: "effective-group" as const,
-              }
-            : {}),
-        },
-    ),
-    dmPolicy,
-    groupPolicy,
-    policy: {
-      groupAllowFromFallbackToAllowFrom: true,
-      mutableIdentifierMatching: allowNameMatching ? "enabled" : "disabled",
-    },
-    allowFrom: configuredDmAllowFrom,
-    groupAllowFrom,
-    command: {
-      allowTextCommands: true,
-      hasControlCommand: params.hasControlCommand === true,
-      directGroupAllowFrom: isDirectMessage ? "effective" : "none",
-    },
-  });
-  return {
-    ...resolved,
-    pairing,
-    isDirectMessage,
-    conversationId,
-    senderId,
-    senderName,
-    msteamsCfg,
-    dmPolicy,
-    channelGate,
-    allowNameMatching,
-    groupPolicy,
-  };
-}

package/src/monitor-handler/inbound-media.test.ts

@@ -1,314 +0,0 @@
-import { describe, expect, it, vi } from "vitest";
-
-vi.mock("../attachments.js", () => ({
-  downloadMSTeamsAttachments: vi.fn(async () => []),
-  downloadMSTeamsGraphMedia: vi.fn(async () => ({ media: [] })),
-  downloadMSTeamsBotFrameworkAttachments: vi.fn(async () => ({ media: [], attachmentCount: 0 })),
-  buildMSTeamsGraphMessageUrls: vi.fn(() => [
-    "https://graph.microsoft.com/v1.0/chats/c/messages/m",
-  ]),
-  extractMSTeamsHtmlAttachmentIds: vi.fn(() => ["att-0", "att-1"]),
-  isBotFrameworkPersonalChatId: vi.fn((id: string | null | undefined) => {
-    if (typeof id !== "string") {
-      return false;
-    }
-    return id.startsWith("a:") || id.startsWith("8:orgid:");
-  }),
-}));
-
-import {
-  buildMSTeamsGraphMessageUrls,
-  downloadMSTeamsAttachments,
-  downloadMSTeamsBotFrameworkAttachments,
-  downloadMSTeamsGraphMedia,
-  extractMSTeamsHtmlAttachmentIds,
-} from "../attachments.js";
-import { resolveMSTeamsInboundMedia } from "./inbound-media.js";
-
-const baseParams = {
-  maxBytes: 1024 * 1024,
-  tokenProvider: { getAccessToken: vi.fn(async () => "token") },
-  conversationType: "personal",
-  conversationId: "19:user_bot@unq.gbl.spaces",
-  activity: { id: "msg-1", replyToId: undefined, channelData: {} },
-  log: { debug: vi.fn() },
-};
-
-function firstGraphMediaCall() {
-  const [call] = vi.mocked(downloadMSTeamsGraphMedia).mock.calls;
-  if (!call) {
-    throw new Error("expected Graph media download call");
-  }
-  return call[0];
-}
-
-function firstBotFrameworkAttachmentCall() {
-  const [call] = vi.mocked(downloadMSTeamsBotFrameworkAttachments).mock.calls;
-  if (!call) {
-    throw new Error("expected Bot Framework attachment download call");
-  }
-  return call[0];
-}
-
-describe("resolveMSTeamsInboundMedia graph fallback trigger", () => {
-  it("triggers Graph fallback when HTML contains <attachment> tags", async () => {
-    vi.mocked(downloadMSTeamsAttachments).mockResolvedValue([]);
-    vi.mocked(extractMSTeamsHtmlAttachmentIds).mockReturnValueOnce(["att-0"]);
-    vi.mocked(downloadMSTeamsGraphMedia).mockResolvedValue({
-      media: [{ path: "/tmp/img.png", contentType: "image/png", placeholder: "[image]" }],
-    });
-
-    await resolveMSTeamsInboundMedia({
-      ...baseParams,
-      attachments: [
-        {
-          contentType: "text/html",
-          content: '<div>A file <attachment id="att-0"></attachment></div>',
-        },
-      ],
-    });
-
-    expect(buildMSTeamsGraphMessageUrls).toHaveBeenCalled();
-    expect(downloadMSTeamsGraphMedia).toHaveBeenCalled();
-  });
-
-  it("does NOT trigger Graph fallback for mention-only HTML (no <attachment> tags)", async () => {
-    vi.mocked(downloadMSTeamsAttachments).mockResolvedValue([]);
-    // Mention cards include `<at>` markers but no `<attachment id="...">`,
-    // so the extractor returns an empty ID list. The fallback must skip.
-    vi.mocked(extractMSTeamsHtmlAttachmentIds).mockReturnValueOnce([]);
-    vi.mocked(downloadMSTeamsGraphMedia).mockClear();
-    vi.mocked(buildMSTeamsGraphMessageUrls).mockClear();
-
-    await resolveMSTeamsInboundMedia({
-      ...baseParams,
-      attachments: [
-        {
-          contentType: "text/html",
-          content: '<div><at id="0">Bot</at> hello there</div>',
-        },
-      ],
-    });
-
-    expect(downloadMSTeamsGraphMedia).not.toHaveBeenCalled();
-    expect(buildMSTeamsGraphMessageUrls).not.toHaveBeenCalled();
-  });
-
-  it("does NOT trigger Graph fallback when no attachments are text/html", async () => {
-    vi.mocked(downloadMSTeamsAttachments).mockResolvedValue([]);
-    // No HTML attachments at all → extractor returns [].
-    vi.mocked(extractMSTeamsHtmlAttachmentIds).mockReturnValueOnce([]);
-    vi.mocked(downloadMSTeamsGraphMedia).mockClear();
-    vi.mocked(buildMSTeamsGraphMessageUrls).mockClear();
-
-    await resolveMSTeamsInboundMedia({
-      ...baseParams,
-      attachments: [
-        { contentType: "image/png", contentUrl: "https://example.com/img.png" },
-        { contentType: "application/pdf", contentUrl: "https://example.com/doc.pdf" },
-      ],
-    });
-
-    expect(downloadMSTeamsGraphMedia).not.toHaveBeenCalled();
-  });
-
-  it("does NOT trigger Graph fallback when direct download succeeds", async () => {
-    vi.mocked(downloadMSTeamsAttachments).mockResolvedValue([
-      { path: "/tmp/img.png", contentType: "image/png", placeholder: "[image]" },
-    ]);
-    vi.mocked(downloadMSTeamsGraphMedia).mockClear();
-
-    await resolveMSTeamsInboundMedia({
-      ...baseParams,
-      attachments: [
-        {
-          contentType: "text/html",
-          content: '<div><attachment id="att-0"></attachment></div>',
-        },
-      ],
-    });
-
-    expect(downloadMSTeamsGraphMedia).not.toHaveBeenCalled();
-  });
-
-  it("forwards log through to downloadMSTeamsGraphMedia for diagnostics", async () => {
-    vi.mocked(downloadMSTeamsAttachments).mockResolvedValue([]);
-    vi.mocked(extractMSTeamsHtmlAttachmentIds).mockReturnValueOnce(["att-0"]);
-    vi.mocked(downloadMSTeamsGraphMedia).mockClear();
-    vi.mocked(downloadMSTeamsGraphMedia).mockResolvedValue({ media: [] });
-    const log = { debug: vi.fn() };
-
-    await resolveMSTeamsInboundMedia({
-      ...baseParams,
-      log,
-      attachments: [
-        {
-          contentType: "text/html",
-          content: '<div><attachment id="att-0"></attachment></div>',
-        },
-      ],
-    });
-
-    const call = firstGraphMediaCall();
-    // The monitor handler's logger is forwarded so graph.ts can report
-    // message fetch failures instead of swallowing them (#51749).
-    expect(call?.logger).toBe(log);
-    expect(log.debug).toHaveBeenCalledWith("graph media fetch empty", {
-      attempts: [
-        {
-          url: "https://graph.microsoft.com/v1.0/chats/c/messages/m",
-          hostedStatus: undefined,
-          attachmentStatus: undefined,
-          hostedCount: undefined,
-          attachmentCount: undefined,
-          tokenError: undefined,
-        },
-      ],
-      attachmentIdCount: 1,
-    });
-  });
-});
-
-describe("resolveMSTeamsInboundMedia bot framework DM routing", () => {
-  const dmParams = {
-    ...baseParams,
-    conversationType: "personal",
-    conversationId: "a:1dRsHCobZ1AxURzY05Dc",
-    serviceUrl: "https://smba.trafficmanager.net/amer/",
-  };
-
-  it("routes 'a:' conversation IDs through the Bot Framework attachment endpoint", async () => {
-    vi.mocked(downloadMSTeamsAttachments).mockResolvedValue([]);
-    vi.mocked(downloadMSTeamsBotFrameworkAttachments).mockClear();
-    vi.mocked(downloadMSTeamsBotFrameworkAttachments).mockResolvedValue({
-      media: [
-        {
-          path: "/tmp/report.pdf",
-          contentType: "application/pdf",
-          placeholder: "<media:document>",
-        },
-      ],
-      attachmentCount: 1,
-    });
-    vi.mocked(downloadMSTeamsGraphMedia).mockClear();
-
-    const mediaList = await resolveMSTeamsInboundMedia({
-      ...dmParams,
-      attachments: [
-        {
-          contentType: "text/html",
-          content: '<div>A file <attachment id="att-0"></attachment></div>',
-        },
-      ],
-    });
-
-    expect(downloadMSTeamsBotFrameworkAttachments).toHaveBeenCalledTimes(1);
-    const call = firstBotFrameworkAttachmentCall();
-    expect(call?.serviceUrl).toBe(dmParams.serviceUrl);
-    expect(call?.attachmentIds).toEqual(["att-0", "att-1"]);
-    expect(downloadMSTeamsGraphMedia).not.toHaveBeenCalled();
-    expect(mediaList).toHaveLength(1);
-    expect(mediaList[0].path).toBe("/tmp/report.pdf");
-  });
-
-  it("skips the Graph fallback entirely for 'a:' conversation IDs", async () => {
-    vi.mocked(downloadMSTeamsAttachments).mockResolvedValue([]);
-    vi.mocked(downloadMSTeamsBotFrameworkAttachments).mockClear();
-    vi.mocked(downloadMSTeamsBotFrameworkAttachments).mockResolvedValue({
-      media: [],
-      attachmentCount: 1,
-    });
-    vi.mocked(downloadMSTeamsGraphMedia).mockClear();
-    vi.mocked(buildMSTeamsGraphMessageUrls).mockClear();
-
-    await resolveMSTeamsInboundMedia({
-      ...dmParams,
-      attachments: [
-        {
-          contentType: "text/html",
-          content: '<div><attachment id="att-0"></attachment></div>',
-        },
-      ],
-    });
-
-    expect(downloadMSTeamsBotFrameworkAttachments).toHaveBeenCalled();
-    expect(buildMSTeamsGraphMessageUrls).not.toHaveBeenCalled();
-    expect(downloadMSTeamsGraphMedia).not.toHaveBeenCalled();
-  });
-
-  it("does NOT call the Bot Framework endpoint for Graph-compatible '19:' IDs", async () => {
-    vi.mocked(downloadMSTeamsAttachments).mockResolvedValue([]);
-    vi.mocked(downloadMSTeamsBotFrameworkAttachments).mockClear();
-    vi.mocked(downloadMSTeamsGraphMedia).mockResolvedValue({ media: [] });
-
-    await resolveMSTeamsInboundMedia({
-      ...baseParams,
-      conversationId: "19:abc@thread.tacv2",
-      serviceUrl: "https://smba.trafficmanager.net/amer/",
-      attachments: [
-        {
-          contentType: "text/html",
-          content: '<div><attachment id="att-0"></attachment></div>',
-        },
-      ],
-    });
-
-    expect(downloadMSTeamsBotFrameworkAttachments).not.toHaveBeenCalled();
-    expect(downloadMSTeamsGraphMedia).toHaveBeenCalled();
-  });
-
-  it("skips BF DM attachment fetch entirely when HTML has no <attachment> tags", async () => {
-    vi.mocked(downloadMSTeamsAttachments).mockResolvedValue([]);
-    vi.mocked(downloadMSTeamsBotFrameworkAttachments).mockClear();
-    vi.mocked(downloadMSTeamsGraphMedia).mockClear();
-    // Mention-only HTML (no `<attachment id="...">` tag) → extractor
-    // returns []. The fallback skips both the Bot Framework and Graph
-    // paths so we do not emit spurious 404 diagnostics (#58617).
-    vi.mocked(extractMSTeamsHtmlAttachmentIds).mockReturnValueOnce([]);
-
-    await resolveMSTeamsInboundMedia({
-      ...dmParams,
-      attachments: [
-        {
-          contentType: "text/html",
-          content: '<div><at id="0">Bot</at> hello</div>',
-        },
-      ],
-    });
-
-    expect(downloadMSTeamsBotFrameworkAttachments).not.toHaveBeenCalled();
-    expect(downloadMSTeamsGraphMedia).not.toHaveBeenCalled();
-  });
-
-  it("logs when serviceUrl is missing for a BF DM with HTML content", async () => {
-    vi.mocked(downloadMSTeamsAttachments).mockResolvedValue([]);
-    vi.mocked(downloadMSTeamsBotFrameworkAttachments).mockClear();
-    vi.mocked(downloadMSTeamsGraphMedia).mockClear();
-    vi.mocked(buildMSTeamsGraphMessageUrls).mockClear();
-    const log = { debug: vi.fn() };
-
-    await resolveMSTeamsInboundMedia({
-      ...baseParams,
-      log,
-      conversationType: "personal",
-      conversationId: "a:bf-dm-id",
-      attachments: [
-        {
-          contentType: "text/html",
-          content: '<div><attachment id="att-0"></attachment></div>',
-        },
-      ],
-    });
-
-    expect(downloadMSTeamsBotFrameworkAttachments).not.toHaveBeenCalled();
-    // Graph fallback is also skipped because the ID is 'a:'
-    expect(downloadMSTeamsGraphMedia).not.toHaveBeenCalled();
-    expect(log.debug).toHaveBeenCalledWith(
-      "bot framework attachment skipped (missing serviceUrl)",
-      {
-        conversationType: "personal",
-        conversationId: "a:bf-dm-id",
-      },
-    );
-  });
-});

package/src/monitor-handler/inbound-media.ts

@@ -1,180 +0,0 @@
-import {
-  buildMSTeamsGraphMessageUrls,
-  downloadMSTeamsAttachments,
-  downloadMSTeamsBotFrameworkAttachments,
-  downloadMSTeamsGraphMedia,
-  extractMSTeamsHtmlAttachmentIds,
-  isBotFrameworkPersonalChatId,
-  type MSTeamsAccessTokenProvider,
-  type MSTeamsAttachmentLike,
-  type MSTeamsHtmlAttachmentSummary,
-  type MSTeamsInboundMedia,
-} from "../attachments.js";
-import type { MSTeamsTurnContext } from "../sdk-types.js";
-
-type MSTeamsLogger = {
-  debug?: (message: string, meta?: Record<string, unknown>) => void;
-  warn?: (message: string, meta?: Record<string, unknown>) => void;
-  error?: (message: string, meta?: Record<string, unknown>) => void;
-};
-
-export async function resolveMSTeamsInboundMedia(params: {
-  attachments: MSTeamsAttachmentLike[];
-  htmlSummary?: MSTeamsHtmlAttachmentSummary;
-  maxBytes: number;
-  allowHosts?: string[];
-  authAllowHosts?: string[];
-  tokenProvider: MSTeamsAccessTokenProvider;
-  conversationType: string;
-  conversationId: string;
-  conversationMessageId?: string;
-  serviceUrl?: string;
-  activity: Pick<MSTeamsTurnContext["activity"], "id" | "replyToId" | "channelData">;
-  log: MSTeamsLogger;
-  /** When true, embeds original filename in stored path for later extraction. */
-  preserveFilenames?: boolean;
-}): Promise<MSTeamsInboundMedia[]> {
-  const {
-    attachments,
-    htmlSummary,
-    maxBytes,
-    tokenProvider,
-    allowHosts,
-    conversationType,
-    conversationId,
-    conversationMessageId,
-    serviceUrl,
-    activity,
-    log,
-    preserveFilenames,
-  } = params;
-
-  let mediaList = await downloadMSTeamsAttachments({
-    attachments,
-    maxBytes,
-    tokenProvider,
-    allowHosts,
-    authAllowHosts: params.authAllowHosts,
-    preserveFilenames,
-    logger: log,
-  });
-
-  if (mediaList.length === 0) {
-    // Gate the Graph/Bot Framework media fallback on the presence of real
-    // `<attachment id="...">` tags inside any `text/html` attachment. Teams
-    // delivers @mention cards and other chrome as `text/html` attachments
-    // too, so keying off contentType alone produces spurious 404 diagnostics
-    // for every mention-only message and masks real file attachments (#58617).
-    const attachmentIds = extractMSTeamsHtmlAttachmentIds(attachments);
-    const hasHtmlFileAttachment = attachmentIds.length > 0;
-
-    // Personal DMs with the bot use Bot Framework conversation IDs (`a:...`
-    // or `8:orgid:...`) which Graph's `/chats/{id}` endpoint rejects with
-    // "Invalid ThreadId". Fetch media via the Bot Framework v3 attachments
-    // endpoint instead, which speaks the same identifier space.
-    if (hasHtmlFileAttachment && isBotFrameworkPersonalChatId(conversationId)) {
-      if (!serviceUrl) {
-        log.debug?.("bot framework attachment skipped (missing serviceUrl)", {
-          conversationType,
-          conversationId,
-        });
-      } else {
-        const bfMedia = await downloadMSTeamsBotFrameworkAttachments({
-          serviceUrl,
-          attachmentIds,
-          tokenProvider,
-          maxBytes,
-          allowHosts,
-          authAllowHosts: params.authAllowHosts,
-          preserveFilenames,
-        });
-        if (bfMedia.media.length > 0) {
-          mediaList = bfMedia.media;
-        } else {
-          log.debug?.("bot framework attachments fetch empty", {
-            conversationType,
-            attachmentCount: bfMedia.attachmentCount ?? attachmentIds.length,
-          });
-        }
-      }
-    }
-
-    if (
-      hasHtmlFileAttachment &&
-      mediaList.length === 0 &&
-      !isBotFrameworkPersonalChatId(conversationId)
-    ) {
-      const messageUrls = buildMSTeamsGraphMessageUrls({
-        conversationType,
-        conversationId,
-        messageId: activity.id ?? undefined,
-        replyToId: activity.replyToId ?? undefined,
-        conversationMessageId,
-        channelData: activity.channelData,
-      });
-      if (messageUrls.length === 0) {
-        log.debug?.("graph message url unavailable", {
-          conversationType,
-          hasChannelData: Boolean(activity.channelData),
-          messageId: activity.id ?? undefined,
-          replyToId: activity.replyToId ?? undefined,
-        });
-      } else {
-        const attempts: Array<{
-          url: string;
-          hostedStatus?: number;
-          attachmentStatus?: number;
-          hostedCount?: number;
-          attachmentCount?: number;
-          tokenError?: boolean;
-        }> = [];
-        for (const messageUrl of messageUrls) {
-          const graphMedia = await downloadMSTeamsGraphMedia({
-            messageUrl,
-            tokenProvider,
-            maxBytes,
-            allowHosts,
-            authAllowHosts: params.authAllowHosts,
-            preserveFilenames,
-            log,
-            logger: log,
-          });
-          attempts.push({
-            url: messageUrl,
-            hostedStatus: graphMedia.hostedStatus,
-            attachmentStatus: graphMedia.attachmentStatus,
-            hostedCount: graphMedia.hostedCount,
-            attachmentCount: graphMedia.attachmentCount,
-            tokenError: graphMedia.tokenError,
-          });
-          if (graphMedia.media.length > 0) {
-            mediaList = graphMedia.media;
-            break;
-          }
-          if (graphMedia.tokenError) {
-            break;
-          }
-        }
-        if (mediaList.length === 0) {
-          log.debug?.("graph media fetch empty", {
-            attempts,
-            attachmentIdCount: attachmentIds.length,
-          });
-        }
-      }
-    }
-  }
-
-  if (mediaList.length > 0) {
-    log.debug?.("downloaded attachments", { count: mediaList.length });
-  } else if (htmlSummary?.imgTags) {
-    log.debug?.("inline images detected but none downloaded", {
-      imgTags: htmlSummary.imgTags,
-      srcHosts: htmlSummary.srcHosts,
-      dataImages: htmlSummary.dataImages,
-      cidImages: htmlSummary.cidImages,
-    });
-  }
-
-  return mediaList;
-}

package/src/monitor-handler/message-handler-mock-support.test-support.ts

@@ -1,28 +0,0 @@
-import { vi } from "vitest";
-
-const runtimeApiMockState = vi.hoisted(() => ({
-  dispatchReplyFromConfigWithSettledDispatcher: vi.fn(async (params: { ctxPayload: unknown }) => ({
-    queuedFinal: false,
-    counts: {},
-    capturedCtxPayload: params.ctxPayload,
-  })),
-}));
-
-export function getRuntimeApiMockState() {
-  return runtimeApiMockState;
-}
-
-vi.mock("klaw/plugin-sdk/inbound-reply-dispatch", () => {
-  return {
-    dispatchReplyFromConfigWithSettledDispatcher:
-      runtimeApiMockState.dispatchReplyFromConfigWithSettledDispatcher,
-  };
-});
-
-vi.mock("../reply-dispatcher.js", () => ({
-  createMSTeamsReplyDispatcher: () => ({
-    dispatcher: {},
-    replyOptions: {},
-    markDispatchIdle: vi.fn(),
-  }),
-}));