fix(table-core): avoid no-op page index reset updates

Author: HaukeSchnau

.changeset/quiet-page-reset.md

@@ -0,0 +1,6 @@
+---
+'@tanstack/react-table': patch
+'@tanstack/table-core': patch
+---
+
+Avoid emitting a pagination page-index reset when the table is already at the reset page.

packages/table-core/src/features/RowPagination.ts

@@ -215,6 +215,16 @@ export const RowPagination: TableFeature = {
     let registered = false
     let queued = false
 
+    const getSafePageIndex = (pageIndex: number) => {
+      const maxPageIndex =
+        typeof table.options.pageCount === 'undefined' ||
+        table.options.pageCount === -1
+          ? Number.MAX_SAFE_INTEGER
+          : table.options.pageCount - 1
+
+      return Math.max(0, Math.min(pageIndex, maxPageIndex))
+    }
+
     table._autoResetPageIndex = () => {
       if (!registered) {
         table._queue(() => {
@@ -254,15 +264,9 @@ export const RowPagination: TableFeature = {
     }
     table.setPageIndex = (updater) => {
       table.setPagination((old) => {
-        let pageIndex = functionalUpdate(updater, old.pageIndex)
-
-        const maxPageIndex =
-          typeof table.options.pageCount === 'undefined' ||
-          table.options.pageCount === -1
-            ? Number.MAX_SAFE_INTEGER
-            : table.options.pageCount - 1
-
-        pageIndex = Math.max(0, Math.min(pageIndex, maxPageIndex))
+        const pageIndex = getSafePageIndex(
+          functionalUpdate(updater, old.pageIndex),
+        )
 
         return {
           ...old,
@@ -271,11 +275,20 @@ export const RowPagination: TableFeature = {
       })
     }
     table.resetPageIndex = (defaultState) => {
-      table.setPageIndex(
+      const pageIndex = getSafePageIndex(
         defaultState
           ? defaultPageIndex
           : (table.initialState?.pagination?.pageIndex ?? defaultPageIndex),
       )
+
+      const currentPageIndex =
+        table.getState().pagination?.pageIndex ?? defaultPageIndex
+
+      if (currentPageIndex === pageIndex) {
+        return
+      }
+
+      table.setPageIndex(pageIndex)
     }
     table.resetPageSize = (defaultState) => {
       table.setPageSize(

packages/table-core/tests/RowPagination.test.ts

@@ -0,0 +1,102 @@
+import { describe, expect, it, vi } from 'vitest'
+import {
+  ColumnDef,
+  createTable,
+  functionalUpdate,
+  getCoreRowModel,
+  PaginationState,
+} from '../src'
+
+type Person = {
+  name: string
+}
+
+const data: Person[] = [{ name: 'Ada' }]
+const columns: ColumnDef<Person>[] = [{ accessorKey: 'name' }]
+
+function createPaginationTable(
+  pagination: PaginationState,
+  onPaginationChange = vi.fn(),
+) {
+  const table = createTable<Person>({
+    data,
+    columns,
+    getCoreRowModel: getCoreRowModel(),
+    onPaginationChange,
+    onStateChange() {},
+    renderFallbackValue: '',
+    state: {
+      pagination,
+    },
+  })
+
+  return { table, onPaginationChange }
+}
+
+describe('RowPagination', () => {
+  it('does not emit a pagination change when resetting to the current page index', () => {
+    const { table, onPaginationChange } = createPaginationTable({
+      pageIndex: 0,
+      pageSize: 10,
+    })
+
+    table.resetPageIndex()
+
+    expect(onPaginationChange).not.toHaveBeenCalled()
+  })
+
+  it('emits a pagination change when resetting from a different page index', () => {
+    const { table, onPaginationChange } = createPaginationTable({
+      pageIndex: 1,
+      pageSize: 10,
+    })
+
+    table.resetPageIndex()
+
+    expect(onPaginationChange).toHaveBeenCalledOnce()
+    expect(
+      functionalUpdate(onPaginationChange.mock.calls[0][0], {
+        pageIndex: 1,
+        pageSize: 10,
+      }),
+    ).toEqual({
+      pageIndex: 0,
+      pageSize: 10,
+    })
+  })
+
+  it('does not emit an auto-reset pagination change when the page index is already reset', async () => {
+    const { table, onPaginationChange } = createPaginationTable({
+      pageIndex: 0,
+      pageSize: 10,
+    })
+
+    table._autoResetPageIndex()
+    await Promise.resolve()
+
+    table._autoResetPageIndex()
+    await Promise.resolve()
+
+    expect(onPaginationChange).not.toHaveBeenCalled()
+  })
+
+  it('does not emit an auto-reset pagination change when core row data changes and the page index is already reset', async () => {
+    const { table, onPaginationChange } = createPaginationTable({
+      pageIndex: 0,
+      pageSize: 10,
+    })
+
+    table.getRowModel()
+    await Promise.resolve()
+    onPaginationChange.mockClear()
+
+    table.setOptions((old) => ({
+      ...old,
+      data: [{ name: 'Grace' }],
+    }))
+    table.getRowModel()
+    await Promise.resolve()
+
+    expect(onPaginationChange).not.toHaveBeenCalled()
+  })
+})