Merge branch 'main' into feat/update-block-remove-liquidity

Author: agualis

lib/modules/pool/actions/add-liquidity/form/useMaximumInputs.tsx

@@ -23,10 +23,11 @@ export function useMaximumInputs() {
 
   // Depending on if the user is using WETH or ETH, we need to filter out the
   // native asset or wrapped native asset.
-  const nativeAssetFilter = (balance: TokenAmount) =>
-    wethIsEth
-      ? wNativeAsset && balance.address !== wNativeAsset.address
-      : nativeAsset && balance.address !== nativeAsset.address
+  const nativeAssetFilter = (balance: TokenAmount) => {
+    return wethIsEth
+      ? balance.address !== wNativeAsset?.address
+      : balance.address !== nativeAsset?.address
+  }
 
   const filteredBalances = useMemo(() => {
     return balances.filter(nativeAssetFilter)

lib/modules/web3/ChainConfig.tsx

@@ -53,8 +53,8 @@ export const rpcOverrides: Record<GqlChain, string | undefined> = {
   [GqlChain.Polygon]: getPrivateRpcUrl(GqlChain.Polygon),
   [GqlChain.Zkevm]: getPrivateRpcUrl(GqlChain.Zkevm),
   [GqlChain.Sepolia]: getPrivateRpcUrl(GqlChain.Sepolia),
-  [GqlChain.Mode]: undefined,
-  [GqlChain.Fraxtal]: undefined,
+  [GqlChain.Mode]: getPrivateRpcUrl(GqlChain.Mode),
+  [GqlChain.Fraxtal]: getPrivateRpcUrl(GqlChain.Fraxtal),
 }
 
 const customMainnet = { iconUrl: '/images/chains/MAINNET.svg', ...mainnet }

lib/shared/components/errors/GenericError.tsx

@@ -2,7 +2,12 @@
 
 import { AlertProps, Text } from '@chakra-ui/react'
 import { ErrorAlert } from './ErrorAlert'
-import { isPausedError, isUserRejectedError, isViemHttpFetchError } from '../../utils/error-filters'
+import {
+  isPausedError,
+  isTooManyRequestsError,
+  isUserRejectedError,
+  isViemHttpFetchError,
+} from '../../utils/error-filters'
 import { ensureError } from '../../utils/errors'
 import { BalAlertLink } from '../alerts/BalAlertLink'
 
@@ -39,6 +44,17 @@ export function GenericError({ error: _error, customErrorName, ...rest }: Props)
       </ErrorAlert>
     )
   }
+  if (isTooManyRequestsError(_error)) {
+    return (
+      <ErrorAlert title={customErrorName} {...rest}>
+        <Text variant="secondary" color="black">
+          Too many RPC requests. Please try again in some minutes. You can report the problem in{' '}
+          <BalAlertLink href="https://discord.balancer.fi/">our discord</BalAlertLink> if the issue
+          persists.
+        </Text>
+      </ErrorAlert>
+    )
+  }
   const errorMessage = error?.shortMessage || error.message
 
   if (errorMessage === 'RPC Request failed.' || errorMessage === 'An unknown RPC error occurred.') {

lib/shared/utils/error-filters.ts

@@ -13,6 +13,19 @@ export function isViemHttpFetchError(error?: Error | null): boolean {
   )
 }
 
+/*
+  Detects 429 Too Many Requests errors thrown by wagmi/viem
+  We should not have many of them as we are using a private RPC provider but they still could happen when:
+  - the provider's rate limiting is not working as expected
+  - the provider is down and we are using a public fallback
+  - others
+*/
+export function isTooManyRequestsError(error?: Error | null): boolean {
+  console.log(error?.message)
+  if (!error) return false
+  return error.message.startsWith('HTTP request failed.') && error.message.includes('Status: 429')
+}
+
 export function isPausedError(error?: Error | null): boolean {
   if (!error) return false
   return isPausedErrorMessage(error.message)