@boxcustodia/library 2.0.0-alpha.31 → 2.0.0-alpha.32

package/dist/hooks/use-pagination/use-pagination.cjs.js

@@ -1 +1 @@
-"use strict";Object.defineProperty(exports,Symbol.toStringTag,{value:"Module"});const n=require("react"),f=require("../internal/use-latest-ref.cjs.js"),G=10,O=1;function m(e,r){return e<=0||Number.isNaN(e)?(process.env.NODE_ENV!=="production"&&(!r||!r.current)&&(r&&(r.current=!0),console.warn(`[usePagination] pageSize must be a positive number, received ${e}. Coercing to 1.`)),1):Math.floor(e)}function U(e,r){return e<=0?0:Math.ceil(e/r)}function d(e,r){return r<=0||Number.isNaN(e)?1:Math.min(Math.max(1,Math.floor(e)),r)}function j({totalItems:e,pageSize:r,defaultPageSize:z=G,onPageSizeChange:b,page:C,defaultPage:A=O,onPageChange:T,onPaginationChange:_}){const[p,R]=n.useState(A),[S,h]=n.useState(z),x=n.useRef(!1),c=m(r??S,x),o=U(e,c),u=d(C??p,o),i=f.useLatestRef(u),y=f.useLatestRef(c),s=f.useLatestRef(o),L=f.useLatestRef(T),N=f.useLatestRef(b),D=f.useLatestRef(_),P=n.useRef(!1),l=n.useRef(null),E=n.useRef(null);n.useEffect(()=>{if(!P.current)return;const t=d(p,s.current);l.current!==t&&(l.current=t,L.current?.(t))},[p]),n.useEffect(()=>{if(!P.current)return;const t=m(S);E.current!==t&&(E.current=t,N.current?.(t))},[S]),n.useEffect(()=>{P.current&&D.current?.({page:u,pageSize:c})},[u,c]),n.useEffect(()=>{P.current=!0},[]);const F=n.useMemo(()=>{const t=g=>{const a=d(g,s.current);a!==i.current&&R(a)};return{goTo:g=>{s.current<=0||t(g)},next:()=>{i.current>=s.current||t(i.current+1)},prev:()=>{i.current<=1||t(i.current-1)},firstPage:()=>{s.current<=0||t(1)},lastPage:()=>{s.current<=0||t(s.current)},setPageSize:g=>{const a=m(g);a!==y.current&&(E.current=a,h(a),N.current?.(a),i.current!==1&&(l.current=1,L.current?.(1)),R(1))}}},[R,h]),v=o<=0?!0:u<=1,M=o<=0?!0:u>=o,q=e<=0?{start:0,end:0}:{start:(u-1)*c+1,end:Math.min(u*c,e)};return{page:u,pageSize:c,pageCount:o,isFirstPage:v,isLastPage:M,hasPrevPage:!v,hasNextPage:!M,range:q,...F}}exports.usePagination=j;
+"use strict";Object.defineProperty(exports,Symbol.toStringTag,{value:"Module"});const n=require("react"),o=require("../internal/use-latest-ref.cjs.js"),O=10,U=1;function d(e,r){return e<=0||Number.isNaN(e)?(process.env.NODE_ENV!=="production"&&(!r||!r.current)&&(r&&(r.current=!0),console.warn(`[usePagination] pageSize must be a positive number, received ${e}. Coercing to 1.`)),1):Math.floor(e)}function j(e,r){return e<=0?0:Math.ceil(e/r)}function h(e,r){return r<=0||Number.isNaN(e)?1:Math.min(Math.max(1,Math.floor(e)),r)}function V({totalItems:e,pageSize:r,defaultPageSize:z=O,onPageSizeChange:b,page:C,defaultPage:A=U,onPageChange:T,onPaginationChange:_}){const[P,S]=n.useState(A),[l,L]=n.useState(z),x=n.useRef(!1),s=d(r??l,x),i=j(e,s),c=h(C??P,i),f=o.useLatestRef(c),y=o.useLatestRef(P),D=o.useLatestRef(s),a=o.useLatestRef(i),E=o.useLatestRef(T),N=o.useLatestRef(b),F=o.useLatestRef(_),R=n.useRef(!1),p=n.useRef(null),m=n.useRef(null);n.useEffect(()=>{if(!R.current)return;const t=h(P,a.current);p.current!==t&&(p.current=t,E.current?.(t))},[P]),n.useEffect(()=>{if(!R.current)return;const t=d(l);m.current!==t&&(m.current=t,N.current?.(t))},[l]),n.useEffect(()=>{R.current&&F.current?.({page:c,pageSize:s})},[c,s]),n.useEffect(()=>{R.current=!0},[]);const q=n.useMemo(()=>{const t=g=>{const u=h(g,a.current);u!==f.current&&(y.current===u&&(p.current=u,E.current?.(u)),S(u))};return{goTo:g=>{a.current<=0||t(g)},next:()=>{f.current>=a.current||t(f.current+1)},prev:()=>{f.current<=1||t(f.current-1)},firstPage:()=>{a.current<=0||t(1)},lastPage:()=>{a.current<=0||t(a.current)},setPageSize:g=>{const u=d(g);u!==D.current&&(m.current=u,L(u),N.current?.(u),f.current!==1&&(p.current=1,E.current?.(1)),S(1))}}},[S,L]),v=i<=0?!0:c<=1,M=i<=0?!0:c>=i,G=e<=0?{start:0,end:0}:{start:(c-1)*s+1,end:Math.min(c*s,e)};return{page:c,pageSize:s,pageCount:i,isFirstPage:v,isLastPage:M,hasPrevPage:!v,hasNextPage:!M,range:G,...q}}exports.usePagination=V;

