From ac927a97cf81d2f33fbc4a34f6b05e12fa0f0ebf Mon Sep 17 00:00:00 2001 From: Claude Date: Sun, 30 Aug 2026 10:52:33 +0000 Subject: [PATCH] refactor(hooks): drop unused parameters from follow callback Both consumers of useFollowComponent call the returned callback with no arguments, so the ref and offset overrides were dead. Removing them also drops the guard on refs that can never be falsy. --- apps/client/src/common/hooks/useFollowComponent.ts | 11 +++-------- 1 file changed, 3 insertions(+), 8 deletions(-) diff --git a/apps/client/src/common/hooks/useFollowComponent.ts b/apps/client/src/common/hooks/useFollowComponent.ts index b70341cb7..3e3afcb5d 100644 --- a/apps/client/src/common/hooks/useFollowComponent.ts +++ b/apps/client/src/common/hooks/useFollowComponent.ts @@ -57,14 +57,9 @@ export default function useFollowComponent({ } }, [followTrigger, doFollow, followRef, scrollRef, setScrollFlag, topOffset]); - const scrollToRefComponent = useCallback( - (componentRef = followRef, containerRef = scrollRef, offset?: number) => { - if (componentRef && containerRef) { - scrollToComponent(componentRef, containerRef, offset ?? resolveTopOffset(topOffset)); - } - }, - [followRef, scrollRef, topOffset], - ); + const scrollToRefComponent = useCallback(() => { + scrollToComponent(followRef, scrollRef, resolveTopOffset(topOffset)); + }, [followRef, scrollRef, topOffset]); return scrollToRefComponent; }