Skip to content

feat: EIP-1898 block identifier support (blockHash)#4186

Open
dputko wants to merge 1 commit intowevm:mainfrom
dputko:feat/eip-1898
Open

feat: EIP-1898 block identifier support (blockHash)#4186
dputko wants to merge 1 commit intowevm:mainfrom
dputko:feat/eip-1898

Conversation

@dputko
Copy link

@dputko dputko commented Dec 19, 2025

Adds EIP-1898 block identifier support (blockHash/requireCanonical) for state‑reading actions (call, getBalance, getCode, getProof, getStorageAt, getTransactionCount), enabling deterministic state queries. Adds shared formatBlockParameter validation, updates types/docs, and adds tests.
Refs #3165.

What parts require more attention from reviewers?

  • Multicall batching is skipped when blockHash is provided (see alternatives below).
  • Dedupe logic for getCode/getTransactionCount now includes blockHash; please sanity‑check caching expectations.

What other alternatives have you explored?

  • Enable batching for blockHash by resolving blockNumber via eth_getBlockByHash. Rejected: extra RPC/latency and undermines batching.
  • Force deployless multicall for blockHash. Rejected: extra complexity for a narrow path.

@changeset-bot
Copy link

changeset-bot bot commented Dec 19, 2025

🦋 Changeset detected

Latest commit: fc5a9b6

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
viem Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@F4ever
Copy link

F4ever commented Jan 23, 2026

@tmm Hi!
Can you take a look into this PR?
Need this so much!

@vercel
Copy link

vercel bot commented Feb 16, 2026

@dputko is attempting to deploy a commit to the Wevm Team on Vercel.

A member of the Team first needs to authorize it.

@infloop infloop mentioned this pull request Feb 17, 2026
1 task
@infloop
Copy link

infloop commented Feb 17, 2026

Hi @jxom @tmm! We really need comments about that PR, please take a look. We are open for discussion.

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