feat: use static salt for deploy task

This is so helpful when testing locally, but carries a little risk that the code is easily front-runnable for ownership
matter-labs · Dec 19, 2024 · 941e020 · 941e020
1 parent 0b7f4b3
commit 941e020
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 3 deletions.
diff --git a/scripts/deploy.ts b/scripts/deploy.ts
@@ -17,7 +17,7 @@ async function deploy(name: string, deployer: Wallet, proxy: boolean, args?: any
   console.log("Deploying", name, "contract...");
   let implContract;
   if (name == FACTORY_NAME) {
-    implContract = await deployFactory(deployer, args![0]);
+    implContract = await deployFactory(deployer, args![0], ethersStaticSalt);
   } else {
     implContract = await create2(name, deployer, ethersStaticSalt, args);
   }

diff --git a/test/utils.ts b/test/utils.ts
@@ -166,10 +166,9 @@ export async function deployFactory(wallet: Wallet, beaconAddress: string, salt?
   const standardCreate2Address = utils.create2Address(wallet.address, factoryBytecodeHash, factorySalt, constructorArgs);
   const accountCode = await wallet.provider.getCode(standardCreate2Address);
   if (accountCode != "0x") {
-    logInfo(`Factory already exists!`);
+    logInfo(`Factory already exists at ${standardCreate2Address}`);
     return AAFactory__factory.connect(standardCreate2Address, wallet);
   }
-  logInfo(`Factory doesn't exist at ${standardCreate2Address}!`);
   const factory = await deployer.deploy(
     bytecodeHash,
     beaconAddress,