@tradeport/sui-trading-sdk 0.4.62 → 0.4.64

package/package.json

@@ -1,62 +1,60 @@
 {
-	"name": "@tradeport/sui-trading-sdk",
-	"license": "MIT",
-	"version": "0.4.62",
-	"main": "dist/index.js",
-	"module": "dist/index.mjs",
-	"types": "dist/index.d.ts",
-	"scripts": {
-		"build": "tsup src/index.ts --sourcemap --format cjs,esm --dts",
-		"release": "pnpm run build && changeset publish",
-		"lint": "tsc && eslint --fix src/* --ignore-path .gitignore --ext .js,.ts",
-		"format": "prettier --write \"**/*.+(js|ts|json)\"",
-		"test": "jest",
-		"prepare": "husky install",
-		"pre-commit": "lint-staged"
-	},
-	"devDependencies": {
-		"@babel/core": "^7.23.3",
-		"@babel/preset-env": "^7.23.3",
-		"@changesets/cli": "^2.26.2",
-		"@types/jest": "^29.5.10",
-		"@types/ms": "^2.1.0",
-		"@types/node": "^20.10.0",
-		"@typescript-eslint/eslint-plugin": "^6.13.1",
-		"@typescript-eslint/parser": "^6.13.1",
-		"babel-jest": "^29.7.0",
-		"dotenv": "^16.3.1",
-		"eslint": "^8.54.0",
-		"eslint-config-prettier": "^9.0.0",
-		"eslint-config-xo": "^0.43.1",
-		"eslint-config-xo-typescript": "^1.0.1",
-		"husky": "^8.0.0",
-		"jest": "^29.7.0",
-		"lint-staged": "^16.2.7",
-		"prettier": "^3.1.0",
-		"ts-jest": "^29.1.1",
-		"tsup": "^8.0.1",
-		"typescript": "^5.3.2"
-	},
-	"dependencies": {
-		"@flowx-finance/sdk": "^1.15.0",
-		"@mysten/kiosk": "^0.7.13",
-		"@mysten/sui": "^1.17.0",
-		"@types/analytics-node": "^3.1.14",
-		"analytics-node": "^6.2.0",
-		"bignumber.js": "^9.3.1",
-		"graphql": "^16.12.0",
-		"graphql-request": "^6.1.0",
-		"ms": "^2.1.3"
-	},
-	"publishConfig": {
-		"access": "public",
-		"registry": "https://registry.npmjs.org/"
-	},
-	"lint-staged": {
-		"*.ts": [
-			"eslint --fix",
-			"bash -c 'tsc --noEmit'",
-			"prettier --write"
-		]
-	}
-}
+  "name": "@tradeport/sui-trading-sdk",
+  "license": "MIT",
+  "version": "0.4.64",
+  "main": "dist/index.js",
+  "module": "dist/index.mjs",
+  "types": "dist/index.d.ts",
+  "devDependencies": {
+    "@babel/core": "^7.23.3",
+    "@babel/preset-env": "^7.23.3",
+    "@changesets/cli": "^2.26.2",
+    "@types/jest": "^29.5.10",
+    "@types/ms": "^2.1.0",
+    "@types/node": "^20.10.0",
+    "@typescript-eslint/eslint-plugin": "^6.13.1",
+    "@typescript-eslint/parser": "^6.13.1",
+    "babel-jest": "^29.7.0",
+    "dotenv": "^16.3.1",
+    "eslint": "^8.54.0",
+    "eslint-config-prettier": "^9.0.0",
+    "eslint-config-xo": "^0.43.1",
+    "eslint-config-xo-typescript": "^1.0.1",
+    "husky": "^8.0.0",
+    "jest": "^29.7.0",
+    "lint-staged": "^16.2.7",
+    "prettier": "^3.1.0",
+    "ts-jest": "^29.1.1",
+    "tsup": "^8.0.1",
+    "typescript": "^5.3.2"
+  },
+  "dependencies": {
+    "@mysten/kiosk": "^0.7.13",
+    "@mysten/sui": "^1.17.0",
+    "@types/analytics-node": "^3.1.14",
+    "analytics-node": "^6.2.0",
+    "bignumber.js": "^9.3.1",
+    "graphql": "^16.12.0",
+    "graphql-request": "^6.1.0",
+    "ms": "^2.1.3"
+  },
+  "publishConfig": {
+    "access": "public",
+    "registry": "https://registry.npmjs.org/"
+  },
+  "lint-staged": {
+    "*.ts": [
+      "eslint --fix",
+      "bash -c 'tsc --noEmit'",
+      "prettier --write"
+    ]
+  },
+  "scripts": {
+    "build": "tsup src/index.ts --sourcemap --format cjs,esm --dts",
+    "release": "pnpm run build && changeset publish",
+    "lint": "tsc && eslint --fix src/* --ignore-path .gitignore --ext .js,.ts",
+    "format": "prettier --write \"**/*.+(js|ts|json)\"",
+    "test": "jest",
+    "pre-commit": "lint-staged"
+  }
+}

