@hsu-react/ui 2.5.4 → 2.5.6

package/src/components/Chart/optionMemo.test.tsx

@@ -0,0 +1,232 @@
+import React, { useState } from "react";
+import { beforeEach, describe, expect, it, vi } from "vitest";
+import { act, render } from "@testing-library/react";
+
+/**
+ * 这一组用例盯的是 `chartOption` 那个 `useMemo` 的**依赖必须是值，不能是每次渲染
+ * 新建的对象**。
+ *
+ * 依赖里一旦混进新建对象（`...coreOption` 这个 rest 包、或写在解构默认值里的
+ * `= { show: false }` / `= ["#fff"]` 字面量），memo 就**恒不命中**：
+ * 父组件每渲染一次 → chartOption 换新引用 → effect 重跑 → `setOption(…, true)`
+ * 整个重设配置 —— 动画重播、鼠标悬浮态被清掉，用户直接看得见。
+ *
+ * 修法不是「让 effect 少跑」，而是让依赖回到值语义：
+ * - rest 包走 `useShallowStable`（内容浅相等就复用同一个引用）；
+ * - 解构默认值里的对象/数组字面量提到模块级常量。
+ *
+ * 所以每个组件都验两条：
+ * 1. 父组件重渲染、props 未变 → 不再 setOption；
+ * 2. 配置真变了（换数据）→ 仍然要 setOption。
+ */
+
+const chart = {
+  setOption: vi.fn(),
+  resize: vi.fn(),
+  dispose: vi.fn(),
+  getDataURL: vi.fn(() => "data:image/png;base64,AAAA"),
+  getOption: vi.fn(() => ({ dataZoom: [{ start: 0, end: 100 }] })),
+  on: vi.fn(),
+  off: vi.fn(),
+  dispatchAction: vi.fn(),
+  getZr: vi.fn(() => ({ on: vi.fn(), off: vi.fn() })),
+  convertToPixel: vi.fn(() => [0, 0]),
+  getWidth: vi.fn(() => 600),
+  getHeight: vi.fn(() => 400),
+};
+
+vi.mock("echarts", () => ({
+  init: () => chart,
+  use: () => {},
+  registerTheme: () => {},
+  graphic: {
+    LinearGradient: class {
+      constructor(..._args: unknown[]) {}
+    },
+  },
+}));
+vi.mock("echarts-gl", () => ({}));
+
+import ChartBar from "./Bar";
+import ChartLine from "./Line";
+import ChartPie from "./Pie";
+import ChartBubble from "./Bubble";
+import ChartHeatmap from "./Heatmap";
+import ChartPolar from "./Polar";
+import ChartRadar from "./Radar";
+import ChartCommon from "./Common";
+import ChartGauge from "./Gauge";
+import ChartPie3D from "./Pie/Pie3D";
+
+// Chart.Bubble 要等 ResizeObserver 报出容器尺寸才会 init，jsdom 的空实现永远不回调，
+// 这里补一个观察即回调的实现
+class FiringResizeObserver {
+  constructor(private cb: ResizeObserverCallback) {}
+  observe(target: Element) {
+    this.cb(
+      [
+        {
+          target,
+          contentRect: { width: 600, height: 400 },
+        } as unknown as ResizeObserverEntry,
+      ],
+      this as unknown as ResizeObserver,
+    );
+  }
+  unobserve() {}
+  disconnect() {}
+}
+(globalThis as { ResizeObserver?: unknown }).ResizeObserver =
+  FiringResizeObserver;
+
+beforeEach(() => {
+  vi.clearAllMocks();
+  vi.spyOn(Element.prototype, "getBoundingClientRect").mockReturnValue({
+    width: 600,
+    height: 400,
+    top: 0,
+    left: 0,
+    right: 600,
+    bottom: 400,
+    x: 0,
+    y: 0,
+    toJSON: () => ({}),
+  } as DOMRect);
+});
+
+const X_AXIS = ["一", "二", "三", "四"];
+const SERIES = [{ name: "A", value: [1, 2, 3, 4] }];
+const SERIES_2 = [{ name: "A", value: [9, 9, 9, 9] }];
+const PIE_DATA = [{ name: "a", value: 1 }];
+const PIE_DATA_2 = [{ name: "a", value: 2 }];
+const BUBBLE_DATA = [{ name: "a", value: 1 }];
+const BUBBLE_DATA_2 = [{ name: "a", value: 5 }];
+const HEATMAP_DATA: Array<[number, number, number]> = [[0, 0, 1]];
+const HEATMAP_DATA_2: Array<[number, number, number]> = [[0, 0, 7]];
+const RADAR_INDICATORS = [{ name: "a", max: 10 }];
+const RADAR_DATA = [1];
+const RADAR_DATA_2 = [8];
+const POLAR_SERIES = { value: 1, name: "a" };
+const POLAR_SERIES_2 = { value: 8, name: "a" };
+const COMMON_SERIES = [{ type: "bar" as const, data: [1, 2, 3] }];
+const COMMON_SERIES_2 = [{ type: "bar" as const, data: [4, 5, 6] }];
+const GAUGE_DATA = [{ value: 1, name: "a" }];
+const GAUGE_DATA_2 = [{ value: 6, name: "a" }];
+const PIE3D_DATA = [{ name: "a", value: 1 }];
+const PIE3D_DATA_2 = [{ name: "a", value: 4 }];
+
+/** 每个组件：[名字, 组件, 稳定 props, 会真正改变配置的 props] */
+const cases: Array<
+  [string, React.ComponentType<Record<string, unknown>>, Record<string, unknown>, Record<string, unknown>]
+> = [
+  [
+    "Chart.Bar",
+    ChartBar as React.ComponentType<Record<string, unknown>>,
+    { xAxisData: X_AXIS, seriesData: SERIES },
+    { seriesData: SERIES_2 },
+  ],
+  [
+    "Chart.Line",
+    ChartLine as React.ComponentType<Record<string, unknown>>,
+    { xAxisData: X_AXIS, seriesData: SERIES },
+    { seriesData: SERIES_2 },
+  ],
+  [
+    "Chart.Pie",
+    ChartPie as React.ComponentType<Record<string, unknown>>,
+    { seriesData: PIE_DATA },
+    { seriesData: PIE_DATA_2 },
+  ],
+  [
+    "Chart.Bubble",
+    ChartBubble as React.ComponentType<Record<string, unknown>>,
+    { data: BUBBLE_DATA },
+    { data: BUBBLE_DATA_2 },
+  ],
+  [
+    "Chart.Heatmap",
+    ChartHeatmap as React.ComponentType<Record<string, unknown>>,
+    { data: HEATMAP_DATA },
+    { data: HEATMAP_DATA_2 },
+  ],
+  [
+    "Chart.Polar",
+    ChartPolar as React.ComponentType<Record<string, unknown>>,
+    { seriesData: POLAR_SERIES },
+    { seriesData: POLAR_SERIES_2 },
+  ],
+  [
+    "Chart.Radar",
+    ChartRadar as React.ComponentType<Record<string, unknown>>,
+    { indicators: RADAR_INDICATORS, data: RADAR_DATA },
+    { data: RADAR_DATA_2 },
+  ],
+  [
+    "Chart.Gauge",
+    ChartGauge as React.ComponentType<Record<string, unknown>>,
+    { seriesData: GAUGE_DATA },
+    { seriesData: GAUGE_DATA_2 },
+  ],
+  [
+    "Chart.Pie.Three",
+    ChartPie3D as React.ComponentType<Record<string, unknown>>,
+    { pieData: PIE3D_DATA },
+    { pieData: PIE3D_DATA_2 },
+  ],
+  [
+    "Chart.Common",
+    ChartCommon as React.ComponentType<Record<string, unknown>>,
+    { xAxis: { type: "category" as const }, series: COMMON_SERIES },
+    { series: COMMON_SERIES_2 },
+  ],
+];
+
+describe("chartOption 的 useMemo 依赖必须是值", () => {
+  for (const [name, Comp, stableProps, changedProps] of cases) {
+    describe(name, () => {
+      it("父组件重渲染但配置未变 → 不重设 option", () => {
+        let bump: () => void = () => {};
+
+        function Consumer() {
+          const [, setTick] = useState(0);
+          bump = () => setTick((n) => n + 1);
+
+          return <Comp {...stableProps} />;
+        }
+
+        render(<Consumer />);
+        const initial = chart.setOption.mock.calls.length;
+        expect(initial).toBeGreaterThan(0);
+
+        act(() => bump());
+        act(() => bump());
+        act(() => bump());
+
+        expect(chart.setOption.mock.calls.length).toBe(initial);
+      });
+
+      it("配置真变了 → 仍然重设 option", () => {
+        let change: () => void = () => {};
+
+        function Consumer() {
+          const [changed, setChanged] = useState(false);
+          change = () => setChanged(true);
+
+          return (
+            <Comp
+              {...stableProps}
+              {...(changed ? changedProps : {})}
+            />
+          );
+        }
+
+        render(<Consumer />);
+        const initial = chart.setOption.mock.calls.length;
+
+        act(() => change());
+
+        expect(chart.setOption.mock.calls.length).toBeGreaterThan(initial);
+      });
+    });
+  }
+});

