Improve exception handling and clarify test comments

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

Author: Copilot

libraries/microsoft-agents-a365-tooling-extensions-agentframework/microsoft_agents_a365/tooling/extensions/agentframework/services/mcp_tool_registration_service.py

@@ -133,8 +133,12 @@ async def add_tool_servers_to_agent(
                     self._logger.info(f"Added MCP plugin '{server_name}' to agent tools")
 
                 except Exception as tool_ex:
+                    try:
+                        server_name = config.mcp_server_name
+                    except AttributeError:
+                        server_name = "Unknown"
                     self._logger.warning(
-                        f"Failed to create MCP plugin for {config.mcp_server_name}: {tool_ex}"
+                        f"Failed to create MCP plugin for {server_name}: {tool_ex}"
                     )
                     continue
 

tests/tooling/test_mcp_server_configuration.py

@@ -123,7 +123,7 @@ def test_parse_manifest_server_config_without_custom_url(self, mock_build_url, s
 
         assert config is not None
         assert config.mcp_server_name == "DefaultServer"
-        # When no custom URL, the built URL goes into mcp_server_unique_name
+        # Without a custom URL, build_mcp_server_url constructs the full URL and stores it in mcp_server_unique_name
         assert config.mcp_server_unique_name == "https://default.server/agents/servers/test_server"
         mock_build_url.assert_called_once_with("test_server")