Skip to content

Record external operation DelegationResolution#333

Merged
sorpaas merged 2 commits intorust-ethereum:v0.xfrom
moonbeam-foundation:manuel/record-external-operation-delegation-resolution
Jul 9, 2025
Merged

Record external operation DelegationResolution#333
sorpaas merged 2 commits intorust-ethereum:v0.xfrom
moonbeam-foundation:manuel/record-external-operation-delegation-resolution

Conversation

@manuelmauro
Copy link
Copy Markdown
Contributor

Description

The current implementation of EIP-7702 is not properly recording delegation resolution as an external operation.

@manuelmauro
Copy link
Copy Markdown
Contributor Author

Is DelegationResolution even needed? Would emitting two times AddressCodeRead be flexible enough? I say this because on frontier this seems to be the case to me. Still having an explicit enum constant could be valuable.

@sorpaas
Copy link
Copy Markdown
Member

sorpaas commented Jul 8, 2025

Is DelegationResolution even needed? Would emitting two times AddressCodeRead be flexible enough?

I don't have preference. At this moment we can do what is better for Frontier. I think for other chains if they want to customize for more flexibility, some more hooks / code changes will be needed anyway than DelegationResolution.

@manuelmauro
Copy link
Copy Markdown
Contributor Author

Since on frontier having an external operation for DelegationResolution does not add complexity and makes the cost calculations more explicit I would slightly lean towards keeping it.

Reference commit on frontier.

@sorpaas sorpaas merged commit 902bd7e into rust-ethereum:v0.x Jul 9, 2025
2 checks passed
manuelmauro added a commit to moonbeam-foundation/evm that referenced this pull request Jul 14, 2025
* fix: 🐛 record external operation DelegationResolution

* refactor: ⚡ optimize delegated code retrieval
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.

2 participants