Skip to content

Conversation

@ManuelBilbao
Copy link
Contributor

Motivation

Inserting placeholders allow us to later include new system contracts in already deployed networks by just upgrading the proxies

Description

@ManuelBilbao ManuelBilbao self-assigned this Oct 18, 2025
@ManuelBilbao ManuelBilbao added the L2 Rollup client label Oct 18, 2025
Copilot AI review requested due to automatic review settings October 18, 2025 00:44
@ManuelBilbao ManuelBilbao requested a review from a team as a code owner October 18, 2025 00:44
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds placeholder proxies for system contracts in the L2 genesis configuration, enabling future upgrades to include new system contracts in already deployed networks by upgrading these pre-deployed proxies.

  • Introduces add_placeholder_proxy function to create proxy contracts without implementation
  • Deploys placeholder proxies for address range 0xff00 to 0xfffc (253 proxies)

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@github-actions
Copy link

Lines of code report

Total lines added: 23
Total lines removed: 0
Total lines changed: 23

Detailed view
+-------------------------------+-------+------+
| File                          | Lines | Diff |
+-------------------------------+-------+------+
| ethrex/cmd/ethrex/build_l2.rs | 397   | +23  |
+-------------------------------+-------+------+

@ManuelBilbao ManuelBilbao added this pull request to the merge queue Oct 21, 2025
Merged via the queue into main with commit 5f2dbf6 Oct 21, 2025
47 checks passed
@ManuelBilbao ManuelBilbao deleted the placeholder_system_contracts branch October 21, 2025 20:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

L2 Rollup client

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants