Skip to content

refactor(broadcast): remove overflow channel - #179

Merged
tisonkun merged 1 commit into
mainfrom
codex/remove-broadcast-overflow
Aug 23, 2026
Merged

refactor(broadcast): remove overflow channel#179
tisonkun merged 1 commit into
mainfrom
codex/remove-broadcast-overflow

Conversation

@tisonkun

Copy link
Copy Markdown
Member

Summary

  • remove the lossy broadcast::overflow channel and its broadcast Cargo feature
  • remove its dedicated tests, benchmarks, API documentation, and feature wiring
  • remove the private poisoned-lock wrapper that was used only by the overflow buffer

Rationale

broadcast::overflow implements the sliding/drop-oldest behavior deferred in #167. Its observable capacity, lag, resume-position, and eviction semantics need a clearer contract, and there is no concrete downstream requirement that justifies carrying the existing API into 0.7.

Removing it now leaves room for the lossless bounded and unbounded broadcast APIs described in #167 without preserving an incompatible public path or error model. Shared waiter infrastructure remains in place because other primitives still use it.

Validation

  • cargo run -p x -- build --locked
  • cargo run -p x -- test
  • cargo run -p x -- check
  • cargo run -p x -- lint
  • cargo run -p x -- semver --release-version 0.7.0

@tisonkun
tisonkun merged commit 5b86f20 into main Aug 23, 2026
9 checks passed
@tisonkun
tisonkun deleted the codex/remove-broadcast-overflow branch August 23, 2026 16:52
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