oh-my-codex 0.21.19 → 0.21.20

package/Cargo.lock

@@ -32,7 +32,7 @@ checksum = "6b947ae49db0d222b1dbc6b113ce7248a3fc3a6ca21b696717bfc000ba4484d8"
 
 [[package]]
 name = "omx-api"
-version = "0.21.19"
+version = "0.21.20"
 dependencies = [
  "serde",
  "serde_json",
@@ -40,14 +40,14 @@ dependencies = [
 
 [[package]]
 name = "omx-explore-harness"
-version = "0.21.19"
+version = "0.21.20"
 dependencies = [
  "libc",
 ]
 
 [[package]]
 name = "omx-mux"
-version = "0.21.19"
+version = "0.21.20"
 dependencies = [
  "serde",
  "serde_json",
@@ -55,7 +55,7 @@ dependencies = [
 
 [[package]]
 name = "omx-runtime"
-version = "0.21.19"
+version = "0.21.20"
 dependencies = [
  "fs2",
  "libc",
@@ -67,7 +67,7 @@ dependencies = [
 
 [[package]]
 name = "omx-runtime-core"
-version = "0.21.19"
+version = "0.21.20"
 dependencies = [
  "fs2",
  "serde",
@@ -76,7 +76,7 @@ dependencies = [
 
 [[package]]
 name = "omx-sparkshell"
-version = "0.21.19"
+version = "0.21.20"
 dependencies = [
  "omx-mux",
 ]

package/Cargo.toml

@@ -11,7 +11,7 @@ resolver = "2"
 
 [workspace.package]
 
-version = "0.21.19"
+version = "0.21.20"
 
 edition = "2021"
 rust-version = "1.73"

package/dist/cli/__tests__/plugin-cache-live-pin-3790.test.js

@@ -266,7 +266,7 @@ describe("issue 3790 live plugin cache pins", () => {
                 const packaged = await resolvePackagedOmxMarketplace(packageRoot);
                 assert.ok(packaged);
                 const pluginManifest = JSON.parse(await readFile(packaged.pluginManifestPath, "utf-8"));
-                assert.equal(pluginManifest.version, "0.21.19");
+                assert.equal(pluginManifest.version, "0.21.20");
                 const published = await materializePackagedOmxPluginCache(codexHomeDir, packaged);
                 assert.equal(published.status, "materialized");
                 assert.ok(legacySnapshots.every((snapshot) => published.cacheDir !== snapshot));

package/dist/team/__tests__/tmux-session.test.js

@@ -4006,6 +4006,36 @@ esac
             assert.equal(result, false, "waitForWorkerReady with timeout=0 should return false, not run checks after deadline");
         });
     });
+    it('enforces deadline on every poll including when ready (Codex finding 4310966027)', async () => {
+        // Test that even if a poll returns true (ready), we recheck the deadline.
+        // Mock tmux to return ready after a delay that causes us to exceed the timeout.
+        const delaySeconds = 0.15;
+        const timeoutMs = 100;
+        // With a timeout shorter than the delay, the poll will complete after deadline
+        // and should return false despite the ready output.
+        await withMockTmuxFixture('omx-worker-ready-deadline-enforce-', (logPath) => `#!/bin/sh
+set -eu
+printf '%s\\n' "$*" >> "${logPath}"
+case "$1" in
+  capture-pane)
+    # Simulate slow capture-pane that takes longer than the remaining timeout
+    sleep ${delaySeconds}
+    cat <<'EOF'
+${READY_HELPER_CAPTURE}
+EOF
+    exit 0
+    ;;
+  *)
+    exit 0
+    ;;
+esac`, async () => {
+            // Poll with a very short timeout that will be exceeded during the capture
+            const result = waitForWorkerReady('omx-team-x', 1, timeoutMs);
+            // Even though capture-pane returns ready, the deadline was exceeded
+            // so we should return false
+            assert.equal(result, false, "waitForWorkerReady should return false when poll completes after deadline");
+        });
+    });
 });
 describe('waitForWorkerReadyAsync parity', () => {
     it('uses visible capture-pane argv without tail flags', async () => {
@@ -4288,6 +4318,36 @@ esac
             assert.equal(result, false, "waitForWorkerReadyAsync with timeout=0 should return false, not run checks after deadline");
         });
     });
+    it('enforces deadline on every poll including when ready (Codex finding 4310966027)', async () => {
+        // Test that even if a poll returns true (ready), we recheck the deadline.
+        // Mock tmux to return ready after a delay that causes us to exceed the timeout.
+        const delaySeconds = 0.15;
+        const timeoutMs = 100;
+        // With a timeout shorter than the delay, the poll will complete after deadline
+        // and should return false despite the ready output.
+        await withMockTmuxFixture('omx-worker-ready-deadline-enforce-async-', (logPath) => `#!/bin/sh
+set -eu
+printf '%s\\n' "$*" >> "${logPath}"
+case "$1" in
+  capture-pane)
+    # Simulate slow capture-pane that takes longer than the remaining timeout
+    sleep ${delaySeconds}
+    cat <<'EOF'
+${READY_HELPER_CAPTURE}
+EOF
+    exit 0
+    ;;
+  *)
+    exit 0
+    ;;
+esac`, async () => {
+            // Poll with a very short timeout that will be exceeded during the capture
+            const result = await waitForWorkerReadyAsync('omx-team-x', 1, timeoutMs);
+            // Even though capture-pane returns ready, the deadline was exceeded
+            // so we should return false
+            assert.equal(result, false, "waitForWorkerReadyAsync should return false when poll completes after deadline");
+        });
+    });
 });
 describe('createTeamSession tmux instance tagging', () => {
     it('rejects split-window output unless it contains one canonical pane id and optional expected receipt', () => {