@eth-optimism/actions-sdk 0.5.0 → 0.6.0

package/src/{swap → actions/swap}/providers/uniswap/UniswapSwapProvider.ts

@@ -1,26 +1,29 @@
 import { formatUnits } from 'viem'
 
-import { UNISWAP } from '@/constants/providers.js'
-import type { SupportedChainId } from '@/constants/supportedChains.js'
-import type { SwapQuoteParamsResolved } from '@/ens/types.js'
-import { expandMarkets, findMarket } from '@/swap/core/markets.js'
-import { SwapProvider } from '@/swap/core/SwapProvider.js'
+import { expandMarkets, findMarket } from '@/actions/swap/core/markets.js'
+import { SwapProvider } from '@/actions/swap/core/SwapProvider.js'
 import {
   getSupportedChainIds,
   getUniswapAddresses,
-} from '@/swap/providers/uniswap/addresses.js'
+} from '@/actions/swap/providers/uniswap/addresses.js'
 import {
   encodeUniversalRouterSwap,
   getQuote,
-} from '@/swap/providers/uniswap/encoding.js'
+} from '@/actions/swap/providers/uniswap/encoding.js'
 import {
   configToMarkets,
   getValidMarketConfigs,
-} from '@/swap/providers/uniswap/markets.js'
+} from '@/actions/swap/providers/uniswap/markets.js'
 import type {
   UniswapMarketConfig,
   UniswapSwapProviderConfig,
-} from '@/swap/providers/uniswap/types.js'
+} from '@/actions/swap/providers/uniswap/types.js'
+import { UNISWAP } from '@/constants/providers.js'
+import type { SupportedChainId } from '@/constants/supportedChains.js'
+import { AssetMetadataRequiredError } from '@/core/error/errors.js'
+import type { ChainManager } from '@/services/ChainManager.js'
+import type { SwapQuoteParamsResolved } from '@/services/nameservices/ens/types.js'
+import type { SwapSettings } from '@/types/actions.js'
 import type { Asset } from '@/types/asset.js'
 import type {
   GetSwapMarketParams,
@@ -30,12 +33,21 @@ import type {
   SwapQuote,
   SwapTransaction,
 } from '@/types/swap/index.js'
+import { resolveApprovalMode } from '@/utils/approve.js'
 import { isNativeAsset, parseAssetAmount } from '@/utils/assets.js'
 
 /**
  * Uniswap V4 swap provider using Universal Router and Permit2 approvals.
  */
 export class UniswapSwapProvider extends SwapProvider<UniswapSwapProviderConfig> {
+  constructor(
+    config: UniswapSwapProviderConfig,
+    chainManager: ChainManager,
+    settings?: SwapSettings,
+  ) {
+    super(config, chainManager, settings)
+  }
+
   protocolSupportedChainIds(): SupportedChainId[] {
     return getSupportedChainIds()
   }
@@ -61,7 +73,10 @@ export class UniswapSwapProvider extends SwapProvider<UniswapSwapProviderConfig>
       deadline: params.deadline,
       recipient: params.recipient,
     })
-    return this.buildSwapTransactions(swapQuote)
+    return this.buildSwapTransactions({
+      ...swapQuote,
+      approvalMode: params.approvalMode,
+    })
   }
 
   protected async _buildApprovals(quote: SwapQuote) {
@@ -73,10 +88,15 @@ export class UniswapSwapProvider extends SwapProvider<UniswapSwapProviderConfig>
         assetOut: quote.assetOut,
         slippage: quote.slippage,
         deadline: quote.deadline,
-        recipient: quote.recipient!,
-        walletAddress: quote.recipient!,
+        recipient: quote.recipient,
+        walletAddress: quote.recipient,
         chainId: quote.chainId,
         amountInRaw: quote.amountInRaw,
+        approvalMode: resolveApprovalMode(
+          quote.approvalMode,
+          this._config.approvalMode,
+          this._settings.approvalMode,
+        ),
       },
       quote.amountInRaw,
       addresses.permit2,
@@ -162,7 +182,7 @@ export class UniswapSwapProvider extends SwapProvider<UniswapSwapProviderConfig>
       quotedAt: now,
       expiresAt: deadline,
       gasEstimate: quote.gasEstimate,
