fix: config path handling

Author: LittleCoinCoin

hatch/mcp_host_config/strategies.py

@@ -433,8 +433,12 @@ def validate_server_config(self, server_config: MCPServerConfig) -> bool:
 
     def read_configuration(self) -> HostConfiguration:
         """Read Kiro configuration file."""
-        config_path = self.get_config_path()
-        if not config_path or not config_path.exists():
+        config_path_str = self.get_config_path()
+        if not config_path_str:
+            return HostConfiguration(servers={})
+        
+        config_path = Path(config_path_str)
+        if not config_path.exists():
             return HostConfiguration(servers={})
 
         try:
@@ -459,10 +463,12 @@ def read_configuration(self) -> HostConfiguration:
 
     def write_configuration(self, config: HostConfiguration, no_backup: bool = False) -> bool:
         """Write configuration to Kiro with backup support."""
-        config_path = self.get_config_path()
-        if not config_path:
+        config_path_str = self.get_config_path()
+        if not config_path_str:
             return False
 
+        config_path = Path(config_path_str)
+        
         try:
             # Ensure directory exists
             config_path.parent.mkdir(parents=True, exist_ok=True)

tests/test_data/configs/mcp_host_test_configs/kiro_simple.json

@@ -0,0 +1,14 @@
+{
+  "mcpServers": {
+    "test_server": {
+      "command": "auggie",
+      "args": [
+        "--mcp"
+      ],
+      "disabled": false,
+      "autoApprove": [
+        "codebase-retrieval"
+      ]
+    }
+  }
+}