feat(evmonly): add eth_getBalance RPC - #4140
Conversation
|
The latest Buf updates on your PR. Results from workflow Buf / buf (pull_request).
|
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #4140 +/- ##
==========================================
- Coverage 61.58% 60.62% -0.97%
==========================================
Files 2196 2080 -116
Lines 193825 180359 -13466
==========================================
- Hits 119365 109336 -10029
+ Misses 62888 60605 -2283
+ Partials 11572 10418 -1154
Flags with carried forward coverage won't be shown. Click here to find out more.
🚀 New features to boost your workflow:
|
PR SummaryMedium Risk Overview The handler accepts Coverage includes unit tests, an HTTP Reviewed by Cursor Bugbot for commit 448897f. Bugbot is set up for automated code reviews on this repo. Configure here. |
There was a problem hiding this comment.
Adds eth_getBalance to the EVM-only JSON-RPC surface, reading the current committed EVM state and rejecting historical selectors, with unit, registration, and Docker integration coverage. The block-selector gating, the nil Sei-address argument (only reachable in EVM-only mode, where it is ignored), the RPC-surface exposure, and the 2^200 + 1 integration assertion all check out; nothing blocking found.
Findings: 0 blocking | 0 non-blocking | 0 posted inline
Blockers
- None at the file/PR level.
Non-blocking
- None at the file/PR level.
Describe your changes and provide context
eth_getBalancefrom the EVM-only JSON-RPC serverlatest,safe,finalized, andpending, while rejecting block heights and hashes until historical state is wiredTesting performed to validate your change
go test -race -count=1 ./giga/evmonly/rpc/...go test -count=1 ./sei-tendermint/internal/rpc/core/...make autobahn-evmonly-integration-test(four local Docker validators, 4,000 finalized transfers, post-transfer balance checks on every validator)golangci-lint run ./giga/evmonly/rpc/... ./sei-tendermint/internal/rpc/core/...golangci-lint fmt --diff