KarpelesLab / libwallet / 25864401376

24%

push

github

wltasset: add Asset.IsNative / Asset.TokenAddress helpers

Hosts have been inventing native-vs-token matchers — branching on
Asset.type ("native" — which libwallet has never emitted; it's always
"fungible" for runtime balance entries), Asset.symbol (collides; an
ERC-20 can ship as "ETH" too), or Asset.name (localised). None are
reliable. The single invariant signal is the ".NATIVE" suffix on
Asset.key — and that's been the convention since the initial commit.

Wrap it as a method so hosts don't reinvent the predicate:

  // Go
  asset.IsNative()      // true when Key ends with ".NATIVE"
  asset.TokenAddress()  // mint/contract part, or "" for native

  // Dart (matching shape)
  asset.isNative        // bool getter
  asset.tokenAddress    // String? — null for native

Both receivers safe on nil/empty. Tests cover the native suffixes,
the token addresses (EVM 0x + Solana base58), and the edge cases
where ".NATIVE" appears as a substring rather than the trailing
segment (must use HasSuffix, not Contains).

Also fixes the misleading Asset.type doc on the Dart side that
listed "native" / "erc20" / "spl-token" as values — that list was
aspirational; the runtime value is always "fungible".

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

14 of 14 new or added lines in 1 file covered. (100.0%)

3565 of 16461 relevant lines covered (21.66%)

113.79 hits per line