Move server_name assignment outside try block to ensure it's always available in exception handler

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

@@ -96,6 +96,9 @@ async def add_tool_servers_to_agent(
 
             # Add servers as MCPStreamableHTTPTool instances
             for config in server_configs:
+                # Use mcp_server_name if available, otherwise fall back to mcp_server_unique_name
+                server_name = config.mcp_server_name or config.mcp_server_unique_name
+                
                 try:
                     # Prepare auth headers
                     headers = {}
@@ -108,9 +111,6 @@ async def add_tool_servers_to_agent(
                         self._orchestrator_name
                     )
 
-                    # Use mcp_server_name if available, otherwise fall back to mcp_server_unique_name
-                    server_name = config.mcp_server_name or config.mcp_server_unique_name
-
                     # Create and configure MCPStreamableHTTPTool
                     mcp_tools = MCPStreamableHTTPTool(
                         name=server_name,
@@ -129,7 +129,7 @@ async def add_tool_servers_to_agent(
 
                 except Exception as tool_ex:
                     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