Skip to content

fix: harden Codex automation lifecycle and error handling - #8

Merged
github-actions[bot] merged 4 commits into
mainfrom
fix/runtime-lifecycle-validation
Sep 10, 2026
Merged

github-actions[bot] merged 4 commits into
mainfrom
fix/runtime-lifecycle-validation

Conversation

@mastertyko

Copy link
Copy Markdown
Owner

Summary

  • Handle app-server stdin failures without crashing the host, reject outstanding requests, and preserve replacement processes.
  • Retry native actions only when failure is known to precede dispatch; never replay a possibly executed action because an RPC error mentions a stale thread.
  • Honor explicit cancellation signals when the host context supplies a different signal.
  • Reject Chrome executions that were waiting on preparation invalidated by a reset.
  • Bind Chrome trust probes to the exact app-server version being considered for trust.

Each fix includes regression coverage. The changes preserve stopped-session errors, bounded pre-dispatch retries, Chrome tab ownership, uncertain-outcome handling, and existing permission behavior.

Verification

  • bun run check: TypeScript passed; 244 tests passed across 22 files.
  • bun run qa:host: passed.
  • npm pack --dry-run: passed.
  • git diff --check: passed.
  • Live native smoke through OMO: computer_use_list_apps returned an application successfully.
  • Live Chrome probe on app-server 0.153.4: open, observe, reload, close, and cleanup passed.
  • Independent final review approved the changes with no blocking findings and reproduced the automated checks.

Notes

The Chrome live probe used a process-local trust override and did not alter persisted trust. Normal use of a newly installed app-server version still requires /codex-computer trust.

The native smoke run also emitted an existing OMO memory-reflection warning about a stale extension context. The same warning occurred before the changes and did not prevent the tool result.

@github-actions
github-actions Bot enabled auto-merge September 10, 2026 20:23
@github-actions
github-actions Bot merged commit 363ee21 into main Sep 10, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant