Skip to content

docs: triage "exceeded retry limit, last status: 429 Too Many Requests" - #6925

Closed
Howard-Jerry wants to merge 1 commit into
lidge-jun:devfrom
Howard-Jerry:docs/codex-retry-429-triage
Closed

Howard-Jerry wants to merge 1 commit into
lidge-jun:devfrom
Howard-Jerry:docs/codex-retry-429-triage

Conversation

@Howard-Jerry

@Howard-Jerry Howard-Jerry commented Oct 11, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Verification

  • Docs-only change: 1 file changed, 18 insertions (a single new section on the existing page; no front matter, sidebar or other docs touched).
  • cd docs-site && bun install && bun run build — exit 0: 371 packages installed, 569 page(s) built, internal-links check checked 79312 internal links across 569 pages with no failures, pagefind + sitemap completed.
  • Branch based on current dev (433764e9d).

Readiness

  • Required local validation passed (commands and results above; scope: docs build + internal links).
  • Branch on the latest dev commit (currently 433764e9d; will rebase if dev moves before review).
  • All correct Codex and CodeRabbit findings resolved (will handle as findings arrive).
  • Ready for review confirmation (will confirm once CI and review bots are clean).

Checklist

  • Scope stays focused and avoids unrelated cleanup.
  • Docs or release notes were updated when needed.
  • Security-sensitive changes were reviewed for secrets, auth, and unsafe defaults. (docs-only; no credentials or private paths included)

Review readiness checklist

This PR stays in draft until every box below is ticked. Tick all four boxes once the requirements are met:

  • Required local validation passed; commands, results, and any full-suite exception are documented.

  • I pushed my PR to a recent dev commit (at most 10 behind; a maintainer may still ask for the exact tip before merge).

  • I resolved all correct Codex and CodeRabbit findings.

  • My PR is ready for review.

@coderabbitai

coderabbitai Bot commented Oct 11, 2026

Copy link
Copy Markdown
Contributor

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

  • Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions

Copy link
Copy Markdown
Contributor

✅ Deterministic PR hygiene checks passed.

@github-actions github-actions Bot added the documentation Improvements or additions to documentation label Oct 11, 2026
@github-actions

Copy link
Copy Markdown
Contributor

⏳ DRAFT

  • review readiness checklist open (0/4 boxes ticked).

What to do

  • Tick all four boxes in the PR description once you're done (currently 0/4).

Review readiness checklist

  • ⬜ Required local validation passed; commands, results, and any full-suite exception are documented.
  • ⬜ I pushed my PR to a recent dev commit (at most 10 behind; a maintainer may still ask for the exact tip before merge).
  • ⬜ I resolved all correct Codex and CodeRabbit findings.
  • ⬜ My PR is ready for review.

0/4 boxes ticked.

This PR stays in draft until every box above is ticked.

@Ingwannu Ingwannu left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Scoped documentation review of 32992c8c53a35e4b2ad94825b134ebdb942baceb:

The distinction between local admission, upstream quota and transport is useful. Please adjust the hard-lock table's instruction "Release it with a validated usage refresh" to "Refresh usage to reevaluate the lock; it releases only when fresh evidence is below the applicable thresholds or validates that the previously blocking window is absent." getMainAccountHardLockStatus in src/codex/main-account-hard-lock.ts still returns blocked for a fresh reading at/above its threshold. A refresh therefore is not a guaranteed unlock. Disabling the policy is a deliberate operator override, not restoration of upstream capacity.

Keep the upstream row explicitly plan-dependent and prefer the returned account windows/reset times: current official pricing guidance distinguishes plan-specific five-hour/weekly limits. The app-server reset contract also requires refreshing limits after a reset and allows a no-eligible-window result; spending a credit is not itself evidence that every limit cleared.

The PR remains Draft with incomplete author readiness; this COMMENT is not approval. I did not run a new docs build, account refresh, credit redemption or paid/live request.

lidge-jun added a commit that referenced this pull request Oct 13, 2026
…7055, #7056, #6925) (#7065)

* fix(chatgpt): preserve launcher when app inspection fails

Carries #7055. Report unknown inspection state and refuse an unconfirmed restart.

Co-authored-by: lcxhh <lcxhh521@gmail.com>

* fix(proxy): bound SOCKS5 TLS handshake lifetime

Carries #7056. Clear deadline and listeners on every TLS handshake exit while keeping certificate verification.

Co-authored-by: lcxhh <lcxhh521@gmail.com>

* docs: triage exceeded-retry-limit 429 errors

Carries #6925. Align refresh release conditions and the resolved credit rediscovery reference with current dev.

Co-authored-by: Howard-Jerry <197214646+Howard-Jerry@users.noreply.github.com>

---------

Co-authored-by: JUN <243035832+lidge-jun@users.noreply.github.com>
Co-authored-by: lcxhh <lcxhh521@gmail.com>
Co-authored-by: Howard-Jerry <197214646+Howard-Jerry@users.noreply.github.com>
@lidge-jun

Copy link
Copy Markdown
Owner

Thanks @Howard-Jerry! This change landed on dev in #7065 (1edd55a), carried unchanged in its own commit with you credited in a Co-authored-by trailer. The fork PR never got the repository's main CI, so I carried it onto an upstream branch to run it. Closing this PR because the carry superseded it.

@lidge-jun lidge-jun closed this Oct 13, 2026
lidge-jun added a commit that referenced this pull request Oct 13, 2026
…top Bad credentials (#7053, #7054) (#7062)

* fix(kiro): accept authorized social device grants

Carries #7053. Preserve token, profile, principal, and HTTP validation.

Co-authored-by: Veritas-7 <234569343+Veritas-7@users.noreply.github.com>

* fix(kiro): quarantine exact desktop bad-credentials refusals

Carries #7054. Keep AWS SSO and other status/message pairs retryable.

Co-authored-by: Veritas-7 <234569343+Veritas-7@users.noreply.github.com>

* fix(chatgpt): preserve launcher when app inspection fails

Carries #7055. Report unknown inspection state and refuse an unconfirmed restart.

Co-authored-by: lcxhh <lcxhh521@gmail.com>

* fix(proxy): bound SOCKS5 TLS handshake lifetime

Carries #7056. Clear deadline and listeners on every TLS handshake exit while keeping certificate verification.

Co-authored-by: lcxhh <lcxhh521@gmail.com>

* docs: triage exceeded-retry-limit 429 errors

Carries #6925. Align refresh release conditions and the resolved credit rediscovery reference with current dev.

Co-authored-by: Howard-Jerry <197214646+Howard-Jerry@users.noreply.github.com>

---------

Co-authored-by: JUN <243035832+lidge-jun@users.noreply.github.com>
Co-authored-by: Veritas-7 <234569343+Veritas-7@users.noreply.github.com>
Co-authored-by: lcxhh <lcxhh521@gmail.com>
Co-authored-by: Howard-Jerry <197214646+Howard-Jerry@users.noreply.github.com>
Co-authored-by: jun <jun@juns-Mac-Studio.local>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants