Skip to content

Conversation

@shemnon
Copy link
Contributor

@shemnon shemnon commented Jul 25, 2025

Implements the eth_config JSON-RPC method as specified in EIP-7910 to provide comprehensive fork configuration details including current, next, and last fork configurations with their respective hashes and fork IDs.

Changes:

  • Add eth_config method definition in src/eth/client.yaml
  • Add ConfigurationResponse, ConfigObject, and BlobSchedule schemas
  • Add bytes4 base type for CRC-32 hashes and fork IDs
  • Add test cases for different scenarios

shemnon added 2 commits July 25, 2025 16:38
Implements the eth_config JSON-RPC method as specified in EIP-7910 to provide
comprehensive fork configuration details including current, next, and last
fork configurations with their respective hashes and fork IDs.

Changes:
- Add eth_config method definition in src/eth/client.yaml
- Add ConfigurationResponse, ConfigObject, and BlobSchedule schemas
- Add bytes4 base type for CRC-32 hashes and fork IDs
- Add test cases for different scenarios
Signed-off-by: Danno Ferrin <[email protected]>
shemnon and others added 5 commits July 27, 2025 20:41
Co-authored-by: Mercy Boma Naps Nkari <[email protected]>
Co-authored-by: Mercy Boma Naps Nkari <[email protected]>
- Remove configHash fields from ConfigurationResponse
- Move forkId fields into ConfigObject
- Invert precompiles map: name->address mapping with alphabetical sorting
- Update test cases to match EIP specification format

Signed-off-by: Danno Ferrin <[email protected]>
…cution-apis into eip-7910-eth_config

Signed-off-by: Danno Ferrin <[email protected]>
Copy link
Collaborator

@kclowes kclowes left a comment

Choose a reason for hiding this comment

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

lgtm!

Copy link
Member

@lightclient lightclient left a comment

Choose a reason for hiding this comment

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

Generally LGTM. How were the tests generated? Would be good if we have that in rpctestgen before merging.

@shemnon
Copy link
Contributor Author

shemnon commented Aug 20, 2025

Tests were generated from the reference implementation and cross-compared with fork definitions.

@fjl fjl added the triage label Oct 24, 2025
@fjl fjl changed the title Add eth_config API definitions for EIP-7910 eth: add eth_config API definitions for EIP-7910 Oct 27, 2025
@fjl
Copy link
Contributor

fjl commented Oct 27, 2025

We have to create these tests with rpctestgen before merging this.

@fjl fjl removed the triage label Oct 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants