chore: fix gas limits

Author: Filipp Makarov

contracts/forwarder/Forwarder.sol

@@ -3,14 +3,15 @@ pragma solidity ^0.8.27;
 
 /**
  * @title EtherForwarder
- * @dev A contract that forwards received Ether to a specified address
+ * @notice A contract that forwards received Ether to a specified address
+ * @author Biconomy
  */
 contract EtherForwarder {
     error ZeroAddress();
     error ForwardFailed();
 
     /**
-     * @dev Forwards the received Ether to the specified destination address
+     * @notice Forwards the received Ether to the specified destination address
      * @param destination The address to forward the Ether to
      */
     function forward(address destination) external payable {
@@ -36,7 +37,7 @@ contract EtherForwarder {
     }
 
     /**
-     * @dev Prevents accidental Ether transfers without a destination
+     * @notice Prevents accidental Ether transfers without a destination
      * intentionally using sting and not a custom error here
      */
     receive() external payable {
@@ -45,7 +46,7 @@ contract EtherForwarder {
     }
 
     /**
-     * @dev Prevents accidental Ether transfers without a destination
+     * @notice Prevents accidental Ether transfers without a destination
      * intentionally using sting and not a custom error here
      */
     fallback() external payable {

contracts/lib/util/EcdsaHelperLib.sol

@@ -3,21 +3,36 @@ pragma solidity ^0.8.27;
 
 import { ECDSA } from "solady/utils/ECDSA.sol";
 
+/**
+ * @title EcdsaHelperLib
+ * @notice A library that provides helper functions for ECDSA operations
+ * @author Biconomy
+ */
 library EcdsaHelperLib {
     using ECDSA for bytes32;
-
     /**
+     * @notice Checks if the signature is valid for the given hash and expected signer
+     * @param expectedSigner The expected signer of the signature
+     * @param hash The hash of the data to be signed
+     * @param signature The signature to be validated
+     * @return bool True if the signature is valid, false otherwise
      * @dev Solady ECDSA does not revert on incorrect signatures.
      *      Instead, it returns address(0) as the recovered address.
      *      Make sure to never pass address(0) as expectedSigner to this function.
      */
+    // solhint-disable-next-line gas-named-return-values
+
     function isValidSignature(address expectedSigner, bytes32 hash, bytes memory signature) internal view returns (bool) {
         if (hash.tryRecover(signature) == expectedSigner) return true;
         if (hash.toEthSignedMessageHash().tryRecover(signature) == expectedSigner) return true;
         return false;
     }
 
     /**
+     * @notice Returns the keccak256 digest of an EIP-712 typed data (EIP-191 version `0x01`).
+     * @param domainSeparator The domain separator of the typed data
+     * @param structHash The struct hash of the typed data
+     * @return digest The keccak256 digest of the typed data
      * @dev Returns the keccak256 digest of an EIP-712 typed data (EIP-191 version `0x01`).
      *
      * The digest is calculated from a `domainSeparator` and a `structHash`, by prefixing them with

test/unit/node-paymaster/percentage-premium/PercentagePremium_Test.t.sol

@@ -69,7 +69,7 @@ contract PercentagePremium_Paymaster_Test is BaseTest {
             callData: callData,
             wallet: wallet,
             preVerificationGasLimit: 50e3,
-            verificationGasLimit: 35e3,
+            verificationGasLimit: 55e3,
             callGasLimit: 100e3
         });
 
@@ -109,7 +109,7 @@ contract PercentagePremium_Paymaster_Test is BaseTest {
     // test percentage user single
     function test_percentage_user_single() public {
         _premiumPercentage = 1_700_000;
-        uint128 pmValidationGasLimit = 15_000;
+        uint128 pmValidationGasLimit = 25_000;
         // ~ 12_000 is raw PM.postOp gas spent
         // here we add more for emitting events in the wrapper + refunds etc in EP
         uint128 pmPostOpGasLimit = 37_000;
@@ -164,7 +164,7 @@ contract PercentagePremium_Paymaster_Test is BaseTest {
         public
     {
         preVerificationGasLimit = bound(preVerificationGasLimit, 1e5, 5e6);
-        verificationGasLimit = uint128(bound(verificationGasLimit, 50e3, 5e6));
+        verificationGasLimit = uint128(bound(verificationGasLimit, 55e3, 5e6));
         callGasLimit = uint128(bound(callGasLimit, 100e3, 5e6));
         premiumPercentage = bound(premiumPercentage, 0, 200e5);
         pmValidationGasLimit = uint128(bound(pmValidationGasLimit, 30e3, 5e6));