@ag-eco/agentplate-cli 0.15.1 → 0.15.2

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@ag-eco/agentplate-cli",
-	"version": "0.15.1",
+	"version": "0.15.2",
 	"description": "Multi-agent orchestration for AI coding agents — spawn workers in git worktrees via tmux, coordinate through SQLite mail, merge with tiered conflict resolution. Pluggable runtime adapters for Claude Code, Pi, and more.",
 	"author": "Jaymin West",
 	"license": "MIT",

package/src/runtimes/codex.test.ts

@@ -20,7 +20,7 @@ describe("CodexRuntime", () => {
 	});
 
 	describe("buildSpawnCommand", () => {
-		test("basic command uses interactive codex with --full-auto", () => {
+		test("basic command uses interactive codex with workspace-write sandbox + no approvals", () => {
 			const opts: SpawnOpts = {
 				model: "gpt-5-codex",
 				permissionMode: "bypass",
@@ -28,7 +28,7 @@ describe("CodexRuntime", () => {
 				env: {},
 			};
 			const cmd = runtime.buildSpawnCommand(opts);
-			expect(cmd).toContain("codex --full-auto");
+			expect(cmd).toContain("codex --sandbox workspace-write --ask-for-approval never");
 			expect(cmd).toContain("--model gpt-5-codex");
 			expect(cmd).toContain("Read AGENTS.md");
 		});
@@ -42,7 +42,7 @@ describe("CodexRuntime", () => {
 					env: {},
 				};
 				const cmd = runtime.buildSpawnCommand(opts);
-				expect(cmd).toContain("codex --full-auto");
+				expect(cmd).toContain("codex --sandbox workspace-write --ask-for-approval never");
 				expect(cmd).not.toContain(" --model ");
 			}
 		});
@@ -173,7 +173,7 @@ describe("CodexRuntime", () => {
 			};
 			const cmd = runtime.buildSpawnCommand(opts);
 			expect(cmd).toBe(
-				"codex --full-auto --model gpt-5-codex 'Read AGENTS.md for your task assignment and begin immediately.'",
+				"codex --sandbox workspace-write --ask-for-approval never --model gpt-5-codex 'Read AGENTS.md for your task assignment and begin immediately.'",
 			);
 		});
 
@@ -256,7 +256,14 @@ describe("CodexRuntime", () => {
 	describe("buildPrintCommand", () => {
 		test("basic command without model", () => {
 			const argv = runtime.buildPrintCommand("Summarize this diff");
-			expect(argv).toEqual(["codex", "exec", "--full-auto", "--ephemeral", "Summarize this diff"]);
+			expect(argv).toEqual([
+				"codex",
+				"exec",
+				"--sandbox",
+				"workspace-write",
+				"--ephemeral",
+				"Summarize this diff",
+			]);
 		});
 
 		test("command with model override", () => {
@@ -264,7 +271,8 @@ describe("CodexRuntime", () => {
 			expect(argv).toEqual([
 				"codex",
 				"exec",
-				"--full-auto",
+				"--sandbox",
+				"workspace-write",
 				"--ephemeral",
 				"--model",
 				"gpt-5-codex",
@@ -277,7 +285,8 @@ describe("CodexRuntime", () => {
 			expect(argv).toEqual([
 				"codex",
 				"exec",
-				"--full-auto",
+				"--sandbox",
+				"workspace-write",
 				"--ephemeral",
 				"--model",
 				"gpt-5.4",
@@ -296,14 +305,14 @@ describe("CodexRuntime", () => {
 			expect(argv[argv.length - 1]).toBe(prompt);
 		});
 
-		test("without model, argv has exactly 5 elements", () => {
+		test("without model, argv has exactly 6 elements", () => {
 			const argv = runtime.buildPrintCommand("prompt text");
-			expect(argv.length).toBe(5);
+			expect(argv.length).toBe(6);
 		});
 
-		test("with model, argv has exactly 7 elements", () => {
+		test("with model, argv has exactly 8 elements", () => {
 			const argv = runtime.buildPrintCommand("prompt text", "gpt-5-codex");
-			expect(argv.length).toBe(7);
+			expect(argv.length).toBe(8);
 		});
 
 		test("does not include --json (print expects plain text stdout)", () => {
@@ -318,27 +327,53 @@ describe("CodexRuntime", () => {
 	});
 
 	describe("detectReady", () => {
-		test("returns ready for empty pane", () => {
-			const state = runtime.detectReady("");
-			expect(state).toEqual({ phase: "ready" });
-		});
-
-		test("returns ready for any pane content", () => {
-			const state = runtime.detectReady("Loading Codex...\nPlease wait");
-			expect(state).toEqual({ phase: "ready" });
-		});
-
-		test("returns ready for NDJSON output", () => {
-			const state = runtime.detectReady(
-				'{"type":"thread.started","thread_id":"abc"}\n{"type":"turn.started"}',
-			);
-			expect(state).toEqual({ phase: "ready" });
-		});
-
-		test("no dialog phase — Codex has no trust dialog", () => {
-			const state = runtime.detectReady("trust this folder");
-			expect(state.phase).not.toBe("dialog");
-			expect(state.phase).toBe("ready");
+		test("returns loading for empty / pre-header pane", () => {
+			expect(runtime.detectReady("")).toEqual({ phase: "loading" });
+			expect(runtime.detectReady("Loading Codex...\nPlease wait")).toEqual({ phase: "loading" });
+		});
+
+		test("returns ready once the Codex header box renders", () => {
+			const pane = [
+				"╭──────────────────────────────────────╮",
+				"│ >_ OpenAI Codex (v0.128.0)           │",
+				"│ model: gpt-5.5 high   /model to change│",
+				"╰──────────────────────────────────────╯",
+				"› Explain this codebase",
+			].join("\n");
+			expect(runtime.detectReady(pane)).toEqual({ phase: "ready" });
+		});
+
+		test("skips the blocking 'Update available' modal via type:2", () => {
+			const pane = [
+				"✨ Update available! 0.128.0 -> 0.131.0",
+				"› 1. Update now (runs `npm install -g @openai/codex`)",
+				"  2. Skip",
+				"  3. Skip until next version",
+				"  Press enter to continue",
+			].join("\n");
+			expect(runtime.detectReady(pane)).toEqual({ phase: "dialog", action: "type:2" });
+		});
+
+		test("passive post-skip update banner is not treated as a modal", () => {
+			// The non-blocking banner says "Run npm install" and has no numbered options.
+			const pane = [
+				"╭───────────────────────────────────────────╮",
+				"│ ✨ Update available! 0.128.0 -> 0.135.0     │",
+				"│ Run npm install -g @openai/codex to update. │",
+				"╰───────────────────────────────────────────╯",
+				"│ >_ OpenAI Codex (v0.128.0)                 │",
+			].join("\n");
+			expect(runtime.detectReady(pane)).toEqual({ phase: "ready" });
+		});
+
+		test("confirms the trust-directory prompt via retryable type:1", () => {
+			const pane = [
+				"Do you trust the contents of this directory?",
+				"› 1. Yes, continue",
+				"  2. No, quit",
+				"  Press enter to continue",
+			].join("\n");
+			expect(runtime.detectReady(pane)).toEqual({ phase: "dialog", action: "type:1" });
 		});
 	});
 
@@ -728,7 +763,7 @@ describe("CodexRuntime integration: spawn command structure", () => {
 			env: { AGENTPLATE_AGENT_NAME: "builder-1" },
 		});
 		expect(cmd).toBe(
-			"codex --full-auto --model gpt-5-codex 'Read AGENTS.md for your task assignment and begin immediately.'",
+			"codex --sandbox workspace-write --ask-for-approval never --model gpt-5-codex 'Read AGENTS.md for your task assignment and begin immediately.'",
 		);
 	});
 
@@ -741,7 +776,9 @@ describe("CodexRuntime integration: spawn command structure", () => {
 			appendSystemPrompt: baseDefinition,
 			env: { AGENTPLATE_AGENT_NAME: "coordinator" },
 		});
-		expect(cmd).toContain("codex --full-auto --model gpt-5-codex");
+		expect(cmd).toContain(
+			"codex --sandbox workspace-write --ask-for-approval never --model gpt-5-codex",
+		);
 		expect(cmd).toContain("# Coordinator");
 		expect(cmd).toContain("You are the coordinator agent.");
 		expect(cmd).toContain("Read AGENTS.md");
@@ -755,7 +792,9 @@ describe("CodexRuntime integration: spawn command structure", () => {
 			appendSystemPromptFile: "/project/.agentplate/agent-defs/coordinator.md",
 			env: { AGENTPLATE_AGENT_NAME: "coordinator" },
 		});
-		expect(cmd).toContain("codex --full-auto --model gpt-5-codex");
+		expect(cmd).toContain(
+			"codex --sandbox workspace-write --ask-for-approval never --model gpt-5-codex",
+		);
 		expect(cmd).toContain("$(cat '/project/.agentplate/agent-defs/coordinator.md')");
 		expect(cmd).toContain("Read AGENTS.md");
 	});

package/src/runtimes/codex.ts

@@ -95,7 +95,10 @@ export class CodexRuntime implements AgentRuntime {
 		const bareModel = CodexRuntime.stripProviderPrefix(opts.model);
 		// When model comes from default manifest aliases (sonnet/opus/haiku),
 		// omit --model so Codex uses the user's configured default model.
-		let cmd = "codex --full-auto";
+		// `--full-auto` was removed in modern codex-cli; the equivalent (write to
+		// the workspace without per-command approval prompts) is
+		// `--sandbox workspace-write --ask-for-approval never`.
+		let cmd = "codex --sandbox workspace-write --ask-for-approval never";
 		if (!CodexRuntime.MANIFEST_ALIASES.has(bareModel)) {
 			cmd += ` --model ${bareModel}`;
 		}
@@ -135,7 +138,9 @@ export class CodexRuntime implements AgentRuntime {
 	 * @returns Argv array for Bun.spawn
 	 */
 	buildPrintCommand(prompt: string, model?: string): string[] {
-		const cmd = ["codex", "exec", "--full-auto", "--ephemeral"];
+		// `--full-auto` is deprecated in `codex exec` (still works, warns); the
+		// non-deprecated form is `--sandbox workspace-write`.
+		const cmd = ["codex", "exec", "--sandbox", "workspace-write", "--ephemeral"];
 		if (model !== undefined) {
 			// Strip provider prefix — Codex CLI expects bare model names.
 			cmd.push("--model", CodexRuntime.stripProviderPrefix(model));
@@ -174,13 +179,37 @@ export class CodexRuntime implements AgentRuntime {
 	}
 
 	/**
-	 * Codex interactive startup is treated as ready once a pane exists.
+	 * Detect Codex TUI readiness from a tmux pane content snapshot.
 	 *
-	 * @param _paneContent - Captured tmux pane content (unused)
-	 * @returns Always `{ phase: "ready" }`
+	 * Observed against codex-cli 0.128.x, the interactive TUI can show up to two
+	 * blocking dialogs before it is usable, which we clear via dialog actions:
+	 * - "Update available" modal (outdated install): "1. Update now / 2. Skip …".
+	 *   Selected via `type:2` (Skip). Distinguished from the passive post-skip
+	 *   "Run npm install …" banner by the "Update now" option text.
+	 * - "Do you trust the contents of this directory?" prompt: confirm the default
+	 *   "Yes, continue" with Enter.
+	 * Ready once the Codex header box ("OpenAI Codex") renders.
+	 *
+	 * @param paneContent - Captured tmux pane content
+	 * @returns Current readiness phase
 	 */
-	detectReady(_paneContent: string): ReadyState {
-		return { phase: "ready" };
+	detectReady(paneContent: string): ReadyState {
+		// Blocking update modal (numbered options) — skip it. The passive banner
+		// shown afterwards says "Run npm install" and lacks the "Update now" option.
+		if (paneContent.includes("Update now") && paneContent.includes("Skip")) {
+			return { phase: "dialog", action: "type:2" };
+		}
+		// Trust-directory prompt → select "1. Yes, continue". Use a `type:` action
+		// (not bare Enter) so the readiness loop re-fires it until the dialog clears
+		// — non-type dialog actions are sent only once (see waitForTuiReady).
+		if (paneContent.includes("Do you trust")) {
+			return { phase: "dialog", action: "type:1" };
+		}
+		// Ready: the Codex header box has rendered.
+		if (paneContent.includes("OpenAI Codex")) {
+			return { phase: "ready" };
+		}
+		return { phase: "loading" };
 	}
 
 	/**

package/src/version.ts

@@ -2,4 +2,4 @@
  * Single source of truth for the package version, shared by every bundled bin
  * (ap/agentplate, lm/loam, sr, tl). Updated by scripts/version-bump.ts.
  */
-export const VERSION = "0.15.1";
+export const VERSION = "0.15.2";