allowing to specify percentage-based factors (like 1.125 for 112.5%)

CHANGELOG.md

@@ -2744,3 +2744,7 @@ If there are any bugs, improvements, optimizations or any new feature proposal f
 #### web3
 
 -   Exported EIP-6963 types are no longer under the `providers` namespace. (#7270)
+
+#### web3-eth
+
+- Allow specifying percentage based factor in Web3Eth.calculateFeeData Param baseFeePerGasFactor #7332

packages/web3-eth/src/web3_eth.ts

@@ -308,7 +308,7 @@ export class Web3Eth extends Web3Context<Web3EthExecutionAPI, RegisteredSubscrip
 	 * ```
 	 */
 	public async calculateFeeData(
-		baseFeePerGasFactor = BigInt(2),
+		baseFeePerGasFactor: bigint | number = BigInt(2),
 		alternativeMaxPriorityFeePerGas = ethUnitMap.Gwei,
 	): Promise<FeeData> {
 		const block = await this.getBlock<{ number: FMT_NUMBER.BIGINT; bytes: FMT_BYTES.HEX }>(
@@ -348,7 +348,15 @@ export class Web3Eth extends Web3Context<Web3EthExecutionAPI, RegisteredSubscrip
 			// and we multiply the `baseFeePerGas` by `baseFeePerGasFactor`, to allow
 			// trying to include the transaction in the next few blocks even if the
 			// baseFeePerGas is increasing fast
-			maxFeePerGas = baseFeePerGas * baseFeePerGasFactor + maxPriorityFeePerGas;
+			let baseFeeMultiplier: bigint;
+			if (typeof baseFeePerGasFactor === 'number') {
+				// Convert number to bigint with three decimal places
+				baseFeeMultiplier = BigInt(Math.floor(baseFeePerGasFactor * 1000)) / BigInt(1000);
+			} else {
+				// It's already a BigInt, so just use it as-is
+				baseFeeMultiplier = baseFeePerGasFactor;
+			}
+			maxFeePerGas = baseFeePerGas * baseFeeMultiplier + maxPriorityFeePerGas;
 		}
 
 		return { gasPrice, maxFeePerGas, maxPriorityFeePerGas, baseFeePerGas };

packages/web3-eth/test/unit/web3_eth_calculate_fee_data.test.ts

@@ -81,4 +81,25 @@ describe('Web3Eth.calculateFeeData', () => {
 			baseFeePerGas,
 		});
 	});
+
+	it('should use default baseFeePerGasFactor if none is provided', async () => {
+		const gasPrice = BigInt(20 * 1000);
+		const baseFeePerGas = BigInt(1000);
+		const maxPriorityFeePerGas = BigInt(100); // this will be used directly
+
+		jest.spyOn(ethRpcMethods, 'getBlockByNumber').mockReturnValueOnce({ baseFeePerGas } as any);
+		jest.spyOn(ethRpcMethods, 'getGasPrice').mockReturnValueOnce(gasPrice as any);
+		jest.spyOn(ethRpcMethods, 'getMaxPriorityFeePerGas').mockReturnValueOnce(
+			maxPriorityFeePerGas as any,
+		);
+
+		const feeData = await web3Eth.calculateFeeData(); // no baseFeePerGasFactor passed
+		const defaultBaseFeePerGasFactor = BigInt(2);
+		expect(feeData).toMatchObject({
+			gasPrice,
+			maxFeePerGas: baseFeePerGas * defaultBaseFeePerGasFactor + maxPriorityFeePerGas,
+			maxPriorityFeePerGas,
+			baseFeePerGas,
+		});
+	});
 });