Review comments.

Signed-off-by: Santiago Pericas-Geertsen <[email protected]>

Author: spericas

examples/calendar/src/main/java/io/helidon/extensions/mcp/examples/calendar/ListCalendarEventTool.java

@@ -39,8 +39,7 @@ final class ListCalendarEventTool implements McpTool {
                         "description": "Event date in the following format YYYY-MM-DD",
                         "type": "string"
                     }
-                },
-                "required": [ ]
+                }
             }
             """;
 

server/src/main/java/io/helidon/extensions/mcp/server/McpServerFeature.java

@@ -387,12 +387,6 @@ private void notificationCancelRpc(JsonRpcRequest req, JsonRpcResponse res) {
     private void initializeRpc(JsonRpcRequest req, JsonRpcResponse res) {
         Optional<McpSession> foundSession = findSession(req);
 
-        // log initial request
-        if (LOGGER.isLoggable(Level.FINEST)) {
-            LOGGER.log(Level.FINEST, "Request:\n" + prettyPrint(req.asJsonObject()));
-            LOGGER.log(Level.FINEST, "Response:\n" + prettyPrint(res.asJsonObject()));
-        }
-
         // is this streamable HTTP?
         if (foundSession.isEmpty()) {
             // create a new session
@@ -425,6 +419,12 @@ private void initializeRpc(JsonRpcRequest req, JsonRpcResponse res) {
             LOGGER.log(Level.FINEST, "SSE transport");
             session.send(res);
         }
+
+        // log initial request
+        if (LOGGER.isLoggable(Level.FINEST)) {
+            LOGGER.log(Level.FINEST, "Request:\n" + prettyPrint(req.asJsonObject()));
+            LOGGER.log(Level.FINEST, "Response:\n" + prettyPrint(res.asJsonObject()));
+        }
     }
 
     private String parseClientVersion(McpParameters parameters) {