Skip to content

Commit 4425ae5

Browse files
spalladinoclaude
andcommitted
fix(wallet-sdk): use Limit mana estimate for conservative fee prediction
Addresses PR #22116 review comment 1: wallet was defaulting to Target mana usage estimate which can underprice during congestion growth. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent 80bb627 commit 4425ae5

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

yarn-project/wallet-sdk/src/base-wallet/base_wallet.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -253,9 +253,9 @@ export abstract class BaseWallet implements Wallet {
253253
/**
254254
* Returns the worst-case min fee across predicted future slots.
255255
* Falls back to getCurrentMinFees if the node doesn't support getPredictedMinFees.
256-
* @param estimate - The mana usage estimate to use for fee prediction. Defaults to Target (steady state).
256+
* @param estimate - The mana usage estimate to use for fee prediction. Defaults to Limit for conservative estimation.
257257
*/
258-
protected async getMinFees(estimate: ManaUsageEstimate = ManaUsageEstimate.Target): Promise<GasFees> {
258+
protected async getMinFees(estimate: ManaUsageEstimate = ManaUsageEstimate.Limit): Promise<GasFees> {
259259
try {
260260
const predicted = await this.aztecNode.getPredictedMinFees(estimate);
261261
if (predicted.length === 0) {

0 commit comments

Comments
 (0)