From b72259d78fed10324744a158c8ed1107a531c98c Mon Sep 17 00:00:00 2001
From: Louis <louis@jan.ai>
Date: Sat, 7 Jun 2025 00:55:12 +0700
Subject: [PATCH] chore: trim all mcp server configs before saving (#5211)

---
 web-app/src/containers/dialogs/AddEditMCPServer.tsx | 11 ++++++-----
 1 file changed, 6 insertions(+), 5 deletions(-)

diff --git a/web-app/src/containers/dialogs/AddEditMCPServer.tsx b/web-app/src/containers/dialogs/AddEditMCPServer.tsx
index acb5c06b6..047b1bc54 100644
--- a/web-app/src/containers/dialogs/AddEditMCPServer.tsx
+++ b/web-app/src/containers/dialogs/AddEditMCPServer.tsx
@@ -203,22 +203,23 @@ export default function AddEditMCPServer({
     // Convert env arrays to object
     const envObj: Record<string, string> = {}
     envKeys.forEach((key, index) => {
-      if (key.trim() !== '') {
-        envObj[key] = envValues[index] || ''
+      const keyName = key.trim()
+      if (keyName !== '') {
+        envObj[keyName] = envValues[index]?.trim() || ''
       }
     })
 
     // Filter out empty args
-    const filteredArgs = args.filter((arg) => arg.trim() !== '')
+    const filteredArgs = args.map((arg) => arg.trim()).filter((arg) => arg)
 
     const config: MCPServerConfig = {
-      command,
+      command: command.trim(),
       args: filteredArgs,
       env: envObj,
     }
 
     if (serverName.trim() !== '') {
-      onSave(serverName, config)
+      onSave(serverName.trim(), config)
       onOpenChange(false)
       resetForm()
     }