@yanhaidao/wecom 2.4.120 → 2.5.110

package/scripts/wecom/menu-click-help.js

@@ -1,59 +0,0 @@
-#!/usr/bin/env node
-
-// 从 stdin 读取平台注入的事件 envelope（JSON）
-let raw = "";
-process.stdin.setEncoding("utf8");
-
-process.stdin.on("data", (chunk) => {
-  raw += chunk;
-});
-
-process.stdin.on("end", () => {
-  let payload = {};
-  try {
-    payload = JSON.parse(raw || "{}");
-  } catch {
-    // 输入异常时返回可读错误，避免脚本抛异常导致无输出
-    process.stdout.write(
-      JSON.stringify({
-        ok: false,
-        action: "reply_text",
-        reply: {
-          text: "事件脚本解析输入失败，请联系管理员。",
-        },
-        chainToAgent: true,
-      }),
-    );
-    return;
-  }
-
-  const message = payload && typeof payload === "object" ? payload.message ?? {} : {};
-  const eventType = String(message.eventType ?? "unknown");
-  const eventKey = String(message.eventKey ?? "");
-  const fromUser = String(message.fromUser ?? "");
-
-  // 这里给出联调用的回显文本，业务可改为真正逻辑
-  const text = [
-    "已收到菜单事件",
-    `eventType=${eventType}`,
-    eventKey ? `eventKey=${eventKey}` : "eventKey=<empty>",
-    fromUser ? `fromUser=${fromUser}` : "",
-    "\n你可以在 openclaw 配置里把这个 eventKey 路由到具体业务脚本。",
-  ]
-    .filter(Boolean)
-    .join("\n");
-
-  process.stdout.write(
-    JSON.stringify({
-      ok: true,
-      action: "reply_text",
-      reply: {
-        text,
-      },
-      chainToAgent: false,
-      audit: {
-        tags: ["menu", "click", eventType],
-      },
-    }),
-  );
-});

package/scripts/wecom/menu-click-help.py

@@ -1,55 +0,0 @@
-#!/usr/bin/env python3
-import json
-import sys
-
-
-def main() -> int:
-    # 从 stdin 读取平台注入的事件 envelope（JSON）
-    try:
-        payload = json.load(sys.stdin)
-    except Exception:
-        # 输入异常时返回可读错误，避免脚本抛异常导致无输出
-        json.dump(
-            {
-                "ok": False,
-                "action": "reply_text",
-                "reply": {"text": "事件脚本解析输入失败，请联系管理员。"},
-                "chainToAgent": True,
-            },
-            sys.stdout,
-            ensure_ascii=False,
-        )
-        return 0
-
-    message = payload.get("message", {}) if isinstance(payload, dict) else {}
-    event_type = str(message.get("eventType") or "unknown")
-    event_key = str(message.get("eventKey") or "")
-    from_user = str(message.get("fromUser") or "")
-
-    # 这里给出联调用的回显文本，业务可改为真正逻辑
-    lines = [
-        "已收到菜单事件",
-        f"eventType={event_type}",
-        f"eventKey={event_key or '<empty>'}",
-    ]
-    if from_user:
-        lines.append(f"fromUser={from_user}")
-    lines.append("")
-    lines.append("你可以在 openclaw 配置里把这个 eventKey 路由到具体业务脚本。")
-
-    json.dump(
-        {
-            "ok": True,
-            "action": "reply_text",
-            "reply": {"text": "\n".join(lines)},
-            "chainToAgent": False,
-            "audit": {"tags": ["menu", "click", event_type]},
-        },
-        sys.stdout,
-        ensure_ascii=False,
-    )
-    return 0
-
-
-if __name__ == "__main__":
-    raise SystemExit(main())

package/src/accounts.ts

@@ -1,34 +0,0 @@
-import type { OpenClawConfig } from "openclaw/plugin-sdk";
-
-import type { ResolvedWecomAccount } from "./types/index.js";
-import {
-  listWecomAccountIds as listWecomAccountIdsFromConfig,
-  resolveDefaultWecomAccountId as resolveDefaultWecomAccountIdFromConfig,
-  resolveWecomAccount as resolveWecomAccountFromConfig,
-} from "./config/accounts.js";
-
-/**
- * Backward-compatible re-export layer.
- * Keep this file as a thin wrapper so older imports continue to work,
- * while all account logic stays single-sourced in `src/config/accounts.ts`.
- */
-export function listWecomAccountIds(cfg: OpenClawConfig): string[] {
-  return listWecomAccountIdsFromConfig(cfg);
-}
-
-export function resolveDefaultWecomAccountId(cfg: OpenClawConfig): string {
-  return resolveDefaultWecomAccountIdFromConfig(cfg);
-}
-
-export function resolveWecomAccount(params: {
-  cfg: OpenClawConfig;
-  accountId?: string | null;
-}): ResolvedWecomAccount {
-  return resolveWecomAccountFromConfig(params);
-}
-
-export function listEnabledWecomAccounts(cfg: OpenClawConfig): ResolvedWecomAccount[] {
-  return listWecomAccountIdsFromConfig(cfg)
-    .map((accountId) => resolveWecomAccountFromConfig({ cfg, accountId }))
-    .filter((account) => account.enabled);
-}

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/event-router.test.ts

