[Test] Combine sol-cap read trigger and min identical responses changes by amit-momin · Pull Request #2219 · smartcontractkit/chainlink-common

amit-momin · 2026-07-02T20:00:00Z

No description provided.

* add field to mark requests as external for solana service * fix format * update comments --------- Co-authored-by: Silas Lenihan <32529249+silaslenihan@users.noreply.github.com>

* pipe get program accounts * add nil check test cases * add mustEmbedUnimplementedClient * handle nil filters * fix tests * run generate * fix mocks

* wire slot to GetBalanceReply * add slot for GetFeeForMessageReply

* initial implementation of generalized monitoring applied to solana server * generate protos from parent directory * kvs renamed * cleanup monitoring * simplify metrics * []any -> KeyValue * UpperCase logs

…ead-methods

…o cre/combine-solcap-read-trigger-min-identical-responses

…-config-to-read-methods' into cre/combine-solcap-read-trigger-min-identical-responses

github-actions · 2026-07-02T20:01:37Z

⚠️ API Diff Results - `github.com/smartcontractkit/chainlink-common`

⚠️ Breaking Changes (7)

`pkg/capabilities/v2/chain-capabilities/solana/server.ClientCapability` (1)

MonitoringContext — ➕ Added

`pkg/capabilities/v2/protoc/pkg` (1)

GenerateServer — Type changed:

func(
  *google.golang.org/protobuf/compiler/protogen.Plugin, 
  *google.golang.org/protobuf/compiler/protogen.File, 
  ServerLanguage, 
  string, 
  string, 
  + bool
)
error

`pkg/chains/solana.SolanaClient` (1)

GetProgramAccounts — ➕ Added

`pkg/types.SolanaService` (1)

mustEmbedUnimplementedClient — ➕ Added

`pkg/types/chains/solana.Client` (3)

GetProgramAccounts — ➕ Added
GetProgramAccounts — ➕ Added
mustEmbedUnimplementedClient — ➕ Added

✅ Compatible Changes (83)

`package github` (1)

com/smartcontractkit/chainlink-common/pkg/capabilities/v2/monitoring — ➕ Added

`pkg/capabilities.RemoteExecutableConfig` (1)

MinResponsesToAggregate — ➕ Added

`pkg/capabilities/pb.(*RemoteExecutableConfig)` (1)

GetMinResponsesToAggregate — ➕ Added

`pkg/capabilities/pb.RemoteExecutableConfig` (1)

MinResponsesToAggregate — ➕ Added

`pkg/capabilities/v2/chain-capabilities/consensus/ocr3/types.(*ReportingPluginConfig)` (1)

GetMinResponsesToAggregate — ➕ Added

`pkg/capabilities/v2/chain-capabilities/consensus/ocr3/types.ReportingPluginConfig` (1)

MinResponsesToAggregate — ➕ Added

`pkg/capabilities/v2/chain-capabilities/solana` (2)

ConvertGetProgramAccountsReplyToProto — ➕ Added
ConvertGetProgramAccountsRequestFromProto — ➕ Added

`pkg/capabilities/v2/chain-capabilities/solana.(*WriteReportRequest)` (2)

LogKVs — ➕ Added
MetricKVs — ➕ Added

`pkg/capabilities/v2/chain-capabilities/solana/server.ClientCapability` (1)

MonitoringContext — ➕ Added

`pkg/chains/solana` (23)

ConvertGetAccountInfoRequestFromProto — ➕ Added
ConvertGetAccountInfoRequestToProto — ➕ Added
ConvertGetProgramAccountsOptsFromProto — ➕ Added
ConvertGetProgramAccountsOptsToProto — ➕ Added
ConvertGetProgramAccountsReplyFromProto — ➕ Added
ConvertGetProgramAccountsReplyToProto — ➕ Added
ConvertGetProgramAccountsRequestFromProto — ➕ Added
ConvertGetProgramAccountsRequestToProto — ➕ Added
ConvertKeyedAccountFromProto — ➕ Added
ConvertKeyedAccountToProto — ➕ Added
ConvertRPCFilterFromProto — ➕ Added
ConvertRPCFilterMemcmpFromProto — ➕ Added
ConvertRPCFilterMemcmpToProto — ➕ Added
ConvertRPCFiltersFromProto — ➕ Added
ConvertRPCFiltersToProto — ➕ Added
ConvertRPCFilterToProto — ➕ Added
GetProgramAccountsOpts — ➕ Added
GetProgramAccountsReply — ➕ Added
GetProgramAccountsRequest — ➕ Added
KeyedAccount — ➕ Added
RPCFilter — ➕ Added
RPCFilterMemcmp — ➕ Added
Solana_GetProgramAccounts_FullMethodName — ➕ Added

