🐛 Bugfix: Fixed the issue of being unable to add MCP services via containerization. (#3213)

[Specification Details]
1. Modify the DEFAULT_NETWORK_NAME when starting the MCP service in the container to match the name in docker-compose.
2. Modify the parameters passed to the add_mcp_service method; custom_headers defaults to None.

Author: YehongPan

backend/services/remote_mcp_service.py

@@ -230,7 +230,7 @@ async def add_mcp_service(
     server_url: str,
     tags: list | None,
     authorization_token: str | None,
-    custom_headers: dict | None,
+    custom_headers: dict | None = None,
     container_config: dict | None,
     registry_json: dict | None,
     enabled: bool = False,

sdk/nexent/container/docker_client.py

@@ -57,7 +57,7 @@ class ContainerConnectionError(Exception):
 class DockerContainerClient(ContainerClient):
     """Docker container client implementation"""
 
-    DEFAULT_NETWORK_NAME = "nexent_nexent"
+    DEFAULT_NETWORK_NAME = "nexent_network"
 
     def __init__(self, config: DockerContainerConfig):
         """

test/sdk/container/test_docker_client.py

@@ -1627,10 +1627,10 @@ def test_ensure_network_exists(self, docker_container_client):
         mock_network = MagicMock()
         docker_container_client.client.networks.get.return_value = mock_network
 
-        docker_container_client._ensure_network("nexent_nexent")
+        docker_container_client._ensure_network("nexent_network")
 
         docker_container_client.client.networks.get.assert_called_once_with(
-            "nexent_nexent")
+            "nexent_network")
         docker_container_client.client.networks.create.assert_not_called()
 
     def test_ensure_network_create_new(self, docker_container_client):
@@ -1640,12 +1640,12 @@ def test_ensure_network_create_new(self, docker_container_client):
         mock_network = MagicMock()
         docker_container_client.client.networks.create.return_value = mock_network
 
-        docker_container_client._ensure_network("nexent_nexent")
+        docker_container_client._ensure_network("nexent_network")
 
         docker_container_client.client.networks.get.assert_called_once_with(
-            "nexent_nexent")
+            "nexent_network")
         docker_container_client.client.networks.create.assert_called_once_with(
-            "nexent_nexent")
+            "nexent_network")
 
     def test_ensure_network_race_condition(self, docker_container_client):
         """Test ensuring network when race condition occurs (another process creates it)"""
@@ -1657,7 +1657,7 @@ def test_ensure_network_race_condition(self, docker_container_client):
         docker_container_client.client.networks.create.side_effect = APIError(
             "Network already exists")
 
-        docker_container_client._ensure_network("nexent_nexent")
+        docker_container_client._ensure_network("nexent_network")
 
         assert docker_container_client.client.networks.get.call_count == 2
         docker_container_client.client.networks.create.assert_called_once()
@@ -1672,15 +1672,15 @@ def test_ensure_network_create_fails_then_get_fails(self, docker_container_clien
             "Create failed")
 
         with pytest.raises(ContainerError, match="Failed to create or get Docker network"):
-            docker_container_client._ensure_network("nexent_nexent")
+            docker_container_client._ensure_network("nexent_network")
 
     def test_ensure_network_get_api_error(self, docker_container_client):
         """Test ensuring network when get raises APIError"""
         docker_container_client.client.networks.get.side_effect = APIError(
             "API error")
 
         with pytest.raises(ContainerError, match="Failed to get Docker network"):
-            docker_container_client._ensure_network("nexent_nexent")
+            docker_container_client._ensure_network("nexent_network")
 
 
 # ---------------------------------------------------------------------------