@@ -1,421 +0,0 @@
-import path from "node:path";
-import { describe, expect, it } from "vitest";
-
-import { routeAgentInboundEvent } from "./event-router.js";
-import type { ResolvedAgentAccount, WecomAgentInboundMessage } from "../types/index.js";
-import type { WecomRuntimeAuditEvent } from "../types/runtime-context.js";
-
-function createAgent(overrides?: Partial<ResolvedAgentAccount>): ResolvedAgentAccount {
-  return {
-    accountId: "default",
-    configured: true,
-    callbackConfigured: true,
-    apiConfigured: true,
-    corpId: "corp-1",
-    corpSecret: "secret",
-    agentId: 1001,
-    token: "token",
-    encodingAESKey: "aes",
-    eventEnabled: true,
-    allowedEventTypes: ["click", "change_contact"],
-    config: {
-      corpId: "corp-1",
-      corpSecret: "secret",
-      agentId: 1001,
-      token: "token",
-      encodingAESKey: "aes",
-    },
-    ...overrides,
-  };
-}
-
-describe("routeAgentInboundEvent", () => {
-  it("ignores unmatched events when unmatchedAction is ignore", async () => {
-    const agent = createAgent({
-      config: {
-        corpId: "corp-1",
-        corpSecret: "secret",
-        agentId: 1001,
-        token: "token",
-        encodingAESKey: "aes",
-        eventRouting: {
-          unmatchedAction: "ignore",
-          routes: [],
-        },
-      },
-    });
-
-    const result = await routeAgentInboundEvent({
-      agent,
-      msgType: "event",
-      eventType: "click",
-      fromUser: "zhangsan",
-      msg: { MsgType: "event", Event: "click", EventKey: "MENU_X" },
-    });
-
-    expect(result.handled).toBe(true);
-    expect(result.chainToAgent).toBe(false);
-    expect(result.reason).toBe("unmatched_event_ignored");
-  });
-
-  it("proxies unmatched events to agent when unmatchedAction is forwardToAgent", async () => {
-    const agent = createAgent({
-      config: {
-        corpId: "corp-1",
-        corpSecret: "secret",
-        agentId: 1001,
-        token: "token",
-        encodingAESKey: "aes",
-        eventRouting: {
-          unmatchedAction: "forwardToAgent",
-          routes: [],
-        },
-      },
-    });
-
-    const result = await routeAgentInboundEvent({
-      agent,
-      msgType: "event",
-      eventType: "click",
-      fromUser: "zhangsan",
-      msg: { MsgType: "event", Event: "click", EventKey: "MENU_X" },
-    });
-
-    expect(result.handled).toBe(false);
-    expect(result.chainToAgent).toBe(true);
-  });
-
-  it("matches builtin echo routes using eventKey", async () => {
-    const agent = createAgent({
-      config: {
-        corpId: "corp-1",
-        corpSecret: "secret",
-        agentId: 1001,
-        token: "token",
-        encodingAESKey: "aes",
-        eventRouting: {
-          routes: [
-            {
-              id: "menu-help",
-              when: { eventType: "click", eventKey: "MENU_HELP" },
-              handler: { type: "builtin", name: "echo" },
-            },
-          ],
-        },
-      },
-    });
-
-    const result = await routeAgentInboundEvent({
-      agent,
-      msgType: "event",
-      eventType: "click",
-      fromUser: "zhangsan",
-      msg: { MsgType: "event", Event: "click", EventKey: "MENU_HELP" },
-    });
-
-    expect(result.handled).toBe(true);
-    expect(result.replyText).toContain("event=click");
-    expect(result.replyText).toContain("eventKey=MENU_HELP");
-    expect(result.matchedRouteId).toBe("menu-help");
-  });
-
-  it("matches change_contact routes using changeType", async () => {
-    const agent = createAgent({
-      config: {
-        corpId: "corp-1",
-        corpSecret: "secret",
-        agentId: 1001,
-        token: "token",
-        encodingAESKey: "aes",
-        eventRouting: {
-          routes: [
-            {
-              id: "contact-create-user",
-              when: { eventType: "change_contact", changeType: "create_user" },
-              handler: { type: "builtin", name: "echo" },
-            },
-          ],
-        },
-      },
-    });
-
-    const msg: WecomAgentInboundMessage = {
-      MsgType: "event",
-      Event: "change_contact",
-      ChangeType: "create_user",
-    };
-    const result = await routeAgentInboundEvent({
-      agent,
-      msgType: "event",
-      eventType: "change_contact",
-      fromUser: "sys",
-      msg,
-    });
-
-    expect(result.handled).toBe(true);
-    expect(result.replyText).toContain("changeType=create_user");
-  });
-
-  it("passes full event params to node scripts", async () => {
-    const fixturePath = path.resolve("src/agent/test-fixtures/reply-event-script.mjs");
-    const agent = createAgent({
-      config: {
-        corpId: "corp-1",
-        corpSecret: "secret",
-        agentId: 1001,
-        token: "token",
-        encodingAESKey: "aes",
-        eventRouting: {
-          routes: [
-            {
-              id: "script-click",
-              when: { eventType: "click", eventKeyPrefix: "MENU_" },
-              handler: { type: "node_script", entry: fixturePath },
-            },
-          ],
-        },
-        scriptRuntime: {
-          enabled: true,
-          allowPaths: [path.resolve("src/agent/test-fixtures")],
-          nodeCommand: process.execPath,
-        },
-      },
-    });
-
-    const result = await routeAgentInboundEvent({
-      agent,
-      msgType: "event",
-      eventType: "click",
-      fromUser: "zhangsan",
-      msg: {
-        ToUserName: "corp-1",
-        FromUserName: "zhangsan",
-        MsgType: "event",
-        Event: "click",
-        EventKey: "MENU_HELP",
-        AgentID: 1001,
-      },
-    });
-
-    expect(result.handled).toBe(true);
-    expect(result.replyText).toBe("script:click:MENU_HELP:");
-  });
-
-  it("supports scripts that explicitly continue the default pipeline", async () => {
-    const fixturePath = path.resolve("src/agent/test-fixtures/reply-event-script.mjs");
-    const agent = createAgent({
-      config: {
-        corpId: "corp-1",
-        corpSecret: "secret",
-        agentId: 1001,
-        token: "token",
-        encodingAESKey: "aes",
-        eventRouting: {
-          routes: [
-            {
-              id: "script-click-pass",
-              when: { eventType: "click", eventKey: "PASS_TO_DEFAULT" },
-              handler: { type: "node_script", entry: fixturePath },
-            },
-          ],
-        },
-        scriptRuntime: {
-          enabled: true,
-          allowPaths: [path.resolve("src/agent/test-fixtures")],
-          nodeCommand: process.execPath,
-        },
-      },
-    });
-
-    const result = await routeAgentInboundEvent({
-      agent,
-      msgType: "event",
-      eventType: "click",
-      fromUser: "zhangsan",
-      msg: {
-        MsgType: "event",
-        Event: "click",
-        EventKey: "PASS_TO_DEFAULT",
-      },
-    });
-
-    expect(result.handled).toBe(true);
-    expect(result.chainToAgent).toBe(true);
-    expect(result.replyText).toBeUndefined();
-  });
-
-  it("passes full event params to python scripts", async () => {
-    const fixturePath = path.resolve("src/agent/test-fixtures/reply-event-script.py");
-    const agent = createAgent({
-      config: {
-        corpId: "corp-1",
-        corpSecret: "secret",
-        agentId: 1001,
-        token: "token",
-        encodingAESKey: "aes",
-        eventRouting: {
-          routes: [
-            {
-              id: "script-python-click",
-              when: { eventType: "click", eventKey: "MENU_PY" },
-              handler: { type: "python_script", entry: fixturePath },
-            },
-          ],
-        },
-        scriptRuntime: {
-          enabled: true,
-          allowPaths: [path.resolve("src/agent/test-fixtures")],
-          pythonCommand: "python3",
-        },
-      },
-    });
-
-    const result = await routeAgentInboundEvent({
-      agent,
-      msgType: "event",
-      eventType: "click",
-      fromUser: "zhangsan",
-      msg: {
-        MsgType: "event",
-        Event: "click",
-        EventKey: "MENU_PY",
-      },
-    });
-
-    expect(result.handled).toBe(true);
-    expect(result.replyText).toBe("python:click:MENU_PY:");
-  });
-
-  it("records audit events for successful script execution", async () => {
-    const fixturePath = path.resolve("src/agent/test-fixtures/reply-event-script.mjs");
-    const auditEvents: WecomRuntimeAuditEvent[] = [];
-    const agent = createAgent({
-      config: {
-        corpId: "corp-1",
-        corpSecret: "secret",
-        agentId: 1001,
-        token: "token",
-        encodingAESKey: "aes",
-        eventRouting: {
-          routes: [
-            {
-              id: "script-audit-success",
-              when: { eventType: "click", eventKey: "MENU_AUDIT" },
-              handler: { type: "node_script", entry: fixturePath },
-            },
-          ],
-        },
-        scriptRuntime: {
-          enabled: true,
-          allowPaths: [path.resolve("src/agent/test-fixtures")],
-          nodeCommand: process.execPath,
-        },
-      },
-    });
-
-    await routeAgentInboundEvent({
-      agent,
-      msgType: "event",
-      eventType: "click",
-      fromUser: "zhangsan",
-      msg: {
-        MsgType: "event",
-        Event: "click",
-        EventKey: "MENU_AUDIT",
-      },
-      auditSink: (event) => auditEvents.push(event),
-    });
-
-    expect(auditEvents).toHaveLength(1);
-    expect(auditEvents[0]?.category).toBe("inbound");
-    expect(auditEvents[0]?.summary).toContain("event route script ok");
-  });
-
-  it("captures invalid script output as a routed error and audits it", async () => {
-    const fixturePath = path.resolve("src/agent/test-fixtures/invalid-json-script.mjs");
-    const auditEvents: WecomRuntimeAuditEvent[] = [];
-    const agent = createAgent({
-      config: {
-        corpId: "corp-1",
-        corpSecret: "secret",
-        agentId: 1001,
-        token: "token",
-        encodingAESKey: "aes",
-        eventRouting: {
-          routes: [
-            {
-              id: "script-invalid-json",
-              when: { eventType: "click", eventKey: "MENU_BAD_JSON" },
-              handler: { type: "node_script", entry: fixturePath },
-            },
-          ],
-        },
-        scriptRuntime: {
-          enabled: true,
-          allowPaths: [path.resolve("src/agent/test-fixtures")],
-          nodeCommand: process.execPath,
-        },
-      },
-    });
-
-    const result = await routeAgentInboundEvent({
-      agent,
-      msgType: "event",
-      eventType: "click",
-      fromUser: "zhangsan",
-      msg: {
-        MsgType: "event",
-        Event: "click",
-        EventKey: "MENU_BAD_JSON",
-      },
-      auditSink: (event) => auditEvents.push(event),
-    });
-
-    expect(result.handled).toBe(true);
-    expect(result.chainToAgent).toBe(false);
-    expect(result.reason).toBe("script_node_script_error");
-    expect(result.error).toContain("not valid JSON");
-    expect(auditEvents).toHaveLength(1);
-    expect(auditEvents[0]?.category).toBe("runtime-error");
-  });
-
-  it("treats invalid eventKeyPattern as non-match instead of throwing", async () => {
-    const auditEvents: WecomRuntimeAuditEvent[] = [];
-    const agent = createAgent({
-      config: {
-        corpId: "corp-1",
-        corpSecret: "secret",
-        agentId: 1001,
-        token: "token",
-        encodingAESKey: "aes",
-        eventRouting: {
-          unmatchedAction: "ignore",
-          routes: [
-            {
-              id: "invalid-pattern",
-              when: { eventType: "click", eventKeyPattern: "(*" },
-              handler: { type: "builtin", name: "echo" },
-            },
-          ],
-        },
-      },
-    });
-
-    const result = await routeAgentInboundEvent({
-      agent,
-      msgType: "event",
-      eventType: "click",
-      fromUser: "zhangsan",
-      msg: { MsgType: "event", Event: "click", EventKey: "MENU_X" },
-      auditSink: (event) => auditEvents.push(event),
-    });
-
-    expect(result.handled).toBe(true);
-    expect(result.chainToAgent).toBe(false);
-    expect(result.reason).toBe("unmatched_event_ignored");
-    expect(auditEvents).toHaveLength(1);
-    expect(auditEvents[0]?.category).toBe("runtime-error");
-    expect(auditEvents[0]?.summary).toContain("invalid route eventKeyPattern");
-    expect(auditEvents[0]?.error).toContain("routeId=invalid-pattern");
-  });
-});