fix(perps): recover HIP-3 DEX discovery on transient failure and guard deposit flow

- HyperLiquidProvider: on transient perpDexs() fetch failure, return [null] without
  caching so the next #ensureReady() call retries DEX discovery instead of reusing a
  stale degraded result. Added #dexDiscoveryComplete flag so #ensureReady resets its
  promise and re-runs #buildAssetMapping until discovery succeeds — trading on main
  DEX continues uninterrupted in the meantime. Flag is cleared on disconnect for
  clean reconnection.

- usePerpsBalanceTokenFilter: call ensureArbitrumNetworkExists() before
  depositWithConfirmation() in handlePerpsDepositPress, matching the existing guard
  in usePerpsHomeActions. Fixes "Invalid chain ID 0xa4b1" for users without Arbitrum
  configured who tap "Add funds" from the pay-with modal.

Author: abretonc7s

app/components/UI/Perps/hooks/usePerpsBalanceTokenFilter.ts

@@ -19,6 +19,7 @@ import { useIsPerpsBalanceSelected } from './useIsPerpsBalanceSelected';
 import { usePerpsPaymentToken } from './usePerpsPaymentToken';
 import Routes from '../../../../constants/navigation/Routes';
 import { usePerpsTrading } from './usePerpsTrading';
+import { usePerpsNetworkManagement } from './usePerpsNetworkManagement';
 import { useConfirmNavigation } from '../../../Views/confirmations/hooks/useConfirmNavigation';
 
 /** URI for the perps balance token icon, shared with PerpsPayRow and pay-with modal. */
@@ -45,18 +46,28 @@ export function usePerpsBalanceTokenFilter(): (
   const formatFiat = useFiatFormatter({ currency: 'usd' });
 
   const { depositWithConfirmation } = usePerpsTrading();
+  const { ensureArbitrumNetworkExists } = usePerpsNetworkManagement();
   const { navigateToConfirmation } = useConfirmNavigation();
 
   const isPerpsDepositAndOrder = hasTransactionType(transactionMeta, [
     TransactionType.perpsDepositAndOrder,
   ]);
 
   const handlePerpsDepositPress = useCallback(() => {
-    navigateToConfirmation({ stack: Routes.PERPS.ROOT });
-    depositWithConfirmation().catch(() => {
-      // Deposit flow handles errors (e.g. user rejection).
-    });
-  }, [navigateToConfirmation, depositWithConfirmation]);
+    ensureArbitrumNetworkExists()
+      .then(() => {
+        navigateToConfirmation({ stack: Routes.PERPS.ROOT });
+        return depositWithConfirmation();
+      })
+      .catch((_err) => {
+        // Deposit flow handles errors (e.g. user rejection or missing network).
+        // ensureArbitrumNetworkExists errors are logged inside the hook itself.
+      });
+  }, [
+    ensureArbitrumNetworkExists,
+    navigateToConfirmation,
+    depositWithConfirmation,
+  ]);
 
   const { onPaymentTokenChange: onPerpsPaymentTokenChange } =
     usePerpsPaymentToken();

app/controllers/perps/providers/HyperLiquidProvider.ts

@@ -317,6 +317,11 @@ export class HyperLiquidProvider implements PerpsProvider {
 
   #cachedAllPerpDexs: ({ name: string } | null)[] | null = null;
 
+  // True once DEX discovery has succeeded with real data (not a fallback).
+  // When false, #ensureReadyPromise is reset after each init so the next
+  // caller retries DEX discovery instead of reusing a degraded mapping.
+  #dexDiscoveryComplete = false;
+
   // Pending promise to deduplicate concurrent getValidatedDexs() calls
   #pendingValidatedDexsPromise: Promise<(string | null)[]> | null = null;
 
@@ -698,8 +703,8 @@ export class HyperLiquidProvider implements PerpsProvider {
       // Verify clients are properly initialized
       this.#clientService.ensureInitialized();
 
-      // Build asset mapping on first call only (flags are immutable)
-      if (this.#symbolToAssetId.size === 0) {
+      // Build asset mapping on first call, or retry if DEX discovery previously failed
+      if (this.#symbolToAssetId.size === 0 || !this.#dexDiscoveryComplete) {
         this.#deps.debugLogger.log(
           'HyperLiquidProvider: Building asset mapping',
           {
@@ -717,8 +722,15 @@ export class HyperLiquidProvider implements PerpsProvider {
     })();
 
     // Await initialization - keep the promise so subsequent calls resolve immediately
-    // The promise is only reset in disconnect() for clean reconnection
+    // The promise is only reset in disconnect() for clean reconnection,
+    // or when DEX discovery was degraded so the next caller retries.
     await this.#ensureReadyPromise;
+    if (!this.#dexDiscoveryComplete) {
+      // DEX discovery failed transiently — reset so next call retries.
+      // Trading still works (main DEX mapping is populated), but HIP-3 markets
+      // will be re-discovered on the next #ensureReady() call.
+      this.#ensureReadyPromise = null;
+    }
     this.#deps.debugLogger.log('[ensureReady] Initialization complete');
   }
 
@@ -1041,25 +1053,23 @@ export class HyperLiquidProvider implements PerpsProvider {
         ensureError(error, 'HyperLiquidProvider.fetchValidatedDexsInternal'),
         this.#getErrorContext('getValidatedDexs.perpDexs'),
       );
-      this.#cachedAllPerpDexs = [null];
-      this.#cachedValidatedDexs = [null];
-      return this.#cachedValidatedDexs;
+      // Do not cache — transient error, allow retry on next call
+      return [null];
     }
 
-    // Cache for buildAssetMapping() to avoid duplicate call
-    this.#cachedAllPerpDexs = allDexs;
-
     // Validate API response
     if (!allDexs || !Array.isArray(allDexs)) {
       this.#deps.debugLogger.log(
         'HyperLiquidProvider: Failed to fetch DEX list (invalid response), falling back to main DEX only',
         { allDexs },
       );
-      this.#cachedAllPerpDexs = [null];
-      this.#cachedValidatedDexs = [null];
-      return this.#cachedValidatedDexs;
+      // Do not cache — may be transient, allow retry on next call
+      return [null];
     }
 
+    // Cache for buildAssetMapping() to avoid duplicate call
+    this.#cachedAllPerpDexs = allDexs;
+
     // Extract HIP-3 DEX names (filter out null which represents main DEX)
     const availableHip3Dexs: string[] = [];
     allDexs.forEach((dex) => {
@@ -1943,6 +1953,13 @@ export class HyperLiquidProvider implements PerpsProvider {
     let dexsToMap: (string | null)[];
     try {
       dexsToMap = await this.#getValidatedDexs();
+      // Mark DEX discovery as complete only when we got real data (not a fallback).
+      // #cachedValidatedDexs is null when #fetchValidatedDexsInternal returned [null]
+      // without caching (transient failure) — in that case we stay incomplete so
+      // #ensureReady resets its promise and retries on the next call.
+      if (this.#cachedValidatedDexs !== null) {
+        this.#dexDiscoveryComplete = true;
+      }
     } catch (dexError) {
       // If getValidatedDexs fails, fall back to main DEX only to keep the provider
       // functional. Without this, a transient perpDexs() failure would permanently
@@ -7318,6 +7335,7 @@ export class HyperLiquidProvider implements PerpsProvider {
       this.#cachedMetaByDex.clear();
       this.#cachedSpotMeta = null;
       this.#perpDexsCache = { data: null, timestamp: 0 };
+      this.#dexDiscoveryComplete = false;
 
       // Await pending initialization before clearing to prevent the IIFE from
       // setting clientsInitialized = true after disconnect completes