KarpelesLab / libwallet / 28647121355

25%

Build Type

push

github

Committed by MagicalTux

Commit Message

wltbase: list registered ERC-20 tokens with live balances in Asset:list

EVM tokens registered via Token:create / Token:discoverToken (or swap
EnsureToken) produced a Token row but never appeared in Asset:list — only
native currencies and Solana SPL tokens did, so adding e.g. USDC on Polygon
showed no balance row at all. Unlike Solana, EVM has no cheap owner→tokens
enumeration, so the user's Token registry is the source of truth:
computeAssets now lists the network's erc20 rows (new
wlttoken.TokensByNetwork) and reads each balance via eth_call balanceOf
(wlttx.EVMERC20BalanceOf, exporting the existing maxSendable helper).

Zero balances are included on purpose — a token the user explicitly
registered shows as "0" rather than vanishing. Per-token RPC failures skip
that token (logged) instead of failing the snapshot. The path rides the
0.4.68 balance-snapshot cache and the 60s poller, so balances_changed now
fires for ERC-20 movements too. Asset.Key mirrors the Solana convention
("<network-id>.<contract>").

Reimplements the feature from Jeremy's `erc20` branch (bb114f1..a420635)
natively: the branch predates the wlttoken package (it carried its own
Token type in wltnet), the Asset:list snapshot-cache rewrite, and the
wltobj rename, and sits 440 commits behind master — unmergeable as-is, but
the gap it addressed was real. The branch can be closed in its favor.

Integration test (wlttest/erc20_assets_test.go): registry round-trip +
network scoping against the temp env, and a live Polygon balanceOf
round-trip (skips if the public RPC is unreachable).

Part of unreleased 0.4.78.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

Coverage Stats

0 of 44 new or added lines in 3 files covered. (0.0%)

1 existing line in 1 file now uncovered.

5027 of 20425 relevant lines covered (24.61%)

93.33 hits per line