hardhat 2.19.5 → 2.20.0

package/src/internal/hardhat-network/provider/modules/eth.ts

@@ -1,11 +1,16 @@
 import { Block } from "@nomicfoundation/ethereumjs-block";
 import { Common } from "@nomicfoundation/ethereumjs-common";
 import {
-  Transaction,
+  LegacyTransaction,
   TransactionFactory,
   TypedTransaction,
 } from "@nomicfoundation/ethereumjs-tx";
-import { Address, toBuffer, toRpcSig } from "@nomicfoundation/ethereumjs-util";
+import {
+  Address,
+  equalsBytes,
+  toBytes,
+  toRpcSig,
+} from "@nomicfoundation/ethereumjs-util";
 import * as t from "io-ts";
 import cloneDeep from "lodash/cloneDeep";
 import { BoundExperimentalHardhatNetworkMessageTraceHook } from "../../../../types";
@@ -376,7 +381,9 @@ export class EthModule extends Base {
     await this._runHardhatNetworkMessageTraceHooks(trace, true);
 
     if (error !== undefined && this._throwOnCallFailures) {
-      const callReturnData = trace?.returnData.toString("hex") ?? "";
+      const callReturnData = Buffer.from(trace?.returnData ?? []).toString(
+        "hex"
+      );
       (error as any).data = `0x${callReturnData}`;
 
       throw error;
@@ -452,7 +459,9 @@ export class EthModule extends Base {
         error
       );
 
-      const callReturnData = trace?.returnData.toString("hex") ?? "";
+      const callReturnData = Buffer.from(trace?.returnData ?? []).toString(
+        "hex"
+      );
       (error as any).data = `0x${callReturnData}`;
 
       throw error;
@@ -819,7 +828,7 @@ export class EthModule extends Base {
     }
 
     const index = block.transactions.findIndex((btx) =>
-      btx.hash().equals(hash)
+      equalsBytes(btx.hash(), hash)
     );
     const tx = block.transactions[index];
     if (tx === undefined) {
@@ -970,10 +979,10 @@ export class EthModule extends Base {
     try {
       tx = TransactionFactory.fromSerializedData(rawTx, {
         common: this._common,
-        disableMaxInitCodeSizeCheck: true,
+        allowUnlimitedInitCodeSize: true,
       });
 
-      this._validateEip3860MaxInitCodeSize(tx.to?.toBuffer(), tx.data);
+      this._validateEip3860MaxInitCodeSize(tx.to?.toBytes(), tx.data);
     } catch (error) {
       // This section of the code is incredibly dependant of TransactionFactory.fromSerializedData
       // AccessListEIP2930Transaction.fromSerializedTx and Transaction.fromSerializedTx
@@ -1014,7 +1023,7 @@ export class EthModule extends Base {
       throw new InvalidArgumentsError("Invalid Signature");
     }
 
-    if (tx instanceof Transaction) {
+    if (tx instanceof LegacyTransaction) {
       this._validateEip155HardforkRequirement(tx);
     }
 
@@ -1266,7 +1275,7 @@ export class EthModule extends Base {
       gasLimit:
         rpcTx.gas !== undefined ? rpcTx.gas : this._node.getBlockGasLimit(),
       value: rpcTx.value !== undefined ? rpcTx.value : 0n,
-      data: rpcTx.data !== undefined ? rpcTx.data : toBuffer([]),
+      data: rpcTx.data !== undefined ? rpcTx.data : toBytes([]),
       nonce:
         rpcTx.nonce !== undefined
           ? rpcTx.nonce
@@ -1503,7 +1512,9 @@ export class EthModule extends Base {
     if (trace.error !== undefined && this._throwOnTransactionFailures) {
       const e = trace.error;
 
-      const returnData = trace.trace?.returnData.toString("hex") ?? "";
+      const returnData = Buffer.from(trace.trace?.returnData ?? []).toString(
+        "hex"
+      );
       (e as any).data = `0x${returnData}`;
       (e as any).transactionHash = bufferToRpcData(tx.hash());
 
@@ -1611,7 +1622,7 @@ export class EthModule extends Base {
       const transactions = result.block.transactions;
       for (let i = 0; i < transactions.length; i++) {
         const blockTx = transactions[i];
-        if (blockTx.hash().equals(tx.hash())) {
+        if (equalsBytes(blockTx.hash(), tx.hash())) {
           return [result, i];
         }
       }
@@ -1638,6 +1649,15 @@ export class EthModule extends Base {
   private _validateTransactionAndCallRequest(
     rpcRequest: RpcCallRequest | RpcTransactionRequest
   ) {
+    if (
+      rpcRequest.blobs !== undefined ||
+      rpcRequest.blobVersionedHashes !== undefined
+    ) {
+      throw new InvalidInputError(
+        `An EIP-4844 (shard blob) transaction was received, but Hardhat doesn't have support for them yet.`
+      );
+    }
+
     if (
       (rpcRequest.maxFeePerGas !== undefined ||
         rpcRequest.maxPriorityFeePerGas !== undefined) &&
@@ -1689,7 +1709,7 @@ You can use them by running Hardhat Network with 'hardfork' ${ACCESS_LIST_MIN_HA
   }
 
   // TODO: Find a better place for this
-  private _validateEip155HardforkRequirement(tx: Transaction) {
+  private _validateEip155HardforkRequirement(tx: LegacyTransaction) {
     // 27 and 28 are only valid for non-EIP-155 legacy txs
     if (tx.v === 27n || tx.v === 28n) {
       return;
@@ -1703,8 +1723,8 @@ You can use them by running Hardhat Network with 'hardfork' ${EIP155_MIN_HARDFOR
   }
 
   private _validateEip3860MaxInitCodeSize(
-    to: Buffer | undefined,
-    data: Buffer
+    to: Uint8Array | undefined,
+    data: Uint8Array
   ) {
     if (!this._common.gteHardfork(EIP3860_MIN_HARDFORK)) {
       // this check is only relevant after shanghai
@@ -1730,6 +1750,12 @@ Enable the 'allowUnlimitedContractSize' option to allow init codes of any length
   }
 
   private _validateRawTransactionHardforkRequirements(rawTx: Buffer) {
+    if (rawTx[0] <= 0x7f && rawTx[0] === 3) {
+      throw new InvalidInputError(
+        `An EIP-4844 (shard blob) transaction was received, but Hardhat doesn't have support for them yet.`
+      );
+    }
+
     if (rawTx[0] <= 0x7f && rawTx[0] !== 1 && rawTx[0] !== 2) {
       throw new InvalidArgumentsError(`Invalid transaction type ${rawTx[0]}.
 

package/src/internal/hardhat-network/provider/modules/logger.ts

@@ -1,6 +1,9 @@
 import { Block } from "@nomicfoundation/ethereumjs-block";
 import { TypedTransaction } from "@nomicfoundation/ethereumjs-tx";
-import { bufferToHex } from "@nomicfoundation/ethereumjs-util";
+import {
+  bytesToHex as bufferToHex,
+  equalsBytes,
+} from "@nomicfoundation/ethereumjs-util";
 import ansiEscapes from "ansi-escapes";
 import chalk, { Chalk } from "chalk";
 import util from "util";
@@ -89,8 +92,8 @@ export class ModulesLogger {
 
   public logBlockFromAutomine(
     result: MineBlockResult,
-    codes: Buffer[],
-    txHashToHighlight: Buffer
+    codes: Uint8Array[],
+    txHashToHighlight: Uint8Array
   ) {
     const { block, blockResult, traces } = result;
     const { results } = blockResult;
@@ -113,7 +116,7 @@ export class ModulesLogger {
           const txTrace = traces[i];
           const code = codes[i];
 
-          const highlightTxHash = tx.hash().equals(txHashToHighlight);
+          const highlightTxHash = equalsBytes(tx.hash(), txHashToHighlight);
 
           this._logTxInsideBlock(tx, txTrace, code, txGasUsed, {
             highlightTxHash,
@@ -216,8 +219,8 @@ export class ModulesLogger {
 
       this._logWithTitle("Transaction", txHash);
 
-      this._logTxFrom(tx.getSenderAddress().toBuffer());
-      this._logTxTo(tx.to?.toBuffer(), txTrace.trace);
+      this._logTxFrom(Buffer.from(tx.getSenderAddress().toBytes()));
+      this._logTxTo(tx.to?.toBytes(), txTrace.trace);
       this._logTxValue(tx.value);
       this._logWithTitle("Gas used", `${txGasUsed} of ${tx.gasLimit}`);
 
@@ -251,8 +254,8 @@ export class ModulesLogger {
 
       this._logWithTitle("Transaction", txHash);
 
-      this._logTxFrom(tx.getSenderAddress().toBuffer());
-      this._logTxTo(tx.to?.toBuffer(), txTrace.trace);
+      this._logTxFrom(tx.getSenderAddress().toBytes());
+      this._logTxTo(tx.to?.toBytes(), txTrace.trace);
       this._logTxValue(tx.value);
       this._logWithTitle("Gas used", `${txGasUsed} of ${tx.gasLimit}`);
 
@@ -565,7 +568,7 @@ export class ModulesLogger {
   private _logTxInsideBlock(
     tx: TypedTransaction,
     txTrace: GatherTracesResult,
-    code: Buffer,
+    code: Uint8Array,
     txGasUsed: bigint,
     {
       highlightTxHash,
@@ -586,8 +589,8 @@ export class ModulesLogger {
 
     this._indent(() => {
       this._logContractAndFunctionName(txTrace.trace, code);
-      this._logTxFrom(tx.getSenderAddress().toBuffer());
-      this._logTxTo(tx.to?.toBuffer(), txTrace.trace);
+      this._logTxFrom(tx.getSenderAddress().toBytes());
+      this._logTxTo(tx.to?.toBytes(), txTrace.trace);
       this._logTxValue(tx.value);
       this._logWithTitle("Gas used", `${txGasUsed} of ${tx.gasLimit}`);
 
@@ -637,7 +640,7 @@ export class ModulesLogger {
 
   private _logContractAndFunctionName(
     trace: MessageTrace | undefined,
-    code: Buffer,
+    code: Uint8Array,
     {
       printNonContractCalled = false,
     }: { printNonContractCalled?: boolean } = {}
@@ -721,7 +724,7 @@ export class ModulesLogger {
     this._methodCollapsedCount = 0;
   }
 
-  private _logTxTo(to: Buffer | undefined, trace?: MessageTrace) {
+  private _logTxTo(to: Uint8Array | undefined, trace?: MessageTrace) {
     if (trace !== undefined && isCreateTrace(trace)) {
       return;
     }
@@ -740,7 +743,7 @@ export class ModulesLogger {
     this._logWithTitle("Value", weiToHumanReadableString(value));
   }
 
-  private _logTxFrom(from: Buffer) {
+  private _logTxFrom(from: Uint8Array) {
     this._logWithTitle("From", bufferToHex(from));
   }
 

package/src/internal/hardhat-network/provider/node-types.ts

@@ -61,14 +61,14 @@ export interface GenesisAccount {
   balance: string | number | bigint;
 }
 
-export type AccessListBufferItem = [Buffer, Buffer[]];
+export type AccessListBufferItem = [Uint8Array, Uint8Array[]];
 
 export interface CallParams {
-  to?: Buffer;
-  from: Buffer;
+  to?: Uint8Array;
+  from: Uint8Array;
   gasLimit: bigint;
   value: bigint;
-  data: Buffer;
+  data: Uint8Array;
   // We use this access list format because @nomicfoundation/ethereumjs-tx access list data
   // forces us to use it or stringify them
   accessList?: AccessListBufferItem[];
@@ -81,15 +81,16 @@ export interface CallParams {
 export type TransactionParams =
   | LegacyTransactionParams
   | AccessListTransactionParams
-  | EIP1559TransactionParams;
+  | EIP1559TransactionParams
+  | BlobTransactionParams;
 
 interface BaseTransactionParams {
   // `to` should be undefined for contract creation
-  to?: Buffer;
-  from: Buffer;
+  to?: Uint8Array;
+  from: Uint8Array;
   gasLimit: bigint;
   value: bigint;
-  data: Buffer;
+  data: Uint8Array;
   nonce: bigint;
 }
 
@@ -112,25 +113,31 @@ export interface EIP1559TransactionParams extends BaseTransactionParams {
   maxPriorityFeePerGas: bigint;
 }
 
+export interface BlobTransactionParams extends EIP1559TransactionParams {
+  blobs: Uint8Array[];
+  blobVersionedHashes: Uint8Array[];
+}
+
 export interface FilterParams {
   fromBlock: bigint;
   toBlock: bigint;
-  addresses: Buffer[];
-  normalizedTopics: Array<Array<Buffer | null> | null>;
+  addresses: Uint8Array[];
+  normalizedTopics: Array<Array<Uint8Array | null> | null>;
 }
 
 export interface Snapshot {
   id: number;
   date: Date;
   latestBlock: Block;
-  stateRoot: Buffer;
+  stateRoot: Uint8Array;
   txPoolSnapshotId: number;
   blockTimeOffsetSeconds: bigint;
   nextBlockTimestamp: bigint;
-  irregularStatesByBlockNumber: Map<bigint, Buffer>;
+  irregularStatesByBlockNumber: Map<bigint, Uint8Array>;
   userProvidedNextBlockBaseFeePerGas: bigint | undefined;
   coinbase: string;
   mixHashGenerator: RandomBufferGenerator;
+  parentBeaconBlockRootGenerator: RandomBufferGenerator;
 }
 
 export type SendTransactionResult =