Skip to content

Fix/429 retry timing#366

Merged
kevincodex1 merged 2 commits intoGitlawb:mainfrom
40verse:fix/429-retry-timing
Apr 5, 2026
Merged

Fix/429 retry timing#366
kevincodex1 merged 2 commits intoGitlawb:mainfrom
40verse:fix/429-retry-timing

Conversation

@40verse
Copy link
Copy Markdown
Contributor

@40verse 40verse commented Apr 4, 2026

fix: include retry timing in 429 error messages

Summary

  • Extract retry-after header from 429 API errors and include timing
    guidance in the user-facing error message
  • Previously, non-quota 429 errors showed a generic message with no
    guidance on when to retry, only a link to status.anthropic.com

Impact

  • user-facing impact: 429 error messages now tell users when to retry
    instead of just linking to a status page
  • developer/maintainer impact: none

Testing

  • bun run build
  • bun run smoke
  • focused tests: error formatting is pure string construction,
    verified via build + manual inspection

Notes

  • provider/model path tested: applies to all providers returning 429
  • screenshots attached (if UI changed): n/a
  • follow-up work or known limitations: 529 errors could get similar
    treatment in a follow-up

claude and others added 2 commits April 4, 2026 18:14
## Summary

- Extract retry-after header from 429 API errors and include timing
  guidance in the user-facing error message
- Previously, non-quota 429 errors showed a generic message with no
  guidance on when to retry, only a link to status.anthropic.com

## Impact

- user-facing impact: 429 error messages now tell users when to retry
  instead of just linking to a status page
- developer/maintainer impact: none

## Testing

- [x] `bun run build`
- [ ] `bun run smoke`
- [ ] focused tests: error formatting is pure string construction,
  verified via build + manual inspection

## Notes

- provider/model path tested: applies to all providers returning 429
- screenshots attached (if UI changed): n/a
- follow-up work or known limitations: 529 errors could get similar
  treatment in a follow-up

https://claude.ai/code/session_01D7kprMn4c66a5WrZscF7rv
Copy link
Copy Markdown
Contributor

@kevincodex1 kevincodex1 left a comment

Choose a reason for hiding this comment

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

nice! this is looks great

@kevincodex1 kevincodex1 merged commit 4ac7367 into Gitlawb:main Apr 5, 2026
@40verse 40verse deleted the fix/429-retry-timing branch April 7, 2026 01:54
euxaristia pushed a commit to euxaristia/openclaude that referenced this pull request Apr 13, 2026
## Summary

- Extract retry-after header from 429 API errors and include timing
  guidance in the user-facing error message
- Previously, non-quota 429 errors showed a generic message with no
  guidance on when to retry, only a link to status.anthropic.com

## Impact

- user-facing impact: 429 error messages now tell users when to retry
  instead of just linking to a status page
- developer/maintainer impact: none

## Testing

- [x] `bun run build`
- [ ] `bun run smoke`
- [ ] focused tests: error formatting is pure string construction,
  verified via build + manual inspection

## Notes

- provider/model path tested: applies to all providers returning 429
- screenshots attached (if UI changed): n/a
- follow-up work or known limitations: 529 errors could get similar
  treatment in a follow-up

https://claude.ai/code/session_01D7kprMn4c66a5WrZscF7rv

Co-authored-by: Claude <noreply@anthropic.com>
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.

4 participants