package/dist/hooks/use-pagination/use-pagination.es.js

@@ -1,83 +1,83 @@
-import { useState as b, useRef as P, useEffect as p, useMemo as V } from "react";
-import { useLatestRef as s } from "../internal/use-latest-ref.es.js";
-const Z = 10, $ = 1;
+import { useState as b, useRef as S, useEffect as l, useMemo as Z } from "react";
+import { useLatestRef as o } from "../internal/use-latest-ref.es.js";
+const $ = 10, j = 1;
 function d(e, t) {
   return e <= 0 || Number.isNaN(e) ? (process.env.NODE_ENV !== "production" && (!t || !t.current) && (t && (t.current = !0), console.warn(
     `[usePagination] pageSize must be a positive number, received ${e}. Coercing to 1.`
   )), 1) : Math.floor(e);
 }
-function j(e, t) {
+function k(e, t) {
   return e <= 0 ? 0 : Math.ceil(e / t);
 }
 function N(e, t) {
   return t <= 0 || Number.isNaN(e) ? 1 : Math.min(Math.max(1, Math.floor(e)), t);
 }
-function B({
+function H({
   totalItems: e,
   pageSize: t,
-  defaultPageSize: x = Z,
+  defaultPageSize: x = $,
   onPageSizeChange: A,
   page: L,
-  defaultPage: _ = $,
+  defaultPage: _ = j,
   onPageChange: D,
   onPaginationChange: F
 }) {
-  const [S, l] = b(_), [m, R] = b(x), T = P(!1), u = d(t ?? m, T), o = j(e, u), n = N(L ?? S, o), i = s(n), G = s(u), c = s(o), v = s(D), z = s(A), U = s(F), g = P(!1), h = P(null), E = P(null);
-  p(() => {
-    if (!g.current) return;
-    const r = N(S, c.current);
-    h.current !== r && (h.current = r, v.current?.(r));
-  }, [S]), p(() => {
-    if (!g.current) return;
-    const r = d(m);
-    E.current !== r && (E.current = r, z.current?.(r));
-  }, [m]), p(() => {
-    g.current && U.current?.({ page: n, pageSize: u });
-  }, [n, u]), p(() => {
-    g.current = !0;
+  const [g, m] = b(_), [h, v] = b(x), T = S(!1), c = d(t ?? h, T), i = k(e, c), u = N(L ?? g, i), s = o(u), G = o(g), U = o(c), a = o(i), E = o(D), z = o(A), y = o(F), P = S(!1), p = S(null), R = S(null);
+  l(() => {
+    if (!P.current) return;
+    const r = N(g, a.current);
+    p.current !== r && (p.current = r, E.current?.(r));
+  }, [g]), l(() => {
+    if (!P.current) return;
+    const r = d(h);
+    R.current !== r && (R.current = r, z.current?.(r));
+  }, [h]), l(() => {
+    P.current && y.current?.({ page: u, pageSize: c });
+  }, [u, c]), l(() => {
+    P.current = !0;
   }, []);
-  const y = V(() => {
+  const O = Z(() => {
     const r = (f) => {
-      const a = N(f, c.current);
-      a !== i.current && l(a);
+      const n = N(f, a.current);
+      n !== s.current && (G.current === n && (p.current = n, E.current?.(n)), m(n));
     };
     return {
       goTo: (f) => {
-        c.current <= 0 || r(f);
+        a.current <= 0 || r(f);
       },
       next: () => {
-        i.current >= c.current || r(i.current + 1);
+        s.current >= a.current || r(s.current + 1);
       },
       prev: () => {
-        i.current <= 1 || r(i.current - 1);
+        s.current <= 1 || r(s.current - 1);
       },
       firstPage: () => {
-        c.current <= 0 || r(1);
+        a.current <= 0 || r(1);
       },
       lastPage: () => {
-        c.current <= 0 || r(c.current);
+        a.current <= 0 || r(a.current);
       },
       setPageSize: (f) => {
-        const a = d(f);
-        a !== G.current && (E.current = a, R(a), z.current?.(a), i.current !== 1 && (h.current = 1, v.current?.(1)), l(1));
+        const n = d(f);
+        n !== U.current && (R.current = n, v(n), z.current?.(n), s.current !== 1 && (p.current = 1, E.current?.(1)), m(1));
       }
     };
-  }, [l, R]), M = o <= 0 ? !0 : n <= 1, C = o <= 0 ? !0 : n >= o, O = e <= 0 ? { start: 0, end: 0 } : {
-    start: (n - 1) * u + 1,
-    end: Math.min(n * u, e)
+  }, [m, v]), M = i <= 0 ? !0 : u <= 1, C = i <= 0 ? !0 : u >= i, V = e <= 0 ? { start: 0, end: 0 } : {
+    start: (u - 1) * c + 1,
+    end: Math.min(u * c, e)
   };
   return {
-    page: n,
-    pageSize: u,
-    pageCount: o,
+    page: u,
+    pageSize: c,
+    pageCount: i,
     isFirstPage: M,
     isLastPage: C,
     hasPrevPage: !M,
     hasNextPage: !C,
-    range: O,
-    ...y
+    range: V,
+    ...O
   };
 }
 export {
-  B as usePagination
+  H as usePagination
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@boxcustodia/library",
-  "version": "2.0.0-alpha.31",
+  "version": "2.0.0-alpha.32",
   "type": "module",
   "main": "dist/index.cjs.js",
   "module": "dist/index.es.js",

package/src/hooks/use-pagination/use-pagination.test.tsx

@@ -324,6 +324,32 @@ describe("usePagination — Controllable Page", () => {
     expect(onPageChange).toHaveBeenCalledWith(3);
   });
 
+  it("Scenario 21b — onPageChange fires when controlled prop diverges from internal state then user navigates back to page 1", () => {
+    // Regression: if the page prop is updated externally (no internal navigation),
+    // pageState stays at its default (1). A subsequent goTo(1) calls setPageState(1)
+    // which is a React no-op → useEffect never fires → onPageChange silently skipped.
+    const onPageChange = vi.fn();
+    let controlledPage = 1;
+    const { result, rerender } = renderHook(() =>
+      usePagination({
+        totalItems: 100,
+        defaultPageSize: 10,
+        page: controlledPage,
+        onPageChange,
+      }),
+    );
+
+    // Parent externally jumps to page 5 — no internal navigation, pageState stays 1
+    controlledPage = 5;
+    rerender();
+    expect(result.current.page).toBe(5);
+    onPageChange.mockClear();
+
+    // User clicks "First" — should fire onPageChange(1)
+    act(() => result.current.goTo(1));
+    expect(onPageChange).toHaveBeenCalledWith(1);
+  });
+
   it("Scenario 22 — onPageChange NOT fired on mount", () => {
     const onPageChange = vi.fn();
     renderHook(() =>

package/src/hooks/use-pagination/use-pagination.ts

@@ -136,6 +136,7 @@ export function usePagination({
 
   // Refs so memoized actions read live values without stale closures
   const pageRef = useLatestRef(page);
+  const pageStateRef = useLatestRef(pageState);
   const pageSizeRef = useLatestRef(pageSize);
   const pageCountRef = useLatestRef(pageCount);
   const onPageChangeRef = useLatestRef(onPageChange);
@@ -186,6 +187,14 @@ export function usePagination({
     const applyPage = (target: number) => {
       const clamped = clampPage(target, pageCountRef.current);
       if (clamped === pageRef.current) return; // no-op for same page
+      // In controlled mode, pageState may already equal `clamped` (diverged from
+      // pageProp), so setPageState would be a no-op and the useEffect that fires
+      // onPageChange would never run. Mirror the same direct-fire pattern used in
+      // setPageSize to guarantee the callback reaches the consumer.
+      if (pageStateRef.current === clamped) {
+        lastEmittedPageRef.current = clamped;
+        onPageChangeRef.current?.(clamped);
+      }
       setPageState(clamped);
     };