From e5b2e9be64fa363590dcc1cfbc2017d66ea30a9e Mon Sep 17 00:00:00 2001
From: oobabooga <112222186+oobabooga@users.noreply.github.com>
Date: Sun, 31 May 2026 15:13:13 -0700
Subject: [PATCH] Move "Confirm tool calls" to the Tools section

---
 .../src/features/chat/chat-settings-sheet.tsx | 43 +++++++++++--------
 1 file changed, 25 insertions(+), 18 deletions(-)

diff --git a/studio/frontend/src/features/chat/chat-settings-sheet.tsx b/studio/frontend/src/features/chat/chat-settings-sheet.tsx
index 8eda82fb02..4f19e95a88 100644
--- a/studio/frontend/src/features/chat/chat-settings-sheet.tsx
+++ b/studio/frontend/src/features/chat/chat-settings-sheet.tsx
@@ -1338,6 +1338,7 @@ export function ChatSettingsPanel({
           <CollapsibleSection label="Tools">
             <div className="flex flex-col gap-5 pt-1">
               <AutoHealToolCallsToggle />
+              <ConfirmToolCallsToggle />
               <MaxToolCallsSlider />
               <ToolCallTimeoutSlider />
             </div>
@@ -1512,13 +1513,35 @@ function AutoHealToolCallsToggle() {
   );
 }
 
+function ConfirmToolCallsToggle() {
+  const confirmToolCalls = useChatRuntimeStore((s) => s.confirmToolCalls);
+  const setConfirmToolCalls = useChatRuntimeStore((s) => s.setConfirmToolCalls);
+
+  return (
+    <div className="flex items-center justify-between gap-3">
+      <div className="flex min-w-0 items-center gap-1.5">
+        <span className="min-w-0 text-[13px] font-medium leading-[1.25] tracking-nav text-nav-fg">
+          Confirm tool calls
+        </span>
+        <InfoHint>
+          When on, every tool call pauses for your approval in the chat before
+          it runs.
+        </InfoHint>
+      </div>
+      <Switch
+        className="panel-switch"
+        checked={confirmToolCalls}
+        onCheckedChange={setConfirmToolCalls}
+      />
+    </div>
+  );
+}
+
 function McpServersSection() {
   const mcpEnabledForChat = useChatRuntimeStore((s) => s.mcpEnabledForChat);
   const setMcpEnabledForChat = useChatRuntimeStore(
     (s) => s.setMcpEnabledForChat,
   );
-  const confirmToolCalls = useChatRuntimeStore((s) => s.confirmToolCalls);
-  const setConfirmToolCalls = useChatRuntimeStore((s) => s.setConfirmToolCalls);
   const [enabledServerCount, setEnabledServerCount] = useState<number | null>(
     null,
   );
@@ -1559,22 +1582,6 @@ function McpServersSection() {
           disabled={enabledServerCount === 0 && !mcpEnabledForChat}
         />
       </div>
-      <div className="flex items-center justify-between gap-3">
-        <div className="flex min-w-0 items-center gap-1.5">
-          <span className="min-w-0 text-[13px] font-medium leading-[1.25] tracking-nav text-nav-fg">
-            Confirm tool calls
-          </span>
-          <InfoHint>
-            When on, every tool call pauses for your approval in the chat
-            before it runs.
-          </InfoHint>
-        </div>
-        <Switch
-          className="panel-switch"
-          checked={confirmToolCalls}
-          onCheckedChange={setConfirmToolCalls}
-        />
-      </div>
       <div className="flex items-center justify-between">
         <span className="text-[11px] text-muted-foreground">
           {enabledServerCount === null