@tradeport/sui-trading-sdk 0.1.12 → 0.1.13

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # @tradeport/sui-trading-sdk
 
+## 0.1.13
+
+### Patch Changes
+
+- 7157dff: Fixed exerciseShortLocks
+
 ## 0.1.12
 
 ### Patch Changes

package/dist/index.js

@@ -1827,9 +1827,7 @@ async function exerciseLongLocks({ walletAddress, transaction, locks }, context)
             query: fetchBidsById,
             variables: { bidIds: [bidId] }
           });
-          bid = res.bids.find(
-            (bid2) => bid2.status === "active"
-          );
+          bid = res.bids.find((bid2) => bid2.status === "active");
           if (!bid) {
             throw new Error(`Active bid ${bidId} is missing for ${lock.nft_type}`);
           }
@@ -2794,52 +2792,53 @@ async function exerciseShortLocks({ walletAddress, transaction, locks }, context
     }
     const transferPolicyId = transferPolicy?.id;
     const sharedObjects = await getSharedObjects(lock.nft_type);
+    if (lock.nft?.token_id) {
+      throw new Error(`The short lock ${lockId} must not have any nft`);
+    }
+    let requiredCoins = BigInt(lock.maker_price) + calculateRoyaltyFee(transferPolicy.rules, 0n);
+    let listing = null;
+    if (listingId) {
+      const res = await gqlChainRequest({
+        chain: "sui",
+        query: fetchListingsById,
+        variables: { listingIds: [listingId] }
+      });
+      listing = res.listings[0];
+      if (!listing) {
+        throw new Error(`Missing listing ${listingId}`);
+      }
+    }
+    if (listing) {
+      if (nftId && listing.nft.id !== nftId) {
+        throw new Error(`The listing ${listingId} must be for token ${listing.nft.token_id}`);
+      }
+      if (!nftId) {
+        nftId = listing.nft.id;
+      }
+      if (listing.nonce && isOriginByteTx(sharedObjects) || !listing.nft?.chain_state?.kiosk_id) {
+        throw new Error(`The listing ${listingId} must be for native kiosk`);
+      }
+      requiredCoins += calculateRoyaltyFee(transferPolicy.rules, BigInt(listing.price));
+    }
+    if (!nftId) {
+      throw new Error(`nftId is required for shorts`);
+    }
+    const nft = (await gqlChainRequest({
+      chain: "sui",
+      query: fetchNftById,
+      variables: { nftId }
+    }))?.nfts?.[0];
+    if (!nft) {
+      throw new Error(`No nft ${nftId} found`);
+    }
+    if (!listing && nft.owner !== walletAddress) {
+      throw new Error(`The nft ${nftId} must be owned by ${walletAddress}`);
+    }
     await kioskTxWrapper({
       tx,
       kioskOwner: walletAddress,
+      kiosk: nft?.chain_state?.kiosk_id,
       async runCommands(kioskTx) {
-        if (lock.nft?.token_id) {
-          throw new Error(`The short lock ${lockId} must not have any nft`);
-        }
-        let requiredCoins = BigInt(lock.maker_price) + calculateRoyaltyFee(transferPolicy.rules, 0n);
-        let listing = null;
-        if (listingId) {
-          const res = await gqlChainRequest({
-            chain: "sui",
-            query: fetchListingsById,
-            variables: { listingIds: [listingId] }
-          });
-          listing = res.listings[0];
-          if (!listing) {
-            throw new Error(`Missing listing ${listingId}`);
-          }
-        }
-        if (listing) {
-          if (nftId && listing.nft.id !== nftId) {
-            throw new Error(`The listing ${listingId} must be for token ${listing.nft.token_id}`);
-          }
-          if (!nftId) {
-            nftId = listing.nft.id;
-          }
-          if (listing.nonce && isOriginByteTx(sharedObjects) || !listing.nft?.chain_state?.kiosk_id) {
-            throw new Error(`The listing ${listingId} must be for native kiosk`);
-          }
-          requiredCoins += calculateRoyaltyFee(transferPolicy.rules, BigInt(listing.price));
-        }
-        if (!nftId) {
-          throw new Error(`nftId is required for shorts`);
-        }
-        const nft = (await gqlChainRequest({
-          chain: "sui",
-          query: fetchNftById,
-          variables: { nftId }
-        }))?.nfts?.[0];
-        if (!nft) {
-          throw new Error(`No nft ${nftId} found`);
-        }
-        if (!listing && nft.owner !== walletAddress) {
-          throw new Error(`The nft ${nftId} must be owned by ${walletAddress}`);
-        }
         const [request] = tx.moveCall({
           target: `${TRADEPORT_PRICE_LOCK_PACKAGE}::tradeport_price_lock::start_confirm_short_lock`,
           typeArguments: [lock.nft_type],
@@ -2882,6 +2881,7 @@ async function exerciseShortLocks({ walletAddress, transaction, locks }, context
           nftType: lock.nft_type,
           price: "0",
           transferRequest,
+          kioskToLockRuleProve: TRADEPORT_KIOSK_TRANSFERS_ESCROW_KIOSK,
           shouldSkipKioskLocking: true
         });
       }