@aztec/ethereum 0.0.1-commit.87a0206 → 0.0.1-commit.88e6f9396

package/dest/deploy_aztec_l1_contracts.js

@@ -19,9 +19,9 @@ import { RollupContract } from './contracts/rollup.js';
 const logger = createLogger('ethereum:deploy_aztec_l1_contracts');
 const JSON_DEPLOY_RESULT_PREFIX = 'JSON DEPLOY RESULT:';
 /**
- * Runs a process with the given command, arguments, and environment.
- * If the process outputs a line starting with JSON_DEPLOY_RESULT_PREFIX,
- * the JSON is parsed and returned.
+ * Runs a process and parses JSON deploy results from stdout.
+ * Lines starting with JSON_DEPLOY_RESULT_PREFIX are parsed and returned.
+ * All other stdout goes to logger.info, stderr goes to logger.warn.
  */ function runProcess(command, args, env, cwd) {
     const { promise, resolve, reject } = promiseWithResolvers();
     const proc = spawn(command, args, {
@@ -37,27 +37,42 @@ const JSON_DEPLOY_RESULT_PREFIX = 'JSON DEPLOY RESULT:';
         ]
     });
     let result;
+    let parseError;
+    let settled = false;
     readline.createInterface({
         input: proc.stdout
     }).on('line', (line)=>{
         const trimmedLine = line.trim();
         if (trimmedLine.startsWith(JSON_DEPLOY_RESULT_PREFIX)) {
             const jsonStr = trimmedLine.slice(JSON_DEPLOY_RESULT_PREFIX.length).trim();
-            // TODO(AD): should this be a zod parse?
-            result = JSON.parse(jsonStr);
+            try {
+                result = JSON.parse(jsonStr);
+            } catch  {
+                parseError = new Error(`Failed to parse deploy result JSON: ${jsonStr.slice(0, 200)}`);
+            }
         } else {
             logger.info(line);
         }
     });
     readline.createInterface({
         input: proc.stderr
-    }).on('line', logger.error.bind(logger));
+    }).on('line', logger.warn.bind(logger));
     proc.on('error', (error)=>{
+        if (settled) {
+            return;
+        }
+        settled = true;
         reject(new Error(`Failed to spawn ${command}: ${error.message}`));
     });
     proc.on('close', (code)=>{
+        if (settled) {
+            return;
+        }
+        settled = true;
         if (code !== 0) {
-            reject(new Error(`${command} exited with code ${code}. See logs for details.\n`));
+            reject(new Error(`${command} exited with code ${code}`));
+        } else if (parseError) {
+            reject(parseError);
         } else {
             resolve(result);
         }
@@ -228,11 +243,8 @@ function cleanupDeployDir() {
     if (isVerifiableChain && !process.env.ETHERSCAN_API_KEY) {
         logger.warn(`Deploying to chain ${chainId} (${chainId === mainnet.id ? 'mainnet' : 'sepolia'}) without ETHERSCAN_API_KEY. ` + `Contracts will NOT be verified on Etherscan. Set ETHERSCAN_API_KEY environment variable to enable verification.`);
     }
-    // From heuristic testing. More caused issues with anvil.
-    const MAGIC_ANVIL_BATCH_SIZE = 8;
-    // Anvil seems to stall with unbounded batch size. Otherwise no max batch size is desirable.
+    const scriptPath = join(getL1ContractsPath(), 'scripts', 'forge_broadcast.js');
     const forgeArgs = [
-        'script',
         FORGE_SCRIPT,
         '--sig',
         'run()',
@@ -240,9 +252,6 @@ function cleanupDeployDir() {
         privateKey,
         '--rpc-url',
         rpcUrl,
-        '--broadcast',
-        '--batch-size',
-        MAGIC_ANVIL_BATCH_SIZE.toString(),
         ...shouldVerify ? [
             '--verify'
         ] : []
@@ -253,7 +262,10 @@ function cleanupDeployDir() {
         FOUNDRY_PROFILE: chainId === mainnet.id ? 'production' : undefined,
         ...getDeployAztecL1ContractsEnvVars(args)
     };
-    const result = await runProcess('forge', forgeArgs, forgeEnv, l1ContractsPath);
+    const result = await runProcess(process.execPath, [
+        scriptPath,
+        ...forgeArgs
+    ], forgeEnv, l1ContractsPath);
     if (!result) {
         throw new Error('Forge script did not output deployment result');
     }
@@ -365,16 +377,15 @@ export function getDeployRollupForUpgradeEnvVars(args) {
     const l1ContractsPath = prepareL1ContractsForDeployment();
     const FORGE_SCRIPT = 'script/deploy/DeployRollupForUpgrade.s.sol';
     await maybeForgeForceProductionBuild(l1ContractsPath, FORGE_SCRIPT, chainId);
+    const scriptPath = join(getL1ContractsPath(), 'scripts', 'forge_broadcast.js');
     const forgeArgs = [
-        'script',
         FORGE_SCRIPT,
         '--sig',
         'run()',
         '--private-key',
         privateKey,
         '--rpc-url',
-        rpcUrl,
-        '--broadcast'
+        rpcUrl
     ];
     const forgeEnv = {
         FOUNDRY_PROFILE: chainId === mainnet.id ? 'production' : undefined,
@@ -383,7 +394,10 @@ export function getDeployRollupForUpgradeEnvVars(args) {
         NETWORK: getActiveNetworkName(),
         ...getDeployRollupForUpgradeEnvVars(args)
     };
-    const result = await runProcess('forge', forgeArgs, forgeEnv, l1ContractsPath);
+    const result = await runProcess(process.execPath, [
+        scriptPath,
+        ...forgeArgs
+    ], forgeEnv, l1ContractsPath);
     if (!result) {
         throw new Error('Forge script did not output deployment result');
     }

package/dest/deploy_l1_contract.js

@@ -5,7 +5,7 @@ import { concatHex, encodeAbiParameters, encodeDeployData, getContractAddress, n
 import { DEPLOYER_ADDRESS } from './deploy_aztec_l1_contracts.js';
 import { RegisterNewRollupVersionPayloadArtifact } from './l1_artifacts.js';
 import { getL1TxUtilsConfigEnvVars } from './l1_tx_utils/config.js';
-import { createL1TxUtilsFromViemWallet } from './l1_tx_utils/factory.js';
+import { createL1TxUtils } from './l1_tx_utils/factory.js';
 import { formatViemError } from './utils.js';
 export class L1Deployer {
     client;
@@ -28,7 +28,7 @@ export class L1Deployer {
         this.salt = maybeSalt ? padHex(numberToHex(maybeSalt), {
             size: 32
         }) : undefined;
-        this.l1TxUtils = createL1TxUtilsFromViemWallet(this.client, {
+        this.l1TxUtils = createL1TxUtils(this.client, {
             logger: this.logger,
             dateProvider
         }, {
@@ -130,7 +130,7 @@ export class L1Deployer {
     let { l1TxUtils } = opts;
     if (!l1TxUtils) {
         const config = getL1TxUtilsConfigEnvVars();
-        l1TxUtils = createL1TxUtilsFromViemWallet(extendedClient, {
+        l1TxUtils = createL1TxUtils(extendedClient, {
             logger
         }, {
             ...config,

package/dest/generated/l1-contracts-defaults.d.ts

@@ -1,7 +1,7 @@
 /** Default L1 contracts configuration values from network-defaults.yml */
 export declare const l1ContractsDefaultEnv: {
     readonly ETHEREUM_SLOT_DURATION: 12;
-    readonly AZTEC_SLOT_DURATION: 36;
+    readonly AZTEC_SLOT_DURATION: 72;
     readonly AZTEC_EPOCH_DURATION: 32;
     readonly AZTEC_TARGET_COMMITTEE_SIZE: 48;
     readonly AZTEC_LAG_IN_EPOCHS_FOR_VALIDATOR_SET: 2;

package/dest/generated/l1-contracts-defaults.js

@@ -2,7 +2,7 @@
 // Do not edit manually - run yarn generate to regenerate
 /** Default L1 contracts configuration values from network-defaults.yml */ export const l1ContractsDefaultEnv = {
     ETHEREUM_SLOT_DURATION: 12,
-    AZTEC_SLOT_DURATION: 36,
+    AZTEC_SLOT_DURATION: 72,
     AZTEC_EPOCH_DURATION: 32,
     AZTEC_TARGET_COMMITTEE_SIZE: 48,
     AZTEC_LAG_IN_EPOCHS_FOR_VALIDATOR_SET: 2,