@openclaw/discord 2026.5.2 → 2026.5.3-beta.2

package/src/actions/handle-action.guild-admin.ts

@@ -1,411 +0,0 @@
-import type { AgentToolResult } from "@mariozechner/pi-agent-core";
-import {
-  readNumberParam,
-  readStringArrayParam,
-  readStringParam,
-} from "openclaw/plugin-sdk/agent-runtime";
-import type { ChannelMessageActionContext } from "openclaw/plugin-sdk/channel-contract";
-import { normalizeOptionalString } from "openclaw/plugin-sdk/text-runtime";
-import { handleDiscordAction } from "../../action-runtime-api.js";
-import {
-  isDiscordModerationAction,
-  readDiscordModerationCommand,
-} from "./runtime.moderation-shared.js";
-import {
-  readDiscordChannelCreateParams,
-  readDiscordChannelEditParams,
-  readDiscordChannelMoveParams,
-} from "./runtime.shared.js";
-
-type Ctx = Pick<
-  ChannelMessageActionContext,
-  "action" | "params" | "cfg" | "accountId" | "requesterSenderId" | "mediaLocalRoots"
->;
-
-export async function tryHandleDiscordMessageActionGuildAdmin(params: {
-  ctx: Ctx;
-  resolveChannelId: () => string;
-}): Promise<AgentToolResult<unknown> | undefined> {
-  const { ctx, resolveChannelId } = params;
-  const { action, params: actionParams, cfg } = ctx;
-  const accountId = ctx.accountId ?? readStringParam(actionParams, "accountId");
-
-  if (action === "member-info") {
-    const userId = readStringParam(actionParams, "userId", { required: true });
-    const guildId = readStringParam(actionParams, "guildId", {
-      required: true,
-    });
-    return await handleDiscordAction(
-      { action: "memberInfo", accountId: accountId ?? undefined, guildId, userId },
-      cfg,
-    );
-  }
-
-  if (action === "role-info") {
-    const guildId = readStringParam(actionParams, "guildId", {
-      required: true,
-    });
-    return await handleDiscordAction(
-      { action: "roleInfo", accountId: accountId ?? undefined, guildId },
-      cfg,
-    );
-  }
-
-  if (action === "emoji-list") {
-    const guildId = readStringParam(actionParams, "guildId", {
-      required: true,
-    });
-    return await handleDiscordAction(
-      { action: "emojiList", accountId: accountId ?? undefined, guildId },
-      cfg,
-    );
-  }
-
-  if (action === "emoji-upload") {
-    const guildId = readStringParam(actionParams, "guildId", {
-      required: true,
-    });
-    const name = readStringParam(actionParams, "emojiName", { required: true });
-    const mediaUrl = readStringParam(actionParams, "media", {
-      required: true,
-      trim: false,
-    });
-    const roleIds = readStringArrayParam(actionParams, "roleIds");
-    return await handleDiscordAction(
-      {
-        action: "emojiUpload",
-        accountId: accountId ?? undefined,
-        guildId,
-        name,
-        mediaUrl,
-        roleIds,
-      },
-      cfg,
-    );
-  }
-
-  if (action === "sticker-upload") {
-    const guildId = readStringParam(actionParams, "guildId", {
-      required: true,
-    });
-    const name = readStringParam(actionParams, "stickerName", {
-      required: true,
-    });
-    const description = readStringParam(actionParams, "stickerDesc", {
-      required: true,
-    });
-    const tags = readStringParam(actionParams, "stickerTags", {
-      required: true,
-    });
-    const mediaUrl = readStringParam(actionParams, "media", {
-      required: true,
-      trim: false,
-    });
-    return await handleDiscordAction(
-      {
-        action: "stickerUpload",
-        accountId: accountId ?? undefined,
-        guildId,
-        name,
-        description,
-        tags,
-        mediaUrl,
-      },
-      cfg,
-    );
-  }
-
-  if (action === "role-add" || action === "role-remove") {
-    const guildId = readStringParam(actionParams, "guildId", {
-      required: true,
-    });
-    const userId = readStringParam(actionParams, "userId", { required: true });
-    const roleId = readStringParam(actionParams, "roleId", { required: true });
-    return await handleDiscordAction(
-      {
-        action: action === "role-add" ? "roleAdd" : "roleRemove",
-        accountId: accountId ?? undefined,
-        guildId,
-        userId,
-        roleId,
-      },
-      cfg,
-    );
-  }
-
-  if (action === "channel-info") {
-    const channelId = readStringParam(actionParams, "channelId", {
-      required: true,
-    });
-    return await handleDiscordAction(
-      { action: "channelInfo", accountId: accountId ?? undefined, channelId },
-      cfg,
-    );
-  }
-
-  if (action === "channel-list") {
-    const guildId = readStringParam(actionParams, "guildId", {
-      required: true,
-    });
-    return await handleDiscordAction(
-      { action: "channelList", accountId: accountId ?? undefined, guildId },
-      cfg,
-    );
-  }
-
-  if (action === "channel-create") {
-    const guildId = readStringParam(actionParams, "guildId", {
-      required: true,
-    });
-    return await handleDiscordAction(
-      {
-        action: "channelCreate",
-        accountId: accountId ?? undefined,
-        ...readDiscordChannelCreateParams({ ...actionParams, guildId }),
-      },
-      cfg,
-    );
-  }
-
-  if (action === "channel-edit") {
-    const channelId = readStringParam(actionParams, "channelId", {
-      required: true,
-    });
-    return await handleDiscordAction(
-      {
-        action: "channelEdit",
-        accountId: accountId ?? undefined,
-        ...readDiscordChannelEditParams({ ...actionParams, channelId }),
-      },
-      cfg,
-    );
-  }
-
-  if (action === "channel-delete") {
-    const channelId = readStringParam(actionParams, "channelId", {
-      required: true,
-    });
-    return await handleDiscordAction(
-      { action: "channelDelete", accountId: accountId ?? undefined, channelId },
-      cfg,
-    );
-  }
-
-  if (action === "channel-move") {
-    const guildId = readStringParam(actionParams, "guildId", {
-      required: true,
-    });
-    const channelId = readStringParam(actionParams, "channelId", {
-      required: true,
-    });
-    return await handleDiscordAction(
-      {
-        action: "channelMove",
-        accountId: accountId ?? undefined,
-        ...readDiscordChannelMoveParams({ ...actionParams, guildId, channelId }),
-      },
-      cfg,
-    );
-  }
-
-  if (action === "category-create") {
-    const guildId = readStringParam(actionParams, "guildId", {
-      required: true,
-    });
-    const name = readStringParam(actionParams, "name", { required: true });
-    const position = readNumberParam(actionParams, "position", {
-      integer: true,
-    });
-    return await handleDiscordAction(
-      {
-        action: "categoryCreate",
-        accountId: accountId ?? undefined,
-        guildId,
-        name,
-        position: position ?? undefined,
-      },
-      cfg,
-    );
-  }
-
-  if (action === "category-edit") {
-    const categoryId = readStringParam(actionParams, "categoryId", {
-      required: true,
-    });
-    const name = readStringParam(actionParams, "name");
-    const position = readNumberParam(actionParams, "position", {
-      integer: true,
-    });
-    return await handleDiscordAction(
-      {
-        action: "categoryEdit",
-        accountId: accountId ?? undefined,
-        categoryId,
-        name: name ?? undefined,
-        position: position ?? undefined,
-      },
-      cfg,
-    );
-  }
-
-  if (action === "category-delete") {
-    const categoryId = readStringParam(actionParams, "categoryId", {
-      required: true,
-    });
-    return await handleDiscordAction(
-      { action: "categoryDelete", accountId: accountId ?? undefined, categoryId },
-      cfg,
-    );
-  }
-
-  if (action === "voice-status") {
-    const guildId = readStringParam(actionParams, "guildId", {
-      required: true,
-    });
-    const userId = readStringParam(actionParams, "userId", { required: true });
-    return await handleDiscordAction(
-      { action: "voiceStatus", accountId: accountId ?? undefined, guildId, userId },
-      cfg,
-    );
-  }
-
-  if (action === "event-list") {
-    const guildId = readStringParam(actionParams, "guildId", {
-      required: true,
-    });
-    return await handleDiscordAction(
-      { action: "eventList", accountId: accountId ?? undefined, guildId },
-      cfg,
-    );
-  }
-
-  if (action === "event-create") {
-    const guildId = readStringParam(actionParams, "guildId", {
-      required: true,
-    });
-    const name = readStringParam(actionParams, "eventName", { required: true });
-    const startTime = readStringParam(actionParams, "startTime", {
-      required: true,
-    });
-    const endTime = readStringParam(actionParams, "endTime");
-    const description = readStringParam(actionParams, "desc");
-    const channelId = readStringParam(actionParams, "channelId");
-    const location = readStringParam(actionParams, "location");
-    const entityType = readStringParam(actionParams, "eventType");
-    const image = readStringParam(actionParams, "image", { trim: false });
-    return await handleDiscordAction(
-      {
-        action: "eventCreate",
-        accountId: accountId ?? undefined,
-        guildId,
-        name,
-        startTime,
-        endTime,
-        description,
-        channelId,
-        location,
-        entityType,
-        image,
-      },
-      cfg,
-      { mediaLocalRoots: ctx.mediaLocalRoots },
-    );
-  }
-
-  if (isDiscordModerationAction(action)) {
-    const moderation = readDiscordModerationCommand(action, {
-      ...actionParams,
-      durationMinutes: readNumberParam(actionParams, "durationMin", { integer: true }),
-      deleteMessageDays: readNumberParam(actionParams, "deleteDays", {
-        integer: true,
-      }),
-    });
-    const senderUserId = normalizeOptionalString(ctx.requesterSenderId);
-    return await handleDiscordAction(
-      {
-        action: moderation.action,
-        accountId: accountId ?? undefined,
-        guildId: moderation.guildId,
-        userId: moderation.userId,
-        durationMinutes: moderation.durationMinutes,
-        until: moderation.until,
-        reason: moderation.reason,
-        deleteMessageDays: moderation.deleteMessageDays,
-        senderUserId,
-      },
-      cfg,
-    );
-  }
-
-  // Some actions are conceptually "admin", but still act on a resolved channel.
-  if (action === "thread-list") {
-    const guildId = readStringParam(actionParams, "guildId", {
-      required: true,
-    });
-    const channelId = readStringParam(actionParams, "channelId");
-    const includeArchived =
-      typeof actionParams.includeArchived === "boolean" ? actionParams.includeArchived : undefined;
-    const before = readStringParam(actionParams, "before");
-    const limit = readNumberParam(actionParams, "limit", { integer: true });
-    return await handleDiscordAction(
-      {
-        action: "threadList",
-        accountId: accountId ?? undefined,
-        guildId,
-        channelId,
-        includeArchived,
-        before,
-        limit,
-      },
-      cfg,
-    );
-  }
-
-  if (action === "thread-reply") {
-    const content = readStringParam(actionParams, "message", {
-      required: true,
-    });
-    const mediaUrl = readStringParam(actionParams, "media", { trim: false });
-    const replyTo = readStringParam(actionParams, "replyTo");
-
-    // `message.thread-reply` (tool) uses `threadId`, while the CLI historically used `to`/`channelId`.
-    // Prefer `threadId` when present to avoid accidentally replying in the parent channel.
-    const threadId = readStringParam(actionParams, "threadId");
-    const channelId = threadId ?? resolveChannelId();
-
-    return await handleDiscordAction(
-      {
-        action: "threadReply",
-        accountId: accountId ?? undefined,
-        channelId,
-        content,
-        mediaUrl: mediaUrl ?? undefined,
-        replyTo: replyTo ?? undefined,
-      },
-      cfg,
-    );
-  }
-
-  if (action === "search") {
-    const guildId = readStringParam(actionParams, "guildId", {
-      required: true,
-    });
-    const query = readStringParam(actionParams, "query", { required: true });
-    return await handleDiscordAction(
-      {
-        action: "searchMessages",
-        accountId: accountId ?? undefined,
-        guildId,
-        content: query,
-        channelId: readStringParam(actionParams, "channelId"),
-        channelIds: readStringArrayParam(actionParams, "channelIds"),
-        authorId: readStringParam(actionParams, "authorId"),
-        authorIds: readStringArrayParam(actionParams, "authorIds"),
-        limit: readNumberParam(actionParams, "limit", { integer: true }),
-      },
-      cfg,
-    );
-  }
-
-  return undefined;
-}

