consensus-specs v1.7.0-alpha.12 compatibility (deferring 7688) - #10954
Merged
Conversation
Still more to do before alpha 12 is fully integrated, but this should be a good start. not devnet6 compatible.
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
Reviewed by Cursor Bugbot for commit f890218. Configure here.
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 subscribe to this conversation on GitHub.
Already have an account?
Sign in.
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.

not devnet6 compatible.
TODO:
MAX_REQUEST_PAYLOADSinEnvelopesByRange聽ethereum/consensus-specs#5383get_dependent_rootwithget_shuffling_dependent_root聽ethereum/consensus-specs#5374Set slot before using it for object construction聽ethereum/consensus-specs#5386 (not sure)Documentation
doc-change-requiredlabel to this PR if updates are required.Changelog
Note
Medium Risk
Touches fork-choice proposer boost, Gloas builder deposit/withdrawal rules, and withdrawal prefix semantics鈥攃onsensus-sensitive but mostly spec conformance with Gloas ref tests gated off.
Overview
Brings Teku in line with consensus-specs v1.7.0-alpha.12 while deferring full Gloas/EIP-7688 (Gloas reference tests are disabled until progressive SSZ lands).
Gloas parameters and constants are updated in mainnet/minimal/swift YAML, REST fixtures, and presets:
MIN_BUILDER_WITHDRAWABILITY_DELAY64 epochs,PAYLOAD_DUE_BPS5000,MAX_BUILDER_DEPOSIT_REQUESTS_PER_PAYLOAD64, and builder withdrawal prefix0xB0.Builder deposit processing now skips non-builder credentials, registers new builders with
PAYLOAD_BUILDER_VERSION, and only resetswithdrawable_epochfor exited builders whose balance was swept; unit tests cover those cases.Fork choice / proposer boost renames dependent-root helpers to
get_shuffling_dependent_rootand passes the current epoch explicitly (specrefs andProposerPreferencesPublishercomments updated).Networking caps execution payload envelope range requests with
getMaxRequestPayloads()instead of the Deneb block limit.Reference tests bump to alpha.12; generic SSZ executors use
SszListSchema.createProgressive, container checks compare hash tree roots from YAML, and test struct schemas switch to uint8/uint32 where the new vectors require it.specrefs track alpha.12 (containers, functions, configs, partial Fulu gossip API tweaks) and mark not-yet-implemented Gloas progressive SSZ items.
Reviewed by Cursor Bugbot for commit caf8733. Bugbot is set up for automated code reviews on this repo. Configure here.