@aptos-labs/ts-sdk 5.2.1 → 6.1.0

package/src/internal/general.ts

@@ -15,9 +15,19 @@ import { GetChainTopUserTransactionsResponse, GetProcessorStatusResponse, Graphq
 import { GetChainTopUserTransactionsQuery, GetProcessorStatusQuery } from "../types/generated/operations";
 import { GetChainTopUserTransactions, GetProcessorStatus } from "../types/generated/queries";
 import { ProcessorType } from "../utils/const";
+import { memoizeAsync } from "../utils/memoize";
+
+/**
+ * Cache TTL for ledger info in milliseconds (10 seconds).
+ * Ledger info changes frequently but we can cache briefly to reduce redundant calls
+ * when building multiple transactions in quick succession.
+ */
+const LEDGER_INFO_CACHE_TTL_MS = 10 * 1000;
 
 /**
  * Retrieves information about the current ledger.
+ * Results are cached for 10 seconds to reduce redundant network calls during
+ * rapid transaction building.
  *
  * @param args - The arguments for retrieving ledger information.
  * @param args.aptosConfig - The configuration object for connecting to the Aptos network.
@@ -25,12 +35,20 @@ import { ProcessorType } from "../utils/const";
  */
 export async function getLedgerInfo(args: { aptosConfig: AptosConfig }): Promise<LedgerInfo> {
   const { aptosConfig } = args;
-  const { data } = await getAptosFullNode<{}, LedgerInfo>({
-    aptosConfig,
-    originMethod: "getLedgerInfo",
-    path: "",
-  });
-  return data;
+  const cacheKey = `ledger-info-${aptosConfig.network}`;
+
+  return memoizeAsync(
+    async () => {
+      const { data } = await getAptosFullNode<{}, LedgerInfo>({
+        aptosConfig,
+        originMethod: "getLedgerInfo",
+        path: "",
+      });
+      return data;
+    },
+    cacheKey,
+    LEDGER_INFO_CACHE_TTL_MS,
+  )();
 }
 
 /**

package/src/internal/queries/getCollectionData.graphql

@@ -14,16 +14,5 @@ query getCollectionData($where_condition: current_collections_v2_bool_exp!) {
     description
     last_transaction_timestamp
     last_transaction_version
-    cdn_asset_uris {
-      cdn_image_uri
-      asset_uri
-      animation_optimizer_retry_count
-      cdn_animation_uri
-      cdn_json_uri
-      image_optimizer_retry_count
-      json_parser_retry_count
-      raw_animation_uri
-      raw_image_uri
-    }
   }
 }

package/src/internal/queries/getNames.graphql

@@ -8,4 +8,9 @@ query getNames(
   current_aptos_names(limit: $limit, where: $where_condition, order_by: $order_by, offset: $offset) {
     ...AnsTokenFragment
   }
+  current_aptos_names_aggregate(where: $where_condition) {
+    aggregate {
+      count
+    }
+  }
 }

package/src/internal/transaction.ts

@@ -246,7 +246,6 @@ export async function waitForTransaction(args: {
       break;
     }
     try {
-      // eslint-disable-next-line no-await-in-loop
       lastTxn = await getTransactionByHash({ aptosConfig, transactionHash });
 
       isPending = lastTxn.type === TransactionResponseType.Pending;
@@ -257,7 +256,7 @@ export async function waitForTransaction(args: {
     } catch (e) {
       handleAPIError(e);
     }
-    // eslint-disable-next-line no-await-in-loop
+
     await sleep(backoffIntervalMs);
     timeElapsed += backoffIntervalMs / 1000; // Convert to seconds
     backoffIntervalMs *= backoffMultiplier;
@@ -322,11 +321,11 @@ export async function waitForIndexer(args: {
 
     if (processorType === undefined) {
       // Get the last success version from all processor
-      // eslint-disable-next-line no-await-in-loop
+
       indexerVersion = await getIndexerLastSuccessVersion({ aptosConfig });
     } else {
       // Get the last success version from the specific processor
-      // eslint-disable-next-line no-await-in-loop
+
       const processor = await getProcessorStatus({ aptosConfig, processorType });
       indexerVersion = processor.last_success_version;
     }
@@ -336,7 +335,6 @@ export async function waitForIndexer(args: {
       break;
     }
 
-    // eslint-disable-next-line no-await-in-loop
     await sleep(200);
   }
 }

package/src/internal/transactionSubmission.ts

@@ -300,6 +300,29 @@ export async function simulateTransaction(
     originMethod: "simulateTransaction",
     contentType: MimeType.BCS_SIGNED_TRANSACTION,
   });
+
+  // If the server's gas estimation produced a value below the network minimum,
+  // retry without estimation so the transaction's original max_gas_amount is used.
+  if (
+    args.options?.estimateMaxGasAmount &&
+    data.length > 0 &&
+    data[0].vm_status === "MAX_GAS_UNITS_BELOW_MIN_TRANSACTION_GAS_UNITS"
+  ) {
+    const { data: retryData } = await postAptosFullNode<Uint8Array, Array<UserTransactionResponse>>({
+      aptosConfig,
+      body: signedTransaction,
+      path: "transactions/simulate",
+      params: {
+        estimate_gas_unit_price: args.options?.estimateGasUnitPrice ?? false,
+        estimate_max_gas_amount: false,
+        estimate_prioritized_gas_unit_price: args.options?.estimatePrioritizedGasUnitPrice ?? false,
+      },
+      originMethod: "simulateTransaction",
+      contentType: MimeType.BCS_SIGNED_TRANSACTION,
+    });
+    return retryData;
+  }
+
   return data;
 }
 

package/src/transactions/authenticator/account.ts

@@ -1,8 +1,6 @@
 // Copyright © Aptos Foundation
 // SPDX-License-Identifier: Apache-2.0
 
-/* eslint-disable @typescript-eslint/naming-convention */
-
 import { Serializer, Deserializer, Serializable } from "../../bcs";
 import { AnyPublicKey, AnySignature } from "../../core/crypto";
 import { Ed25519PublicKey, Ed25519Signature } from "../../core/crypto/ed25519";
@@ -258,7 +256,6 @@ export class AccountAuthenticatorMultiKey extends AccountAuthenticator {
  * It allows skipping the public/auth key check during the simulation.
  */
 export class AccountAuthenticatorNoAccountAuthenticator extends AccountAuthenticator {
-  // eslint-disable-next-line class-methods-use-this
   serialize(serializer: Serializer): void {
     serializer.serializeU32AsUleb128(AccountAuthenticatorVariant.NoAccountAuthenticator);
   }

package/src/transactions/authenticator/transaction.ts

@@ -1,8 +1,6 @@
 // Copyright © Aptos Foundation
 // SPDX-License-Identifier: Apache-2.0
 
-/* eslint-disable @typescript-eslint/naming-convention */
-
 import { AccountAuthenticator } from "./account";
 import { Deserializer, Serializable, Serializer } from "../../bcs";
 import { AccountAddress } from "../../core";

package/src/transactions/instances/multiAgentTransaction.ts

@@ -1,8 +1,6 @@
 // Copyright © Aptos Foundation
 // SPDX-License-Identifier: Apache-2.0
 
-/* eslint-disable @typescript-eslint/naming-convention */
-
 import { Deserializer } from "../../bcs/deserializer";
 import { Serializable, Serializer } from "../../bcs/serializer";
 import { AccountAddress } from "../../core";

package/src/transactions/instances/rawTransaction.ts

@@ -1,8 +1,6 @@
 // Copyright © Aptos Foundation
 // SPDX-License-Identifier: Apache-2.0
 
-/* eslint-disable @typescript-eslint/naming-convention */
-
 import { Deserializer } from "../../bcs/deserializer";
 import { Serializable, Serializer } from "../../bcs/serializer";
 import { ChainId } from "./chainId";

package/src/transactions/instances/signedTransaction.ts

@@ -1,8 +1,6 @@
 // Copyright © Aptos Foundation
 // SPDX-License-Identifier: Apache-2.0
 
-/* eslint-disable @typescript-eslint/naming-convention */
-
 import { TransactionAuthenticator } from "../authenticator/transaction";
 import { Deserializer } from "../../bcs/deserializer";
 import { Serializable, Serializer } from "../../bcs/serializer";

package/src/transactions/instances/simpleTransaction.ts

@@ -1,8 +1,6 @@
 // Copyright © Aptos Foundation
 // SPDX-License-Identifier: Apache-2.0
 
-/* eslint-disable @typescript-eslint/naming-convention */
-
 import { Deserializer } from "../../bcs/deserializer";
 import { Serializable, Serializer } from "../../bcs/serializer";
 import { AccountAddress } from "../../core";

package/src/transactions/instances/transactionPayload.ts

@@ -1,7 +1,7 @@
 // Copyright © Aptos Foundation
 // SPDX-License-Identifier: Apache-2.0
 
-/* eslint-disable @typescript-eslint/naming-convention */
+/* eslint-disable max-classes-per-file */
 
 import { Deserializer } from "../../bcs/deserializer";
 import { Serializable, Serializer } from "../../bcs/serializer";
@@ -22,7 +22,7 @@ import {
   I256,
 } from "../../bcs/serializable/movePrimitives";
 import { MoveVector, Serialized } from "../../bcs/serializable/moveStructs";
-import { AccountAddress, Hex } from "../../core";
+import { AccountAddress } from "../../core";
 import { Identifier } from "./identifier";
 import { ModuleId } from "./moduleId";
 import type { EntryFunctionArgument, ScriptFunctionArgument, TransactionArgument } from "./transactionArgument";
@@ -588,8 +588,8 @@ export class TransactionInnerPayloadV1 extends TransactionInnerPayload {
   }
 
   static load(deserializer: Deserializer): TransactionInnerPayloadV1 {
-    let executable = TransactionExecutable.deserialize(deserializer);
-    let extra_config = TransactionExtraConfig.deserialize(deserializer);
+    const executable = TransactionExecutable.deserialize(deserializer);
+    const extra_config = TransactionExtraConfig.deserialize(deserializer);
     return new TransactionInnerPayloadV1(executable, extra_config);
   }
 }
@@ -627,7 +627,7 @@ export class TransactionExecutableScript extends TransactionExecutable {
   }
 
   static load(deserializer: Deserializer): TransactionExecutableScript {
-    let script = Script.deserialize(deserializer);
+    const script = Script.deserialize(deserializer);
     return new TransactionExecutableScript(script);
   }
 }
@@ -646,7 +646,7 @@ export class TransactionExecutableEntryFunction extends TransactionExecutable {
   }
 
   static load(deserializer: Deserializer): TransactionExecutableEntryFunction {
-    let entryFunction = EntryFunction.deserialize(deserializer);
+    const entryFunction = EntryFunction.deserialize(deserializer);
     return new TransactionExecutableEntryFunction(entryFunction);
   }
 }
