Skip to content

Add Canton client and Manual Exec#161

Open
RodrigoAD wants to merge 17 commits intomainfrom
add-canton-client
Open

Add Canton client and Manual Exec#161
RodrigoAD wants to merge 17 commits intomainfrom
add-canton-client

Conversation

@RodrigoAD
Copy link
Copy Markdown
Member

@RodrigoAD RodrigoAD commented Mar 2, 2026

  • Add Canton JSON ledger API Client wrapper
  • Add Canton contract discovery through ACS and EDS
  • Add CCV verifications fetch through CCIP Indexer
  • Add Manual Execution SDK and Command
  • Add Send Message SDK

@vercel
Copy link
Copy Markdown

vercel Bot commented Mar 2, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
ccip-tools-ts Ready Ready Preview, Comment Mar 26, 2026 10:43am

Request Review

Comment thread ccip-sdk/src/canton/index.ts
Comment thread ccip-sdk/src/canton/index.ts Outdated
@RodrigoAD RodrigoAD force-pushed the add-canton-client branch from 9a3bffd to abb9065 Compare March 9, 2026 12:24
@RodrigoAD RodrigoAD changed the title Add canton client Add Canton client and Manual Exec Mar 9, 2026
@RodrigoAD RodrigoAD marked this pull request as ready for review March 9, 2026 12:29
@RodrigoAD RodrigoAD requested a review from a team as a code owner March 9, 2026 12:29
@RodrigoAD RodrigoAD requested a review from andrevmatos March 9, 2026 12:29
Comment thread ccip-sdk/src/canton/explicit-disclosures/acs.ts Outdated
Comment thread ccip-sdk/src/canton/explicit-disclosures/acs.ts Outdated
Comment on lines +493 to +494
templateId: acsDisclosures.ccipReceiver.templateId,
contractId: acsDisclosures.ccipReceiver.contractId,
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This would actually have to be specified by the user themselves - CCIPReceiver is mainly an example contract

Comment thread ccip-sdk/src/selectors.ts
},
// end:generate

// generate:
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

think this should be removed?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

This is to add and autogenerate Canton selectors, we need it

Comment thread ccip-sdk/src/chain.ts
Comment thread ccip-sdk/src/evm/abi/OffRamp_2_0.ts Outdated
| 'api'
| 'cantonJwt'
| 'cantonParty'
| 'cantonEdsUrl'
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

@andrevmatos we need somehow all these values from the user input, what's the recommended way to get them through the CLI?

This way I understand is not ideal

@@ -0,0 +1,7637 @@
/* eslint-disable */
/**
* This file was auto-generated by openapi-typescript.
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Unfortunately we there's no lib that gives us this info.

Let me know if you prefer any other way to handle this @andrevmatos

@@ -0,0 +1,7637 @@
/* eslint-disable */
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

We could autogenerate this with the generate tool

edsDisclosureProvider: EdsDisclosureProvider,
transferInstructionClient: TransferInstructionClient,
tokenMetadataClient: TokenMetadataClient,
ccipParty: string,
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Check if we can autodiscover this

| 'api'
| 'cantonJwt'
| 'cantonParty'
| 'cantonEdsUrl'
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Offchain services URLs could be hardcoded

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.

3 participants