Skip to content

Add REST-SSZ wire containers required for REST-SSZ engine - #10242

Open
SamAg19 wants to merge 7 commits into
sigp:unstablefrom
SamAg19:rest-ssz-wire-types
Open

SamAg19 wants to merge 7 commits into
sigp:unstablefrom
SamAg19:rest-ssz-wire-types

Conversation

@SamAg19

@SamAg19 SamAg19 commented Oct 7, 2026

Copy link
Copy Markdown
Contributor

Issue Addressed

Introduces REST-SSZ wire types required for the REST-SSZ engine API implementation based on the spec

Proposed Changes

PR #9652 has the complete implementation of the REST-SSZ engine API and support uptil Gloas but since the diff is 6k+, reviewing such a PR will not be possible and hence I will be creating multiple smaller PRs from PR #9652 as mentioned in this comment to improve reviewability starting with the following PR of adding REST-SSZ wire types.

  • Introduces ssz_structures.rs which holds the wire containers required for the REST-SSZ engine along with the conversions between those containers and the internal SSZ containers.
  • Derives Encode and Decode on existing types that can be reused instead of redeclaring them in ssz_structures.rs
  • Adding MaxExecutionRequestsPerPayload, MaxErrorBytes and MaxVersionedHashesPerRequest types to the EthSpec trait in accordance with the spec

@SamAg19

SamAg19 commented Oct 19, 2026

Copy link
Copy Markdown
Contributor Author

@macladson Updated the branch to incorporate PR #10378 fixes that addresses the cargo-audit in the check-code CI run. Everything should pass now.

@michaelsproul michaelsproul added the optimization Something to make Lighthouse run more efficiently. label Oct 20, 2026

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

optimization Something to make Lighthouse run more efficiently.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants