Fix env loading in all MCP servers

All 6 MCP servers had a bug where they would `break` after loading
the first .env file found (~/.env). This caused project-specific
API keys in Researcher-Agent/.env to never be loaded.

Fix: Load ALL .env files with override=True, so project-specific
keys properly override base config. Order: ~/.env (base) ->
project root (overrides) -> server dir (highest priority).

Affected servers:
- fundamentals-basket
- valuation-basket
- volatility-basket
- macro-basket
- news-basket
- sentiment-basket

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

mcp-servers/fundamentals-basket/server.py

@@ -21,18 +21,16 @@ from datetime import datetime
 from pathlib import Path
 from typing import Optional
 
-# Load environment variables
+# Load environment variables (later files override earlier ones)
 from dotenv import load_dotenv
-
 env_paths = [
-    Path.home() / ".env",
-    Path(__file__).parent / ".env",
-    Path(__file__).parent.parent.parent / ".env",
+    Path.home() / ".env",  # Home directory (base)
+    Path(__file__).parent.parent.parent / ".env",  # Project root (overrides)
+    Path(__file__).parent / ".env",  # MCP server directory (highest priority)
 ]
 for env_path in env_paths:
     if env_path.exists():
-        load_dotenv(env_path)
-        break
+        load_dotenv(env_path, override=True)
 
 # MCP SDK
 from mcp.server import Server

mcp-servers/macro-basket/server.py

@@ -20,17 +20,16 @@ from datetime import datetime
 from pathlib import Path
 from typing import Optional
 
-# Load environment variables
+# Load environment variables (later files override earlier ones)
 from dotenv import load_dotenv
 env_paths = [
-    Path.home() / ".env",
-    Path(__file__).parent / ".env",
-    Path(__file__).parent.parent.parent / ".env",
+    Path.home() / ".env",  # Home directory (base)
+    Path(__file__).parent.parent.parent / ".env",  # Project root (overrides)
+    Path(__file__).parent / ".env",  # MCP server directory (highest priority)
 ]
 for env_path in env_paths:
     if env_path.exists():
-        load_dotenv(env_path)
-        break
+        load_dotenv(env_path, override=True)
 
 # MCP SDK
 from mcp.server import Server

mcp-servers/news-basket/server.py

@@ -39,17 +39,16 @@ from typing import Optional
 from config.company_name_filters import clean_company_name
 from config.domain_filters import NEWS_DOMAINS, NYT_NEWS_DESKS, NEWSAPI_DOMAINS
 
-# Load environment variables
+# Load environment variables (later files override earlier ones)
 from dotenv import load_dotenv
 env_paths = [
-    Path.home() / ".env",
-    Path(__file__).parent / ".env",
-    Path(__file__).parent.parent.parent / ".env",
+    Path.home() / ".env",  # Home directory (base)
+    Path(__file__).parent.parent.parent / ".env",  # Project root (overrides)
+    Path(__file__).parent / ".env",  # MCP server directory (highest priority)
 ]
 for env_path in env_paths:
     if env_path.exists():
-        load_dotenv(env_path)
-        break
+        load_dotenv(env_path, override=True)
 
 # MCP SDK
 from mcp.server import Server

mcp-servers/sentiment-basket/server.py

@@ -24,16 +24,15 @@ from pathlib import Path
 # Load environment variables from .env
 from dotenv import load_dotenv
 
-# Try loading from multiple locations
+# Load from multiple locations (later files override earlier ones)
 env_paths = [
-    Path.home() / ".env",  # Home directory
-    Path(__file__).parent / ".env",  # MCP server directory
-    Path(__file__).parent.parent.parent / ".env",  # Project root
+    Path.home() / ".env",  # Home directory (base)
+    Path(__file__).parent.parent.parent / ".env",  # Project root (overrides)
+    Path(__file__).parent / ".env",  # MCP server directory (highest priority)
 ]
 for env_path in env_paths:
     if env_path.exists():
-        load_dotenv(env_path)
-        break
+        load_dotenv(env_path, override=True)
 
 # MCP SDK
 from mcp.server import Server

mcp-servers/valuation-basket/server.py

@@ -25,17 +25,16 @@ from pathlib import Path
 from typing import Optional
 from concurrent.futures import ThreadPoolExecutor
 
-# Load environment variables
+# Load environment variables (later files override earlier ones)
 from dotenv import load_dotenv
 env_paths = [
-    Path.home() / ".env",
-    Path(__file__).parent / ".env",
-    Path(__file__).parent.parent.parent / ".env",
+    Path.home() / ".env",  # Home directory (base)
+    Path(__file__).parent.parent.parent / ".env",  # Project root (overrides)
+    Path(__file__).parent / ".env",  # MCP server directory (highest priority)
 ]
 for env_path in env_paths:
     if env_path.exists():
-        load_dotenv(env_path)
-        break
+        load_dotenv(env_path, override=True)
 
 # MCP SDK
 from mcp.server import Server

mcp-servers/volatility-basket/server.py

@@ -24,19 +24,16 @@ from pathlib import Path
 from typing import Optional
 import statistics
 
-# Load environment variables from .env
+# Load environment variables (later files override earlier ones)
 from dotenv import load_dotenv
-
-# Try loading from multiple locations
 env_paths = [
-    Path.home() / ".env",  # Home directory
-    Path(__file__).parent / ".env",  # MCP server directory
-    Path(__file__).parent.parent.parent / ".env",  # Project root
+    Path.home() / ".env",  # Home directory (base)
+    Path(__file__).parent.parent.parent / ".env",  # Project root (overrides)
+    Path(__file__).parent / ".env",  # MCP server directory (highest priority)
 ]
 for env_path in env_paths:
     if env_path.exists():
-        load_dotenv(env_path)
-        break
+        load_dotenv(env_path, override=True)
 
 # MCP SDK
 from mcp.server import Server