add lane expansion#1210
Open
tt-cll wants to merge 9 commits into
Open
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
This PR extends the CCV deployment tooling to support “lane expansion” via a new EVM lane-configuration adapter, and adds new on-chain committee-verifier products for allowed-finality configuration and per-destination sender allowlists. It also updates deployment plumbing to optionally package lane operations into MCMS proposals and bumps several Go module dependencies accordingly.
Changes:
- Add an EVM
LaneConfigAdapterimplementation (plus internal tests) and register it for lane expansion/router promotion flows. - Add on-chain changesets + EVM adapter support for
SetAllowedFinalityConfigandUpdateSenderAllowlist, with unit tests. - Update lane expansion / router promotion to resolve remote ramps via the remote chain adapter and optionally emit MCMS timelock proposals.
Reviewed changes
Copilot reviewed 20 out of 23 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| integration/evm/go.mod | Adds local replace for chainlink-ccv/deployment and bumps CCIP + protos deps. |
| integration/evm/go.sum | Updates checksums for bumped integration module dependencies. |
| integration/evm/adapters/init.go | Registers the new EVM lane config adapter and ensures MCMS reader registration via blank import. |
| integration/evm/adapters/lane_config_adapter.go | New EVM lane configuration adapter + input resolution helpers and validation. |
| integration/evm/adapters/lane_config_adapter_internal_test.go | Internal unit tests for lane-config input resolution/validation. |
| integration/evm/adapters/protocol_contracts_deploy_adapter_test.go | Updates imports to reference the local adapters package. |
| integration/evm/adapters/committee_verifier_deploy_adapter_test.go | Updates imports to reference the local adapters package. |
| integration/evm/adapters/ccv_committee_verifier_onchain.go | Adds EVM on-chain adapter methods for allowed finality config + allowlist updates. |
| integration/evm/adapters/deploy_defaults_test.go | Removes deploy-defaults tests previously targeting CCIP deploy adapter behavior. |
| deployment/go.mod | Bumps chainlink-deployments-framework, adds chainlink-ccip/deployment, updates indirect deps. |
| deployment/go.sum | Updates checksums for bumped deployment module dependencies. |
| deployment/adapters/lane_config.go | Extends lane config adapter interface to support remote ramp resolution + new RemoteLaneConfig fields. |
| deployment/adapters/committee_verifier_onchain.go | Extends on-chain committee verifier adapter interface with finality + allowlist operations. |
| deployment/changesets/lane_expansion.go | Adds inbound signer overrides, remote ramp resolution via remote adapter, and MCMS output packaging. |
| deployment/changesets/lane_expansion_test.go | Updates lane config adapter stub to satisfy new interface methods. |
| deployment/changesets/promote_lane_router.go | Adds optional MCMS input passthrough to the shared lane apply path. |
| deployment/changesets/set_allowed_finality_config.go | New changeset to set allowed-finality config across chains for a committee qualifier. |
| deployment/changesets/set_allowed_finality_config_test.go | Unit tests for allowed-finality changeset validation and apply behavior. |
| deployment/changesets/update_sender_allowlist.go | New changeset to update/toggle per-destination sender allowlists across chains. |
| deployment/changesets/update_sender_allowlist_test.go | Unit tests for sender allowlist changeset validation and apply behavior. |
| deployment/changesets/add_nop_to_committee_test.go | Extends test stubs to satisfy the expanded on-chain adapter interface. |
| build/devenv/go.mod | Bumps CCIP EVM + deployment deps in devenv module. |
| build/devenv/go.sum | Updates devenv module checksums for bumped dependencies. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
Code coverage report:
|
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.
Description
Testing
Checklist
changelogdirectory)just lint fix- no new lint errorsjust generate- mocks and protobufs are up to date