Skip to content

Conversation

@RazorClient
Copy link

This Pr introduces all the types required + the helper functions for the same to implement

EIP 7919- Pureth Meta

The files inside eth/ssz have the type definitions and the recast functions to be per the

The open question was

  • How to introduce the new fields in the receipts as per
Field Description
from The transaction sender’s address.
gas_used How much gas this individual transaction used. Replaces cumulative_gas_used.
contract_address For transactions deploying a contract, the new contract address.
logs Logs emitted during transaction execution.
status EIP-658 transaction status code.
authorities For transactions with an authorization list, the list of EIP-7702 authority addresses. Non-successful authorizations are represented with an all-zero address.

The answer that came up was to change stored receipts to make room for the new fields and pass them on to the makereceipts and have them produce the new stored receipt variant with eip7807 and then recast them into seq[ssz_receipts] and then calc the root

So we need to save the new variant in the db too, as the stored receipt is what is present in the db and do tests to make sure its correct

@RazorClient
Copy link
Author

cc @advaita-saha for making sure that the db can know the new stored receipts and also test it out properly what tests do i need to add ?
i have made some tests inside tests/common/test_receipts.nim

You had mentioned we might need tests in test_eth_types_serialization too with a version of genTestOpt
how would that work with stored receipts?

@RazorClient RazorClient changed the title EIP 7919 Changes EIP 7919: Pureth Meta Oct 17, 2025
logs* : seq[Log]
eip7807ReceiptType*: Eip7807ReceiptType
authorities* : seq[Address]
txGasUsed* : uint64 # Gas used by THIS transaction only
Copy link
Contributor

Choose a reason for hiding this comment

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

is txGasUsed required from spec?
If yes please mentioned spec link in comments

Copy link
Author

Choose a reason for hiding this comment

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

Copy link
Contributor

Choose a reason for hiding this comment

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

Mention it in the code as a comment

@tersec
Copy link
Contributor

tersec commented Nov 6, 2025

CI won't run here until the eth/common/headers.nim merge conflict is resolved.

@RazorClient
Copy link
Author

Addressed conflict!

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.

3 participants