Skip to content

Comments

feat: add ConditionalDeployer#18295

Closed
0xiamflux wants to merge 1 commit intoethereum-optimism:developfrom
defi-wonderland:sc-feat/l2cm-conditional-deployer
Closed

feat: add ConditionalDeployer#18295
0xiamflux wants to merge 1 commit intoethereum-optimism:developfrom
defi-wonderland:sc-feat/l2cm-conditional-deployer

Conversation

@0xiamflux
Copy link
Contributor

Add Conditional Deployer

Adds a wrapper around Create2Deployer to be used during Network Upgrade Transactions, allowing for non-reverting deterministic deployments even when address collisions occur.

Changes

ConditionalDeployer.sol

  • A wrapper around Create2Deployer for conditional deployment of contracts, in case of an address collision simply returns the address without reverting.

/// @notice Intermediary contract for deploying predeploy implementations during network upgrades.
contract ConditionalDeployer {
/// @notice Address of the Create2Deployer preinstall.
address payable private constant CREATE2_DEPLOYER = payable(0x13b0D85CcB8bf860b6b79AF3029fCA081AE9beF2);
Copy link
Contributor

Choose a reason for hiding this comment

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

Nit, would suggest we either use arachnid's deployer (0x4e59b44847b379578588920cA78FbF26c0B4956C, that's the current forge default + what we use for L1), or CreateX (0xba5Ed099633D3B313e4D5F7bdc1305d3c28ba5Ed, may become the forge default and can change L1 to match that later). Both are already preinstalls: https://docs.optimism.io/chain-operators/reference/features/preinstalls

@opgitgovernance opgitgovernance added the S-stale Status: Will be closed unless there is activity label Nov 29, 2025
@opgitgovernance
Copy link
Contributor

This pr has been automatically marked as stale and will be closed in 5 days if no updates

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-stale Status: Will be closed unless there is activity

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants