From 62ff965cca742ca9384560137958095d70a68d49 Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Sat, 11 Apr 2026 15:01:32 +0000
Subject: [PATCH 1/2] Initial plan


From 20acdbfc0a7e43a8b5bdf06cd83cac5f2e50e236 Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Sat, 11 Apr 2026 15:08:23 +0000
Subject: [PATCH 2/2] Fix cross-platform process creation in tests: use cmd /c
 more on Windows

Agent-Logs-Url: https://github.com/github/copilot-sdk-java/sessions/a5638a0d-f412-4810-b724-45bc5f10438d

Co-authored-by: edburns <75821+edburns@users.noreply.github.com>
---
 .../com/github/copilot/sdk/CliServerManagerTest.java  |  7 ++++++-
 .../com/github/copilot/sdk/JsonRpcClientTest.java     | 11 ++++++++---
 2 files changed, 14 insertions(+), 4 deletions(-)

diff --git a/src/test/java/com/github/copilot/sdk/CliServerManagerTest.java b/src/test/java/com/github/copilot/sdk/CliServerManagerTest.java
index 32257b0a5..e59d03ae8 100644
--- a/src/test/java/com/github/copilot/sdk/CliServerManagerTest.java
+++ b/src/test/java/com/github/copilot/sdk/CliServerManagerTest.java
@@ -70,13 +70,18 @@ void connectToServerTcpMode() throws Exception {
         }
     }
 
+    private static Process startBlockingProcess() throws IOException {
+        boolean isWindows = System.getProperty("os.name").toLowerCase().contains("windows");
+        return (isWindows ? new ProcessBuilder("cmd", "/c", "more") : new ProcessBuilder("cat")).start();
+    }
+
     @Test
     void connectToServerStdioMode() throws Exception {
         var options = new CopilotClientOptions();
         var manager = new CliServerManager(options);
 
         // Create a dummy process for stdio mode
-        Process process = new ProcessBuilder("cat").start();
+        Process process = startBlockingProcess();
         try {
             JsonRpcClient client = manager.connectToServer(process, null, null);
             assertNotNull(client);
diff --git a/src/test/java/com/github/copilot/sdk/JsonRpcClientTest.java b/src/test/java/com/github/copilot/sdk/JsonRpcClientTest.java
index 8b5c1858c..4fb43f4b6 100644
--- a/src/test/java/com/github/copilot/sdk/JsonRpcClientTest.java
+++ b/src/test/java/com/github/copilot/sdk/JsonRpcClientTest.java
@@ -133,9 +133,14 @@ void testIsConnectedWithSocketClosed() throws Exception {
         pair.serverSocket.close();
     }
 
+    private static Process startBlockingProcess() throws IOException {
+        boolean isWindows = System.getProperty("os.name").toLowerCase().contains("windows");
+        return (isWindows ? new ProcessBuilder("cmd", "/c", "more") : new ProcessBuilder("cat")).start();
+    }
+
     @Test
     void testIsConnectedWithProcess() throws Exception {
-        Process proc = new ProcessBuilder("cat").start();
+        Process proc = startBlockingProcess();
         try (var client = JsonRpcClient.fromProcess(proc)) {
             assertTrue(client.isConnected());
         }
@@ -143,7 +148,7 @@ void testIsConnectedWithProcess() throws Exception {
 
     @Test
     void testIsConnectedWithProcessDead() throws Exception {
-        Process proc = new ProcessBuilder("cat").start();
+        Process proc = startBlockingProcess();
         var client = JsonRpcClient.fromProcess(proc);
         proc.destroy();
         proc.waitFor(5, TimeUnit.SECONDS);
@@ -155,7 +160,7 @@ void testIsConnectedWithProcessDead() throws Exception {
 
     @Test
     void testGetProcessReturnsProcess() throws Exception {
-        Process proc = new ProcessBuilder("cat").start();
+        Process proc = startBlockingProcess();
         try (var client = JsonRpcClient.fromProcess(proc)) {
             assertSame(proc, client.getProcess());
         }