feat(rpc): Add support for the `EthTraceReplayBlockTransactions` V2 API #6444

akaladarshi · 2026-01-19T18:32:30Z

Summary of changes

Changes introduced in this pull request:

Adds support for the EthTraceReplayBlockTransactions V2 API
Small Refactor
Fix the discrepancy in EthTraceReplayBlockTransactions response

Reference issue to close (if applicable)

Closes #6308

Change checklist

I have performed a self-review of my own code,
I have made corresponding changes to the documentation. All new code adheres to the team's documentation standards,
I have added tests that prove my fix is effective or that my feature works (if possible),
I have made sure the CHANGELOG is up-to-date. All user-facing changes should be reflected in this document.

Summary by CodeRabbit

New Features
- Added a V2 Ethereum trace-replay RPC endpoint (new API path) returning the same trace items with updated parameters.
Changes
- Trace responses now always include stateDiff and vmTrace (nullable) for compatibility.
- Improved validation and clearer error responses for trace type parameters.
- Aligned V1/V2 handling to a shared implementation for consistent results.
Tests
- Extended RPC test coverage and updated snapshots to include the V2 endpoint.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

coderabbitai · 2026-01-19T18:32:47Z

Walkthrough

Adds a V2 RPC method EthTraceReplayBlockTransactionsV2, centralizes replay logic into a new async helper eth_trace_replay_block_transactions, ensures V1 delegates to the helper, changes serialization to always emit state_diff and vm_trace (nullable), and updates tests/snapshots and registration.

Changes

Cohort / File(s)	Summary
RPC method implementation `src/rpc/methods/eth.rs`	Added `EthTraceReplayBlockTransactionsV2` (Params = `(ExtBlockNumberOrHash, Vec<String>)`, Ok = `Vec<EthReplayBlockTransactionTrace>`). Introduced `eth_trace_replay_block_transactions<DB>` async helper used by both V1 and V2. V2 validates `traceTypes` and uses V2 tipset resolver.
Types / serialization `src/rpc/methods/eth/types.rs`	Removed `#[serde(skip_serializing_if = "Option::is_none")]` from `state_diff` and `vm_trace` on `EthReplayBlockTransactionTrace`, so those fields are serialized as `null` when absent.
RPC registration `src/rpc/mod.rs`	Registered `EthTraceReplayBlockTransactionsV2` in the `for_each_rpc_method` macro.
Tests & snapshots `src/tool/subcommands/api_cmd/api_compare_tests.rs`, `src/tool/subcommands/api_cmd/test_snapshots.txt`	Added test entry for V2 (`EthTraceReplayBlockTransactionsV2::request`) and replaced/added snapshot files for the V2 output.
Docs / changelog `CHANGELOG.md`	Documented addition of API v2 method and change to always include `stateDiff` and `vmTrace` (nullable) in responses.

Sequence Diagram(s)

mermaid
sequenceDiagram
participant Client
participant RPC as "RPC Method (V1/V2)"
participant Resolver as "Tipset Resolver"
participant Tracer as "Trace Executor"
participant DB as "DB / State Viewer"
participant Assembler as "Trace Assembler"

Client->>RPC: ReplayBlockTransactions request
RPC->>Resolver: resolve tipset (V1 or V2)
Resolver-->>RPC: tipset
RPC->>Tracer: execute traces for tipset
Tracer-->>RPC: raw trace results
RPC->>DB: resolve tx hashes & fetch state
DB-->>RPC: tx hashes / state snapshots
RPC->>Assembler: build envs, capture vm_trace/state_diff, assemble results
Assembler-->>Client: Vec

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Possibly related PRs

feat: enable V2 API support for basic Eth and Filecoin RPC methods #6171 — related modifications to Eth trace replay and V2 API support for trace endpoints.
fix(ci): rpc parity test #6071 — similar change to stop skipping optional fields in serde serialization so fields serialize as null when absent.

Suggested reviewers

hanabi1224
sudo-shashank

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The pull request title clearly and specifically describes the main change: adding V2 API support for EthTraceReplayBlockTransactions, which aligns with the primary objective of the changeset.
Linked Issues check	✅ Passed	The pull request implements the RPC v2 endpoint for Filecoin.EthTraceReplayBlockTransactions as required by issue `#6308`, adding the V2 variant with appropriate parameter types and return types.
Out of Scope Changes check	✅ Passed	All changes are directly related to implementing the V2 API endpoint. The refactoring of shared logic between V1 and V2, serialization adjustments, and test updates are all in scope for this feature implementation.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

codecov · 2026-01-20T09:46:43Z

Codecov Report

❌ Patch coverage is 70.17544% with 17 lines in your changes missing coverage. Please review.
✅ Project coverage is 58.76%. Comparing base (f96325d) to head (0741774).

Files with missing lines	Patch %	Lines
src/rpc/methods/eth.rs	70.17%	10 Missing and 7 partials ⚠️

Additional details and impacted files

Files with missing lines	Coverage Δ
src/rpc/methods/eth/types.rs	`66.06% <ø> (ø)`
src/rpc/mod.rs	`24.47% <ø> (ø)`
src/rpc/methods/eth.rs	`67.67% <70.17%> (+0.01%)`	⬆️

