Add TON support #486

krebernisak · 2025-09-25T15:25:24Z

No description provided.

changeset-bot · 2025-09-25T15:25:28Z

🦋 Changeset detected

Latest commit: 099a18a

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

This PR includes changesets to release 1 package

Name	Type
@smartcontractkit/mcms	Minor

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

gustavogama-cll · 2025-09-25T16:11:09Z

.envrc

is nix mandatory for ton?

Hey! Nix is added to support dev environments for devs that use Nix.

Nix is also the way to get chainlink-ton packages, in this case I will need the TON contracts package, which I can easily pull from source with Nix. This is required to deploy contracts.

Same as we import Go bindings from a commit with Go,
we import TON contract artifacts from a commit with Nix.

Nix is an orthogonal add-on - this means that folks can use it or set up things manually.

For TON, some tools like Go need to be in a path and TON compiled contracts need to be available on a path defined with an env var. One can just run a prepared Nix shell to get all of that, or set up things manually.

Once I get everything working, I will also document how to get a TON env running manually as well.

gustavogama-cll

I have a few requests, if you please.

add a changeset file
remove the .envrc file. We don't mind if you want to use nix, but we don't want to enforce it by default
consider adding unit tests; I understand it's just an initial PR, but we do try have good coverage in the library. Ideally, in every PR.

gustavogama-cll · 2025-09-25T16:31:59Z

I see the PR was marked as draft later... If it's not ready for review yet, please ignore my remarks.

krebernisak · 2025-09-26T15:03:37Z

I see the PR was marked as draft later... If it's not ready for review yet, please ignore my remarks.

Thx for a quick review! Yeah, still an early draft trying to move things forward.

e2e/tests/ton/timelock_inspection.go

ecPablo · 2025-11-04T02:35:44Z

sdk/ton/decoder.go

+// TODO: unify and move these definitions to smartcontractkit/chainlink-ton
+var TLBsByContract = map[string]map[uint64]interface{}{
+	// Jetton contract types
+	"com.github.ton-blockchain.jetton-contract.contracts.jetton-wallet": wallet.TLBs,
+	"com.github.ton-blockchain.jetton-contract.contracts.jetton-minter": minter.TLBs,
+	// CCIP contract types
+	"com.chainlink.ton.ccip.Router":           router.TLBs,
+	"com.chainlink.ton.ccip.OnRamp":           onramp.TLBs,
+	"com.chainlink.ton.ccip.OffRamp":          offramp.TLBs,
+	"com.chainlink.ton.ccip.FeeQuoter":        feequoter.TLBs,
+	"com.chainlink.ton.ccip.CCIPSendExecutor": ccipsendexecutor.TLBs,
+	// MCMS contract types
+	"com.chainlink.ton.lib.access.RBAC": rbac.TLBs,
+	"com.chainlink.ton.mcms.MCMS":       mcms.TLBs,
+	"com.chainlink.ton.mcms.Timelock":   timelock.TLBs,
+}


just flagging that this probably needs to be moved to chainlink-token as the TODO indicates

Sure! I've left a couple of TODOs re: code structure across the PR.

Once I'm done with e2e tests, I'll these before merging. Some utility code and bindings can be moved and hosted in https://github.com/smartcontractkit/chainlink-ton

ecPablo · 2025-11-04T02:36:21Z

sdk/ton/decoder.go

+	}
+
+	// Extract the input keys and args (tree/map lvl 0)
+	keys, err := lib.DecodeTLBStructKeys(v, tlbs)


might want to handle the err here

ecPablo · 2025-11-04T02:42:31Z

sdk/ton/executor_test.go

+
+			tx, err := executor.ExecuteOperation(ctx, tt.metadata, tt.nonce, tt.proof, tt.op)
+
+			assert.Equal(t, tt.wantTxHash, tx.Hash)


nit: might want to move this after the error check just to ensure err is nil before asserting

sdk/ton/inspector.go

ecPablo · 2025-11-04T02:51:03Z

e2e/tests/runner_test.go

 }
