MCP-281 Clean up proxied servers logic

[nquinquenel]

No description provided.

[hashicorp-vault-sonar-prod]

MCP-281

[sonarqubecloud]

SonarQube reviewer guide

Summary: Update tool namespacing from forward slash to underscore separator, add stderr logging for proxied servers, increase default timeout, and improve argument filtering for proxied tool execution.

Review Focus:

• The namespacing change from / to _ is a breaking change affecting tool naming across the codebase—verify all references are updated consistently.
• The new stderr handler in McpClientManager captures diagnostic logs; ensure the logging pattern [serverName] doesn't conflict with existing log formats.
• The Tool.toMap() filtering of null values and ctx parameter is critical for proxied server communication—confirm this doesn't break any expected argument forwarding.
• Timeout increase from 30s to 60s and initialization timeout addition of 1 minute should be validated against real proxied server startup times.

Start review at: src/main/java/org/sonarsource/sonarqube/mcp/client/McpClientManager.java. This is the core logic handling proxied server communication, stderr logging setup, and tool name construction. The changes here directly impact how proxied tools are exposed and how diagnostic information flows back to users.

💬 Please send your feedback

Quality Gate passed

Issues
0 New issues
0 Accepted issues
0 Dependency risks

Measures
0 Security Hotspots
88.2% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud