refactor

OffchainLabs · Oct 9, 2024 · 34fc9fb · 34fc9fb
1 parent 9ff31f8
commit 34fc9fb
Showing 1 changed file with 18 additions and 13 deletions.
diff --git a/src/createRollupPrepareDeploymentParamsConfig.ts b/src/createRollupPrepareDeploymentParamsConfig.ts
@@ -89,21 +89,26 @@ export function createRollupPrepareDeploymentParamsConfig<TChain extends Chain |
     );
   }
 
-  const paramsByParentBlockTime: {
+  let paramsByParentBlockTime: {
     confirmPeriodBlocks: bigint;
     sequencerInboxMaxTimeVariation: SequencerInboxMaxTimeVariation;
-  } = parentChainIsCustom
-    ? {
-        // ok to use non-null assertion here because we already checked above
-        confirmPeriodBlocks: confirmPeriodBlocks!,
-        // ok to use non-null assertion here because we already checked above
-        sequencerInboxMaxTimeVariation: sequencerInboxMaxTimeVariation!,
-      }
-    : {
-        confirmPeriodBlocks: confirmPeriodBlocks ?? getDefaultConfirmPeriodBlocks(parentChainId),
-        sequencerInboxMaxTimeVariation:
-          sequencerInboxMaxTimeVariation ?? getDefaultSequencerInboxMaxTimeVariation(parentChainId),
-      };
+  };
+
+  if (parentChainIsCustom) {
+    // ok to use non-null assertions here because we already checked above
+    paramsByParentBlockTime = {
+      confirmPeriodBlocks: confirmPeriodBlocks!,
+      sequencerInboxMaxTimeVariation: sequencerInboxMaxTimeVariation!,
+    };
+  } else {
+    const defaultConfirmPeriodBlocks = getDefaultConfirmPeriodBlocks(parentChainId);
+    const defaultSequencerInboxMTV = getDefaultSequencerInboxMaxTimeVariation(parentChainId);
+
+    paramsByParentBlockTime = {
+      confirmPeriodBlocks: confirmPeriodBlocks ?? defaultConfirmPeriodBlocks,
+      sequencerInboxMaxTimeVariation: sequencerInboxMaxTimeVariation ?? defaultSequencerInboxMTV,
+    };
+  }
 
   return {
     ...defaults,