Skip to content

fix(queue): bound retries and Deep Research waits - #61

Merged
MarsLuay merged 1 commit into
mainfrom
issue-accept/29-5bfb87fa-9f93-42b8-83fe-e4312ab55da7
Sep 20, 2026
Merged

MarsLuay merged 1 commit into
mainfrom
issue-accept/29-5bfb87fa-9f93-42b8-83fe-e4312ab55da7

Conversation

@MarsLuay

Copy link
Copy Markdown
Owner

Summary

  • Make automatic retries finite by default (3 attempts, exponential backoff) through retryCurrentCommandIfEnabled(), with queueUnlimitedRetryWait as the only explicit unlimited wait/retry mode.
  • Keep Deep Research waits finite: longer max duration plus a stale-progress timeout. Progress can reset the stale timer; only unlimited mode removes the final bound.
  • Persist retry budget and research wait metadata in QUEUE_DURABLE_STATE_KEY, classify retryable vs compatibility/stop failures, and pause once on exhaustion with a popup-visible reason.

Fixes #29

Test plan

  • npm test (success-after-retry, exhaustion, finite/unlimited, finite Deep Research, stalled-research, stop-during-wait, recovery, non-retryable, exactly-once)
  • npm run check (test + lint + typecheck)
  • Confirm a default Deep Research wait eventually pauses instead of hanging
  • Confirm unlimited retry remains interruptible via Stop

Make automatic retries finite by default with exponential backoff, keep unlimited mode explicit, and stop Deep Research from waiting forever unless that setting is enabled.
Copilot AI lite review requested due to automatic review settings September 20, 2026 18:47

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@MarsLuay
MarsLuay merged commit dd86cab into main Sep 20, 2026
2 checks passed
@MarsLuay
MarsLuay deleted the issue-accept/29-5bfb87fa-9f93-42b8-83fe-e4312ab55da7 branch September 20, 2026 18: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.

[Suggestion] Improve retry and error handling for transient ChatGPT failures

2 participants