Skip to content

fix: backport tracerConfig fix#4547

Open
skosito wants to merge 1 commit intorelease/v36-rpc-onlyfrom
backport-tracer-config-fix
Open

fix: backport tracerConfig fix#4547
skosito wants to merge 1 commit intorelease/v36-rpc-onlyfrom
backport-tracer-config-fix

Conversation

@skosito
Copy link
Member

@skosito skosito commented Feb 14, 2026

Description

original PR: #4490

How Has This Been Tested?

  • Tested CCTX in localnet
  • Tested in development environment
  • Go unit tests
  • Go integration tests
  • Tested via GitHub Actions

Note

Medium Risk
Touches tracing RPC request shaping and config serialization; mistakes could change trace behavior or break compatibility for debug endpoints, but scope is limited to tracing paths and covered by unit tests.

Overview
Fixes debug_traceTransaction/debug_traceBlock* to take rpctypes.TraceConfig (with tracerConfig) instead of directly exposing evmtypes.TraceConfig.

Adds toEVMTraceConfig conversion in backend tracing to marshal tracerConfig objects into TracerJsonConfig before calling QueryClient.TraceTx/TraceBlock, and updates mocks/tests to cover passing a map-based tracer config through to the EVM query request.

Written by Cursor Bugbot for commit e9fa131. Configure here.

@skosito skosito requested a review from a team as a code owner February 14, 2026 02:22
@skosito skosito added the no-changelog Skip changelog CI check label Feb 14, 2026
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 14, 2026

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

🗂️ Base branches to auto review (1)
  • develop

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch backport-tracer-config-fix

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

Copy link
Contributor

@ws4charlie ws4charlie left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-changelog Skip changelog CI check

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants