refactor: change recommendPage to info object to avoid parameter pollution

Co-Authored-By: Claude <noreply@anthropic.com>

Author: 刘欢

docs/examples/jumper.tsx

@@ -9,9 +9,10 @@ function onShowSizeChange(current, pageSize) {
   console.log(pageSize);
 }
 
-function onChange(current, pageSize) {
+function onChange(current, pageSize, info) {
   console.log('onChange:current=', current);
   console.log('onChange:pageSize=', pageSize);
+  console.log('onChange:recommendPage=', info?.recommendPage);
 }
 
 const App = () => (

src/Pagination.tsx

@@ -209,7 +209,7 @@ const Pagination: React.FC<PaginationProps> = (props) => {
     setInternalInputVal(nextCurrent);
     onShowSizeChange?.(current, size);
     setCurrent(nextCurrent);
-    onChange?.(nextCurrent, size, recommendPage);
+    onChange?.(nextCurrent, size, { recommendPage });
   }
 
   function handleChange(page: number) {

src/interface.ts

@@ -58,7 +58,11 @@ export interface PaginationData {
 
 export interface PaginationProps
   extends Partial<PaginationData>, React.AriaAttributes {
-  onChange?: (page: number, pageSize: number, recommendPage?: number) => void;
+  onChange?: (
+    page: number,
+    pageSize: number,
+    info?: { recommendPage?: number },
+  ) => void;
   onShowSizeChange?: (current: number, size: number) => void;
   itemRender?: (
     page: number,

tests/index.test.tsx

@@ -483,7 +483,7 @@ describe('current value on onShowSizeChange when total is 0', () => {
     fireEvent.keyDown(input, { key: 'Down', keyCode: 40, which: 40 });
     fireEvent.keyDown(input, { key: 'Enter', keyCode: 13, which: 13 });
     expect(onShowSizeChange).toHaveBeenLastCalledWith(1, 20);
-    expect(onChange).toHaveBeenLastCalledWith(1, 20, 1);
+    expect(onChange).toHaveBeenLastCalledWith(1, 20, { recommendPage: 1 });
   });
 
   it('when total is 0, pager should show `1` and being disabled', () => {

tests/simple.test.tsx

@@ -131,7 +131,7 @@ describe('simple Pagination', () => {
     const pageSize1 = container.querySelectorAll('.rc-select-item')[0];
     fireEvent.click(pageSize1);
     expect(onChange).toHaveBeenCalled();
-    expect(onChange).toHaveBeenLastCalledWith(1, 10, 1);
+    expect(onChange).toHaveBeenLastCalledWith(1, 10, { recommendPage: 1 });
   });
 
   it('should support keyboard event', () => {

tests/sizer.test.tsx

@@ -67,7 +67,7 @@ describe('Pagination with sizer', () => {
     const pageSize1 = container.querySelectorAll('.rc-select-item')[0];
     fireEvent.click(pageSize1);
     expect(onChange).toHaveBeenCalled();
-    expect(onChange).toHaveBeenLastCalledWith(1, 10, 1);
+    expect(onChange).toHaveBeenLastCalledWith(1, 10, { recommendPage: 1 });
   });
 
   // https://github.com/ant-design/ant-design/issues/26580