Update @github/copilot to 1.0.57-3 (#1519)

* Update @github/copilot to 1.0.57-3

- Updated nodejs and test harness dependencies
- Re-ran code generators
- Formatted generated code

* Re-enable pending-work resume e2e tests now that runtime race is fixed

Co-authored-by: stephentoub <2642209+stephentoub@users.noreply.github.com>

---------

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: stephentoub <2642209+stephentoub@users.noreply.github.com>

Author: 3 people

dotnet/test/E2E/PendingWorkResumeE2ETests.cs

@@ -19,14 +19,7 @@ public class PendingWorkResumeE2ETests(E2ETestFixture fixture, ITestOutputHelper
     private static readonly TimeSpan PendingWorkTimeout = TimeSpan.FromSeconds(60);
     private const string SharedToken = "pending-work-resume-shared-token";
 
-    // Skipped after the runtime 1.0.56 bump. Runtime PR #9040 (commit b8e1220b45) changed
-    // SDKServer.handleConnectionClosed to tear down the session when the last RPC client
-    // disconnects, so the in-memory pending permission request is gone before the resumed
-    // client can satisfy it and HandlePendingPermissionRequest returns success=false. This
-    // test models same-process ForceStop+resume; it needs to be redesigned to either keep
-    // an owner connected (warm resume) or to model a true process restart against the
-    // persisted session state.
-    [Fact(Skip = "Runtime 1.0.56 cleans up the session on last-client disconnect (copilot-agent-runtime PR #9040), so the in-memory pending request is gone before resume can satisfy it. Test needs redesign.")]
+    [Fact]
     public async Task Should_Continue_Pending_Permission_Request_After_Resume()
     {
         var originalPermissionRequest = new TaskCompletionSource<PermissionRequest>(TaskCreationOptions.RunContinuationsAsynchronously);
@@ -95,12 +88,7 @@ static string ResumePermissionTool([Description("Value to transform")] string va
             $"ORIGINAL_SHOULD_NOT_RUN_{value}";
     }
 
-    // Skipped for the same reason as Should_Continue_Pending_Permission_Request_After_Resume:
-    // runtime 1.0.56 (copilot-agent-runtime PR #9040) tears down the session when the last
-    // RPC client disconnects, so the in-memory pending external tool call is gone before
-    // the resumed client can satisfy it. Needs redesign to keep an owner connected (warm)
-    // or to model true process-restart resume from persisted state.
-    [Fact(Skip = "Runtime 1.0.56 cleans up the session on last-client disconnect (copilot-agent-runtime PR #9040), so the in-memory pending tool call is gone before resume can satisfy it. Test needs redesign.")]
+    [Fact]
     public async Task Should_Continue_Pending_External_Tool_Request_After_Resume()
     {
         var originalToolStarted = new TaskCompletionSource<string>(TaskCreationOptions.RunContinuationsAsynchronously);

go/internal/e2e/pending_work_resume_e2e_test.go

@@ -25,15 +25,6 @@ func TestPendingWorkResumeE2E(t *testing.T) {
 	ctx := testharness.NewTestContext(t)
 
 	t.Run("should continue pending permission request after resume", func(t *testing.T) {
-		// Skipped after the runtime 1.0.56 bump. Runtime PR #9040 (commit b8e1220b45)
-		// changed SDKServer.handleConnectionClosed to tear down the session when the
-		// last RPC client disconnects, so the in-memory pending permission request is
-		// gone before the resumed client can satisfy it and HandlePendingPermissionRequest
-		// returns Success=false. This test models same-process ForceStop+resume; it
-		// needs to be redesigned to either keep an owner connected (warm resume) or to
-		// model a true process restart against the persisted session state.
-		t.Skip("Runtime 1.0.56 cleans up the session on last-client disconnect (copilot-agent-runtime PR #9040), so the in-memory pending request is gone before resume can satisfy it. Test needs redesign.")
-
 		ctx.ConfigureForTest(t)
 
 		_, cliURL := startTcpServer(t, ctx)
@@ -147,14 +138,6 @@ func TestPendingWorkResumeE2E(t *testing.T) {
 	})
 
 	t.Run("should continue pending external tool request after resume", func(t *testing.T) {
-		// Skipped for the same reason as "should continue pending permission request
-		// after resume": runtime 1.0.56 (copilot-agent-runtime PR #9040) tears down
-		// the session when the last RPC client disconnects, so the in-memory pending
-		// external tool call is gone before the resumed client can satisfy it. Needs
-		// redesign to keep an owner connected (warm) or to model true process-restart
-		// resume from persisted state.
-		t.Skip("Runtime 1.0.56 cleans up the session on last-client disconnect (copilot-agent-runtime PR #9040), so the in-memory pending tool call is gone before resume can satisfy it. Test needs redesign.")
-
 		ctx.ConfigureForTest(t)
 
 		_, cliURL := startTcpServer(t, ctx)

nodejs/package-lock.json

@@ -56,7 +56,7 @@
     "author": "GitHub",
     "license": "MIT",
     "dependencies": {
-        "@github/copilot": "^1.0.57-2",
+        "@github/copilot": "^1.0.57-3",
         "vscode-jsonrpc": "^8.2.1",
         "zod": "^4.3.6"
     },

nodejs/package.json

@@ -166,14 +166,7 @@ describe("Pending work resume", async () => {
         return `localhost:${port}`;
     }
 
-    // Skipped after the runtime 1.0.56 bump. Runtime PR #9040 (commit b8e1220b45)
-    // changed SDKServer.handleConnectionClosed to tear down the session when the
-    // last RPC client disconnects, so the in-memory pending permission request is
-    // gone before the resumed client can satisfy it and handlePendingPermissionRequest
-    // returns success=false. This test models same-process ForceStop+resume; it needs
-    // to be redesigned to either keep an owner connected (warm resume) or to model
-    // a true process restart against the persisted session state.
-    it.skip(
+    it(
         "should continue pending permission request after resume",
         { timeout: TEST_TIMEOUT_MS },
         async () => {
@@ -246,13 +239,7 @@ describe("Pending work resume", async () => {
         }
     );
 
-    // Skipped for the same reason as "should continue pending permission request
-    // after resume": runtime 1.0.56 (copilot-agent-runtime PR #9040) tears down the
-    // session when the last RPC client disconnects, so the in-memory pending external
-    // tool call is gone before the resumed client can satisfy it. Needs redesign to
-    // keep an owner connected (warm) or to model true process-restart resume from
-    // persisted state.
-    it.skip(
+    it(
         "should continue pending external tool request after resume",
         { timeout: TEST_TIMEOUT_MS },
         async () => {

nodejs/samples/package-lock.json

@@ -128,20 +128,6 @@ async def _safe_force_stop(client: CopilotClient) -> None:
 
 
 class TestPendingWorkResume:
-    # Skipped after the runtime 1.0.56 bump. Runtime PR #9040 (commit b8e1220b45)
-    # changed SDKServer.handleConnectionClosed to tear down the session when the last
-    # RPC client disconnects, so the in-memory pending permission request is gone
-    # before the resumed client can satisfy it and handle_pending_permission_request
-    # returns success=False. This test models same-process force_stop+resume; it
-    # needs to be redesigned to either keep an owner connected (warm resume) or to
-    # model a true process restart against the persisted session state.
-    @pytest.mark.skip(
-        reason=(
-            "Runtime 1.0.56 cleans up the session on last-client disconnect "
-            "(copilot-agent-runtime PR #9040), so the in-memory pending request "
-            "is gone before resume can satisfy it. Test needs redesign."
-        )
-    )
     async def test_should_continue_pending_permission_request_after_resume(
         self, ctx: E2ETestContext
     ):
@@ -222,19 +208,6 @@ def resumed_tool_handler(args):
         finally:
             await _safe_force_stop(server)
 
-    # Skipped for the same reason as
-    # test_should_continue_pending_permission_request_after_resume: runtime 1.0.56
-    # (copilot-agent-runtime PR #9040) tears down the session when the last RPC
-    # client disconnects, so the in-memory pending external tool call is gone before
-    # the resumed client can satisfy it. Needs redesign to keep an owner connected
-    # (warm) or to model true process-restart resume from persisted state.
-    @pytest.mark.skip(
-        reason=(
-            "Runtime 1.0.56 cleans up the session on last-client disconnect "
-            "(copilot-agent-runtime PR #9040), so the in-memory pending tool call "
-            "is gone before resume can satisfy it. Test needs redesign."
-        )
-    )
     async def test_should_continue_pending_external_tool_request_after_resume(
         self, ctx: E2ETestContext
     ):