Skip to content

Conversation

Copy link

Copilot AI commented Dec 2, 2025

Part of repository maintenance

Motivation

Build was producing deprecation warnings from recent MCP SDK 0.16.0 and SonarLint Core 10.37.1 updates.

Changes

API Updates

McpError construction (StdioServerTransportProvider.java)

// Before: deprecated constructor
return Mono.error(new McpError("No session to close"));

// After: builder pattern with error codes
return Mono.error(McpError.builder(McpSchema.ErrorCodes.INTERNAL_ERROR)
  .message("No active session to notify clients")
  .build());

AnalyzeFilesAndTrackParams (BackendService.java)

  • Removed deprecated startTime parameter from constructor

Build Configuration

Added -Xlint:deprecation to JavaCompile tasks for early detection of future API changes.

Notes

  • All 462 tests pass
  • One remaining warning from external SonarLint Core library (RawIssueDto) - outside our control
  • Error message improved from "No session to close" to "No active session to notify clients" for clarity

Checklist

  • Explanation provided
  • Code follows SonarSource style
  • Tests updated and passing
  • No JIRA ticket (repository maintenance)

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • develocity.sonar.build
    • Triggering command: /usr/lib/jvm/temurin-17-jdk-amd64/bin/java /usr/lib/jvm/temurin-17-jdk-amd64/bin/java --add-opens=java.base/java.lang=ALL-UNNAMED --add-opens=java.base/java.lang.invoke=ALL-UNNAMED --add-opens=java.base/java.util=ALL-UNNAMED --add-opens=java.prefs/java.util.prefs=ALL-UNNAMED --add-exports=jdk.compiler/com.sun.tools.javac.api=ALL-UNNAMED --add-exports=jdk.compiler/com.sun.tools.javac.util=ALL-UNNAMED --add-opens=java.base/java.util=ALL-UNNAMED --add-opens=java.prefs/java.util.prefs=ALL-UNNAMED --add-opens=java.base/java.nio.charset=ALL-UNNAMED --add-opens=java.base/java.net=ALL-UNNAMED --add-opens=java.base/java.util.concurrent.atomic=ALL-UNNAMED --add-opens=java.xml/javax.xml.namespace=ALL-UNNAMED -XX:MaxMetaspaceSize=512M -Xmx8G -Dfile.encoding=UTF-8 -Duser.country -Duser.language=en -Duser.variant -cp (dns block)
    • Triggering command: /usr/lib/jvm/temurin-21-jdk-amd64/bin/java /usr/lib/jvm/temurin-21-jdk-amd64/bin/java --add-opens=java.base/java.lang=ALL-UNNAMED --add-opens=java.base/java.lang.invoke=ALL-UNNAMED --add-opens=java.base/java.util=ALL-UNNAMED --add-opens=java.prefs/java.util.prefs=ALL-UNNAMED --add-exports=jdk.compiler/com.sun.tools.javac.api=ALL-UNNAMED --add-exports=jdk.compiler/com.sun.tools.javac.util=ALL-UNNAMED --add-opens=java.base/java.util=ALL-UNNAMED --add-opens=java.prefs/java.util.prefs=ALL-UNNAMED --add-opens=java.base/java.nio.charset=ALL-UNNAMED --add-opens=java.base/java.net=ALL-UNNAMED --add-opens=java.base/java.util.concurrent.atomic=ALL-UNNAMED --add-opens=java.xml/javax.xml.namespace=ALL-UNNAMED -XX:MaxMetaspaceSize=512M -Xmx8G -Dfile.encoding=UTF-8 -Duser.country -Duser.language=en -Duser.variant -cp (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Original prompt

Find issues in our repository and fix them


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI changed the title [WIP] Fix issues in the repository Fix deprecation warnings in MCP SDK and SonarLint Core usage Dec 2, 2025
Copilot AI requested a review from nquinquenel December 2, 2025 14:44
@nquinquenel nquinquenel closed this Dec 2, 2025
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