Skip to content

update chainlink-sui#22173

Merged
FelixFan1992 merged 1 commit intodevelopfrom
upgrade-chainlink-sui
Apr 24, 2026
Merged

update chainlink-sui#22173
FelixFan1992 merged 1 commit intodevelopfrom
upgrade-chainlink-sui

Conversation

@FelixFan1992
Copy link
Copy Markdown
Contributor

Requires

Supports

@github-actions
Copy link
Copy Markdown
Contributor

I see you updated files related to core. Please run make gocs in the root directory to add a changeset as well as in the text include at least one of the following tags:

  • #added For any new functionality added.
  • #breaking_change For any functionality that requires manual action for the node to boot.
  • #bugfix For bug fixes.
  • #changed For any change to the existing functionality.
  • #db_update For any feature that introduces updates to database schema.
  • #deprecation_notice For any upcoming deprecation functionality.
  • #internal For changesets that need to be excluded from the final changelog.
  • #nops For any feature that is NOP facing and needs to be in the official Release Notes for the release.
  • #removed For any functionality/config that is removed.
  • #updated For any functionality that is updated.
  • #wip For any change that is not ready yet and external communication about it should be held off till it is feature complete.

@github-actions
Copy link
Copy Markdown
Contributor

✅ No conflicts with other open PRs targeting develop

@trunk-io
Copy link
Copy Markdown

trunk-io Bot commented Apr 24, 2026

Static BadgeStatic BadgeStatic BadgeStatic Badge

Failed Test Failure Summary Logs
TestCompute_SpendValueRelativeToComputeTime The test 'TestCompute_SpendValueRelativeToComputeTime' failed during execution. Logs ↗︎
TestCompute_SpendValueRelativeToComputeTime/3s The test failed because a computed value was not less than the expected threshold, indicating a logic or calculation error. Logs ↗︎
Test_RegistrySynchronizer1_2_UpkeepReceivedLog The test failed due to an error fetching blockchain data because the current block height is too low. Logs ↗︎

View Full Report ↗︎Docs

@FelixFan1992 FelixFan1992 marked this pull request as ready for review April 24, 2026 14:52
Copilot AI review requested due to automatic review settings April 24, 2026 14:52
@FelixFan1992 FelixFan1992 requested review from a team as code owners April 24, 2026 14:52
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Risk Rating: MEDIUM — dependency/version bumps across multiple Go modules plus a public plugin gitRef update; low code-risk, but moderate integration risk if versions are skewed.

Updates the repository’s Sui relayer dependencies (chainlink-sui and the block-vision Sui SDK) across root/integration/system-test/deployment modules, and updates the public plugin reference to the newer chainlink-sui pseudo-version.

Changes:

  • Bump github.com/smartcontractkit/chainlink-sui to v0.0.0-20260420191233-429504ae18dc in multiple modules and plugin config.
  • Bump github.com/block-vision/sui-go-sdk from v1.1.4 to v1.2.1 where referenced.
  • Refresh corresponding go.sum entries across impacted modules.

Reviewed changes

Copilot reviewed 8 out of 15 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
system-tests/tests/go.sum Updates sui-go-sdk and chainlink-sui checksums for system test module.
system-tests/tests/go.mod Bumps indirect sui-go-sdk and chainlink-sui versions for system test module.
system-tests/lib/go.sum Updates sui-go-sdk and chainlink-sui checksums for system test library module.
system-tests/lib/go.mod Bumps indirect sui-go-sdk and chainlink-sui versions for system test library module.
plugins/plugins.public.yaml Updates public plugin gitRef for chainlink-sui relayer install.
integration-tests/load/go.sum Updates chainlink-sui + chainlink-sui/deployment checksums for load tests module.
integration-tests/load/go.mod Bumps indirect chainlink-sui + chainlink-sui/deployment for load tests module.
integration-tests/go.sum Updates chainlink-sui + chainlink-sui/deployment checksums for integration tests module.
integration-tests/go.mod Bumps direct chainlink-sui + chainlink-sui/deployment versions for integration tests module.
go.sum Updates root module sui-go-sdk and chainlink-sui checksums.
go.mod Bumps root module chainlink-sui and indirect sui-go-sdk versions.
deployment/go.sum Updates deployment module sui-go-sdk and chainlink-sui checksums.
deployment/go.mod Bumps deployment module sui-go-sdk and chainlink-sui versions.
core/scripts/go.sum Updates scripts module sui-go-sdk and chainlink-sui checksums.
core/scripts/go.mod Bumps scripts module indirect sui-go-sdk and chainlink-sui versions.

