src/animation/AnimationStateMachine.ts (1)

15-23: ⚡ Quick win

Update JSDoc comments to reflect the renamed interface.

The JSDoc comments still reference "RechartsAnimation" instead of "AnimationStateMachine" in the state descriptions (lines 17-20). For consistency and clarity, these should be updated to match the renamed interface.

📝 Proposed JSDoc update

   /**
    * Returns the state machine current state
-   * - `init`:       RechartsAnimation had just been created. It immediately calls `onAnimationStart`
-   * - `pending`:    RechartsAnimation is now paused for `animationBegin` milliseconds until the transition begins
-   * - `active`:     RechartsAnimation is transitioning items on screen
-   * - `completed`:  RechartsAnimation has completed its transition and executed `onAnimationEnd`.
+   * - `init`:       AnimationStateMachine had just been created. It immediately calls `onAnimationStart`
+   * - `pending`:    AnimationStateMachine is now paused for `animationBegin` milliseconds until the transition begins
+   * - `active`:     AnimationStateMachine is transitioning items on screen
+   * - `completed`:  AnimationStateMachine has completed its transition and executed `onAnimationEnd`.
    *                 This state is final and the animation is no longer allowed to transition to other states.
    */

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@src/animation/AnimationStateMachine.ts` around lines 15 - 23, Update the
JSDoc comments for the getState() method in AnimationStateMachine to replace the
outdated references to "RechartsAnimation" with "AnimationStateMachine".
Specifically, update lines 17-20 in the comment block where it describes the
different animation states (init, pending, active, completed) to ensure the
comments refer to AnimationStateMachine instead of the old RechartsAnimation
name for consistency and clarity.

src/animation/JavascriptAnimate.tsx (1)

57-57: ⚡ Quick win

Rename local variable for consistency with CSSTransitionAnimate.

The local variable is still named animationManager, but the prop and hook have been renamed to animationController. In the parallel file CSSTransitionAnimate.tsx (line 68), the local variable was renamed to match the new naming. For consistency within this PR and to align variable names with prop/hook names, consider renaming this variable to animationController.

♻️ Proposed refactor

-  const animationManager = useAnimationController(props.animationController);
+  const animationController = useAnimationController(props.animationController);
   const [style, setStyle] = useState<number>(isActive ? from : to);

And update the usage and dependency array:

-    return animationManager(timeoutController, animation, setStyle);
-  }, [animationManager, animationId, isActive, canBegin, duration, easing, begin, onAnimationStart, onAnimationEnd]);
+    return animationController(timeoutController, animation, setStyle);
+  }, [animationController, animationId, isActive, canBegin, duration, easing, begin, onAnimationStart, onAnimationEnd]);

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@src/animation/JavascriptAnimate.tsx` at line 57, The local variable
`animationManager` assigned from the `useAnimationController` hook call should
be renamed to `animationController` to maintain consistency with the prop name
and hook name, aligning with the changes already made in the parallel file
`CSSTransitionAnimate.tsx`. Find the variable declaration of `animationManager`
at the `useAnimationController(props.animationController)` call and rename it to
`animationController` throughout the component, including all references to this
variable and any dependency arrays that include this variable.