From 33058b534e50b6fe6507c1c2b5d53b1014893f09 Mon Sep 17 00:00:00 2001
From: yuchou87 <yuchou87@gmail.com>
Date: Sat, 21 Feb 2026 13:45:00 +0800
Subject: [PATCH] fix(mcp): reject empty keys in loadEnvFile

A line like '=value' would result in envVars[""] = "value", producing
an invalid environment entry for the child process. Return an error
instead when the key is empty.
---
 pkg/mcp/manager.go | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/pkg/mcp/manager.go b/pkg/mcp/manager.go
index 1fc253774..bbc6925ea 100644
--- a/pkg/mcp/manager.go
+++ b/pkg/mcp/manager.go
@@ -73,6 +73,10 @@ func loadEnvFile(path string) (map[string]string, error) {
 		key := strings.TrimSpace(parts[0])
 		value := strings.TrimSpace(parts[1])
 
+		if key == "" {
+			return nil, fmt.Errorf("invalid format at line %d: empty key", lineNum)
+		}
+
 		// Remove surrounding quotes if present
 		if len(value) >= 2 {
 			if (value[0] == '"' && value[len(value)-1] == '"') ||