... and 7 files with indirect coverage changes

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f96325d...0741774. Read the comment docs.

🚀 New features to boost your workflow:

❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

LesnyRumcajs · 2026-01-20T16:18:10Z

src/rpc/methods/eth.rs


-        let state = StateTree::new_from_root(ctx.store_owned(), &state_root)?;
+pub enum EthTraceReplayBlockTransactionsV2 {}
+impl RpcMethod<2> for EthTraceReplayBlockTransactionsV2 {


description?

I didn't notice this earlier that some of the other API's also doesn't have a Description, I will create an issue for this so we can add in all API's.

LesnyRumcajs · 2026-01-20T16:21:45Z

src/rpc/methods/eth.rs

+        ctx: Ctx<impl Blockstore + Send + Sync + 'static>,
+        (block_param, trace_types): Self::Params,
+    ) -> Result<Self::Ok, ServerError> {
+        if trace_types.as_slice() != ["trace"] {


is there a point in using anyhow here? Seems to me it'd be easier to define a new method on ServerError, e.g., ServerError::unsupported_params. It'd be cheaper.

I guess we can use the ServerError::invalid_params("only 'trace' is supported", None)

…snapshot tests

LesnyRumcajs · 2026-01-21T09:46:02Z

src/rpc/methods/eth/types.rs

 #[serde(rename_all = "camelCase")]
 pub struct EthReplayBlockTransactionTrace {
    pub output: EthBytes,
-    #[serde(skip_serializing_if = "Option::is_none")]


so this was incorrect?

what's the struct in Lotus?

akaladarshi force-pushed the akaladarshi/replay-block-tx-v2 branch from 62d9e13 to 1b2504a Compare January 20, 2026 08:59

akaladarshi marked this pull request as ready for review January 20, 2026 08:59

akaladarshi requested a review from a team as a code owner January 20, 2026 08:59

akaladarshi requested review from LesnyRumcajs and hanabi1224 and removed request for a team January 20, 2026 08:59

akaladarshi added the RPC requires calibnet RPC checks to run on CI label Jan 20, 2026

LesnyRumcajs reviewed Jan 20, 2026

View reviewed changes

akaladarshi added 4 commits January 21, 2026 00:38

Add EthTraceReplayBlockTransactionsV2 API support

10b6635

add api snapshot test

222b07b

fix eth replay block transaction response descrepancy and update the …

4386854

…snapshot tests

address comments

505c6f5

akaladarshi force-pushed the akaladarshi/replay-block-tx-v2 branch from 6cbc78a to 505c6f5 Compare January 20, 2026 19:08

update changelog

0741774

LesnyRumcajs reviewed Jan 21, 2026

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

feat(rpc): Add support for the `EthTraceReplayBlockTransactions` V2 API #6444

feat(rpc): Add support for the `EthTraceReplayBlockTransactions` V2 API #6444

akaladarshi commented Jan 19, 2026 •

edited by coderabbitai bot

Loading

Uh oh!

coderabbitai bot commented Jan 19, 2026 •

edited

Loading

Uh oh!

codecov bot commented Jan 20, 2026 •

edited

Loading

Uh oh!

LesnyRumcajs Jan 20, 2026

Uh oh!

akaladarshi Jan 20, 2026 •

edited

Loading

Uh oh!

LesnyRumcajs Jan 20, 2026

Uh oh!

akaladarshi Jan 20, 2026

Uh oh!

LesnyRumcajs Jan 21, 2026

Uh oh!

LesnyRumcajs Jan 21, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

feat(rpc): Add support for the EthTraceReplayBlockTransactions V2 API #6444

Are you sure you want to change the base?

feat(rpc): Add support for the EthTraceReplayBlockTransactions V2 API #6444

Conversation

akaladarshi commented Jan 19, 2026 • edited by coderabbitai bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary of changes

Reference issue to close (if applicable)

Other information and links

Change checklist

Summary by CodeRabbit

Uh oh!

coderabbitai bot commented Jan 19, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Walkthrough

Changes

Sequence Diagram(s)

Estimated code review effort

Possibly related PRs

Suggested reviewers

Uh oh!

codecov bot commented Jan 20, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Report

Uh oh!

LesnyRumcajs Jan 20, 2026

Choose a reason for hiding this comment

Uh oh!

akaladarshi Jan 20, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

LesnyRumcajs Jan 20, 2026

Choose a reason for hiding this comment

Uh oh!

akaladarshi Jan 20, 2026

Choose a reason for hiding this comment

Uh oh!

LesnyRumcajs Jan 21, 2026

Choose a reason for hiding this comment

Uh oh!

LesnyRumcajs Jan 21, 2026

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

feat(rpc): Add support for the `EthTraceReplayBlockTransactions` V2 API #6444

feat(rpc): Add support for the `EthTraceReplayBlockTransactions` V2 API #6444

akaladarshi commented Jan 19, 2026 •

edited by coderabbitai bot

Loading

coderabbitai bot commented Jan 19, 2026 •

edited

Loading

codecov bot commented Jan 20, 2026 •

edited

Loading

akaladarshi Jan 20, 2026 •

edited

Loading