+
+func TestTONSuite(t *testing.T) {
+	suite.Run(t, new(tone2e.TimelockInspectionTestSuite))


I'd strongly suggest to add the rest e2e tests before merging this one if possible. In past implementations like EVM/Solana they were super helpful to catch errors early. It saved us a lot of time specially when integrating with CLD. I know its a lot of work, so if not possible in this one at least having a followup before releasing the first TON mcms lib version

Sure! At this point (1) implementation done, (2) unit test done (EVM parity), (3) e2e tests WIP

krebernisak requested a review from a team as a code owner September 25, 2025 15:25

product-security-plaid-test bot requested a review from ecPablo September 25, 2025 15:25

krebernisak marked this pull request as draft September 25, 2025 15:25

gustavogama-cll reviewed Sep 25, 2025

View reviewed changes

krebernisak force-pushed the feat/ton-support branch from d2cd41a to 0154364 Compare October 22, 2025 09:31

krebernisak force-pushed the feat/ton-support branch 4 times, most recently from 01797f9 to 3e808a9 Compare November 3, 2025 12:48

krebernisak marked this pull request as ready for review November 3, 2025 14:10

krebernisak force-pushed the feat/ton-support branch from ee5b77b to bc628b4 Compare November 3, 2025 15:25

ecPablo reviewed Nov 4, 2025

View reviewed changes

krebernisak added 14 commits November 4, 2025 11:51

Setup env

36eca40

Add TON domain - add configurer

0ac14d4

Encode SetConfig message

ab29df2

Add ton.ConfigTransformer

8d4ffc3

Implement transformer.ToConfig

f13911c

Add sdk/ton/encoder.go

821891d

Add TON Encoder.HashMetadata

6325452

Add sdk/ton/inspector.go

60bc62a

Add sdk.Inspector#GetRootMetadata

9e04ccc

Add sdk/ton/timelock_inspector.go

77a2f90

Add sdk/ton/transaction.go

e74fda1

Add sdk/ton/timelock_executor.go

d501d4c

Add sdk/ton/timelock_converter.go

cd949ad

Add sdk/ton/executor.go

9b4480b

krebernisak added 22 commits November 4, 2025 11:51

Add signature and proof encoders

65a458c

Update signatures and proof encoders type

b4833c3

Add sdk/ton/decoder.go

96012a5

Bump github.com/smartcontractkit/chainlink-ton

7fe80ad

Add cselectors.FamilyTon as option

da11bcc

Add changesets and test entrypoints

1cea1ff

Add TON e2e setup + TimelockInspectionTestSuite stub (evm)

ee976cd

Add Configurer option to not send transactions (NONEVM-2862) - TON

63526fd

Add sdk/ton/decoded_operation_test.go

f3b1fd1

Add sdk/ton/decoder_test.go

ebd0385

Add sdk/ton/encoder_test.go

3250299

Add sdk/ton/config_transformer_test.go

ab2ff59

Add sdk/ton/timelock_converter_test.go

4163a29

Configure mockery v2

7903018

Bump chainlink-ton

645e48c

Add wallet.TonAPI mock

da04273

Add sdk/ton/configurer_test.go

b2b676d

Add sdk/ton/executor_test.go

2d8d725

Add sdk/ton/timelock_executor_test.go

868545a

Add timelock inspector getRoleMembers helper func

d14c388

Add sdk/ton/timelock_inspector_test.go

e35bd66

Add sdk/ton/timelock_inspector_test.go

099a18a

krebernisak force-pushed the feat/ton-support branch from 9e33a9f to 099a18a Compare November 4, 2025 10:52

krebernisak mentioned this pull request Nov 4, 2025

Dynamic msg TL-B - JSON decoding + chainlink-ton-extras pkg smartcontractkit/chainlink-ton#269

Merged


		tx, err := executor.ExecuteOperation(ctx, tt.metadata, tt.nonce, tt.proof, tt.op)

		assert.Equal(t, tt.wantTxHash, tx.Hash)

Add TON support #486

Are you sure you want to change the base?

Add TON support #486

Uh oh!

Conversation

krebernisak commented Sep 25, 2025

Uh oh!

changeset-bot bot commented Sep 25, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

🦋 Changeset detected

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

gustavogama-cll left a comment

Choose a reason for hiding this comment

Uh oh!

gustavogama-cll commented Sep 25, 2025

Uh oh!

krebernisak commented Sep 26, 2025

Uh oh!

Uh oh!

Uh oh!

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!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

changeset-bot bot commented Sep 25, 2025 •

edited

Loading