From ef08fd7c71bc004410da4791c45cd87a17fc7193 Mon Sep 17 00:00:00 2001
From: hugocasa <hugo@casademont.ch>
Date: Fri, 18 Sep 2026 16:02:51 +0200
Subject: [PATCH] fix: ask for a tool name on every kind of agent tool

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
---
 .../flows/common/FlowCardHeader.svelte        | 28 +++++++++++--------
 1 file changed, 16 insertions(+), 12 deletions(-)

diff --git a/frontend/src/lib/components/flows/common/FlowCardHeader.svelte b/frontend/src/lib/components/flows/common/FlowCardHeader.svelte
index 063367cc85..323a04abe3 100644
--- a/frontend/src/lib/components/flows/common/FlowCardHeader.svelte
+++ b/frontend/src/lib/components/flows/common/FlowCardHeader.svelte
@@ -59,6 +59,9 @@
 	let toolNameError = $derived(
 		isAgentTool ? getToolNameError(summary ?? '', undefined, siblingToolNames) : undefined
 	)
+	// An agent tool's summary is the function name the model is given, not free text, so the field
+	// asks for a name whatever the tool runs: a script, an MCP server or another agent.
+	let summaryPlaceholder = $derived(isAgentTool ? 'Tool name' : 'Summary')
 
 	const dispatch = createEventDispatcher()
 	const customUi: FlowBuilderWhitelabelCustomUi | undefined = getContext('customUi')
@@ -191,6 +194,14 @@
 	})
 </script>
 
+{#snippet summaryInput()}
+	<input
+		bind:value={summary}
+		placeholder={summaryPlaceholder}
+		class={twMerge('w-full grow', toolNameError && '!border-red-400')}
+	/>
+{/snippet}
+
 <div class="flex flex-col gap-1 px-4 py-2">
 	<div
 		class="overflow-x-auto scrollbar-hidden flex items-center justify-between flex-nowrap w-full"
@@ -210,7 +221,7 @@
 							code={flowModuleValue.content}
 							class="w-full"
 							elementProps={{
-								placeholder: isAgentTool ? 'Tool name' : 'Summary'
+								placeholder: summaryPlaceholder
 							}}
 							hideError={isAgentTool}
 							{siblingToolNames}
@@ -225,22 +236,15 @@
 							<DropdownV2 size="sm" placement="bottom-end" items={scriptItems} />
 						{/if}
 
-						<div class="flex min-w-[8rem] flex-1 flex-col">
-							<input
-								bind:value={summary}
-								placeholder={isAgentTool ? 'Tool name' : 'Summary'}
-								class={twMerge('w-full grow', toolNameError && '!border-red-400')}
-							/>
-							{#if toolNameError && !isAgentTool}
-								<p class="text-3xs text-red-400 leading-tight mt-0.5">{toolNameError}</p>
-							{/if}
+						<div class="flex min-w-[8rem] flex-1">
+							{@render summaryInput()}
 						</div>
 					{:else if flowModuleValue.type === 'flow'}
 						<Badge color="indigo" capitalize>flow</Badge>
-						<input bind:value={summary} placeholder="Summary" class="w-full grow" />
+						{@render summaryInput()}
 					{:else if flowModuleValue.type === 'aiagent'}
 						<Badge color="indigo">AI Agent</Badge>
-						<input bind:value={summary} placeholder="Summary" class="w-full grow" />
+						{@render summaryInput()}
 					{/if}
 				</div>
 			</span>