Skip to content

add ILs peer sync - #11180

Merged
mehdi-aouadi merged 2 commits into
Consensys:focil-devnet-0from
mehdi-aouadi:9063-peer-sync
Aug 27, 2026
Merged

add ILs peer sync#11180
mehdi-aouadi merged 2 commits into
Consensys:focil-devnet-0from
mehdi-aouadi:9063-peer-sync

Conversation

@mehdi-aouadi

@mehdi-aouadi mehdi-aouadi commented Aug 26, 2026

Copy link
Copy Markdown
Contributor

PR Description

Add InclusionListsByCommitteeIndices RPC endpoint

Fixed Issue(s)

fixes #9063

Documentation

  • I thought about documentation and added the doc-change-required label to this PR if updates are required.

Changelog

  • I thought about adding a changelog entry, and added one if I deemed necessary.

Note

Medium Risk
Changes P2P request SSZ layout and RPC response selection for inclusion lists, which affects interoperability with peers and what data is served during sync.

Overview
Extends InclusionListsByCommitteeIndices so requests carry a dependent_root alongside slot and committee indices, and wires outbound peer RPC through SyncSource / DefaultEth2Peer (with throttling wrapper delegating unchanged).

On the serve path, InclusionListManager.getInclusionLists now filters stored lists by that root and drops equivocating validators (conflicting messages for the same slot, root, and index). RPC getMaximumResponseChunks is aligned with the number of requested committee bits rather than a fixed config cap.

NodeDataProvider and the inclusion-list RPC handler pass the new field through; tests cover manager lookup/equivocation behavior and peer request encoding.

Reviewed by Cursor Bugbot for commit 5af7db1. Bugbot is set up for automated code reviews on this repo. Configure here.

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

Reviewed by Cursor Bugbot for commit 0846987. Configure here.

@mehdi-aouadi
mehdi-aouadi merged commit a5315f7 into Consensys:focil-devnet-0 Aug 27, 2026
157 of 159 checks passed
@mehdi-aouadi
mehdi-aouadi deleted the 9063-peer-sync branch August 27, 2026 11:06
@github-actions github-actions Bot locked and limited conversation to collaborators Aug 27, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant