Improve error message clarity in StdioServerTransportProvider

Copilot · nquinquenel · Copilot · commit 2bf2c30ed8f8 · 2025-12-02T14:37:44.000Z
Co-authored-by: nquinquenel &lt;14952624+nquinquenel@users.noreply.github.com&gt;
diff --git a/src/main/java/org/sonarsource/sonarqube/mcp/transport/StdioServerTransportProvider.java b/src/main/java/org/sonarsource/sonarqube/mcp/transport/StdioServerTransportProvider.java
@@ -122,7 +122,7 @@ public void setSessionFactory(McpServerSession.Factory sessionFactory) {
   public Mono<Void> notifyClients(String method, Object params) {
     if (this.session == null) {
       return Mono.error(McpError.builder(McpSchema.ErrorCodes.INTERNAL_ERROR)
-        .message("No session to close")
+        .message("No active session to notify clients")
         .build());
     }
     return this.session.sendNotification(method, params)

Original file line number	Diff line number	Diff line change
`@@ -122,7 +122,7 @@ public void setSessionFactory(McpServerSession.Factory sessionFactory) {`
`122`	`122`	`public Mono<Void> notifyClients(String method, Object params) {`
`123`	`123`	`if (this.session == null) {`
`124`	`124`	`return Mono.error(McpError.builder(McpSchema.ErrorCodes.INTERNAL_ERROR)`
`125`		`- .message("No session to close")`
	`125`	`+ .message("No active session to notify clients")`
`126`	`126`	`.build());`
`127`	`127`	`}`
`128`	`128`	`return this.session.sendNotification(method, params)`