fix: use vm common for default new blocks

style: prefer nullish coalescing operator

Author: ryanio

packages/vm/lib/runCall.ts

@@ -33,27 +33,29 @@ export interface RunCallOpts {
  * @ignore
  */
 export default function runCall(this: VM, opts: RunCallOpts): Promise<EVMResult> {
-  const block = opts.block || new Block()
+  const block = opts.block ?? Block.fromBlockData({}, { common: this._common })
 
   const txContext = new TxContext(
-    opts.gasPrice || new BN(0),
-    opts.origin || opts.caller || Address.zero()
+    opts.gasPrice ?? new BN(0),
+    opts.origin ?? opts.caller ?? Address.zero()
   )
+
   const message = new Message({
     caller: opts.caller,
-    gasLimit: opts.gasLimit ? opts.gasLimit : new BN(0xffffff),
-    to: opts.to ? opts.to : undefined,
+    gasLimit: opts.gasLimit ?? new BN(0xffffff),
+    to: opts.to ?? undefined,
     value: opts.value,
     data: opts.data,
     code: opts.code,
-    depth: opts.depth || 0,
-    isCompiled: opts.compiled || false,
-    isStatic: opts.static || false,
-    salt: opts.salt || null,
-    selfdestruct: opts.selfdestruct || {},
-    delegatecall: opts.delegatecall || false,
+    depth: opts.depth ?? 0,
+    isCompiled: opts.compiled ?? false,
+    isStatic: opts.static ?? false,
+    salt: opts.salt ?? null,
+    selfdestruct: opts.selfdestruct ?? {},
+    delegatecall: opts.delegatecall ?? false,
   })
 
   const evm = new EVM(this, txContext, block)
+
   return evm.executeMessage(message)
 }

packages/vm/lib/runCode.ts

@@ -71,35 +71,28 @@ export interface RunCodeOpts {
  * @ignore
  */
 export default function runCode(this: VM, opts: RunCodeOpts): Promise<ExecResult> {
-  if (!opts.block) {
-    opts.block = new Block()
-  }
+  const block = opts.block ?? Block.fromBlockData({}, { common: this._common })
 
   // Backwards compatibility
-  if (!opts.txContext) {
-    opts.txContext = new TxContext(
-      opts.gasPrice || new BN(0),
-      opts.origin || opts.caller || Address.zero()
-    )
-  }
-  if (!opts.message) {
-    opts.message = new Message({
+  const txContext =
+    opts.txContext ??
+    new TxContext(opts.gasPrice ?? new BN(0), opts.origin ?? opts.caller ?? Address.zero())
+
+  const message =
+    opts.message ??
+    new Message({
       code: opts.code,
       data: opts.data,
       gasLimit: opts.gasLimit,
-      to: opts.address || Address.zero(),
+      to: opts.address ?? Address.zero(),
       caller: opts.caller,
       value: opts.value,
-      depth: opts.depth || 0,
-      selfdestruct: opts.selfdestruct || {},
-      isStatic: opts.isStatic || false,
+      depth: opts.depth ?? 0,
+      selfdestruct: opts.selfdestruct ?? {},
+      isStatic: opts.isStatic ?? false,
     })
-  }
 
-  let evm = opts.evm
-  if (!evm) {
-    evm = new EVM(this, opts.txContext, opts.block)
-  }
+  const evm = opts.evm ?? new EVM(this, txContext, block)
 
-  return evm.runInterpreter(opts.message, { pc: opts.pc })
+  return evm.runInterpreter(message, { pc: opts.pc })
 }

packages/vm/lib/runTx.ts

@@ -57,10 +57,7 @@ export default async function runTx(this: VM, opts: RunTxOpts): Promise<RunTxRes
   }
 
   // create a reasonable default if no block is given
-  if (!opts.block) {
-    const common = opts.tx.common
-    opts.block = Block.fromBlockData({}, { common })
-  }
+  opts.block = opts.block ?? Block.fromBlockData({}, { common: opts.tx.common })
 
   if (opts.block.header.gasLimit.lt(opts.tx.gasLimit)) {
     throw new Error('tx has a higher gas limit than the block')