Refactor update_tools to use context manager for automatic cleanup

[Copilot]

The update_tools function creates MCP clients but requires callers to manually manage cleanup, leading to verbose finally blocks and potential for resource leaks.

Changes

• Added MCPToolsManager context manager: Encapsulates client creation, tool fetching, and automatic cleanup on exit
• Added update_tools_with_cleanup() factory function: Returns configured context manager instance
• Updated get_servers endpoint: Replaced manual client management with context manager pattern

Before/After

Before:

mcp_stdio_client = MCPStdioClient()
mcp_streamable_http_client = MCPStreamableHttpClient()
try:
    mode, tool_list, _ = await update_tools(
        server_name, server_config,
        mcp_stdio_client=mcp_stdio_client,
        mcp_streamable_http_client=mcp_streamable_http_client,
    )
    # Use tools
finally:
    await mcp_stdio_client.disconnect()
    await mcp_streamable_http_client.disconnect()

After:

async with update_tools_with_cleanup(server_name, server_config) as (mode, tool_list, _):
    # Use tools

The original update_tools function remains unchanged for backward compatibility with components that manage client lifecycle externally (e.g., MCPToolsComponent).

---

💬 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.}