Add Codex OAuth LLM provider#401
Open
BBossss wants to merge 1 commit intoOpen-Less:betafrom
Open
Conversation
PR Reviewer Guide 🔍(Review updated until commit 8391e56)Here are some key observations to aid the review process:
|
BBossss
added a commit
to BBossss/openless
that referenced
this pull request
May 10, 2026
Codex OAuth polishing should use the lighter Spark default for short dictation cleanup, and Windows builds need to locate Codex auth when HOME is absent in GUI sessions. This keeps the PR provider usable without requiring users to hand-copy auth files or choose a heavier model first. Constraint: PR Open-Less#401 targets a forked beta branch and must append to the existing pull request. Rejected: Keep gpt-5.5 as the default | it is overpowered for polishing and was observed timing out in the app. Rejected: Use only HOME for Codex auth lookup | Windows GUI environments commonly expose USERPROFILE instead. Confidence: high Scope-risk: narrow Directive: Keep Codex OAuth defaults optimized for short low-latency dictation polishing unless product requirements change. Tested: cargo test --manifest-path src-tauri/Cargo.toml --lib Tested: npm run build Tested: git diff --check Co-authored-by: OmX <omx@oh-my-codex.dev>
|
Persistent review updated to latest commit 33f59c1 |
9 tasks
Author
|
/review |
|
Persistent review updated to latest commit 963e906 |
Codex OAuth should let users spend their existing Codex subscription on short dictation polish, translation, and QA without storing another API key in OpenLess. The provider reads the local Codex auth file, keeps OpenAI-compatible providers on their existing path, uses a lighter Spark default for latency, and broadens Codex auth discovery for Windows GUI environments. The same change keeps Volcengine on the duration-priced BigASR resource validated locally and preserves proxy behavior so Codex network paths can use system proxy settings while loopback/local provider checks avoid the proxy. Constraint: Codex OAuth credentials must remain sourced from the Codex auth file and must not be copied into OpenLess credential storage. Constraint: PR Open-Less#401 is a fork PR from BBossss/openless:beta to appergb/openless:beta. Rejected: Require a separate OpenAI API key | the requested path was to reuse the existing Codex subscription. Rejected: Keep gpt-5.5 as the default | it is unnecessarily heavy for short dictation polish and was observed timing out. Rejected: Use only HOME for Codex auth lookup | Windows GUI sessions commonly expose USERPROFILE instead. Confidence: high Scope-risk: moderate Reversibility: clean Directive: Do not persist Codex OAuth tokens into OpenLess credentials; keep using the Codex auth file as the source of truth. Directive: Keep Codex OAuth defaults optimized for short low-latency dictation polishing unless product requirements change. Tested: git diff --cached --check Tested: cargo test --manifest-path src-tauri/Cargo.toml --lib Tested: npm run build Tested: PR Open-Less#401 CI passed on Windows, macOS, and Linux before this history-only squash. Co-authored-by: OmX <omx@oh-my-codex.dev>
|
Persistent review updated to latest commit 8391e56 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
User description
Summary
Verification
Notes
PR Type
Enhancement, Bug fix, Tests
Description
Add Codex OAuth provider support
~/.codex/auth.jsonRoute LLM actions through active provider
Update provider UI and locales
Fix defaults, proxy handling, and tests
Diagram Walkthrough
File Walkthrough
3 files
Reformat partial size parsing branchReformat Whisper constructor and testsReformat history retention calculation1 files
Switch default Volcengine resource ID5 files
Add Codex-aware provider commandsRoute active polishing through providerImplement Codex OAuth LLM clientMap Codex provider to localized nameAdd Codex OAuth preset to settings5 files
Add Codex OAuth provider translationAdd Codex OAuth provider translationAdd Codex OAuth provider translationAdd Codex OAuth provider translationAdd Codex OAuth provider translation1 files
Enable reqwest system proxy support