@@ -656,6 +656,7 @@ export class TransactionExecutableEmpty extends TransactionExecutable {
     serializer.serializeU32AsUleb128(TransactionExecutableVariants.Empty);
   }
 
+  // eslint-disable-next-line @typescript-eslint/no-unused-vars
   static load(_: Deserializer): TransactionExecutableEmpty {
     return new TransactionExecutableEmpty();
   }

package/src/transactions/management/accountSequenceNumber.ts

@@ -111,7 +111,6 @@ export class AccountSequenceNumber {
    * @category Transactions
    */
   async nextSequenceNumber(): Promise<bigint | null> {
-    /* eslint-disable no-await-in-loop */
     while (this.lock) {
       await sleep(this.sleepTime);
     }
@@ -195,7 +194,6 @@ export class AccountSequenceNumber {
   async synchronize(): Promise<void> {
     if (this.lastUncommintedNumber === this.currentNumber) return;
 
-    /* eslint-disable no-await-in-loop */
     while (this.lock) {
       await sleep(this.sleepTime);
     }

package/src/transactions/management/transactionWorker.ts

@@ -1,5 +1,3 @@
-/* eslint-disable no-await-in-loop */
-
 import EventEmitter from "eventemitter3";
 import { AptosConfig } from "../../api/aptosConfig";
 import { Account } from "../../account";
@@ -11,6 +9,12 @@ import { AccountSequenceNumber } from "./accountSequenceNumber";
 import { AsyncQueue, AsyncQueueCancelledError } from "./asyncQueue";
 import { SimpleTransaction } from "../instances/simpleTransaction";
 
+/**
+ * Maximum number of transactions to keep in the history arrays to prevent unbounded memory growth.
+ * When this limit is exceeded, the oldest entries are removed.
+ */
+const MAX_TRANSACTION_HISTORY_SIZE = 10000;
+
 /**
  * @group Implementation
  * @category Transactions
@@ -124,19 +128,35 @@ export class TransactionWorker extends EventEmitter<TransactionWorkerEvents> {
   outstandingTransactions = new AsyncQueue<[Promise<PendingTransactionResponse>, bigint]>();
 
   /**
-   * transactions that have been submitted to chain
+   * transactions that have been submitted to chain.
+   * Limited to MAX_TRANSACTION_HISTORY_SIZE entries to prevent unbounded memory growth.
    * @group Implementation
    * @category Transactions
    */
   sentTransactions: Array<[string, bigint, any]> = [];
 
   /**
-   * transactions that have been committed to chain
+   * transactions that have been committed to chain.
+   * Limited to MAX_TRANSACTION_HISTORY_SIZE entries to prevent unbounded memory growth.
    * @group Implementation
    * @category Transactions
    */
   executedTransactions: Array<[string, bigint, any]> = [];
 
+  /**
+   * Adds a transaction to the history array while enforcing the maximum size limit.
+   * Removes the oldest entries when the limit is exceeded.
+   * @private
+   */
+  private addToTransactionHistory(history: Array<[string, bigint, any]>, entry: [string, bigint, any]): void {
+    history.push(entry);
+    // Remove oldest entries if we exceed the limit (remove ~10% when triggered)
+    if (history.length > MAX_TRANSACTION_HISTORY_SIZE) {
+      const removeCount = Math.ceil(MAX_TRANSACTION_HISTORY_SIZE * 0.1);
+      history.splice(0, removeCount);
+    }
+  }
+
   /**
    * Initializes a new instance of the class, providing a framework for receiving payloads to be processed.
    *
@@ -181,7 +201,6 @@ export class TransactionWorker extends EventEmitter<TransactionWorkerEvents> {
    */
   async submitNextTransaction() {
     try {
-      /* eslint-disable no-constant-condition */
       while (true) {
         const sequenceNumber = await this.accountSequnceNumber.nextSequenceNumber();
         if (sequenceNumber === null) return;
@@ -217,7 +236,6 @@ export class TransactionWorker extends EventEmitter<TransactionWorkerEvents> {
    */
   async processTransactions() {
     try {
-      /* eslint-disable no-constant-condition */
       while (true) {
         const awaitingTransactions = [];
         const sequenceNumbers = [];
@@ -240,7 +258,7 @@ export class TransactionWorker extends EventEmitter<TransactionWorkerEvents> {
           sequenceNumber = sequenceNumbers[i];
           if (sentTransaction.status === promiseFulfilledStatus) {
             // transaction sent to chain
-            this.sentTransactions.push([sentTransaction.value.hash, sequenceNumber, null]);
+            this.addToTransactionHistory(this.sentTransactions, [sentTransaction.value.hash, sequenceNumber, null]);
             // check sent transaction execution
             this.emit(TransactionWorkerEventsEnum.TransactionSent, {
               message: `transaction hash ${sentTransaction.value.hash} has been committed to chain`,
@@ -249,7 +267,11 @@ export class TransactionWorker extends EventEmitter<TransactionWorkerEvents> {
             await this.checkTransaction(sentTransaction, sequenceNumber);
           } else {
             // send transaction failed
-            this.sentTransactions.push([sentTransaction.status, sequenceNumber, sentTransaction.reason]);
+            this.addToTransactionHistory(this.sentTransactions, [
+              sentTransaction.status,
+              sequenceNumber,
+              sentTransaction.reason,
+            ]);
             this.emit(TransactionWorkerEventsEnum.TransactionSendFailed, {
               message: `failed to commit transaction ${this.sentTransactions.length} with error ${sentTransaction.reason}`,
               error: sentTransaction.reason,
@@ -285,14 +307,22 @@ export class TransactionWorker extends EventEmitter<TransactionWorkerEvents> {
         const executedTransaction = sentTransactions[i];
         if (executedTransaction.status === promiseFulfilledStatus) {
           // transaction executed to chain
-          this.executedTransactions.push([executedTransaction.value.hash, sequenceNumber, null]);
+          this.addToTransactionHistory(this.executedTransactions, [
+            executedTransaction.value.hash,
+            sequenceNumber,
+            null,
+          ]);
           this.emit(TransactionWorkerEventsEnum.TransactionExecuted, {
             message: `transaction hash ${executedTransaction.value.hash} has been executed on chain`,
             transactionHash: sentTransaction.value.hash,
           });
         } else {
           // transaction execution failed
-          this.executedTransactions.push([executedTransaction.status, sequenceNumber, executedTransaction.reason]);
+          this.addToTransactionHistory(this.executedTransactions, [
+            executedTransaction.status,
+            sequenceNumber,
+            executedTransaction.reason,
+          ]);
           this.emit(TransactionWorkerEventsEnum.TransactionExecutionFailed, {
             message: `failed to execute transaction ${this.executedTransactions.length} with error ${executedTransaction.reason}`,
             error: executedTransaction.reason,

package/src/transactions/transactionBuilder/remoteAbi.ts

@@ -76,6 +76,8 @@ import {
   isBcsI256,
 } from "./helpers";
 import { MoveFunction, MoveModule } from "../../types";
+import { warnIfDevelopment } from "../../utils/helpers";
+import { memoizeAsync } from "../../utils/memoize";
 
 const TEXT_ENCODER = new TextEncoder();
 
@@ -99,8 +101,15 @@ export function standardizeTypeTags(typeArguments?: Array<TypeArgument>): Array<
   );
 }
 
+/**
+ * Cache TTL for module ABIs in milliseconds (5 minutes).
+ * ABIs rarely change, so a longer cache is acceptable.
+ */
+const MODULE_ABI_CACHE_TTL_MS = 5 * 60 * 1000;
+
 /**
  * Fetches the ABI of a specified module from the on-chain module ABI.
+ * Results are cached for 5 minutes to reduce redundant network calls.
  *
  * @param moduleAddress - The address of the module from which to fetch the ABI.
  * @param moduleName - The name of the module containing the ABI.
@@ -113,8 +122,16 @@ export async function fetchModuleAbi(
   moduleName: string,
   aptosConfig: AptosConfig,
 ): Promise<MoveModule | undefined> {
-  const moduleBytecode = await getModule({ aptosConfig, accountAddress: moduleAddress, moduleName });
-  return moduleBytecode.abi;
+  const cacheKey = `module-abi-${aptosConfig.network}-${moduleAddress}-${moduleName}`;
+
+  return memoizeAsync(
+    async () => {
+      const moduleBytecode = await getModule({ aptosConfig, accountAddress: moduleAddress, moduleName });
+      return moduleBytecode.abi;
+    },
+    cacheKey,
+    MODULE_ABI_CACHE_TTL_MS,
+  )();
 }
 
 /**
@@ -628,10 +645,8 @@ function parseArg(
     }
 
     if (arg instanceof Uint8Array && options?.allowUnknownStructs) {
-      // eslint-disable-next-line no-console
-      console.warn(
-        // eslint-disable-next-line max-len
-        `Unsupported struct input type for argument ${position}. Continuing since 'allowUnknownStructs' is enabled.`,
+      warnIfDevelopment(
+        `[Aptos SDK] Unsupported struct input type for argument ${position}. Continuing since 'allowUnknownStructs' is enabled.`,
       );
       return new FixedBytes(arg);
     }

package/src/transactions/transactionBuilder/transactionBuilder.ts

@@ -24,6 +24,7 @@ import { getInfo } from "../../internal/utils";
 import { getLedgerInfo } from "../../internal/general";
 import { getGasPriceEstimation } from "../../internal/transaction";
 import { NetworkToChainId } from "../../utils/apiEndpoints";
+import { MIN_MAX_GAS_AMOUNT } from "../../utils/const";
 import { normalizeBundle } from "../../utils/normalizeBundle";
 import {
   AccountAuthenticator,
@@ -201,7 +202,7 @@ export function generateTransactionPayloadWithABI(
   }
 
   // Check all BCS types, and convert any non-BCS types
-  const functionArguments: Array<EntryFunctionArgumentTypes> = args.functionArguments.map((arg, i) =>
+  const functionArguments: Array<EntryFunctionArgumentTypes> | undefined = args.functionArguments?.map((arg, i) =>
     /**
      * Converts the argument for a specified function using its ABI and type arguments.
      * This function helps ensure that the correct number of arguments is provided for the function call.
@@ -220,10 +221,9 @@ export function generateTransactionPayloadWithABI(
   );
 
   // Check that all arguments are accounted for
-  if (functionArguments.length !== functionAbi.parameters.length) {
+  if ((functionArguments?.length ?? 0) !== functionAbi.parameters.length) {
     throw new Error(
-      // eslint-disable-next-line max-len
-      `Too few arguments for '${moduleAddress}::${moduleName}::${functionName}', expected ${functionAbi.parameters.length} but got ${functionArguments.length}`,
+      `Too few arguments for '${moduleAddress}::${moduleName}::${functionName}', expected ${functionAbi.parameters.length} but got ${functionArguments?.length ?? 0}`,
     );
   }
 
@@ -232,7 +232,7 @@ export function generateTransactionPayloadWithABI(
     `${moduleAddress}::${moduleName}`,
     functionName,
     typeArguments,
-    functionArguments,
+    functionArguments ?? [],
   );
 
   // Send it as multi sig if it's a multisig payload
@@ -314,7 +314,6 @@ export function generateViewFunctionPayloadWithABI(args: InputViewFunctionDataWi
   // Check that all arguments are accounted for
   if (functionArguments.length !== functionAbi.parameters.length) {
     throw new Error(
-      // eslint-disable-next-line max-len
       `Too few arguments for '${moduleAddress}::${moduleName}::${functionName}', expected ${functionAbi.parameters.length} but got ${functionArguments.length}`,
     );
   }
@@ -392,7 +391,8 @@ export async function generateRawTransaction(args: {
     const getSequenceNumber = async () => {
       if (options?.accountSequenceNumber !== undefined) {
         return options.accountSequenceNumber;
-      } else if (options?.replayProtectionNonce !== undefined) {
+      }
+      if (options?.replayProtectionNonce !== undefined) {
         // If replay nonce is provided, use it as the sequence number
         // This is an unused value, so it's specifically to show that the sequence number is not used
         return 0xdeadbeefn;
@@ -413,7 +413,7 @@ export async function generateRawTransaction(args: {
       try {
         // Check if main signer has been created on chain, if not assign sequence number 0
         return await getSequenceNumber();
-      } catch (e: any) {
+      } catch {
         return 0;
       }
     } else {
@@ -426,8 +426,11 @@ export async function generateRawTransaction(args: {
     getSequenceNumberForAny(),
   ]);
 
+  const userMaxGas = options?.maxGasAmount
+    ? BigInt(options.maxGasAmount)
+    : BigInt(aptosConfig.getDefaultMaxGasAmount());
   const { maxGasAmount, gasUnitPrice, expireTimestamp, replayProtectionNonce } = {
-    maxGasAmount: options?.maxGasAmount ? BigInt(options.maxGasAmount) : BigInt(aptosConfig.getDefaultMaxGasAmount()),
+    maxGasAmount: userMaxGas < BigInt(MIN_MAX_GAS_AMOUNT) ? BigInt(MIN_MAX_GAS_AMOUNT) : userMaxGas,
     gasUnitPrice: options?.gasUnitPrice ?? BigInt(gasEstimate),
     expireTimestamp:
       options?.expireTimestamp ?? BigInt(Math.floor(Date.now() / 1000) + aptosConfig.getDefaultTxnExpirySecFromNow()),
@@ -460,12 +463,14 @@ export function convertPayloadToInnerPayload(
       new TransactionExecutableScript(payload.script),
       new TransactionExtraConfigV1(undefined, replayProtectionNonce),
     );
-  } else if (payload instanceof TransactionPayloadEntryFunction) {
+  }
+  if (payload instanceof TransactionPayloadEntryFunction) {
     return new TransactionInnerPayloadV1(
       new TransactionExecutableEntryFunction(payload.entryFunction),
       new TransactionExtraConfigV1(undefined, replayProtectionNonce),
     );
-  } else if (payload instanceof TransactionPayloadMultiSig) {
+  }
+  if (payload instanceof TransactionPayloadMultiSig) {
     const innerPayload = payload.multiSig.transaction_payload;
     let executable: TransactionExecutable;
     if (innerPayload === undefined || innerPayload?.transaction_payload === undefined) {
@@ -481,9 +486,8 @@ export function convertPayloadToInnerPayload(
       executable,
       new TransactionExtraConfigV1(payload.multiSig.multisig_address, replayProtectionNonce),
     );
-  } else {
-    throw new Error(`Unsupported payload type: ${payload}`);
   }
+  throw new Error(`Unsupported payload type: ${payload}`);
 }
 
 /**

package/src/transactions/typeTag/index.ts

@@ -2,7 +2,7 @@
 // SPDX-License-Identifier: Apache-2.0
 
 /* eslint-disable @typescript-eslint/no-unused-vars */
-/* eslint-disable class-methods-use-this */
+
 /* eslint-disable max-classes-per-file */
 import { Deserializer } from "../../bcs/deserializer";
 import { Serializable, Serializer } from "../../bcs/serializer";

package/src/transactions/typeTag/parser.ts

@@ -299,7 +299,6 @@ export function parseTypeTag(typeStr: string, options?: { allowGenerics?: boolea
         throw new TypeTagParserError(typeStr, TypeTagParserErrorType.UnexpectedWhitespaceCharacter);
       }
 
-      // eslint-disable-next-line no-continue
       continue;
     } else {
       // Any other characters just append to the current string
@@ -415,7 +414,7 @@ function parseTypeTagInner(str: string, types: Array<TypeTag>, allowGenerics: bo
       let address: AccountAddress;
       try {
         address = AccountAddress.fromString(structParts[0]);
-      } catch (error: any) {
+      } catch {
         throw new TypeTagParserError(str, TypeTagParserErrorType.InvalidAddress);
       }
 

package/src/transactions/types.ts

@@ -204,7 +204,7 @@ export type InputGenerateTransactionPayloadDataWithRemoteABI =
 export type InputEntryFunctionData = {
   function: MoveFunctionId;
   typeArguments?: Array<TypeArgument>;
-  functionArguments: Array<EntryFunctionArgumentTypes | SimpleEntryFunctionArgumentTypes>;
+  functionArguments?: Array<EntryFunctionArgumentTypes | SimpleEntryFunctionArgumentTypes>;
   abi?: EntryFunctionABI;
 };