From 37d2caf5e9a1c1fe8c805487edaa2848e1cb1a4e Mon Sep 17 00:00:00 2001
From: Cursor Agent <cursoragent@cursor.com>
Date: Tue, 24 Feb 2026 22:02:35 +0000
Subject: [PATCH] fix: prevent double-toggle on tool/skill switches by stopping
 event propagation

The card onClick and Switch onChange both fired, canceling each other out.
Stop propagation on the Switch click so only one handler runs.

Co-authored-by: Ishaan Jaff <ishaan-jaff@users.noreply.github.com>
---
 .../src/components/agents/add_agent_wizard.tsx                | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/ui/litellm-dashboard/src/components/agents/add_agent_wizard.tsx b/ui/litellm-dashboard/src/components/agents/add_agent_wizard.tsx
index 1c0df7d7d7e..8d0bf79e236 100644
--- a/ui/litellm-dashboard/src/components/agents/add_agent_wizard.tsx
+++ b/ui/litellm-dashboard/src/components/agents/add_agent_wizard.tsx
@@ -115,7 +115,7 @@ const ToolCard: React.FC<{
       <div className="text-sm font-medium text-gray-900">{tool.name}</div>
       <div className="text-xs text-gray-500">{tool.description}</div>
     </div>
-    <Switch checked={selected} onChange={onToggle} size="small" />
+    <Switch checked={selected} size="small" onClick={(_, e) => e.stopPropagation()} onChange={onToggle} />
   </div>
 );
 
@@ -134,7 +134,7 @@ const SkillCard: React.FC<{
   >
     <div className="flex items-center justify-between">
       <span className="text-sm font-medium text-gray-900">{skill.name}</span>
-      <Switch checked={selected} onChange={onToggle} size="small" />
+      <Switch checked={selected} size="small" onClick={(_, e) => e.stopPropagation()} onChange={onToggle} />
     </div>
     <div className="text-xs text-gray-500 mt-1">{skill.description}</div>
   </div>