feat: add diffyscan configs for EDF on Hoodi - #206
Open
chasingrainbows wants to merge 2 commits into
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Add diffyscan configs to verify the EDF/DSM v5 deployment on Hoodi (vote #83, LIP-37).
Changes
config_samples/ethereum/hoodi/edf/hoodi_edf_config.json- DelegationFactory and 17 DelegationContracts from lidofinance/execution-delegation-framework. Constructor args are pinned in the config because the contracts were deployed through the factory, so the explorer has no constructor args for them. Values were read from chain and cross-checked with deployed-hoodi.json and the vote script.config_samples/ethereum/hoodi/edf/hoodi_edf_core_config.json- DepositSecurityModule v5 and the new LidoLocator implementation from lidofinance/core (feat/edf). DSM needsdeployment_from(its constructor reverts when the sender is the zero address) and anallowed_diffsrule for six immutable positions: the ATTEST/PAUSE/UNVET message prefix hashes depend on address(this), so they cannot match in the deployment simulation. The pinned values equal the on-chain getters.config_samples/ethereum/hoodi/edf/hoodi_edf_easy_track_config.json- SetDepositsReserveTarget factory from lidofinance/easy-track (deposit-reserve branch). Allowed diffs: four import lines rewritten by brownie verification, plus the CBOR metadata hash; the executable bytecode is identical. Same pattern as the existing vaults and srv3 easy track configs..gitignore- add node_modules.The new configs use the Etherscan V2 API (
api.etherscan.io+explorer_chain_id: 560048), since the V1 endpoint used by the older Hoodi configs is deprecated.Testing
uv run pytest -q- 374 passed.Related Issues
Linear: STC-889