refactor(telegram): drop the flood-stack compat shims - #1434
Merged
Merged
Conversation
The four src/telegram shims (flood_wait, rate_limiter, rate_limit_gate, flood_breaker) were re-export-only bridges kept by the extraction PR (#1432) so ~50 importer files would not change in the migration commit. All imports now go to telethon_floodgate directly and the shims are deleted — no duplicated code remains in this repository. - the backward-compat run_with_flood_wait re-export in client_pool is removed too (zero importers and zero patch targets, verified) - DIALOG_SWEEP_MAX_CALLS keeps its module-path import (not in the package root __all__) - pybreaker stays a dependency (notifier.py imports it directly) Verified: zero references to the four shim paths anywhere; ruff clean (64 import blocks re-sorted); import-linter contract kept; full suite in the CI-config venv: 9383 passed parallel + 990 serial, failure set identical to main modulo the known flakes (quality_scoring, search_routes, integration, photo_loader, debug_memory — the last two pass in isolation). Co-Authored-By: Claude Code <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Final cleanup of the extraction epic: the four
src/telegramre-export shims (flood_wait,rate_limiter,rate_limit_gate,flood_breaker) are deleted; all ~50 importer files (26 src + 22 tests) now importtelethon_floodgatedirectly. Also removes the zero-consumerrun_with_flood_waitbackward-compat re-export inclient_pool.No code is deleted that still exists here — the bodies moved to the package in #1432; this removes the import-path bridge only.
Verified
DIALOG_SWEEP_MAX_CALLScross-invariant test switched to the module-path import (not in the package root__all__)🤖 Generated with Claude Code