Skip to content

fix(sdk-coin-flrp): pad base fee for Flare atomic exports - #9541

Merged
ArunBala-Bitgo merged 1 commit into
masterfrom
arunchockalingam504/cecho-2004-fix-flare-atomic-export-fee-calculation
Aug 24, 2026
Merged

fix(sdk-coin-flrp): pad base fee for Flare atomic exports#9541
ArunBala-Bitgo merged 1 commit into
masterfrom
arunchockalingam504/cecho-2004-fix-flare-atomic-export-fee-calculation

Conversation

@ArunBala-Bitgo

Copy link
Copy Markdown
Contributor

Summary

  • C-chain atomic exports call newExportTxFromBaseFee with the caller-supplied base fee unpadded, unlike atomic imports which apply a 10% buffer to absorb C-chain base-fee volatility between signing and broadcast (CECHO-1821).
  • This caused exports to broadcast with an underfunded fee once the C-chain base fee rose after signing, surfacing as an "insufficient funds" broadcast error despite adequate wallet balance.
  • Applies the same 10% padding used for imports to the export base fee in ExportInCTxBuilder.

Test plan

  • Added a regression test asserting the export fee reflects the 10% base-fee padding
  • npx mocha --grep "ExportInCTxBuilder" — 20 passing
  • npx tsc --noEmit — no errors

Ticket: CECHO-2004

🤖 Generated with Claude Code

C-chain exports call newExportTxFromBaseFee with the caller-supplied
base fee unpadded, unlike atomic imports which apply a 10% buffer to
absorb base-fee volatility between signing and broadcast (CECHO-1821).
This caused exports to broadcast with an underfunded fee once the
C-chain base fee rose after signing, surfacing as an "insufficient
funds" error despite adequate balance.

Apply the same 10% padding used for imports to the export base fee.

Ticket: CECHO-2004
@ArunBala-Bitgo
ArunBala-Bitgo requested a review from a team as a code owner August 24, 2026 03:14
@linear-code

linear-code Bot commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

CECHO-2004

@ArunBala-Bitgo
ArunBala-Bitgo merged commit 58b203b into master Aug 24, 2026
25 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants