fix(token-lists): preserve first-wins semantics in chainsById and drop dead catch

viem/chains exports 702 chain objects spanning 675 unique IDs; 27 export
names share a chainId, and 10 of those collisions have divergent
nativeCurrency. The previous Map(...map) construction was last-wins, silently
changing buildNativeToken results versus the original find()-first-wins
behaviour for chainIds 9, 166, 260, 999, 1001, 1337, 8217, 9496, 200810,
200901 -- most consequentially chainId 1337 (Localhost ETH/18 → Tempo USD/6).
Replace with an explicit first-wins loop.

Hoist chainsById to the top of the module so both the filter in
combineTokenLists and buildNativeToken reference the same declaration without
a forward-reference.

After the chainsById.has(token.chainId) filter, every token entering the
reduce accumulator has a known chain, making the try/catch/console.error
branch unreachable. Remove it; the throw inside buildNativeToken remains as
a defensive invariant for callers that bypass the filter.

Add a positive-survival assertion to the unsupported-chainId test to guard
against future filter inversions silently dropping valid tokens.

Author: gabitoesmiapodo

src/hooks/useTokenLists.test.ts

@@ -241,6 +241,7 @@ describe('useTokenLists', () => {
     expect(result.current.tokens.some((t) => t.chainId === 3)).toBe(false)
     expect(result.current.tokensByChainId[3]).toBeUndefined()
     expect(errorSpy).not.toHaveBeenCalled()
+    expect(result.current.tokens.some((t) => t.address === mockToken1.address)).toBe(true)
 
     errorSpy.mockRestore()
   })

src/hooks/useTokenLists.ts

@@ -13,6 +13,11 @@ import { type Token, type TokenList, tokenSchema } from '@/src/types/token'
 import { logger } from '@/src/utils/logger'
 import tokenListsCache, { type TokensMap, updateTokenListsCache } from '@/src/utils/tokenListsCache'
 
+const chainsById: Map<number, (typeof chains)[keyof typeof chains]> = new Map()
+for (const chain of Object.values(chains)) {
+  if (!chainsById.has(chain.id)) chainsById.set(chain.id, chain)
+}
+
 /**
  * Loads and processes token lists from configured sources
  *
@@ -122,18 +127,9 @@ function combineTokenLists(results: Array<UseSuspenseQueryResult<TokenList>>): T
   const tokensMap = uniqueTokens.reduce<TokensMap>(
     (acc, token) => {
       if (!acc.tokensByChainId[token.chainId]) {
-        try {
-          // if there's a native token for the chain
-          const nativeToken = buildNativeToken(token.chainId)
-
-          // add it to the list
-          acc.tokensByChainId[token.chainId] = [nativeToken]
-          acc.tokens.push(nativeToken)
-        } catch (err) {
-          console.error(err)
-          // if there's no native token for the chain, ignore the error
-          acc.tokensByChainId[token.chainId] = []
-        }
+        const nativeToken = buildNativeToken(token.chainId)
+        acc.tokensByChainId[token.chainId] = [nativeToken]
+        acc.tokens.push(nativeToken)
       }
 
       acc.tokens.push(token)
@@ -197,10 +193,6 @@ export async function fetchTokenList(url: string): Promise<TokenList> {
   }
 }
 
-const chainsById: Map<number, (typeof chains)[keyof typeof chains]> = new Map(
-  Object.values(chains).map((c) => [c.id, c]),
-)
-
 /**
  * Builds a native token object based on the chain ID.
  *