Refactor code structure for improved readability and maintainability.

Author: brunoborges

src/main/java/com/github/copilot/sdk/CopilotSession.java

@@ -750,7 +750,7 @@ public CompletableFuture<List<AbstractSessionEvent>> getMessages() {
                     if (response.getEvents() != null) {
                         for (JsonNode eventNode : response.getEvents()) {
                             try {
-                                AbstractSessionEvent event = SessionEventParser.parse(eventNode.toString());
+                                AbstractSessionEvent event = SessionEventParser.parse(eventNode);
                                 if (event != null) {
                                     events.add(event);
                                 }

src/main/java/com/github/copilot/sdk/RpcHandlerDispatcher.java

@@ -82,7 +82,7 @@ private void handleSessionEvent(JsonNode params) {
 
             CopilotSession session = sessions.get(sessionId);
             if (session != null && eventNode != null) {
-                AbstractSessionEvent event = SessionEventParser.parse(eventNode.toString());
+                AbstractSessionEvent event = SessionEventParser.parse(eventNode);
                 if (event != null) {
                     session.dispatchEvent(event);
                 }

src/main/java/com/github/copilot/sdk/events/SessionEventParser.java

@@ -88,36 +88,6 @@ public class SessionEventParser {
         TYPE_MAP.put("skill.invoked", SkillInvokedEvent.class);
     }
 
-    /**
-     * Parses a JSON string into the appropriate SessionEvent subclass.
-     *
-     * @param json
-     *            the JSON string representing an event
-     * @return the parsed event, or {@code null} if parsing fails or type is unknown
-     */
-    public static AbstractSessionEvent parse(String json) {
-        try {
-            JsonNode node = MAPPER.readTree(json);
-            String type = node.has("type") ? node.get("type").asText() : null;
-
-            if (type == null) {
-                LOG.warning("Missing 'type' field in event: " + json);
-                return null;
-            }
-
-            Class<? extends AbstractSessionEvent> eventClass = TYPE_MAP.get(type);
-            if (eventClass == null) {
-                LOG.fine("Unknown event type: " + type);
-                return null;
-            }
-
-            return MAPPER.treeToValue(node, eventClass);
-        } catch (Exception e) {
-            LOG.log(Level.SEVERE, "Failed to parse session event", e);
-            return null;
-        }
-    }
-
     /**
      * Parses a JsonNode into the appropriate SessionEvent subclass.
      *

src/test/java/com/github/copilot/sdk/MetadataApiTest.java

@@ -81,7 +81,7 @@ private static String findCopilotInPath() {
     // ===== ToolExecutionProgressEvent Tests =====
 
     @Test
-    void testToolExecutionProgressEventParsing() {
+    void testToolExecutionProgressEventParsing() throws Exception {
         String json = """
                 {
                     "type": "tool.execution_progress",
@@ -94,7 +94,7 @@ void testToolExecutionProgressEventParsing() {
                 }
                 """;
 
-        var event = SessionEventParser.parse(json);
+        var event = SessionEventParser.parse(MAPPER.readTree(json));
 
         assertNotNull(event);
         assertInstanceOf(ToolExecutionProgressEvent.class, event);