feat(ctb): Change json tx getter func to printer

packages/contracts-bedrock/deploy/020-SystemDictatorSteps-1.ts

@@ -12,7 +12,7 @@ import {
   getContractsFromArtifacts,
   getDeploymentAddress,
   doPhase,
-  jsonifyTransaction,
+  printJsonTransaction,
   printTenderlySimulationLink,
   printCastCommand,
 } from '../src/deploy-utils'
@@ -99,8 +99,7 @@ const deployFn: DeployFunction = async (hre) => {
       console.log(`Please transfer AddressManager owner to MSD`)
       console.log(`AddressManager address: ${AddressManager.address}`)
       console.log(`MSD address: ${SystemDictator.address}`)
-      console.log(`JSON:`)
-      console.log(jsonifyTransaction(tx))
+      printJsonTransaction(tx)
       printCastCommand(tx)
       await printTenderlySimulationLink(SystemDictator.provider, tx)
     }
@@ -137,8 +136,7 @@ const deployFn: DeployFunction = async (hre) => {
         `L1StandardBridgeProxy address: ${L1StandardBridgeProxy.address}`
       )
       console.log(`MSD address: ${SystemDictator.address}`)
-      console.log(`JSON:`)
-      console.log(jsonifyTransaction(tx))
+      printJsonTransaction(tx)
       printCastCommand(tx)
       await printTenderlySimulationLink(SystemDictator.provider, tx)
     }
@@ -175,8 +173,7 @@ const deployFn: DeployFunction = async (hre) => {
       console.log(`Please transfer L1ERC721Bridge (proxy) owner to MSD`)
       console.log(`L1ERC721BridgeProxy address: ${L1ERC721BridgeProxy.address}`)
       console.log(`MSD address: ${SystemDictator.address}`)
-      console.log(`JSON:`)
-      console.log(jsonifyTransaction(tx))
+      printJsonTransaction(tx)
       printCastCommand(tx)
       await printTenderlySimulationLink(SystemDictator.provider, tx)
     }

packages/contracts-bedrock/deploy/021-SystemDictatorSteps-2.ts

@@ -10,7 +10,7 @@ import '@nomiclabs/hardhat-ethers'
 import {
   assertContractVariable,
   getContractsFromArtifacts,
-  jsonifyTransaction,
+  printJsonTransaction,
   isStep,
   doStep,
   printTenderlySimulationLink,
@@ -206,8 +206,7 @@ const deployFn: DeployFunction = async (hre) => {
         )
       )
       console.log(`MSD address: ${SystemDictator.address}`)
-      console.log(`JSON:`)
-      console.log(jsonifyTransaction(tx))
+      printJsonTransaction(tx)
       printCastCommand(tx)
       await printTenderlySimulationLink(SystemDictator.provider, tx)
     }
@@ -318,8 +317,7 @@ const deployFn: DeployFunction = async (hre) => {
       const tx = await OptimismPortal.populateTransaction.unpause()
       console.log(`Please unpause the OptimismPortal...`)
       console.log(`OptimismPortal address: ${OptimismPortal.address}`)
-      console.log(`JSON:`)
-      console.log(jsonifyTransaction(tx))
+      printJsonTransaction(tx)
       printCastCommand(tx)
       await printTenderlySimulationLink(SystemDictator.provider, tx)
     }
@@ -348,8 +346,7 @@ const deployFn: DeployFunction = async (hre) => {
       const tx = await SystemDictator.populateTransaction.finalize()
       console.log(`Please finalize deployment...`)
       console.log(`MSD address: ${SystemDictator.address}`)
-      console.log(`JSON:`)
-      console.log(jsonifyTransaction(tx))
+      printJsonTransaction(tx)
       printCastCommand(tx)
       await printTenderlySimulationLink(SystemDictator.provider, tx)
     }

packages/contracts-bedrock/src/deploy-utils.ts

@@ -305,8 +305,10 @@ export const getDeploymentAddress = async (
  * @param tx Ethers transaction object.
  * @returns JSON-ified transaction object.
  */
-export const jsonifyTransaction = (tx: ethers.PopulatedTransaction): string => {
-  return JSON.stringify(
+export const printJsonTransaction = (tx: ethers.PopulatedTransaction): void => {
+  console.log(
+    'JSON transaction parameters:\n' +
+      JSON.stringify(
         {
           from: tx.from,
           to: tx.to,
@@ -317,6 +319,7 @@ export const jsonifyTransaction = (tx: ethers.PopulatedTransaction): string => {
         null,
         2
       )
+  )
 }
 
 /**
@@ -388,11 +391,8 @@ export const doStep = async (opts: {
     ]()
     console.log(`Please execute step ${opts.step}...`)
     console.log(`MSD address: ${opts.SystemDictator.address}`)
-    console.log(`JSON:`)
-    console.log(jsonifyTransaction(tx))
-    console.log(
+    printJsonTransaction(tx)
     await printTenderlySimulationLink(opts.SystemDictator.provider, tx)
-    )
   }
 
   // Wait for the step to complete.
@@ -444,11 +444,8 @@ export const doPhase = async (opts: {
     ]()
     console.log(`Please execute phase ${opts.phase}...`)
     console.log(`MSD address: ${opts.SystemDictator.address}`)
-    console.log(`JSON:`)
-    console.log(jsonifyTransaction(tx))
-    console.log(
+    printJsonTransaction(tx)
     await printTenderlySimulationLink(opts.SystemDictator.provider, tx)
-    )
   }
 
   // Wait for the step to complete.
@@ -495,6 +492,8 @@ export const printTenderlySimulationLink = async (
  */
 export const printCastCommand = (tx: ethers.PopulatedTransaction): void => {
   if (process.env.CAST_COMMANDS) {
-    console.log(`cast send ${tx.to} ${tx.data} --from ${tx.from} --value ${tx.value}`)
+    console.log(
+      `cast send ${tx.to} ${tx.data} --from ${tx.from} --value ${tx.value}`
+    )
   }
 }