types: align legacy menu item contract

Author: ZQDesigned

src/MenuItem.tsx

@@ -9,7 +9,7 @@ import PrivateContext from './context/PrivateContext';
 import useActive from './hooks/useActive';
 import useDirectionStyle from './hooks/useDirectionStyle';
 import Icon from './Icon';
-import type { MenuInfo, ItemData, MenuItemType } from './interface';
+import type { LegacyMenuItemInfo, MenuInfo, ItemData, MenuItemType } from './interface';
 import { warnItemProp } from './utils/warnUtil';
 
 export interface MenuItemProps
@@ -44,9 +44,8 @@ type LegacyMenuItemProps = Omit<React.ComponentProps<typeof Overflow.Item>, 'tit
   onTitleClick?: () => void;
 };
 
-type LegacyMenuItemHandle = {
+type LegacyMenuItemHandle = Omit<LegacyMenuItemInfo, 'props'> & {
   props: LegacyMenuItemProps;
-  element: HTMLLIElement | null;
 };
 
 // Keep exposing a legacy-compatible handle for deprecated `info.item`.
@@ -178,7 +177,7 @@ const InternalMenuItem = React.forwardRef((props: MenuItemProps, ref: React.Ref<
       key: eventKey,
       // Note: For legacy code is reversed which not like other antd component
       keyPath: [...connectedKeys].reverse(),
-      item: legacyMenuItemRef.current as unknown as React.ReactInstance,
+      item: legacyMenuItemRef.current,
       domEvent: e,
       itemData: propsItemData || itemData,
     };

src/interface.ts

@@ -103,11 +103,23 @@ export interface RenderIconInfo {
 
 export type RenderIconType = React.ReactNode | ((props: RenderIconInfo) => React.ReactNode);
 
+/**
+ * @deprecated Compatibility handle for deprecated `info.item`.
+ * Avoid relying on this shape since it will be removed in a future major version.
+ */
+export interface LegacyMenuItemInfo {
+  props: {
+    eventKey?: string;
+    [key: string]: unknown;
+  };
+  element: HTMLLIElement | null;
+}
+
 export interface MenuInfo {
   key: string;
   keyPath: string[];
   /** @deprecated This will not support in future. You should avoid to use this */
-  item: React.ReactInstance;
+  item: LegacyMenuItemInfo;
   domEvent: React.MouseEvent<HTMLElement> | React.KeyboardEvent<HTMLElement>;
   itemData: ItemData;
 }

src/utils/warnUtil.ts

@@ -1,10 +1,11 @@
 import { warning } from '@rc-component/util';
+import type { LegacyMenuItemInfo } from '../interface';
 
 /**
  * `onClick` still exposes deprecated `info.item` for backward compatibility.
  * Keep warning since function components no longer provide a React node instance.
  */
-export function warnItemProp<T extends { item: React.ReactInstance }>({ item, ...restInfo }: T): T {
+export function warnItemProp<T extends { item: LegacyMenuItemInfo }>({ item, ...restInfo }: T): T {
   Object.defineProperty(restInfo, 'item', {
     get: () => {
       warning(