Draft
Conversation
WIP: full ePBS implementation
GetValidatorIndexByPubkey returns indices with BuilderIndexFlag set for builder pubkeys in the shared pubkey cache. The deposits page was using these raw flagged indices, showing incorrect very high validator numbers for 0x03 deposits. Strip the flag, resolve the actual builder via GetBuilderByIndex, and display builder status/name correctly — matching the pattern already used in the exits handler. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
The Deposit Requests tab on the slot page was showing the raw pubkey cache index (with BuilderIndexFlag bit 40 set) for 0x03 builder deposits, resulting in very large incorrect validator numbers like 1099511627776. Strip the flag, set IsBuilder, and render with formatBuilderWithIndex and the hard-hat icon. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…ilder-index Fix builder index display in slot deposit requests view
Remove fields from ChainSpec that are no longer present in chain configs following ethereum-genesis-generator#271: MIN_EPOCHS_FOR_BLOCK_REQUESTS, ATTESTATION_SUBNET_PREFIX_BITS, MAX_REQUEST_BLOB_SIDECARS, MAX_REQUEST_BLOB_SIDECARS_ELECTRA, MAX_REQUEST_DATA_COLUMN_SIDECARS. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…c-vars chore: remove unused chainspec variables
Update the go-eth2-client dependency to include the new ptc_window field in the gloas BeaconState, required for consensus-specs v1.7.0-alpha.4 compatibility. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Now that pk910/go-eth2-client#9 is merged, point the replace directive back to pk910's dynssz-codegen-gloas branch. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Update go-eth2-client for consensus-specs alpha.4
When subscribeStream returns nil (e.g. context canceled), startStream now returns early instead of falling through to the select loop where it would dereference nil basicStream fields. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
barnabasbusa
force-pushed
the
gloas-support
branch
from
c37b1f4 to
4afed9e
Compare
Fix nil pointer panic in beacon event stream
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 join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
docker image:
ethpandaops/dora:gloas-supportspecs version:
v1.7.0-alpha.4specs path: specs/gloas
based on #227