feat(ur-sdk): migrating out of range -> in range or out of range -> opposite side out of range

sdks/universal-router-sdk/src/swapRouter.ts

@@ -36,8 +36,8 @@ export interface MigrateV3ToV4Options {
   v4AddLiquidityOptions: V4AddLiquidityOptions
 }
 
-function isMint(options: V4AddLiquidityOptions): options is MintOptions {
-  return Object.keys(options).some((k) => k === 'recipient')
+function isMigrate(options: V4AddLiquidityOptions): options is MintOptions {
+  return 'migrateOptions' in options && options.migrateOptions?.migrate === true
 }
 
 export abstract class SwapRouter {
@@ -87,6 +87,10 @@ export abstract class SwapRouter {
     const v4PositionManagerAddress =
       positionManagerOverride ?? CHAIN_TO_ADDRESSES_MAP[v4Pool.chainId as SupportedChainsType].v4PositionManagerAddress
 
+    const universalRouterAddress = UNIVERSAL_ROUTER_ADDRESS(
+      UniversalRouterVersion.V2_0,
+      options.inputPosition.pool.chainId as SupportedChainsType
+    )
     // owner of the v3 nft must be the receiver of the v4 nft
 
     // validate the parameters
@@ -110,8 +114,8 @@ export abstract class SwapRouter {
       options.v3RemoveLiquidityOptions.collectOptions.recipient === v4PositionManagerAddress,
       'RECIPIENT_NOT_POSITION_MANAGER'
     )
-    invariant(isMint(options.v4AddLiquidityOptions), 'MINT_REQUIRED')
-    invariant(options.v4AddLiquidityOptions.migrate, 'MIGRATE_REQUIRED')
+    // Migration must be a mint operation, not an increase because the UR should not have permission to increase liquidity on a v4 position
+    invariant(isMigrate(options.v4AddLiquidityOptions), 'MIGRATE_REQUIRED')
 
     const planner = new RoutePlanner()
 
@@ -132,10 +136,6 @@ export abstract class SwapRouter {
     // add position permit to the universal router planner
     if (options.v3RemoveLiquidityOptions.permit) {
       // permit spender should be UR
-      const universalRouterAddress = UNIVERSAL_ROUTER_ADDRESS(
-        UniversalRouterVersion.V2_0,
-        options.inputPosition.pool.chainId as SupportedChainsType
-      )
       invariant(universalRouterAddress == options.v3RemoveLiquidityOptions.permit.spender, 'INVALID_SPENDER')
       // don't need to transfer it because v3posm uses isApprovedOrOwner()
       encodeV3PositionPermit(planner, options.v3RemoveLiquidityOptions.permit, options.v3RemoveLiquidityOptions.tokenId)
@@ -162,6 +162,28 @@ export abstract class SwapRouter {
       planner.addCommand(CommandType.V3_POSITION_MANAGER_CALL, [v3Call])
     }
 
+    // if migrate options has a currency, require a batch permit
+    if (options.v4AddLiquidityOptions.migrateOptions?.neededCurrency) {
+      // need to permit the UR to spend your currency on permit2
+      if (options.v4AddLiquidityOptions.batchPermit) {
+        invariant(
+          options.v4AddLiquidityOptions.batchPermit.permitBatch.spender == universalRouterAddress,
+          'INVALID_SPENDER'
+        )
+        planner.addCommand(CommandType.PERMIT2_PERMIT_BATCH, [
+          options.v4AddLiquidityOptions.batchPermit.permitBatch,
+          options.v4AddLiquidityOptions.batchPermit.signature,
+        ])
+      }
+      planner.addCommand(CommandType.PERMIT2_TRANSFER_FROM, [
+        options.v4AddLiquidityOptions.migrateOptions.neededCurrency,
+        options.v3RemoveLiquidityOptions.collectOptions.recipient,
+        options.v4AddLiquidityOptions.migrateOptions.neededAmount,
+      ])
+      // remove batchPermit so it doesn't get encoded again later
+      delete options.v4AddLiquidityOptions.batchPermit
+    }
+
     // encode v4 mint
     const v4AddParams = V4PositionManager.addCallParameters(options.outputPosition, options.v4AddLiquidityOptions)
     // only modifyLiquidities can be called by the UniversalRouter