codex-multi-auth 1.3.1 → 1.3.2

package/README.md

@@ -333,9 +333,9 @@ codex auth doctor --json
 
 ## Release Notes
 
-- Current stable: [docs/releases/v1.3.1.md](docs/releases/v1.3.1.md)
-- Previous stable: [docs/releases/v1.2.6.md](docs/releases/v1.2.6.md)
-- Earlier stable: [docs/releases/v1.2.5.md](docs/releases/v1.2.5.md)
+- Current stable: [docs/releases/v1.3.2.md](docs/releases/v1.3.2.md)
+- Previous stable: [docs/releases/v1.3.1.md](docs/releases/v1.3.1.md)
+- Earlier stable: [docs/releases/v1.3.0.md](docs/releases/v1.3.0.md)
 - Full release archive: [docs/README.md#release-history](docs/README.md#release-history)
 - Archived prerelease: [docs/releases/v0.1.0-beta.0.md](docs/releases/v0.1.0-beta.0.md)
 

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "codex-multi-auth",
-	"version": "1.3.1",
+	"version": "1.3.2",
 	"description": "Multi-account OAuth manager and codex auth wrapper for the official @openai/codex CLI, with switching, health checks, and recovery tools",
 	"main": "./dist/index.js",
 	"types": "./dist/index.d.ts",

package/scripts/codex.js

@@ -232,10 +232,10 @@ function resolveUnsupportedModelRetryTarget(
 	return undefined;
 }
 
-function replaceRequestedModel(args, nextModel) {
-	if (!nextModel) {
-		return [...args];
-	}
+function replaceRequestedModel(args, nextModel) {
+	if (!nextModel) {
+		return [...args];
+	}
 
 	const nextArgs = [...args];
 	for (let i = 0; i < nextArgs.length; i += 1) {
@@ -250,56 +250,88 @@ function replaceRequestedModel(args, nextModel) {
 		}
 	}
 
-	return nextArgs;
-}
-
-function forwardToRealCodexOnce(codexBin, args, env = process.env, cleanup) {
-	return new Promise((resolve) => {
-		let settled = false;
-		let stdout = "";
-		let stderr = "";
-		const finalize = (exitCode) => {
-			if (settled) {
-				return;
-			}
+	return nextArgs;
+}
+
+function shouldCaptureForwardedCodexOutput(env = process.env) {
+	const override = (env.CODEX_MULTI_AUTH_CAPTURE_FORWARD_OUTPUT ?? "").trim();
+	if (override === "1") {
+		return true;
+	}
+	if (override === "0") {
+		return false;
+	}
+	// Windows child processes can report undefined isTTY; treat that as non-TTY so retry capture remains available.
+	return process.stdout.isTTY !== true || process.stderr.isTTY !== true;
+}
+
+function forwardToRealCodexOnce(
+	codexBin,
+	args,
+	env = process.env,
+	cleanup,
+	options = {},
+) {
+	return new Promise((resolve) => {
+		let settled = false;
+		let stdout = "";
+		let stderr = "";
+		const captureOutput = options.captureOutput === true;
+		const finalize = (exitCode) => {
+			if (settled) {
+				return;
+			}
 			settled = true;
 			try {
 				cleanup?.();
 			} catch {
 				// Best-effort cleanup only.
-			}
-			resolve({
-				exitCode,
-				output: `${stdout}\n${stderr}`.trim(),
-			});
-		};
-
-		const command = codexBin.launchWithNode ? process.execPath : codexBin.path;
-		const commandArgs = codexBin.launchWithNode ? [codexBin.path, ...args] : args;
-		const child = spawn(command, commandArgs, {
-			stdio: ["inherit", "pipe", "pipe"],
-			env,
-		});
-
-		child.stdout?.on("data", (chunk) => {
-			const text = typeof chunk === "string" ? chunk : chunk.toString("utf8");
-			stdout += text;
-			process.stdout.write(chunk);
-		});
-		child.stderr?.on("data", (chunk) => {
-			const text = typeof chunk === "string" ? chunk : chunk.toString("utf8");
-			stderr += text;
-			process.stderr.write(chunk);
-		});
-
-		child.once("error", (error) => {
-			const message = `Failed to launch real Codex CLI: ${String(error)}`;
-			stderr += `${stderr ? "\n" : ""}${message}`;
-			console.error(message);
-			finalize(1);
-		});
-
-		child.once("close", (code, signal) => {
+			}
+			resolve({
+				exitCode,
+				// No-capture output stays empty by design so retry parsing cannot
+				// reintroduce pipes that break terminal passthrough.
+				output: `${stdout}\n${stderr}`.trim(),
+			});
+		};
+
+		const command = codexBin.launchWithNode ? process.execPath : codexBin.path;
+		const commandArgs = codexBin.launchWithNode ? [codexBin.path, ...args] : args;
+		let child;
+		const failLaunch = (error) => {
+			const message = `Failed to launch real Codex CLI: ${String(error)}`;
+			stderr += `${stderr ? "\n" : ""}${message}`;
+			console.error(message);
+			finalize(1);
+		};
+		try {
+			child = spawn(command, commandArgs, {
+				stdio: captureOutput ? ["inherit", "pipe", "pipe"] : "inherit",
+				env,
+			});
+		} catch (error) {
+			failLaunch(error);
+			return;
+		}
+
+		if (captureOutput) {
+			child.stdout?.on("data", (chunk) => {
+				const text = typeof chunk === "string" ? chunk : chunk.toString("utf8");
+				stdout += text;
+				process.stdout.write(chunk);
+			});
+			child.stderr?.on("data", (chunk) => {
+				const text = typeof chunk === "string" ? chunk : chunk.toString("utf8");
+				stderr += text;
+				process.stderr.write(chunk);
+			});
+		}
+
+		child.once("error", (error) => {
+			failLaunch(error);
+		});
+
+		child.once("close", (code, signal) => {
 			if (signal) {
 				const signalNumber = signal === "SIGINT" ? 130 : 1;
 				finalize(signalNumber);
@@ -328,12 +360,15 @@ async function forwardToRealCodex(codexBin, rawArgs, baseEnv = process.env) {
 			compatibilityRequestedModel,
 			baseEnv,
 		);
-		const result = await forwardToRealCodexOnce(
-			codexBin,
-			compatibility.args,
-			compatibility.env,
-			compatibility.cleanup,
-		);
+		const result = await forwardToRealCodexOnce(
+			codexBin,
+			compatibility.args,
+			compatibility.env,
+			compatibility.cleanup,
+			{
+				captureOutput: shouldCaptureForwardedCodexOutput(compatibility.env),
+			},
+		);
 		lastExitCode = result.exitCode;
 		if (result.exitCode === 0) {
 			return result.exitCode;