Open
Conversation
- Expand V4 spec from 1 line to 7 detailed specification points mirroring V3 style - Add support for multiple blob versioned hashes in example (3 blobs) - Clarify missing blob handling (return null at corresponding positions) - Add support for partial blobs with null cells and null proofs - Update response schema to allow null entries (oneOf pattern) - Include example with: present blob, partial blob with null cell, missing blob Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
kamilsa
changed the title
There was a problem hiding this comment.
Pull request overview
Adds new Engine API capabilities needed for EIP-8070 “sparse blobpools”, enabling the CL to (1) communicate its blob column custody set to the EL and (2) request partial blob cell data (with proofs) from the EL blobpool.
Changes:
- Introduces
engine_blobCustodyUpdatedV1for CL→EL custody-set updates via a 16-byte bitarray. - Introduces
engine_getBlobsV4for fetching requested blob cells + KZG proofs (including partial/missing data semantics). - Adds shared schema types (
bytes16,BlobCellsAndProofsV1) and updates fork documentation/spellcheck wordlist accordingly.
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| wordlist.txt | Adds the new versioned schema/method-related token for spellchecking. |
| src/schemas/base-types.yaml | Adds a reusable bytes16 primitive used by the new methods’ bitarray params. |
| src/engine/openrpc/schemas/blob.yaml | Defines BlobCellsAndProofsV1 response object for partial cell/proof returns. |
| src/engine/openrpc/methods/blob.yaml | Adds OpenRPC method entries for engine_blobCustodyUpdatedV1 and engine_getBlobsV4 (with examples). |
| src/engine/amsterdam.md | Documents the new structure and method specs in the Amsterdam fork markdown (including ToC updates). |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
kamilsa
changed the title
healthykim
approved these changes
Apr 23, 2026
4 tasks
4 tasks
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
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.
Companion PR to ethereum/EIPs#11444 . The current PR is expected to be merged first.
To support EIP-8070 (sparse blobpools) :
engine_getBlobsV4: provides the mechanism for the CL to request partial blobs data from the the EL's blobpoolengine_forkchoiceupdatedv4: updated with the column custody field to update EL about CL's custody column set