Port PR #523: improve account rotation timeout safety#6
Merged
Conversation
…safety - Implement AbortSignal.timeout (180s default) to prevent hanging on stuck accounts - Fix TUI/web server freeze by immediately exiting request loop on user interrupt - Add proactive pool quota refresh when majority of accounts are rate-limited - MARK unhealthy accounts on timeout to trigger automatic rotation (cherry picked from commit 52d9b51)
(cherry picked from commit 3757ec3)
- AccountManager.shouldRefreshAllQuotas now includes soft quota checks in its blockedCount calculation - AccountManager.updateQuotaCache now uses stable refreshToken instead of stale positional index - Quota results now include refreshToken for reliable identification during async refreshes (cherry picked from commit f12df55)
- Increase default request_timeout_seconds to 600s (10 mins) - Apply 3x multiplier for streaming requests (max 30 mins) to prevent premature timeouts on active streams - Fix actualTimeoutSec logic in catch block by moving declaration outside try scope (cherry picked from commit b1306a4)
(cherry picked from commit 203d4db)
- Ensure proactive quota refresh only triggers on the first endpoint attempt - Persist account cooldown state immediately after a request timeout - Refine shouldRefreshAllQuotas to use family-specific rate limiting check - Update test descriptions to match implementation (75% or more) (cherry picked from commit a4745c1)
…e on disk errors (cherry picked from commit 506698b)
- Improve AbortSignal.any polyfill with proper listener cleanup - Add AbortSignal.timeout guard and fallback for older Node.js versions - Implement mergeAbortSignals helper for safe signal combination - Add exhaustive listener cleanup on signal abortion (cherry picked from commit 2b9b575)
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
|
thx bro i dont mind if u skid my work ily :D |
Owner
Author
|
Not sure why they archived the main project but I'll try to merge the PR
powered by AI
…--
Em qua., 1 de abr. de 2026, 08:59, Nunya9k ***@***.***>
escreveu:
*9nunya* left a comment (insign/opencode-antigravity-auth-updated#6)
<#6?email_source=notifications&email_token=AAIPXVOC3SCYAVTRCMWFM6T4TUADPA5CNFSNUABFM5UWIORPF5TWS5BNNB2WEL2JONZXKZKDN5WW2ZLOOQXTIMJWHE2TOOJTG442M4TFMFZW63VGMF2XI2DPOKSWK5TFNZ2LK4DSL5RW63LNMVXHIX3POBSW4X3DNRUWG2Y#issuecomment-4169579379>
thx bro i dont mind if u skid my work ily :D
—
Reply to this email directly, view it on GitHub
<#6?email_source=notifications&email_token=AAIPXVOC3SCYAVTRCMWFM6T4TUADPA5CNFSNUABFM5UWIORPF5TWS5BNNB2WEL2JONZXKZKDN5WW2ZLOOQXTIMJWHE2TOOJTG442M4TFMFZW63VGMF2XI2DPOKSWK5TFNZ2LK4DSL5RW63LNMVXHIX3POBSW4X3DNRUWG2Y#issuecomment-4169579379>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAIPXVJ4QWVFZRNWKQUGUQL4TUADPAVCNFSM6AAAAACXIUIIWOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHM2DCNRZGU3TSMZXHE>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
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
2 participants
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.
Port of archived upstream PR NoeFabris#523 from https://github.com/NoeFabris/opencode-antigravity-auth/pull/523\n\nOriginal author: @9nunya\n\nNotes:\n- Cherry-picked all upstream commits from PR NoeFabris#523\n- Verified locally with npm test