Skip to content

update SUI CLI to sui 1.63.3#328

Draft
FelixFan1992 wants to merge 76 commits intodevelopfrom
upgrade-to-1.63
Draft

update SUI CLI to sui 1.63.3#328
FelixFan1992 wants to merge 76 commits intodevelopfrom
upgrade-to-1.63

Conversation

@FelixFan1992
Copy link
Collaborator

No description provided.

@FelixFan1992 FelixFan1992 requested a review from a team as a code owner January 16, 2026 21:54
@FelixFan1992 FelixFan1992 marked this pull request as draft January 16, 2026 21:54
@github-actions
Copy link

👋 FelixFan1992, thanks for creating this pull request!

To help reviewers, please consider creating future PRs as drafts first. This allows you to self-review and make any final changes before notifying the team.

Once you're ready, you can mark it as "Ready for review" to request feedback. Thanks!

}
}

#[test]
Copy link
Collaborator Author

@FelixFan1992 FelixFan1992 Jan 19, 2026

Choose a reason for hiding this comment

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

currently, we can use a hack:

[published.testnet]
chain-id = "4c78adac"
published-at = "0x0000000000000000000000000000000000000000000000000000000000000000"
original-id = "0x0000000000000000000000000000000000000000000000000000000000000000"
version = 1

but this is not ideal. waiting for Sui team to release a new approach

@faisal-chainlink
Copy link
Collaborator

Seems like the primary remaining item in this PR is the insufficient gas error. Likely due to account selection in the CLI.

@faisal-chainlink
Copy link
Collaborator

faisal-chainlink commented Feb 2, 2026

@FelixFan1992 Currently all tests should pass with the exception of the upgrades via MCMS. The issue with MCMS upgrades is that we are still using the CLI the build the compiled package.

Note: I'm running the relaying and binding tests with go test ./... -v -tags=integration -count=1 -p=1 from the main directory. For the /integration-tests directory, it's the same command but needs to be run within the subdirectory since it's a separate module. This is slightly different from how CI is running them but we can handle that afterwards.

When using the --with-unpublished-dependencies, I'm seeing an "incompatible upgrade" error. However, if we use the regular publishing flow to generate the byte code of a package, we must also fill in all the addresses of the dependencies which isn't currently done.

That should be the last missing piece.


The summary is that we are now creating the Published.toml file or pub.[env].toml file whenever we want to handle a given contract deployment. We start by traversing all the deps for that contract and build their toml files one by one to enable publishing / compiling.

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.

3 participants