Skip to content

Remove zksync wrapper generation - #552

Open
friedemannf wants to merge 1 commit into
developfrom
zksync-wrappers
Open

Remove zksync wrapper generation#552
friedemannf wants to merge 1 commit into
developfrom
zksync-wrappers

Conversation

@friedemannf

@friedemannf friedemannf commented Sep 10, 2026

Copy link
Copy Markdown
Member

Continuation of: #551

  • Removes zkSync compilation, wrappers + wrapper generation.
  • Bumps gethwrappers/helpers

Blocks: #550

@github-actions

Copy link
Copy Markdown
Contributor

📊 API Diff Results

No changes detected for module github.com/smartcontractkit/chainlink-evm/contracts/cre/gobindings

View full report

@github-actions

Copy link
Copy Markdown
Contributor

⚠️ API Diff Results - github.com/smartcontractkit/chainlink-evm/gethwrappers

⚠️ Breaking Changes (25)

generated/mock_ethusd_aggregator_wrapper (2)
  • DeployMockETHUSDAggregatorZk — 🗑️ Removed

  • ZkBytecode — 🗑️ Removed

package github (3)
  • com/smartcontractkit/chainlink-evm/gethwrappers/generation/generate/zksync — 🗑️ Removed

  • com/smartcontractkit/chainlink-evm/gethwrappers/generation/zksync — 🗑️ Removed

  • com/smartcontractkit/chainlink-evm/gethwrappers/zksync — 🗑️ Removed

shared/generated/initial/burn_mint_erc677 (2)
  • DeployBurnMintERC677Zk — 🗑️ Removed

  • ZkBytecode — 🗑️ Removed

shared/generated/initial/link_token (2)
  • DeployLinkTokenZk — 🗑️ Removed

  • ZkBytecode — 🗑️ Removed

shared/generated/initial/mock_v3_aggregator_contract (2)
  • DeployMockV3AggregatorZk — 🗑️ Removed

  • ZkBytecode — 🗑️ Removed

shared/generated/initial/multicall3 (2)
  • DeployMulticall3Zk — 🗑️ Removed

  • ZkBytecode — 🗑️ Removed

shared/generated/initial/weth9_zksync (2)
  • DeployWETH9ZKSyncZk — 🗑️ Removed

  • ZkBytecode — 🗑️ Removed

shared/generated/latest/burn_mint_erc677 (2)
  • DeployBurnMintERC677Zk — 🗑️ Removed

  • ZkBytecode — 🗑️ Removed

shared/generated/latest/link_token (2)
  • DeployLinkTokenZk — 🗑️ Removed

  • ZkBytecode — 🗑️ Removed

shared/generated/latest/mock_v3_aggregator_contract (2)
  • DeployMockV3AggregatorZk — 🗑️ Removed

  • ZkBytecode — 🗑️ Removed

shared/generated/latest/multicall3 (2)
  • DeployMulticall3Zk — 🗑️ Removed

  • ZkBytecode — 🗑️ Removed

shared/generated/latest/weth9_zksync (2)
  • DeployWETH9ZKSyncZk — 🗑️ Removed

  • ZkBytecode — 🗑️ Removed


📄 View full apidiff report

@friedemannf
friedemannf marked this pull request as ready for review September 10, 2026 16:15
@friedemannf
friedemannf requested review from a team as code owners September 10, 2026 16:15
Copilot AI lite review requested due to automatic review settings September 10, 2026 16:15
@friedemannf
friedemannf requested review from a team as code owners September 10, 2026 16:15

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Approval recommended

No unresolved review comments remain, and the requested zkSync cleanup is complete.

Pull request overview

Removes legacy zkSync compilation and wrapper-generation support while retaining standard Solidity wrapper generation.

Changes:

  • Deletes zkSync tooling, generators, templates, scripts, and generated bindings.
  • Removes related Make, documentation, ignore-file, and CI configuration.
  • Updates dependencies and bumps gethwrappers/helpers.
File summaries
File Description
tools/bin/install_forge_zksync Removes the zkSync Forge installer.
gethwrappers/zksync/go_generate.go Removes zkSync generation directives.
gethwrappers/wrap/wrap.go Simplifies wrapper generation.
gethwrappers/shared/generated/latest/weth9_zksync/weth9_zksync_zksync.go Removes generated zkSync binding.
gethwrappers/shared/generated/latest/multicall3/multicall3_zksync.go Removes generated zkSync binding.
gethwrappers/shared/generated/latest/mock_v3_aggregator_contract/mock_v3_aggregator_contract_zksync.go Removes generated zkSync binding.
gethwrappers/shared/generated/initial/weth9_zksync/weth9_zksync_zksync.go Removes generated zkSync binding.
gethwrappers/shared/generated/initial/multicall3/multicall3_zksync.go Removes generated zkSync binding.
gethwrappers/shared/generated/initial/mock_v3_aggregator_contract/mock_v3_aggregator_contract_zksync.go Removes generated zkSync binding.
gethwrappers/go.sum Cleans dependency checksums.
gethwrappers/go.mod Removes zkSync dependencies and updates helpers.
gethwrappers/generation/zksync/wrap.go Removes the zkSync generator entrypoint.
gethwrappers/generation/generate/zksync/zksyncwrapper.go Removes zkSync generation implementation.
gethwrappers/generation/generate/zksync/vars.go Removes generator placeholders.
gethwrappers/generation/generate/zksync/template.go Removes the zkSync deployment template.
gethwrappers/generated/mock_ethusd_aggregator_wrapper/mock_ethusd_aggregator_wrapper_zksync.go Removes generated zkSync binding.
contracts/scripts/zksync_compile_all Removes the zkSync compilation script.
contracts/GNUmakefile Removes zkSync tooling and targets.
contracts/FOUNDRY_GUIDE.md Removes obsolete zkSync instructions.
contracts/cre/gobindings/wrap/wrap.go Simplifies CRE wrapper generation.
contracts/cre/gobindings/go.sum Cleans dependency checksums.
contracts/cre/gobindings/go.mod Removes zkSync dependencies and updates helpers.
contracts/cre/GNUmakefile Removes the zkSync wrapper target.
.gitignore Removes the obsolete zkSync Forge entry.
.github/workflows/solidity.yml Removes zkSync compilation and generation CI.
Review details
  • Files reviewed: 14/29 changed files
  • Comments generated: 0
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

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