Skip to content

fix: bump default fee model from 1 to 100 sat/KB to match ARC endpoint requirements#205

Draft
Copilot wants to merge 2 commits into
mainfrom
copilot/fix-default-feemodel-value
Draft

fix: bump default fee model from 1 to 100 sat/KB to match ARC endpoint requirements#205
Copilot wants to merge 2 commits into
mainfrom
copilot/fix-default-feemodel-value

Conversation

Copilot AI commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

Both ARC endpoints shipped with the library (arc.taal.com, arc.gorillapool.io) enforce a minimum of 100 sat/KB, but StorageProvider.defaultOptions() was returning { model: 'sat/kb', value: 1 }. Wallets created with defaults would broadcast transactions that silently go unmined.

Changes

  • StorageProvider.defaultOptions() — default feeModel.value: 1100
  • validateStorageFeeModel() — fallback value when no fee model is provided: 1100
  • Setup.createStorageKnex(), SetupClient.createStorageIdb(), MonitorDaemon.createSetup() — hardcoded inline fee model values updated to match
// Before
feeModel: { model: 'sat/kb', value: 1 }

// After
feeModel: { model: 'sat/kb', value: 100 }

Test utilities that explicitly set value: 1 for controlled fee-calculation unit tests are intentionally unchanged.

Copilot AI changed the title [WIP] Fix default feeModel value to 100 sat/KB fix: bump default fee model from 1 to 100 sat/KB to match ARC endpoint requirements Jun 17, 2026
Copilot AI requested a review from sirdeggen June 17, 2026 05:27
@sonarqubecloud

Copy link
Copy Markdown

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