Skip to content

feat(transaction-controller): support saved gas fee levels#8993

Draft
pedronfigueiredo wants to merge 1 commit into
mainfrom
pnf/save-custom-gas-settings
Draft

feat(transaction-controller): support saved gas fee levels#8993
pedronfigueiredo wants to merge 1 commit into
mainfrom
pnf/save-custom-gas-settings

Conversation

@pedronfigueiredo
Copy link
Copy Markdown
Contributor

Explanation

This expands transaction-controller saved gas fee handling so clients can resolve saved preferences with full transaction metadata instead of only a chain ID. That lets consumers key saved gas settings by both chain and account while keeping the controller API responsible for applying saved fees.

The saved gas payload now supports gas fee estimate levels (low, medium, high) and legacy custom gasPrice values in addition to EIP-1559 custom fee values. Saved estimate levels reuse the current gas fee flow estimate for the saved level; custom values override only the specific saved fields.

Transactions that already include gas fee params continue to use those initial params, so dapp-proposed fees keep precedence over saved preferences. Swap transaction types continue to skip saved gas fees.

References

Related to MetaMask extension work to reintroduce saved gas settings per account and per chain.

Checklist

  • I’ve updated the test suite for new or updated code as appropriate
  • I’ve updated documentation (JSDoc, Markdown, etc.) for new or updated code as appropriate
  • I’ve communicated my changes to consumers by updating changelogs for packages I’ve changed
  • I’ve introduced breaking changes in this PR and have prepared client adoption work to resolve them

@pedronfigueiredo pedronfigueiredo force-pushed the pnf/save-custom-gas-settings branch from 8f9511a to 83cc6ba Compare June 3, 2026 16:40
@pedronfigueiredo pedronfigueiredo force-pushed the pnf/save-custom-gas-settings branch from 83cc6ba to ba63c2b Compare June 3, 2026 16:45
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.

1 participant