@wopr-network/platform-ui-core 1.1.5 → 1.1.6

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@wopr-network/platform-ui-core",
-  "version": "1.1.5",
+  "version": "1.1.6",
   "description": "Brand-agnostic AI agent platform UI — deploy as any brand via env vars",
   "repository": {
     "type": "git",

package/src/__tests__/chat-store.test.ts

@@ -3,6 +3,8 @@ import {
   clearChatHistory,
   getSessionId,
   loadChatHistory,
+  MAX_CHAT_HISTORY,
+  MAX_MESSAGE_CONTENT_LENGTH,
   saveChatHistory,
 } from "@/lib/chat/chat-store";
 import type { ChatMessage } from "@/lib/chat/types";
@@ -84,4 +86,94 @@ describe("chat-store", () => {
       expect(loadChatHistory()).toEqual([]);
     });
   });
+
+  describe("saveChatHistory limits", () => {
+    it("keeps only the last MAX_CHAT_HISTORY messages", () => {
+      const total = MAX_CHAT_HISTORY + 50;
+      const messages: ChatMessage[] = Array.from({ length: total }, (_, i) => ({
+        id: String(i),
+        role: "user" as const,
+        content: `msg ${i}`,
+        timestamp: 1000 + i,
+      }));
+      saveChatHistory(messages);
+      const loaded = loadChatHistory();
+      expect(loaded).toHaveLength(MAX_CHAT_HISTORY);
+      // Should keep the LAST MAX_CHAT_HISTORY (indices 50-149)
+      expect(loaded[0].id).toBe("50");
+      expect(loaded[MAX_CHAT_HISTORY - 1].id).toBe(String(total - 1));
+    });
+
+    it("truncates message content exceeding MAX_MESSAGE_CONTENT_LENGTH and appends ellipsis", () => {
+      const longContent = "x".repeat(MAX_MESSAGE_CONTENT_LENGTH + 1000);
+      const messages: ChatMessage[] = [
+        { id: "1", role: "user", content: longContent, timestamp: 1000 },
+      ];
+      saveChatHistory(messages);
+      const loaded = loadChatHistory();
+      expect(loaded).toHaveLength(1);
+      expect(loaded[0].content).toHaveLength(MAX_MESSAGE_CONTENT_LENGTH + 1);
+      expect(loaded[0].content).toBe(`${"x".repeat(MAX_MESSAGE_CONTENT_LENGTH)}…`);
+    });
+
+    it("does not mutate the original messages array when saving", () => {
+      const longContent = "x".repeat(MAX_MESSAGE_CONTENT_LENGTH + 1000);
+      const messages: ChatMessage[] = [
+        { id: "1", role: "user", content: longContent, timestamp: 1000 },
+      ];
+      const originalContent = messages[0].content;
+      saveChatHistory(messages);
+      // localStorage copy is truncated; in-memory array must be unchanged
+      expect(messages[0].content).toBe(originalContent);
+      expect(messages[0].content).toHaveLength(MAX_MESSAGE_CONTENT_LENGTH + 1000);
+    });
+
+    it("slices by Unicode codepoints so emoji are not split", () => {
+      // Each emoji is 2 UTF-16 code units but 1 codepoint.
+      // String.slice() cuts by code units; [...str].slice() cuts by codepoints.
+      // A string of MAX_MESSAGE_CONTENT_LENGTH emoji has 2*MAX code units.
+      // With .slice() the last emoji would be split; with spread it must not be.
+      const emoji = "\u{1F600}"; // 😀 — 2 code units
+      const longContent = emoji.repeat(MAX_MESSAGE_CONTENT_LENGTH + 100);
+      const messages: ChatMessage[] = [
+        { id: "1", role: "user", content: longContent, timestamp: 1000 },
+      ];
+      saveChatHistory(messages);
+      const loaded = loadChatHistory();
+      const stored = loaded[0].content;
+      // The stored content (minus ellipsis) must decode to valid codepoints only.
+      // If the last emoji were split, it would contain a lone surrogate which
+      // JSON.parse would either mangle or preserve as an invalid codepoint.
+      const withoutEllipsis = stored.slice(0, -1); // remove "…"
+      expect([...withoutEllipsis].every((ch) => ch === emoji)).toBe(true);
+    });
+
+    it("does not truncate content at or below the limit", () => {
+      const exactContent = "y".repeat(MAX_MESSAGE_CONTENT_LENGTH);
+      const messages: ChatMessage[] = [
+        { id: "1", role: "user", content: exactContent, timestamp: 1000 },
+      ];
+      saveChatHistory(messages);
+      const loaded = loadChatHistory();
+      expect(loaded[0].content).toBe(exactContent);
+    });
+
+    it("applies both message count and content length limits together", () => {
+      const total = MAX_CHAT_HISTORY + 10;
+      const messages: ChatMessage[] = Array.from({ length: total }, (_, i) => ({
+        id: String(i),
+        role: "bot" as const,
+        content: "z".repeat(MAX_MESSAGE_CONTENT_LENGTH + 1000),
+        timestamp: 1000 + i,
+      }));
+      saveChatHistory(messages);
+      const loaded = loadChatHistory();
+      expect(loaded).toHaveLength(MAX_CHAT_HISTORY);
+      expect(loaded[0].id).toBe("10");
+      for (const msg of loaded) {
+        expect(msg.content).toHaveLength(MAX_MESSAGE_CONTENT_LENGTH + 1);
+        expect(msg.content.endsWith("…")).toBe(true);
+      }
+    });
+  });
 });