@@ -17,7 +17,7 @@ plugins:
- moduleURI: "github.com/smartcontractkit/chainlink-sui"
# IMP: consult chainlink-sui team before updating this version
# current version is v0.0.0-20260409184948-5b16fae57fe0
Copy link

Copilot AI Apr 24, 2026

Choose a reason for hiding this comment

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

The inline comment for the Sui plugin still states the previous chainlink-sui version, but gitRef has been updated. Please update or remove the "current version is ..." comment so it doesn't mislead future updates.

Suggested change
# current version is v0.0.0-20260409184948-5b16fae57fe0

Copilot uses AI. Check for mistakes.
Comment thread deployment/go.mod
github.com/smartcontractkit/chainlink-solana/contracts v0.0.0-20260421131224-c46cbfe7bc6c
github.com/smartcontractkit/chainlink-sui v0.0.0-20260409184948-5b16fae57fe0
github.com/smartcontractkit/chainlink-sui v0.0.0-20260420191233-429504ae18dc
github.com/smartcontractkit/chainlink-sui/deployment v0.0.0-20260409184948-5b16fae57fe0
Copy link

Copilot AI Apr 24, 2026

Choose a reason for hiding this comment

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

chainlink-sui was updated, but chainlink-sui/deployment remains pinned to the older pseudo-version in the same require block. If deployment artifacts/APIs are expected to stay in lockstep with the relayer, this mismatch can lead to pulling incompatible packages. Consider updating chainlink-sui/deployment to the corresponding pseudo-version (or add a brief comment explaining why it must stay on the older commit).

Suggested change
github.com/smartcontractkit/chainlink-sui/deployment v0.0.0-20260409184948-5b16fae57fe0
github.com/smartcontractkit/chainlink-sui/deployment v0.0.0-20260420191233-429504ae18dc

Copilot uses AI. Check for mistakes.
Comment thread core/scripts/go.mod
github.com/smartcontractkit/chainlink-solana/contracts v0.0.0-20260421131224-c46cbfe7bc6c // indirect
github.com/smartcontractkit/chainlink-sui v0.0.0-20260409184948-5b16fae57fe0 // indirect
github.com/smartcontractkit/chainlink-sui v0.0.0-20260420191233-429504ae18dc // indirect
github.com/smartcontractkit/chainlink-sui/deployment v0.0.0-20260409184948-5b16fae57fe0 // indirect
Copy link

Copilot AI Apr 24, 2026

Choose a reason for hiding this comment

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

chainlink-sui was bumped but chainlink-sui/deployment is still on the previous pseudo-version immediately below. If both modules are meant to be upgraded together, please bump chainlink-sui/deployment as well (or document why it intentionally stays back).

Suggested change
github.com/smartcontractkit/chainlink-sui/deployment v0.0.0-20260409184948-5b16fae57fe0 // indirect
github.com/smartcontractkit/chainlink-sui/deployment v0.0.0-20260420191233-429504ae18dc // indirect

Copilot uses AI. Check for mistakes.
Comment thread system-tests/lib/go.mod
github.com/smartcontractkit/chainlink-solana/contracts v0.0.0-20260421131224-c46cbfe7bc6c // indirect
github.com/smartcontractkit/chainlink-sui v0.0.0-20260409184948-5b16fae57fe0 // indirect
github.com/smartcontractkit/chainlink-sui v0.0.0-20260420191233-429504ae18dc // indirect
github.com/smartcontractkit/chainlink-sui/deployment v0.0.0-20260409184948-5b16fae57fe0 // indirect
Copy link

Copilot AI Apr 24, 2026

Choose a reason for hiding this comment

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

chainlink-sui was updated but chainlink-sui/deployment remains on the older pseudo-version right after it. If deployment packages should track the relayer version, please bump chainlink-sui/deployment too (or add rationale for the version skew).

Suggested change
github.com/smartcontractkit/chainlink-sui/deployment v0.0.0-20260409184948-5b16fae57fe0 // indirect
github.com/smartcontractkit/chainlink-sui/deployment v0.0.0-20260420191233-429504ae18dc // indirect

Copilot uses AI. Check for mistakes.
@pavel-raykov pavel-raykov enabled auto-merge April 24, 2026 14:55
@pavel-raykov pavel-raykov added this pull request to the merge queue Apr 24, 2026
@cl-sonarqube-production
Copy link
Copy Markdown

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarQube

@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Apr 24, 2026
@FelixFan1992 FelixFan1992 added this pull request to the merge queue Apr 24, 2026
Merged via the queue into develop with commit ed9af27 Apr 24, 2026
341 of 349 checks passed
@FelixFan1992 FelixFan1992 deleted the upgrade-chainlink-sui branch April 24, 2026 16:42
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