fix(line): count rich menu limits by grapheme

vincentkoc · vincentkoc · commit 8b8b72c16514 · 2026-07-03T23:31:52.000-07:00
diff --git a/extensions/line/src/rich-menu.test.ts b/extensions/line/src/rich-menu.test.ts
@@ -5,6 +5,7 @@ import path from "node:path";
 import type { OpenClawConfig } from "openclaw/plugin-sdk/config-contracts";
 import { afterAll, afterEach, beforeEach, describe, expect, it, vi } from "vitest";
 import {
+  createRichMenu,
   createDefaultMenuConfig,
   createGridLayout,
   datetimePickerAction,
@@ -14,19 +15,33 @@ import {
   uriAction,
 } from "./rich-menu.js";
 
-const { setRichMenuImageMock, MessagingApiBlobClientMock } = vi.hoisted(() => {
+const {
+  createRichMenuMock,
+  setRichMenuImageMock,
+  MessagingApiClientMock,
+  MessagingApiBlobClientMock,
+} = vi.hoisted(() => {
+  const createRichMenuMockLocal = vi.fn();
   const setRichMenuImageMockLocal = vi.fn();
+  const MessagingApiClientMockLocal = vi.fn(function () {
+    return { createRichMenu: createRichMenuMockLocal };
+  });
   const MessagingApiBlobClientMockLocal = vi.fn(function () {
     return { setRichMenuImage: setRichMenuImageMockLocal };
   });
   return {
+    createRichMenuMock: createRichMenuMockLocal,
     setRichMenuImageMock: setRichMenuImageMockLocal,
+    MessagingApiClientMock: MessagingApiClientMockLocal,
     MessagingApiBlobClientMock: MessagingApiBlobClientMockLocal,
   };
 });
 
 vi.mock("@line/bot-sdk", () => ({
-  messagingApi: { MessagingApiBlobClient: MessagingApiBlobClientMock },
+  messagingApi: {
+    MessagingApiClient: MessagingApiClientMock,
+    MessagingApiBlobClient: MessagingApiBlobClientMock,
+  },
 }));
 
 afterAll(() => {
@@ -241,6 +256,37 @@ const richMenuUploadCfg: OpenClawConfig = {
   },
 };
 
+describe("createRichMenu", () => {
+  beforeEach(() => {
+    createRichMenuMock.mockReset();
+    createRichMenuMock.mockResolvedValue({ richMenuId: "rich-menu-1" });
+    MessagingApiClientMock.mockClear();
+  });
+
+  it("truncates names and chat bar text by grapheme cluster", async () => {
+    const emoji = "😀";
+    const familyEmoji = "👨‍👩‍👧‍👦";
+
+    await createRichMenu(
+      {
+        size: { width: 2500, height: 843 },
+        name: emoji.repeat(301),
+        chatBarText: familyEmoji.repeat(15),
+        areas: [],
+      },
+      { cfg: richMenuUploadCfg },
+    );
+
+    expect(MessagingApiClientMock).toHaveBeenCalledWith({ channelAccessToken: "line-token" });
+    expect(createRichMenuMock).toHaveBeenCalledWith(
+      expect.objectContaining({
+        name: emoji.repeat(300),
+        chatBarText: familyEmoji.repeat(14),
+      }),
+    );
+  });
+});
+
 describe("uploadRichMenuImage", () => {
   let tempRoot: string;
 
diff --git a/extensions/line/src/rich-menu.ts b/extensions/line/src/rich-menu.ts
@@ -4,7 +4,6 @@ import { getAgentScopedMediaLocalRoots } from "openclaw/plugin-sdk/agent-media-p
 import type { OpenClawConfig } from "openclaw/plugin-sdk/config-contracts";
 import { mimeTypeFromFilePath } from "openclaw/plugin-sdk/media-mime";
 import { logVerbose } from "openclaw/plugin-sdk/runtime-env";
-import { truncateUtf16Safe } from "openclaw/plugin-sdk/text-utility-runtime";
 import { loadWebMediaRaw } from "openclaw/plugin-sdk/web-media";
 import { resolveLineAccount } from "./accounts.js";
 import { datetimePickerAction, messageAction, postbackAction, uriAction } from "./actions.js";
@@ -15,6 +14,8 @@ type RichMenuResponse = messagingApi.RichMenuResponse;
 type RichMenuArea = messagingApi.RichMenuArea;
 type Action = messagingApi.Action;
 const USER_BATCH_SIZE = 500;
+// LINE counts rich-menu names and chat-bar text in grapheme clusters, unlike most message fields.
+const graphemeSegmenter = new Intl.Segmenter(undefined, { granularity: "grapheme" });
 
 export interface RichMenuSize {
   width: 2500;
@@ -79,6 +80,19 @@ function chunkUserIds(userIds: string[]): string[][] {
   return batches;
 }
 
+function truncateGraphemes(input: string, maxLength: number): string {
+  let result = "";
+  let count = 0;
+  for (const { segment } of graphemeSegmenter.segment(input)) {
+    if (count >= maxLength) {
+      break;
+    }
+    result += segment;
+    count += 1;
+  }
+  return result;
+}
+
 export async function createRichMenu(
   menu: CreateRichMenuParams,
   opts: RichMenuOpts,
@@ -88,8 +102,8 @@ export async function createRichMenu(
   const richMenuRequest: RichMenuRequest = {
     size: menu.size,
     selected: menu.selected ?? false,
-    name: truncateUtf16Safe(menu.name, 300),
-    chatBarText: truncateUtf16Safe(menu.chatBarText, 14),
+    name: truncateGraphemes(menu.name, 300),
+    chatBarText: truncateGraphemes(menu.chatBarText, 14),
     areas: menu.areas as RichMenuArea[],
   };
 