`pkg/chains/solana.(*GetAccountInfoWithOptsRequest)` (1)

GetIsExternal — ➕ Added

`pkg/chains/solana.(*GetBalanceReply)` (1)

GetSlot — ➕ Added

`pkg/chains/solana.(*GetFeeForMessageReply)` (1)

GetSlot — ➕ Added

`pkg/chains/solana.(*GetMultipleAccountsWithOptsRequest)` (1)

GetIsExternal — ➕ Added

`pkg/chains/solana.(*GetTransactionRequest)` (1)

GetIsExternal — ➕ Added

`pkg/chains/solana.(*SimulateTXRequest)` (1)

GetIsExternal — ➕ Added

`pkg/chains/solana.GetAccountInfoWithOptsRequest` (1)

IsExternal — ➕ Added

`pkg/chains/solana.GetBalanceReply` (1)

Slot — ➕ Added

`pkg/chains/solana.GetFeeForMessageReply` (1)

Slot — ➕ Added

`pkg/chains/solana.GetMultipleAccountsWithOptsRequest` (1)

IsExternal — ➕ Added

`pkg/chains/solana.GetTransactionRequest` (1)

IsExternal — ➕ Added

`pkg/chains/solana.SimulateTXRequest` (1)

IsExternal — ➕ Added

`pkg/chains/solana.SolanaServer` (1)

GetProgramAccounts — ➕ Added

`pkg/chains/solana.UnimplementedSolanaServer` (1)

GetProgramAccounts — ➕ Added

`pkg/http` (3)

GetResponseSizeLimit — ➕ Added
LimitedTransport — ➕ Added
WithResponseSizeLimit — ➕ Added

`pkg/settings/cresettings.chainRead` (1)

Solana — ➕ Added

`pkg/types.UnimplementedSolanaService` (1)

UnimplementedSolanaClient — ➕ Added

`pkg/types/chains/solana` (8)

ClientMustEmbed — ➕ Added
GetProgramAccountsOpts — ➕ Added
GetProgramAccountsReply — ➕ Added
GetProgramAccountsRequest — ➕ Added
KeyedAccount — ➕ Added
RPCFilter — ➕ Added
RPCFilterMemcmp — ➕ Added
UnimplementedSolanaClient — ➕ Added

`pkg/types/chains/solana.GetAccountInfoRequest` (1)

IsExternal — ➕ Added

`pkg/types/chains/solana.GetBalanceReply` (1)

Slot — ➕ Added

`pkg/types/chains/solana.GetFeeForMessageReply` (1)

Slot — ➕ Added

`pkg/types/chains/solana.GetMultipleAccountsRequest` (1)

IsExternal — ➕ Added

`pkg/types/chains/solana.GetTransactionRequest` (1)

IsExternal — ➕ Added

`pkg/types/chains/solana.SimulateTXRequest` (1)

IsExternal — ➕ Added

`pkg/types/chains/solana.UnimplementedSolanaClient` (10)

GetAccountInfoWithOpts — ➕ Added
GetBalance — ➕ Added
GetBlock — ➕ Added
GetFeeForMessage — ➕ Added
GetMultipleAccountsWithOpts — ➕ Added
GetProgramAccounts — ➕ Added
GetSignatureStatuses — ➕ Added
GetSlotHeight — ➕ Added
GetTransaction — ➕ Added
SimulateTX — ➕ Added

`pkg/types/mocks` (4)

SolanaService_GetProgramAccounts_Call — ➕ Added
SolanaService_mustEmbedUnimplementedClient_Call — ➕ Added
SolanaServiceShim — ➕ Added
WrapSolanaService — ➕ Added

`pkg/types/mocks.(*SolanaService)` (1)

GetProgramAccounts — ➕ Added

