fix: add missing return after exitWithNoMatchingRequestError in CI mode (#345)

* fix: add missing return after exitWithNoMatchingRequestError in CI mode

When running tests in CI mode with no cached response, the proxy would
crash with ERR_HTTP_HEADERS_SENT because exitWithNoMatchingRequestError()
calls onError() (which writes headers), but then execution continued to
super.performRequest() which also tries to write headers.

This caused cascading test failures because the proxy process would die
and subsequent tests couldn't communicate with it.

This is why the 'sendAndWait throws on timeout' test is currently
skipped in CI. With this fix, that test could potentially be enabled.

* test: enable sendAndWait timeout test in CI

Now that the harness properly returns after exitWithNoMatchingRequestError,
this test can run in CI without crashing the proxy.

* fix broken go test

---------

Co-authored-by: Adrien Friggeri <adrien@friggeri.net>

Author: brunoborges

go/internal/e2e/session_test.go

@@ -388,11 +388,7 @@ func TestSession(t *testing.T) {
 		}
 
 		// Resume using a new client
-		newClient := copilot.NewClient(&copilot.ClientOptions{
-			CLIPath: ctx.CLIPath,
-			Cwd:     ctx.WorkDir,
-			Env:     ctx.Env(),
-		})
+		newClient := ctx.NewClient()
 		defer newClient.ForceStop()
 
 		session2, err := newClient.ResumeSession(t.Context(), sessionID)

nodejs/test/e2e/session.test.ts

@@ -384,9 +384,9 @@ describe("Send Blocking Behavior", async () => {
         expect(events).toContain("assistant.message");
     });
 
-    // Skip in CI - this test validates client-side timeout behavior, not LLM responses.
-    // The test intentionally times out before receiving a response, so there's no snapshot to replay.
-    it.skipIf(process.env.CI === "true")("sendAndWait throws on timeout", async () => {
+    // This test validates client-side timeout behavior.
+    // The snapshot has no assistant response since we expect timeout before completion.
+    it("sendAndWait throws on timeout", async () => {
         const session = await client.createSession();
 
         // Use a slow command to ensure timeout triggers before completion

test/harness/replayingCapiProxy.ts

@@ -292,6 +292,7 @@ export class ReplayingCapiProxy extends CapturingHttpProxy {
             state.workDir,
             state.toolResultNormalizers,
           );
+          return;
         }
         super.performRequest(options);
       } catch (err) {