fix: [UIE-10326] - IAM / Entities pagination on page refresh

packages/manager/.changeset/pr-13451-fixed-1772457963491.md

@@ -0,0 +1,5 @@
+---
+"@linode/manager": Fixed
+---
+
+IAM / Entities & Roles client-side pagination on page refresh ([#13451](https://github.com/linode/manager/pull/13451))

packages/manager/src/hooks/usePaginationV2.test.ts

@@ -804,7 +804,7 @@ describe('usePaginationV2', () => {
       expect(mockNavigate).not.toHaveBeenCalled();
     });
 
-    it('should handle page clamping to 1 for empty data', async () => {
+    it('should not reset page when data is empty (e.g. still loading) to preserve URL on reload', async () => {
       queryClient.setQueryData(['profile', 'preferences'], {
         pageSizes: { 'test-key': 25 },
       });
@@ -826,9 +826,9 @@ describe('usePaginationV2', () => {
         expect(result.current.page).toBe(1);
       });
 
-      await waitFor(() => {
-        expect(mockNavigate).toHaveBeenCalled();
-      });
+      // Should not trigger navigation when totalCount is 0 so that page from URL
+      // is preserved on reload while data is still loading
+      expect(mockNavigate).not.toHaveBeenCalled();
     });
 
     it('should auto-reset page when data changes and current page becomes invalid', async () => {

packages/manager/src/hooks/usePaginationV2.ts

@@ -143,10 +143,15 @@ export const usePaginationV2 = <T extends TableSearchParams, D = unknown>({
   }, [clientSidePaginationData, clampedPage, pageSize]);
 
   React.useEffect(() => {
-    if (paginatedData !== undefined && clampedPage !== page) {
+    if (
+      paginatedData !== undefined &&
+      totalCount !== undefined &&
+      totalCount > 0 &&
+      clampedPage !== page
+    ) {
       setPage(clampedPage);
     }
-  }, [clampedPage, page, paginatedData, setPage]);
+  }, [clampedPage, page, paginatedData, setPage, totalCount]);
 
   return {
     handlePageChange: setPage,