Skip to content

fix(acp): coordinate HTTP rate-limit retries - #3618

Closed
dbett4 wants to merge 6 commits into
block:mainfrom
dbett4:codex/buzz-health-20260729
Closed

fix(acp): coordinate HTTP rate-limit retries#3618
dbett4 wants to merge 6 commits into
block:mainfrom
dbett4:codex/buzz-health-20260729

Conversation

@dbett4

@dbett4 dbett4 commented Jul 29, 2026

Copy link
Copy Markdown

Summary

  • share one process-local HTTP 429 cooldown across RestClient clones
  • serialize bridge admission so queued requests re-check the gate after a peer arms it
  • honor integer Retry-After hints with conservative defaults and a bounded cap
  • add concurrent regression coverage for the retry-storm path

Verification

  • cargo fmt --check -- crates/buzz-acp/src/relay.rs
  • cargo test -p buzz-acp
  • 636 unit tests and 9 lifecycle tests passed

@dbett4
dbett4 requested a review from a team as a code owner July 29, 2026 21:00
@dbett4
dbett4 force-pushed the codex/buzz-health-20260729 branch from a223252 to a9747fa Compare July 29, 2026 21:02
dbett4 added 6 commits August 10, 2026 19:22
Signed-off-by: Dave Bettner <dbett4@gmail.com>
Signed-off-by: Dave Bettner <dbett4@gmail.com>
Signed-off-by: Dave Bettner <dbett4@gmail.com>
Signed-off-by: Dave Bettner <dbett4@gmail.com>
Signed-off-by: Dave Bettner <dbett4@gmail.com>
Signed-off-by: Dave Bettner <dbett4@gmail.com>
@dbett4

dbett4 commented Aug 12, 2026

Copy link
Copy Markdown
Author

Superseded by #5620.

This PR bundled six unrelated commits under a title describing only the first, which made it hard to review. #5620 is just the rate-limit fix — +297/−9 across three files, rebased onto current main — with the concurrency test verified as a real detector rather than only a passing test.

Closing this. The five agent reply-credential commits will be resubmitted separately if they are wanted.

@dbett4 dbett4 closed this Aug 12, 2026
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.

1 participant