Skip to content

tcapi: chain-agnostic address lookups#1121

Draft
makramkd wants to merge 6 commits into
mainfrom
mk/try2/CCIP-10587
Draft

tcapi: chain-agnostic address lookups#1121
makramkd wants to merge 6 commits into
mainfrom
mk/try2/CCIP-10587

Conversation

@makramkd
Copy link
Copy Markdown
Collaborator

@makramkd makramkd commented May 19, 2026

Description

The current tcapi std test suite uses EVM-specific contract types and versions. AltVMs already use different types and versions, so we need a way for the test cases to fetch the appropriate contract address.

This PR introduces a new interface, AddressResolver, that can resolve specific addresses given a datastore, chain selector, and a qualifier. It is registered per-family in the main devenv chainreg.Registry.

Testing

Checklist

  • Breaking changes documented in changelog (see changelog directory)
  • Cross link related PRs (in this or other repositories)
  • just lint fix - no new lint errors
  • just generate - mocks and protobufs are up to date

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 2, 2026

Code coverage report:

Package main mk/try2/CCIP-10587 Diff
github.com/smartcontractkit/chainlink-ccv/aggregator 49.35% 49.35% +0.00%
github.com/smartcontractkit/chainlink-ccv/bootstrap 54.14% 54.14% +0.00%
github.com/smartcontractkit/chainlink-ccv/cli 65.13% 65.13% +0.00%
github.com/smartcontractkit/chainlink-ccv/cmd 15.54% 15.54% +0.00%
github.com/smartcontractkit/chainlink-ccv/common 56.54% 56.54% +0.00%
github.com/smartcontractkit/chainlink-ccv/executor 45.97% 45.97% +0.00%
github.com/smartcontractkit/chainlink-ccv/indexer 37.73% 37.68% -0.05%
github.com/smartcontractkit/chainlink-ccv/integration 46.23% 46.13% -0.10%
github.com/smartcontractkit/chainlink-ccv/pkg 84.62% 84.62% +0.00%
github.com/smartcontractkit/chainlink-ccv/pricer 0.00% 0.00% +0.00%
github.com/smartcontractkit/chainlink-ccv/protocol 63.06% 63.06% +0.00%
github.com/smartcontractkit/chainlink-ccv/verifier 34.48% 34.48% +0.00%
Total 46.60% 46.50% -0.10%

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.

2 participants