Use 'url' field consistently in both manifest and gateway responses

Co-authored-by: pontemonti <7850950+pontemonti@users.noreply.github.com>

Author: Copilot

libraries/microsoft-agents-a365-tooling/microsoft_agents_a365/tooling/services/mcp_tool_server_configuration_service.py

@@ -501,9 +501,7 @@ def _extract_server_url(self, server_element: Dict[str, Any]) -> Optional[str]:
         Returns:
             Server URL string or None.
         """
-        # Check for 'mcpServerUrl' (manifest) or 'url' (gateway)
-        if "mcpServerUrl" in server_element and isinstance(server_element["mcpServerUrl"], str):
-            return server_element["mcpServerUrl"]
+        # Check for 'url' field in both manifest and gateway responses
         if "url" in server_element and isinstance(server_element["url"], str):
             return server_element["url"]
         return None

tests/tooling/test_mcp_server_configuration.py

@@ -72,22 +72,17 @@ def mock_manifest_data(self) -> Dict[str, Any]:
                 {
                     "mcpServerName": "TestServer2",
                     "mcpServerUniqueName": "test_server_2",
-                    "mcpServerUrl": "https://custom.server.com/mcp",
+                    "url": "https://custom.server.com/mcp",
                 },
             ]
         }
 
     def test_extract_server_url_from_manifest(self, service):
         """Test extracting custom URL from manifest element."""
-        # Test with mcpServerUrl field
-        element = {"mcpServerUrl": "https://custom.url.com"}
-        url = service._extract_server_url(element)
-        assert url == "https://custom.url.com"
-
         # Test with url field
-        element = {"url": "https://another.url.com"}
+        element = {"url": "https://custom.url.com"}
         url = service._extract_server_url(element)
-        assert url == "https://another.url.com"
+        assert url == "https://custom.url.com"
 
         # Test with no URL
         element = {}
@@ -99,7 +94,7 @@ def test_parse_manifest_server_config_with_custom_url(self, service):
         server_element = {
             "mcpServerName": "CustomServer",
             "mcpServerUniqueName": "custom_server",
-            "mcpServerUrl": "https://my.custom.server/mcp",
+            "url": "https://my.custom.server/mcp",
         }
 
         config = service._parse_manifest_server_config(server_element)