@copilotkit/web-inspector 1.57.4 → 1.58.0-canary.thread-id-propagation

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@copilotkit/web-inspector",
-  "version": "1.57.4",
+  "version": "1.58.0-canary.thread-id-propagation",
   "description": "Lit-based web component for the CopilotKit web inspector",
   "repository": {
     "type": "git",
@@ -27,7 +27,7 @@
     "lit": "^3.2.0",
     "lucide": "^0.525.0",
     "marked": "^12.0.2",
-    "@copilotkit/core": "1.57.4"
+    "@copilotkit/core": "1.58.0-canary.thread-id-propagation"
   },
   "devDependencies": {
     "@tailwindcss/cli": "^4.1.11",

package/vitest.config.ts

@@ -5,6 +5,7 @@ export default defineConfig({
     environment: "jsdom",
     globals: true,
     clearMocks: true,
+    setupFiles: ["./vitest.setup.ts"],
     reporters: [["default", { summary: false }]],
     silent: true,
   },

package/vitest.setup.ts

@@ -0,0 +1,76 @@
+// Node 25 ships an experimental built-in `localStorage` global (gated on the
+// `--localstorage-file` flag, but the accessor exists unconditionally). When
+// vitest boots the jsdom environment, jsdom defines its own `localStorage` on
+// the synthetic `window`, but Node's global accessor still wins on
+// `globalThis.localStorage` AND — because vitest's jsdom integration aliases
+// `window` to `globalThis` — also on `window.localStorage`. The result is that
+// `window.localStorage` resolves to Node's stub object which has no `clear`,
+// `setItem`, `removeItem`, etc., breaking every test that touches localStorage.
+//
+// This setup file installs a proper in-memory Storage implementation on both
+// `globalThis` and `window` BEFORE any test code runs. The shim is a plain
+// object (not a class) so `vi.spyOn(window.localStorage, "getItem")` works —
+// vitest needs the methods to be own properties on the spied target.
+//
+// We re-install the shim in `beforeEach` so a test that did
+// `vi.restoreAllMocks()` (which restores spied methods) still sees the shim's
+// methods, and so each test starts with a fresh empty store.
+
+import { beforeEach } from "vitest";
+
+function createStorageShim(): Storage {
+  const store = new Map<string, string>();
+  const shim = {
+    get length() {
+      return store.size;
+    },
+    key(index: number): string | null {
+      return Array.from(store.keys())[index] ?? null;
+    },
+    getItem(key: string): string | null {
+      return store.has(key) ? store.get(key)! : null;
+    },
+    setItem(key: string, value: string): void {
+      store.set(String(key), String(value));
+    },
+    removeItem(key: string): void {
+      store.delete(key);
+    },
+    clear(): void {
+      store.clear();
+    },
+  } as Storage;
+  return shim;
+}
+
+function installLocalStorageShim(): void {
+  const shim = createStorageShim();
+  // Override the Node 25 global accessor (and any jsdom accessor) with a
+  // plain data property pointing at our shim. `configurable: true` so a
+  // subsequent install can replace it.
+  Object.defineProperty(globalThis, "localStorage", {
+    value: shim,
+    writable: true,
+    configurable: true,
+    enumerable: true,
+  });
+  if (typeof window !== "undefined" && window !== (globalThis as unknown)) {
+    Object.defineProperty(window, "localStorage", {
+      value: shim,
+      writable: true,
+      configurable: true,
+      enumerable: true,
+    });
+  }
+}
+
+// Install once at module load so any top-level code in test files (e.g.
+// imports that read localStorage on init) sees the shim.
+installLocalStorageShim();
+
+// Re-install before each test so `vi.restoreAllMocks()` from a prior test
+// can't leave behind spied/replaced methods, and each test starts with an
+// empty store.
+beforeEach(() => {
+  installLocalStorageShim();
+});