feat: remove Tron non tradable assets from token selector

[GeorgeGkas]

Description

Tron Snap currently treats 'energy' and 'bandwidth' as assets, which causes them to appear in asset lists, including the Swaps asset pickers. Since these are not tradeable assets, we need to implement filtering on the asset pickers to exclude them from the UI. We ensure that 'energy' and 'bandwidth' are not displayed in asset pickers where tradeable assets are expected.

Changelog

CHANGELOG entry: Remove non-tradable assets from asset picker

Related issues

Fixes: https://consensyssoftware.atlassian.net/browse/SWAPS-3419

Manual testing steps

- Ensure `Energy`, `Bandwidth` and `Max bandwidth` tokens are not rendered in asset picker.
- Ensure that all tokens excepts the one mentioned above are still there.

Screenshots/Recordings

Before

After

Pre-merge author checklist

• I’ve followed MetaMask Contributor Docs and MetaMask Mobile Coding Standards.
• I've completed the PR template to the best of my ability
• I’ve included tests if applicable
• I’ve documented my code using JSDoc format if applicable
• I’ve applied the right labels on the PR (see labeling guidelines). Not required for external contributors.

Pre-merge reviewer checklist

• I've manually tested the PR (e.g. pull and build branch, run the app, test code being changed).
• I confirm that this PR addresses all acceptance criteria described in the ticket it closes and includes the necessary testing evidence such as recordings and or screenshots.

---

Note

Introduces isTradableToken and updates useTokens to filter Tron Mainnet resource tokens (Energy, Bandwidth, Max Bandwidth) from bridge token lists, with comprehensive tests.

• Bridge:
  • Filtering: Add utils/isTradableToken to mark non-tradable Tron Mainnet tokens (Energy, Bandwidth, Max Bandwidth) as excluded.
  • Hook updates: Integrate isTradableToken into hooks/useTokens filtering for tokensWithoutBalance, allTokens, and tokensToRender.
  • Tests: Add unit tests for isTradableToken covering EVM, Solana, Tron (mainnet/testnet), CAIP IDs, casing/whitespace, and name presence.

^{Written by Cursor Bugbot for commit dac5ef9. This will update automatically on new commits. Configure here.}

[github-actions]

CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
100.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[bfullam]

Adding this as a separate .filter() means an additional iteration through the entire array, which is fairly inefficient especially for larger lists. Instead, we can optimize this with a util function like isTradableToken() that can be used as part of the existing filter pipes. E.g.

.filter((token) => {
if (!isTradableToken(token)) return false;
const tokenKey = getTokenKey(token);
return !excludedTokensSet.has(tokenKey);
});

[GeorgeGkas]

yeah I thought about that but did not consider the fact that lists can be so big and neither how to ecapsulate it into the existing ones for some reason, isTradableToken is a nice touch.

[bfullam]

Here we can/should compare to the actual chainId, stored in TrxScope.Mainnet. E.g.

token.chainId === TrxScope.Mainnet