feat: removeLiquidity

Author: luchobonatti

src/constants/common.ts

@@ -0,0 +1,2 @@
+export const DEFAULT_DEADLINE = 60 * 10 // 10 minutes
+export const DEFAULT_SLIPPAGE_TOLERANCE = 50

src/core/uniDevKitV4.ts

@@ -17,6 +17,10 @@ import type {
   PreparePermit2DataResult,
 } from '@/types/utils/permit2'
 import { buildAddLiquidityCallData } from '@/utils/buildAddLiquidityCallData'
+import {
+  type BuildRemoveLiquidityCallDataParams,
+  buildRemoveLiquidityCallData,
+} from '@/utils/buildRemoveLiquidityCallData'
 import { buildSwapCallData } from '@/utils/buildSwapCallData'
 import { getPool } from '@/utils/getPool'
 import { getPoolKeyFromPoolId } from '@/utils/getPoolKeyFromPoolId'
@@ -234,4 +238,14 @@ export class UniDevKitV4 {
   async preparePermit2Data(params: PreparePermit2DataParams): Promise<PreparePermit2DataResult> {
     return preparePermit2Data(params, this.instance)
   }
+
+  /**
+   * Builds a remove liquidity call data for a given remove liquidity parameters.
+   * @param params @type {BuildRemoveLiquidityCallDataParams}
+   * @returns Promise resolving to remove liquidity call data including calldata and value
+   * @throws Error if SDK instance is not found or if remove liquidity call data is invalid
+   */
+  async buildRemoveLiquidityCallData(params: BuildRemoveLiquidityCallDataParams) {
+    return buildRemoveLiquidityCallData(params, this.instance)
+  }
 }

src/utils/buildAddLiquidityCallData.ts

@@ -1,15 +1,14 @@
+import { DEFAULT_SLIPPAGE_TOLERANCE } from '@/constants/common'
+import { percentFromBips } from '@/helpers/percent'
 import type { UniDevKitV4Instance } from '@/types'
 import type {
   BuildAddLiquidityCallDataResult,
   BuildAddLiquidityParams,
 } from '@/types/utils/buildAddLiquidityCallData'
-import { Percent } from '@uniswap/sdk-core'
+import { getDefaultDeadline } from '@/utils/getDefaultDeadline'
 import { TickMath, encodeSqrtRatioX96, nearestUsableTick } from '@uniswap/v3-sdk'
 import { Position, V4PositionManager } from '@uniswap/v4-sdk'
 
-const DEFAULT_DEADLINE = 1800n // 30 minutes
-const DEFAULT_SLIPPAGE_TOLERANCE = 50
-
 /**
  * Builds the calldata and native value required to add liquidity to a Uniswap V4 pool.
  *
@@ -80,14 +79,10 @@ export async function buildAddLiquidityCallData(
     permit2BatchSignature,
   } = params
 
-  console.log('params', params)
-
   try {
-    const deadline =
-      deadlineParam ??
-      (await instance.client.getBlock().then((b) => b.timestamp + DEFAULT_DEADLINE)).toString()
+    const deadline = deadlineParam ?? (await getDefaultDeadline(instance)).toString()
 
-    const slippagePercent = new Percent(slippageTolerance, 10_000)
+    const slippagePercent = percentFromBips(slippageTolerance)
     const createPool = pool.liquidity.toString() === '0'
 
     const tickLower = tickLowerParam ?? nearestUsableTick(TickMath.MIN_TICK, pool.tickSpacing)

src/utils/buildRemoveLiquidityCallData.ts

@@ -1,34 +1,33 @@
-import { Percent } from '@uniswap/sdk-core'
-import { type Position, V4PositionManager } from '@uniswap/v4-sdk'
+import { DEFAULT_SLIPPAGE_TOLERANCE } from '@/constants/common'
+import { percentFromBips } from '@/helpers/percent'
+import type { UniDevKitV4Instance } from '@/types'
+import { getDefaultDeadline } from '@/utils/getDefaultDeadline'
+import { getPosition } from '@/utils/getPosition'
+import { V4PositionManager } from '@uniswap/v4-sdk'
 
 /**
  * Parameters required to build the calldata for removing liquidity from a Uniswap v4 position.
  */
 export interface BuildRemoveLiquidityCallDataParams {
-  /**
-   * The position object representing the liquidity position to modify.
-   */
-  position: Position
-
   /**
    * The percentage of liquidity to remove from the position.
    */
   liquidityPercentage: number
 
   /**
-   * The deadline for the transaction.
+   * The tokenId of the position to remove liquidity from.
    */
-  deadline: string
+  tokenId: string
 
   /**
    * The slippage tolerance for the transaction.
    */
-  slippageTolerance: number
+  slippageTolerance?: number
 
   /**
-   * The tokenId of the position to remove liquidity from.
+   * The deadline for the transaction. (default: 5 minutes from now)
    */
-  tokenId: string
+  deadline?: string
 }
 
 /**
@@ -41,7 +40,7 @@ export interface BuildRemoveLiquidityCallDataParams {
  * ```typescript
  * const { calldata, value } = buildRemoveLiquidityCallData({
  *   position,
- *   liquidityPercentage: new Percent(1, 1), // 100%
+ *   liquidityPercentage: 10_000, // 100%
  * });
  *
  * const tx = await sendTransaction({
@@ -51,18 +50,29 @@ export interface BuildRemoveLiquidityCallDataParams {
  * });
  * ```
  */
-export function buildRemoveLiquidityCallData({
-  position,
-  liquidityPercentage,
-  deadline,
-  slippageTolerance,
-  tokenId,
-}: BuildRemoveLiquidityCallDataParams) {
+export async function buildRemoveLiquidityCallData(
+  {
+    liquidityPercentage,
+    deadline: deadlineParam,
+    slippageTolerance,
+    tokenId,
+  }: BuildRemoveLiquidityCallDataParams,
+  instance: UniDevKitV4Instance,
+) {
+  // Get position data
+  const positionData = await getPosition({ tokenId }, instance)
+  if (!positionData) {
+    throw new Error('Position not found')
+  }
+
+  const deadline = deadlineParam ?? (await getDefaultDeadline(instance)).toString()
+
+  // Build remove liquidity call data
   try {
-    const { calldata, value } = V4PositionManager.removeCallParameters(position, {
-      slippageTolerance: new Percent(slippageTolerance, 100),
+    const { calldata, value } = V4PositionManager.removeCallParameters(positionData.position, {
+      slippageTolerance: percentFromBips(slippageTolerance ?? DEFAULT_SLIPPAGE_TOLERANCE),
       deadline: deadline,
-      liquidityPercentage: new Percent(liquidityPercentage, 100),
+      liquidityPercentage: percentFromBips(liquidityPercentage),
       tokenId: tokenId,
     })
 

src/utils/getDefaultDeadline.ts

@@ -0,0 +1,9 @@
+import { DEFAULT_DEADLINE } from '@/constants/common'
+import type { UniDevKitV4Instance } from '@/types'
+
+export async function getDefaultDeadline(
+  instance: UniDevKitV4Instance,
+  timeFromNow: number = DEFAULT_DEADLINE,
+): Promise<bigint> {
+  return (await instance.client.getBlock()).timestamp + BigInt(timeFromNow)
+}