oh-my-codex 0.21.2 → 0.21.4

package/dist/cli/index.js

@@ -176,7 +176,7 @@ Options:
                 (shorthand for: -c model_reasoning_effort="xhigh")
   --madmax      DANGEROUS: bypass Codex approvals and sandbox
                 (alias for --dangerously-bypass-approvals-and-sandbox)
-  --spark       Use the Codex spark model (~1.3x faster) for team workers only
+  --spark       Use the configured spark-lane model for team workers only
                 Workers get the configured low-complexity team model; leader model unchanged
   --madmax-spark  spark model for workers + bypass approvals for leader and workers
                 (shorthand for: --spark --madmax)
@@ -2438,10 +2438,38 @@ export function registerDetachedHudLayoutReconcileHook(options) {
         env: buildDetachedHudHookEnv(options.baseEnv ?? process.env, options.sessionId, detachedLeaderPaneId, tmuxEnvValue, options.omxBin, options.omxRootOverride),
     });
 }
-export const DETACHED_TMUX_HISTORY_LIMIT = 500;
+// #3611: this clamp bounds tmux scrollback memory for detached leader sessions,
+// but it also decides how much assistant output survives in the pane. At 500
+// lines a long session drops multi-line responses out of scrollback while the
+// session still holds them (`/copy` returns the whole response), which reads as
+// "the response was cut to one line". Keep the clamp, but at a size that holds a
+// realistic transcript, and let operators override it.
+export const DETACHED_TMUX_HISTORY_LIMIT = 5000;
+export const DETACHED_TMUX_HISTORY_LIMIT_MIN = 500;
+export const DETACHED_TMUX_HISTORY_LIMIT_MAX = 200_000;
+export const DETACHED_TMUX_HISTORY_LIMIT_ENV = "OMX_TMUX_HISTORY_LIMIT";
 const TMUX_HOOK_INDEX_MAX = 1_000_000;
+// Resolves the scrollback clamp applied to OMX-owned tmux leader sessions.
+// Invalid, non-integer, or out-of-range overrides fall back to the default so a
+// typo can never silently shrink a user's transcript to nothing.
+export function resolveDetachedTmuxHistoryLimit(env = process.env) {
+    const raw = env[DETACHED_TMUX_HISTORY_LIMIT_ENV];
+    if (raw === undefined)
+        return DETACHED_TMUX_HISTORY_LIMIT;
+    const trimmed = raw.trim();
+    if (!/^[0-9]+$/.test(trimmed))
+        return DETACHED_TMUX_HISTORY_LIMIT;
+    const parsed = Number.parseInt(trimmed, 10);
+    if (!Number.isSafeInteger(parsed))
+        return DETACHED_TMUX_HISTORY_LIMIT;
+    if (parsed < DETACHED_TMUX_HISTORY_LIMIT_MIN)
+        return DETACHED_TMUX_HISTORY_LIMIT_MIN;
+    if (parsed > DETACHED_TMUX_HISTORY_LIMIT_MAX)
+        return DETACHED_TMUX_HISTORY_LIMIT_MAX;
+    return parsed;
+}
 function setDetachedTmuxSessionHistoryLimit(sessionName, leaderPaneId) {
-    const boundedHistoryLimit = String(DETACHED_TMUX_HISTORY_LIMIT);
+    const boundedHistoryLimit = String(resolveDetachedTmuxHistoryLimit());
     try {
         execTmuxFileSync(["set-option", "-q", "-t", sessionName, "history-limit", boundedHistoryLimit], { stdio: "ignore" });
     }
@@ -6316,8 +6344,8 @@ async function runCodex(cwd, args, sessionId, workerDefaultModel, codexHomeOverr
                             // evaluation sees it. Retrying this idempotent first owner-guarded
                             // mutation once gives the server a command boundary without relaxing
                             // any part of the captured session, window, pane, PID, or owner fence.
-                            runDetachedLeaderMutation(authority, ["set-option", "-q", "-t", authority.sessionName, "history-limit", String(DETACHED_TMUX_HISTORY_LIMIT)], true, true);
-                            runDetachedLeaderMutation(authority, ["set-option", "-pq", "-t", authority.paneId, "history-limit", String(DETACHED_TMUX_HISTORY_LIMIT)]);
+                            runDetachedLeaderMutation(authority, ["set-option", "-q", "-t", authority.sessionName, "history-limit", String(resolveDetachedTmuxHistoryLimit())], true, true);
+                            runDetachedLeaderMutation(authority, ["set-option", "-pq", "-t", authority.paneId, "history-limit", String(resolveDetachedTmuxHistoryLimit())]);
                             // #3266: the owned leader pane must close with its process so a normal
                             // child exit destroys the session naturally even when the user's tmux
                             // config inherits remain-on-exit=on/failed.