From bba52fa96956c16640b3ce56e0d4aca3d4c2f07f Mon Sep 17 00:00:00 2001 From: mateo-berri <277851410+mateo-berri@users.noreply.github.com> Date: Sat, 6 Jun 2026 21:27:41 +0000 Subject: [PATCH] fix: strip trailing _copy before appending to avoid accumulation Addresses Greptile feedback: duplicating Server_copy no longer creates Server_copy_copy; instead it stays Server_copy. https://claude.ai/code/session_01Wx49QQnsCngnJ7hsfaC1yC --- .../src/components/mcp_tools/create_mcp_server.tsx | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/ui/litellm-dashboard/src/components/mcp_tools/create_mcp_server.tsx b/ui/litellm-dashboard/src/components/mcp_tools/create_mcp_server.tsx index dbafe41f994..5867bca2d59 100644 --- a/ui/litellm-dashboard/src/components/mcp_tools/create_mcp_server.tsx +++ b/ui/litellm-dashboard/src/components/mcp_tools/create_mcp_server.tsx @@ -297,9 +297,10 @@ const CreateMCPServer: React.FC = ({ setTransportType(transport); const copySuffix = "_copy"; - const baseName = duplicateServer.server_name || duplicateServer.alias || ""; + const baseName = (duplicateServer.server_name || duplicateServer.alias || "").replace(/_copy$/, ""); + const baseAlias = (duplicateServer.alias || duplicateServer.server_name || "").replace(/_copy$/, ""); const newName = baseName + copySuffix; - const newAlias = (duplicateServer.alias || baseName) + copySuffix; + const newAlias = baseAlias + copySuffix; const prefillValues: Record = { server_name: newName,