Skip to content

Update relayer ccip provider interface to use dedicated args#1497

Merged
archseer merged 1 commit intomainfrom
feature/update-relayer-ccip-provider-interface
Sep 3, 2025
Merged

Update relayer ccip provider interface to use dedicated args#1497
archseer merged 1 commit intomainfrom
feature/update-relayer-ccip-provider-interface

Conversation

@amit-momin
Copy link
Copy Markdown
Contributor

@amit-momin amit-momin commented Aug 21, 2025

Added dedicated args specifically for the NewCCIPProvider method in the Relayer to support initializing the default provider in the Relayer. The generic relayer args were not providing the contract reader and chain reader config to initialize both for the default chain accessor.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Aug 21, 2025

apidiff results - backwards-incompatible changes detected ❌

Module: github-com-smartcontractkit-chainlink-common

Status: ❌ 6 incompatible, 1 compatible

Incompatible Changes (6)

Element Change
./pkg/loop.(*RelayerService).NewCCIPProvider changed from func(context.Context, github.com/smartcontractkit/chainlink-common/pkg/types.RelayArgs) (github.com/smartcontractkit/chainlink-common/pkg/types.CCIPProvider, error) to func(context.Context, github.com/smartcontractkit/chainlink-common/pkg/types.CCIPProviderArgs) (github.com/smartcontractkit/chainlink-common/pkg/types.CCIPProvider, error)
./pkg/loop/mocks.(*Relayer_NewCCIPProvider_Call).Run changed from func(func(_a0 context.Context, _a1 github.com/smartcontractkit/chainlink-common/pkg/types.RelayArgs)) *Relayer_NewCCIPProvider_Call to func(func(_a0 context.Context, _a1 github.com/smartcontractkit/chainlink-common/pkg/types.CCIPProviderArgs)) *Relayer_NewCCIPProvider_Call
./pkg/loop/mocks.(*Relayer_NewCCIPProvider_Call).RunAndReturn changed from func(func(context.Context, github.com/smartcontractkit/chainlink-common/pkg/types.RelayArgs) (github.com/smartcontractkit/chainlink-common/pkg/types.CCIPProvider, error)) *Relayer_NewCCIPProvider_Call to func(func(context.Context, github.com/smartcontractkit/chainlink-common/pkg/types.CCIPProviderArgs) (github.com/smartcontractkit/chainlink-common/pkg/types.CCIPProvider, error)) *Relayer_NewCCIPProvider_Call
./pkg/loop/mocks.(*Relayer).NewCCIPProvider changed from func(context.Context, github.com/smartcontractkit/chainlink-common/pkg/types.RelayArgs) (github.com/smartcontractkit/chainlink-common/pkg/types.CCIPProvider, error) to func(context.Context, github.com/smartcontractkit/chainlink-common/pkg/types.CCIPProviderArgs) (github.com/smartcontractkit/chainlink-common/pkg/types.CCIPProvider, error)
./pkg/types.(*UnimplementedRelayer).NewCCIPProvider changed from func(context.Context, RelayArgs) (CCIPProvider, error) to func(context.Context, CCIPProviderArgs) (CCIPProvider, error)
./pkg/types.Relayer.NewCCIPProvider changed from func(context.Context, RelayArgs) (CCIPProvider, error) to func(context.Context, CCIPProviderArgs) (CCIPProvider, error)

(Full summary: https://github.com/smartcontractkit/chainlink-common/actions/runs/17418440861/#summary-49451847127)

@ogtownsend
Copy link
Copy Markdown
Contributor

Hey @archseer and @jmank88 , could I get a restamp on this PR please?

@archseer archseer merged commit 5f5c55e into main Sep 3, 2025
20 of 22 checks passed
@archseer archseer deleted the feature/update-relayer-ccip-provider-interface branch September 3, 2025 13:13
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.

5 participants