@msafe/sui3-sdk 1.0.17 → 1.0.18

package/dist/index.d.cts

@@ -366,12 +366,6 @@ declare function selectGasFunding(input: {
     owner: string;
     gasBudget: bigint;
 }): Promise<GasFundingResult>;
-/**
- * Estimate gas budget (no storageRebate) using a probe build/simulate.
- * Prefers classic coin gas when coin objects exist so the estimate matches the
- * cheaper payment path. Budget is left unset so Mysten can dry-run with MAX_GAS
- * overrides (avoids locking the full balance away from GasCoin splits).
- */
 declare function estimateGasBudget(input: {
     tx: Transaction;
     suiClient: SuiGrpcClient;

package/dist/index.d.ts

@@ -366,12 +366,6 @@ declare function selectGasFunding(input: {
     owner: string;
     gasBudget: bigint;
 }): Promise<GasFundingResult>;
-/**
- * Estimate gas budget (no storageRebate) using a probe build/simulate.
- * Prefers classic coin gas when coin objects exist so the estimate matches the
- * cheaper payment path. Budget is left unset so Mysten can dry-run with MAX_GAS
- * overrides (avoids locking the full balance away from GasCoin splits).
- */
 declare function estimateGasBudget(input: {
     tx: Transaction;
     suiClient: SuiGrpcClient;

package/dist/index.js

@@ -528,13 +528,33 @@ async function selectGasFunding(input) {
   }
   throw new InsufficientGasFundsError(gasBudget, coinBalance, addressBalance);
 }
+function cloneWithClearedGasConfig(tx) {
+  const data = structuredClone(tx.getData());
+  data.gasData.budget = null;
+  data.gasData.price = null;
+  data.gasData.payment = null;
+  data.expiration = null;
+  return Transaction.from(JSON.stringify(data));
+}
+function isPositiveBudget(budget) {
+  if (budget == null) {
+    return false;
+  }
+  try {
+    return BigInt(budget) > 0n;
+  } catch {
+    return false;
+  }
+}
 async function estimateGasBudget(input) {
   const { tx, suiClient, owner, gasPrice } = input;
   const funding = await loadSuiGasFunding(suiClient, owner, tx);
   if (funding.total <= 0n) {
     throw new InsufficientGasFundsError(0n, funding.coinBalance, funding.addressBalance);
   }
-  const probe = Transaction.from(tx);
+  const { budget, payment } = tx.getData().gasData;
+  const mustResetGas = payment != null && payment.length === 0 || budget != null && !isPositiveBudget(budget);
+  const probe = mustResetGas ? cloneWithClearedGasConfig(tx) : Transaction.from(tx);
   probe.setSender(owner);
   probe.setGasPrice(gasPrice);
   if (funding.paymentCoins.length > 0) {
@@ -566,14 +586,21 @@ async function prepareGasFunding(input) {
   const { payment, budget: existingBudget } = tx.getData().gasData;
   const bakedAddressBalanceGas = payment != null && payment.length === 0;
   let { gasBudget } = input;
+  if (gasBudget != null && gasBudget <= 0n) {
+    gasBudget = void 0;
+  }
+  const existingPositive = isPositiveBudget(existingBudget) ? BigInt(existingBudget) : null;
   if (gasBudget == null) {
-    if (bakedAddressBalanceGas || existingBudget == null) {
+    if (bakedAddressBalanceGas || existingPositive == null) {
       gasBudget = await estimateGasBudget({ tx, suiClient, owner, gasPrice });
     } else {
-      gasBudget = BigInt(existingBudget);
+      gasBudget = existingPositive;
     }
   }
   tx.setGasBudget(gasBudget);
+  if (bakedAddressBalanceGas) {
+    tx.setExpiration(null);
+  }
   return selectGasFunding({ tx, suiClient, owner, gasBudget });
 }
 async function preferClassicGasPayment(input) {