@openclaw/whatsapp 2026.5.2-beta.2 → 2026.5.3-beta.1

package/src/channel-react-action.test.ts

@@ -1,294 +0,0 @@
-import { beforeEach, describe, expect, it, vi } from "vitest";
-import { handleWhatsAppReactAction } from "./channel-react-action.js";
-import type { OpenClawConfig } from "./runtime-api.js";
-
-const hoisted = vi.hoisted(() => ({
-  handleWhatsAppAction: vi.fn(async () => ({ content: [{ type: "text", text: '{"ok":true}' }] })),
-}));
-
-vi.mock("./channel-react-action.runtime.js", async () => {
-  return {
-    handleWhatsAppAction: hoisted.handleWhatsAppAction,
-    resolveReactionMessageId: ({
-      args,
-      toolContext,
-    }: {
-      args: Record<string, unknown>;
-      toolContext?: { currentMessageId?: string | number | null };
-    }) => args.messageId ?? toolContext?.currentMessageId ?? null,
-    readStringOrNumberParam: (params: Record<string, unknown>, key: string) => {
-      const value = params[key];
-      if (typeof value === "number" && Number.isFinite(value)) {
-        return value;
-      }
-      if (typeof value === "string" && value.trim()) {
-        return value;
-      }
-      return undefined;
-    },
-    isWhatsAppGroupJid: (value?: string | null) => (value ?? "").trim().endsWith("@g.us"),
-    normalizeWhatsAppTarget: (value?: string | null) => {
-      const raw = (value ?? "").trim();
-      if (!raw) {
-        return null;
-      }
-      const stripped = raw.replace(/^whatsapp:/, "");
-      if (stripped.endsWith("@g.us")) {
-        return stripped;
-      }
-      return stripped.startsWith("+") ? stripped : `+${stripped.replace(/^\+/, "")}`;
-    },
-    readStringParam: (
-      params: Record<string, unknown>,
-      key: string,
-      options?: { required?: boolean; allowEmpty?: boolean },
-    ) => {
-      const value = params[key];
-      if (value == null) {
-        if (options?.required) {
-          const err = new Error(`${key} required`);
-          err.name = "ToolInputError";
-          throw err;
-        }
-        return undefined;
-      }
-      const text = typeof value === "string" ? value : "";
-      if (!options?.allowEmpty && !text.trim()) {
-        if (options?.required) {
-          const err = new Error(`${key} required`);
-          err.name = "ToolInputError";
-          throw err;
-        }
-        return undefined;
-      }
-      return text;
-    },
-  };
-});
-
-describe("whatsapp react action messageId resolution", () => {
-  const baseCfg = {
-    channels: { whatsapp: { actions: { reactions: true }, allowFrom: ["*"] } },
-  } as OpenClawConfig;
-
-  beforeEach(() => {
-    hoisted.handleWhatsAppAction.mockClear();
-  });
-
-  it("uses explicit messageId when provided", async () => {
-    await handleWhatsAppReactAction({
-      action: "react",
-      params: { messageId: "explicit-id", emoji: "👍", to: "+1555" },
-      cfg: baseCfg,
-      accountId: "default",
-    });
-    expect(hoisted.handleWhatsAppAction).toHaveBeenCalledWith(
-      expect.objectContaining({ messageId: "explicit-id" }),
-      baseCfg,
-    );
-  });
-
-  it("falls back to toolContext.currentMessageId when messageId omitted", async () => {
-    await handleWhatsAppReactAction({
-      action: "react",
-      params: { emoji: "❤️", to: "+1555" },
-      cfg: baseCfg,
-      accountId: "default",
-      toolContext: {
-        currentChannelId: "whatsapp:+1555",
-        currentChannelProvider: "whatsapp",
-        currentMessageId: "ctx-msg-42",
-      },
-    });
-    expect(hoisted.handleWhatsAppAction).toHaveBeenCalledWith(
-      expect.objectContaining({ messageId: "ctx-msg-42" }),
-      baseCfg,
-    );
-  });
-
-  it("converts numeric toolContext messageId to string", async () => {
-    await handleWhatsAppReactAction({
-      action: "react",
-      params: { emoji: "🎉", to: "+1555" },
-      cfg: baseCfg,
-      accountId: "default",
-      toolContext: {
-        currentChannelId: "whatsapp:+1555",
-        currentChannelProvider: "whatsapp",
-        currentMessageId: 12345,
-      },
-    });
-    expect(hoisted.handleWhatsAppAction).toHaveBeenCalledWith(
-      expect.objectContaining({ messageId: "12345" }),
-      baseCfg,
-    );
-  });
-
-  it("throws ToolInputError when messageId missing and no toolContext", async () => {
-    const err = await handleWhatsAppReactAction({
-      action: "react",
-      params: { emoji: "👍", to: "+1555" },
-      cfg: baseCfg,
-      accountId: "default",
-    }).catch((e: unknown) => e);
-    expect(err).toBeInstanceOf(Error);
-    expect((err as Error).name).toBe("ToolInputError");
-  });
-
-  it("skips context fallback when targeting a different chat", async () => {
-    const err = await handleWhatsAppReactAction({
-      action: "react",
-      params: { emoji: "👍", to: "+9999" },
-      cfg: baseCfg,
-      accountId: "default",
-      toolContext: {
-        currentChannelId: "whatsapp:+1555",
-        currentChannelProvider: "whatsapp",
-        currentMessageId: "ctx-msg-42",
-      },
-    }).catch((e: unknown) => e);
-    expect(err).toBeInstanceOf(Error);
-    expect((err as Error).name).toBe("ToolInputError");
-  });
-
-  it("uses context fallback when target matches current chat", async () => {
-    await handleWhatsAppReactAction({
-      action: "react",
-      params: { emoji: "👍", to: "12345@g.us" },
-      cfg: baseCfg,
-      accountId: "default",
-      requesterSenderId: "123@lid",
-      toolContext: {
-        currentChannelId: "whatsapp:12345@g.us",
-        currentChannelProvider: "whatsapp",
-        currentMessageId: "ctx-msg-42",
-      },
-    });
-    expect(hoisted.handleWhatsAppAction).toHaveBeenCalledWith(
-      expect.objectContaining({
-        messageId: "ctx-msg-42",
-        participant: "123@lid",
-      }),
-      baseCfg,
-    );
-  });
-
-  it("keeps direct-chat reactions without an inferred participant", async () => {
-    await handleWhatsAppReactAction({
-      action: "react",
-      params: { emoji: "👍", to: "+1555" },
-      cfg: baseCfg,
-      accountId: "default",
-      requesterSenderId: "123@lid",
-      toolContext: {
-        currentChannelId: "whatsapp:+1555",
-        currentChannelProvider: "whatsapp",
-        currentMessageId: "ctx-msg-42",
-      },
-    });
-    expect(hoisted.handleWhatsAppAction).toHaveBeenCalledWith(
-      expect.objectContaining({
-        messageId: "ctx-msg-42",
-        participant: undefined,
-      }),
-      baseCfg,
-    );
-  });
-
-  it("prefers explicit participant over inferred current-message participant", async () => {
-    await handleWhatsAppReactAction({
-      action: "react",
-      params: {
-        emoji: "👍",
-        to: "12345@g.us",
-        participant: "555@s.whatsapp.net",
-      },
-      cfg: baseCfg,
-      accountId: "default",
-      requesterSenderId: "123@lid",
-      toolContext: {
-        currentChannelId: "whatsapp:12345@g.us",
-        currentChannelProvider: "whatsapp",
-        currentMessageId: "ctx-msg-42",
-      },
-    });
-    expect(hoisted.handleWhatsAppAction).toHaveBeenCalledWith(
-      expect.objectContaining({
-        messageId: "ctx-msg-42",
-        participant: "555@s.whatsapp.net",
-      }),
-      baseCfg,
-    );
-  });
-
-  it("does not reuse the current-chat participant for cross-chat reactions", async () => {
-    const err = await handleWhatsAppReactAction({
-      action: "react",
-      params: { emoji: "👍", to: "99999@g.us" },
-      cfg: baseCfg,
-      accountId: "default",
-      requesterSenderId: "123@lid",
-      toolContext: {
-        currentChannelId: "whatsapp:12345@g.us",
-        currentChannelProvider: "whatsapp",
-        currentMessageId: "ctx-msg-42",
-      },
-    }).catch((e: unknown) => e);
-    expect(err).toBeInstanceOf(Error);
-    expect((err as Error).name).toBe("ToolInputError");
-    expect(hoisted.handleWhatsAppAction).not.toHaveBeenCalled();
-  });
-
-  it("does not infer participant when messageId is explicitly provided", async () => {
-    await handleWhatsAppReactAction({
-      action: "react",
-      params: { emoji: "👍", to: "12345@g.us", messageId: "older-msg-7" },
-      cfg: baseCfg,
-      accountId: "default",
-      requesterSenderId: "123@lid",
-      toolContext: {
-        currentChannelId: "whatsapp:12345@g.us",
-        currentChannelProvider: "whatsapp",
-        currentMessageId: "ctx-msg-42",
-      },
-    });
-    expect(hoisted.handleWhatsAppAction).toHaveBeenCalledWith(
-      expect.objectContaining({
-        messageId: "older-msg-7",
-        participant: undefined,
-      }),
-      baseCfg,
-    );
-  });
-
-  it("skips context fallback when source is another provider", async () => {
-    const err = await handleWhatsAppReactAction({
-      action: "react",
-      params: { emoji: "👍", to: "+1555" },
-      cfg: baseCfg,
-      accountId: "default",
-      toolContext: {
-        currentChannelId: "telegram:-1003841603622",
-        currentChannelProvider: "telegram",
-        currentMessageId: "tg-msg-99",
-      },
-    }).catch((e: unknown) => e);
-    expect(err).toBeInstanceOf(Error);
-    expect((err as Error).name).toBe("ToolInputError");
-  });
-
-  it("skips context fallback when currentChannelId is missing with explicit target", async () => {
-    const err = await handleWhatsAppReactAction({
-      action: "react",
-      params: { emoji: "👍", to: "+1555" },
-      cfg: baseCfg,
-      accountId: "default",
-      toolContext: {
-        currentChannelProvider: "whatsapp",
-        currentMessageId: "ctx-msg-42",
-      },
-    }).catch((e: unknown) => e);
-    expect(err).toBeInstanceOf(Error);
-    expect((err as Error).name).toBe("ToolInputError");
-  });
-});

