Skip to content

Remove trailing headers handling for rate limit headers#2298

Merged
dgageot merged 1 commit intodocker:mainfrom
dgageot:board/remove-trailing-headers-rate-limit-handl-873dc5c9
Apr 1, 2026
Merged

Remove trailing headers handling for rate limit headers#2298
dgageot merged 1 commit intodocker:mainfrom
dgageot:board/remove-trailing-headers-rate-limit-handl-873dc5c9

Conversation

@dgageot
Copy link
Copy Markdown
Member

@dgageot dgageot commented Mar 31, 2026

Remove the mechanism that drained Anthropic SSE response trailers to extract X-RateLimit-* and Retry-After headers.

This includes:

  • Removing the RateLimit struct from chat
  • Removing getResponseTrailer plumbing from the Anthropic client and stream adapters
  • Removing lastHTTPResponse / WithResponseInto from the Anthropic client
  • Removing all downstream propagation through streamResult, MessageUsage, and event types

@dgageot dgageot requested a review from a team as a code owner March 31, 2026 15:04
Remove the mechanism that drained Anthropic SSE response trailers to extract
X-RateLimit-* and Retry-After headers. This includes removing the RateLimit
struct from chat, the getResponseTrailer plumbing from the Anthropic client
and adapters, and all downstream propagation through streamResult,
MessageUsage, and event types.

Assisted-By: docker-agent
@dgageot dgageot force-pushed the board/remove-trailing-headers-rate-limit-handl-873dc5c9 branch from de00067 to 1e8d0e1 Compare March 31, 2026 15:15
@dgageot dgageot merged commit cca8638 into docker:main Apr 1, 2026
4 checks passed
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.

3 participants