fix: setting local state for first tool use set if no models, then user can click and change tool use

Author: alashchev17

refact-agent/gui/src/components/ChatForm/ChatControls.tsx

@@ -35,6 +35,7 @@ import {
   setAutomaticPatch,
   setChatMode,
   setEnabledCheckpoints,
+  setToolUse,
 } from "../../features/Chat/Thread";
 import { useAppSelector, useAppDispatch, useCapsForToolUse } from "../../hooks";
 import { getChatById } from "../../features/History/historySlice";
@@ -355,14 +356,14 @@ export const ChatControls: React.FC<ChatControlsProps> = ({
   host,
 }) => {
   const refs = useTourRefs();
+  const dispatch = useAppDispatch();
   const isStreaming = useAppSelector(selectIsStreaming);
   const isWaiting = useAppSelector(selectIsWaiting);
   const messages = useAppSelector(selectMessages);
   const toolUse = useAppSelector(selectToolUse);
-  const { handleManualToolUseChange } = useCapsForToolUse();
   const onSetToolUse = useCallback(
-    (value: ToolUse) => handleManualToolUseChange(value),
-    [handleManualToolUseChange],
+    (value: ToolUse) => dispatch(setToolUse(value)),
+    [dispatch],
   );
 
   const showControls = useMemo(

refact-agent/gui/src/hooks/useCapsForToolUse.ts

@@ -25,7 +25,7 @@ const PAID_AGENT_LIST = [
 ];
 
 export function useCapsForToolUse() {
-  const [wasInteracted, setWasInteracted] = useState(false);
+  const [wasAdjusted, setWasAdjusted] = useState(false);
   const caps = useGetCapsQuery();
   const toolUse = useAppSelector(selectThreadToolUse);
   const usage = useAgentUsage();
@@ -129,39 +129,31 @@ export function useCapsForToolUse() {
     };
 
     const handleAutomaticToolUseChange = () => {
-      if (!caps.isSuccess || wasInteracted) return;
+      if (!caps.isSuccess || wasAdjusted) return;
 
       const newToolUse = determineNewToolUse();
       if (newToolUse) {
         dispatch(setToolUse(newToolUse));
+        setWasAdjusted(true);
       }
     };
 
     handleAutomaticToolUseChange();
   }, [
     dispatch,
-    wasInteracted,
+    wasAdjusted,
     caps.isSuccess,
     toolUse,
     modelsSupportingAgent,
     modelsSupportingTools,
   ]);
 
-  const handleManualToolUseChange = useCallback(
-    (newToolUse: ToolUse) => {
-      setWasInteracted(true);
-      dispatch(setToolUse(newToolUse));
-    },
-    [dispatch],
-  );
-
   return {
     usableModels,
     usableModelsForPlan,
     currentModel,
     setCapModel,
     isMultimodalitySupportedForCurrentModel,
     loading: !caps.data && (caps.isFetching || caps.isLoading),
-    handleManualToolUseChange,
   };
 }