refactor(useThrottledCallback): use named function in 'useEffect', replace 'useMemo' with 'usePreservedReference', and add leading+trailing test

Author: sukvvon

packages/core/src/hooks/useThrottledCallback/useThrottledCallback.spec.ts

@@ -83,6 +83,20 @@ describe('useThrottledCallback', () => {
     expect(onChange).not.toBeCalled();
   });
 
+  it('should handle leading and trailing edges together', () => {
+    const onChange = vi.fn();
+    const { result } = renderHookSSR(() =>
+      useThrottledCallback({ onChange, timeThreshold: 100, edges: ['leading', 'trailing'] })
+    );
+
+    result.current(true);
+    expect(onChange).toBeCalledTimes(1);
+    expect(onChange).toBeCalledWith(true);
+
+    vi.advanceTimersByTime(100);
+    expect(onChange).toBeCalledTimes(1);
+  });
+
   it('should handle value toggling', () => {
     const onChange = vi.fn();
     const { result } = renderHookSSR(() => useThrottledCallback({ onChange, timeThreshold: 100 }));

packages/core/src/hooks/useThrottledCallback/useThrottledCallback.ts

@@ -1,6 +1,7 @@
-import { useCallback, useEffect, useMemo, useRef } from 'react';
+import { useCallback, useEffect, useRef } from 'react';
 
 import { usePreservedCallback } from '../usePreservedCallback/index.ts';
+import { usePreservedReference } from '../usePreservedReference/index.ts';
 import { throttle } from '../useThrottle/throttle.ts';
 
 type ThrottleOptions = {
@@ -39,16 +40,14 @@ export function useThrottledCallback({
   const handleChange = usePreservedCallback(onChange);
   const ref = useRef({ value: false, clearPreviousThrottle: () => {} });
 
-  useEffect(() => {
+  useEffect(function cleanupThrottleOnUnmount() {
     const current = ref.current;
     return () => {
       current.clearPreviousThrottle();
     };
   }, []);
 
-  const preservedEdges = useMemo(() => {
-    return edges;
-  }, [edges]);
+  const preservedEdges = usePreservedReference(edges);
 
   return useCallback(
     (nextValue: boolean) => {