test(java): re-enable ModeHandlers exit_plan_mode E2E assertions (#2032)

* test(java): re-enable ModeHandlers exit_plan_mode E2E assertions

Remove the CLI 1.0.57 @disabled workaround and assert the canonical
action order (autopilot, interactive, exit_only) plus recommendedAction,
matching the other language SDKs after snapshot updates.

Fixes #1547

Co-authored-by: Cursor <cursoragent@cursor.com>

* fix(java): use agentMode instead of mode for exit_plan_mode E2E test

The test was using MessageOptions.setMode("plan") which sets the
wire-protocol 'mode' field (prompt formatting only, e.g. the [[PLAN]]
prefix). The field that actually switches the CLI's active agent mode
and exposes the exit_plan_mode tool to the model is 'agentMode', set
via MessageOptions.setAgentMode(AgentMode.PLAN) - matching how the
Node.js and .NET reference E2E tests invoke this same scenario.

This was root-caused by capturing the Node SDK's actual outgoing
session.send payload and comparing it against Java's: Node sends
agentMode: "plan", Java was only sending mode: "plan". Without
agentMode set, the CLI never enters plan UI mode, so
exit_plan_mode is not registered as an available tool and the model's
tool call is rejected by the real (non-replayed) CLI, causing the
replay-proxy snapshot mismatch failure seen in CI.

Reproduced the original failure locally, confirmed this fix resolves
it, and validated on both JDK 25 and JDK 17 per java/README.md's
two-step build/test process.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

---------

Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Steve Sanderson <SteveSandersonMS@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Author: 4 people

java/src/test/java/com/github/copilot/ModeHandlersTest.java

@@ -7,18 +7,19 @@
 import static org.junit.jupiter.api.Assertions.*;
 
 import java.util.HashMap;
+import java.util.List;
 import java.util.Map;
 import java.util.concurrent.CompletableFuture;
 import java.util.concurrent.TimeUnit;
 
 import org.junit.jupiter.api.AfterAll;
 import org.junit.jupiter.api.BeforeAll;
-import org.junit.jupiter.api.Disabled;
 import org.junit.jupiter.api.Test;
 
 import com.github.copilot.generated.ExitPlanModeAction;
 import com.github.copilot.generated.ExitPlanModeCompletedEvent;
 import com.github.copilot.generated.ExitPlanModeRequestedEvent;
+import com.github.copilot.rpc.AgentMode;
 import com.github.copilot.rpc.AutoModeSwitchRequest;
 import com.github.copilot.rpc.AutoModeSwitchResponse;
 import com.github.copilot.rpc.CopilotClientOptions;
@@ -68,7 +69,6 @@ private void configureAuthenticatedUser(String testName) throws Exception {
     }
 
     @Test
-    @Disabled("Snapshot needs re-recording for CLI 1.0.57: https://github.com/github/copilot-sdk/issues/1547")
     void shouldInvokeExitPlanModeHandlerWhenModelUsesTool() throws Exception {
         final String summary = "Greeting file implementation plan";
         configureAuthenticatedUser("should_invoke_exit_plan_mode_handler_when_model_uses_tool");
@@ -99,20 +99,23 @@ void shouldInvokeExitPlanModeHandlerWhenModelUsesTool() throws Exception {
 
             var response = session.sendAndWait(new MessageOptions().setPrompt(
                     "Create a brief implementation plan for adding a greeting.txt file, then request approval with exit_plan_mode.")
-                    .setMode("plan")).get(120, TimeUnit.SECONDS);
+                    .setAgentMode(AgentMode.PLAN)).get(120, TimeUnit.SECONDS);
 
             var request = handlerCalled.get(10, TimeUnit.SECONDS);
             assertEquals(summary, request.getSummary());
-            assertNotNull(request.getActions());
-            assertTrue(request.getActions().contains("interactive"));
+            // Canonical action order after CLI 1.0.57+ (aligned with #2023 / other SDKs).
+            assertEquals(List.of("autopilot", "interactive", "exit_only"), request.getActions());
+            assertEquals("interactive", request.getRecommendedAction());
             assertNotNull(request.getPlanContent());
 
             var reqEvent = requestedEvent.get(10, TimeUnit.SECONDS);
             assertEquals(request.getSummary(), reqEvent.getData().summary());
+            assertEquals(ExitPlanModeAction.INTERACTIVE, reqEvent.getData().recommendedAction());
 
             var compEvent = completedEvent.get(10, TimeUnit.SECONDS);
             assertTrue(compEvent.getData().approved());
             assertEquals(ExitPlanModeAction.INTERACTIVE, compEvent.getData().selectedAction());
+            assertEquals("Approved by the Java E2E test", compEvent.getData().feedback());
 
             assertNotNull(response);