Conversation
|
👋 0xsuryansh, thanks for creating this pull request! To help reviewers, please consider creating future PRs as drafts first. This allows you to self-review and make any final changes before notifying the team. Once you're ready, you can mark it as "Ready for review" to request feedback. Thanks! |
CORA - Pending ReviewersAll codeowners have approved! ✅ Legend: ✅ Approved | ❌ Changes Requested | 💬 Commented | 🚫 Dismissed | ⏳ Pending | ❓ Unknown For more details, see the full review summary. |
|
✅ No conflicts with other open PRs targeting |
There was a problem hiding this comment.
Pull request overview
Risk Rating: LOW
Updates CCIP deployment logic to treat USDCTokenPoolProxy as version 2.0.0 (instead of 1.7.0) across shared version constants, on-chain state loading, and candidate changeset validation/tests.
Changes:
- Replace
deployment.Version1_7_0withdeployment.Version2_0_0and introduce the2.0.0semver constant. - Update CCIP token pool version allowlist and on-chain stateview switch cases for
USDCTokenPoolProxy. - Update v1_6 candidate changeset logic and tests to read/write datastore refs using
2.0.0.
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| deployment/version.go | Replaces the previously referenced 1.7.0 constant with 2.0.0. |
| deployment/ccip/shared/token_pools.go | Updates the supported token pool version set to include 2.0.0. |
| deployment/ccip/shared/stateview/state.go | Loads USDCTokenPoolProxy state using type+version 2.0.0. |
| deployment/ccip/changeset/v1_6/cs_ccip_home.go | Updates candidate changeset validation/datastore lookup to use 2.0.0 for USDCTokenPoolProxy. |
| deployment/ccip/changeset/v1_6/cs_ccip_home_test.go | Updates assertions and datastore refs to use 2.0.0. |
Scrupulous human review areas:
- Confirm the on-chain
TypeAndVersion()returned byUSDCTokenPoolProxydeployments is expected to beUSDCTokenPoolProxy 2.0.0(sostateview.LoadChainStatewill correctly classify the contract).
|




No description provided.