Skip to content

EIP-7688: Forward Compatible Consensus Data Structures - #9450

Merged
mergify[bot] merged 40 commits into
sigp:unstablefrom
macladson:eip-7688
Aug 4, 2026
Merged

EIP-7688: Forward Compatible Consensus Data Structures#9450
mergify[bot] merged 40 commits into
sigp:unstablefrom
macladson:eip-7688

Conversation

@macladson

@macladson macladson commented Jun 10, 2026

Copy link
Copy Markdown
Member

Relevant Links

Proposed Changes

Adds the necessary types and components to support EIP-7688, along with the cargo patches to pull in the progressive versions of the crates in our SSZ stack.

Testing

participants_matrix:
  el:
    - el_type: nethermind
      el_image: ethpandaops/nethermind:master
  cl:
    - cl_type: nimbus
      cl_image: ethpandaops/nimbus-eth2:glamsterdam-devnet-7-minimal
    - cl_type: lodestar
      cl_image: ethpandaops/lodestar:glamsterdam-devnet-7
    - cl_type: lighthouse
      cl_image: ethpandaops/lighthouse:glamsterdam-devnet-7
    - cl_type: teku
      cl_image: ethpandaops/teku:glamsterdam-devnet-7

ethereum_genesis_generator_params:
  image: "ethpandaops/ethereum-genesis-generator:glamsterdam-devnet-7"
dora_params:
  image: ethpandaops/dora:glamsterdam-devnet-7

global_log_level: debug

network_params:
  preset: minimal
  gloas_fork_epoch: 1

additional_services:
  - dora

Additional Info

Based on the original work by @michaelsproul in #8505

@eserilev

eserilev commented Jul 8, 2026

Copy link
Copy Markdown
Member

5436 was also added as part of the alpha spec 12 changes

@eserilev eserilev left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Finished my review, outside of my nits i think this should be g2g. imo I think we should get this into unstable asap as its blocking alpha spec 12 and 13

Comment thread beacon_node/http_api/src/beacon/pool.rs Outdated
Comment thread beacon_node/http_api/src/validator/mod.rs Outdated
@macladson macladson added ready-for-review The code is ready for review and removed blocked waiting-on-author The reviewer has suggested changes and awaits thier implementation. labels Aug 3, 2026
@mergify

mergify Bot commented Aug 3, 2026

Copy link
Copy Markdown

Some required checks have failed. Could you please take a look @macladson? 🙏

@mergify mergify Bot added waiting-on-author The reviewer has suggested changes and awaits thier implementation. and removed ready-for-review The code is ready for review labels Aug 3, 2026
@macladson

Copy link
Copy Markdown
Member Author

For the other Claude findings I have addressed 3 and 5, but left 1, 2 and 4 (size-bounds and limits) for a follow-up PR.

@macladson macladson added ready-for-review The code is ready for review and removed waiting-on-author The reviewer has suggested changes and awaits thier implementation. labels Aug 3, 2026

@eserilev eserilev left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM!

@eserilev eserilev added ready-for-merge This PR is ready to merge. and removed ready-for-review The code is ready for review labels Aug 4, 2026
@mergify mergify Bot added the queued label Aug 4, 2026
@mergify

mergify Bot commented Aug 4, 2026

Copy link
Copy Markdown

Merge Queue Status

This pull request spent 29 minutes 35 seconds in the queue, including 27 minutes 39 seconds running CI.

Required conditions to merge

@mergify
mergify Bot merged commit e58ec88 into sigp:unstable Aug 4, 2026
38 checks passed
@mergify mergify Bot removed the queued label Aug 4, 2026
@macladson
macladson deleted the eip-7688 branch August 4, 2026 08:24
mergify Bot pushed a commit that referenced this pull request Aug 12, 2026
## Issue Addressed

Follow-up to #9708 , resolving the outstanding switch of `InclusionList.transactions` to `ProgressiveTransactions`. 
This was unblocked by the merge of #9450.

## Proposed Changes

- Switch `InclusionList.transactions` from `Transactions<E>` to `ProgressiveTransactions`, matching [the spec type](https://github.com/ethereum/consensus-specs/blob/master/specs/heze/beacon-chain.md#inclusionlist)
- Since no field depends on `EthSpec` anymore, `InclusionList` and `SignedInclusionList` drop this type parameter
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

gloas ready-for-merge This PR is ready to merge.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants