jerry/agents-marketplace
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

shared/tx: bump live-tx setMinFee floor 200K → 500K (all builders)

Browse Source 
The comment at every setMinFee call site said "500K is a comfortable
floor" but the actual value was 200K — the floor wasn't high enough to
cover lucid 0.4.30's Conway-era fee underestimation. Smoke-tested
mainnet Accept tx hit Ogmios code 3122: lucid declared 343,132 lovelace
fee but Ogmios required 347,726 (off by ~5K). With the floor at 200K
the floor was inert (343K > 200K), so the underestimated fee stood.

Bumping all six builder paths (PostEscrow, Claim/Submit, Accept,
Reclaim, PostAdvert, UpdateAdvert/RetireAdvert) to 500K — comfortably
above the empirical minimum across operations, ~0.15 AP3X surcharge
per tx, excess becomes change.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This commit is contained in:
david
2026-05-19 15:56:37 +02:00
parent 8244912806
commit 4f1ec9aa28
1 changed files with 6 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
packages/shared/src/tx/internal/liveCbor.ts
View File

@@ -374,7 +374,7 @@ export async function buildLiveTxForEscrow(
// lucid 0.4.30 under-estimates Conway fees by ~1K lovelace on // lucid 0.4.30 under-estimates Conway fees by ~1K lovelace on
// PostEscrow (Ogmios 3122). 500K is a comfortable floor; excess // PostEscrow (Ogmios 3122). 500K is a comfortable floor; excess
// becomes change. Same fix as Claim/Submit/Accept paths. // becomes change. Same fix as Claim/Submit/Accept paths.
.setMinFee(200_000n); .setMinFee(500_000n);
const completed = await txBuilder.complete(); const completed = await txBuilder.complete();
signed = await completed.sign.withWallet().complete(); signed = await completed.sign.withWallet().complete();
@@ -472,7 +472,7 @@ async function buildSpendOpenOrClaimedTx(opts: {
// without masking the savings from CIP-33 ref-scripts (natural // without masking the savings from CIP-33 ref-scripts (natural
// ref-script tx fee is ~0.24 AP3X; inlined is ~0.34 AP3X — both above // ref-script tx fee is ~0.24 AP3X; inlined is ~0.34 AP3X — both above
// the floor, so lucid's estimate wins and the size delta surfaces). // the floor, so lucid's estimate wins and the size delta surfaces).
.setMinFee(200_000n); .setMinFee(500_000n);
// Let lucid pick coins + compute fee normally; production wallets have // Let lucid pick coins + compute fee normally; production wallets have
// multiple real UTxOs (no need for the test-side leftover-as-fee hack). // multiple real UTxOs (no need for the test-side leftover-as-fee hack).
@@ -610,7 +610,7 @@ export async function buildLiveTxForAccept(params: LiveAcceptParams): Promise<Bu
.addSignerKey(params.buyerKey.pubKeyHash) .addSignerKey(params.buyerKey.pubKeyHash)
.validFrom(params.tipMs - 60_000) .validFrom(params.tipMs - 60_000)
.validTo(params.windowEnd) .validTo(params.windowEnd)
.setMinFee(200_000n); .setMinFee(500_000n);
// Same UPLC-eval delegation as Claim/Submit (see buildSpendOpenOrClaimedTx // Same UPLC-eval delegation as Claim/Submit (see buildSpendOpenOrClaimedTx
// for the rationale). lucid 0.4.30's CML WASM evaluator falsely rejects // for the rationale). lucid 0.4.30's CML WASM evaluator falsely rejects
@@ -678,7 +678,7 @@ export async function buildLiveTxForReclaim(params: LiveReclaimParams): Promise<
// already verified tipMs >= deliver_by, so a 60s back-pad is safe. // already verified tipMs >= deliver_by, so a 60s back-pad is safe.
.validFrom(params.tipMs - 60_000) .validFrom(params.tipMs - 60_000)
.validTo(params.tipMs + 60_000) .validTo(params.tipMs + 60_000)
.setMinFee(200_000n); .setMinFee(500_000n);
// Same UPLC eval delegation as Accept — lucid's CML evaluator falsely // Same UPLC eval delegation as Accept — lucid's CML evaluator falsely
// rejects valid Plutus V3 spends; route to Ogmios. // rejects valid Plutus V3 spends; route to Ogmios.
@@ -738,7 +738,7 @@ export async function buildLiveTxForAdvert(
.addSignerKey(walletKey.pubKeyHash) .addSignerKey(walletKey.pubKeyHash)
.validFrom(advertisedAt - 60_000) .validFrom(advertisedAt - 60_000)
.validTo(advertisedAt + 5 * 60_000) .validTo(advertisedAt + 5 * 60_000)
.setMinFee(200_000n); .setMinFee(500_000n);
const completed = await txBuilder.complete(); const completed = await txBuilder.complete();
signed = await completed.sign.withWallet().complete(); signed = await completed.sign.withWallet().complete();
@@ -801,7 +801,7 @@ export async function buildLiveTxForRetireAdvert(
// propagation lag the same as sibling builders. // propagation lag the same as sibling builders.
.validFrom(nowMs - 60_000) .validFrom(nowMs - 60_000)
.validTo(nowMs + 5 * 60_000) .validTo(nowMs + 5 * 60_000)
.setMinFee(200_000n); .setMinFee(500_000n);
// Same UPLC-eval delegation as Claim/Submit/Accept — see // Same UPLC-eval delegation as Claim/Submit/Accept — see
// buildSpendOpenOrClaimedTx for the lucid 0.4.30 false-rejection rationale. // buildSpendOpenOrClaimedTx for the lucid 0.4.30 false-rejection rationale.