KarpelesLab / libwallet / 25848034620

24%

push

github

wltswap: strip Asset.Key prefix from tokenIn/tokenOut addresses

Hosts pipe Asset.Key directly into TokenRef.Address — the prefixed
"<type>.<chainId>.<mint>" form Asset:list returns. The aggregators
only accept bare mints / contracts, so Jupiter / 1inch rejected the
qualified strings:

  HTTP 400: {"error":"Invalid outputMint"}
  outputMint=solana.mainnet.EPjFWdd5AufqSSqeM2qN1xzybapC8G4wEGGkZwyTDt1v

Normalise at the swap.go entry points (swapQuote, swapMaxSpendable)
so the prefix never reaches the adapter / aggregator wire. The
NATIVE sentinel works in either form. Bare addresses pass through
unchanged; the round-trip is a no-op on those.

solanaNativeMintOrAddr and oneInchTokenOrSentinel also strip the
prefix as a belt-and-suspenders so the bug can't reappear via a
future adapter that takes a TokenRef.Address without going through
the swap.go entry.

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

7 of 11 new or added lines in 3 files covered. (63.64%)

3499 of 16359 relevant lines covered (21.39%)

115.1 hits per line