| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
Sorry, something went wrong.
|
The following comment was made by an LLM, it may be inaccurate: Based on my search results, I found one potentially related PR: PR #38958: fix(session): cap retry attempts to prevent infinite thinking on API errors This PR appears to address a similar issue of capping retry attempts to prevent infinite sessions on API errors. It may be worth reviewing to see if it's a prior attempt at fixing the same problem or if it's been superseded by PR #41699. All other results in the search were either PR #41699 itself or unrelated retry handling fixes. No other duplicate PRs were identified. |
Sorry, something went wrong.
|
Re the duplicate check: #38958 hardcodes a cap of 5 (single file, no tests). Since #41939 merged the fixed cap upstream, this PR is now rebased on it and adds the per-provider knobs: configurable retry (0 disables) and backoffDelay, with tests, docs, and schema/SDK updates. |
Sorry, something went wrong.
|
Thanks for updating your PR! It now meets our contributing guidelines. 👍 |
Sorry, something went wrong.
Builds on the fixed RETRY_MAX_RETRIES cap from anomalyco#41939: providers differ in how aggressively they should be retried, so expose 'retry' (max attempts, 0 disables) and 'backoffDelay' (initial backoff ms) provider options in opencode.json. retry-after headers from the provider still take precedence over the configured backoff. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
|
Hope this PR merged ASAP, fixed max_retry in #41939 broken everything. |
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
Issue for this PR
Closes #21960
Type of change
What does this PR do?
Follow-up to #41939, which added the fixed RETRY_MAX_RETRIES = 5 cap. That default is right for most cases, but the retry budget users want differs per provider (an aggressive retry loop against a rate-limited Bedrock account makes things worse; a flaky self-hosted proxy may need more than 5). This PR makes the cap and the initial backoff configurable per provider:
{ "provider": { "amazon-bedrock": { "options": { "retry": 3, "backoffDelay": 1000 } } } }Both options are stripped before SDK construction (like chunkTimeout) so they never reach the provider client.
How did you verify your code works?
Screenshots / recordings
N/A (no UI change)
Checklist