feat: enforce ERC20 compliance in NativeCoinAdapter & enhance tracer support #46
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Replace `evm.Call` with direct balance updates in coinManager. - Prevent invoking `receive`/`fallback` functions on recipient contracts, aligning with standard ERC20 behavior.
Refactor the prestate tracer test framework to use generics, resolving precision issues with uint64 fields (e.g., nonce, extra) during JSON marshaling. Also, inject Anzeon system contracts into the genesis block for call tracer and flat call tracer tests to properly support NativeManager operations. - Refactor `prestate_test.go` to use `testcase[T any]` - Fix `uint64` parsing issues in `account` struct - Inject system contracts in `calltrace_test.go` and `flat_calltrace_test.go`
Update the suicide_anzeon test case to correctly reflect the missing baseFee distribution in the trace result. - Update `suicide_anzeon.json` result to include correct balance changes
Enhance the prestate tracer to correctly capture state changes induced by NativeManager operations. Add `CaptureEnter` hook to track addresses involved in native calls. - Implement `CaptureEnter` in `native/prestate.go` for address lookup
Add comprehensive tracer test cases for NativeManager operations, including NativeCoinAdapter (mint, burn, transfer) and AccountManager (blacklist).
Cover prestate, call, and flat call tracers.
- Add `native_coin_adapter_{mint,burn,transfer}.json`
- Add `account_manager_blacklist.json`
eomti-wm
requested review from
0xmhha,
code0xff,
colinkim,
egonspace and
hominlee-wemade
eomti-wm
added
enhancement
egonspace
approved these changes
Jan 23, 2026
code0xff
approved these changes
Jan 23, 2026
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
This PR addresses two key objectives:
NativeCoinAdapterChanges
NativeCoinAdapter ERC20 Compliance
NativeCoinAdapterstrictly follows ERC20 standard behavior.mintandtransferoperations incore/vm/native_manager.go.evm.Callwith direct implementationnativeTransfer. This ensures that token transfers modify balances without invoking the recipient'sreceiveorfallbackfunctions, aligning with the passive nature of standard ERC20 transfers.NativeManager Tracing Support
mint,burn,transfer,blacklist).CaptureEnterhook innative/prestate.go.NativeManageroperations, even when state changes occur directly vianativeTransfer.Tracer Test Framework Refactor
prestate_test.goto use Go generics (testcase[T]), resolvinguint64precision loss during JSON unmarshaling tointerface{}.InjectContractslogic to test runners to correctly simulate the Anzeon system contract environment.Test Coverage Expansion
NativeCoinAdapter(mint,burn,transfer) andAccountManager(blacklist).authorizedtests were omitted as the logic is identical toblacklist.)prestate_tracer,call_tracer, andcall_tracer_flat.