-
-
Notifications
You must be signed in to change notification settings - Fork 80.7k
Feature request: native Codex quota/auth diagnosis plus brokered reauth execution #54278
Copy link
Copy link
Open
Labels
P2Normal backlog priority with limited blast radius.Normal backlog priority with limited blast radius.clawsweeper:needs-maintainer-reviewClawSweeper marked this issue as needing maintainer review before automation.ClawSweeper marked this issue as needing maintainer review before automation.clawsweeper:needs-product-decisionClawSweeper marked this issue as needing a product or behavior decision.ClawSweeper marked this issue as needing a product or behavior decision.clawsweeper:needs-security-reviewClawSweeper marked this issue as needing security-sensitive review.ClawSweeper marked this issue as needing security-sensitive review.clawsweeper:no-new-fix-prClawSweeper does not recommend queueing a new automated fix PR for this issue.ClawSweeper does not recommend queueing a new automated fix PR for this issue.impact:auth-providerAuth, provider routing, model choice, or SecretRef resolution may break.Auth, provider routing, model choice, or SecretRef resolution may break.impact:securitySecurity boundary, credential, authz, sandbox, or sensitive-data risk.Security boundary, credential, authz, sandbox, or sensitive-data risk.issue-rating: 🌊 off-meta tidepoolIssue quality rating does not apply to this item.Issue quality rating does not apply to this item.maturity:stableIssue affects a taxonomy feature currently scored M4/M5.Issue affects a taxonomy feature currently scored M4/M5.staleMarked as stale due to inactivityMarked as stale due to inactivity
Description
Metadata
Metadata
Assignees
Labels
P2Normal backlog priority with limited blast radius.Normal backlog priority with limited blast radius.clawsweeper:needs-maintainer-reviewClawSweeper marked this issue as needing maintainer review before automation.ClawSweeper marked this issue as needing maintainer review before automation.clawsweeper:needs-product-decisionClawSweeper marked this issue as needing a product or behavior decision.ClawSweeper marked this issue as needing a product or behavior decision.clawsweeper:needs-security-reviewClawSweeper marked this issue as needing security-sensitive review.ClawSweeper marked this issue as needing security-sensitive review.clawsweeper:no-new-fix-prClawSweeper does not recommend queueing a new automated fix PR for this issue.ClawSweeper does not recommend queueing a new automated fix PR for this issue.impact:auth-providerAuth, provider routing, model choice, or SecretRef resolution may break.Auth, provider routing, model choice, or SecretRef resolution may break.impact:securitySecurity boundary, credential, authz, sandbox, or sensitive-data risk.Security boundary, credential, authz, sandbox, or sensitive-data risk.issue-rating: 🌊 off-meta tidepoolIssue quality rating does not apply to this item.Issue quality rating does not apply to this item.maturity:stableIssue affects a taxonomy feature currently scored M4/M5.Issue affects a taxonomy feature currently scored M4/M5.staleMarked as stale due to inactivityMarked as stale due to inactivity
Type
Fields
Priority
None yet
Summary
OpenClaw should provide native Codex/ChatGPT OAuth reliability handling by separating quota-window exhaustion from true OAuth credential failure, and by supporting brokered reauth execution when reauthentication is actually required.
User-facing goal
As a user, I want one reliable ChatGPT/Codex OAuth experience:
Problem
Today, quota exhaustion and unrecoverable OAuth failure can be confused operationally.
For example:
refresh_token_reusedshould trigger guided reauthThese are different states, but they are easy to collapse into a single generic "auth failed" workflow.
Requested feature set
OpenClaw should distinguish at least:
okexpiring_soonquota_waitreauth_requiredBehavior:
quota_wait: do not trigger reauthreauth_required: prompt for or invoke a bounded reauth adapterOpenClaw should surface Codex rate-limit state natively, including both the 5-hour and weekly windows.
Recommended display format:
Formatting rules:
When OpenClaw determines the state is
reauth_required, it should support invoking a narrow reauth adapter rather than assuming it must handle browser-based reauth itself.This is especially important for Docker/sandboxed deployments where:
Conceptually:
Control UI impact
Yes, this should include Control UI changes.
Suggested Control UI behavior:
quota_wait, show renewal times instead of suggesting reauthreauth_required, show a clear guided reauth action if a reauth adapter is availableWhy this matters
For users, this is one feature set: a more reliable ChatGPT/Codex OAuth experience.
Internally, it can still be split into:
That split keeps the trust boundary clean while delivering one coherent recovery UX.
Additional context
I implemented a local host-side prototype in a Docker-based OpenClaw sandbox that:
quota_waitseparately fromreauth_requiredreauth_requiredI am not attaching that patch directly here because parts of it are deployment-specific, but it validated the behavior and UX split described above.