package/src/components/Select/_hooks/useSelectComposition.test.ts

@@ -0,0 +1,47 @@
+import { afterEach, beforeEach, describe, expect, it, vi } from "vitest";
+import { renderHook } from "@testing-library/react";
+
+import { useSelectComposition } from "./useSelectComposition";
+
+/**
+ * 这里的 onSearch 只被当作「要不要监听输入法事件」的真值判断，effect 体从不调它。
+ * 它直接进依赖数组，消费方传内联箭头就会每渲染一次拆装一次 window 监听。
+ */
+describe("useSelectComposition", () => {
+  let addSpy: ReturnType<typeof vi.spyOn>;
+
+  const countAdds = () =>
+    addSpy.mock.calls.filter(([type]) => type === "compositionstart").length;
+
+  beforeEach(() => {
+    addSpy = vi.spyOn(window, "addEventListener");
+  });
+  afterEach(() => {
+    addSpy.mockRestore();
+  });
+
+  it("没传 onSearch 不注册；从无到有传入时会注册", () => {
+    const { rerender } = renderHook(
+      ({ onSearch }: { onSearch?: (v: string) => void }) =>
+        useSelectComposition({ onSearch }),
+      { initialProps: {} as { onSearch?: (v: string) => void } },
+    );
+    expect(countAdds()).toBe(0);
+
+    rerender({ onSearch: () => {} });
+    expect(countAdds()).toBe(1);
+  });
+
+  it("onSearch 换引用不会反复拆装 window 监听", () => {
+    const { rerender } = renderHook(
+      ({ onSearch }: { onSearch?: (v: string) => void }) =>
+        useSelectComposition({ onSearch }),
+      { initialProps: { onSearch: () => {} } },
+    );
+    expect(countAdds()).toBe(1);
+
+    rerender({ onSearch: () => {} });
+    rerender({ onSearch: () => {} });
+    expect(countAdds()).toBe(1);
+  });
+});

package/src/components/Select/_hooks/useSelectComposition.ts

@@ -11,6 +11,7 @@ export function useSelectComposition({
   onSearch,
 }: UseSelectCompositionProps) {
   const [isComposing, setComposing] = useState<boolean>(false);
+  const hasOnSearch = !!onSearch;
 
   useEffect(() => {
     const compositionend = () => {
@@ -20,7 +21,7 @@ export function useSelectComposition({
       setComposing(true);
     };
 
-    if (onSearch) {
+    if (hasOnSearch) {
       window.addEventListener("compositionstart", compositionstart);
       window.addEventListener("compositionend", compositionend);
     }
@@ -29,7 +30,10 @@ export function useSelectComposition({
       window.removeEventListener("compositionstart", compositionstart);
       window.removeEventListener("compositionend", compositionend);
     };
-  }, [onSearch]);
+    // onSearch 在这里只被当作「要不要监听输入法事件」的真值判断，effect 体从不调它。
+    // 直接进依赖数组的话，消费方传内联箭头就会每渲染一次拆一次、装一次 window 监听；
+    // 只取「有没有传」这个布尔量，从无到有时照样会重新注册。
+  }, [hasOnSearch]);
 
   return { isComposing };
 }

package/src/components/Table/_hooks/useAutoScrolling.test.tsx

@@ -0,0 +1,95 @@
+import React, { useRef } from "react";
+import { afterEach, beforeEach, describe, expect, it, vi } from "vitest";
+import { act, render, waitFor } from "@testing-library/react";
+
+import useAutoScrolling from "./useAutoScrolling";
+
+/**
+ * onAutoScrollEndAdd 在这个 hook 里身兼两职：既是「有没有加载更多」的开关，
+ * 又是真正去加载的动作。整条 rAF 循环挂在它的引用上（loop 依赖它、启停 effect 依赖 loop），
+ * 消费方传内联箭头就会每渲染一次重启一次滚动循环 —— 表格滚到一半被打回起点。
+ */
+
+const DATA = [{ key: "1" }, { key: "2" }];
+
+function Host({
+  onAutoScrollEndAdd,
+}: {
+  onAutoScrollEndAdd?: () => Promise<boolean>;
+}) {
+  const ref = useRef<HTMLDivElement>(null);
+
+  useAutoScrolling({
+    autoScrolling: true,
+    ref,
+    dataSource: DATA,
+    onAutoScrollEndAdd,
+  });
+
+  return (
+    <div ref={ref}>
+      <div className="ant-table-body">
+        <table>
+          <tbody className="ant-table-tbody" />
+        </table>
+      </div>
+    </div>
+  );
+}
+
+describe("useAutoScrolling", () => {
+  let cancelSpy: ReturnType<typeof vi.spyOn>;
+
+  beforeEach(() => {
+    // 让 rAF 不真的跑，cancelAnimationFrame 就只会由 effect 的 cleanup 触发，
+    // 于是「循环有没有被重启」可以直接数出来
+    vi.spyOn(window, "requestAnimationFrame").mockImplementation(
+      () => 1 as unknown as number,
+    );
+    cancelSpy = vi
+      .spyOn(window, "cancelAnimationFrame")
+      .mockImplementation(() => {});
+  });
+  afterEach(() => {
+    vi.restoreAllMocks();
+  });
+
+  /** hook 靠 MutationObserver 把自己标成 ready，这里手动改一次 DOM 触发它 */
+  const makeReady = async (container: HTMLElement) => {
+    await act(async () => {
+      container
+        .querySelector(".ant-table-tbody")!
+        .appendChild(document.createElement("tr"));
+      await Promise.resolve();
+    });
+    await waitFor(() =>
+      expect(window.requestAnimationFrame).toHaveBeenCalled(),
+    );
+  };
+
+  it("onAutoScrollEndAdd 换引用不会重启滚动循环", async () => {
+    const { container, rerender } = render(
+      <Host onAutoScrollEndAdd={() => Promise.resolve(true)} />,
+    );
+    await makeReady(container);
+
+    const before = cancelSpy.mock.calls.length;
+
+    rerender(<Host onAutoScrollEndAdd={() => Promise.resolve(true)} />);
+    rerender(<Host onAutoScrollEndAdd={() => Promise.resolve(true)} />);
+
+    expect(cancelSpy.mock.calls.length).toBe(before);
+  });
+
+  it("从无到有传入 onAutoScrollEndAdd 时会接上加载更多（循环按新开关重建一次）", async () => {
+    const { container, rerender } = render(<Host />);
+    await makeReady(container);
+
+    const before = cancelSpy.mock.calls.length;
+
+    rerender(<Host onAutoScrollEndAdd={() => Promise.resolve(true)} />);
+
+    // 开关从 false 变 true，循环必须按新语义重建，否则「加载更多」永远接不上
+    expect(cancelSpy.mock.calls.length).toBeGreaterThan(before);
+  });
+});

package/src/components/Table/_hooks/useAutoScrolling.ts

@@ -1,6 +1,7 @@
 import { useMutationObserver } from "ahooks";
 import { useCallback, useEffect, useRef, useState } from "react";
 import { TableProps } from "..";
+import { useLatestRef } from "../../../hooks/useLatestRef";
 
 const DEFAULT_INTERVAL = 2000;
 const DEFAULT_SPEED = 25;
@@ -39,6 +40,13 @@ const useAutoScrolling = (props: UseAutoScrollingProps) => {
     autoScrollingOffset = 0,
   } = props;
 
+  // onAutoScrollEndAdd 在这里身兼两职：既是「有没有加载更多」的开关，又是真正去加载的动作。
+  // 整个 rAF 循环（loop → 依赖它 → 启停 effect 依赖 loop）都挂在它的引用上，
+  // 消费方传内联箭头就会每渲染一次重启一次滚动循环 —— 表格滚到一半被打回起点。
+  // 拆成两半：开关看布尔量（从无到有传入时仍会正确接上加载更多），动作取 ref 里的最新引用。
+  const onAutoScrollEndAddRef = useLatestRef(onAutoScrollEndAdd);
+  const hasOnAutoScrollEndAdd = !!onAutoScrollEndAdd;
+
   const validSpeed =
     autoScrollingSpeed > 0 ? autoScrollingSpeed : DEFAULT_SPEED;
   const smoothStep = (validSpeed * SMOOTH_SCROLL_TICK) / 1000;
@@ -324,12 +332,13 @@ const useAutoScrolling = (props: UseAutoScrollingProps) => {
   );
 
   const runLoadMore = useCallback(() => {
-    if (!onAutoScrollEndAdd || pendingLoadMoreRef.current) return;
+    const loadMore = onAutoScrollEndAddRef.current;
+    if (!loadMore || pendingLoadMoreRef.current) return;
     pendingLoadMoreRef.current = true;
     phaseRef.current = "loading";
     const token = ++loadTokenRef.current;
 
-    onAutoScrollEndAdd()
+    loadMore()
       .then((hasMore) => {
         if (token !== loadTokenRef.current) return;
         const currentBody = getBody();
@@ -352,7 +361,7 @@ const useAutoScrolling = (props: UseAutoScrollingProps) => {
         if (token !== loadTokenRef.current) return;
         pendingLoadMoreRef.current = false;
       });
-  }, [autoScrollLoop, getBody, onAutoScrollEndAdd]);
+  }, [autoScrollLoop, getBody, onAutoScrollEndAddRef]);
 
   const loop = useCallback(() => {
     rafRef.current = requestAnimationFrame(loop);
@@ -391,7 +400,7 @@ const useAutoScrolling = (props: UseAutoScrollingProps) => {
         // In seamless mode the scroll never actually reaches the bottom; just keep scrolling
         phaseRef.current = "idle";
       } else if (isAtBottom(body)) {
-        if (onAutoScrollEndAdd) {
+        if (hasOnAutoScrollEndAdd) {
           runLoadMore();
         } else {
           if (autoScrollLoop) {
@@ -457,7 +466,7 @@ const useAutoScrolling = (props: UseAutoScrollingProps) => {
     isAtBottom,
     isBodyScrollable,
     isSeamless,
-    onAutoScrollEndAdd,
+    hasOnAutoScrollEndAdd,
     ready,
     ref,
     runLoadMore,
@@ -506,7 +515,7 @@ const useAutoScrolling = (props: UseAutoScrollingProps) => {
       return;
     }
 
-    if (onAutoScrollEndAdd) {
+    if (hasOnAutoScrollEndAdd) {
       // In load-more mode, if scrolling had stopped, resume auto-scrolling when new data arrives
       if (phaseRef.current === "stopped") {
         phaseRef.current = autoScrollMode === "row" ? "wait" : "idle";
@@ -530,7 +539,7 @@ const useAutoScrolling = (props: UseAutoScrollingProps) => {
     interval,
     isBodyScrollable,
     isSeamless,
-    onAutoScrollEndAdd,
+    hasOnAutoScrollEndAdd,
   ]);
 
   useEffect(() => {

package/src/hooks/useShallowStable.ts

@@ -0,0 +1,65 @@
+import { useRef } from "react";
+
+/**
+ * 把一个**内容没变、但每次渲染都新建**的对象收敛成**引用恒定**的对象：
+ * 浅比较（自有键 ＋ `Object.is` 比值）相等就继续返回上一次那个引用。
+ *
+ * 用途只有一个：让它可以直接进 `useMemo` / `useEffect` 的依赖数组。
+ *
+ * React 的依赖数组按 `Object.is` 比引用，所以依赖里只要混进一个「每次渲染都新建
+ * 的对象」，memo 就**恒不命中**、effect 就**每渲染必重跑**。组件内部最容易出现
+ * 这种对象的地方是 rest 解构：
+ *
+ * ```tsx
+ * const { className, style, ...coreOption } = props; // coreOption 每次都是新对象
+ * const option = useMemo(() => ({ ...coreOption }), [coreOption]); // 永远不命中
+ * ```
+ *
+ * 消费方就算把每一个 prop 都 memo 好，也躲不掉——新对象是组件自己建的。
+ * 正确写法是让这个对象回到值语义：
+ *
+ * ```tsx
+ * const { className, style, ...restOption } = props;
+ * const coreOption = useShallowStable(restOption); // 内容不变 → 引用不变
+ * ```
+ *
+ * 注意这**不是**「缓存结果跳过副作用」：memo 该重算时照样重算、effect 该重跑时
+ * 照样重跑——只要 rest 里任何一个值真的变了，浅比较就不相等，新引用立刻透出去。
+ *
+ * 渲染期间读写 ref 的安全性：并发渲染下被丢弃的那次渲染可能已经把 ref 换成了它
+ * 自己的对象，但那个对象与当次 props 浅相等，所以后续渲染拿到的引用**内容永远
+ * 与当前 props 一致**，不会读到过期值。
+ */
+export function useShallowStable<T extends object>(value: T): T {
+  const ref = useRef(value);
+
+  if (!shallowEqual(ref.current, value)) {
+    ref.current = value;
+  }
+
+  return ref.current;
+}
+
+function shallowEqual(a: object, b: object) {
+  if (Object.is(a, b)) return true;
+
+  const aKeys = Object.keys(a);
+  const bKeys = Object.keys(b);
+  if (aKeys.length !== bKeys.length) return false;
+
+  for (const key of aKeys) {
+    if (!Object.prototype.hasOwnProperty.call(b, key)) return false;
+    if (
+      !Object.is(
+        (a as Record<string, unknown>)[key],
+        (b as Record<string, unknown>)[key],
+      )
+    ) {
+      return false;
+    }
+  }
+
+  return true;
+}
+
+export default useShallowStable;

package/src/layout/Menu/_hooks/useOnlyLvOneMenu.test.tsx

@@ -0,0 +1,56 @@
+import React, { useState } from "react";
+import { describe, expect, it, vi } from "vitest";
+import { render } from "@testing-library/react";
+import { MemoryRouter } from "react-router-dom";
+
+import { useOnlyLvOneMenu } from "./useOnlyLvOneMenu";
+import type { MenuType } from "..";
+
+const ITEMS = [
+  { key: "/orch", label: "编排", children: [{ key: "/orch/a", label: "A" }] },
+] as unknown as MenuType[];
+
+/**
+ * 这个 effect 干的是：路由变了就把当前一级菜单的子项交给消费方，并同步展开态。
+ * getCurrChildItems 是 effect 体里发出去的通知 —— 它进依赖数组，
+ * 配上每次都新建数组的 setOpenkeys([item.key])，就是标准的死循环配方。
+ */
+function Host({ onChildren }: { onChildren: (c: MenuType[]) => void }) {
+  const [, setOpenkeys] = useState<string[]>([]);
+  const [, setMenuKey] = useState("");
+  const [children, setChildren] = useState<MenuType[]>([]);
+
+  useOnlyLvOneMenu({
+    items: ITEMS,
+    onlyLvOneMenu: true,
+    // 内联箭头 + 每次都是新数组：最坏情况的消费方写法
+    getCurrChildItems: (c) => {
+      onChildren(c);
+      setChildren([...c]);
+    },
+    setOpenkeys,
+    setMenuKey,
+  });
+
+  return <div data-testid="count">{children.length}</div>;
+}
+
+describe("useOnlyLvOneMenu", () => {
+  it("内联 getCurrChildItems 回写 state 不会死循环", () => {
+    const spy = vi.fn();
+
+    const { getByTestId } = render(
+      <MemoryRouter
+        initialEntries={["/orch/a"]}
+        future={{ v7_startTransition: true, v7_relativeSplatPath: true }}
+      >
+        <Host onChildren={spy} />
+      </MemoryRouter>,
+    );
+
+    expect(getByTestId("count").textContent).toBe("1");
+    // 进了循环这里会是几十上百次，直到 React 抛 Maximum update depth exceeded
+    expect(spy.mock.calls.length).toBeLessThanOrEqual(2);
+    expect(spy).toHaveBeenCalledWith(ITEMS[0].children);
+  });
+});

package/src/layout/Menu/_hooks/useOnlyLvOneMenu.ts

@@ -1,4 +1,5 @@
 import { useEffect } from "react";
+import { useLatestRef } from "../../../hooks/useLatestRef";
 import { useLocation } from "react-router";
 import { MenuType } from "..";
 
@@ -21,6 +22,11 @@ export const useOnlyLvOneMenu = ({
   setMenuKey,
 }: UseOnlyLvOneMenuOptions) => {
   const location = useLocation();
+  // getCurrChildItems 是 effect 体里发出去的通知，本身进依赖数组是这一类死循环的标准配方：
+  // 消费方传内联箭头 → 每次渲染新引用 → effect 重跑 → setOpenkeys([item.key]) 每次都是
+  // 新数组必定触发重渲染 → 又是新引用 …… 直到 React 抛 Maximum update depth exceeded。
+  // 通知取 ref 里的最新引用，依赖数组只留真正的输入。
+  const getCurrChildItemsRef = useLatestRef(getCurrChildItems);
 
   useEffect(() => {
     if (onlyLvOneMenu) {
@@ -28,14 +34,14 @@ export const useOnlyLvOneMenu = ({
         (i) => i.key === "/" + location.pathname.split("/").filter(Boolean)[0]
       );
       if (item) {
-        getCurrChildItems?.(item.children || []);
+        getCurrChildItemsRef.current?.(item.children || []);
         setOpenkeys([item.key]);
         setMenuKey(item.key);
       }
     }
   }, [
     items,
-    getCurrChildItems,
+    getCurrChildItemsRef,
     location,
     onlyLvOneMenu,
     setOpenkeys,