feat: add server name validation to MCP config loader

Author: Karanraj-6

pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "uipath-mcp"
-version = "0.1.1"
+version = "0.1.2"
 description = "UiPath MCP SDK"
 readme = { file = "README.md", content-type = "text/markdown" }
 requires-python = ">=3.11"

src/uipath_mcp/_cli/_utils/_config.py

@@ -1,6 +1,7 @@
 import json
 import logging
 import os
+import re
 from typing import Any
 
 logger = logging.getLogger(__name__)
@@ -66,16 +67,30 @@ def exists(self) -> bool:
         """Check if mcp.json exists"""
         return os.path.exists(self.config_path)
 
+    @staticmethod
+    def validate_server_name(name: str) -> None:
+        """
+        Validate the server name.
+
+        The server name must only contain letters (a-z, A-Z), numbers (0-9), and hyphens (-).
+        Raises a ValueError if the name is invalid.
+        """
+        if not re.match(r"^[a-zA-Z0-9-]+$", name):
+            raise ValueError(
+                f'Invalid server name "{name}": only letters, numbers, and hyphens are allowed.'
+            )
+
     def _load_config(self) -> None:
         """Load and process MCP configuration."""
         try:
             with open(self.config_path, "r") as f:
                 self._raw_config = json.load(f)
 
             servers_config = self._raw_config.get("servers", {})
-            self._servers = {
-                name: McpServer(name, config) for name, config in servers_config.items()
-            }
+            self._servers = {}
+            for name in servers_config.keys():
+                self.validate_server_name(name)
+                self._servers[name] = McpServer(name, servers_config[name])
 
         except json.JSONDecodeError as e:
             logger.error(f"Invalid JSON in {self.config_path}: {str(e)}")