From 5cd2811b9dc3e2fefbbc7893ab911156643a7826 Mon Sep 17 00:00:00 2001 From: jh-block Date: Thu, 12 Mar 2026 15:16:14 +0100 Subject: [PATCH] fix: remove unnecessary onFinish from useCallback dependency arrays MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit ESLint flagged onFinish as an unnecessary dependency in handleSubmit, submitElicitationResponse, and onMessageUpdate — it's already captured via the submitToSession closure. --- ui/desktop/src/hooks/useChatStream.ts | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/ui/desktop/src/hooks/useChatStream.ts b/ui/desktop/src/hooks/useChatStream.ts index d4f34381b8..c0415dca7b 100644 --- a/ui/desktop/src/hooks/useChatStream.ts +++ b/ui/desktop/src/hooks/useChatStream.ts @@ -647,7 +647,7 @@ export function useChatStream({ await submitToSession(sessionId, newMessage, currentMessages); }, - [sessionId, onFinish, submitToSession] + [sessionId, submitToSession] ); const submitElicitationResponse = useCallback( @@ -668,7 +668,7 @@ export function useChatStream({ await submitToSession(sessionId, responseMessage, currentMessages); }, - [sessionId, onFinish, submitToSession] + [sessionId, submitToSession] ); const setRecipeUserParams = useCallback( @@ -815,7 +815,7 @@ export function useChatStream({ }); } }, - [sessionId, handleSubmit, onFinish, submitToSession] + [sessionId, handleSubmit, submitToSession] ); const setChatState = useCallback((newState: ChatState) => {