@keep-network/tbtc-v2 1.0.0-dev.5 → 1.0.0-dev.6

package/deploy/{09_disable_moving_funds.ts → 18_disable_moving_funds.ts}

@@ -51,11 +51,20 @@ const func: DeployFunction = async (hre: HardhatRuntimeEnvironment) => {
     walletParameters.walletClosingPeriod
   )
 
-  // To emphasize the fact that moving funds is disabled, we set the
-  // movingFundsTimeout to uint32 max value (2^32-1 = 4294967295) and
-  // movingFundsTimeoutSlashingAmount to zero.
+  // To emphasize the fact that moving funds is disabled, we set:
+  // - movingFundsTimeout to uint32 max value (2^32-1 = 4294967295),
+  // - movedFundsSweepTimeout to uint32 max value,
+  // - movingFundsTimeoutSlashingAmount to 0,
+  // - movedFundsSweepTimeoutSlashingAmount to 0,
+  // - movingFundsTimeoutNotifierRewardMultiplier to 0,
+  // - movedFundsSweepTimeoutNotifierRewardMultiplier to 0.
   const movingFundsTimeout = ethers.BigNumber.from("4294967295")
+  const movedFundsSweepTimeout = ethers.BigNumber.from("4294967295")
   const movingFundsTimeoutSlashingAmount = ethers.BigNumber.from("0")
+  const movedFundsSweepTimeoutSlashingAmount = ethers.BigNumber.from("0")
+  const movingFundsTimeoutNotifierRewardMultiplier = ethers.BigNumber.from("0")
+  const movedFundsSweepTimeoutNotifierRewardMultiplier =
+    ethers.BigNumber.from("0")
 
   // Fetch the current values of other moving funds parameters to keep them unchanged.
   const movingFundsParameters = await read("Bridge", "movingFundsParameters")
@@ -69,12 +78,12 @@ const func: DeployFunction = async (hre: HardhatRuntimeEnvironment) => {
     movingFundsParameters.movingFundsTimeoutResetDelay,
     movingFundsTimeout,
     movingFundsTimeoutSlashingAmount,
-    movingFundsParameters.movingFundsTimeoutNotifierRewardMultiplier,
+    movingFundsTimeoutNotifierRewardMultiplier,
     movingFundsParameters.movingFundsCommitmentGasOffset,
     movingFundsParameters.movedFundsSweepTxMaxTotalFee,
-    movingFundsParameters.movedFundsSweepTimeout,
-    movingFundsParameters.movedFundsSweepTimeoutSlashingAmount,
-    movingFundsParameters.movedFundsSweepTimeoutNotifierRewardMultiplier
+    movedFundsSweepTimeout,
+    movedFundsSweepTimeoutSlashingAmount,
+    movedFundsSweepTimeoutNotifierRewardMultiplier
   )
 }
 

package/deploy/19_authorize_spv_maintainer_in_bridge.ts

@@ -0,0 +1,31 @@
+import { HardhatRuntimeEnvironment } from "hardhat/types"
+import { DeployFunction } from "hardhat-deploy/types"
+
+const func: DeployFunction = async function (hre: HardhatRuntimeEnvironment) {
+  const { getNamedAccounts, deployments } = hre
+  const { execute } = deployments
+  const { deployer, spvMaintainer } = await getNamedAccounts()
+
+  await execute(
+    "Bridge",
+    { from: deployer, log: true, waitConfirmations: 1 },
+    "setSpvMaintainerStatus",
+    spvMaintainer,
+    true
+  )
+}
+
+export default func
+
+func.tags = ["AuthorizeSpvMaintainer"]
+func.dependencies = ["Bridge"]
+
+// SPV maintainer can submit SPV proofs to the Bridge. We authorize spvMaintainer
+// account for Hardhat network (unit tests) and Goerli (testnet) but we DO NOT
+// want to authorize it for Mainnet deployment. SPV maintainer will be authorized
+// separately by the Governance when sweeping will be activated.
+//
+// Note that at this point MaintainerProxy contract is already authorized in the
+// Bridge (see AuthorizeMaintainerProxyInBridge tag).
+func.skip = async (hre: HardhatRuntimeEnvironment): Promise<boolean> =>
+  hre.network.name === "mainnet"

package/deploy/{16_transfer_bank_ownership.ts → 20_transfer_bank_ownership.ts}

@@ -11,5 +11,5 @@ const func: DeployFunction = async function (hre: HardhatRuntimeEnvironment) {
 export default func
 
 func.tags = ["TransferBankOwnership"]
-func.dependencies = ["Bank"]
+func.dependencies = ["Bank", "BankUpdateBridge"]
 func.runAtTheEnd = true

package/deploy/{17_transfer_bridge_governance.ts → 21_transfer_bridge_governance.ts}

@@ -18,5 +18,15 @@ const func: DeployFunction = async function (hre: HardhatRuntimeEnvironment) {
 export default func
 
 func.tags = ["TransferBridgeGovernance"]
-func.dependencies = ["Bridge"]
+func.dependencies = [
+  "Bridge",
+  "AuthorizeTBTCVault",
+  "AuthorizeMaintainerProxyInBridge",
+  "SetDepositParameters",
+  "SetWalletParameters",
+  "DisableFraudChallenges",
+  "DisableRedemptions",
+  "DisableMovingFunds",
+  "AuthorizeSpvMaintainer",
+]
 func.runAtTheEnd = true

package/deploy/{23_authorize_maintainer_proxy_in_reimbursement_pool.ts → 26_authorize_maintainer_proxy_in_reimbursement_pool.ts}

@@ -20,3 +20,10 @@ export default func
 
 func.tags = ["AuthorizeMaintainerProxyInReimbursementPool"]
 func.dependencies = ["ReimbursementPool", "MaintainerProxy"]
+
+// On mainnet, the ReimbursementPool ownership is passed to the Threshold
+// Council / DAO and that address is not controlled by the dev team.
+// Hence, this step can be executed only for non-mainnet networks such as
+// Hardhat (unit tests) and Goerli (testnet).
+func.skip = async (hre: HardhatRuntimeEnvironment): Promise<boolean> =>
+  hre.network.name === "mainnet"

package/deploy/{24_authorize_bridge_in_reimbursement_pool.ts → 27_authorize_bridge_in_reimbursement_pool.ts}

@@ -20,3 +20,10 @@ export default func
 
 func.tags = ["AuthorizeBridgeInReimbursementPool"]
 func.dependencies = ["ReimbursementPool", "Bridge"]
+
+// On mainnet, the ReimbursementPool ownership is passed to the Threshold
+// Council / DAO and that address is not controlled by the dev team.
+// Hence, this step can be executed only for non-mainnet networks such as
+// Hardhat (unit tests) and Goerli (testnet).
+func.skip = async (hre: HardhatRuntimeEnvironment): Promise<boolean> =>
+  hre.network.name === "mainnet"