Skip to content

fix(sse): clear network dedup when provider breaker resets (#13899) - #16077

Merged
diegosouzapw merged 1 commit into
release/v3.8.52from
fix/13899-breaker-reset-dedup
Oct 9, 2026
Merged

diegosouzapw merged 1 commit into
release/v3.8.52from
fix/13899-breaker-reset-dedup

Conversation

@diegosouzapw

Copy link
Copy Markdown
Collaborator

Re-land of #13899 by @onexer88-blip (closes #13887). The original PR has been a contributor draft since 2026-09-19 and could not be merged; the validated net diff is carried here with the original commit authorship preserved.

clearProviderFailure() now also clears the network-error and per-connection failure dedup windows, so a fresh network blip right after a provider breaker reset is counted instead of swallowed. Includes the contributor's regression test and changelog fragment.

Validation (local): breaker-network-error-guard 16/16; 262 related tests with 1 failure, auto-combo-candidates-locked-model-visible, which also fails on the tip without this change; typecheck:core, check:open-sse-typecheck, check:file-size green.

⚠️ base-red inherited: #15306

clearProviderFailure() now also clears the network-error and per-connection failure dedup windows, so a fresh network blip right after a provider breaker reset is counted instead of swallowed (#13887).

Co-authored-by: diegosouzapw <8016841+diegosouzapw@users.noreply.github.com>
@diegosouzapw
diegosouzapw merged commit 5bbe3c6 into release/v3.8.52 Oct 9, 2026
10 of 13 checks passed
@diegosouzapw
diegosouzapw deleted the fix/13899-breaker-reset-dedup branch October 9, 2026 16:37
@github-actions

github-actions Bot commented Oct 9, 2026

Copy link
Copy Markdown
Contributor

CI Coverage Report

  • Coverage job: skipped
  • PR test policy: success

Coverage artifact was not available for this run.

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.

omni

2 participants