Predeploys should be proxies

[RogerPodacter]

Note

Moves Ethscriptions to proxied 0x330… predeploys, wires implementations via code-namespace, and updates template/constants and tooling accordingly.

• Genesis/Predeploy Setup:
  • Switch Ethscriptions to proxied predeploys across 0x3300… with EIP-1967 admin set to Predeploys.PROXY_ADMIN and implementations in code-namespace via setImplementation.
  • Create genesis Ethscriptions by etching GenesisEthscriptions to the implementation address, calling via proxy, then swapping in real Ethscriptions implementation without resetting nonce.
  • Add _setImplementationCodeNamed and _setCodeAt helpers; templates (ERC20, ERC721) etched at implementation addresses (not proxied).
• Predeploys Library:
  • Update ETHSCRIPTIONS to 0x3300…0001; add ERC20_TEMPLATE_PROXY/IMPLEMENTATION and ERC721_TEMPLATE_PROXY/IMPLEMENTATION constants.
  • Introduce isOPPredeployNamespace, isEthscriptionsPredeployNamespace, and broaden isPredeployNamespace; keep predeployToCodeNamespace for both namespaces.
• Contracts:
  • CollectionsManager and TokenManager: point erc721Template/erc20Template to *_TEMPLATE_IMPLEMENTATION constants.
• Ruby Tooling:
  • SysConfig: set ETHSCRIPTIONS_ADDRESS to 0x3300…0001.
  • event_decoder.rb and storage_reader.rb: use SysConfig::ETHSCRIPTIONS_ADDRESS instead of hardcoded address.

^{Written by Cursor Bugbot for commit 232e539. This will update automatically on new commits. Configure here.}

[Copilot]

Pull Request Overview

This PR migrates Ethscriptions predeploy contracts to use a proxy pattern, aligning with OP Stack architecture. The main change is moving the ETHSCRIPTIONS address from 0xEeee... to 0x3300...0001 to match the Ethscriptions namespace, and introducing proxy/implementation separation for system contracts.

Key changes:

• Updated ETHSCRIPTIONS_ADDRESS constant from 0xEeee... to 0x3300...0001 across the codebase
• Introduced proxy/implementation separation for predeploy contracts with distinct addresses
• Refactored genesis deployment script to create proxies across the full 0x3300... namespace

Reviewed Changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
lib/sys_config.rb	Updated ETHSCRIPTIONS_ADDRESS to new namespace
lib/storage_reader.rb	Changed to use centralized SysConfig constant
lib/event_decoder.rb	Changed to use centralized SysConfig constant
contracts/src/libraries/Predeploys.sol	Added proxy/implementation address pairs and namespace helper functions
contracts/src/TokenManager.sol	Updated to reference ERC20_TEMPLATE_IMPLEMENTATION instead of ERC20_TEMPLATE
contracts/src/CollectionsManager.sol	Updated to reference ERC721_TEMPLATE_IMPLEMENTATION instead of ERC721_TEMPLATE
contracts/script/L2Genesis.s.sol	Refactored to deploy proxies across namespace and wire up implementations

---

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

[cursor]

Bug: ERC20/721 Template Address Casing Mismatch

The ERC20_TEMPLATE_IMPLEMENTATION and ERC721_TEMPLATE_IMPLEMENTATION addresses have inconsistent hex casing compared to the pattern derived by the predeployToCodeNamespace function. While the underlying address values are functionally equivalent in the EVM, this casing mismatch creates an inconsistency in the addressing scheme, which could lead to confusion or issues with string comparisons and checksum validations.