package/src/channel-react-action.ts

@@ -1,84 +0,0 @@
-import {
-  isWhatsAppGroupJid,
-  resolveReactionMessageId,
-  handleWhatsAppAction,
-  normalizeWhatsAppTarget,
-  readStringOrNumberParam,
-  readStringParam,
-  type OpenClawConfig,
-} from "./channel-react-action.runtime.js";
-
-const WHATSAPP_CHANNEL = "whatsapp" as const;
-
-export async function handleWhatsAppReactAction(params: {
-  action: string;
-  params: Record<string, unknown>;
-  cfg: OpenClawConfig;
-  accountId?: string | null;
-  requesterSenderId?: string | null;
-  toolContext?: {
-    currentChannelId?: string | null;
-    currentChannelProvider?: string | null;
-    currentMessageId?: string | number | null;
-  };
-}) {
-  if (params.action !== "react") {
-    throw new Error(`Action ${params.action} is not supported for provider ${WHATSAPP_CHANNEL}.`);
-  }
-  const isWhatsAppSource = params.toolContext?.currentChannelProvider === WHATSAPP_CHANNEL;
-  const explicitTarget =
-    readStringParam(params.params, "chatJid") ?? readStringParam(params.params, "to");
-  const normalizedTarget = explicitTarget ? normalizeWhatsAppTarget(explicitTarget) : null;
-  const normalizedCurrent =
-    isWhatsAppSource && params.toolContext?.currentChannelId
-      ? normalizeWhatsAppTarget(params.toolContext.currentChannelId)
-      : null;
-  const isCrossChat =
-    normalizedTarget != null &&
-    (normalizedCurrent == null || normalizedTarget !== normalizedCurrent);
-  const scopedContext =
-    !isWhatsAppSource || isCrossChat || !params.toolContext
-      ? undefined
-      : {
-          currentChannelId: params.toolContext.currentChannelId ?? undefined,
-          currentChannelProvider: params.toolContext.currentChannelProvider ?? undefined,
-          currentMessageId: params.toolContext.currentMessageId ?? undefined,
-        };
-  const messageIdRaw = resolveReactionMessageId({
-    args: params.params,
-    toolContext: scopedContext,
-  });
-  if (messageIdRaw == null) {
-    readStringParam(params.params, "messageId", { required: true });
-  }
-  const messageId = String(messageIdRaw);
-  const explicitMessageId = readStringOrNumberParam(params.params, "messageId");
-  const emoji = readStringParam(params.params, "emoji", { allowEmpty: true });
-  const remove = typeof params.params.remove === "boolean" ? params.params.remove : undefined;
-  const explicitParticipant = readStringParam(params.params, "participant");
-  const inferredParticipant =
-    explicitParticipant ||
-    explicitMessageId != null ||
-    !isWhatsAppSource ||
-    isCrossChat ||
-    !isWhatsAppGroupJid(explicitTarget ?? params.toolContext?.currentChannelId ?? "")
-      ? undefined
-      : typeof params.requesterSenderId === "string" && params.requesterSenderId.trim().length > 0
-        ? params.requesterSenderId.trim()
-        : undefined;
-  return await handleWhatsAppAction(
-    {
-      action: "react",
-      chatJid:
-        readStringParam(params.params, "chatJid") ??
-        readStringParam(params.params, "to", { required: true }),
-      messageId,
-      emoji,
-      remove,
-      participant: explicitParticipant ?? inferredParticipant,
-      accountId: params.accountId ?? undefined,
-      fromMe: typeof params.params.fromMe === "boolean" ? params.params.fromMe : undefined,
-    },
-    params.cfg,
-  );
-}

package/src/channel.runtime.ts

@@ -1,117 +0,0 @@
-import {
-  startWebLoginWithQr as startWebLoginWithQrImpl,
-  waitForWebLogin as waitForWebLoginImpl,
-} from "../login-qr-runtime.js";
-import { getActiveWebListener as getActiveWebListenerImpl } from "./active-listener.js";
-import {
-  getWebAuthAgeMs as getWebAuthAgeMsImpl,
-  logWebSelfId as logWebSelfIdImpl,
-  logoutWeb as logoutWebImpl,
-  readWebAuthSnapshot as readWebAuthSnapshotImpl,
-  readWebAuthState as readWebAuthStateImpl,
-  readWebAuthExistsBestEffort as readWebAuthExistsBestEffortImpl,
-  readWebAuthExistsForDecision as readWebAuthExistsForDecisionImpl,
-  readWebAuthSnapshotBestEffort as readWebAuthSnapshotBestEffortImpl,
-  readWebSelfId as readWebSelfIdImpl,
-  webAuthExists as webAuthExistsImpl,
-} from "./auth-store.js";
-import { monitorWebChannel as monitorWebChannelImpl } from "./auto-reply/monitor.js";
-import { loginWeb as loginWebImpl } from "./login.js";
-import { whatsappSetupWizard as whatsappSetupWizardImpl } from "./setup-surface.js";
-
-type GetActiveWebListener = typeof import("./active-listener.js").getActiveWebListener;
-type GetWebAuthAgeMs = typeof import("./auth-store.js").getWebAuthAgeMs;
-type LogWebSelfId = typeof import("./auth-store.js").logWebSelfId;
-type LogoutWeb = typeof import("./auth-store.js").logoutWeb;
-type ReadWebAuthSnapshot = typeof import("./auth-store.js").readWebAuthSnapshot;
-type ReadWebAuthState = typeof import("./auth-store.js").readWebAuthState;
-type ReadWebAuthExistsBestEffort = typeof import("./auth-store.js").readWebAuthExistsBestEffort;
-type ReadWebAuthExistsForDecision = typeof import("./auth-store.js").readWebAuthExistsForDecision;
-type ReadWebAuthSnapshotBestEffort = typeof import("./auth-store.js").readWebAuthSnapshotBestEffort;
-type ReadWebSelfId = typeof import("./auth-store.js").readWebSelfId;
-type WebAuthExists = typeof import("./auth-store.js").webAuthExists;
-type LoginWeb = typeof import("./login.js").loginWeb;
-type StartWebLoginWithQr = typeof import("../login-qr-runtime.js").startWebLoginWithQr;
-type WaitForWebLogin = typeof import("../login-qr-runtime.js").waitForWebLogin;
-type WhatsAppSetupWizard = typeof import("./setup-surface.js").whatsappSetupWizard;
-type MonitorWebChannel = typeof import("./auto-reply/monitor.js").monitorWebChannel;
-
-export function getActiveWebListener(
-  ...args: Parameters<GetActiveWebListener>
-): ReturnType<GetActiveWebListener> {
-  return getActiveWebListenerImpl(...args);
-}
-
-export function getWebAuthAgeMs(...args: Parameters<GetWebAuthAgeMs>): ReturnType<GetWebAuthAgeMs> {
-  return getWebAuthAgeMsImpl(...args);
-}
-
-export function logWebSelfId(...args: Parameters<LogWebSelfId>): ReturnType<LogWebSelfId> {
-  return logWebSelfIdImpl(...args);
-}
-
-export function logoutWeb(...args: Parameters<LogoutWeb>): ReturnType<LogoutWeb> {
-  return logoutWebImpl(...args);
-}
-
-export function readWebAuthSnapshot(
-  ...args: Parameters<ReadWebAuthSnapshot>
-): ReturnType<ReadWebAuthSnapshot> {
-  return readWebAuthSnapshotImpl(...args);
-}
-
-export function readWebAuthState(
-  ...args: Parameters<ReadWebAuthState>
-): ReturnType<ReadWebAuthState> {
-  return readWebAuthStateImpl(...args);
-}
-
-export function readWebAuthExistsBestEffort(
-  ...args: Parameters<ReadWebAuthExistsBestEffort>
-): ReturnType<ReadWebAuthExistsBestEffort> {
-  return readWebAuthExistsBestEffortImpl(...args);
-}
-
-export function readWebAuthExistsForDecision(
-  ...args: Parameters<ReadWebAuthExistsForDecision>
-): ReturnType<ReadWebAuthExistsForDecision> {
-  return readWebAuthExistsForDecisionImpl(...args);
-}
-
-export function readWebAuthSnapshotBestEffort(
-  ...args: Parameters<ReadWebAuthSnapshotBestEffort>
-): ReturnType<ReadWebAuthSnapshotBestEffort> {
-  return readWebAuthSnapshotBestEffortImpl(...args);
-}
-
-export function readWebSelfId(...args: Parameters<ReadWebSelfId>): ReturnType<ReadWebSelfId> {
-  return readWebSelfIdImpl(...args);
-}
-
-export function webAuthExists(...args: Parameters<WebAuthExists>): ReturnType<WebAuthExists> {
-  return webAuthExistsImpl(...args);
-}
-
-export function loginWeb(...args: Parameters<LoginWeb>): ReturnType<LoginWeb> {
-  return loginWebImpl(...args);
-}
-
-export async function startWebLoginWithQr(
-  ...args: Parameters<StartWebLoginWithQr>
-): ReturnType<StartWebLoginWithQr> {
-  return await startWebLoginWithQrImpl(...args);
-}
-
-export async function waitForWebLogin(
-  ...args: Parameters<WaitForWebLogin>
-): ReturnType<WaitForWebLogin> {
-  return await waitForWebLoginImpl(...args);
-}
-
-export const whatsappSetupWizard: WhatsAppSetupWizard = { ...whatsappSetupWizardImpl };
-
-export function monitorWebChannel(
-  ...args: Parameters<MonitorWebChannel>
-): ReturnType<MonitorWebChannel> {
-  return monitorWebChannelImpl(...args);
-}