@yanhaidao/wecom 2.4.160 → 2.5.110

package/src/agent/api-client.upload.test.ts

@@ -1,109 +0,0 @@
-import { beforeEach, describe, expect, it, vi } from "vitest";
-import type { ResolvedAgentAccount } from "../types/index.js";
-
-const { wecomFetchMock, resolveProxyMock } = vi.hoisted(() => ({
-  wecomFetchMock: vi.fn(),
-  resolveProxyMock: vi.fn(() => undefined),
-}));
-
-vi.mock("../http.js", () => ({
-  wecomFetch: wecomFetchMock,
-  readResponseBodyAsBuffer: vi.fn(),
-}));
-
-vi.mock("../config/index.js", () => ({
-  resolveWecomEgressProxyUrlFromNetwork: resolveProxyMock,
-}));
-
-import { uploadMedia } from "../transport/agent-api/core.js";
-
-function createAgent(agentId: number): ResolvedAgentAccount {
-  return {
-    accountId: `acct-${agentId}`,
-    configured: true,
-    corpId: "corp",
-    corpSecret: "secret",
-    agentId,
-    token: "token",
-    encodingAESKey: "aes",
-    config: {} as any,
-  };
-}
-
-function jsonResponse(body: unknown): Response {
-  return new Response(JSON.stringify(body), {
-    status: 200,
-    headers: { "Content-Type": "application/json" },
-  });
-}
-
-describe("wecom agent uploadMedia", () => {
-  beforeEach(() => {
-    wecomFetchMock.mockReset();
-    resolveProxyMock.mockReset();
-    resolveProxyMock.mockReturnValue(undefined);
-  });
-
-  it("uses text/plain for .txt uploads", async () => {
-    wecomFetchMock.mockResolvedValueOnce(jsonResponse({ access_token: "token-1", expires_in: 7200 }));
-    wecomFetchMock.mockResolvedValueOnce(jsonResponse({ errcode: 0, errmsg: "ok", media_id: "m-1" }));
-
-    const mediaId = await uploadMedia({
-      agent: createAgent(10001),
-      type: "file",
-      buffer: Buffer.from("hello txt"),
-      filename: "note.txt",
-    });
-
-    expect(mediaId).toBe("m-1");
-    const [, init] = wecomFetchMock.mock.calls[1] as [string, RequestInit];
-    const body = init.body as Buffer;
-    const bodyText = body.toString("utf8");
-    expect(bodyText).toContain('filename="note.txt"');
-    expect(bodyText).toContain("Content-Type: text/plain");
-  });
-
-  it("uses docx mime and normalizes non-ascii filename", async () => {
-    wecomFetchMock.mockResolvedValueOnce(jsonResponse({ access_token: "token-2", expires_in: 7200 }));
-    wecomFetchMock.mockResolvedValueOnce(jsonResponse({ errcode: 0, errmsg: "ok", media_id: "m-2" }));
-
-    const mediaId = await uploadMedia({
-      agent: createAgent(10002),
-      type: "file",
-      buffer: Buffer.from("docx bytes"),
-      filename: "需求文档.docx",
-    });
-
-    expect(mediaId).toBe("m-2");
-    const [, init] = wecomFetchMock.mock.calls[1] as [string, RequestInit];
-    const body = init.body as Buffer;
-    const bodyText = body.toString("utf8");
-    expect(bodyText).toContain('filename="file.docx"');
-    expect(bodyText).toContain(
-      "Content-Type: application/vnd.openxmlformats-officedocument.wordprocessingml.document",
-    );
-  });
-
-  it("retries with octet-stream when preferred mime upload fails", async () => {
-    wecomFetchMock.mockResolvedValueOnce(jsonResponse({ access_token: "token-3", expires_in: 7200 }));
-    wecomFetchMock.mockResolvedValueOnce(jsonResponse({ errcode: 40005, errmsg: "invalid media type" }));
-    wecomFetchMock.mockResolvedValueOnce(jsonResponse({ errcode: 0, errmsg: "ok", media_id: "m-3" }));
-
-    const mediaId = await uploadMedia({
-      agent: createAgent(10003),
-      type: "file",
-      buffer: Buffer.from("yaml bytes"),
-      filename: "config.yaml",
-    });
-
-    expect(mediaId).toBe("m-3");
-    expect(wecomFetchMock).toHaveBeenCalledTimes(3);
-
-    const [, firstUploadInit] = wecomFetchMock.mock.calls[1] as [string, RequestInit];
-    const [, retryUploadInit] = wecomFetchMock.mock.calls[2] as [string, RequestInit];
-    const firstUploadBody = (firstUploadInit.body as Buffer).toString("utf8");
-    const retryUploadBody = (retryUploadInit.body as Buffer).toString("utf8");
-    expect(firstUploadBody).toContain("Content-Type: application/yaml");
-    expect(retryUploadBody).toContain("Content-Type: application/octet-stream");
-  });
-});

