Address review feedback: add parser tests for new fields, fix Javadoc @see refs

Co-authored-by: brunoborges <129743+brunoborges@users.noreply.github.com>

Author: Copilot

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

@@ -460,7 +460,7 @@ public CompletableFuture<List<ModelInfo>> listModels() {
      *
      * @return a future that resolves with the last session ID, or {@code null} if
      *         no sessions exist
-     * @see #resumeSession(String)
+     * @see #resumeSession(String, com.github.copilot.sdk.json.ResumeSessionConfig)
      */
     public CompletableFuture<String> getLastSessionId() {
         return ensureConnected().thenCompose(
@@ -498,7 +498,7 @@ public CompletableFuture<Void> deleteSession(String sessionId) {
      *
      * @return a future that resolves with a list of session metadata
      * @see SessionMetadata
-     * @see #resumeSession(String)
+     * @see #resumeSession(String, com.github.copilot.sdk.json.ResumeSessionConfig)
      */
     public CompletableFuture<List<SessionMetadata>> listSessions() {
         return listSessions(null);
@@ -528,7 +528,7 @@ public CompletableFuture<List<SessionMetadata>> listSessions() {
      * @return a future that resolves with a list of session metadata
      * @see SessionMetadata
      * @see SessionListFilter
-     * @see #resumeSession(String)
+     * @see #resumeSession(String, com.github.copilot.sdk.json.ResumeSessionConfig)
      */
     public CompletableFuture<List<SessionMetadata>> listSessions(SessionListFilter filter) {
         return ensureConnected().thenCompose(connection -> {

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

@@ -1343,6 +1343,7 @@ void testAssistantMessageEventAllFields() throws Exception {
                             }
                         ],
                         "parentToolCallId": "parent-tc",
+                        "interactionId": "interaction-msg-1",
                         "reasoningOpaque": "opaque-data",
                         "reasoningText": "My reasoning",
                         "encryptedContent": "enc123"
@@ -1356,6 +1357,7 @@ void testAssistantMessageEventAllFields() throws Exception {
         assertEquals("msg-rich", data.messageId());
         assertEquals("Full response", data.content());
         assertEquals("parent-tc", data.parentToolCallId());
+        assertEquals("interaction-msg-1", data.interactionId());
         assertEquals("opaque-data", data.reasoningOpaque());
         assertEquals("My reasoning", data.reasoningText());
         assertEquals("enc123", data.encryptedContent());
@@ -1390,6 +1392,59 @@ void testAssistantMessageDeltaEventAllFields() throws Exception {
         assertEquals("ptc-1", data.parentToolCallId());
     }
 
+    @Test
+    void testAssistantStreamingDeltaEventAllFields() throws Exception {
+        String json = """
+                {
+                    "type": "assistant.streaming_delta",
+                    "data": {
+                        "totalResponseSizeBytes": 4096.0
+                    }
+                }
+                """;
+
+        var event = (AssistantStreamingDeltaEvent) parseJson(json);
+        assertNotNull(event);
+        assertEquals("assistant.streaming_delta", event.getType());
+        assertEquals(4096.0, event.getData().totalResponseSizeBytes());
+    }
+
+    @Test
+    void testAssistantMessageEventIncludesInteractionId() throws Exception {
+        String json = """
+                {
+                    "type": "assistant.message",
+                    "data": {
+                        "messageId": "msg-with-interaction",
+                        "content": "Response",
+                        "interactionId": "interaction-abc-123"
+                    }
+                }
+                """;
+
+        var event = (AssistantMessageEvent) parseJson(json);
+        assertNotNull(event);
+        assertEquals("interaction-abc-123", event.getData().interactionId());
+    }
+
+    @Test
+    void testAssistantTurnStartEventIncludesInteractionId() throws Exception {
+        String json = """
+                {
+                    "type": "assistant.turn_start",
+                    "data": {
+                        "turnId": "turn-with-interaction",
+                        "interactionId": "interaction-xyz-456"
+                    }
+                }
+                """;
+
+        var event = (AssistantTurnStartEvent) parseJson(json);
+        assertNotNull(event);
+        assertEquals("turn-with-interaction", event.getData().turnId());
+        assertEquals("interaction-xyz-456", event.getData().interactionId());
+    }
+
     @Test
     void testAssistantUsageEventAllFields() throws Exception {
         String json = """
@@ -1410,6 +1465,23 @@ void testAssistantUsageEventAllFields() throws Exception {
                         "quotaSnapshots": {
                             "premium": 100,
                             "standard": 500
+                        },
+                        "copilotUsage": {
+                            "totalNanoAiu": 1234567.0,
+                            "tokenDetails": [
+                                {
+                                    "tokenType": "input",
+                                    "tokenCount": 500.0,
+                                    "batchSize": 100.0,
+                                    "costPerBatch": 0.001
+                                },
+                                {
+                                    "tokenType": "output",
+                                    "tokenCount": 200.0,
+                                    "batchSize": 100.0,
+                                    "costPerBatch": 0.002
+                                }
+                            ]
                         }
                     }
                 }
@@ -1431,6 +1503,15 @@ void testAssistantUsageEventAllFields() throws Exception {
         assertEquals("ptc-usage", data.parentToolCallId());
         assertNotNull(data.quotaSnapshots());
         assertEquals(2, data.quotaSnapshots().size());
+
+        // Verify copilotUsage
+        assertNotNull(data.copilotUsage());
+        assertEquals(1234567.0, data.copilotUsage().totalNanoAiu());
+        assertNotNull(data.copilotUsage().tokenDetails());
+        assertEquals(2, data.copilotUsage().tokenDetails().size());
+        assertEquals("input", data.copilotUsage().tokenDetails().get(0).tokenType());
+        assertEquals(500.0, data.copilotUsage().tokenDetails().get(0).tokenCount());
+        assertEquals("output", data.copilotUsage().tokenDetails().get(1).tokenType());
     }
 
     @Test
@@ -1546,6 +1627,8 @@ void testToolExecutionCompleteEventWithError() throws Exception {
                     "data": {
                         "toolCallId": "tc-err-1",
                         "success": false,
+                        "model": "claude-3-5-sonnet",
+                        "interactionId": "interaction-tool-1",
                         "isUserRequested": true,
                         "error": {
                             "message": "File not found",
@@ -1565,6 +1648,8 @@ void testToolExecutionCompleteEventWithError() throws Exception {
         var data = event.getData();
         assertEquals("tc-err-1", data.toolCallId());
         assertFalse(data.success());
+        assertEquals("claude-3-5-sonnet", data.model());
+        assertEquals("interaction-tool-1", data.interactionId());
         assertTrue(data.isUserRequested());
         assertEquals("ptc-complete", data.parentToolCallId());
 

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

@@ -229,7 +229,7 @@ void testCanReceiveAndReturnComplexTypes(TestInfo testInfo) throws Exception {
 
     /**
      * Verifies that a custom tool is invoked with the permission handler being
-     * called and can inspect the tool name.
+     * called and can inspect the permission request kind.
      *
      * @see Snapshot: tools/invokes_custom_tool_with_permission_handler
      */
@@ -257,7 +257,7 @@ void testInvokesCustomToolWithPermissionHandler(TestInfo testInfo) throws Except
             CopilotSession session = client.createSession(
                     new SessionConfig().setTools(List.of(encryptTool)).setOnPermissionRequest((request, invocation) -> {
                         permissionRequests.add(request);
-                        return CompletableFuture.completedFuture(new PermissionRequestResult().setKind("approved"));
+                        return PermissionHandler.APPROVE_ALL.handle(request, invocation);
                     })).get();
 
             AssistantMessageEvent response = session