tokenRegistry bindings and deployment by vicentevieytes · Pull Request #767 · smartcontractkit/chainlink-ton

vicentevieytes · 2026-06-18T22:32:37Z

Adds a setter to the minimal TokenRegistry to be able to set TokenInfo
Modifies OnRamp dynamic config and storage to be able to fit the TokenRegistry address
Implementes token registry bindings and updates deployment sequence to include the token registry on the OnRamp

Copilot

Pull request overview

This PR adds TokenRegistry contract support across the TON CCIP stack by introducing new Go bindings, registering the contract type in the bindings registry, wiring TokenRegistry into CCIP deployment/state, and updating the generated TS wrapper + e2e test to reflect the updated ABI/storage layout.

Changes:

Added Go TL-B bindings for the CCIP TokenRegistry contract and registered it in the global bindings registry.
Extended CCIP deployment/state/config to deploy TokenRegistry and pass its address into OnRamp storage.
Updated Tolk/TS artifacts (storage includes id, new SetTokenInfo message) and adjusted the e2e token transfer test to initialize TokenRegistry storage correctly.

Reviewed changes

Copilot reviewed 15 out of 16 changed files in this pull request and generated 4 comments.

Show a summary per file

File	Description
pkg/ccip/bindings/tokenregistry/tokenregistry.go	New Go TL-B bindings for TokenRegistry messages/storage.
pkg/ccip/bindings/onramp/onramp.go	Updated OnRamp storage comment around TokenRegistry field.
pkg/bindings/index.go	Registered TokenRegistry contract type + TLBs in bindings registry.
deployment/state/state.go	Added TokenRegistry to CCIPChainState and contract type list.
deployment/ccip/sequence/deploy_ccip.go	Deploy TokenRegistry before ramps and inject address into OnRamp storage.
deployment/ccip/cs_test_helpers.go	Added TokenRegistryParams to deployment test config.
deployment/ccip/config/deploy.go	Added TokenRegistryParams to chain deployment config + validation hook.
deployment/ccip/1_6_0/sequences/deploy_chain_contracts.go	Persist/extract TokenRegistry address in chain state and set default params.
contracts/wrappers/TokenRegistry.compile.ts	New blueprint compile config for TokenRegistry.
contracts/wrappers/gen/ccip/TokenRegistry.ts	Updated generated wrapper for new storage/id + SetTokenInfo message.
contracts/tests/ccip/e2e/CCIPSendWithTokenTransfer.spec.ts	Updated test to provide TokenRegistry storage `id`.
contracts/scripts/generateContractsPkg.ts	Added TokenRegistry to contracts package generation list.
contracts/contracts/ccip/token_registry/storage.tolk	Storage now includes `id` and store() persists it.
contracts/contracts/ccip/token_registry/messages.tolk	Added SetTokenInfo message and included in incoming union.
contracts/contracts/ccip/token_registry/contract.tolk	Added handler for SetTokenInfo; adjusted metadata/version constants.
contracts/contracts/ccip/onramp/storage.tolk	Comment formatting tweak around tokenRegistry field.

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

… vv/token-registry-bindings-and-deployment

…ddress Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

fix typo Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

krebernisak · 2026-06-26T08:44:15Z

+struct OnRamp_TokenRegistryDeployment {
+    //TODO: deployableCode: cell;
+    //      tokenRegistryCode: cell;
+    tokenRegistry: address;


Not sure why don't we immediately start using this as a sharded TR and put code here instead of a single address.

Think it's not a big pull to just replace this address with code and implement calculateTRAddr from code/token. and send it a get info message.

Agree here. Deployment becomes simpler because OnRamp can then be configured before deploying the tokenRegistry and knowing its address

Moved to a sharded implementation under the router, please take another look @patricios-space @krebernisak

patricios-space · 2026-06-26T14:07:46Z


+fun onSetTokenInfo(msg: TokenRegistry_SetTokenInfo, sender: address) {
+    //TODO access control
+    var st = TokenRegistry_Storage.load();


We should not be able to change the minter address.

for now this is a convenient way to set the info on the registry, I'll move to the correct interface in a follow up

vicentevieytes · 2026-07-01T14:45:24Z

    val msg = lazy CCIPSendExecutor_InMessage.fromSlice(in.body);
    match (msg) {
        // Sender must be this contract (called when initializing the contract)
-        CCIPSendExecutor_ExecuteV2 => {


I removed this V2 version of Execute and just updated the original type and functions. It was causing friction to iterate.

We'll change how we get the executor code so that it's inlined in the onramp in the future.

Besides that, we should think about compatibility issues between all the other contracts in an upgreade path

patricios-space · 2026-07-01T17:10:18Z

+    var tokenRegistry: address? = null;
+    if (!msg.tokenAmounts.empty()) {
+        val tokenAmount = msg.tokenAmounts.iter().next(); // Only one tokenAmount per msg is supported
+        //TODO should we assert that there is only one token in the tokenAmounts? Or do we just ignore after the first one?


I would assert here. I'm doing that in the offramp flow

patricios-space

Let's get this merged and continue iterating on a follow up PR

patricios-space · 2026-07-01T17:11:15Z

+                    info: msg.tokenInfo
+                }
+            });
+            setTokenInfoMsg.send(SEND_MODE_REGULAR); //TODO check send_mode


You can use Deployable_InitializeAndSend instead

patricios-space · 2026-07-01T17:16:40Z


            onCCIPSend(sendMsg, msg.sender, in.valueCoins);
        }
+        Router_TokenRegistrySetTokenInfo => {


I agree with @krebernisak on having a separate TokenAdminRegistryManager contract here

patricios-space · 2026-07-01T17:17:10Z

@@ -1,8 +1,8 @@
 import "types"

 struct TokenRegistry_Storage {


We should remember renaming all TokenRegistry appearances to TokenAdminRegistry

patricios-space · 2026-07-01T17:21:00Z

We will need a deploy_token_registry.go sequence

If we add a TokenAdminRegistryManager we will be deploying it in deploy_ccip.go I guess

Yeah, when we move to a separate contract. For now we can just rely on the Router deploying the individual registers

vicentevieytes added 9 commits June 18, 2026 19:32

tokenRegistry bindings and deployment

65078ef

rename module to tokenregistry

428c325

rename to lowercase

ba96184

fix tokenRegistry not in metadata

b504846

rename wrapper compile file

2f857ff

add missing shortToLong FQN

c6e0034

golint

2d013d1

ID instead of Id

e1676a4

golint deployment

53d64ba

vicentevieytes marked this pull request as ready for review June 19, 2026 02:42

vicentevieytes requested a review from a team as a code owner June 19, 2026 02:42

vicentevieytes requested review from Copilot and patricios-space June 19, 2026 02:42

Copilot started reviewing on behalf of vicentevieytes June 19, 2026 02:43 View session

Copilot AI reviewed Jun 19, 2026

View reviewed changes

Comment thread pkg/ccip/bindings/tokenregistry/tokenregistry.go

Comment thread deployment/ccip/sequence/deploy_ccip.go Outdated

Comment thread deployment/ccip/sequence/deploy_ccip.go Outdated

Comment thread deployment/state/state.go Outdated

vicentevieytes added 3 commits June 19, 2026 10:58

fix token registry address not stored in address book

15a9002

re-organize dynamic config type

199cfff

update wrappers and tests with new DynamicConfig layout

9e5dcae

vicentevieytes marked this pull request as draft June 23, 2026 13:30

vicentevieytes and others added 7 commits June 23, 2026 16:34

fix off-chain bindings and deployment

a57762f

Merge branch 'main' of github.com:smartcontractkit/chainlink-ton into…

276fdc8

… vv/token-registry-bindings-and-deployment

golint

c0031f3

TokenPool on ReturnTokenInfo should be raw address instead of maybe a…

5fd6a27

…ddress Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

Set minter address and token pool address as addressNone instead of null

3ee2029

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

Potential fix for pull request finding

2e92d5b

fix typo Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

fix typo

a9d37a1

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

vicentevieytes marked this pull request as ready for review June 23, 2026 19:54

vicentevieytes requested a review from krebernisak June 23, 2026 19:55

update generated onramp wrappers

5aa4221

krebernisak previously approved these changes Jun 26, 2026

View reviewed changes

patricios-space reviewed Jun 26, 2026

View reviewed changes

Comment thread contracts/contracts/ccip/token_registry/storage.tolk Outdated

patricios-space reviewed Jun 26, 2026

View reviewed changes

vicentevieytes added 4 commits June 26, 2026 16:51

shard token registry under the Router

b2c893f

revert changes to onRamp data structure

36af62e

restore firedrill onramp as well

babfe11

move to sharded implementation under the Router

266f955

vicentevieytes dismissed krebernisak’s stale review via 266f955 June 30, 2026 17:47

vicentevieytes added 4 commits June 30, 2026 14:49

remove tokenregistry deployment step

42b0aec

fix issues that would cause failing tests

6576a46

fix router golden hex and add tokenRegistryDeployment to builder

715128f

golint

2aa3fe0

patricios-space reviewed Jun 30, 2026

View reviewed changes

Comment thread contracts/contracts/ccip/onramp/messages.tolk Outdated

remove todo

22aa113

vicentevieytes commented Jul 1, 2026

View reviewed changes

fix comment

afa2faf

vicentevieytes requested a review from patricios-space July 1, 2026 15:18

patricios-space reviewed Jul 1, 2026

View reviewed changes

patricios-space approved these changes Jul 1, 2026

View reviewed changes

vicentevieytes merged commit dfd58a9 into main Jul 1, 2026
36 checks passed

vicentevieytes deleted the vv/token-registry-bindings-and-deployment branch July 1, 2026 22:44

Uh oh!

Conversation

vicentevieytes commented Jun 18, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

patricios-space Jun 26, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

patricios-space Jun 26, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

patricios-space left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

vicentevieytes commented Jun 18, 2026 •

edited

Loading

patricios-space Jun 26, 2026 •

edited

Loading

patricios-space Jun 26, 2026 •

edited

Loading