Consolidate normalizeTxParams method usage

packages/block/src/block/constructors.ts

@@ -5,12 +5,12 @@ import {
   createTxFromBlockBodyData,
   createTxFromSerializedData,
   createTxFromTxData,
+  normalizeTxParams,
 } from '@ethereumjs/tx'
 import {
   CLRequestFactory,
   ConsolidationRequest,
   DepositRequest,
-  TypeOutput,
   Withdrawal,
   WithdrawalRequest,
   bigIntToHex,
@@ -22,9 +22,6 @@ import {
   hexToBytes,
   intToHex,
   isHexString,
-  setLengthLeft,
-  toBytes,
-  toType,
 } from '@ethereumjs/util'
 
 import { generateCliqueBlockExtraData } from '../consensus/clique.js'
@@ -59,27 +56,6 @@ import type {
   WithdrawalBytes,
 } from '@ethereumjs/util'
 
-export function normalizeTxParams(_txParams: any) {
-  const txParams = Object.assign({}, _txParams)
-
-  txParams.gasLimit = toType(txParams.gasLimit ?? txParams.gas, TypeOutput.BigInt)
-  txParams.data = txParams.data === undefined ? txParams.input : txParams.data
-
-  // check and convert gasPrice and value params
-  txParams.gasPrice = txParams.gasPrice !== undefined ? BigInt(txParams.gasPrice) : undefined
-  txParams.value = txParams.value !== undefined ? BigInt(txParams.value) : undefined
-
-  // strict byte length checking
-  txParams.to =
-    txParams.to !== null && txParams.to !== undefined
-      ? setLengthLeft(toBytes(txParams.to), 20)
-      : null
-
-  txParams.v = toType(txParams.v, TypeOutput.BigInt)
-
-  return txParams
-}
-
 /**
  * Static constructor to create a block from a block data dictionary
  *

packages/tx/src/index.ts

@@ -18,3 +18,6 @@ export {
 
 // Types
 export * from './types.js'
+
+// Utils
+export * from './util.js'

packages/tx/src/transactionFactory.ts

@@ -4,7 +4,6 @@ import { create1559FeeMarketTx, create1559FeeMarketTxFromRLP } from './1559/cons
 import { create2930AccessListTx, create2930AccessListTxFromRLP } from './2930/constructors.js'
 import { create4844BlobTx, create4844BlobTxFromRLP } from './4844/constructors.js'
 import { create7702EOACodeTx, create7702EOACodeTxFromRLP } from './7702/constructors.js'
-import { normalizeTxParams } from './fromRpc.js'
 import {
   createLegacyTx,
   createLegacyTxFromBytesArray,
@@ -18,6 +17,7 @@ import {
   isFeeMarketEIP1559TxData,
   isLegacyTxData,
 } from './types.js'
+import { normalizeTxParams } from './util.js'
 
 import type { Transaction, TxData, TxOptions, TypedTxData } from './types.js'
 import type { EthersProvider } from '@ethereumjs/util'

packages/tx/src/util.ts

@@ -1,8 +1,11 @@
 import {
   type PrefixedHexString,
+  TypeOutput,
   bytesToHex,
   hexToBytes,
   setLengthLeft,
+  toBytes,
+  toType,
   validateNoLeadingZeroes,
 } from '@ethereumjs/util'
 
@@ -16,6 +19,7 @@ import type {
   AuthorizationListBytes,
   AuthorizationListItem,
   TransactionType,
+  TypedTxData,
 } from './types.js'
 import type { Common } from '@ethereumjs/common'
 
@@ -249,3 +253,42 @@ export function validateNotArray(values: { [key: string]: any }) {
     }
   }
 }
+
+/**
+ * Normalizes values for transactions that are received from an RPC provider to be properly usable within
+ * the ethereumjs context
+ * @param txParamsFromRPC a transaction in the standard JSON-RPC format
+ * @returns a normalized {@link TypedTxData} object with valid values
+ */
+export const normalizeTxParams = (txParamsFromRPC: any): TypedTxData => {
+  const txParams = Object.assign({}, txParamsFromRPC)
+
+  txParams.gasLimit = toType(txParams.gasLimit ?? txParams.gas, TypeOutput.BigInt)
+  txParams.data = txParams.data === undefined ? txParams.input : txParams.data
+
+  // check and convert gasPrice and value params
+  txParams.gasPrice = txParams.gasPrice !== undefined ? BigInt(txParams.gasPrice) : undefined
+  txParams.value = txParams.value !== undefined ? BigInt(txParams.value) : undefined
+
+  // strict byte length checking
+  txParams.to =
+    txParams.to !== null && txParams.to !== undefined
+      ? setLengthLeft(toBytes(txParams.to), 20)
+      : null
+
+  // Normalize the v/r/s values. If RPC returns '0x0', ensure v/r/s are set to `undefined` in the tx.
+  // If this is not done, then the transaction creation will throw, because `v` is `0`.
+  // Note: this still means that `isSigned` will return `false`.
+  // v/r/s values are `0x0` on networks like Optimism, where the tx is a system tx.
+  // For instance: https://optimistic.etherscan.io/tx/0xf4304cb09b3f58a8e5d20fec5f393c96ccffe0269aaf632cb2be7a8a0f0c91cc
+
+  txParams.v = txParams.v === '0x0' ? '0x' : txParams.v
+  txParams.r = txParams.r === '0x0' ? '0x' : txParams.r
+  txParams.s = txParams.s === '0x0' ? '0x' : txParams.s
+
+  if (txParams.v !== '0x' || txParams.r !== '0x' || txParams.s !== '0x') {
+    txParams.v = toType(txParams.v, TypeOutput.BigInt)
+  }
+
+  return txParams
+}

packages/tx/test/fromRpc.spec.ts

@@ -2,13 +2,13 @@ import { Common, Hardfork, Mainnet, createCustomCommon } from '@ethereumjs/commo
 import { bytesToHex, randomBytes } from '@ethereumjs/util'
 import { assert, describe, it } from 'vitest'
 
-import { normalizeTxParams } from '../src/fromRpc.js'
 import {
   TransactionType,
   createTxFromJsonRpcProvider,
   createTxFromRPC,
   createTxFromTxData,
 } from '../src/index.js'
+import { normalizeTxParams } from '../src/util.js'
 
 import optimismTx from './json/optimismTx.json'
 import rpcTx from './json/rpcTx.json'