Skip to content

Conversation

@jadepark-dev
Copy link
Contributor

@jadepark-dev jadepark-dev commented Dec 8, 2025

This PR bumps chainlink-ton, and clean ups CCIP test helpers

chainlink-ton PR to be merged first:

@github-actions
Copy link
Contributor

github-actions bot commented Dec 8, 2025

CORA - Pending Reviewers

Codeowners Entry Overall Num Files Owners
* 1 @smartcontractkit/foundations, @smartcontractkit/core
go.md 1 @smartcontractkit/core, @smartcontractkit/foundations
go.mod 6 @smartcontractkit/core, @smartcontractkit/foundations
go.sum 6 @smartcontractkit/core, @smartcontractkit/foundations

Legend: ✅ Approved | ❌ Changes Requested | 💬 Commented | 🚫 Dismissed | ⏳ Pending | ❓ Unknown

For more details, see the full review summary.

@github-actions
Copy link
Contributor

github-actions bot commented Dec 8, 2025

I see you updated files related to core. Please run pnpm changeset in the root directory to add a changeset as well as in the text include at least one of the following tags:

  • #added For any new functionality added.
  • #breaking_change For any functionality that requires manual action for the node to boot.
  • #bugfix For bug fixes.
  • #changed For any change to the existing functionality.
  • #db_update For any feature that introduces updates to database schema.
  • #deprecation_notice For any upcoming deprecation functionality.
  • #internal For changesets that need to be excluded from the final changelog.
  • #nops For any feature that is NOP facing and needs to be in the official Release Notes for the release.
  • #removed For any functionality/config that is removed.
  • #updated For any functionality that is updated.
  • #wip For any change that is not ready yet and external communication about it should be held off till it is feature complete.

Copy link
Contributor

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

This PR updates chainlink-ton and its related dependencies to version f56790ae3631 (dated 2025-12-11) across the codebase, along with several golang.org/x packages. The update includes corresponding changes to CCIP test helpers to accommodate API changes in the updated chainlink-ton package.

  • Bumps chainlink-ton and chainlink-ton/deployment packages to f56790ae3631
  • Updates TON contract version reference to 08209efcf3f3
  • Refactors TON message sending in test helpers to use new SendCCIPMessage API and tonrouter.CCIPSend struct

Reviewed changes

Copilot reviewed 14 out of 21 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
system-tests/tests/go.mod Updates chainlink-ton dependencies and golang.org/x packages
system-tests/lib/go.mod Updates chainlink-ton dependencies and golang.org/x packages
plugins/plugins.public.yaml Updates TON plugin git reference to new version
integration-tests/load/go.mod Updates chainlink-ton dependencies and golang.org/x packages
integration-tests/go.mod Updates chainlink-ton dependencies and golang.org/x packages
go.mod Updates chainlink-ton, ethereum/go-ethereum, rs/zerolog, and golang.org/x packages; adds new indirect dependencies
go.md Adds chainlink-ton to chainlink-framework/metrics dependency relationship
deployment/go.mod Updates chainlink-ton dependencies and golang.org/x packages
deployment/ccip/changeset/testhelpers/test_ton_helpers.go Handles error return from tonlogpoller.NewService constructor
deployment/ccip/changeset/testhelpers/test_helpers_solana_v0_1_1.go Refactors TON request sending to use new SendCCIPMessage API and tonrouter.CCIPSend
deployment/ccip/changeset/testhelpers/test_helpers_solana_v0_1_0.go Refactors TON request sending to use new SendCCIPMessage API and tonrouter.CCIPSend
deployment/ccip/changeset/testhelpers/test_environment.go Updates TON contract version reference to 08209efcf3f3
deployment/ccip/changeset/testhelpers/messagingtest/helpers.go Updates message construction to use tonrouter.CCIPSend with DestChainSelector field
core/scripts/go.mod Updates chainlink-ton dependencies and golang.org/x packages

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

@trunk-io
Copy link

trunk-io bot commented Dec 11, 2025

Static BadgeStatic BadgeStatic BadgeStatic Badge

View Full Report ↗︎Docs

@cl-sonarqube-production
Copy link

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarQube

@jadepark-dev jadepark-dev merged commit 7522a5c into develop Dec 17, 2025
423 of 438 checks passed
@jadepark-dev jadepark-dev deleted the jade/bump-ton branch December 17, 2025 20:22
@jadepark-dev jadepark-dev restored the jade/bump-ton branch December 17, 2025 20:22
@jadepark-dev
Copy link
Contributor Author

jadepark-dev commented Dec 17, 2025

accidentally pushed reset state, the PR has been reopened #20653

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