Support windows paths and process running

Author: xtmq

src/CodexJsonRpcConnection.ts

@@ -8,7 +8,9 @@ import path from "node:path";
 import {createJSONRPCReader, createJSONRPCWriter} from "./StdUtils";
 
 export function startCodexConnection(codexPath: string, logPath?: string): MessageConnection {
-    const codex: ChildProcessWithoutNullStreams = spawn(codexPath, ["app-server"]);
+    const codex: ChildProcessWithoutNullStreams = spawn(`"${codexPath}" app-server`, {
+        shell: process.platform === 'win32'
+    });
 
     if (logPath) {
         attachLogs(codex, logPath);

src/__tests__/acp-test-utils.ts

@@ -3,6 +3,7 @@ import {type CodexConnectionEvent, CodexAppServerClient} from '../CodexAppServer
 import {startCodexConnection} from "../CodexJsonRpcConnection";
 import {CodexAcpServer} from "../CodexAcpServer";
 import type {AgentSideConnection} from "@agentclientprotocol/sdk";
+import path from "node:path";
 
 export type MethodCallEvent = { method: string; args: any[] };
 
@@ -32,14 +33,15 @@ export interface TestFixture {
 }
 
 export function createTestFixture(): TestFixture {
-    const pathToCodex = "././node_modules/.bin/codex"
+    const pathToCodex = path.resolve(process.cwd(), "node_modules", ".bin", process.platform === 'win32' ? "codex.cmd" : "codex");
     const acpConnectionEvents: MethodCallEvent[] = []
     const acpEventHandlers: ((event: MethodCallEvent) => void)[] = [];
     const acpConnection = createSmartMock<AgentSideConnection>((event) => {
         acpConnectionEvents.push(event);
         acpEventHandlers.forEach(handler => handler(event));
     });
-    const codexAppServerClient = new CodexAppServerClient(startCodexConnection(pathToCodex));
+    const codexConnection = startCodexConnection(pathToCodex);
+    const codexAppServerClient = new CodexAppServerClient(codexConnection);
 
     const codexAcpClient = new CodexAcpClient(codexAppServerClient);
     const codexAcpAgent = new CodexAcpServer(acpConnection, codexAcpClient);