From e3f8813dfc346d0d72901ee112ab1df11f0fc47b Mon Sep 17 00:00:00 2001
From: mateo-berri <277851410+mateo-berri@users.noreply.github.com>
Date: Sat, 6 Jun 2026 22:33:24 +0000
Subject: [PATCH] fix: copy env_vars when duplicating MCP server

Addresses Greptile feedback: servers using ${VAR_NAME} interpolation in
static_headers would have broken references after duplication because
env_vars was not being copied.

https://claude.ai/code/session_01Wx49QQnsCngnJ7hsfaC1yC
---
 .../src/components/mcp_tools/create_mcp_server.tsx       | 9 +++++++++
 1 file changed, 9 insertions(+)

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 b5f1cd593ce..ef728db4a46 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
@@ -347,6 +347,15 @@ const CreateMCPServer: React.FC<CreateMCPServerProps> = ({
       }));
     }
 
+    if (duplicateServer.env_vars && duplicateServer.env_vars.length > 0) {
+      prefillValues.env_vars = duplicateServer.env_vars.map((v) => ({
+        name: v.name,
+        value: v.value,
+        scope: v.scope,
+        description: v.description || "",
+      }));
+    }
+
     if (duplicateServer.is_byok) {
       prefillValues.is_byok = true;
       prefillValues.byok_description = duplicateServer.byok_description || [];