Skip to content

fix: bump chainlink-proto dependency - #1457

Merged
huangzhen1997 merged 2 commits into
mainfrom
jh/bump-protos-main
Sep 17, 2026
Merged

huangzhen1997 merged 2 commits into
mainfrom
jh/bump-protos-main

Conversation

@huangzhen1997

@huangzhen1997 huangzhen1997 commented Sep 16, 2026

Copy link
Copy Markdown
Contributor

Summary

Bump github.com/smartcontractkit/chainlink-protos/cre/go from v0.0.0-20260804191526-b7a850ae7648 to v0.0.0-20260916140808-f5d22b3cb3ee (latest main) in all modules that pin it.

Why

  • The old commit b7a850ae7648 is not reachable from the main branch of chainlink-protos. Upstream squash-merges drop pre-merge commits from the branch history.
  • Repositories that validate dependency versions against the default branch fail when they bump their chainlink-ccv pin. chainlink-ccip-solana hit this failure after fix: devenv grafana service log #1452 merged.
  • The pin arrived through the chainlink-common bump in fix: devenv grafana service log #1452; chainlink-common@v0.11.2-0.20260915214759-02f2214d5823 requires the old version.
  • The module proxy still serves the old version, so builds pass. The defect affects dependency validation in downstream repositories only.

Validation

  • gomods tidy and just tidy complete with no further changes.
  • Root, build/devenv, build/devenv/fakes, deployment, and integration/evm modules build.

References

  • chainlink-ccip-solana validation failure: dependency chainlink-protos/cre/go@v0.0.0-20260804191526-b7a850ae7648 not found in (main)
  • Pin introduced by fix: devenv grafana service log #1452

@huangzhen1997 huangzhen1997 changed the title bump chainlink-proto dependency fix: bump chainlink-proto dependency Sep 16, 2026
@github-actions

Copy link
Copy Markdown

Code coverage report:

Package main jh/bump-protos-main Diff
github.com/smartcontractkit/chainlink-ccv/aggregator 50.82% 50.85% +0.03%
github.com/smartcontractkit/chainlink-ccv/bootstrap 72.00% 72.00% +0.00%
github.com/smartcontractkit/chainlink-ccv/cli 50.62% 50.62% +0.00%
github.com/smartcontractkit/chainlink-ccv/cmd 39.10% 39.10% +0.00%
github.com/smartcontractkit/chainlink-ccv/common 48.13% 48.13% +0.00%
github.com/smartcontractkit/chainlink-ccv/executor 42.80% 42.80% +0.00%
github.com/smartcontractkit/chainlink-ccv/indexer 36.78% 36.82% +0.04%
github.com/smartcontractkit/chainlink-ccv/integration 61.39% 61.39% +0.00%
github.com/smartcontractkit/chainlink-ccv/internal 0.00% 0.00% +0.00%
github.com/smartcontractkit/chainlink-ccv/migration 78.70% 78.70% +0.00%
github.com/smartcontractkit/chainlink-ccv/pkg 100.00% 100.00% +0.00%
github.com/smartcontractkit/chainlink-ccv/pricer 0.00% 0.00% +0.00%
github.com/smartcontractkit/chainlink-ccv/protocol 67.04% 67.04% +0.00%
github.com/smartcontractkit/chainlink-ccv/tools 43.61% 43.61% +0.00%
github.com/smartcontractkit/chainlink-ccv/verifier 36.94% 36.94% +0.00%
Total 51.10% 51.10% +0.00%

@huangzhen1997
huangzhen1997 marked this pull request as ready for review September 16, 2026 22:24
@huangzhen1997
huangzhen1997 requested review from a team as code owners September 16, 2026 22:24
Copilot AI lite review requested due to automatic review settings September 16, 2026 22:24

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Approval recommended

No unresolved issues were identified, and all dependency pins and checksums are updated consistently.

Pull request overview

Updates the Chainlink CRE proto dependency to a commit reachable from upstream main.

Changes:

  • Bumps the dependency across five Go modules.
  • Refreshes corresponding go.sum checksums.
  • Removes references to the unreachable version.
File summaries
File Description
integration/evm/go.sum Refreshes EVM checksums
integration/evm/go.mod Updates EVM dependency pin
go.sum Updates root checksums
go.mod Updates root dependency pin
deployment/go.sum Refreshes deployment checksums
deployment/go.mod Updates deployment dependency pin
build/devenv/go.sum Refreshes devenv checksums
build/devenv/go.mod Updates devenv dependency pin
build/devenv/fakes/go.sum Refreshes fakes checksums
build/devenv/fakes/go.mod Updates fakes dependency pin
Review details
  • Files reviewed: 5/10 changed files
  • Comments generated: 0
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@huangzhen1997
huangzhen1997 added this pull request to the merge queue Sep 17, 2026
Merged via the queue into main with commit 1b32fa7 Sep 17, 2026
49 checks passed
@huangzhen1997
huangzhen1997 deleted the jh/bump-protos-main branch September 17, 2026 13: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.

4 participants