Skip to content

chore: add scripts to upgrade all core contracts - #544

Merged
evercoinx merged 4 commits into
mainfrom
serge/core-upgrade
May 20, 2026
Merged

chore: add scripts to upgrade all core contracts#544
evercoinx merged 4 commits into
mainfrom
serge/core-upgrade

Conversation

@evercoinx

@evercoinx evercoinx commented May 20, 2026

Copy link
Copy Markdown
Contributor

Note

Medium Risk
Adds mainnet/testnet upgrade and post-upgrade verification scripts for UUPS proxies; while code changes are off-chain, incorrect usage or addresses could trigger unintended on-chain upgrades.

Overview
Introduces network-specific Forge scripts to upgrade all Core UUPS proxies on Ethereum mainnet and Sepolia, adding chainid guards and documented canonical proxy addresses.

Makes UpgradeAllContracts.run() public virtual to allow overrides, and adds VerifyCoreEthereum/VerifyCoreSepolia scripts that sanity-check the applied upgrade (e.g., admin role ownership, adapter wiring, expected function selectors, delays, and WETH price feed status).

Reviewed by Cursor Bugbot for commit 6551815. Bugbot is set up for automated code reviews on this repo. Configure here.

@evercoinx
evercoinx requested a review from dB2510 May 20, 2026 07:43
@evercoinx evercoinx self-assigned this May 20, 2026
Comment thread src/script/UpgradeAllContractsEthereum.s.sol
Comment thread src/script/VerifyCoreEthereum.s.sol

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 6551815. Configure here.

Comment thread src/script/VerifyCoreEthereum.s.sol
@evercoinx
evercoinx merged commit 7ead05d into main May 20, 2026
2 checks passed
@evercoinx
evercoinx deleted the serge/core-upgrade branch May 20, 2026 18: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.

2 participants