Skip to content

Conversation

@patricios-space
Copy link
Collaborator

@patricios-space patricios-space commented Dec 23, 2025

NONEVM-3269 Define explicitly dummy addresses for wrapped TON and LinkToken Jetton contracts

@patricios-space patricios-space force-pushed the feat/define-link-and-ton-dummy-addreses branch from a2ee6d3 to 3cb64d9 Compare December 23, 2025 16:15
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 introduces centralized dummy address definitions for TON and LINK tokens in the Go codebase, replacing hardcoded addresses scattered across test and deployment files.

Key Changes:

  • Added LinkTokenAddr and updated TonTokenAddr comment in pkg/ton/tvm/address.go to clearly document these as dummy addresses
  • Introduced TokenPrice utility function to calculate token prices with proper decimal handling
  • Refactored test and deployment code to use centralized address constants

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated no comments.

Show a summary per file
File Description
pkg/ton/tvm/address.go Defines dummy addresses for TON and LINK tokens as package-level constants
integration-tests/model/fee_quoter_test.go Replaces hardcoded address literals with centralized dummy address constants
integration-tests/deployment/ccip/token_price_test.go Adds comprehensive test coverage for the new TokenPrice utility function
deployment/ccip/cs_test_helpers.go Refactors token price calculation to use new TokenPrice utility function
deployment/ccip/cs_deploy_ton_chain.go Updates LINK token address initialization to use centralized dummy address constant
deployment/ccip/config/tokenPrice.go Implements new utility function for calculating token prices based on USD price and token decimals
Comments suppressed due to low confidence (1)

deployment/ccip/cs_deploy_ton_chain.go:3

  • The import order has been changed with 'mcms' now appearing before 'chainlink-deployments-framework' imports. Go convention typically groups imports in order: standard library, third-party packages, then local packages. Consider maintaining consistent import ordering throughout the codebase.
import (

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

@patricios-space patricios-space changed the title feat: define link and ton dummy addresses in go [NONEVM-3269] Define explicitly dummy addresses for TON and Link Dec 26, 2025
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.

1 participant