feat(rpc): add web3, net namespaces and expand debug/eth for tooling compatibility by prestwich · Pull Request #120 · init4tech/node-components

prestwich · 2026-03-25T16:37:08Z

Summary

[Claude Code]

Adds standard Ethereum JSON-RPC namespaces and methods for general tooling compatibility (Blockscout, Routescan, wallets, etc.). Driven by ENG-960.

web3 namespace: clientVersion, sha3
net namespace: version, listening
eth: protocolVersion
debug: getRawHeader, getRawTransaction, getRawBlock, getRawReceipts, traceBlock (raw RLP), traceCall

Total: 56 routes across 5 namespaces (eth 41, debug 9, signet 2, web3 2, net 2).

Methods that don't apply to Signet (mining, keystore, txpool, proofs, blobs, net_peerCount) are not registered and return method_not_found.

Key implementation details

debug_traceBlock decodes raw RLP, recovers senders, reuses extracted trace_block_inner shared tracing loop
debug_traceCall combines EVM state resolution (like eth_call) with the tracer infrastructure
debug_getRawReceipts reconstructs consensus ReceiptEnvelope from ColdReceipt (strips RPC log metadata)
debug_getRawBlock assembles full block from separately-stored header + transactions
New DebugError variants: RlpDecode, SenderRecovery

Test plan

All 35 existing tests pass (updated test_not_supported → test_protocol_version)
New unit tests for web3_sha3, web3_clientVersion, DebugError variants
Clippy clean (--all-features --all-targets)
Manual testing with Blockscout/Routescan against a running node

Review Fixes

Replace EvmHalt with Internal for task panic/cancellation in all await_handler! sites
Replace EvmHalt with Internal for resolve_evm_block error mapping in debug_traceCall
Use unwrap_or_default() for optional tracer options per Geth spec (all 4 handlers)
Change pub(super) to pub(crate) on all debug handler functions
Replace use super::* glob import with explicit imports in web3 tests
Remove accidentally committed planning/spec documents

Reviewed and updated by Claude Code.

🤖 Generated with Claude Code

prestwich · 2026-03-30T17:29:11Z

[Claude Code]

Code review

Found 1 issue:

Planning/spec documents appear to be committed by mistake — docs/superpowers/plans/2026-03-25-parity-trace-namespace.md and docs/superpowers/specs/2026-03-25-parity-trace-namespace-design.md. This was also flagged in PR feat: handle reorgs in get_filter_changes with reorg watermark #98 for similar files.

node-components/docs/superpowers/plans/2026-03-25-parity-trace-namespace.md

Lines 1 to 5 in c0379af

    
           # Parity `trace_` Namespace Implementation Plan 
        
           > **For agentic workers:** REQUIRED SUB-SKILL: Use superpowers:subagent-driven-development (recommended) or superpowers:executing-plans to implement this plan task-by-task. Steps use checkbox (`- [ ]`) syntax for tracking. 
        
           **Goal:** Add a Parity/OpenEthereum `trace_` JSON-RPC namespace (9 methods) to signet-rpc for Blockscout and general tooling compatibility.

node-components/docs/superpowers/specs/2026-03-25-parity-trace-namespace-design.md

Lines 1 to 5 in c0379af

    
           # Parity `trace_` Namespace 
        
           Add the Parity/OpenEthereum `trace_` JSON-RPC namespace to signet-rpc for 
        
           Blockscout and general tooling compatibility. Driven by ENG-1064 and ENG-1065.

Additionally, several lower-confidence findings were identified (not posted here as they scored below threshold): pub(super) on new handlers (CLAUDE.md violation), opts.ok_or() instead of unwrap_or_default() (spec compliance), and EvmHalt misuse for task panics (regression from afe39f5 fix). These may warrant attention during triage.

🤖 Generated with Claude Code

_{- If this code review was useful, please react with 👍. Otherwise, react with 👎.}

Co-Authored-By: Claude Sonnet 4.6 <[email protected]>

Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>

Extract trace_block_inner() so it can be reused by the new debug_traceBlock (raw RLP) handler. Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>

Decodes an RLP-encoded block, recovers transaction senders, and traces all transactions using the shared trace_block_inner loop. Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>

Co-Authored-By: Claude Sonnet 4.6 <[email protected]>

eth_protocolVersion is now implemented, so the test that expected method_not_found is updated to assert the "0x44" response. Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>

- Document partial-move pattern in get_raw_receipts - Consolidate duplicate -32602 error code arms in DebugError - Fix resolve_header error mapping: use Resolve(e) instead of BlockNotFound(id) to preserve the distinction between "block not found" (-32001) and "storage layer failed" (-32000). The resolve_error_message helper already sanitizes DB errors to the opaque "server error" string, so no internals are leaked. Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>

9 methods (trace_block, trace_transaction, trace_replayBlockTransactions, trace_replayTransaction, trace_call, trace_callMany, trace_rawTransaction, trace_get, trace_filter) for Blockscout/tooling compatibility. ENG-1064, ENG-1065. Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>

