feat(network): rewire PMA consumers to use transport and connected_peers (8/n)#15576
Merged
shreyan-gupta merged 2 commits intomasterfrom Apr 28, 2026
Merged
feat(network): rewire PMA consumers to use transport and connected_peers (8/n)#15576shreyan-gupta merged 2 commits intomasterfrom
shreyan-gupta merged 2 commits intomasterfrom
Conversation
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #15576 +/- ##
=======================================
Coverage 69.46% 69.47%
=======================================
Files 942 942
Lines 214969 214991 +22
Branches 214969 214991 +22
=======================================
+ Hits 149334 149357 +23
- Misses 59716 59720 +4
+ Partials 5919 5914 -5
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
5e71525 to
6284e77
Compare
5ca1a60 to
a622b9d
Compare
00169d9 to
cb81b43
Compare
a622b9d to
86d7eb3
Compare
cb81b43 to
c3d7b23
Compare
24d7e74 to
f58c995
Compare
73be26e to
2ade8e3
Compare
f58c995 to
be1d301
Compare
2ade8e3 to
bae8f46
Compare
7bd84df to
fa8bbac
Compare
bae8f46 to
4f340b9
Compare
fa8bbac to
a0dcdca
Compare
98e8627 to
c26743f
Compare
a0dcdca to
c2ed5f7
Compare
c26743f to
7590600
Compare
23388bd to
942b085
Compare
7f980ec to
dc3dced
Compare
40ce58d to
a5c7cb8
Compare
167fa39 to
c7e60b3
Compare
a83854c to
b774ffb
Compare
3ef6472 to
623360b
Compare
4a69807 to
9869ba3
Compare
0f151fe to
87b937e
Compare
pugachAG
approved these changes
Apr 28, 2026
87b937e to
19f37fb
Compare
27c372d to
864d458
Compare
…ted_peers Rewire PMA handlers to use the transport trait and connected_peers instead of direct Pool reads: - Block broadcast: self.state.tier2.broadcast_message → self.transport.broadcast_message(T2) - BlockRequest, EpochSyncRequest: self.state.tier2.send_message → self.transport.send_message(T2) - SyncSnapshotHosts: self.state.tier2.broadcast_message → self.transport.broadcast_message(T2) - highest_height_peers: reads connected_peers instead of tier2.load().ready - unreliable_peers: reads connected_peers instead of tier2.load().ready - PartialEncodedChunkRequest peer matching: reads connected_peers instead of tier2.load().ready Remaining PMA Pool reads (is_outbound_bootstrap_needed, maybe_stop_active_connection, monitor_peers_trigger, get_network_info, report_bandwidth_stats_trigger) need Pool-level info (outbound_handshakes count, per-connection bandwidth stats) and will move with the Pools in PR 9. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…est_height_peer_info
19f37fb to
62b76bf
Compare
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
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.
transport.broadcast_message(T2, ...)transport.send_message(T2, ...)transport.broadcast_message(T2, ...)state.peers.tier2()+to_highest_height_peer_infohelperstate.peers.tier2()instead ofstate.tier2.load().ready