xrpl 4.5.0-smartescrow.0 → 4.5.0-smartescrow.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "xrpl",
-  "version": "4.5.0-smartescrow.0",
+  "version": "4.5.0-smartescrow.1",
   "license": "ISC",
   "description": "A TypeScript/JavaScript API for interacting with the XRP Ledger in Node.js and the browser",
   "files": [
@@ -29,7 +29,7 @@
     "bignumber.js": "^9.0.0",
     "eventemitter3": "^5.0.1",
     "ripple-address-codec": "^5.0.0",
-    "ripple-binary-codec": "^2.6.0-smartescrow.0",
+    "ripple-binary-codec": "^2.6.0-smartescrow.1",
     "ripple-keypairs": "^2.0.0"
   },
   "devDependencies": {
@@ -47,8 +47,7 @@
     "ws": "^8.14.2"
   },
   "scripts": {
-    "build": "run-s build:lib build:snippets build:web",
-    "build:snippets": "tsc --build ./snippets/tsconfig.json",
+    "build": "run-s build:lib build:web",
     "build:lib": "tsc --build tsconfig.build.json",
     "build:web": "webpack",
     "build:browserTests": "webpack --config ./test/webpack.config.js",
@@ -64,11 +63,8 @@
     "test:browser": "npm run build && npm run build:browserTests && karma start ./karma.config.js",
     "test:watch": "jest --watch --config=jest.config.unit.js --verbose false --silent=false",
     "format": "prettier --write '{src,test}/**/*.ts'",
-    "lint": "eslint . --ext .ts --max-warnings 0",
-    "perf": "./scripts/perf_test.sh",
-    "compile:snippets": "tsc -p snippets/tsconfig.json",
-    "start:snippet": "npm run compile:snippets && node",
-    "inspect:snippet": "npm run compile:snippets && node inspect"
+    "lint": "eslint . --max-warnings 0",
+    "perf": "./scripts/perf_test.sh"
   },
   "prettier": "@xrplf/prettier-config",
   "repository": {
@@ -86,5 +82,5 @@
   "engines": {
     "node": ">=18.0.0"
   },
-  "gitHead": "d3fde692bb6d5875183bc3230ff70e40d366a32a"
+  "gitHead": "d93eff401a6632db1587569afa410ad125f14a1f"
 }

package/src/Wallet/fundWallet.ts

@@ -211,7 +211,7 @@ async function processError(response: Response, body): Promise<never> {
     new XRPLFaucetError(
       `Request failed: ${JSON.stringify({
         // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment -- json response could be anything
-        body: body || {},
+        body: body ?? {},
         contentType: response.headers.get('Content-Type'),
         statusCode: response.status,
       })}`,

package/src/Wallet/index.ts

@@ -252,6 +252,7 @@ export class Wallet {
       )
     }
 
+    // eslint-disable-next-line n/no-sync -- Using async would break fromMnemonic; this rule should be disabled entirely later.
     const seed = mnemonicToSeedSync(mnemonic)
     const masterNode = HDKey.fromMasterSeed(seed)
     const node = masterNode.derive(

package/src/client/connection.ts

@@ -301,7 +301,18 @@ export class Connection extends EventEmitter {
     >(request, timeout ?? this.config.timeout)
     this.trace('send', message)
     websocketSendAsync(this.ws, message).catch((error) => {
-      this.requestManager.reject(id, error)
+      try {
+        this.requestManager.reject(id, error)
+      } catch (err) {
+        if (err instanceof XrplError) {
+          this.trace(
+            'send',
+            `send errored after connection was closed: ${err.toString()}`,
+          )
+        } else {
+          this.trace('send', String(err))
+        }
+      }
     })
 
     return responsePromise

package/src/client/index.ts

@@ -132,16 +132,16 @@ type RequestNextPageType =
 type RequestNextPageReturnMap<T> = T extends AccountChannelsRequest
   ? AccountChannelsResponse
   : T extends AccountLinesRequest
-  ? AccountLinesResponse
-  : T extends AccountObjectsRequest
-  ? AccountObjectsResponse
-  : T extends AccountOffersRequest
-  ? AccountOffersResponse
-  : T extends AccountTxRequest
-  ? AccountTxResponse
-  : T extends LedgerDataRequest
-  ? LedgerDataResponse
-  : never
+    ? AccountLinesResponse
+    : T extends AccountObjectsRequest
+      ? AccountObjectsResponse
+      : T extends AccountOffersRequest
+        ? AccountOffersResponse
+        : T extends AccountTxRequest
+          ? AccountTxResponse
+          : T extends LedgerDataRequest
+            ? LedgerDataResponse
+            : never
 
 /**
  * Get the response key / property name that contains the listed data for a
@@ -476,7 +476,7 @@ class Client extends EventEmitter<EventTypes> {
      * If limit is not provided, fetches all data over multiple requests.
      * NOTE: This may return much more than needed. Set limit when possible.
      */
-    const countTo: number = request.limit == null ? Infinity : request.limit
+    const countTo: number = request.limit ?? Infinity
     let count = 0
     let marker: unknown = request.marker
     const results: U[] = []
@@ -673,9 +673,7 @@ class Client extends EventEmitter<EventTypes> {
     tx.Flags = convertTxFlagsToNumber(tx)
 
     const promises: Array<Promise<void>> = []
-    if (tx.NetworkID == null) {
-      tx.NetworkID = txNeedsNetworkID(this) ? this.networkID : undefined
-    }
+    tx.NetworkID ??= txNeedsNetworkID(this) ? this.networkID : undefined
     if (tx.Sequence == null) {
       promises.push(setNextValidSequenceNumber(this, tx))
     }
@@ -884,6 +882,7 @@ class Client extends EventEmitter<EventTypes> {
    * @param transaction - A {@link Transaction} in JSON format
    * @param signersCount - The expected number of signers for this transaction.
    * Only used for multisigned transactions.
+   * @returns The prepared transaction with required fields autofilled.
    * @deprecated Use autofill instead, provided for users familiar with v1
    */
   public async prepareTransaction(

package/src/client/partialPayment.ts

@@ -68,6 +68,7 @@ function amountsEqual(
 /* eslint-enable complexity */
 /* eslint-enable @typescript-eslint/consistent-type-assertions */
 
+/* eslint-disable complexity -- required here for multiple checks */
 function isPartialPayment(
   tx?: Transaction,
   metadata?: TransactionMetadata | string,
@@ -105,6 +106,7 @@ function isPartialPayment(
 
   return !amountsEqual(delivered, amount)
 }
+/* eslint-enable complexity */
 
 function txHasPartialPayment(response: TxResponse): boolean {
   return isPartialPayment(response.result.tx_json, response.result.meta)

package/src/models/ledger/FeeSettings.ts

@@ -25,9 +25,11 @@ export interface FeeSettingsPostAmendmentFields {
   /** The incremental owner reserve for owning objects, as drops of XRP. */
   ReserveIncrementDrops: string
 
-  ExtensionComputeLimit: number
+  ExtensionComputeLimit?: number
 
-  ExtensionSizeLimit: number
+  ExtensionSizeLimit?: number
+
+  GasPrice?: number
 }
 
 export interface FeeSettingsBase

package/src/models/methods/index.ts

@@ -1,5 +1,6 @@
 /* eslint-disable no-inline-comments -- Necessary for important note */
 /* eslint-disable max-lines -- There is a lot to export */
+/* eslint-disable prettier/prettier -- Required here to keep formatting in line */
 import type { APIVersion, DEFAULT_API_VERSION } from '../common'
 
 import {

package/src/models/methods/subscribe.ts

@@ -158,9 +158,12 @@ export interface LedgerStreamResponse {
    * If this ledger version includes a SetFee pseudo-transaction the new.
    * Transaction cost applies starting with the following ledger version.
    */
+  extension_compute?: number
+  extension_size?: number
   fee_base: number
   /** The reference transaction cost in "fee units". This is not returned after the SetFees amendment is enabled. */
   fee_ref?: number
+  gas_price?: string
   /** The identifying hash of the ledger version that was closed. */
   ledger_hash: string
   /** The ledger index of the ledger that was closed. */
@@ -506,22 +509,22 @@ export type EventTypes =
 export type OnEventToListenerMap<T extends EventTypes> = T extends 'connected'
   ? () => void
   : T extends 'disconnected'
-  ? (code: number) => void
-  : T extends 'ledgerClosed'
-  ? (ledger: LedgerStream) => void
-  : T extends 'validationReceived'
-  ? (validation: ValidationStream) => void
-  : T extends 'transaction'
-  ? (transaction: TransactionStream) => void
-  : T extends 'peerStatusChange'
-  ? (peerStatus: PeerStatusStream) => void
-  : T extends 'consensusPhase'
-  ? (consensus: ConsensusStream) => void
-  : T extends 'manifestReceived'
-  ? (manifest: ManifestRequest) => void
-  : T extends 'path_find'
-  ? (path: PathFindStream) => void
-  : T extends 'error'
-  ? // eslint-disable-next-line @typescript-eslint/no-explicit-any -- needs to be any for overload
-    (...err: any[]) => void
-  : (...args: never[]) => void
+    ? (code: number) => void
+    : T extends 'ledgerClosed'
+      ? (ledger: LedgerStream) => void
+      : T extends 'validationReceived'
+        ? (validation: ValidationStream) => void
+        : T extends 'transaction'
+          ? (transaction: TransactionStream) => void
+          : T extends 'peerStatusChange'
+            ? (peerStatus: PeerStatusStream) => void
+            : T extends 'consensusPhase'
+              ? (consensus: ConsensusStream) => void
+              : T extends 'manifestReceived'
+                ? (manifest: ManifestRequest) => void
+                : T extends 'path_find'
+                  ? (path: PathFindStream) => void
+                  : T extends 'error'
+                    ? // eslint-disable-next-line @typescript-eslint/no-explicit-any -- needs to be any for overload
+                      (...err: any[]) => void
+                    : (...args: never[]) => void

package/src/models/transactions/MPTokenIssuanceCreate.ts

@@ -173,7 +173,7 @@ export function validateMPTokenIssuanceCreate(
     const isTfMPTCanTransfer =
       typeof flags === 'number'
         ? isFlagEnabled(flags, MPTokenIssuanceCreateFlags.tfMPTCanTransfer)
-        : flags.tfMPTCanTransfer ?? false
+        : (flags.tfMPTCanTransfer ?? false)
 
     if (tx.TransferFee < 0 || tx.TransferFee > MAX_TRANSFER_FEE) {
       throw new ValidationError(

package/src/models/transactions/MPTokenIssuanceSet.ts

@@ -73,12 +73,12 @@ export function validateMPTokenIssuanceSet(tx: Record<string, unknown>): void {
   const isTfMPTLock =
     typeof flags === 'number'
       ? isFlagEnabled(flags, MPTokenIssuanceSetFlags.tfMPTLock)
-      : flags.tfMPTLock ?? false
+      : (flags.tfMPTLock ?? false)
 
   const isTfMPTUnlock =
     typeof flags === 'number'
       ? isFlagEnabled(flags, MPTokenIssuanceSetFlags.tfMPTUnlock)
-      : flags.tfMPTUnlock ?? false
+      : (flags.tfMPTUnlock ?? false)
 
   if (isTfMPTLock && isTfMPTUnlock) {
     throw new ValidationError('MPTokenIssuanceSet: flag conflict')

package/src/models/transactions/NFTokenCreateOffer.ts

@@ -151,7 +151,7 @@ export function validateNFTokenCreateOffer(tx: Record<string, unknown>): void {
   const isTfSellNFToken =
     typeof flags === 'number'
       ? isFlagEnabled(flags, NFTokenCreateOfferFlags.tfSellNFToken)
-      : flags.tfSellNFToken ?? false
+      : (flags.tfSellNFToken ?? false)
 
   if (isTfSellNFToken) {
     validateNFTokenSellOfferCases(tx)

package/src/models/transactions/batch.ts

@@ -7,8 +7,10 @@ import {
   GlobalFlags,
   GlobalFlagsInterface,
   isArray,
+  isNull,
   isRecord,
   isString,
+  isValue,
   validateBaseTransaction,
   validateOptionalField,
   validateRequiredField,
@@ -40,18 +42,6 @@ export interface BatchFlagsInterface extends GlobalFlagsInterface {
   tfIndependent?: boolean
 }
 
-export type BatchInnerTransaction = SubmittableTransaction & {
-  Fee?: '0'
-
-  SigningPubKey?: ''
-
-  TxnSignature?: never
-
-  Signers?: never
-
-  LastLedgerSequence?: never
-}
-
 export interface BatchSigner {
   BatchSigner: {
     Account: string
@@ -73,10 +63,48 @@ export interface Batch extends BaseTransaction {
   BatchSigners?: BatchSigner[]
 
   RawTransactions: Array<{
-    RawTransaction: BatchInnerTransaction
+    RawTransaction: SubmittableTransaction
   }>
 }
 
+function validateBatchInnerTransaction(
+  tx: Record<string, unknown>,
+  index: number,
+): void {
+  if (tx.TransactionType === 'Batch') {
+    throw new ValidationError(
+      `Batch: RawTransactions[${index}] is a Batch transaction. Cannot nest Batch transactions.`,
+    )
+  }
+
+  // Check for the `tfInnerBatchTxn` flag in the inner transactions
+  if (!hasFlag(tx, GlobalFlags.tfInnerBatchTxn, 'tfInnerBatchTxn')) {
+    throw new ValidationError(
+      `Batch: RawTransactions[${index}] must contain the \`tfInnerBatchTxn\` flag.`,
+    )
+  }
+  validateOptionalField(tx, 'Fee', isValue('0'), {
+    paramName: `RawTransactions[${index}].RawTransaction.Fee`,
+    txType: 'Batch',
+  })
+  validateOptionalField(tx, 'SigningPubKey', isValue(''), {
+    paramName: `RawTransactions[${index}].RawTransaction.SigningPubKey`,
+    txType: 'Batch',
+  })
+  validateOptionalField(tx, 'TxnSignature', isNull, {
+    paramName: `RawTransactions[${index}].RawTransaction.TxnSignature`,
+    txType: 'Batch',
+  })
+  validateOptionalField(tx, 'Signers', isNull, {
+    paramName: `RawTransactions[${index}].RawTransaction.Signers`,
+    txType: 'Batch',
+  })
+  validateOptionalField(tx, 'LastLedgerSequence', isNull, {
+    paramName: `RawTransactions[${index}].RawTransaction.LastLedgerSequence`,
+    txType: 'Batch',
+  })
+}
+
 /**
  * Verify the form and type of a Batch at runtime.
  *
@@ -101,18 +129,7 @@ export function validateBatch(tx: Record<string, unknown>): void {
     })
 
     const rawTx = rawTxObj.RawTransaction
-    if (rawTx.TransactionType === 'Batch') {
-      throw new ValidationError(
-        `Batch: RawTransactions[${index}] is a Batch transaction. Cannot nest Batch transactions.`,
-      )
-    }
-
-    // Check for the `tfInnerBatchTxn` flag in the inner transactions
-    if (!hasFlag(rawTx, GlobalFlags.tfInnerBatchTxn, 'tfInnerBatchTxn')) {
-      throw new ValidationError(
-        `Batch: RawTransactions[${index}] must contain the \`tfInnerBatchTxn\` flag.`,
-      )
-    }
+    validateBatchInnerTransaction(rawTx, index)
 
     // Full validation of each `RawTransaction` object is done in `validate` to avoid dependency cycles
   })
@@ -132,19 +149,19 @@ export function validateBatch(tx: Record<string, unknown>): void {
 
     const signer = signerRecord.BatchSigner
     validateRequiredField(signer, 'Account', isString, {
-      paramName: `BatchSigners[${index}].Account`,
+      paramName: `BatchSigners[${index}].BatchSigner.Account`,
       txType: 'Batch',
     })
     validateOptionalField(signer, 'SigningPubKey', isString, {
-      paramName: `BatchSigners[${index}].SigningPubKey`,
+      paramName: `BatchSigners[${index}].BatchSigner.SigningPubKey`,
       txType: 'Batch',
     })
     validateOptionalField(signer, 'TxnSignature', isString, {
-      paramName: `BatchSigners[${index}].TxnSignature`,
+      paramName: `BatchSigners[${index}].BatchSigner.TxnSignature`,
       txType: 'Batch',
     })
     validateOptionalField(signer, 'Signers', isArray, {
-      paramName: `BatchSigners[${index}].Signers`,
+      paramName: `BatchSigners[${index}].BatchSigner.Signers`,
       txType: 'Batch',
     })
   })

package/src/models/transactions/common.ts

@@ -152,6 +152,28 @@ export function isNumber(num: unknown): num is number {
   return typeof num === 'number'
 }
 
+/**
+ * Verify the form and type of a null value at runtime.
+ *
+ * @param inp - The value to check the form and type of.
+ * @returns Whether the value is properly formed.
+ */
+export function isNull(inp: unknown): inp is null {
+  return inp == null
+}
+
+/**
+ * Verify that a certain field has a certain exact value at runtime.
+ *
+ * @param value The value to compare against.
+ * @returns Whether the number is properly formed and within the bounds.
+ */
+export function isValue<V>(value: V): (inp: unknown) => inp is V {
+  // eslint-disable-next-line func-style -- returning a function
+  const isValueInternal = (inp: unknown): inp is V => inp === value
+  return isValueInternal
+}
+
 /**
  * Checks whether the given value is a valid XRPL number string.
  * Accepts integer, decimal, or scientific notation strings.

package/src/models/transactions/delegateSet.ts

@@ -9,7 +9,7 @@ import {
 } from './common'
 
 const PERMISSIONS_MAX_LENGTH = 10
-const NON_DELEGATABLE_TRANSACTIONS = new Set([
+const NON_DELEGABLE_TRANSACTIONS = new Set([
   'AccountSet',
   'SetRegularKey',
   'SignerListSet',
@@ -97,7 +97,7 @@ export function validateDelegateSet(tx: Record<string, unknown>): void {
     if (typeof permissionValue !== 'string') {
       throw new ValidationError('DelegateSet: PermissionValue must be a string')
     }
-    if (NON_DELEGATABLE_TRANSACTIONS.has(permissionValue)) {
+    if (NON_DELEGABLE_TRANSACTIONS.has(permissionValue)) {
       throw new ValidationError(
         `DelegateSet: PermissionValue contains a non-delegatable transaction ${permissionValue}`,
       )

package/src/models/transactions/escrowFinish.ts

@@ -9,7 +9,7 @@ import {
   validateRequiredField,
   MAX_AUTHORIZED_CREDENTIALS,
 } from './common'
-import { TransactionMetadataBase } from './metadata'
+import type { TransactionMetadataBase } from './metadata'
 
 /**
  * Deliver XRP from a held payment to the recipient.

package/src/models/transactions/metadata.ts

@@ -97,15 +97,15 @@ export type TransactionMetadata<T extends BaseTransaction = Transaction> =
   T extends Payment
     ? PaymentMetadata
     : T extends NFTokenMint
-    ? NFTokenMintMetadata
-    : T extends NFTokenCreateOffer
-    ? NFTokenCreateOfferMetadata
-    : T extends NFTokenAcceptOffer
-    ? NFTokenAcceptOfferMetadata
-    : T extends NFTokenCancelOffer
-    ? NFTokenCancelOfferMetadata
-    : T extends MPTokenIssuanceCreate
-    ? MPTokenIssuanceCreateMetadata
-    : T extends EscrowFinish
-    ? EscrowFinishMetadata
-    : TransactionMetadataBase
+      ? NFTokenMintMetadata
+      : T extends NFTokenCreateOffer
+        ? NFTokenCreateOfferMetadata
+        : T extends NFTokenAcceptOffer
+          ? NFTokenAcceptOfferMetadata
+          : T extends NFTokenCancelOffer
+            ? NFTokenCancelOfferMetadata
+            : T extends MPTokenIssuanceCreate
+              ? MPTokenIssuanceCreateMetadata
+              : T extends EscrowFinish
+                ? EscrowFinishMetadata
+                : TransactionMetadataBase

package/src/models/transactions/payment.ts

@@ -241,7 +241,7 @@ function checkPartialPayment(tx: Record<string, unknown>): void {
     const isTfPartialPayment =
       typeof flags === 'number'
         ? isFlagEnabled(flags, PaymentFlags.tfPartialPayment)
-        : flags.tfPartialPayment ?? false
+        : (flags.tfPartialPayment ?? false)
 
     if (!isTfPartialPayment) {
       throw new ValidationError(

package/src/models/transactions/setFee.ts

@@ -32,6 +32,12 @@ export interface SetFeePostAmendment extends BaseTransaction {
    * The incremental reserve, in drops
    */
   ReserveIncrementDrops: string
+
+  ExtensionComputeLimit?: number
+
+  ExtensionSizeLimit?: number
+
+  GasPrice?: number
 }
 
 /**

package/src/models/utils/flags.ts

@@ -118,7 +118,7 @@ export function convertTxFlagsToNumber(tx: Transaction): number {
       }
 
       return txFlags[flag]
-        ? resultFlags | (flagEnum[flag] || GlobalFlags[flag])
+        ? resultFlags | (flagEnum[flag] ?? GlobalFlags[flag])
         : resultFlags
     }, 0)
   }

package/src/sugar/autofill.ts

@@ -21,6 +21,9 @@ const REQUIRED_NETWORKID_VERSION = '1.11.0'
 
 const MICRO_DROPS_PER_DROP = 1_000_000
 
+const WASM_FIXED_UPLOAD_COST = 100
+const WASM_DROPS_PER_BYTE = 5
+
 /**
  * Determines whether the source rippled version is not later than the target rippled version.
  * Example usage: isNotLaterRippledVersion('1.10.0', '1.11.0') returns true.
@@ -279,7 +282,7 @@ async function fetchGasPrice(client: Client): Promise<BigNumber> {
  * @param [signersCount=0] - The number of signers (default is 0). Only used for multisigning.
  * @returns A promise that returns the fee.
  */
-// eslint-disable-next-line max-lines-per-function -- needed here due to the complexity of the fee calculation
+// eslint-disable-next-line max-lines-per-function, complexity -- necessary to check for many transaction types.
 async function calculateFeePerTransactionType(
   client: Client,
   tx: Transaction,
@@ -295,7 +298,9 @@ async function calculateFeePerTransactionType(
 
   // EscrowCreate transaction with FinishFunction
   if (tx.TransactionType === 'EscrowCreate' && tx.FinishFunction != null) {
-    baseFee = baseFee.plus(1000)
+    baseFee = baseFee
+      .plus(WASM_FIXED_UPLOAD_COST)
+      .plus((WASM_DROPS_PER_BYTE * tx.FinishFunction.length) / 2)
   } else if (tx.TransactionType === 'EscrowFinish') {
     // EscrowFinish Transaction with Fulfillment/ComputationAllowance
     if (tx.Fulfillment != null) {
@@ -314,14 +319,17 @@ async function calculateFeePerTransactionType(
       baseFee = baseFee.plus(extraFee)
     }
   } else if (tx.TransactionType === 'Batch') {
-    const rawTxFees = await tx.RawTransactions.reduce(async (acc, rawTxn) => {
-      const resolvedAcc = await acc
-      const fee = await calculateFeePerTransactionType(
-        client,
-        rawTxn.RawTransaction,
-      )
-      return BigNumber.sum(resolvedAcc, fee)
-    }, Promise.resolve(new BigNumber(0)))
+    const rawTxFees = await tx.RawTransactions.reduce(
+      async (acc, rawTxn) => {
+        const resolvedAcc = await acc
+        const fee = await calculateFeePerTransactionType(
+          client,
+          rawTxn.RawTransaction,
+        )
+        return BigNumber.sum(resolvedAcc, fee)
+      },
+      Promise.resolve(new BigNumber(0)),
+    )
     baseFee = BigNumber.sum(baseFee.times(2), rawTxFees)
   } else if (isSpecialTxCost) {
     baseFee = await fetchOwnerReserveFee(client)
@@ -417,12 +425,9 @@ export async function checkAccountDeleteBlockers(
  */
 export function handleDeliverMax(tx: Payment): void {
   if (tx.DeliverMax != null) {
-    // eslint-disable-next-line @typescript-eslint/no-unnecessary-condition -- needed here
-    if (tx.Amount == null) {
-      // If only DeliverMax is provided, use it to populate the Amount field
-      // eslint-disable-next-line no-param-reassign -- known RPC-level property
-      tx.Amount = tx.DeliverMax
-    }
+    // If only DeliverMax is provided, use it to populate the Amount field
+    // eslint-disable-next-line @typescript-eslint/no-unnecessary-condition, no-param-reassign -- needed here
+    tx.Amount ??= tx.DeliverMax
 
     // eslint-disable-next-line @typescript-eslint/no-unnecessary-condition -- needed here
     if (tx.Amount != null && tx.Amount !== tx.DeliverMax) {
@@ -450,7 +455,7 @@ export async function autofillBatchTxn(
 ): Promise<void> {
   const accountSequences: Record<string, number> = {}
 
-  for await (const rawTxn of tx.RawTransactions) {
+  for (const rawTxn of tx.RawTransactions) {
     const txn = rawTxn.RawTransaction
 
     // Sequence processing
@@ -459,6 +464,7 @@ export async function autofillBatchTxn(
         txn.Sequence = accountSequences[txn.Account]
         accountSequences[txn.Account] += 1
       } else {
+        // eslint-disable-next-line no-await-in-loop -- It has to wait
         const nextSequence = await getNextValidSequenceNumber(
           client,
           txn.Account,
@@ -472,28 +478,24 @@ export async function autofillBatchTxn(
 
     if (txn.Fee == null) {
       txn.Fee = '0'
-      // eslint-disable-next-line @typescript-eslint/no-unnecessary-condition -- needed for JS checks
     } else if (txn.Fee !== '0') {
       throw new XrplError('Must have `Fee of "0" in inner Batch transaction.')
     }
 
     if (txn.SigningPubKey == null) {
       txn.SigningPubKey = ''
-      // eslint-disable-next-line @typescript-eslint/no-unnecessary-condition -- needed for JS checks
     } else if (txn.SigningPubKey !== '') {
       throw new XrplError(
         'Must have `SigningPubKey` of "" in inner Batch transaction.',
       )
     }
 
-    // eslint-disable-next-line @typescript-eslint/no-unnecessary-condition -- needed for JS checks
     if (txn.TxnSignature != null) {
       throw new XrplError(
         'Must not have `TxnSignature` in inner Batch transaction.',
       )
     }
 
-    // eslint-disable-next-line @typescript-eslint/no-unnecessary-condition -- needed for JS checks
     if (txn.Signers != null) {
       throw new XrplError('Must not have `Signers` in inner Batch transaction.')
     }

package/src/sugar/getFeeXrp.ts

@@ -35,10 +35,8 @@ export default async function getFeeXrp(
   }
 
   const baseFeeXrp = new BigNumber(baseFee)
-  if (serverInfo.load_factor == null) {
-    // https://github.com/ripple/rippled/issues/3812#issuecomment-816871100
-    serverInfo.load_factor = 1
-  }
+  // https://github.com/ripple/rippled/issues/3812#issuecomment-816871100
+  serverInfo.load_factor ??= 1
   let fee = baseFeeXrp.times(serverInfo.load_factor).times(feeCushion)
 
   // Cap fee to `client.maxFeeXRP`

package/src/sugar/getOrderbook.ts

@@ -127,7 +127,7 @@ export function createBookOffersRequest(
     ledger_index: options.ledger_index ?? 'validated',
     ledger_hash: options.ledger_hash === null ? undefined : options.ledger_hash,
     limit: options.limit ?? DEFAULT_LIMIT,
-    taker: options.taker ? options.taker : undefined,
+    taker: options.taker ?? undefined,
   }
 
   return request