Repository navigation
fix: improve Codex usage polling reliability - #30
Merged
Merged
Conversation
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
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.
What does this PR do?
Temporary Codex usage failures currently surface a generic connection error. This change retries recoverable failures and keeps the last reading visibly stale with its age, while showing specific guidance for login errors, rate limits, timeouts, and invalid responses.
Retry-Afterand increasing rate-limit cooldowns, including after manual refresh or provider switching.Related issue
None.
Checklist
task checkpasses locally (type-check, Clippy lint, all tests)Screenshots / notes
Validated on macOS: 92 frontend tests and 112 Rust tests pass, along with Clippy, TypeScript checks, and
npm run build. Local HTTP fixtures cover recovery, retry limits, timeouts, authentication failures, invalid responses, and server cooldowns without contacting the live usage API. Tests also cover stale-data age, account isolation, and history exclusion.