package/src/components/channel-wizard/field-qr.tsx

@@ -163,7 +163,7 @@ export function FieldQR({ field, value: _value, onChange, error, botId }: FieldQ
 
               {/* bg-white is intentional -- QR codes require white background for scanability */}
               <div className="h-40 w-40 rounded-sm bg-white p-3 min-[375px]:h-48 min-[375px]:w-48">
-                {/* biome-ignore lint/performance/noImgElement: QR code is a base64 data URL, not optimizable by next/image */}
+                {/* biome-ignore lint/performance/noImgElement: QR PNG is a base64 data URI from the API — next/image does not support data: URIs (cannot optimize, resize, or lazy-load inline blobs). Raw <img> is the only option here. */}
                 <img
                   src={qrPng}
                   alt="Scan this QR code with your phone"
@@ -205,7 +205,7 @@ export function FieldQR({ field, value: _value, onChange, error, botId }: FieldQ
               {/* bg-white is intentional -- QR codes require white background for scanability */}
               <div className="relative h-40 w-40 rounded-sm bg-white p-3 min-[375px]:h-48 min-[375px]:w-48">
                 {qrPng && (
-                  // biome-ignore lint/performance/noImgElement: QR code is a base64 data URL
+                  // biome-ignore lint/performance/noImgElement: base64 data URI — see comment above for rationale
                   <img src={qrPng} alt="Expired QR code" className="h-full w-full opacity-40" />
                 )}
                 {/* Dark overlay */}

package/src/lib/chat/chat-store.ts

@@ -5,6 +5,12 @@ import type { ChatMessage } from "./types";
 const HISTORY_KEY = storageKey("chat-history");
 const SESSION_KEY = storageKey("chat-session");
 
+/** Maximum number of messages persisted to localStorage. */
+export const MAX_CHAT_HISTORY = 100;
+
+/** Maximum character length for a single message's content field. */
+export const MAX_MESSAGE_CONTENT_LENGTH = 4_000;
+
 const ChatMessageSchema = z.object({
   id: z.string(),
   role: z.enum(["user", "bot", "event"]),
@@ -46,9 +52,15 @@ export function loadChatHistory(): ChatMessage[] {
 export function saveChatHistory(messages: ChatMessage[]): void {
   if (typeof window === "undefined") return;
   try {
-    localStorage.setItem(HISTORY_KEY, JSON.stringify(messages));
+    const trimmed = messages.slice(-MAX_CHAT_HISTORY).map((msg) => {
+      const codepoints = [...msg.content];
+      return codepoints.length > MAX_MESSAGE_CONTENT_LENGTH
+        ? { ...msg, content: `${codepoints.slice(0, MAX_MESSAGE_CONTENT_LENGTH).join("")}…` }
+        : msg;
+    });
+    localStorage.setItem(HISTORY_KEY, JSON.stringify(trimmed));
   } catch {
-    // ignore
+    // ignore — quota exceeded or private browsing
   }
 }