Fix Codex probe child process leak on timeout and Object.assign isDefault: undefined

- Add AbortSignal support to probeCodexAccount and use Effect.ensuring in
  probeCodexCapabilities to kill the child process when the timeout fires,
  mirroring the existing pattern in probeClaudeCapabilities.

- Replace Object.assign with explicit object construction in
  adjustModelsForSubscription to avoid setting an explicit
  isDefault: undefined property on non-default context window options.

Author: cursoragent

apps/server/src/provider/Layers/ClaudeProvider.ts

@@ -333,7 +333,9 @@ export function adjustModelsForSubscription(
       capabilities: {
         ...caps,
         contextWindowOptions: caps.contextWindowOptions.map((opt) =>
-          Object.assign({}, opt, { isDefault: opt.value === "1m" ? true : undefined }),
+          opt.value === "1m"
+            ? { value: opt.value, label: opt.label, isDefault: true as const }
+            : { value: opt.value, label: opt.label },
         ),
       },
     };

apps/server/src/provider/Layers/CodexProvider.ts

@@ -313,15 +313,22 @@ const CAPABILITIES_PROBE_TIMEOUT_MS = 8_000;
 const probeCodexCapabilities = (input: {
   readonly binaryPath: string;
   readonly homePath?: string;
-}) =>
-  Effect.tryPromise(() => probeCodexAccount(input)).pipe(
+}) => {
+  const abort = new AbortController();
+  return Effect.tryPromise(() => probeCodexAccount({ ...input, signal: abort.signal })).pipe(
+    Effect.ensuring(
+      Effect.sync(() => {
+        if (!abort.signal.aborted) abort.abort();
+      }),
+    ),
     Effect.timeoutOption(CAPABILITIES_PROBE_TIMEOUT_MS),
     Effect.result,
     Effect.map((result) => {
       if (Result.isFailure(result)) return undefined;
       return Option.isSome(result.success) ? result.success.value : undefined;
     }),
   );
+};
 
 const runCodexCommand = (args: ReadonlyArray<string>) =>
   Effect.gen(function* () {

apps/server/src/provider/codexAppServer.ts

@@ -43,6 +43,7 @@ export function killCodexChildProcess(child: ChildProcessWithoutNullStreams): vo
 export async function probeCodexAccount(input: {
   readonly binaryPath: string;
   readonly homePath?: string;
+  readonly signal?: AbortSignal;
 }): Promise<CodexAccountSnapshot> {
   return await new Promise((resolve, reject) => {
     const child = spawn(input.binaryPath, ["app-server"], {
@@ -82,6 +83,12 @@ export async function probeCodexAccount(input: {
         ),
       );
 
+    if (input.signal?.aborted) {
+      fail(new Error("Codex account probe aborted."));
+      return;
+    }
+    input.signal?.addEventListener("abort", () => fail(new Error("Codex account probe aborted.")));
+
     const writeMessage = (message: unknown) => {
       if (!child.stdin.writable) {
         fail(new Error("Cannot write to codex app-server stdin."));