Skip to content

Add CCIP types required for Solana transmitter and codec migrations out of core#1555

Closed
amit-momin wants to merge 8 commits intoogt/extra-data-codec-registry-interface-v3from
feature/add-ccip-provider-plugin-type
Closed

Add CCIP types required for Solana transmitter and codec migrations out of core#1555
amit-momin wants to merge 8 commits intoogt/extra-data-codec-registry-interface-v3from
feature/add-ccip-provider-plugin-type

Conversation

@amit-momin
Copy link
Contributor

@amit-momin amit-momin commented Sep 17, 2025

@github-actions
Copy link

github-actions bot commented Sep 17, 2025

apidiff results - backwards-incompatible changes detected ❌

Module: github-com-smartcontractkit-chainlink-common

Status: ❌ 2 incompatible, 5 compatible

Incompatible Changes (2)

Element Change
./pkg/types.CCIPProviderArgs.OffRampAddress changed from string to github.com/smartcontractkit/chainlink-common/pkg/types/ccipocr3.UnknownAddress
./pkg/types.CCIPProviderArgs.PluginType changed from uint32 to github.com/smartcontractkit/chainlink-common/pkg/types/ccipocr3.PluginType

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

@amit-momin amit-momin changed the title Add CCIP PluginType Add CCIP PluginType and Transmitter field Sep 17, 2025
@amit-momin amit-momin force-pushed the feature/add-ccip-provider-plugin-type branch from 3e113e6 to 3ace32f Compare September 17, 2025 22:05
@amit-momin amit-momin changed the base branch from main to ogt/extra-data-codec-registry-interface-v3 September 17, 2025 22:06
Comment on lines +377 to +396

// PluginType represents the type of CCIP plugin.
// It mirrors the OCRPluginType in Internal.sol.
type PluginType uint8

const (
PluginTypeCCIPCommit PluginType = 0
PluginTypeCCIPExec PluginType = 1
)

func (pt PluginType) String() string {
switch pt {
case PluginTypeCCIPCommit:
return "CCIPCommit"
case PluginTypeCCIPExec:
return "CCIPExec"
default:
return "Unknown"
}
}
Copy link
Contributor

Choose a reason for hiding this comment

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

👍 and we can alias the one from chainlink/core/capabilities/ccip/types/types.go to point here

@ogtownsend ogtownsend force-pushed the ogt/extra-data-codec-registry-interface-v3 branch 3 times, most recently from d712adb to 86877f3 Compare September 18, 2025 16:15
@amit-momin amit-momin force-pushed the feature/add-ccip-provider-plugin-type branch from 6ed0349 to eec97e7 Compare September 18, 2025 16:54
@amit-momin amit-momin changed the title Add CCIP PluginType and Transmitter field Add CCIP types required for Solana transmitter and codec migrations out of core Sep 18, 2025
@amit-momin amit-momin force-pushed the feature/add-ccip-provider-plugin-type branch from aa8b416 to 185e5e4 Compare September 18, 2025 17:16
@ogtownsend ogtownsend force-pushed the ogt/extra-data-codec-registry-interface-v3 branch from 06d4446 to 661a3a9 Compare September 22, 2025 18:09
@amit-momin amit-momin force-pushed the feature/add-ccip-provider-plugin-type branch from 3994214 to 915d7ed Compare September 22, 2025 20:49
@ogtownsend ogtownsend force-pushed the ogt/extra-data-codec-registry-interface-v3 branch from 661a3a9 to 3d1af8f Compare September 24, 2025 22:28
@amit-momin
Copy link
Contributor Author

Changes included as part of #1561

@amit-momin amit-momin closed this Sep 26, 2025
@amit-momin amit-momin deleted the feature/add-ccip-provider-plugin-type branch October 2, 2025 16:45
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