package/src/SuiTradingClient.ts

@@ -13,6 +13,10 @@ import {
 } from './methods/acceptCollectionBid/acceptCollectionBid';
 import { acceptNftBids, type AcceptNftBids } from './methods/acceptNftBids/acceptNftBids';
 import { applyFtStrategy, type ApplyFtStrategy } from './methods/applyFtStrategy/applyFtStrategy';
+import {
+	type ApplyNftStrategy,
+	applyTradeportNftStrategy,
+} from './methods/applyNftStrategy/applyNftStrategy';
 import { buyListings, type BuyListings } from './methods/buyListings/buyListings';
 import { cancelMultiBid, type CancelMultiBid } from './methods/cancelMultiBid/cancelMultiBid';
 import {
@@ -27,8 +31,8 @@ import {
 	type MigrateNftsFromUnsharedToSharedKiosks,
 } from './methods/migrateNftsFromUnsharedToSharedKiosks/migrateNftsFromUnsharedToSharedKiosks';
 import {
-	placeCollectionBids,
 	type PlaceCollectionBid,
+	placeCollectionBids,
 } from './methods/placeCollectionBids/placeCollectionBids';
 import { placeNftBids, type PlaceNftBids } from './methods/placeNftBids/placeNftBids';
 import {
@@ -36,6 +40,10 @@ import {
 	type RemoveCollectionBids,
 } from './methods/removeCollectionBids/removeCollectionBids';
 import { removeNftBids, type RemoveNftBids } from './methods/removeNftBids/removeNftBids';
+import {
+	type SponsorNftListing,
+	sponsorNftListing,
+} from './methods/sponsorNftListing/sponsorNftListing';
 import { transferNfts, type TransferNfts } from './methods/transferNfts/transferNfts';
 import { unlistListings, type UnlistListings } from './methods/unlistListings/unlistListings';
 import { updateMultiBid, type UpdateMultiBid } from './methods/updateMultiBid/updateMultiBid';
@@ -44,28 +52,12 @@ import {
 	type WithdrawProfitsFromKiosks,
 } from './methods/withdrawProfitsFromKiosks/withdrawProfitsFromKiosks';
 import { addLeadingZerosAfter0x } from './utils/addLeadingZerosAfter0x';
-import {
-	type SponsorNftListing,
-	sponsorNftListing,
-} from './methods/sponsorNftListing/sponsorNftListing';
-import {
-	type ApplyNftStrategy,
-	applyTradeportNftStrategy,
-} from './methods/applyNftStrategy/applyNftStrategy';
-import {
-	buildSwapRoutes,
-	calculateAmountToSwap,
-	type SwapCoins,
-	swapCoins,
-	type SwapRoutesArgs,
-} from './methods/swapCoins/swapCoins';
-import { type Coin, type GetRoutesResult } from '@flowx-finance/sdk';
 
 type ApiConfig = {
 	apiUser: string;
 	apiKey: string;
 	apiUrl?: string;
-	defiRouterUrl?: string;
+	tradeportRouterUrl?: string;
 	apiVercelId?: string;
 	suiNodeUrl?: string;
 	graphQLClient?: GraphQLClient;
@@ -73,7 +65,7 @@ type ApiConfig = {
 export interface RequestContext {
 	apiUser: string;
 	apiKey: string;
-	defiRouterUrl?: string;
+	tradeportRouterUrl?: string;
 	suiClient: ReturnType<typeof createSuiClient>;
 	kioskClient: ReturnType<typeof createKioskClient>;
 }
@@ -81,7 +73,7 @@ export interface RequestContext {
 class SuiTradingClient {
 	private readonly apiUser: string;
 	private readonly apiKey: string;
-	private readonly defiRouterUrl: string;
+	private readonly tradeportRouterUrl?: string;
 	private readonly suiClient: ReturnType<typeof createSuiClient>;
 	private readonly kioskClient: ReturnType<typeof createKioskClient>;
 	private readonly allowedApiUsersForUnsharedKiosksMigration = ['tradeport.xyz', 'mercato.xyz'];
@@ -91,13 +83,13 @@ class SuiTradingClient {
 		apiKey,
 		apiVercelId,
 		apiUrl,
-		defiRouterUrl,
+		tradeportRouterUrl,
 		suiNodeUrl,
 		graphQLClient,
 	}: ApiConfig) {
 		this.apiUser = apiUser;
 		this.apiKey = apiKey;
-		this.defiRouterUrl = defiRouterUrl ?? 'https://api.indexer.xyz/router';
+		this.tradeportRouterUrl = tradeportRouterUrl;
 		this.suiClient = createSuiClient(suiNodeUrl ?? getFullnodeUrl('mainnet'));
 		this.kioskClient = createKioskClient(this.suiClient);
 
@@ -116,35 +108,34 @@ class SuiTradingClient {
 		// trackMethodCall({ apiUser, apiKey, event: 'INITIALIZE_CLIENT' });
 	}
 
-	public async buyListings({ listingIds, walletAddress, tx }: BuyListings): Promise<Transaction> {
+	public async buyListings(args: BuyListings): Promise<Transaction> {
 		const context: RequestContext = {
 			apiUser: this.apiUser,
 			apiKey: this.apiKey,
-			defiRouterUrl: this.defiRouterUrl,
+			tradeportRouterUrl: this.tradeportRouterUrl,
 			suiClient: this.suiClient,
 			kioskClient: this.kioskClient,
 		};
-
-		return buyListings({ listingIds, walletAddress, tx }, context);
+		return buyListings(args, context);
 	}
 
-	public async listNfts({ nfts, walletAddress }: ListNfts) {
+	public async listNfts({ nfts, walletAddress, tx }: ListNfts) {
 		const context: RequestContext = {
 			apiUser: this.apiUser,
 			apiKey: this.apiKey,
-			defiRouterUrl: this.defiRouterUrl,
+			tradeportRouterUrl: this.tradeportRouterUrl,
 			suiClient: this.suiClient,
 			kioskClient: this.kioskClient,
 		};
 
-		return listNfts({ nfts, walletAddress }, context);
+		return listNfts({ nfts, walletAddress, tx }, context);
 	}
 
 	public async unlistListings({ listingIds, walletAddress }: UnlistListings) {
 		const context: RequestContext = {
 			apiUser: this.apiUser,
 			apiKey: this.apiKey,
-			defiRouterUrl: this.defiRouterUrl,
+			tradeportRouterUrl: this.tradeportRouterUrl,
 			suiClient: this.suiClient,
 			kioskClient: this.kioskClient,
 		};
@@ -156,7 +147,7 @@ class SuiTradingClient {
 		const context: RequestContext = {
 			apiUser: this.apiUser,
 			apiKey: this.apiKey,
-			defiRouterUrl: this.defiRouterUrl,
+			tradeportRouterUrl: this.tradeportRouterUrl,
 			suiClient: this.suiClient,
 			kioskClient: this.kioskClient,
 		};
@@ -167,7 +158,7 @@ class SuiTradingClient {
 	public async removeNftBids({ bidIds, tx }: RemoveNftBids) {
 		const context: RequestContext = {
 			apiUser: this.apiUser,
-			defiRouterUrl: this.defiRouterUrl,
+			tradeportRouterUrl: this.tradeportRouterUrl,
 			apiKey: this.apiKey,
 			suiClient: this.suiClient,
 			kioskClient: this.kioskClient,
@@ -180,7 +171,7 @@ class SuiTradingClient {
 		const context: RequestContext = {
 			apiUser: this.apiUser,
 			apiKey: this.apiKey,
-			defiRouterUrl: this.defiRouterUrl,
+			tradeportRouterUrl: this.tradeportRouterUrl,
 			suiClient: this.suiClient,
 			kioskClient: this.kioskClient,
 		};
@@ -201,7 +192,7 @@ class SuiTradingClient {
 		const context: RequestContext = {
 			apiUser: this.apiUser,
 			apiKey: this.apiKey,
-			defiRouterUrl: this.defiRouterUrl,
+			tradeportRouterUrl: this.tradeportRouterUrl,
 			suiClient: this.suiClient,
 			kioskClient: this.kioskClient,
 		};
@@ -229,7 +220,7 @@ class SuiTradingClient {
 		const context: RequestContext = {
 			apiUser: this.apiUser,
 			apiKey: this.apiKey,
-			defiRouterUrl: this.defiRouterUrl,
+			tradeportRouterUrl: this.tradeportRouterUrl,
 			suiClient: this.suiClient,
 			kioskClient: this.kioskClient,
 		};
@@ -321,7 +312,7 @@ class SuiTradingClient {
 		const context: RequestContext = {
 			apiUser: this.apiUser,
 			apiKey: this.apiKey,
-			defiRouterUrl: this.defiRouterUrl,
+			tradeportRouterUrl: this.tradeportRouterUrl,
 			suiClient: this.suiClient,
 			kioskClient: this.kioskClient,
 		};
@@ -333,7 +324,7 @@ class SuiTradingClient {
 		const context: RequestContext = {
 			apiUser: this.apiUser,
 			apiKey: this.apiKey,
-			defiRouterUrl: this.defiRouterUrl,
+			tradeportRouterUrl: this.tradeportRouterUrl,
 			suiClient: this.suiClient,
 			kioskClient: this.kioskClient,
 		};
@@ -345,7 +336,7 @@ class SuiTradingClient {
 		const context: RequestContext = {
 			apiUser: this.apiUser,
 			apiKey: this.apiKey,
-			defiRouterUrl: this.defiRouterUrl,
+			tradeportRouterUrl: this.tradeportRouterUrl,
 			suiClient: this.suiClient,
 			kioskClient: this.kioskClient,
 		};
@@ -363,7 +354,7 @@ class SuiTradingClient {
 		const context: RequestContext = {
 			apiUser: this.apiUser,
 			apiKey: this.apiKey,
-			defiRouterUrl: this.defiRouterUrl,
+			tradeportRouterUrl: this.tradeportRouterUrl,
 			suiClient: this.suiClient,
 			kioskClient: this.kioskClient,
 		};
@@ -386,7 +377,7 @@ class SuiTradingClient {
 		const context: RequestContext = {
 			apiUser: this.apiUser,
 			apiKey: this.apiKey,
-			defiRouterUrl: this.defiRouterUrl,
+			tradeportRouterUrl: this.tradeportRouterUrl,
 			suiClient: this.suiClient,
 			kioskClient: this.kioskClient,
 		};
@@ -402,7 +393,7 @@ class SuiTradingClient {
 		const context: RequestContext = {
 			apiUser: this.apiUser,
 			apiKey: this.apiKey,
-			defiRouterUrl: this.defiRouterUrl,
+			tradeportRouterUrl: this.tradeportRouterUrl,
 			suiClient: this.suiClient,
 			kioskClient: this.kioskClient,
 		};
@@ -419,19 +410,14 @@ class SuiTradingClient {
 	}
 
 	public async updateMultiBid(args: UpdateMultiBid) {
-		return updateMultiBid(args);
+		return updateMultiBid(args, {
+			suiClient: this.suiClient,
+			tradeportRouterUrl: this.tradeportRouterUrl,
+		});
 	}
 
 	public async sponsorNftListing(args: SponsorNftListing) {
-		const context: RequestContext = {
-			apiUser: this.apiUser,
-			apiKey: this.apiKey,
-			defiRouterUrl: this.defiRouterUrl,
-			suiClient: this.suiClient,
-			kioskClient: this.kioskClient,
-		};
-
-		return sponsorNftListing(args, context);
+		return sponsorNftListing(args);
 	}
 
 	public async applyFtStrategy(args: ApplyFtStrategy) {
@@ -441,20 +427,6 @@ class SuiTradingClient {
 	public async applyTradeportNftStrategy(args: ApplyNftStrategy) {
 		return applyTradeportNftStrategy(args);
 	}
-
-	public async swapCoins(args: SwapCoins) {
-		return swapCoins(args, { suiClient: this.suiClient, defiRouterUrl: this.defiRouterUrl });
-	}
-
-	public async buildSwapRoutes(args: SwapRoutesArgs): Promise<GetRoutesResult<Coin, Coin>> {
-		return buildSwapRoutes(args, { defiRouterUrl: this.defiRouterUrl });
-	}
-
-	public async calculateAmountToSwap(
-		args: Pick<SwapCoins, 'coinInType' | 'coinOutType' | 'coinInAmount'>,
-	): Promise<bigint> {
-		return calculateAmountToSwap(args);
-	}
 }
 
 export default SuiTradingClient;

package/src/index.ts

@@ -1,3 +1,2 @@
 import SuiTradingClient from './SuiTradingClient';
-
 export { SuiTradingClient };

package/src/methods/buyListings/buyListings.ts

@@ -9,8 +9,6 @@ import {
 import { gqlChainRequest } from '../../graphql/gqlChainRequest';
 import { fetchListingsById } from '../../graphql/queries/fetchListingsById';
 import { addThirdPartyTxFee } from '../../helpers/addThirdPartyTxFee';
-import { deserializeOrCreateTxBlock } from '../../helpers/deserializeOrCreateTxBlock';
-import { extractSwapResultCoinFromTxBlock } from '../../helpers/extractSwapResultCoin';
 import { getNftType } from '../../helpers/getNftType';
 import {
 	preProcessSharedBulkBuyingData,
@@ -74,7 +72,7 @@ export type BuyListing = {
 export type BuyListings = {
 	listingIds: string[];
 	walletAddress: string;
-	tx?: Transaction | string;
+	tx?: Transaction;
 	kioskTx?: KioskTransaction;
 	coinToSplit?: any;
 	beforeResolveKioskTransferRequest?: (transferRequest: any) => void | Promise<void>;
@@ -103,8 +101,7 @@ export const buyListings = async (
 	}
 
 	const listingsForTracking = [];
-	const tx = deserializeOrCreateTxBlock({ existingTx });
-	const swapResultCoin = extractSwapResultCoinFromTxBlock(tx);
+	const tx = existingTx ?? new Transaction();
 
 	const tocenTokenIds: string[] = [];
 	let tocenNftType = '';
@@ -157,7 +154,7 @@ export const buyListings = async (
 			sellerKiosk: listing.nft?.chain_state?.kiosk_id,
 			collectionId: listing?.nft?.collection_id,
 			nftContractId: listing?.nft?.contract_id,
-			coinToSplit: swapResultCoin ? swapResultCoin : coinToSplit,
+			coinToSplit,
 			marketFeeDecimalPercent,
 			beforeResolveKioskTransferRequest,
 			sharedKioskState,
@@ -211,10 +208,6 @@ export const buyListings = async (
 
 	sharedKioskState?.kioskTx?.finalize();
 
-	if (swapResultCoin) {
-		tx.transferObjects([swapResultCoin], walletAddress);
-	}
-
 	if (tocenTokenIds?.length > 0) {
 		addTocenBuyTxHandler({
 			tx,
@@ -233,5 +226,5 @@ export const buyListings = async (
 	// 	});
 	// }
 
-	return Transaction.from(tx);
+	return tx;
 };

package/src/methods/createMultiBid/createMultiBid.ts

@@ -1,13 +1,12 @@
-import { type Transaction } from '@mysten/sui/transactions';
+import { Transaction } from '@mysten/sui/transactions';
 import { TRADEPORT_MULTI_BID_PACKAGE, TRADEPORT_MULTI_BID_STORE } from '../../constants';
-import { deserializeOrCreateTxBlock } from '../../helpers/deserializeOrCreateTxBlock';
 import { updateMultiBid } from '../updateMultiBid/updateMultiBid';
 
 export interface CreateMultiBid {
 	walletAddress: string;
 	name?: string;
 	amount?: bigint;
-	tx?: Transaction | string;
+	tx?: Transaction;
 }
 
 export async function createMultiBid({
@@ -16,7 +15,7 @@ export async function createMultiBid({
 	amount,
 	tx: existingTx,
 }: CreateMultiBid): Promise<{ multiBidChainId: any; tx: Transaction }> {
-	const tx = deserializeOrCreateTxBlock({ existingTx });
+	const tx = existingTx ?? new Transaction();
 
 	const multiBidChainId = tx.moveCall({
 		target: `${TRADEPORT_MULTI_BID_PACKAGE}::tradeport_biddings::create_multi_bid`,

package/src/methods/listNfts/listNfts.ts

@@ -9,13 +9,13 @@ import {
 import { gqlChainRequest } from '../../graphql/gqlChainRequest';
 import { fetchNftsWithListingsById } from '../../graphql/queries/fetchNftsById';
 import { getNftType } from '../../helpers/getNftType';
+import { isExpiredListing } from '../../helpers/isExpiredListing';
 import { type SharedKioskState } from '../../helpers/kiosk/sharedKioskState.type';
-import { relistNft } from '../relistNft/relistNft';
-import { addTradePortListTxHandler } from './addListTxs';
 import { isOriginByteCollection } from '../../helpers/originByte/isOriginByteCollection';
+import { relistNft } from '../relistNft/relistNft';
 import { addSponsorListingTx } from '../sponsorNftListing/addSponsorNftListingTx';
 import { type SponsorNftListingOptions } from '../sponsorNftListing/sponsorNftListing';
-import { isExpiredListing } from '../../helpers/isExpiredListing';
+import { addTradePortListTxHandler } from './addListTxs';
 
 export type ListNftTx = {
 	tx: Transaction;
@@ -41,10 +41,11 @@ export type ListNfts = {
 		sponsorOptions?: SponsorNftListingOptions;
 	}>;
 	walletAddress: string;
+	tx?: Transaction;
 };
 
 export const listNfts = async (
-	{ nfts, walletAddress }: ListNfts,
+	{ nfts, walletAddress, tx: existingTx }: ListNfts,
 	context: RequestContext,
 ): Promise<Transaction> => {
 	const res = await gqlChainRequest({
@@ -58,7 +59,7 @@ export const listNfts = async (
 	}
 
 	const nftsForTracking = [];
-	const tx = new Transaction();
+	const tx = existingTx ?? new Transaction();
 
 	const sharedKioskState: SharedKioskState = {
 		kioskTx: undefined,
@@ -120,12 +121,9 @@ export const listNfts = async (
 		if (inputNft?.sponsorOptions?.shouldSponsor) {
 			await addSponsorListingTx({
 				tx,
-				suiClient: context.suiClient,
 				nftTokenId: nft?.token_id,
 				nftType,
-				walletAddress,
 				sponsorOptions: inputNft?.sponsorOptions,
-				defiRouterUrl: context.defiRouterUrl,
 			});
 		}
 
@@ -150,5 +148,5 @@ export const listNfts = async (
 	// 	});
 	// }
 
-	return tx instanceof Transaction ? tx : Transaction.from(tx);
+	return tx;
 };

package/src/methods/placeCollectionBids/placeCollectionBids.ts

@@ -5,8 +5,6 @@ import { type RequestContext } from '../../SuiTradingClient';
 import { ORIGIN_BYTE_NFT_TYPES_MISSING_ORDERBOOK } from '../../constants';
 import { gqlChainRequest } from '../../graphql/gqlChainRequest';
 import { fetchCollectionsByIdWithOneNft } from '../../graphql/queries/fetchCollectionsById';
-import { deserializeOrCreateTxBlock } from '../../helpers/deserializeOrCreateTxBlock';
-import { extractSwapResultCoinFromTxBlock } from '../../helpers/extractSwapResultCoin';
 import { getNftType } from '../../helpers/getNftType';
 import { getSharedObjects } from '../../helpers/getSharedObjects';
 import { getOrCreateOBKiosk } from '../../helpers/originByte/getOrCreateOBKiosk';
@@ -43,7 +41,7 @@ export type PlaceCollectionBid = {
 	multiBidId?: string;
 	multiBidChainId?: any;
 	expireAt?: Date;
-	tx?: Transaction | string;
+	tx?: Transaction;
 };
 
 export type PlaceCollectionBids = {
@@ -56,11 +54,19 @@ export type PlaceCollectionBids = {
 	walletAddress: string;
 	multiBidId?: string;
 	multiBidChainId?: any;
-	tx?: Transaction | string;
+	tx?: Transaction;
+	coinToSplit?: any;
 };
 
 export const placeCollectionBids = async (
-	{ collections, walletAddress, multiBidId, multiBidChainId, tx: existingTx }: PlaceCollectionBids,
+	{
+		collections,
+		walletAddress,
+		multiBidId,
+		multiBidChainId,
+		tx: existingTx,
+		coinToSplit,
+	}: PlaceCollectionBids,
 	context: RequestContext,
 ): Promise<Transaction> => {
 	const res = await gqlChainRequest({
@@ -74,8 +80,7 @@ export const placeCollectionBids = async (
 	}
 
 	const collectionsForTracking = [];
-	const tx = deserializeOrCreateTxBlock({ existingTx });
-	const swapResultCoin = extractSwapResultCoinFromTxBlock(tx);
+	const tx = existingTx ?? new Transaction();
 
 	for (const collection of res.collections) {
 		const nftType = getNftType({
@@ -98,7 +103,7 @@ export const placeCollectionBids = async (
 			multiBidId,
 			multiBidChainId,
 			expireAt: collections?.find((c) => c.id === collection?.id)?.expireAt,
-			coinToSplit: swapResultCoin,
+			coinToSplit,
 		};
 
 		const numOfBids = collections?.find((c) => c.id === collection?.id)?.numOfBids;
@@ -135,10 +140,6 @@ export const placeCollectionBids = async (
 		});
 	}
 
-	if (swapResultCoin) {
-		tx.transferObjects([swapResultCoin], walletAddress);
-	}
-
 	// if (process.env.ENABLE_SEGMENT_TRACKING === 'true' && collectionsForTracking.length > 0) {
 	// 	trackMethodCall({
 	// 		apiUser: context.apiUser,
@@ -148,5 +149,5 @@ export const placeCollectionBids = async (
 	// 	});
 	// }
 
-	return Transaction.from(tx);
+	return tx;
 };

package/src/methods/placeNftBids/placeNftBids.ts

@@ -7,8 +7,6 @@ import {
 } from '../../constants';
 import { gqlChainRequest } from '../../graphql/gqlChainRequest';
 import { fetchNftsById } from '../../graphql/queries/fetchNftsById';
-import { deserializeOrCreateTxBlock } from '../../helpers/deserializeOrCreateTxBlock';
-import { extractSwapResultCoinFromTxBlock } from '../../helpers/extractSwapResultCoin';
 import { getNftType } from '../../helpers/getNftType';
 import { addTradePortPlaceNftBidTxHandler } from './addPlaceNftBidTxs';
 
@@ -38,15 +36,15 @@ export type PlaceNftBids = {
 	walletAddress: string;
 	multiBidId?: string;
 	multiBidChainId?: any;
-	tx?: Transaction | string;
+	tx?: Transaction;
+	coinToSplit?: any;
 };
 
 export const placeNftBids = async (
-	{ bids, walletAddress, multiBidId, multiBidChainId, tx: existingTx }: PlaceNftBids,
+	{ bids, walletAddress, multiBidId, multiBidChainId, tx: existingTx, coinToSplit }: PlaceNftBids,
 	context: RequestContext,
 ): Promise<Transaction> => {
-	const tx = deserializeOrCreateTxBlock({ existingTx });
-	const swapResultCoin = extractSwapResultCoinFromTxBlock(tx);
+	const tx = existingTx ?? new Transaction();
 
 	const res = await gqlChainRequest({
 		chain: 'sui',
@@ -87,7 +85,7 @@ export const placeNftBids = async (
 			multiBidId,
 			multiBidChainId,
 			expireAt: bid?.expireAt,
-			coinToSplit: swapResultCoin,
+			coinToSplit,
 		};
 
 		await addTradePortPlaceNftBidTxHandler(txData);
@@ -100,10 +98,6 @@ export const placeNftBids = async (
 		});
 	}
 
-	if (swapResultCoin) {
-		tx.transferObjects([swapResultCoin], walletAddress);
-	}
-
 	// if (process.env.ENABLE_SEGMENT_TRACKING === 'true' && nftsForTracking.length > 0) {
 	// 	trackMethodCall({
 	// 		apiUser: context.apiUser,
@@ -113,5 +107,5 @@ export const placeNftBids = async (
 	// 	});
 	// }
 
-	return Transaction.from(tx);
+	return tx;
 };