diff --git a/sui/README.md b/sui/README.md index d5d70aecb..44aea8f1c 100644 --- a/sui/README.md +++ b/sui/README.md @@ -81,9 +81,9 @@ node sui/deploy-contract.js upgrade axelar_gateway policy should be one of the following: -- `0`: Allow any upgrade. -- `128`: Upgrade policy to just add code. https://docs.sui.io/references/framework/sui-framework/package#function-only_additive_upgrades -- `192`: Upgrade policy to just change dependencies. https://docs.sui.io/references/framework/sui-framework/package#function-only_dep_upgrades +- `any_upgrade`: Allow any upgrade. +- `code_upgrade`: Upgrade policy to just add code. https://docs.sui.io/references/framework/sui-framework/package#function-only_additive_upgrades +- `dep_upgrade`: Upgrade policy to just change dependencies. https://docs.sui.io/references/framework/sui-framework/package#function-only_dep_upgrades Provide `--txFilePath` with `--offline` to generate tx data file for offline signing. diff --git a/sui/deploy-contract.js b/sui/deploy-contract.js index ac1277ea2..f2af1c511 100644 --- a/sui/deploy-contract.js +++ b/sui/deploy-contract.js @@ -113,8 +113,8 @@ async function deploy(contractName, config, chain, options) { ], }); - if (policy !== '0') { - const upgradeType = policy === '128' ? 'only_additive_upgrades' : 'only_dep_upgrades'; + if (policy !== 'any_upgrade') { + const upgradeType = policy === 'code_upgrade' ? 'only_additive_upgrades' : 'only_dep_upgrades'; tx.moveCall({ target: `${suiPackageAddress}::package::${upgradeType}`, @@ -206,9 +206,9 @@ if (require.main === module) { .addOption(new Option('--nonce ', 'nonce for the signer (defaults to HashZero)')) .addOption(new Option('--previousSigners ', 'number of previous signers to retain').default('15')) .addOption( - new Option('--policy ', 'upgrade policy for upgrade cap: For example, use "0" to allow all types of upgrades') - .choices(['0', '128', '192']) - .default('0'), + new Option('--policy ', 'upgrade policy for upgrade cap: For example, use "any_upgrade" to allow all types of upgrades') + .choices(['any_upgrade', 'code_upgrade', 'dep_upgrade']) + .default('any_upgrade'), ) .action((contractName, options) => { mainProcessor([contractName], options, deploy); diff --git a/sui/deploy-utils.js b/sui/deploy-utils.js index c3a6ebff9..fc46e5ff6 100644 --- a/sui/deploy-utils.js +++ b/sui/deploy-utils.js @@ -12,8 +12,8 @@ async function upgradePackage(client, keypair, packageName, packageConfig, build const { policy, offline } = options; const sender = options.sender || keypair.toSuiAddress(); - if (!['0', '128', '192'].includes(policy)) { - throw new Error(`Unknown upgrade policy: ${policy}`); + if (!['any_upgrade', 'code_upgrade', 'dep_upgrade'].includes(policy)) { + throw new Error(`Unknown upgrade policy: ${policy}. Supported policies: any_upgrade, code_upgrade, dep_upgrade`); } const upgradeCap = packageConfig.objects?.upgradeCap;