Use 'or' operator for cleaner fallback logic in server_name assignment

Copilot · pontemonti · Copilot · commit ad365b0fc4c4 · 2025-12-30T21:15:11.000Z
Co-authored-by: pontemonti &lt;7850950+pontemonti@users.noreply.github.com&gt;
diff --git a/libraries/microsoft-agents-a365-tooling-extensions-agentframework/microsoft_agents_a365/tooling/extensions/agentframework/services/mcp_tool_registration_service.py b/libraries/microsoft-agents-a365-tooling-extensions-agentframework/microsoft_agents_a365/tooling/extensions/agentframework/services/mcp_tool_registration_service.py
@@ -108,7 +108,7 @@ async def add_tool_servers_to_agent(
                         self._orchestrator_name
                     )
 
-                    server_name = config.mcp_server_name if config.mcp_server_name else config.mcp_server_unique_name
+                    server_name = config.mcp_server_name or config.mcp_server_unique_name
 
                     # Create and configure MCPStreamableHTTPTool
                     mcp_tools = MCPStreamableHTTPTool(

Original file line number	Diff line number	Diff line change
`@@ -108,7 +108,7 @@ async def add_tool_servers_to_agent(`
`108`	`108`	`self._orchestrator_name`
`109`	`109`	`)`
`110`	`110`
`111`		`- server_name = config.mcp_server_name if config.mcp_server_name else config.mcp_server_unique_name`
	`111`	`+ server_name = config.mcp_server_name or config.mcp_server_unique_name`
`112`	`112`
`113`	`113`	`# Create and configure MCPStreamableHTTPTool`
`114`	`114`	`mcp_tools = MCPStreamableHTTPTool(`