Skip to content

NONEVM-1891: add TVM support in Fee Quoter for barebone TON CCIP e2e#962

Merged
jadepark-dev merged 32 commits intomainfrom
NONEVM-1891-Add-TON-address-validation-support-to-FeeQuoter-contract-on-EVM-CCIP
Jun 17, 2025
Merged

NONEVM-1891: add TVM support in Fee Quoter for barebone TON CCIP e2e#962
jadepark-dev merged 32 commits intomainfrom
NONEVM-1891-Add-TON-address-validation-support-to-FeeQuoter-contract-on-EVM-CCIP

Conversation

@jadepark-dev
Copy link
Copy Markdown
Contributor

@jadepark-dev jadepark-dev commented May 30, 2025

This PR adds TVM(TON) support to the Fee Quoter contract for TON CCIP integration NONEVM-1891

Primary Changes

  • Added TVM chain family selector support to enable TON address validation in the Fee Quoter contract
  • Implemented _validateTVMAddress function to validate 36-byte TON user-friendly addresses with basic format checks
  • Extended fee calculation logic to include TVM chain family
  • Added tests for the internal lib

@jadepark-dev jadepark-dev marked this pull request as ready for review June 2, 2025 18:07
@jadepark-dev jadepark-dev requested a review from a team as a code owner June 2, 2025 18:07
@RensR
Copy link
Copy Markdown
Collaborator

RensR commented Jun 4, 2025

Please change the version of the feeQuoter to 1.6.1-dev to indicate this is unaudited and not 1.6.0

Comment thread chains/evm/contracts/libraries/Internal.sol Outdated
@jadepark-dev jadepark-dev requested a review from archseer June 4, 2025 11:42
Comment thread chains/evm/contracts/libraries/Internal.sol
@jadepark-dev jadepark-dev requested a review from matYang June 11, 2025 15:52
Comment thread chains/evm/contracts/test/helpers/InternalTestHelper.sol Outdated
Comment thread chains/evm/contracts/test/libraries/Internal.t.sol Outdated
Comment thread chains/evm/contracts/test/libraries/Internal.t.sol Outdated
Comment thread chains/evm/contracts/test/libraries/Internal.t.sol Outdated
Comment thread chains/evm/contracts/test/libraries/Internal.t.sol Outdated
Comment thread chains/evm/contracts/test/libraries/Internal.t.sol Outdated
Comment thread chains/evm/contracts/test/libraries/Internal.t.sol Outdated
Comment thread chains/evm/contracts/test/libraries/Internal.t.sol Outdated
Comment thread chains/evm/contracts/test/libraries/Internal.t.sol Outdated
@github-actions
Copy link
Copy Markdown

Metric NONEVM-1891-Add-TON-address-validation-support-to-FeeQuoter-contract-on-EVM-CCIP main
Coverage 70.0% 69.6%

@jadepark-dev jadepark-dev requested a review from RensR June 17, 2025 10:30
@jadepark-dev jadepark-dev added this pull request to the merge queue Jun 17, 2025
Merged via the queue into main with commit 81753bd Jun 17, 2025
46 checks passed
@jadepark-dev jadepark-dev deleted the NONEVM-1891-Add-TON-address-validation-support-to-FeeQuoter-contract-on-EVM-CCIP branch June 17, 2025 10:56
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