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

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