fix: align types and remove unneeded deps (#12)

* fix: align types and remove unneeded deps

* remove unneeded arg

* remove from test

* remove return null

src/CustomCursor.tsx

@@ -94,7 +94,7 @@ export const CustomCursor: React.FC<CustomCursorProps> = React.memo(
   }) => {
     const { position, setPosition, targetPosition, isVisible } =
       useMousePosition(containerRef, offsetX, offsetY);
-    useSmoothAnimation(position, targetPosition, smoothFactor, setPosition);
+    useSmoothAnimation(targetPosition, smoothFactor, setPosition);
 
     const [portalContainer, setPortalContainer] =
       React.useState<HTMLElement | null>(null);

src/hooks/useSmoothAnimation.ts

@@ -4,11 +4,10 @@ import { Position, TargetPosition } from '../types';
 const SMOOTHING_THRESHOLD = 0.1;
 
 export function useSmoothAnimation(
-  position: Position,
   targetPosition: TargetPosition,
   smoothFactor: number,
   setPosition: React.Dispatch<React.SetStateAction<Position>>
-) {
+): void {
   // Memoize the smoothing calculation
   const calculateNewPosition = useCallback(
     (currentPosition: Position) => {
@@ -69,20 +68,6 @@ export function useSmoothAnimation(
       return;
     }
 
-    // Skip animation if position is null
-    if (position.x === null || position.y === null) {
-      return;
-    }
-
     return animate();
-  }, [
-    smoothFactor,
-    position.x,
-    position.y,
-    targetPosition,
-    animate,
-    setPosition,
-  ]);
-
-  return null;
+  }, [smoothFactor, targetPosition, animate, setPosition]);
 }

src/index.ts

@@ -1,3 +1,5 @@
 export { default as CustomCursor } from './CustomCursor';
 // If you have types to export:
 export type { CustomCursorProps } from './CustomCursor';
+
+export type { Position } from './types';

src/types.ts

@@ -1,5 +1,3 @@
-import type { ReactNode } from 'react';
-
 export type Position = {
   x: number | null;
   y: number | null;

test/hooks/useSmoothAnimation.test.tsx

@@ -12,12 +12,9 @@ describe('useSmoothAnimation', () => {
 
   it('smoothly animates position changes', () => {
     const setPosition = jest.fn();
-    const position = { x: 0, y: 0 };
     const targetPosition = { x: 100, y: 100 };
 
-    renderHook(() =>
-      useSmoothAnimation(position, targetPosition, 0.5, setPosition)
-    );
+    renderHook(() => useSmoothAnimation(targetPosition, 0.5, setPosition));
 
     jest.advanceTimersByTime(16); // One frame
     expect(setPosition).toHaveBeenCalled();