Merge pull request #117 from copilot-community-sdk/feat/records-for-data-types

Refactor data types to use records

Author: brunoborges

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

@@ -48,7 +48,7 @@
  * 	var session = client.createSession(new SessionConfig().setModel("gpt-5")).get();
  *
  * 	session.on(AssistantMessageEvent.class, msg -> {
- * 		System.out.println(msg.getData().getContent());
+ * 		System.out.println(msg.getData().content());
  * 	});
  *
  * 	session.send(new MessageOptions().setPrompt("Hello!")).get();

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

@@ -63,7 +63,7 @@
  *
  * // Register type-safe event handlers
  * session.on(AssistantMessageEvent.class, msg -> {
- * 	System.out.println(msg.getData().getContent());
+ * 	System.out.println(msg.getData().content());
  * });
  * session.on(SessionIdleEvent.class, idle -> {
  * 	System.out.println("Session is idle");
@@ -453,7 +453,7 @@ public Closeable on(Consumer<AbstractSessionEvent> handler) {
      * <pre>{@code
      * // Handle assistant messages
      * session.on(AssistantMessageEvent.class, msg -> {
-     * 	System.out.println(msg.getData().getContent());
+     * 	System.out.println(msg.getData().content());
      * });
      *
      * // Handle session idle

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

@@ -37,7 +37,7 @@
  * <pre>{@code
  * session.on(event -> {
  * 	if (event instanceof AssistantMessageEvent msg) {
- * 		System.out.println("Assistant: " + msg.getData().getContent());
+ * 		System.out.println("Assistant: " + msg.getData().content());
  * 	} else if (event instanceof SessionIdleEvent) {
  * 		System.out.println("Session is idle");
  * 	}

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

@@ -21,7 +21,7 @@
  * <pre>{@code
  * session.on(event -> {
  * 	if (event instanceof AssistantMessageEvent msg) {
- * 		String content = msg.getData().getContent();
+ * 		String content = msg.getData().content();
  * 		System.out.println("Assistant: " + content);
  * 	}
  * });
@@ -70,233 +70,25 @@ public void setData(AssistantMessageData data) {
      * Contains the assistant message content and metadata.
      */
     @JsonIgnoreProperties(ignoreUnknown = true)
-    public static class AssistantMessageData {
-
-        @JsonProperty("messageId")
-        private String messageId;
-
-        @JsonProperty("content")
-        private String content;
-
-        @JsonProperty("toolRequests")
-        private List<ToolRequest> toolRequests;
-
-        @JsonProperty("parentToolCallId")
-        private String parentToolCallId;
-
-        @JsonProperty("reasoningOpaque")
-        private String reasoningOpaque;
-
-        @JsonProperty("reasoningText")
-        private String reasoningText;
-
-        @JsonProperty("encryptedContent")
-        private String encryptedContent;
-
-        /**
-         * Gets the unique message identifier.
-         *
-         * @return the message ID
-         */
-        public String getMessageId() {
-            return messageId;
-        }
-
-        /**
-         * Sets the message identifier.
-         *
-         * @param messageId
-         *            the message ID
-         */
-        public void setMessageId(String messageId) {
-            this.messageId = messageId;
-        }
-
-        /**
-         * Gets the text content of the assistant's message.
-         *
-         * @return the message content
-         */
-        public String getContent() {
-            return content;
-        }
-
-        /**
-         * Sets the message content.
-         *
-         * @param content
-         *            the message content
-         */
-        public void setContent(String content) {
-            this.content = content;
-        }
-
-        /**
-         * Gets the list of tool requests made by the assistant.
-         *
-         * @return the tool requests, or {@code null} if none
-         */
-        public List<ToolRequest> getToolRequests() {
+    public record AssistantMessageData(@JsonProperty("messageId") String messageId,
+            @JsonProperty("content") String content, @JsonProperty("toolRequests") List<ToolRequest> toolRequests,
+            @JsonProperty("parentToolCallId") String parentToolCallId,
+            @JsonProperty("reasoningOpaque") String reasoningOpaque,
+            @JsonProperty("reasoningText") String reasoningText,
+            @JsonProperty("encryptedContent") String encryptedContent) {
+
+        /** Returns a defensive copy of the tool requests list. */
+        @Override
+        public List<ToolRequest> toolRequests() {
             return toolRequests == null ? null : Collections.unmodifiableList(toolRequests);
         }
 
-        /**
-         * Sets the tool requests.
-         *
-         * @param toolRequests
-         *            the tool requests
-         */
-        public void setToolRequests(List<ToolRequest> toolRequests) {
-            this.toolRequests = toolRequests;
-        }
-
-        /**
-         * Gets the parent tool call ID if this message is in response to a tool.
-         *
-         * @return the parent tool call ID, or {@code null}
-         */
-        public String getParentToolCallId() {
-            return parentToolCallId;
-        }
-
-        /**
-         * Sets the parent tool call ID.
-         *
-         * @param parentToolCallId
-         *            the parent tool call ID
-         */
-        public void setParentToolCallId(String parentToolCallId) {
-            this.parentToolCallId = parentToolCallId;
-        }
-
-        /**
-         * Gets the opaque reasoning content (encrypted/encoded).
-         *
-         * @return the opaque reasoning content, or {@code null}
-         */
-        public String getReasoningOpaque() {
-            return reasoningOpaque;
-        }
-
-        /**
-         * Sets the opaque reasoning content.
-         *
-         * @param reasoningOpaque
-         *            the opaque reasoning content
-         */
-        public void setReasoningOpaque(String reasoningOpaque) {
-            this.reasoningOpaque = reasoningOpaque;
-        }
-
-        /**
-         * Gets the human-readable reasoning text.
-         *
-         * @return the reasoning text, or {@code null}
-         */
-        public String getReasoningText() {
-            return reasoningText;
-        }
-
-        /**
-         * Sets the reasoning text.
-         *
-         * @param reasoningText
-         *            the reasoning text
-         */
-        public void setReasoningText(String reasoningText) {
-            this.reasoningText = reasoningText;
-        }
-
-        /**
-         * Gets the encrypted content.
-         *
-         * @return the encrypted content, or {@code null}
-         */
-        public String getEncryptedContent() {
-            return encryptedContent;
-        }
-
-        /**
-         * Sets the encrypted content.
-         *
-         * @param encryptedContent
-         *            the encrypted content
-         */
-        public void setEncryptedContent(String encryptedContent) {
-            this.encryptedContent = encryptedContent;
-        }
-
         /**
          * Represents a request from the assistant to invoke a tool.
          */
         @JsonIgnoreProperties(ignoreUnknown = true)
-        public static class ToolRequest {
-
-            @JsonProperty("toolCallId")
-            private String toolCallId;
-
-            @JsonProperty("name")
-            private String name;
-
-            @JsonProperty("arguments")
-            private Object arguments;
-
-            /**
-             * Gets the unique tool call identifier.
-             *
-             * @return the tool call ID
-             */
-            public String getToolCallId() {
-                return toolCallId;
-            }
-
-            /**
-             * Sets the tool call identifier.
-             *
-             * @param toolCallId
-             *            the tool call ID
-             */
-            public void setToolCallId(String toolCallId) {
-                this.toolCallId = toolCallId;
-            }
-
-            /**
-             * Gets the name of the tool to invoke.
-             *
-             * @return the tool name
-             */
-            public String getName() {
-                return name;
-            }
-
-            /**
-             * Sets the tool name.
-             *
-             * @param name
-             *            the tool name
-             */
-            public void setName(String name) {
-                this.name = name;
-            }
-
-            /**
-             * Gets the arguments to pass to the tool.
-             *
-             * @return the tool arguments (typically a Map or JsonNode)
-             */
-            public Object getArguments() {
-                return arguments;
-            }
-
-            /**
-             * Sets the tool arguments.
-             *
-             * @param arguments
-             *            the tool arguments
-             */
-            public void setArguments(Object arguments) {
-                this.arguments = arguments;
-            }
+        public record ToolRequest(@JsonProperty("toolCallId") String toolCallId, @JsonProperty("name") String name,
+                @JsonProperty("arguments") Object arguments) {
         }
     }
 }