-      quotedRecipient: recipient,
+      recipient,
     }
   }
 
@@ -210,7 +230,7 @@ export class UniswapSwapProvider extends SwapProvider<UniswapSwapProviderConfig>
       | UniswapMarketConfig
       | undefined
     if (config?.fee === undefined || config?.tickSpacing === undefined) {
-      throw new Error(
+      throw new AssetMetadataRequiredError(
         `fee and tickSpacing must be configured for pair ${assetIn.metadata.symbol}/${assetOut.metadata.symbol}`,
       )
     }

package/src/{swap → actions/swap}/providers/uniswap/__tests__/UniswapSwapProvider.test.ts

@@ -3,10 +3,10 @@ import { baseSepolia } from 'viem/chains'
 import { describe, expect, it, vi } from 'vitest'
 
 import { MockWETHAsset } from '@/__mocks__/MockAssets.js'
+import type { UniswapSwapProviderConfig } from '@/actions/swap/providers/uniswap/types.js'
+import { UniswapSwapProvider } from '@/actions/swap/providers/uniswap/UniswapSwapProvider.js'
 import type { SupportedChainId } from '@/constants/supportedChains.js'
 import type { ChainManager } from '@/services/ChainManager.js'
-import type { UniswapSwapProviderConfig } from '@/swap/providers/uniswap/types.js'
-import { UniswapSwapProvider } from '@/swap/providers/uniswap/UniswapSwapProvider.js'
 import type { Asset } from '@/types/asset.js'
 
 const CHAIN_ID = baseSepolia.id as SupportedChainId

package/src/{swap → actions/swap}/providers/uniswap/__tests__/sdk.test.ts

@@ -1,12 +1,19 @@
-import { type Address, type PublicClient, zeroAddress } from 'viem'
+import {
+  type Address,
+  decodeAbiParameters,
+  decodeFunctionData,
+  type PublicClient,
+  zeroAddress,
+} from 'viem'
 import { describe, expect, it, vi } from 'vitest'
 
-import type { SupportedChainId } from '@/constants/supportedChains.js'
+import { UNIVERSAL_ROUTER_ABI } from '@/actions/swap/providers/uniswap/abis.js'
 import {
   calculatePriceImpact,
   encodeUniversalRouterSwap,
   getQuote,
-} from '@/swap/providers/uniswap/encoding.js'
+} from '@/actions/swap/providers/uniswap/encoding.js'
+import type { SupportedChainId } from '@/constants/supportedChains.js'
 import type { Asset } from '@/types/asset.js'
 
 const USDC: Asset = {
@@ -302,6 +309,61 @@ describe('encodeUniversalRouterSwap', () => {
     expect(calldata.length).toBeGreaterThan(10)
   })
 
+  it('tags V4 action bytes per Uniswap v4-periphery Actions.sol', () => {
+    // Regression: SWAP_EXACT_OUT_SINGLE was previously encoded as 0x07, which
+    // V4Router treats as multi-hop SWAP_EXACT_IN. The router decoded our
+    // single-hop EXACT_OUTPUT_SINGLE_PARAMS struct as a PathKey[] path and
+    // bare-reverted on pool lookup. Correct codes:
+    //   0x06 SWAP_EXACT_IN_SINGLE
+    //   0x08 SWAP_EXACT_OUT_SINGLE
+    const decodeActions = (calldata: `0x${string}`): `0x${string}` => {
+      const { args } = decodeFunctionData({
+        abi: UNIVERSAL_ROUTER_ABI,
+        data: calldata,
+      })
+      const [, inputs] = args as readonly [
+        `0x${string}`,
+        ReadonlyArray<`0x${string}`>,
+        bigint,
+      ]
+      const [actions] = decodeAbiParameters(
+        [{ type: 'bytes' }, { type: 'bytes[]' }],
+        inputs[0]!,
+      )
+      return actions as `0x${string}`
+    }
+
+    const exactIn = encodeUniversalRouterSwap({
+      amountInRaw: 100000000n,
+      assetIn: USDC,
+      assetOut: WETH,
+      slippage: 0.005,
+      deadline: 1700000000,
+      recipient: '0xrecipient' as Address,
+      chainId: CHAIN_ID,
+      quote: baseQuote,
+      universalRouterAddress: '0xrouter' as Address,
+      fee: FEE,
+      tickSpacing: TICK_SPACING,
+    })
+    expect(decodeActions(exactIn)).toBe('0x060c0f')
+
+    const exactOut = encodeUniversalRouterSwap({
+      amountOutRaw: 500000000000000000n,
+      assetIn: USDC,
+      assetOut: WETH,
+      slippage: 0.005,
+      deadline: 1700000000,
+      recipient: '0xrecipient' as Address,
+      chainId: CHAIN_ID,
+      quote: baseQuote,
+      universalRouterAddress: '0xrouter' as Address,
+      fee: FEE,
+      tickSpacing: TICK_SPACING,
+    })
+    expect(decodeActions(exactOut)).toBe('0x080c0f')
+  })
+
   it('produces different calldata for exact-in vs exact-out', () => {
     const exactIn = encodeUniversalRouterSwap({
       amountInRaw: 100000000n,

package/src/{swap → actions/swap}/providers/uniswap/addresses.ts

@@ -12,6 +12,7 @@ import {
 
 import { PERMIT2_ADDRESS } from '@/constants/contracts.js'
 import type { SupportedChainId } from '@/constants/supportedChains.js'
+import { ChainNotSupportedError } from '@/core/error/errors.js'
 
 /**
  * Uniswap contract addresses
@@ -97,7 +98,10 @@ export function getUniswapAddresses(
 ): UniswapAddresses {
   const addresses = UNISWAP_ADDRESSES[chainId]
   if (!addresses) {
-    throw new Error(`Uniswap not supported on chain ${chainId}`)
+    throw new ChainNotSupportedError({
+      chainId,
+      supportedChainIds: Object.keys(UNISWAP_ADDRESSES).map(Number),
+    })
   }
   return addresses
 }

package/src/{swap → actions/swap}/providers/uniswap/encoding.ts

@@ -7,7 +7,6 @@ import {
   zeroAddress,
 } from 'viem'
 
-import type { SupportedChainId } from '@/constants/supportedChains.js'
 import {
   CURRENCY_AMOUNT_PARAMS,
   EXACT_INPUT_SINGLE_PARAMS,
@@ -16,7 +15,8 @@ import {
   POOL_KEY_ABI_TYPE,
   QUOTER_ABI,
   UNIVERSAL_ROUTER_ABI,
-} from '@/swap/providers/uniswap/abis.js'
+} from '@/actions/swap/providers/uniswap/abis.js'
+import type { SupportedChainId } from '@/constants/supportedChains.js'
 import type { Asset } from '@/types/asset.js'
 import type { SwapPrice, SwapRoute } from '@/types/swap/index.js'
 import { getAssetAddress, isNativeAsset } from '@/utils/assets.js'
@@ -213,7 +213,7 @@ const V4_SWAP = 0x10
 
 // V4 action types
 const SWAP_EXACT_IN_SINGLE = 0x06
-const SWAP_EXACT_OUT_SINGLE = 0x07
+const SWAP_EXACT_OUT_SINGLE = 0x08
 const SETTLE_ALL = 0x0c
 const TAKE_ALL = 0x0f
 

package/src/{swap → actions/swap}/providers/uniswap/markets.ts

@@ -1,9 +1,9 @@
 import { type Address, encodeAbiParameters, keccak256 } from 'viem'
 
+import { assetPairs } from '@/actions/swap/core/markets.js'
+import { POOL_KEY_ABI_TYPE } from '@/actions/swap/providers/uniswap/abis.js'
 import { UNISWAP } from '@/constants/providers.js'
 import type { SupportedChainId } from '@/constants/supportedChains.js'
-import { assetPairs } from '@/swap/core/markets.js'
-import { POOL_KEY_ABI_TYPE } from '@/swap/providers/uniswap/abis.js'
 import type { Asset } from '@/types/asset.js'
 import type { SwapMarket } from '@/types/swap/index.js'
 

package/src/{swap → actions/swap}/providers/velodrome/VelodromeSwapProvider.ts

@@ -1,28 +1,33 @@
 import { formatUnits } from 'viem'
 
-import { VELODROME } from '@/constants/providers.js'
-import type { SupportedChainId } from '@/constants/supportedChains.js'
-import type { SwapQuoteParamsResolved } from '@/ens/types.js'
-import { expandMarkets, findMarket } from '@/swap/core/markets.js'
-import { SwapProvider } from '@/swap/core/SwapProvider.js'
+import { expandMarkets, findMarket } from '@/actions/swap/core/markets.js'
+import { SwapProvider } from '@/actions/swap/core/SwapProvider.js'
 import {
   getChainConfig,
   getSupportedChainIds,
   getValidMarketConfigs,
-} from '@/swap/providers/velodrome/config.js'
+} from '@/actions/swap/providers/velodrome/config.js'
 import {
-  buildTokenApproval,
   encodePoolSwap,
   fetchPoolQuote,
-} from '@/swap/providers/velodrome/encoding/index.js'
+} from '@/actions/swap/providers/velodrome/encoding/index.js'
 import {
   configToMarkets,
   resolvePoolConfig,
-} from '@/swap/providers/velodrome/markets.js'
+} from '@/actions/swap/providers/velodrome/markets.js'
 import type {
   VelodromeMarketConfig,
   VelodromeSwapProviderConfig,
-} from '@/swap/providers/velodrome/types.js'
+} from '@/actions/swap/providers/velodrome/types.js'
+import { VELODROME } from '@/constants/providers.js'
+import type { SupportedChainId } from '@/constants/supportedChains.js'
+import {
+  ExactOutputNotSupportedError,
+  MarketNotAllowedError,
+} from '@/core/error/errors.js'
+import type { ChainManager } from '@/services/ChainManager.js'
+import type { SwapQuoteParamsResolved } from '@/services/nameservices/ens/types.js'
+import type { SwapSettings } from '@/types/actions.js'
 import type { Asset } from '@/types/asset.js'
 import type {
   GetSwapMarketParams,
@@ -32,6 +37,12 @@ import type {
   SwapQuote,
   SwapTransaction,
 } from '@/types/swap/index.js'
+import {
+  buildErc20ApprovalTx,
+  checkTokenAllowance,
+  resolveApprovalMode,
+  resolveErc20ApprovalAmount,
+} from '@/utils/approve.js'
 import { getAssetAddress, isNativeAsset } from '@/utils/assets.js'
 
 /**
@@ -40,6 +51,14 @@ import { getAssetAddress, isNativeAsset } from '@/utils/assets.js'
  * v2 routers (Optimism, Base), leaf routers (Relay chains), and Universal Router (Base Sepolia).
  */
 export class VelodromeSwapProvider extends SwapProvider<VelodromeSwapProviderConfig> {
+  constructor(
+    config: VelodromeSwapProviderConfig,
+    chainManager: ChainManager,
+    settings?: SwapSettings,
+  ) {
+    super(config, chainManager, settings)
+  }
+
   /** @returns Chain IDs where Velodrome/Aerodrome contracts are deployed */
   protocolSupportedChainIds(): SupportedChainId[] {
     return getSupportedChainIds()
@@ -55,9 +74,7 @@ export class VelodromeSwapProvider extends SwapProvider<VelodromeSwapProviderCon
     params: ResolvedSwapParams,
   ): Promise<SwapTransaction> {
     if (params.amountOutRaw !== undefined) {
-      throw new Error(
-        'Velodrome/Aerodrome does not support exact-output swaps. Provide amountIn instead of amountOut.',
-      )
+      throw new ExactOutputNotSupportedError('Velodrome/Aerodrome')
     }
 
     const swapQuote = await this._getQuote({
@@ -73,7 +90,10 @@ export class VelodromeSwapProvider extends SwapProvider<VelodromeSwapProviderCon
       deadline: params.deadline,
       recipient: params.recipient,
     })
-    return this.buildSwapTransactions(swapQuote)
+    return this.buildSwapTransactions({
+      ...swapQuote,
+      approvalMode: params.approvalMode,
+    })
   }
 
   /**
@@ -119,9 +139,7 @@ export class VelodromeSwapProvider extends SwapProvider<VelodromeSwapProviderCon
     const { chainId, assetIn, assetOut } = params
 
     if (params.amountOut !== undefined) {
-      throw new Error(
-        'Velodrome/Aerodrome does not support exact-output swaps. Provide amountIn instead of amountOut.',
-      )
+      throw new ExactOutputNotSupportedError('Velodrome/Aerodrome')
     }
 
     const chain = getChainConfig(chainId)
@@ -182,25 +200,44 @@ export class VelodromeSwapProvider extends SwapProvider<VelodromeSwapProviderCon
       quotedAt: now,
       expiresAt: deadline,
       gasEstimate: internalQuote.gasEstimate,
-      quotedRecipient: recipient,
+      recipient,
     }
   }
 
   protected async _buildApprovals(quote: SwapQuote) {
+    if (isNativeAsset(quote.assetIn)) {
+      return { tokenApproval: undefined }
+    }
+
     const chain = getChainConfig(quote.chainId)
     const publicClient = this.chainManager.getPublicClient(quote.chainId)
+    const token = getAssetAddress(quote.assetIn, quote.chainId)
+    const spender = chain.contracts.router
+    const required = quote.amountInRaw
 
-    const tokenApproval = isNativeAsset(quote.assetIn)
-      ? undefined
-      : await buildTokenApproval(
-          getAssetAddress(quote.assetIn, quote.chainId),
-          chain.contracts.router,
-          chain.metadata.routerType,
-          quote.amountInRaw,
-          quote.recipient!,
-          publicClient,
-        )
+    const allowance = await checkTokenAllowance({
+      publicClient,
+      token,
+      owner: quote.recipient,
+      spender,
+    })
+
+    if (allowance >= required) {
+      return { tokenApproval: undefined }
+    }
 
+    const tokenApproval = buildErc20ApprovalTx(
+      token,
+      spender,
+      resolveErc20ApprovalAmount(
+        resolveApprovalMode(
+          quote.approvalMode,
+          this._config.approvalMode,
+          this._settings.approvalMode,
+        ),
+        required,
+      ),
+    )
     return { tokenApproval }
   }
 
@@ -217,9 +254,12 @@ export class VelodromeSwapProvider extends SwapProvider<VelodromeSwapProviderCon
       | VelodromeMarketConfig
       | undefined
     if (!config) {
-      throw new Error(
-        `No market config for pair ${assetIn.metadata.symbol}/${assetOut.metadata.symbol}`,
-      )
+      throw new MarketNotAllowedError({
+        assetInSymbol: assetIn.metadata.symbol,
+        assetOutSymbol: assetOut.metadata.symbol,
+        chainId,
+        reason: 'No market config for this pair',
+      })
     }
     return resolvePoolConfig(config)
   }

package/src/{swap → actions/swap}/providers/velodrome/__tests__/VelodromeSwapProvider.network.test.ts

@@ -14,10 +14,10 @@ import { createPublicClient, http } from 'viem'
 import { base, optimism } from 'viem/chains'
 import { afterAll, beforeAll, describe, expect, it } from 'vitest'
 
+import type { VelodromeSwapProviderConfig } from '@/actions/swap/providers/velodrome/types.js'
+import { VelodromeSwapProvider } from '@/actions/swap/providers/velodrome/VelodromeSwapProvider.js'
 import type { SupportedChainId } from '@/constants/supportedChains.js'
 import type { ChainManager } from '@/services/ChainManager.js'
-import type { VelodromeSwapProviderConfig } from '@/swap/providers/velodrome/types.js'
-import { VelodromeSwapProvider } from '@/swap/providers/velodrome/VelodromeSwapProvider.js'
 import type { Asset } from '@/types/asset.js'
 
 // ── Real mainnet assets ──

package/src/{swap → actions/swap}/providers/velodrome/__tests__/VelodromeSwapProvider.routing.test.ts

@@ -9,10 +9,10 @@ import {
   MockUSDCAsset as USDC,
   MockWETHAsset as WETH,
 } from '@/__mocks__/MockAssets.js'
+import type { VelodromeSwapProviderConfig } from '@/actions/swap/providers/velodrome/types.js'
+import { VelodromeSwapProvider } from '@/actions/swap/providers/velodrome/VelodromeSwapProvider.js'
 import type { SupportedChainId } from '@/constants/supportedChains.js'
 import type { ChainManager } from '@/services/ChainManager.js'
-import type { VelodromeSwapProviderConfig } from '@/swap/providers/velodrome/types.js'
-import { VelodromeSwapProvider } from '@/swap/providers/velodrome/VelodromeSwapProvider.js'
 
 const OP_CHAIN_ID = optimism.id as SupportedChainId
 const BASE_CHAIN_ID = base.id as SupportedChainId
@@ -91,7 +91,7 @@ describe('VelodromeSwapProvider router type routing', () => {
   })
 
   describe('universal router (Base Sepolia)', () => {
-    it('executes swap via Universal Router', async () => {
+    it('executes swap via Universal Router with standard approve', async () => {
       const provider = createProvider(BASE_SEPOLIA_CHAIN_ID)
       const result = await provider.execute({
         amountIn: 100,
@@ -102,12 +102,13 @@ describe('VelodromeSwapProvider router type routing', () => {
       })
 
       expect(result.transactionData.swap).toBeDefined()
-      // Universal Router: uses ERC20.transfer (not approve)
+      // Universal Router pulls tokens via transferFrom (payerIsUser=true), so the
+      // approval is a standard ERC20 approve(router, amount) — NOT a transfer.
+      // See encoding.v2.test.ts and the regression test in VelodromeSwapProvider.test.ts.
       expect(result.transactionData.tokenApproval).toBeDefined()
-      // tokenApproval should be a transfer() call, not approve()
       const approvalData = result.transactionData.tokenApproval!.data
-      // transfer() selector = 0xa9059cbb
-      expect(approvalData.startsWith('0xa9059cbb')).toBe(true)
+      // approve(spender, amount) selector = 0x095ea7b3
+      expect(approvalData.startsWith('0x095ea7b3')).toBe(true)
     })
 
     it('quotes via pool.getAmountOut for Universal Router', async () => {

package/src/{swap → actions/swap}/providers/velodrome/__tests__/VelodromeSwapProvider.test.ts

@@ -1,4 +1,5 @@
 import type { Address, PublicClient } from 'viem'
+import { decodeFunctionData, erc20Abi, maxUint256 } from 'viem'
 import { mode, optimism } from 'viem/chains'
 import { describe, expect, it, vi } from 'vitest'
 
@@ -8,15 +9,17 @@ import {
   MockUSDCAsset as USDC,
   MockWETHAsset as WETH,
 } from '@/__mocks__/MockAssets.js'
+import type { VelodromeSwapProviderConfig } from '@/actions/swap/providers/velodrome/types.js'
+import { VelodromeSwapProvider } from '@/actions/swap/providers/velodrome/VelodromeSwapProvider.js'
 import type { SupportedChainId } from '@/constants/supportedChains.js'
 import type { ChainManager } from '@/services/ChainManager.js'
-import type { VelodromeSwapProviderConfig } from '@/swap/providers/velodrome/types.js'
-import { VelodromeSwapProvider } from '@/swap/providers/velodrome/VelodromeSwapProvider.js'
 import type { Asset } from '@/types/asset.js'
 
 const CHAIN_ID = optimism.id as SupportedChainId
 
-function createMockChainManager(): ChainManager {
+function createMockChainManager(
+  supportedChains: SupportedChainId[] = [CHAIN_ID],
+): ChainManager {
   const mockPublicClient = {
     readContract: vi
       .fn()
@@ -35,7 +38,7 @@ function createMockChainManager(): ChainManager {
   return {
     getPublicClient: vi.fn().mockReturnValue(mockPublicClient),
     tryGetPublicClient: vi.fn().mockReturnValue(undefined),
-    getSupportedChains: vi.fn().mockReturnValue([CHAIN_ID]),
+    getSupportedChains: vi.fn().mockReturnValue(supportedChains),
   } as unknown as ChainManager
 }
 
@@ -95,6 +98,44 @@ describe('VelodromeSwapProvider', () => {
       expect(result.transactionData.permit2Approval).toBeUndefined()
     })
 
+    it('approves the exact swap amount in default exact mode', async () => {
+      const provider = createProvider()
+      const result = await provider.execute({
+        amountIn: 100,
+        assetIn: USDC,
+        assetOut: OP,
+        chainId: CHAIN_ID,
+        walletAddress: MOCK_WALLET,
+      })
+
+      const decoded = decodeFunctionData({
+        abi: erc20Abi,
+        data: result.transactionData.tokenApproval!.data,
+      })
+      expect(decoded.functionName).toBe('approve')
+      // 100 USDC at 6 decimals
+      expect(decoded.args[1]).toBe(100_000_000n)
+    })
+
+    it('approves maxUint256 when approvalMode is "max"', async () => {
+      const provider = createProvider()
+      const result = await provider.execute({
+        amountIn: 100,
+        assetIn: USDC,
+        assetOut: OP,
+        chainId: CHAIN_ID,
+        walletAddress: MOCK_WALLET,
+        approvalMode: 'max',
+      })
+
+      const decoded = decodeFunctionData({
+        abi: erc20Abi,
+        data: result.transactionData.tokenApproval!.data,
+      })
+      expect(decoded.functionName).toBe('approve')
+      expect(decoded.args[1]).toBe(maxUint256)
+    })
+
     it('throws for exact-output swaps', async () => {
       const provider = createProvider()
 
@@ -477,7 +518,7 @@ describe('VelodromeSwapProvider', () => {
           amountIn: 100,
           chainId: MODE_CHAIN_ID,
         }),
-      ).rejects.toThrow('CL pools not supported on chain')
+      ).rejects.toThrow('is not supported')
     })
   })
 

package/src/{swap → actions/swap}/providers/velodrome/__tests__/encoding.cl.test.ts

@@ -1,11 +1,14 @@
+import type { Hex } from 'viem'
+import { decodeAbiParameters } from 'viem'
 import { describe, expect, it } from 'vitest'
 
 import { MockUSDCAsset, MockWETHAsset } from '@/__mocks__/MockAssets.js'
-import { UNIVERSAL_ROUTER_ABI } from '@/swap/providers/velodrome/abis.js'
+import { UNIVERSAL_ROUTER_ABI } from '@/actions/swap/providers/velodrome/abis.js'
 import {
   encodeCLSwap,
   encodeSwap,
-} from '@/swap/providers/velodrome/encoding/index.js'
+} from '@/actions/swap/providers/velodrome/encoding/index.js'
+import { V3_SWAP_EXACT_IN_INPUT_PARAMS } from '@/actions/swap/providers/velodrome/encoding/routers/cl.js'
 
 import {
   BASE_CHAIN_ID,
@@ -39,6 +42,32 @@ describe('encodeCLSwap', () => {
     expect(deadline).toBe(BigInt(DEADLINE))
   })
 
+  // Regression for #438: payerIsUser must be true so the router pulls tokens via
+  // transferFrom against an ERC20 allowance. See encoding.v2.test.ts for context.
+  it('encodes V3_SWAP_EXACT_IN with payerIsUser = true', () => {
+    const data = encodeCLSwap({
+      assetIn: MockUSDCAsset,
+      assetOut: MockWETHAsset,
+      amountInRaw: 1000000n,
+      amountOutMin: 400000000000000000n,
+      tickSpacing: 100,
+      recipient: RECIPIENT,
+      deadline: DEADLINE,
+      chainId: BASE_CHAIN_ID,
+    })
+
+    const { args } = decode<[Hex, Hex[], bigint]>(UNIVERSAL_ROUTER_ABI, data)
+    const [, inputs] = args
+    const decoded = decodeAbiParameters(
+      V3_SWAP_EXACT_IN_INPUT_PARAMS,
+      inputs[0],
+    )
+    const payerIsUserIdx = V3_SWAP_EXACT_IN_INPUT_PARAMS.findIndex(
+      (p) => p.name === 'payerIsUser',
+    )
+    expect(decoded[payerIsUserIdx]).toBe(true)
+  })
+
   it('produces different calldata than V2 universal router swap', () => {
     const clData = encodeCLSwap({
       assetIn: MockUSDCAsset,