fix op max cost calculations

pimlicolabs · Oct 14, 2024 · 3345aae · 3345aae
1 parent e1c8a0a
commit 3345aae
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 2 deletions.
diff --git a/docs/pages/infra/paymaster/erc20-paymaster/how-to/calculate-costs.mdx b/docs/pages/infra/paymaster/erc20-paymaster/how-to/calculate-costs.mdx
@@ -30,6 +30,8 @@ To find your userOperation's max cost, we will need to sum up all it's gas value
 - `paymasterVerificationGasLimit` (v0.7 only)
 - `paymasterPostOpGasLimit` (v0.7 only)
 
+Once the gas values are summed up, we multiply it by the maxFeePerGas to get the max cost for the userOperation.
+
 ```ts
 // [!include ~/snippets/erc20-paymaster/calculate-costs.ts:calculateMaxCost]
 ```

diff --git a/docs/snippets/erc20-paymaster/calculate-costs.ts b/docs/snippets/erc20-paymaster/calculate-costs.ts
@@ -83,12 +83,14 @@ const op = await smartAccountClient.prepareUserOperation({
 	},
 })
 
-const userOperationMaxCost =
+const userOperationMaxGas =
 	op.preVerificationGas +
 	op.verificationGasLimit +
 	op.callGasLimit +
 	(op.paymasterVerificationGasLimit || 0n) +
 	(op.paymasterPostOpGasLimit || 0n)
+
+const userOperationMaxCost = userOperationMaxGas * op.maxFeePerGas
 // [!endregion calculateMaxCost]
 
 // [!region calculateCostInToken]

diff --git a/docs/snippets/erc20-paymaster/how-to.ts b/docs/snippets/erc20-paymaster/how-to.ts
@@ -79,9 +79,11 @@ const userOperationMaxGas =
 	(userOperation.paymasterPostOpGasLimit || 0n) +
 	(userOperation.paymasterVerificationGasLimit || 0n)
 
+const userOperationMaxCost = userOperationMaxGas * userOperation.maxFeePerGas
+
 // using formula here https://github.com/pimlicolabs/singleton-paymaster/blob/main/src/base/BaseSingletonPaymaster.sol#L334-L341
 const maxCostInToken =
-	((userOperationMaxGas + postOpGas * userOperation.maxFeePerGas) * exchangeRate) / BigInt(1e18)
+	((userOperationMaxCost + postOpGas * userOperation.maxFeePerGas) * exchangeRate) / BigInt(1e18)
 
 // [!endregion getTokenQuotes]