Merge pull request #332 from pimlicolabs/revert-331-add-second-simula…

…tion-logs Revert "Add logs to check if the second simulation fails"
pimlicolabs · Oct 19, 2024 · e6d99a0 · e6d99a0
2 parents 8b04215 + 3a947aa
commit e6d99a0
Showing 1 changed file with 5 additions and 6 deletions.
diff --git a/src/rpc/rpcHandler.ts b/src/rpc/rpcHandler.ts
@@ -468,7 +468,7 @@ export class RpcHandler implements IRpcEndpoint {
             entryPoint,
             queuedUserOperations,
             true,
-            deepHexlify({ ...stateOverrides })
+            stateOverrides
         )
 
         let { verificationGasLimit, callGasLimit } =
@@ -527,8 +527,9 @@ export class RpcHandler implements IRpcEndpoint {
             callGasLimit = 0n
         }
 
-        // Run second time without state overrides
-        try {
+        // If a balance override is provided for the sender, perform an additional simulation
+        // to verify the userOperation succeeds with the specified balance.
+        if (stateOverrides?.[userOperation.sender]?.balance) {
             await this.validator.getExecutionResult(
                 {
                     ...userOperation,
@@ -541,10 +542,8 @@ export class RpcHandler implements IRpcEndpoint {
                 entryPoint,
                 queuedUserOperations,
                 false,
-                deepHexlify({ ...stateOverrides })
+                deepHexlify(stateOverrides)
             )
-        } catch (err) {
-            this.logger.error(err, "Second simulations fail")
         }
 
         if (isVersion07(userOperation)) {