-
-
Notifications
You must be signed in to change notification settings - Fork 79.1k
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:fix-shape-clearClawSweeper found a clear likely implementation shape for this issue.ClawSweeper found a clear likely implementation shape for this issue.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: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.
Metadata
Metadata
Assignees
Labels
P2Normal backlog priority with limited blast radius.Normal backlog priority with limited blast radius.clawsweeper:fix-shape-clearClawSweeper found a clear likely implementation shape for this issue.ClawSweeper found a clear likely implementation shape for this issue.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: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.
Type
Fields
Give feedbackNo fields configured for issues without a type.
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.