Skip to content

revert(L2): remove system address refunding from FeeDisburser - #414

Merged
0xth4nh merged 1 commit into
mainfrom
revert/fee-disburser-system-address-refunds
Aug 24, 2026
Merged

0xth4nh merged 1 commit into
mainfrom
revert/fee-disburser-system-address-refunds

Conversation

@0xth4nh

@0xth4nh 0xth4nh commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Reverses the FeeDisburser system-address top-up added in feat(L2): add system address refunding to FeeDisburser #358. disburseFees() again withdraws from the fee vaults and bridges the full collected balance to L1_WALLET.
  • Drops initialize, systemAddresses / targetBalances, the custom reentrancy guard, and the ProxyAdminOwnedBase / Initializable inheritance that existed only for that config.
  • Restores ABI, storage layout, and snapshots/semver-lock.json hashes to match the reverted 1.0.0 source (regenerated with Foundry 1.5.1).

Test plan

  • forge test --match-path test/L2/FeeDisburser.t.sol -vv
  • just semver-lock then git diff --exit-code snapshots/semver-lock.json

Made with Cursor

Restore FeeDisburser to bridging the full collected balance to L1,
undoing the system-address top-up added in #358.

Co-authored-by: Cursor <cursoragent@cursor.com>
@cb-heimdall

cb-heimdall commented Aug 24, 2026

Copy link
Copy Markdown
Collaborator

✅ Heimdall Review Status

Requirement Status More Info
Reviews 1/1
Denominator calculation
Show calculation
1 if user is bot 0
1 if user is external 0
2 if repo is sensitive 0
From .codeflow.yml 1
Additional review requirements
Show calculation
Max 0
0
From CODEOWNERS 0
Global minimum 0
Max 1
1
1 if commit is unverified 0
Sum 1

@0xth4nh
0xth4nh merged commit ae76303 into main Aug 24, 2026
6 checks passed
@0xth4nh
0xth4nh deleted the revert/fee-disburser-system-address-refunds branch August 24, 2026 16:28
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.

3 participants