Skip to content

Conversation

@yorhodes
Copy link
Member

@yorhodes yorhodes commented Nov 25, 2025

Description

Backward compatibility

Testing

Summary by CodeRabbit

  • Chores
    • Updated USDC cross-chain token address mappings and routing configurations across multiple supported blockchain networks to reflect new on-chain references.
    • Added a changeset indicating a patch release and updated the deployment artifacts description for the CCTP v2 fast deployment.

✏️ Tip: You can customize this high-level summary in your review settings.

@changeset-bot
Copy link

changeset-bot bot commented Nov 25, 2025

🦋 Changeset detected

Latest commit: 2c945c7

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@hyperlane-xyz/registry Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@github-actions
Copy link
Contributor

github-actions bot commented Nov 25, 2025

Check Warp Deploy Summary

Warp Route ID Status
USDC/mainnet-cctp-v2-fast

Last updated: 2025-11-28 22:14:12 UTC

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Nov 25, 2025

📝 Walkthrough

Walkthrough

Updated USDC mainnet CCTP v2 fast deployment config: all addressOrDenom token addresses and their per-chain connections entries were replaced with new addresses across supported chains. YAML structure and keys remain unchanged; only address values were migrated.

Changes

Cohort / File(s) Summary
USDC Mainnet CCTP Configuration
deployments/warp_routes/USDC/mainnet-cctp-v2-fast-config.yaml
Replaced every token addressOrDenom value and updated all per-chain connections entries with new token addresses for all listed chains (arbitrum, avalanche, base, ethereum, hyperevm, ink, linea, optimism, plume, polygon, sei, sonic, unichain, worldchain, etc.). No structural/key changes.
Changeset
.changeset/sixty-bats-whisper.md
Added changeset declaring a patch release for @hyperlane-xyz/registry with description "Update CCTP v2 fast deployment artifacts." No code behavior changes.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

  • Attention areas:
    • Verify each new address is assigned to the intended chain and token entry.
    • Confirm YAML syntax/indentation and that no addresses were accidentally duplicated or omitted.
    • Cross-check that connection lists reference the correct new addresses for all chains.

Possibly related PRs

Suggested reviewers

  • xeno097
  • nambrot

Poem

In the swamp of configs, addresses got new shoes,
Chains now whisper different lovely cues,
Tokens lined up, tidy and neat,
New connections hum steady and sweet,
Migration done — no more old blues. 🐸✨

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Description check ⚠️ Warning The PR description contains only template placeholders with no substantive content filled in - all required sections are empty. Fill in the Description, Backward compatibility, and Testing sections with actual details about the redeployment, compatibility implications, and validation performed.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title 'Redeploy cctpv2 with core 10.0.4' directly describes the main change - redeploying CCTP v2 configuration with a new core version.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch cctp-v2-deploy

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 8eca2a1 and 2c945c7.

📒 Files selected for processing (1)
  • .changeset/sixty-bats-whisper.md (1 hunks)
✅ Files skipped from review due to trivial changes (1)
  • .changeset/sixty-bats-whisper.md
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
  • GitHub Check: check-warp-deploy

Tip

📝 Customizable high-level summaries are now available in beta!

You can now customize how CodeRabbit generates the high-level summary in your pull requests — including its content, structure, tone, and formatting.

  • Provide your own instructions using the high_level_summary_instructions setting.
  • Format the summary however you like (bullet lists, tables, multi-section layouts, contributor stats, etc.).
  • Use high_level_summary_in_walkthrough to move the summary from the description to the walkthrough section.

Example instruction:

"Divide the high-level summary into five sections:

  1. 📝 Description — Summarize the main change in 50–60 words, explaining what was done.
  2. 📓 References — List relevant issues, discussions, documentation, or related PRs.
  3. 📦 Dependencies & Requirements — Mention any new/updated dependencies, environment variable changes, or configuration updates.
  4. 📊 Contributor Summary — Include a Markdown table showing contributions:
    | Contributor | Lines Added | Lines Removed | Files Changed |
  5. ✔️ Additional Notes — Add any extra reviewer context.
    Keep each section concise (under 200 words) and use bullet or numbered lists for clarity."

Note: This feature is currently in beta for Pro-tier users, and pricing will be announced later.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Collaborator

@paulbalaji paulbalaji left a comment

Choose a reason for hiding this comment

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

went through explorers and marked as proxy where i could to complete verification, left comments on the ones that weren't verified - unsure how strict we are on it

also satisifed that you can ignore the check-warp-deploy ci failure because it works fine locally without public rpcs

yarn tsx scripts/check/check-deploy.ts -e mainnet3 -m warp --warpRouteId USDC/mainnet-cctp-v2-fast
warp checker found no violations

@@ -1,280 +1,280 @@
tokens:
- addressOrDenom: "0xC9d7f943d2056a5927FFCE4b689dA17A9719374C"
- addressOrDenom: "0xb0B8D4C6EF212D76d5079df5Ff7A0888A27e9b32"
Copy link
Collaborator

Choose a reason for hiding this comment

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

- token: ethereum|sonic|0xc508E179297ef72e65909e4D072758C2d0B92B83
- token: ethereum|unichain|0xc508E179297ef72e65909e4D072758C2d0B92B83
- token: ethereum|worldchain|0xc508E179297ef72e65909e4D072758C2d0B92B83
- token: ethereum|avalanche|0x8dadbDE67eD0589d90cdE3C940045F10092AcC11
Copy link
Collaborator

Choose a reason for hiding this comment

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

- token: ethereum|optimism|0xb0B8D4C6EF212D76d5079df5Ff7A0888A27e9b32
- token: ethereum|plume|0x0706DE5b87F03C0c6fDC8a03D987D06f776eF6bf
- token: ethereum|polygon|0x8dadbDE67eD0589d90cdE3C940045F10092AcC11
- token: ethereum|sei|0xAE3d8F7C4d7527BfaF2f15c43D1B6ea39802838B
Copy link
Collaborator

Choose a reason for hiding this comment

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

proxy + impl not verified

- token: ethereum|plume|0x0706DE5b87F03C0c6fDC8a03D987D06f776eF6bf
- token: ethereum|polygon|0x8dadbDE67eD0589d90cdE3C940045F10092AcC11
- token: ethereum|sei|0xAE3d8F7C4d7527BfaF2f15c43D1B6ea39802838B
- token: ethereum|sonic|0x8dadbDE67eD0589d90cdE3C940045F10092AcC11
Copy link
Collaborator

Choose a reason for hiding this comment

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

not verified

- token: ethereum|polygon|0x8dadbDE67eD0589d90cdE3C940045F10092AcC11
- token: ethereum|sei|0xAE3d8F7C4d7527BfaF2f15c43D1B6ea39802838B
- token: ethereum|sonic|0x8dadbDE67eD0589d90cdE3C940045F10092AcC11
- token: ethereum|unichain|0x5b7A1a03F26C05D68ada7CE67d49db263e2D0d9B
Copy link
Collaborator

Choose a reason for hiding this comment

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

not verified

Copy link
Contributor

@xeno097 xeno097 left a comment

Choose a reason for hiding this comment

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

The check job failure is due to an rpc error but running warp check locally gives no config errors and contract seem not to be verified

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.

4 participants