`pkg/types/mocks.(*SolanaService_Expecter)` (1)

GetProgramAccounts — ➕ Added

📄 View full apidiff report

amit-momin and others added 16 commits May 29, 2026 17:06

Added MinResponsesToAggregate config

3961b14

add field to mark requests as external for solana service (#2066)

609e75f

* add field to mark requests as external for solana service * fix format * update comments --------- Co-authored-by: Silas Lenihan <32529249+silaslenihan@users.noreply.github.com>

bump protos

c521c58

make tidy

1169e6d

Plex 2406/pipe get program accounts (#2126)

a45c846

* pipe get program accounts * add nil check test cases * add mustEmbedUnimplementedClient * handle nil filters * fix tests * run generate * fix mocks

Merge branch 'bump-protos' into feature/solcap-read_trigger

08c3cd7

add proto conversions for GetProgramAccounts (#2128)

27a72f7

Merge branch 'main' into feature/solcap-read_trigger

5402cc8

wire slot to GetBalanceReply (#2141)

e951bb2

* wire slot to GetBalanceReply * add slot for GetFeeForMessageReply

PLEX-2935/generalize monitoring (#2132)

7471895

* initial implementation of generalized monitoring applied to solana server * generate protos from parent directory * kvs renamed * cleanup monitoring * simplify metrics * []any -> KeyValue * UpperCase logs

Add Solana limit settings (#2186)

d1f5898

Merge branch 'main' into feature/solcap-read_trigger

809397d

Merge branch 'main' into feature/solcap-read_trigger

d861fa9

Merge branch 'main' into cre/add-min-response-aggregation-config-to-r…

715aee9

…ead-methods

Merge remote-tracking branch 'origin/feature/solcap-read_trigger' int…

d2d2400

…o cre/combine-solcap-read-trigger-min-identical-responses

Merge remote-tracking branch 'origin/cre/add-min-response-aggregation…

6b7d10f

…-config-to-read-methods' into cre/combine-solcap-read-trigger-min-identical-responses

amit-momin added the do-not-merge label Jul 2, 2026

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

[Test] Combine sol-cap read trigger and min identical responses changes#2219

[Test] Combine sol-cap read trigger and min identical responses changes#2219
amit-momin wants to merge 16 commits into
mainfrom
cre/combine-solcap-read-trigger-min-identical-responses

amit-momin commented Jul 2, 2026

Uh oh!

github-actions Bot commented Jul 2, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Uh oh!

Conversation

amit-momin commented Jul 2, 2026

Uh oh!

github-actions Bot commented Jul 2, 2026

⚠️ API Diff Results - github.com/smartcontractkit/chainlink-common

⚠️ Breaking Changes (7)

pkg/capabilities/v2/chain-capabilities/solana/server.ClientCapability (1)

pkg/capabilities/v2/protoc/pkg (1)

pkg/chains/solana.SolanaClient (1)

pkg/types.SolanaService (1)

pkg/types/chains/solana.Client (3)

✅ Compatible Changes (83)

package github (1)

pkg/capabilities.RemoteExecutableConfig (1)

pkg/capabilities/pb.(*RemoteExecutableConfig) (1)

pkg/capabilities/pb.RemoteExecutableConfig (1)

pkg/capabilities/v2/chain-capabilities/consensus/ocr3/types.(*ReportingPluginConfig) (1)

pkg/capabilities/v2/chain-capabilities/consensus/ocr3/types.ReportingPluginConfig (1)

pkg/capabilities/v2/chain-capabilities/solana (2)

pkg/capabilities/v2/chain-capabilities/solana.(*WriteReportRequest) (2)

pkg/capabilities/v2/chain-capabilities/solana/server.ClientCapability (1)

pkg/chains/solana (23)

pkg/chains/solana.(*GetAccountInfoWithOptsRequest) (1)

pkg/chains/solana.(*GetBalanceReply) (1)

pkg/chains/solana.(*GetFeeForMessageReply) (1)

pkg/chains/solana.(*GetMultipleAccountsWithOptsRequest) (1)

pkg/chains/solana.(*GetTransactionRequest) (1)

pkg/chains/solana.(*SimulateTXRequest) (1)

pkg/chains/solana.GetAccountInfoWithOptsRequest (1)

pkg/chains/solana.GetBalanceReply (1)

pkg/chains/solana.GetFeeForMessageReply (1)

pkg/chains/solana.GetMultipleAccountsWithOptsRequest (1)

pkg/chains/solana.GetTransactionRequest (1)

pkg/chains/solana.SimulateTXRequest (1)

pkg/chains/solana.SolanaServer (1)

pkg/chains/solana.UnimplementedSolanaServer (1)

pkg/http (3)

pkg/settings/cresettings.chainRead (1)

pkg/types.UnimplementedSolanaService (1)

pkg/types/chains/solana (8)

pkg/types/chains/solana.GetAccountInfoRequest (1)

pkg/types/chains/solana.GetBalanceReply (1)

pkg/types/chains/solana.GetFeeForMessageReply (1)

pkg/types/chains/solana.GetMultipleAccountsRequest (1)

pkg/types/chains/solana.GetTransactionRequest (1)

pkg/types/chains/solana.SimulateTXRequest (1)

pkg/types/chains/solana.UnimplementedSolanaClient (10)

pkg/types/mocks (4)

pkg/types/mocks.(*SolanaService) (1)

pkg/types/mocks.(*SolanaService_Expecter) (1)

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

⚠️ API Diff Results - `github.com/smartcontractkit/chainlink-common`

`pkg/capabilities/v2/chain-capabilities/solana/server.ClientCapability` (1)

`pkg/capabilities/v2/protoc/pkg` (1)

`pkg/chains/solana.SolanaClient` (1)

`pkg/types.SolanaService` (1)

`pkg/types/chains/solana.Client` (3)

`package github` (1)

`pkg/capabilities.RemoteExecutableConfig` (1)

`pkg/capabilities/pb.(*RemoteExecutableConfig)` (1)

`pkg/capabilities/pb.RemoteExecutableConfig` (1)

`pkg/capabilities/v2/chain-capabilities/consensus/ocr3/types.(*ReportingPluginConfig)` (1)

`pkg/capabilities/v2/chain-capabilities/consensus/ocr3/types.ReportingPluginConfig` (1)

`pkg/capabilities/v2/chain-capabilities/solana` (2)

`pkg/capabilities/v2/chain-capabilities/solana.(*WriteReportRequest)` (2)

`pkg/capabilities/v2/chain-capabilities/solana/server.ClientCapability` (1)

`pkg/chains/solana` (23)

`pkg/chains/solana.(*GetAccountInfoWithOptsRequest)` (1)

`pkg/chains/solana.(*GetBalanceReply)` (1)

`pkg/chains/solana.(*GetFeeForMessageReply)` (1)

`pkg/chains/solana.(*GetMultipleAccountsWithOptsRequest)` (1)

`pkg/chains/solana.(*GetTransactionRequest)` (1)

`pkg/chains/solana.(*SimulateTXRequest)` (1)

`pkg/chains/solana.GetAccountInfoWithOptsRequest` (1)

`pkg/chains/solana.GetBalanceReply` (1)

`pkg/chains/solana.GetFeeForMessageReply` (1)

`pkg/chains/solana.GetMultipleAccountsWithOptsRequest` (1)

`pkg/chains/solana.GetTransactionRequest` (1)

`pkg/chains/solana.SimulateTXRequest` (1)

`pkg/chains/solana.SolanaServer` (1)

`pkg/chains/solana.UnimplementedSolanaServer` (1)

`pkg/http` (3)

`pkg/settings/cresettings.chainRead` (1)

`pkg/types.UnimplementedSolanaService` (1)

`pkg/types/chains/solana` (8)

`pkg/types/chains/solana.GetAccountInfoRequest` (1)

`pkg/types/chains/solana.GetBalanceReply` (1)

`pkg/types/chains/solana.GetFeeForMessageReply` (1)

`pkg/types/chains/solana.GetMultipleAccountsRequest` (1)

`pkg/types/chains/solana.GetTransactionRequest` (1)

`pkg/types/chains/solana.SimulateTXRequest` (1)

`pkg/types/chains/solana.UnimplementedSolanaClient` (10)

`pkg/types/mocks` (4)

`pkg/types/mocks.(*SolanaService)` (1)

`pkg/types/mocks.(*SolanaService_Expecter)` (1)