Move MCP client imports out of loop iteration in get_servers

[Copilot]

Addresses performance feedback on #10988: imports were being executed on every server check iteration.

Changes

• Moved MCPStdioClient and MCPStreamableHttpClient imports from check_server inner function to parent get_servers function
• Imports now execute once per endpoint call instead of N times (once per configured MCP server)

async def get_servers(...):
    import asyncio
    from lfx.base.mcp.util import MCPStdioClient, MCPStreamableHttpClient  # ← moved here
    
    server_list = await get_server_list(...)
    
    async def check_server(server_name: str) -> dict:
        # imports removed from here
        mcp_stdio_client = MCPStdioClient()
        mcp_streamable_http_client = MCPStreamableHttpClient()
        ...
    
    tasks = [check_server(server) for server in server_list["mcpServers"]]

---

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

[coderabbitai]

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}