Skip to content

Retry async candidate load after rate-limit window - #139

Merged
adamtheturtle merged 1 commit into
mainfrom
fix/issue-136
Aug 23, 2026
Merged

Retry async candidate load after rate-limit window#139
adamtheturtle merged 1 commit into
mainfrom
fix/issue-136

Conversation

@adamtheturtle

@adamtheturtle adamtheturtle commented Aug 23, 2026

Copy link
Copy Markdown
Owner

Wait out the remaining rate-limit interval instead of returning immediately. Closes #136


Note

Low Risk
Small lifecycle/timing change in candidate loading; generation tokens still gate stale results, so risk is limited to brief delay/retry behavior.

Overview
When the palette is re-presented inside the 50ms load spacing window, the async candidate .task now sleeps for the remaining delay and retries instead of bailing out and never loading.

CandidateLoadGeneration.delayUntilNextLoadAllowed() reports that remaining interval. During the wait, isLoading is cleared so retained results stay visible without a stuck spinner. Generation tokens still reject cancelled or stale providers.

Reviewed by Cursor Bugbot for commit ff11331. Bugbot is set up for automated code reviews on this repo. Configure here.

@adamtheturtle
adamtheturtle merged commit 56dff56 into main Aug 23, 2026
@adamtheturtle
adamtheturtle deleted the fix/issue-136 branch August 23, 2026 16:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Rate-limited reopen stays empty

1 participant