@mocrane/wecom 2026.2.5 → 2026.3.4

package/src/monitor.inbound-filter.test.ts

@@ -0,0 +1,63 @@
+import { describe, expect, it } from "vitest";
+
+import { shouldProcessBotInboundMessage } from "./monitor.js";
+
+describe("shouldProcessBotInboundMessage", () => {
+  it("skips payloads without sender id", () => {
+    const result = shouldProcessBotInboundMessage({
+      msgtype: "text",
+      from: {},
+      text: { content: "hello" },
+    });
+    expect(result.shouldProcess).toBe(false);
+    expect(result.reason).toBe("missing_sender");
+  });
+
+  it("skips system sender payloads", () => {
+    const result = shouldProcessBotInboundMessage({
+      msgtype: "text",
+      from: { userid: "sys" },
+      text: { content: "hello" },
+    });
+    expect(result.shouldProcess).toBe(false);
+    expect(result.reason).toBe("system_sender");
+  });
+
+  it("skips group payloads without chatid", () => {
+    const result = shouldProcessBotInboundMessage({
+      msgtype: "text",
+      chattype: "group",
+      from: { userid: "zhangsan" },
+      text: { content: "hello" },
+    });
+    expect(result.shouldProcess).toBe(false);
+    expect(result.reason).toBe("missing_chatid");
+  });
+
+  it("accepts normal direct-user messages", () => {
+    const result = shouldProcessBotInboundMessage({
+      msgtype: "text",
+      chattype: "single",
+      from: { userid: "zhangsan" },
+      text: { content: "hello" },
+    });
+    expect(result.shouldProcess).toBe(true);
+    expect(result.reason).toBe("user_message");
+    expect(result.senderUserId).toBe("zhangsan");
+    expect(result.chatId).toBe("zhangsan");
+  });
+
+  it("accepts normal group messages with chatid", () => {
+    const result = shouldProcessBotInboundMessage({
+      msgtype: "text",
+      chattype: "group",
+      chatid: "wr123",
+      from: { userid: "zhangsan" },
+      text: { content: "hello" },
+    });
+    expect(result.shouldProcess).toBe(true);
+    expect(result.reason).toBe("user_message");
+    expect(result.senderUserId).toBe("zhangsan");
+    expect(result.chatId).toBe("wr123");
+  });
+});

package/src/monitor.integration.test.ts

@@ -21,7 +21,7 @@ function createMockRequest(bodyObj: any, query: URLSearchParams): IncomingMessag
     const socket = new Socket();
     const req = new IncomingMessage(socket);
     req.method = "POST";
-    req.url = `/wecom?${query.toString()}`;
+    req.url = `/plugins/wecom/bot/default?${query.toString()}`;
     req.push(JSON.stringify(bodyObj));
     req.push(null);
     return req;
@@ -108,7 +108,7 @@ describe("Monitor Integration: Inbound Image", () => {
             config: {} as any,
             runtime: { log: console.log, error: console.error },
             core: mockCore as any,
-            path: "/wecom"
+            path: "/plugins/wecom/bot/default"
         });
     });
 
@@ -198,6 +198,8 @@ describe("Monitor Integration: Inbound Image", () => {
         // Expect Context Injection
         expect(ctx.MediaPath).toBe("/tmp/saved-image.jpg");
         expect(ctx.MediaType).toBe("image/jpeg");
+        expect(ctx.Surface).toBe("wecom");
+        expect(ctx.OriginatingChannel).toBe("wecom");
 
         expect(undiciFetch).toHaveBeenCalledWith(
             imageUrl,