Skip to content

MCP-282 Setup CAG#198

Draft
nquinquenel wants to merge 1 commit intomasterfrom
poc/nq/cag-setup
Draft

MCP-282 Setup CAG#198
nquinquenel wants to merge 1 commit intomasterfrom
poc/nq/cag-setup

Conversation

@nquinquenel
Copy link
Member

No description provided.

@hashicorp-vault-sonar-prod hashicorp-vault-sonar-prod bot changed the title Setup CAG MCP-282 Setup CAG Feb 9, 2026
@hashicorp-vault-sonar-prod
Copy link

hashicorp-vault-sonar-prod bot commented Feb 9, 2026

MCP-282

@nquinquenel nquinquenel force-pushed the task/nq/cleanup-proxied branch from 0586b09 to cdbdc0f Compare February 9, 2026 12:14
@nquinquenel nquinquenel force-pushed the task/nq/cleanup-proxied branch from cdbdc0f to a55fa16 Compare February 9, 2026 12:19
@nquinquenel nquinquenel force-pushed the poc/nq/cag-setup branch 2 times, most recently from 606eaee to e5e93ca Compare February 10, 2026 18:20
@nquinquenel nquinquenel force-pushed the task/nq/cleanup-proxied branch from a55fa16 to 8eabaad Compare February 10, 2026 18:24
@nquinquenel nquinquenel force-pushed the poc/nq/cag-setup branch 2 times, most recently from d405b99 to 294a1cc Compare February 11, 2026 08:13
@nquinquenel nquinquenel changed the base branch from task/nq/cleanup-proxied to master March 3, 2026 15:32
@nquinquenel nquinquenel force-pushed the poc/nq/cag-setup branch 2 times, most recently from 9a0c1f1 to e2d7d62 Compare March 3, 2026 15:51
@sonarqubecloud
Copy link

sonarqubecloud bot commented Mar 3, 2026

SonarQube reviewer guide

Review in SonarQube

Summary: Integrates Code Analysis Generation (CAG) support into the SonarQube MCP server by adding a managed stdio transport for proxied MCP servers, removing tool name namespacing, updating Docker configuration with Python support, and implementing stderr log level parsing.

Review Focus:

  • The new ManagedStdioClientTransport class handles process lifecycle management with timeout-based termination and forcible cleanup—verify the timeout logic (5s) and signal handling are robust
  • Tool name changes: proxied tools now expose original names without namespace prefixes (e.g., get_weather instead of context/get_weather)—confirm this doesn't conflict with existing SonarQube tools
  • Docker changes include copying sonar-code-context-mcp binary and requirements.txt; verify the docker-entrypoint.sh properly handles signal forwarding to the Java process
  • Log level parsing in logProxiedServerOutput() uses case-insensitive pattern matching on stderr—check if all expected log formats are covered

Start review at: src/main/java/org/sonarsource/sonarqube/mcp/client/ManagedStdioClientTransport.java. This is the most complex new addition with 330 lines handling process management, threading, and resource cleanup.

💬 Please send your feedback

Quality Gate Passed Quality Gate passed

Issues
0 New issues
10 Accepted issues
0 Dependency risks

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

See analysis details on SonarQube Cloud

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants