@box/box-ai-agent-selector 0.40.1 → 0.40.3

package/dist/esm/lib/box-ai-agent-selector.js

@@ -4629,7 +4629,7 @@ function zc({
     formatMessage: f
   } = bi(), [c, l] = X(!1), p = o === fn.ERROR, [b, v] = X(p), [x, w] = Ei(Si), P = wi(), g = o === fn.SUCCESS, m = o === fn.IN_PROGRESS, y = f(Qe.selectAgent), C = f(Qe.defaultAgent);
   let O = "";
-  g ? O = (u == null ? void 0 : u.name) || y : p && (O = C);
+  g ? O = (u == null ? void 0 : u.name) ?? y : p && (O = C);
   const A = Y(null), {
     Wrapper: S,
     wrapperProps: T

package/dist/esm/lib/hooks/__tests__/useLocalStorageAIAgents.tests.js

@@ -1,24 +1,37 @@
-import { renderHook as n, act as c } from "@testing-library/react";
-import l from "../useLocalStorageAIAgents.js";
+import { renderHook as l, act as d } from "@testing-library/react";
+import c from "../useLocalStorageAIAgents.js";
 describe("useLocalStorageAIAgents", () => {
-  const t = "testAgentKey", a = "agent-456";
-  let u, i, d;
-  beforeEach(() => {
-    u = jest.spyOn(Storage.prototype, "setItem"), i = jest.spyOn(Storage.prototype, "getItem"), d = jest.spyOn(Storage.prototype, "removeItem"), localStorage.clear();
+  const o = "testAgentKey", r = "agent-456";
+  let a = {};
+  const g = jest.fn((e) => a[e]);
+  let u = jest.fn((e, t) => {
+    a[e] = t;
+  });
+  const i = jest.fn((e) => {
+    delete a[e];
+  });
+  Object.defineProperty(window, "localStorage", {
+    value: {
+      getItem: (e) => g(e),
+      setItem: (e, t) => u(e, t),
+      removeItem: (e) => i(e)
+    }
+  }), beforeEach(() => {
+    a = {};
   }), afterEach(() => {
     jest.clearAllMocks();
   }), test("should initialize with null when localStorage is empty", () => {
     const {
       result: e
-    } = n(() => l(t)), [o] = e.current;
-    expect(o).toBeNull(), expect(i).toHaveBeenCalledWith(t);
+    } = l(() => c(o)), [t] = e.current;
+    expect(t).toBeNull(), expect(g(o)).toBeUndefined();
   }), test("should initialize with value from localStorage if it exists and is valid", () => {
     const e = "agent-123";
-    localStorage.setItem(t, e);
+    a[o] = e;
     const {
-      result: o
-    } = n(() => l(t)), [r] = o.current;
-    expect(r).toBe(e), expect(i).toHaveBeenCalledWith(t);
+      result: t
+    } = l(() => c(o)), [n] = t.current;
+    expect(n).toBe(e), expect(g).toHaveBeenCalledWith(o);
   }), test("should return null if localStorage value contains invalid characters", () => {
     [
       `><script>alert('XSS Attack!');<\/script><div class="`,
@@ -27,48 +40,60 @@ describe("useLocalStorageAIAgents", () => {
       "${process.env.SECRET_KEY}",
       "../../../etc/passwd",
       "agent@123!"
-    ].forEach((o) => {
-      localStorage.setItem(t, o);
+    ].forEach((t) => {
+      a[o] = t;
       const {
-        result: r
-      } = n(() => l(t)), [s] = r.current;
-      expect(s).toBeNull(), expect(d).toHaveBeenCalledWith(t);
+        result: n
+      } = l(() => c(o)), [s] = n.current;
+      expect(s).toBeNull(), expect(i).toHaveBeenCalledWith(o);
     });
   }), test("should update localStorage when state changes", () => {
     const {
       result: e,
-      rerender: o
-    } = n(() => l(t));
-    let [r, s] = e.current;
-    expect(r).not.toBe(a), c(() => {
-      s(a);
-    }), o(), [r] = e.current, expect(r).toBe(a), expect(u).toHaveBeenCalledWith(t, a);
+      rerender: t
+    } = l(() => c(o));
+    let [n, s] = e.current;
+    expect(n).not.toBe(r), d(() => {
+      s(r);
+    }), t(), [n] = e.current, expect(n).toBe(r), expect(u).toHaveBeenCalledWith(o, r);
   }), test("should remove localStorage item when setting to null", () => {
     const {
       result: e,
-      rerender: o
-    } = n(() => l(t));
-    let [r, s] = e.current;
-    expect(r).toBeNull(), c(() => {
-      s(a);
-    }), o(), [r] = e.current, expect(r).toBe(a), c(() => {
+      rerender: t
+    } = l(() => c(o));
+    let [n, s] = e.current;
+    expect(n).toBeNull(), d(() => {
+      s(r);
+    }), t(), [n] = e.current, expect(n).toBe(r), d(() => {
       s(null);
-    }), o(), [r] = e.current, expect(r).toBeNull(), expect(d).toHaveBeenCalledWith(t);
+    }), t(), [n] = e.current, expect(n).toBeNull(), expect(i).toHaveBeenCalledWith(o);
   }), test("should handle valid alphanumeric characters with underscores and hyphens", () => {
     const e = "valid-agent_123", {
-      result: o,
-      rerender: r
-    } = n(() => l(t)), [, s] = o.current;
-    c(() => {
+      result: t,
+      rerender: n
+    } = l(() => c(o)), [, s] = t.current;
+    d(() => {
       s(e);
-    }), r();
-    const [g] = o.current;
-    expect(g).toBe(e), expect(u).toHaveBeenCalledWith(t, e);
+    }), n();
+    const [h] = t.current;
+    expect(h).toBe(e), expect(u).toHaveBeenCalledWith(o, e);
   }), test("should return null for whitespace-only values in localStorage", () => {
-    localStorage.setItem(t, "   ");
+    a[o] = "   ";
     const {
       result: e
-    } = n(() => l(t)), [o] = e.current;
-    expect(o).toBeNull();
+    } = l(() => c(o)), [t] = e.current;
+    expect(t).toBeNull();
+  }), test("should keep agent selection when localStorage full", () => {
+    u = jest.fn((h, p) => {
+      throw new Error("Quota exceeded");
+    });
+    const {
+      result: e,
+      rerender: t
+    } = l(() => c(o));
+    let [n, s] = e.current;
+    expect(n).not.toBe(r), d(() => {
+      s(r);
+    }), t(), [n] = e.current, expect(n).toBe(r), expect(u).toHaveBeenCalledWith(o, r), expect(i).toHaveBeenCalledWith(o);
   });
 });

package/dist/esm/lib/hooks/useLocalStorageAIAgents.js

@@ -1,21 +1,28 @@
-import { useState as l, useEffect as s } from "react";
-const c = (e, o = !0) => {
-  const [t, a] = l(() => {
+import { useState as l, useEffect as n } from "react";
+const s = (t, o = !0) => {
+  const [e, a] = l(() => {
     try {
-      const r = localStorage.getItem(e);
+      const r = localStorage.getItem(t);
       if (!r)
         return null;
       if (o && !/^[a-zA-Z0-9_-]+$/.test(r))
         throw new Error("Invalid characters in stored value");
       return r;
     } catch {
-      return localStorage.removeItem(e), null;
+      return localStorage.removeItem(t), null;
     }
   });
-  return s(() => {
-    t ? localStorage.setItem(e, t) : localStorage.removeItem(e);
-  }, [t, e]), [t, a];
+  return n(() => {
+    try {
+      if (e) {
+        localStorage.setItem(t, e);
+        return;
+      }
+    } catch {
+    }
+    localStorage.removeItem(t);
+  }, [e, t]), [e, a];
 };
 export {
-  c as default
+  s as default
 };

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@box/box-ai-agent-selector",
-    "version": "0.40.1",
+    "version": "0.40.3",
     "license": "SEE LICENSE IN LICENSE",
     "peerDependencies": {
         "@ariakit/react": "^0.4.15",
@@ -13,10 +13,10 @@
     "devDependencies": {
         "@ariakit/react": "^0.4.15",
         "@box/babel-plugin-target-attributes": "1.3.0",
-        "@box/blueprint-web": "^11.10.2",
-        "@box/blueprint-web-assets": "^4.42.0",
+        "@box/blueprint-web": "^11.10.3",
+        "@box/blueprint-web-assets": "^4.43.0",
         "@box/eslint-plugin-blueprint": "^1.0.3",
-        "@box/storybook-utils": "^0.12.1",
+        "@box/storybook-utils": "^0.12.2",
         "@testing-library/react": "^15.0.6",
         "react-intl": "^6.4.2"
     },