Skip to content

✨♻️🥢: add FeeVaultSplitter.sol and FeeVaultSplitter.t.sol, update Gasback.sol and Gasback.t.sol#19

Open
j6i wants to merge 2 commits into
Vectorized:mainfrom
shape-network:j6i/add-splitter
Open

✨♻️🥢: add FeeVaultSplitter.sol and FeeVaultSplitter.t.sol, update Gasback.sol and Gasback.t.sol#19
j6i wants to merge 2 commits into
Vectorized:mainfrom
shape-network:j6i/add-splitter

Conversation

@j6i

@j6i j6i commented Jun 13, 2026

Copy link
Copy Markdown
Contributor

Description

  • ✨: add FeeVaultSplitter.sol, used to split incoming sequencer fees from the BaseFeeVault
  • add FeeVaultSplitter.t.sol to test FeeVaultSplitter.sol
  • ♻️: updates to Gasback.sol
    • update accrued calculation
    • remove minVaultBalance, accruedRecipient, setAccruedRecipient, withdrawAccruedToAccruedRecipient
    • add baseFeeVaultShareNumerator, setBaseFeeVaultShareNumerator, _expectedBaseFeeVaultShareNumerator
    • lower gasbackRatioNumerator
  • add @openzeppelin-contracts-4.9.5/ dep
  • 🥢: fix "accural" misspelling (accural -> accrual)

Checklist

Ensure you completed all of the steps below before submitting your pull request:

  • Ran forge fmt?
  • Ran forge snapshot?
  • Ran forge test?

Pull requests with an incomplete checklist will be thrown out.

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