Repository navigation
Conversation
Anthropic returns {"type":"overloaded_error","message":"Overloaded"}
as a stream error when at capacity. This was not recognized by
parseStreamError() (which only handled type=="error") or the fallback
JSON check in retryable(), causing the session to halt with a terminal
error instead of retrying with exponential backoff.
- Add overloaded_error handling to parseStreamError() as a retryable
api_error
- Widen ParsedStreamError.isRetryable from literal false to boolean
- Add overloaded_error check to retryable() fallback JSON path as
a safety net
|
Thanks for your contribution! This PR doesn't have a linked issue. All PRs must reference an existing issue. Please:
See CONTRIBUTING.md for details. |
|
Thanks for updating your PR! It now meets our contributing guidelines. 👍 |
|
Automated PR Cleanup Thank you for contributing to opencode. Due to the high volume of PRs from users and AI agents, we periodically close older PRs using automated criteria so maintainers can focus review time on the most active and community-supported contributions. This PR was closed because it matched the following cleanup criteria:
PRs created within the last month are not affected by this cleanup. If you believe this PR was closed incorrectly, or if you are still actively working on it, please leave a comment explaining why it should be reopened. A maintainer can review and reopen it if appropriate. Thanks again for taking the time to contribute. |
Issue for this PR
Closes #20384
Type of change
What does this PR do?
Anthropic returns
{"type":"overloaded_error","message":"Overloaded"}as a stream error when their API is at capacity. This was not recognized by the error handling pipeline, causing sessions to halt with a terminal error instead of retrying.The root cause:
parseStreamError()inerror.tsonly checked forbody.type === "error", so"overloaded_error"fell through. The error became aNamedError.Unknown, andretryable()inretry.tsdidn't match it either since its JSON fallback path only checked fortype === "error"variants.The fix adds
overloaded_errorhandling in two places:parseStreamError()— recognizestype: "overloaded_error"and returns it as a retryableapi_error. Also widensParsedStreamError.isRetryablefrom literalfalsetobooleanso stream errors can be retryable.retryable()fallback — addsjson.type === "overloaded_error"check as a safety net for the JSON parsing path.With this fix, overloaded errors trigger the existing exponential backoff retry logic (2s → 4s → 8s → 16s → 30s cap) instead of killing the session.
How did you verify your code works?
fromError()→parseStreamError()→retryable()confirming the classification bugparseStreamError()now returns{type: "api_error", isRetryable: true}, whichfromError()wraps asMessageV2.APIError, whichretryable()matches at line 54-58 (existingisRetryable+"Overloaded"check)Checklist