@algorandfoundation/algokit-utils 10.0.0-alpha.2 → 10.0.0-alpha.20

package/types/composer.mjs

@@ -1,8 +1,8 @@
 import { EMPTY_SIGNATURE, MAX_TRANSACTION_GROUP_SIZE } from "../packages/common/src/constants.mjs";
 import { OnApplicationComplete } from "../packages/transact/src/transactions/app-call.mjs";
 import { TransactionType } from "../packages/transact/src/transactions/transaction-type.mjs";
-import { Transaction, assignFee, calculateFee, decodeTransaction, encodeTransactionRaw, groupTransactions } from "../packages/transact/src/transactions/transaction.mjs";
-import { decodeSignedTransaction, encodeSignedTransactions } from "../packages/transact/src/transactions/signed-transaction.mjs";
+import { Transaction, assignFee, calculateFee, decodeTransaction, encodeTransactionRaw, groupTransactions, validateTransaction } from "../packages/transact/src/transactions/transaction.mjs";
+import { decodeSignedTransaction, encodeSignedTransactions, validateSignedTransaction } from "../packages/transact/src/transactions/signed-transaction.mjs";
 import { makeEmptyTransactionSigner } from "../packages/transact/src/signer.mjs";
 import { AlgoAmount } from "./amount.mjs";
 import { Config } from "../config.mjs";
