fix(token-lists): drop tokens whose chainId is absent from viem/chains

@uniswap/default-token-list still ships tokens for Ropsten (chainId 3)
and Rinkeby (chainId 4), both removed from viem/chains. combineTokenLists
caught the buildNativeToken throw but logged it via console.error before
ignoring it, producing noise on every cold load.

Extend the existing safeParse filter to also drop tokens whose chainId
is not present in viem/chains, mirroring the existing non-EVM drop. This
removes the throw at source and keeps tokensByChainId free of
unreachable buckets.

Closes #465

Author: gabitoesmiapodo

src/hooks/useTokenLists.test.ts

@@ -222,6 +222,29 @@ describe('useTokenLists', () => {
     expect(nativeToken?.symbol).toBe('ETH')
   })
 
+  it('filters out tokens whose chainId is not present in viem/chains and does not log', () => {
+    // biome-ignore lint/suspicious/noExplicitAny: mocking internal combine param
+    vi.mocked(tanstackQuery.useSuspenseQueries).mockImplementation(({ combine }: any) => {
+      const ropstenToken: Token = {
+        address: '0xB4FBF271143F4FBf7B91A5ded31805e42b2208d6',
+        chainId: 3,
+        decimals: 18,
+        name: 'Wrapped Ether (Ropsten)',
+        symbol: 'WETH',
+      }
+      return combine([mockSuspenseQueryResult([mockToken1, ropstenToken])])
+    })
+
+    const errorSpy = vi.spyOn(console, 'error').mockImplementation(() => {})
+    const { result } = renderHook(() => useTokenLists(), { wrapper })
+
+    expect(result.current.tokens.some((t) => t.chainId === 3)).toBe(false)
+    expect(result.current.tokensByChainId[3]).toBeUndefined()
+    expect(errorSpy).not.toHaveBeenCalled()
+
+    errorSpy.mockRestore()
+  })
+
   it('filters out tokens that fail schema validation', () => {
     // biome-ignore lint/suspicious/noExplicitAny: mocking internal combine param
     vi.mocked(tanstackQuery.useSuspenseQueries).mockImplementation(({ combine }: any) => {

src/hooks/useTokenLists.ts

@@ -110,12 +110,14 @@ function combineTokenLists(results: Array<UseSuspenseQueryResult<TokenList>>): T
         .flatMap((result) => result.data.tokens)
         // tokenSchema enforces EVM address format (0x + 40 hex chars), so non-EVM entries
         // (e.g. Solana tokens from @uniswap/default-token-list v18+) are silently dropped here.
+        // Tokens whose chainId is not present in viem/chains (e.g. deprecated testnets like
+        // Ropsten/Rinkeby still shipped by @uniswap/default-token-list) are also dropped so
+        // buildNativeToken never throws and tokensByChainId stays free of unreachable buckets.
         // Supporting non-EVM chains would require changes to the address schema, chain config,
         // wallet integration, and contract lookup -- out of scope for this EVM-focused starter kit.
         .filter((token) => {
-          const result = tokenSchema.safeParse(token)
-
-          return result.success
+          if (!tokenSchema.safeParse(token).success) return false
+          return supportedChainIds.has(token.chainId)
         })
         .map((token) => [tokenKey(token), token]),
     ).values(),
@@ -201,6 +203,8 @@ export async function fetchTokenList(url: string): Promise<TokenList> {
   }
 }
 
+const supportedChainIds = new Set<number>(Object.values(chains).map((c) => c.id))
+
 /**
  * Builds a native token object based on the chain ID.
  *