From 4cf529476fe2e720005eba5e8ebaa4f89fbb0605 Mon Sep 17 00:00:00 2001
From: ManishMadan2882 <manishmadan321@gmail.com>
Date: Wed, 22 Jul 2026 18:31:57 +0530
Subject: [PATCH] (fix)workflow: stable outside-click handler, drop trailing
 whitespace

Address Copilot review on #2600:
- pass a useCallback handler to useOutsideAlerter so the global mousedown
  listener isn't torn down/re-added on every WorkflowBuilder render
- remove trailing spaces after {displayName} in AgentPageHeader

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
---
 frontend/src/agents/AgentPageHeader.tsx          | 2 +-
 frontend/src/agents/workflow/WorkflowBuilder.tsx | 8 +++++---
 2 files changed, 6 insertions(+), 4 deletions(-)

diff --git a/frontend/src/agents/AgentPageHeader.tsx b/frontend/src/agents/AgentPageHeader.tsx
index 96aa5f85..6d693f58 100644
--- a/frontend/src/agents/AgentPageHeader.tsx
+++ b/frontend/src/agents/AgentPageHeader.tsx
@@ -91,7 +91,7 @@ export default function AgentPageHeader({
           <BreadcrumbItem>
             {currentPage === 'overview' ? (
               <BreadcrumbPage title={displayName} className="w-[16ch] truncate">
-                {displayName}              
+                {displayName}
               </BreadcrumbPage>
             ) : (
               <BreadcrumbLink asChild>
diff --git a/frontend/src/agents/workflow/WorkflowBuilder.tsx b/frontend/src/agents/workflow/WorkflowBuilder.tsx
index e03d5d3f..d821d9f8 100644
--- a/frontend/src/agents/workflow/WorkflowBuilder.tsx
+++ b/frontend/src/agents/workflow/WorkflowBuilder.tsx
@@ -781,11 +781,13 @@ function WorkflowBuilderInner() {
     setShowWorkflowSettings(false);
   }, []);
 
+  const handleCloseWorkflowSettings = useCallback(() => {
+    setShowWorkflowSettings(false);
+  }, []);
+
   useOutsideAlerter(
     workflowSettingsRef,
-    () => {
-      setShowWorkflowSettings(false);
-    },
+    handleCloseWorkflowSettings,
     [],
     false,
   );