12 tasks covering TraceError, param types, config, Parity tracer functions, block replay helpers, 9 trace_ endpoints, and router wiring. Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>

- Replace EvmHalt with Internal for task panic/cancellation errors - Replace EvmHalt with Internal for resolve_evm_block error mapping - Use unwrap_or_default() for optional tracer options per Geth spec - Change pub(super) to pub(crate) on all debug handler functions - Replace glob import with explicit imports in web3 test module - Remove accidentally committed planning/spec documents Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>

Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>

prestwich requested a review from a team as a code owner March 25, 2026 16:37

prestwich mentioned this pull request Mar 25, 2026

feat(rpc): implement Parity trace_ namespace (9 methods) #121

Merged

6 tasks

prestwich requested a review from Fraser999 March 25, 2026 23:24

prestwich marked this pull request as draft March 26, 2026 11:07

prestwich force-pushed the prestwich/eng-960-rpc-namespace-completeness branch from 9af104c to 4dd2e29 Compare March 26, 2026 15:29

prestwich force-pushed the prestwich/eng-1899 branch from dfd1229 to b87e46b Compare March 26, 2026 15:37

prestwich force-pushed the prestwich/eng-960-rpc-namespace-completeness branch from 4dd2e29 to 03bbd7c Compare March 26, 2026 15:38

prestwich force-pushed the prestwich/eng-1899 branch from b87e46b to 112c417 Compare March 26, 2026 21:41

prestwich force-pushed the prestwich/eng-960-rpc-namespace-completeness branch from 03bbd7c to c0379af Compare March 26, 2026 21:41

rswanson approved these changes Mar 27, 2026

View reviewed changes

Fraser999 reviewed Mar 30, 2026

View reviewed changes

Comment thread crates/rpc/src/debug/endpoints.rs Outdated

Comment thread crates/rpc/src/debug/endpoints.rs Outdated

Comment thread docs/superpowers/plans/2026-03-25-parity-trace-namespace.md Outdated

Comment thread docs/superpowers/specs/2026-03-25-parity-trace-namespace-design.md Outdated

prestwich force-pushed the prestwich/eng-960-rpc-namespace-completeness branch from c0379af to f3d693c Compare March 30, 2026 17:35

prestwich and others added 14 commits March 31, 2026 07:30

feat(rpc): add eth_protocolVersion

d94ae12

Co-Authored-By: Claude Sonnet 4.6 <[email protected]>

feat(rpc): add web3 namespace (clientVersion, sha3)

7a18d43

Co-Authored-By: Claude Sonnet 4.6 <[email protected]>

feat(rpc): add net namespace (version, listening)

2ac5514

Co-Authored-By: Claude Sonnet 4.6 <[email protected]>

feat(rpc): extend DebugError with RlpDecode and SenderRecovery variants

318de83

Co-Authored-By: Claude Sonnet 4.6 <[email protected]>

feat(rpc): add debug_getRawHeader and debug_getRawTransaction

84201ce

Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>

feat(rpc): add debug_getRawBlock and debug_getRawReceipts

eb53770

Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>

refactor(rpc): extract shared tracing loop from trace_block

2c34ed8

Extract trace_block_inner() so it can be reused by the new debug_traceBlock (raw RLP) handler. Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>

feat(rpc): add debug_traceBlock (raw RLP input)

88ee2b9

Decodes an RLP-encoded block, recovers transaction senders, and traces all transactions using the shared trace_block_inner loop. Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>

feat(rpc): add debug_traceCall

b8f3022

Co-Authored-By: Claude Sonnet 4.6 <[email protected]>

test(rpc): update test_not_supported → test_protocol_version

ef0f5c7

eth_protocolVersion is now implemented, so the test that expected method_not_found is updated to assert the "0x44" response. Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>

docs: add Parity trace namespace implementation plan

c2a7712

12 tasks covering TraceError, param types, config, Parity tracer functions, block replay helpers, 9 trace_ endpoints, and router wiring. Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>

prestwich force-pushed the prestwich/eng-960-rpc-namespace-completeness branch from f3d693c to f945ca7 Compare March 31, 2026 11:30

prestwich changed the base branch from prestwich/eng-1899 to develop March 31, 2026 11:31

prestwich marked this pull request as ready for review March 31, 2026 11:31

prestwich requested review from Evalir and Fraser999 March 31, 2026 11:31

prestwich enabled auto-merge (squash) March 31, 2026 12:04

chore: trigger CI

21c3877

Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>

prestwich merged commit 3ef2bcc into develop Mar 31, 2026
6 checks passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

feat(rpc): add web3, net namespaces and expand debug/eth for tooling compatibility#120

feat(rpc): add web3, net namespaces and expand debug/eth for tooling compatibility#120
prestwich merged 15 commits into
developfrom
prestwich/eng-960-rpc-namespace-completeness

prestwich commented Mar 25, 2026 •

edited

Loading

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

prestwich commented Mar 30, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Uh oh!

Conversation

prestwich commented Mar 25, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Key implementation details

Test plan

Review Fixes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

prestwich commented Mar 30, 2026

Code review

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

prestwich commented Mar 25, 2026 •

edited

Loading