fix: address review feedback — stale closure and unused prop

- Save hoveredIndex to local variable before clearing to avoid
  confusing stale closure read (Cursor Bugbot + Greptile feedback)
- Remove unused onMouseLeave prop from IconButtonProps since
  mouse-leave is handled by the container div

AI Disclosure: This commit was authored by Claude Opus 4.6 (Anthropic),
an AI agent operated by Maxwell Calkin (@MaxwellCalkin).

Author: MaxwellCalkin

apps/sim/app/(landing)/components/hero/components/icon-button.tsx

@@ -7,7 +7,6 @@ interface IconButtonProps {
   children: React.ReactNode
   onClick?: () => void
   onMouseEnter?: () => void
-  onMouseLeave?: () => void
   style?: React.CSSProperties
   'aria-label': string
   isActive?: boolean
@@ -17,7 +16,6 @@ export function IconButton({
   children,
   onClick,
   onMouseEnter,
-  onMouseLeave,
   style,
   'aria-label': ariaLabel,
   isActive = false,
@@ -28,7 +26,6 @@ export function IconButton({
       aria-label={ariaLabel}
       onClick={onClick}
       onMouseEnter={onMouseEnter}
-      onMouseLeave={onMouseLeave}
       className='relative flex items-center justify-center rounded-xl p-2 outline-none'
       style={style}
     >

apps/sim/app/(landing)/components/hero/hero.tsx

@@ -245,11 +245,12 @@ export default function Hero() {
    * Handle mouse leave on icon container
    */
   const handleIconContainerMouseLeave = () => {
+    const lastIndex = hoveredIndex
     setIsUserHovering(false)
     setHoveredIndex(null)
     // Start from the next icon after the last hovered one
-    if (hoveredIndex !== null) {
-      setAutoHoverIndex((hoveredIndex + 1) % visibleIconCount)
+    if (lastIndex !== null) {
+      setAutoHoverIndex((lastIndex + 1) % visibleIconCount)
     }
   }