@copilotkit/react-core 1.57.2 → 1.57.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@copilotkit/react-core",
-  "version": "1.57.2",
+  "version": "1.57.3",
   "private": false,
   "keywords": [
     "ai",
@@ -81,11 +81,11 @@
     "untruncate-json": "^0.0.1",
     "use-stick-to-bottom": "^1.1.1",
     "zod-to-json-schema": "^3.24.5",
-    "@copilotkit/core": "1.57.2",
-    "@copilotkit/shared": "1.57.2",
-    "@copilotkit/web-inspector": "1.57.2",
-    "@copilotkit/runtime-client-gql": "1.57.2",
-    "@copilotkit/a2ui-renderer": "1.57.2"
+    "@copilotkit/a2ui-renderer": "1.57.3",
+    "@copilotkit/core": "1.57.3",
+    "@copilotkit/runtime-client-gql": "1.57.3",
+    "@copilotkit/shared": "1.57.3",
+    "@copilotkit/web-inspector": "1.57.3"
   },
   "devDependencies": {
     "@tailwindcss/cli": "^4.1.11",

package/src/v2/components/chat/__tests__/CopilotChatPerf.e2e.test.tsx

@@ -115,17 +115,58 @@ describe("CopilotChat perf — re-render regression", () => {
     renderCounts.clear();
   });
 
-  // TanStack Virtual schedules rAF callbacks for measurement. On Node 20 the
-  // jsdom timing is different — draining a fixed number of rAF rounds is not
-  // enough because the observer keeps scheduling new callbacks. When
-  // @testing-library/react's auto-cleanup unmounts the component AFTER our
-  // afterEach, any pending rAF fires against a torn-down window (null) and
-  // throws: TypeError: Cannot read properties of null ('requestAnimationFrame').
+  // @tanstack/virtual-core 3.13.18 has a latent bug: `scrollToIndex` schedules
+  // a nested rAF that calls `this.targetWindow.requestAnimationFrame(verify)`
+  // with no null-check. The virtualizer's cleanup (run on React unmount) sets
+  // `targetWindow = null`, so if the outer rAF fires after unmount, the inner
+  // schedule throws `Cannot read properties of null (reading 'requestAnimationFrame')`.
   //
-  // Fix: after draining what we can, replace rAF with a no-op so any callback
-  // scheduled during (or after) RTL cleanup is silently swallowed. Restore the
-  // real rAF at the start of the next test via beforeEach.
+  // Vitest reports this as an unhandled error and exits non-zero even though
+  // every test passes. We wrap rAF (on BOTH globalThis and window — they're
+  // separate bindings in vitest+jsdom; tanstack uses `targetWindow.rAF` which
+  // resolves to `window.rAF`) so callbacks that hit the known tanstack bug
+  // are swallowed. After each test we drain pending callbacks and replace rAF
+  // with a no-op to keep stragglers from leaking into the next test.
   const realRAF = globalThis.requestAnimationFrame;
+  const realWindowRAF =
+    typeof window !== "undefined" ? window.requestAnimationFrame : realRAF;
+
+  const isKnownTanstackTeardownError = (err: unknown): boolean => {
+    const message =
+      (err as { message?: string } | null | undefined)?.message ?? "";
+    return (
+      message.includes("Cannot read properties of null") &&
+      message.includes("requestAnimationFrame")
+    );
+  };
+
+  const wrapRAF = (real: typeof requestAnimationFrame) =>
+    ((cb: FrameRequestCallback) =>
+      real((time) => {
+        try {
+          cb(time);
+        } catch (err) {
+          if (isKnownTanstackTeardownError(err)) return;
+          throw err;
+        }
+      })) as typeof requestAnimationFrame;
+
+  const noopRAF = ((_cb: FrameRequestCallback) =>
+    0) as typeof requestAnimationFrame;
+
+  const installSafeRAF = () => {
+    globalThis.requestAnimationFrame = wrapRAF(realRAF);
+    if (typeof window !== "undefined") {
+      window.requestAnimationFrame = wrapRAF(realWindowRAF);
+    }
+  };
+
+  const installNoopRAF = () => {
+    globalThis.requestAnimationFrame = noopRAF;
+    if (typeof window !== "undefined") {
+      window.requestAnimationFrame = noopRAF;
+    }
+  };
 
   afterEach(async () => {
     // Best-effort drain of already-queued rAF callbacks.
@@ -134,13 +175,12 @@ describe("CopilotChat perf — re-render regression", () => {
       await new Promise<void>((r) => realRAF(() => r()));
     });
     // Neuter rAF so callbacks scheduled during RTL cleanup are harmless.
-    globalThis.requestAnimationFrame = ((_cb: FrameRequestCallback) =>
-      0) as typeof requestAnimationFrame;
+    installNoopRAF();
   });
 
   beforeEach(() => {
-    // Restore real rAF for the upcoming test.
-    globalThis.requestAnimationFrame = realRAF;
+    // Install the error-swallowing wrap on rAF for the upcoming test.
+    installSafeRAF();
   });
 
   it("completed messages do not re-render when a new message is added", async () => {