From bb66bc49506019d5607f67ebb3ec2cd2cc26bc4e Mon Sep 17 00:00:00 2001 From: talksik Date: Sun, 12 Apr 2026 09:36:53 -0700 Subject: [PATCH] refactor: remove unnecessary refs --- js/src/hooks/use-channel.ts | 12 +++--------- 1 file changed, 3 insertions(+), 9 deletions(-) diff --git a/js/src/hooks/use-channel.ts b/js/src/hooks/use-channel.ts index 0cc1fd7..2971ed4 100644 --- a/js/src/hooks/use-channel.ts +++ b/js/src/hooks/use-channel.ts @@ -22,12 +22,6 @@ export function useChannel(channelId: string | null): UseChannelResult { const [presence, setPresence] = useState([]); const [messages, setMessages] = useState([]); - // Keep a ref to avoid re-subscribing when sendMessage changes - const clientRef = useRef(client); - const channelRef = useRef(channelId); - clientRef.current = client; - channelRef.current = channelId; - useEffect(() => { if (!client || !channelId) { setPresence([]); @@ -80,11 +74,11 @@ export function useChannel(channelId: string | null): UseChannelResult { const sendMessage = useCallback( (payload: unknown) => { - if (clientRef.current && channelRef.current) { - clientRef.current.sendMessage(channelRef.current, payload); + if (client && channelId) { + client?.sendMessage(channelId, payload); } }, - [], + [client, channelId], ); return { presence, messages, sendMessage };