@sodax/dapp-kit 1.3.0-beta → 1.3.1-beta-rc1

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@sodax/dapp-kit",
   "license": "MIT",
-  "version": "1.3.0-beta",
+  "version": "1.3.1-beta-rc1",
   "description": "dapp-kit of New World",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",
@@ -16,8 +16,8 @@
   },
   "dependencies": {
     "viem": "2.29.2",
-    "@sodax/sdk": "1.3.0-beta",
-    "@sodax/types": "1.3.0-beta"
+    "@sodax/sdk": "1.3.1-beta-rc1",
+    "@sodax/types": "1.3.1-beta-rc1"
   },
   "devDependencies": {
     "@types/react": "19.0.8",

package/src/hooks/mm/useMMAllowance.ts

@@ -38,14 +38,27 @@ export function useMMAllowance({
 
   const defaultQueryOptions = {
     queryKey: ['mm', 'allowance', params?.token, params?.action],
-    enabled: !!spokeProvider,
+    /**
+     * IMPORTANT: Skip allowance checks for 'borrow' and 'withdraw' actions.
+     *
+     * Reason: According to the SDK's MoneyMarketService.isAllowanceValid() implementation,
+     * borrow and withdraw actions do NOT require ERC-20 token approval. The SDK's
+     * isAllowanceValid() method always returns `true` for these actions without making
+     * any on-chain allowance checks.
+     *
+     * This optimization prevents unnecessary RPC calls and avoids showing confusing states for actions that don't actually need approval.
+     *
+     * Only 'supply' and 'repay' actions require token approval and should trigger allowance checks.
+     */
+    enabled: !!spokeProvider && !!params && params.action !== 'borrow' && params.action !== 'withdraw',
     refetchInterval: 5000,
+    gcTime: 0, // Don't cache failed queries
   };
 
   queryOptions = {
     ...defaultQueryOptions,
     ...queryOptions, // override default query options if provided
-  }
+  };
 
   return useQuery({
     ...queryOptions,
@@ -53,6 +66,21 @@ export function useMMAllowance({
       if (!spokeProvider) throw new Error('Spoke provider is required');
       if (!params) throw new Error('Params are required');
 
+      /**
+       * Early return for borrow/withdraw actions: these actions do NOT require ERC-20 token approval.
+       *
+       * The SDK's MoneyMarketService.isAllowanceValid() always returns `true` for borrow/withdraw
+       * without checking on-chain allowance. This is because:
+       * - Borrow: User receives tokens (no approval needed)
+       * - Withdraw: User withdraws their own supplied tokens (no approval needed)
+       *
+       * By returning `true` here, we avoid unnecessary RPC calls and ensure consistent behavior
+       * with the SDK's implementation.
+       */
+      if (params.action === 'borrow' || params.action === 'withdraw') {
+        return true;
+      }
+
       const allowance = await sodax.moneyMarket.isAllowanceValid(params, spokeProvider);
 
       if (!allowance.ok) {

package/src/hooks/provider/useSpokeProvider.ts

@@ -67,9 +67,12 @@ export function useSpokeProvider(
           spokeChainConfig[spokeChainId] as SonicSpokeChainConfig,
         );
       }
+      // EVM RPC: flat RpcConfig keyed by chain id (same shape as app's rpcConfig)
+      const evmRpcUrl = rpcConfig[spokeChainId];
       return new EvmSpokeProvider(
         walletProvider as IEvmWalletProvider,
         spokeChainConfig[spokeChainId] as EvmSpokeChainConfig,
+        typeof evmRpcUrl === 'string' ? evmRpcUrl : undefined,
       );
     }
 

package/src/hooks/staking/useConvertedAssets.ts

@@ -23,12 +23,12 @@ import { useQuery, type UseQueryResult } from '@tanstack/react-query';
 export function useConvertedAssets(amount: bigint | undefined, refetchInterval = 10000): UseQueryResult<bigint, Error> {
   const { sodax } = useSodaxContext();
 
-  console.log('useConvertedAssets hook called with:', { amount: amount?.toString(), sodax: !!sodax });
+  // console.log('useConvertedAssets hook called with:', { amount: amount?.toString(), sodax: !!sodax });
 
   return useQuery({
     queryKey: ['soda', 'convertedAssets', amount?.toString()],
     queryFn: async () => {
-      console.log('useConvertedAssets queryFn called with amount:', amount?.toString());
+      // console.log('useConvertedAssets queryFn called with amount:', amount?.toString());
       if (!amount || amount <= 0n) {
         throw new Error('Amount must be greater than 0');
       }