Skip to content

chore: Adds prestate tracer to traceTransaction endpoint #3898

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

Conversation

konstantinabl
Copy link
Contributor

@konstantinabl konstantinabl commented Jul 7, 2025

Description:

This PR allows users to use prestateTracer with the debug_traceTransaction endpoint.

  • Added prestatTracer as an option in traceTransaction method
  • Added acceptance/integration and unit tests

Related issue(s):

Fixes #3817

Notes for reviewer:

Checklist

  • Documented (Code comments, README, etc.)
  • Tested (unit, integration, etc.)

@konstantinabl konstantinabl linked an issue Jul 7, 2025 that may be closed by this pull request
Copy link

github-actions bot commented Jul 7, 2025

Test Results

 20 files  ±0  277 suites  +1   17m 49s ⏱️ -5s
703 tests +1  698 ✅ +4  5 💤 ±0  0 ❌  - 3 
719 runs  +2  714 ✅ +5  5 💤 ±0  0 ❌  - 3 

Results for commit 8a85fb9. ± Comparison against base commit d838e16.

This pull request removes 2 and adds 3 tests. Note that renamed tests count towards both.
"after all" hook in "@web-socket-batch-1 eth_getBlockByHash" ‑ RPC Server Acceptance Tests Acceptance tests @web-socket-batch-1 eth_getBlockByHash "after all" hook in "@web-socket-batch-1 eth_getBlockByHash"
should fail with INVALID_PARAMETER when using PrestateTracer ‑ RPC Server Acceptance Tests Acceptance tests @api-conformity @debug API Acceptance Tests debug_traceTransaction Edge Cases - Parameter Validation should fail with INVALID_PARAMETER when using PrestateTracer
should trace a transaction using PrestateTracer with onlyTopCall=false ‑ RPC Server Acceptance Tests Acceptance tests @api-conformity @debug API Acceptance Tests debug_traceTransaction PrestateTracer should trace a transaction using PrestateTracer with onlyTopCall=false
should trace a transaction using PrestateTracer with onlyTopCall=true ‑ RPC Server Acceptance Tests Acceptance tests @api-conformity @debug API Acceptance Tests debug_traceTransaction PrestateTracer should trace a transaction using PrestateTracer with onlyTopCall=true
should trace a transaction using PrestateTracer ‑ RPC Server Acceptance Tests Acceptance tests @api-conformity @debug API Acceptance Tests debug_traceTransaction PrestateTracer should trace a transaction using PrestateTracer

♻️ This comment has been updated with latest results.

@konstantinabl konstantinabl self-assigned this Jul 8, 2025
@konstantinabl konstantinabl added the enhancement New feature or request label Jul 8, 2025
@konstantinabl konstantinabl added this to the 0.70.0 milestone Jul 8, 2025
Signed-off-by: Konstantina Blazhukova <[email protected]>
@konstantinabl konstantinabl marked this pull request as ready for review July 8, 2025 13:52
@konstantinabl konstantinabl requested review from a team as code owners July 8, 2025 13:52
@konstantinabl konstantinabl requested a review from simzzz July 8, 2025 13:52
Signed-off-by: Konstantina Blazhukova <[email protected]>
Signed-off-by: Konstantina Blazhukova <[email protected]>
@konstantinabl konstantinabl merged commit 642df65 into main Jul 9, 2025
43 of 45 checks passed
@konstantinabl konstantinabl deleted the 3817-add-prestatetracer-to-debug_tracetransaction branch July 9, 2025 08:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add prestateTracer to debug_traceTransaction
3 participants