package/src/actions/handle-action.test.ts

@@ -1,306 +0,0 @@
-import type { OpenClawConfig } from "openclaw/plugin-sdk/config-types";
-import { beforeEach, describe, expect, it, vi } from "vitest";
-
-const runtimeModule = await import("./runtime.js");
-const handleDiscordActionMock = vi
-  .spyOn(runtimeModule, "handleDiscordAction")
-  .mockResolvedValue({ content: [], details: { ok: true } });
-const { handleDiscordMessageAction } = await import("./handle-action.js");
-
-describe("handleDiscordMessageAction", () => {
-  beforeEach(() => {
-    handleDiscordActionMock.mockClear();
-  });
-
-  it("uses trusted requesterSenderId for moderation and ignores params senderUserId", async () => {
-    await handleDiscordMessageAction({
-      action: "timeout",
-      params: {
-        guildId: "guild-1",
-        userId: "user-2",
-        durationMin: 5,
-        senderUserId: "spoofed-admin-id",
-      },
-      cfg: {
-        channels: { discord: { token: "tok", actions: { moderation: true } } },
-      } as OpenClawConfig,
-      requesterSenderId: "trusted-sender-id",
-      toolContext: { currentChannelProvider: "discord" },
-    });
-
-    expect(handleDiscordActionMock).toHaveBeenCalledWith(
-      expect.objectContaining({
-        action: "timeout",
-        guildId: "guild-1",
-        userId: "user-2",
-        durationMinutes: 5,
-        senderUserId: "trusted-sender-id",
-      }),
-      expect.objectContaining({
-        channels: {
-          discord: expect.objectContaining({
-            token: "tok",
-          }),
-        },
-      }),
-    );
-  });
-
-  it("falls back to toolContext.currentMessageId for reactions", async () => {
-    await handleDiscordMessageAction({
-      action: "react",
-      params: {
-        channelId: "123",
-        emoji: "ok",
-      },
-      cfg: {
-        channels: { discord: { token: "tok" } },
-      } as OpenClawConfig,
-      toolContext: { currentMessageId: "9001" },
-    });
-
-    expect(handleDiscordActionMock).toHaveBeenCalledWith(
-      expect.objectContaining({
-        action: "react",
-        channelId: "123",
-        messageId: "9001",
-        emoji: "ok",
-      }),
-      expect.any(Object),
-      expect.any(Object),
-    );
-  });
-
-  it("falls back to Discord toolContext.currentChannelId for reaction targets", async () => {
-    await handleDiscordMessageAction({
-      action: "react",
-      params: {
-        emoji: "ok",
-      },
-      cfg: {
-        channels: { discord: { token: "tok" } },
-      } as OpenClawConfig,
-      toolContext: {
-        currentChannelProvider: "discord",
-        currentChannelId: "user:U1",
-        currentMessageId: "9001",
-      },
-    });
-
-    expect(handleDiscordActionMock).toHaveBeenCalledWith(
-      expect.objectContaining({
-        action: "react",
-        channelId: "user:U1",
-        messageId: "9001",
-        emoji: "ok",
-      }),
-      expect.any(Object),
-      expect.any(Object),
-    );
-  });
-
-  it("falls back to Discord toolContext.currentChannelId for sends", async () => {
-    await handleDiscordMessageAction({
-      action: "send",
-      params: {
-        message: "hello",
-      },
-      cfg: {
-        channels: { discord: { token: "tok" } },
-      } as OpenClawConfig,
-      toolContext: {
-        currentChannelProvider: "discord",
-        currentChannelId: "channel:123",
-      },
-    });
-
-    expect(handleDiscordActionMock).toHaveBeenCalledWith(
-      expect.objectContaining({
-        action: "sendMessage",
-        to: "channel:123",
-        content: "hello",
-      }),
-      expect.any(Object),
-      expect.any(Object),
-    );
-  });
-
-  it("maps upload-file to Discord sendMessage with media read context", async () => {
-    const mediaReadFile = vi.fn(async () => Buffer.from("image"));
-    const mediaAccess = {
-      localRoots: ["/tmp/agent-root"],
-      readFile: mediaReadFile,
-    };
-
-    await handleDiscordMessageAction({
-      action: "upload-file",
-      params: {
-        target: "channel:123",
-        filePath: "/tmp/agent-root/image.png",
-        message: "caption",
-        filename: "image.png",
-        replyTo: "message-1",
-        silent: true,
-        __sessionKey: "session-1",
-        __agentId: "agent-1",
-      },
-      cfg: {
-        channels: { discord: { token: "tok" } },
-      } as OpenClawConfig,
-      mediaAccess,
-      mediaLocalRoots: ["/tmp/agent-root"],
-      mediaReadFile,
-    });
-
-    expect(handleDiscordActionMock).toHaveBeenCalledWith(
-      expect.objectContaining({
-        action: "sendMessage",
-        to: "channel:123",
-        content: "caption",
-        mediaUrl: "/tmp/agent-root/image.png",
-        filename: "image.png",
-        replyTo: "message-1",
-        silent: true,
-        __sessionKey: "session-1",
-        __agentId: "agent-1",
-      }),
-      expect.any(Object),
-      {
-        mediaAccess,
-        mediaLocalRoots: ["/tmp/agent-root"],
-        mediaReadFile,
-      },
-    );
-  });
-
-  it("falls back to Discord toolContext.currentChannelId for upload-file", async () => {
-    await handleDiscordMessageAction({
-      action: "upload-file",
-      params: {
-        path: "/tmp/agent-root/image.png",
-      },
-      cfg: {
-        channels: { discord: { token: "tok" } },
-      } as OpenClawConfig,
-      toolContext: {
-        currentChannelProvider: "discord",
-        currentChannelId: "channel:123",
-      },
-    });
-
-    expect(handleDiscordActionMock).toHaveBeenCalledWith(
-      expect.objectContaining({
-        action: "sendMessage",
-        to: "channel:123",
-        content: "",
-        mediaUrl: "/tmp/agent-root/image.png",
-      }),
-      expect.any(Object),
-      expect.any(Object),
-    );
-  });
-
-  it("requires a file path for upload-file", async () => {
-    await expect(
-      handleDiscordMessageAction({
-        action: "upload-file",
-        params: {
-          to: "channel:123",
-        },
-        cfg: {
-          channels: { discord: { token: "tok" } },
-        } as OpenClawConfig,
-      }),
-    ).rejects.toThrow(/upload-file requires filePath, path, or media/i);
-
-    expect(handleDiscordActionMock).not.toHaveBeenCalled();
-  });
-
-  it("forwards top-level components on sends", async () => {
-    const components = { blocks: [{ type: "text", text: "Pick one" }] };
-
-    await handleDiscordMessageAction({
-      action: "send",
-      params: {
-        message: "hello",
-        components,
-      },
-      cfg: {
-        channels: { discord: { token: "tok" } },
-      } as OpenClawConfig,
-      toolContext: {
-        currentChannelProvider: "discord",
-        currentChannelId: "channel:123",
-      },
-    });
-
-    expect(handleDiscordActionMock).toHaveBeenCalledWith(
-      expect.objectContaining({
-        action: "sendMessage",
-        to: "channel:123",
-        content: "hello",
-        components,
-      }),
-      expect.any(Object),
-      expect.any(Object),
-    );
-  });
-
-  it("does not use another provider's current target for Discord sends", async () => {
-    await expect(
-      handleDiscordMessageAction({
-        action: "send",
-        params: {
-          message: "hello",
-        },
-        cfg: {
-          channels: { discord: { token: "tok" } },
-        } as OpenClawConfig,
-        toolContext: {
-          currentChannelProvider: "telegram",
-          currentChannelId: "channel:123",
-        },
-      }),
-    ).rejects.toThrow(/channel target is required/i);
-
-    expect(handleDiscordActionMock).not.toHaveBeenCalled();
-  });
-
-  it("does not use another provider's current target for Discord reactions", async () => {
-    await expect(
-      handleDiscordMessageAction({
-        action: "react",
-        params: {
-          emoji: "ok",
-        },
-        cfg: {
-          channels: { discord: { token: "tok" } },
-        } as OpenClawConfig,
-        toolContext: {
-          currentChannelProvider: "telegram",
-          currentChannelId: "user:U1",
-          currentMessageId: "9001",
-        },
-      }),
-    ).rejects.toThrow(/channel target is required/i);
-
-    expect(handleDiscordActionMock).not.toHaveBeenCalled();
-  });
-
-  it("rejects reactions when no message id source is available", async () => {
-    await expect(
-      handleDiscordMessageAction({
-        action: "react",
-        params: {
-          channelId: "123",
-          emoji: "ok",
-        },
-        cfg: {
-          channels: { discord: { token: "tok" } },
-        } as OpenClawConfig,
-      }),
-    ).rejects.toThrow(/messageId required/i);
-
-    expect(handleDiscordActionMock).not.toHaveBeenCalled();
-  });
-});