Skip to content

Commit 835e69d

Browse files
author
priyanshu.solanki
committed
fixed the issue of UI rendering for deleted mcp servers
1 parent a32f9db commit 835e69d

File tree

1 file changed

+14
-7
lines changed
  • apps/sim/app/workspace/[workspaceId]/w/[workflowId]/components/panel/components/deploy/components/deploy-modal/components/mcp-tool

1 file changed

+14
-7
lines changed

apps/sim/app/workspace/[workspaceId]/w/[workflowId]/components/panel/components/deploy/components/deploy-modal/components/mcp-tool/mcp-tool.tsx

Lines changed: 14 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -229,7 +229,7 @@ interface ToolOnServerProps {
229229
currentInputFormat: Array<{ name: string; type: string }>
230230
currentParameterSchema: Record<string, unknown>
231231
workflowDescription: string | null | undefined
232-
onRemoved: () => void
232+
onRemoved: (serverId: string) => void
233233
onUpdated: () => void
234234
}
235235

@@ -258,7 +258,7 @@ function ToolOnServer({
258258
serverId: server.id,
259259
toolId: tool.id,
260260
})
261-
onRemoved()
261+
onRemoved(server.id)
262262
} catch (error) {
263263
logger.error('Failed to remove tool:', error)
264264
}
@@ -576,9 +576,16 @@ export function McpToolDeploy({
576576
onAddedToServer,
577577
])
578578

579-
const handleToolChanged = useCallback(() => {
580-
// Clear the tools map to force re-query when ServerToolsQuery components re-render
581-
setServerToolsMap({})
579+
const handleToolChanged = useCallback((removedServerId?: string) => {
580+
// If a tool was removed from a specific server, clear just that entry
581+
// The ServerToolsQuery component will re-query and update the map
582+
if (removedServerId) {
583+
setServerToolsMap((prev) => {
584+
const next = { ...prev }
585+
delete next[removedServerId]
586+
return next
587+
})
588+
}
582589
refetchServers()
583590
}, [refetchServers])
584591

@@ -711,8 +718,8 @@ export function McpToolDeploy({
711718
currentInputFormat={inputFormat}
712719
currentParameterSchema={parameterSchema}
713720
workflowDescription={workflowDescription}
714-
onRemoved={handleToolChanged}
715-
onUpdated={handleToolChanged}
721+
onRemoved={(serverId) => handleToolChanged(serverId)}
722+
onUpdated={() => handleToolChanged()}
716723
/>
717724
))}
718725
</div>

0 commit comments

Comments
 (0)