KarpelesLab / libwallet / 25776160941

24%

push

github

wltnet/explorer: append ?cluster=<id> for non-mainnet Solana txs

TransactionUrl previously just appended `/tx/<hash>` to the explorer
base. Solana explorers (explorer.solana.com, solscan.io, solana.fm)
host one URL per address/tx across every cluster and switch on
`?cluster=`. Without the suffix, devnet/testnet tx links resolved
to "Transaction not found" because the explorer defaulted to mainnet.

Mainnet (and the "mainnet-beta" alias) stay bare — that's the
explorer default. Any other ChainId — devnet, testnet, custom local
clusters — gets `?cluster=<id>` appended. Custom explorers that
don't honour the suffix ignore it harmlessly.

The existing tests only asserted the path shape and never exercised
non-mainnet clusters — extended to cover devnet, testnet, the
"mainnet-beta" alias, custom-explorer + devnet, and empty-ChainId.

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

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

3339 of 15898 relevant lines covered (21.0%)

119.35 hits per line