feat: support Codex ChatGPT OAuth auth - #126
Conversation
rajpratham1
left a comment
There was a problem hiding this comment.
LGTM. This is a well-structured implementation of ChatGPT OAuth support for the Codex harness. The feature is integrated consistently across configuration, runtime, developer tooling, and security validation, with strong accompanying test coverage. I don't see any blocking issues in the current diff. The main thing to verify through CI is that both API-key and OAuth authentication paths continue to work correctly and that production restrictions behave as expected.
|
Thanks for this, @eddie333016 — and sorry for the slow response. We're closing this PR. Per We don't want your idea to get lost, so we've captured it in #745 to triage on our side. Please feel free to open an |
Summary
Validation
QM_OAUTH_FINAL_420AFB8; model calls completed with nonzero usage, the child received no ambient API key, base URL, or Codex access token, and the auth lock was cleared.Need help on this PR? Tag
@codesmith-botwith what you need. Autofix is disabled.