Skip to content

feat: add script verifying deployment#161

Open
dujardin09 wants to merge 1 commit into
mainfrom
feat/verifyMerklDeployment
Open

feat: add script verifying deployment#161
dujardin09 wants to merge 1 commit into
mainfrom
feat/verifyMerklDeployment

Conversation

@dujardin09
Copy link
Copy Markdown

No description provided.

Copilot AI review requested due to automatic review settings May 11, 2026 17:31
Copy link
Copy Markdown

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

Adds a Foundry script to validate a Merkl deployment on-chain by checking ownership, proxy wiring, key configuration values, and expected nonce-derived deployment addresses.

Changes:

  • Introduces VerifyMerklDeployment Forge script with two run(...) entrypoints (default and explicit angleLabs/deployer).
  • Adds checks for ProxyAdmin, AccessControlManager, Distributor, DistributionCreator, optional Disputer, and expected CREATE-nonce derived addresses.
  • Emits a summary and reverts if any verification failures are detected.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

_expectEq(c.distributor(), distributor, "distributor wired");
_expectEq(c.feeRecipient(), DUMPER, "feeRecipient == DUMPER");
_expectEqUint(c.defaultFees(), 0.03 gwei, "defaultFees == 0.03 gwei");
_expectEqUint(c.campaignSpecificFees(4), 5 * 1e6, "campaignSpecificFees[4] == 5%");
AccessControlManager m = AccessControlManager(acm);

_expect(m.isGovernor(angleLabs), "angleLabs is governor");
_expectNot(m.isGovernor(deployer) && deployer != angleLabs, "deployer is NOT governor");
@dujardin09 dujardin09 self-assigned this May 12, 2026
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