[material-ui][Pagination] getItemAriaLabel called with null page number contrary to type definition

[sydneyjodon-wk]

Steps to reproduce

Link to live example: https://codesandbox.io/p/devbox/88qvg2?file=%2Fsrc%2FDemo.tsx

Steps:

1. Use getItemAriaLabel
2. Check page args that are passed in on the initial render of Pagination

Current behavior

Hello 👋

It looks like the getItemAriaLabel prop on Pagination is typed as:

getItemAriaLabel?: (type: UsePaginationItem['type'], page: number, selected: boolean) => string;

with the non-null page param, but null seems to be coming through on initial render of Pagination (see demo):

###

Expected behavior

Would it be possible to either change the behavior of passing null in for page or else change the typing of getItemAriaLabel to make it clear that null is an option for page?

Context

We are trying to use strict null safety

Your environment

See https://codesandbox.io/p/devbox/88qvg2?file=%2Fsrc%2FDemo.tsx

Thank you!

Search keywords: getItemAriaLabel page null

[siriwatknp]

That's true.

Here is a suggested fix, PR welcome:

diff --git a/packages/mui-material/src/Pagination/Pagination.d.ts b/packages/mui-material/src/Pagination/Pagination.d.ts
index a20361d6b5..50d83916d3 100644
--- a/packages/mui-material/src/Pagination/Pagination.d.ts
+++ b/packages/mui-material/src/Pagination/Pagination.d.ts
@@ -46,7 +46,11 @@ export interface PaginationProps
    * @param {bool} selected If true, the current page is selected.
    * @returns {string}
    */
-  getItemAriaLabel?: (type: UsePaginationItem['type'], page: number, selected: boolean) => string;
+  getItemAriaLabel?: (
+    type: UsePaginationItem['type'],
+    page: UsePaginationItem['page'],
+    selected: boolean,
+  ) => string;
 
   /**
    * Render the item.

[sydneyjodon-wk]

@siriwatknp thank you! Here's that PR #43399