package/src/agent/handler.event-filter.test.ts

@@ -1,100 +0,0 @@
-import { describe, expect, it } from "vitest";
-
-import { shouldProcessAgentInboundMessage, shouldSuppressAgentReplyText } from "./handler.js";
-
-describe("shouldProcessAgentInboundMessage", () => {
-    it("allows enter_agent/subscribe through the filter (handled earlier by static welcome)", () => {
-        const enterAgent = shouldProcessAgentInboundMessage({
-            msgType: "event",
-            eventType: "enter_agent",
-            fromUser: "zhangsan",
-        });
-        expect(enterAgent.shouldProcess).toBe(true);
-        expect(enterAgent.reason).toBe("allowed_event:enter_agent");
-
-        const subscribe = shouldProcessAgentInboundMessage({
-            msgType: "event",
-            eventType: "subscribe",
-            fromUser: "lisi",
-        });
-        expect(subscribe.shouldProcess).toBe(true);
-        expect(subscribe.reason).toBe("allowed_event:subscribe");
-    });
-
-    it("skips unknown event callbacks so they do not create sessions", () => {
-        const unknown = shouldProcessAgentInboundMessage({
-            msgType: "event",
-            eventType: "some_random_event",
-            fromUser: "zhangsan",
-        });
-        expect(unknown.shouldProcess).toBe(false);
-        expect(unknown.reason).toBe("event:some_random_event");
-    });
-
-    it("skips system sender callbacks", () => {
-        const systemSender = shouldProcessAgentInboundMessage({
-            msgType: "text",
-            fromUser: "sys",
-        });
-        expect(systemSender.shouldProcess).toBe(false);
-        expect(systemSender.reason).toBe("system_sender");
-    });
-
-    it("skips messages with missing sender id", () => {
-        const missingSender = shouldProcessAgentInboundMessage({
-            msgType: "text",
-            fromUser: "   ",
-        });
-        expect(missingSender.shouldProcess).toBe(false);
-        expect(missingSender.reason).toBe("missing_sender");
-    });
-
-
-    it("allows group chat messages when sender id is missing", () => {
-        const groupWithoutSender = shouldProcessAgentInboundMessage({
-            msgType: "file",
-            fromUser: "   ",
-            chatId: "wrbchat_123",
-        });
-        expect(groupWithoutSender.shouldProcess).toBe(true);
-        expect(groupWithoutSender.reason).toBe("missing_sender_but_group_chat");
-    });
-
-    it("allows normal user text message processing", () => {
-        const normalMessage = shouldProcessAgentInboundMessage({
-            msgType: "text",
-            fromUser: "wangwu",
-        });
-        expect(normalMessage.shouldProcess).toBe(true);
-        expect(normalMessage.reason).toBe("user_message");
-    });
-});
-
-describe("shouldSuppressAgentReplyText", () => {
-    it("keeps plain text replies when no media reply has been seen", () => {
-        expect(
-            shouldSuppressAgentReplyText({
-                text: "这里是正常文本",
-                mediaReplySeen: false,
-            }),
-        ).toBe(false);
-    });
-
-    it("suppresses companion text once the reply flow includes media", () => {
-        expect(
-            shouldSuppressAgentReplyText({
-                text: "文件已发送，请查收",
-                mediaReplySeen: true,
-            }),
-        ).toBe(true);
-    });
-
-    it("does not suppress empty text even after media replies", () => {
-        expect(
-            shouldSuppressAgentReplyText({
-                text: "   ",
-                mediaReplySeen: true,
-            }),
-        ).toBe(false);
-    });
-});