@@ -205,6 +205,7 @@ var TransactionComposer = class TransactionComposer {
 	*/
 	addTransaction(transaction, signer) {
 		if (transaction.group) throw new Error("Cannot add a transaction to the composer because it is already in a group");
+		validateTransaction(transaction);
 		this.push({
 			data: {
 				txn: transaction,
@@ -570,6 +571,7 @@ var TransactionComposer = class TransactionComposer {
 	*  // Max fee doesn't make sense with extraFee AND staticFee
 	*  //  already specified, but here for completeness
 	*  maxFee: (3000).microAlgo(),
+	*  rejectVersion: 1,
 	*  // Signer only needed if you want to provide one,
 	*  //  generally you'd register it with AlgorandClient
 	*  //  against the sender and not need to pass it in
@@ -620,6 +622,7 @@ var TransactionComposer = class TransactionComposer {
 	*  // Max fee doesn't make sense with extraFee AND staticFee
 	*  //  already specified, but here for completeness
 	*  maxFee: (3000).microAlgo(),
+	*  rejectVersion: 1,
 	*})
 	* ```
 	*/
@@ -665,6 +668,7 @@ var TransactionComposer = class TransactionComposer {
 	*  // Max fee doesn't make sense with extraFee AND staticFee
 	*  //  already specified, but here for completeness
 	*  maxFee: (3000).microAlgo(),
+	*  rejectVersion: 1,
 	*})
 	* ```
 	*/
@@ -712,6 +716,7 @@ var TransactionComposer = class TransactionComposer {
 	*  // Max fee doesn't make sense with extraFee AND staticFee
 	*  //  already specified, but here for completeness
 	*  maxFee: (3000).microAlgo(),
+	*  rejectVersion: 1,
 	*})
 	* ```
 	*/
@@ -775,6 +780,7 @@ var TransactionComposer = class TransactionComposer {
 	*  // Max fee doesn't make sense with extraFee AND staticFee
 	*  //  already specified, but here for completeness
 	*  maxFee: (3000).microAlgo(),
+	*  rejectVersion: 1,
 	*})
 	* ```
 	*/
@@ -835,6 +841,7 @@ var TransactionComposer = class TransactionComposer {
 	*  // Max fee doesn't make sense with extraFee AND staticFee
 	*  //  already specified, but here for completeness
 	*  maxFee: (3000).microAlgo(),
+	*  rejectVersion: 1,
 	*})
 	* ```
 	*/
@@ -894,6 +901,7 @@ var TransactionComposer = class TransactionComposer {
 	*  // Max fee doesn't make sense with extraFee AND staticFee
 	*  //  already specified, but here for completeness
 	*  maxFee: (3000).microAlgo(),
+	*  rejectVersion: 1,
 	*})
 	* ```
 	*/
@@ -953,6 +961,7 @@ var TransactionComposer = class TransactionComposer {
 	*  // Max fee doesn't make sense with extraFee AND staticFee
 	*  //  already specified, but here for completeness
 	*  maxFee: (3000).microAlgo(),
+	*  rejectVersion: 1,
 	*})
 	* ```
 	*/
@@ -1076,7 +1085,7 @@ var TransactionComposer = class TransactionComposer {
 		if (!this.transactionsWithSigners) {
 			const suggestedParams = await this.getSuggestedParams();
 			const builtTransactions = await this._buildTransactions(suggestedParams);
-			this.rawBuildTransactions = builtTransactions.transactions;
+			this.rawBuildTransactions = builtTransactions.transactions.map((txn) => new Transaction({ ...txn }));
 			const groupAnalysis = (this.composerConfig.coverAppCallInnerTransactionFees || this.composerConfig.populateAppCallResources) && builtTransactions.transactions.some((txn) => txn.type === TransactionType.AppCall) ? await this.analyzeGroupRequirements(builtTransactions.transactions, suggestedParams, this.composerConfig) : void 0;
 			this.populateTransactionAndGroupResources(builtTransactions.transactions, groupAnalysis);
 			this.transactionsWithSigners = builtTransactions.transactions.map((txn, index) => {
@@ -1101,11 +1110,14 @@ var TransactionComposer = class TransactionComposer {
 		const transactions = new Array();
 		let transactionIndex = 0;
 		for (const ctxn of this.txns) if (ctxn.type === "txn") {
+			validateTransaction(ctxn.data.txn);
 			transactions.push(ctxn.data.txn);
 			if (ctxn.data.signer) signers.set(transactionIndex, ctxn.data.signer);
 			transactionIndex++;
 		} else if (ctxn.type === "asyncTxn") {
-			transactions.push(await ctxn.data.txn);
+			const transaction = await ctxn.data.txn;
+			validateTransaction(transaction);
+			transactions.push(transaction);
 			if (ctxn.data.signer) signers.set(transactionIndex, ctxn.data.signer);
 			transactionIndex++;
 		} else {
@@ -1136,7 +1148,7 @@ var TransactionComposer = class TransactionComposer {
 					transaction = buildAssetOptOut(ctxn.data, suggestedParams, defaultValidityWindow);
 					break;
 				case "appCall":
-					if (!("appId" in ctxn.data)) transaction = await buildAppCreate(ctxn.data, this.appManager, suggestedParams, defaultValidityWindow);
+					if (ctxn.data.appId === void 0 || ctxn.data.appId === 0) transaction = await buildAppCreate(ctxn.data, this.appManager, suggestedParams, defaultValidityWindow);
 					else if ("approvalProgram" in ctxn.data && "clearStateProgram" in ctxn.data) transaction = await buildAppUpdate(ctxn.data, this.appManager, suggestedParams, defaultValidityWindow);
 					else transaction = buildAppCall(ctxn.data, suggestedParams, defaultValidityWindow);
 					break;
@@ -1144,7 +1156,7 @@ var TransactionComposer = class TransactionComposer {
 					transaction = buildKeyReg(ctxn.data, suggestedParams, defaultValidityWindow);
 					break;
 				case "methodCall":
-					if (!("appId" in ctxn.data)) transaction = await buildAppCreateMethodCall(ctxn.data, this.appManager, suggestedParams, defaultValidityWindow);
+					if (ctxn.data.appId === void 0 || ctxn.data.appId === 0) transaction = await buildAppCreateMethodCall(ctxn.data, this.appManager, suggestedParams, defaultValidityWindow);
 					else if ("approvalProgram" in ctxn.data && "clearStateProgram" in ctxn.data) transaction = await buildAppUpdateMethodCall(ctxn.data, this.appManager, suggestedParams, defaultValidityWindow);
 					else transaction = await buildAppCallMethodCall(ctxn.data, suggestedParams, defaultValidityWindow);
 					break;
@@ -1156,6 +1168,7 @@ var TransactionComposer = class TransactionComposer {
 				extraFee: ctxn.data.extraFee?.microAlgos,
 				maxFee: ctxn.data.maxFee?.microAlgos
 			});
+			validateTransaction(transaction);
 			transactions.push(transaction);
 			if (ctxn.data.signer) {
 				const signer = "signer" in ctxn.data.signer ? ctxn.data.signer.signer : ctxn.data.signer;
@@ -1312,13 +1325,13 @@ var TransactionComposer = class TransactionComposer {
 			sortedResources.assets?.sort(compare);
 			sortedResources.apps?.sort(compare);
 			sortedResources.boxes?.sort((a, b) => {
-				return compare(`${a.app}-${a.name}`, `${b.app}-${b.name}`);
+				return compare(`${a.appId}-${a.name}`, `${b.appId}-${b.name}`);
 			});
 			sortedResources.appLocals?.sort((a, b) => {
-				return compare(`${a.app}-${a.account}`, `${b.app}-${b.account}`);
+				return compare(`${a.appId}-${a.address}`, `${b.appId}-${b.address}`);
 			});
 			sortedResources.assetHoldings?.sort((a, b) => {
-				return compare(`${a.asset}-${a.account}`, `${b.asset}-${b.account}`);
+				return compare(`${a.assetId}-${a.address}`, `${b.assetId}-${b.address}`);
 			});
 		}
 		return {
@@ -1353,11 +1366,12 @@ var TransactionComposer = class TransactionComposer {
 	* ```
 	*/
 	async send(params) {
-		if (this.composerConfig.coverAppCallInnerTransactionFees !== (params?.coverAppCallInnerTransactionFees ?? false) || this.composerConfig.populateAppCallResources !== (params?.populateAppCallResources ?? true)) {
-			this.composerConfig = {
-				coverAppCallInnerTransactionFees: params?.coverAppCallInnerTransactionFees ?? false,
-				populateAppCallResources: params?.populateAppCallResources ?? true
-			};
+		const effectiveConfig = {
+			coverAppCallInnerTransactionFees: params?.coverAppCallInnerTransactionFees ?? this.composerConfig.coverAppCallInnerTransactionFees,
+			populateAppCallResources: params?.populateAppCallResources ?? this.composerConfig.populateAppCallResources
+		};
+		if (this.composerConfig.coverAppCallInnerTransactionFees !== effectiveConfig.coverAppCallInnerTransactionFees || this.composerConfig.populateAppCallResources !== effectiveConfig.populateAppCallResources) {
+			this.composerConfig = effectiveConfig;
 			this.reset();
 		}
 		try {
@@ -1445,7 +1459,7 @@ var TransactionComposer = class TransactionComposer {
 		}
 	}
 	async simulate(options) {
-		const { skipSignatures = false, resultOnFailure = false,...rawOptions } = options ?? {};
+		const { skipSignatures = false, resultOnFailure = false, ...rawOptions } = options ?? {};
 		if (skipSignatures) {
 			rawOptions.allowEmptySignatures = true;
 			rawOptions.fixSigners = true;
@@ -1453,7 +1467,7 @@ var TransactionComposer = class TransactionComposer {
 		let transactionsWithSigner;
 		if (!this.transactionsWithSigners) {
 			const builtTransactions = await this.buildTransactions();
-			transactionsWithSigner = (builtTransactions.transactions.length > 0 ? groupTransactions(builtTransactions.transactions) : builtTransactions.transactions).map((txn, index) => ({
+			transactionsWithSigner = (builtTransactions.transactions.length > 1 ? groupTransactions(builtTransactions.transactions) : builtTransactions.transactions).map((txn, index) => ({
 				txn,
 				signer: skipSignatures ? makeEmptyTransactionSigner() : builtTransactions.signers.get(index) ?? makeEmptyTransactionSigner()
 			}));
@@ -1482,10 +1496,10 @@ var TransactionComposer = class TransactionComposer {
 		if (simulateResult?.failureMessage && !resultOnFailure) {
 			const errorMessage = `Transaction failed at transaction(s) ${simulateResult.failedAt?.join(", ") || "unknown"} in the group. ${simulateResult.failureMessage}`;
 			const error = new Error(errorMessage);
-			if (Config.debug) await Config.events.emitAsync(EventType.TxnGroupSimulated, { simulateTransaction: simulateResponse });
+			if (Config.debug) await Config.events.emitAsync(EventType.TxnGroupSimulated, { simulateResponse });
 			throw await this.transformError(error);
 		}
-		if (Config.debug && Config.traceAll) await Config.events.emitAsync(EventType.TxnGroupSimulated, { simulateTransaction: simulateResponse });
+		if (Config.debug && Config.traceAll) await Config.events.emitAsync(EventType.TxnGroupSimulated, { simulateResponse });
 		const abiReturns = this.parseAbiReturnValues(simulateResult.txnResults.map((t) => t.txnResult));
 		return {
 			confirmations: simulateResult.txnResults.map((t) => t.txnResult),
@@ -1529,7 +1543,9 @@ var TransactionComposer = class TransactionComposer {
 		signerEntries.forEach(([, indexes], signerIndex) => {
 			const stxs = signedGroups[signerIndex];
 			indexes.forEach((txIndex, stxIndex) => {
-				signedTransactions[txIndex] = decodeSignedTransaction(stxs[stxIndex]);
+				const stxn = decodeSignedTransaction(stxs[stxIndex]);
+				validateSignedTransaction(stxn);
+				signedTransactions[txIndex] = stxn;
 			});
 		});
 		const unsignedIndexes = signedTransactions.map((stxn, index) => stxn === void 0 ? index : null).filter((index) => index !== null);
@@ -1542,11 +1558,8 @@ var TransactionComposer = class TransactionComposer {
 			const confirmation = confirmations[i];
 			const txn = this.txns[i];
 			if (txn?.type !== "methodCall") continue;
-			const method = txn.data.method;
-			if (method.returns.type !== "void") {
-				const abiReturn = AppManager.getABIReturn(confirmation, method);
-				if (abiReturn !== void 0) abiReturns.push(abiReturn);
-			}
+			const abiReturn = AppManager.getABIReturn(confirmation, txn.data.method);
+			if (abiReturn !== void 0) abiReturns.push(abiReturn);
 		}
 		return abiReturns;
 	}