fix: audit cleanup

Author: horsefacts

.gas-snapshot

@@ -55,8 +55,8 @@ jobs:
       - name: Wait for contract to be deployed
         uses: nick-fields/retry@v2
         with:
-          timeout_seconds: 5
-          retry_wait_seconds: 5
+          timeout_seconds: 30
+          retry_wait_seconds: 10
           max_attempts: 10
           shell: bash
           command: |

.github/workflows/ci.yml

@@ -23,7 +23,7 @@ contract Bundler is IBundler {
     /**
      * @inheritdoc IBundler
      */
-    string public constant VERSION = "2025.06.13";
+    string public constant VERSION = "2025.06.16";
 
     /*//////////////////////////////////////////////////////////////
                                 IMMUTABLES
@@ -88,16 +88,11 @@ contract Bundler is IBundler {
 
     function _addKeys(address fidOwner, SignerParams[] calldata signerParams) internal {
         uint256 signersLen = signerParams.length;
-        for (uint256 i; i < signersLen;) {
+        for (uint256 i; i < signersLen; ++i) {
             SignerParams calldata signer = signerParams[i];
             keyGateway.addFor(
                 fidOwner, signer.keyType, signer.key, signer.metadataType, signer.metadata, signer.deadline, signer.sig
             );
-
-            // Safety: i can be incremented unchecked since it is bound by signerParams.length.
-            unchecked {
-                ++i;
-            }
         }
     }
 

lib/forge-std

@@ -23,7 +23,7 @@ contract TierRegistry is ITierRegistry, Migration {
     /**
      * @inheritdoc ITierRegistry
      */
-    string public constant VERSION = "2025.06.01";
+    string public constant VERSION = "2025.06.16";
 
     /*//////////////////////////////////////////////////////////////
                                 STORAGE

src/Bundler.sol

@@ -60,7 +60,7 @@ interface ITierRegistry is IMigration {
      * @param forDays The number of days of subscription purchased
      * @param payer The address that paid
      */
-    event PurchasedTier(uint256 indexed fid, uint256 indexed tier, uint256 forDays, address payer);
+    event PurchasedTier(uint256 indexed fid, uint256 indexed tier, uint256 forDays, address indexed payer);
 
     /**
      * @notice Emitted when a tier is deactivated

src/TierRegistry.sol

@@ -29,7 +29,7 @@ contract BundlerTest is BundlerTestSuite {
     }
 
     function testVersion() public {
-        assertEq(bundler.VERSION(), "2025.06.13");
+        assertEq(bundler.VERSION(), "2025.06.16");
     }
 
     /*//////////////////////////////////////////////////////////////

src/interfaces/ITierRegistry.sol

@@ -59,7 +59,7 @@ abstract contract TestSuiteSetup is Test {
     function _assumeClean(
         address a
     ) internal {
-        assumeNoPrecompiles(a);
+        assumeNotPrecompile(a);
         vm.assume(!isKnownContract[a]);
         vm.assume(a != ADMIN);
         vm.assume(a != address(0));

test/Bundler/Bundler.t.sol

@@ -16,7 +16,7 @@ contract TierRegistryTest is TierRegistryTestSuite {
                                  EVENTS
     //////////////////////////////////////////////////////////////*/
 
-    event PurchasedTier(uint256 indexed fid, uint256 indexed tier, uint256 forDays, address payer);
+    event PurchasedTier(uint256 indexed fid, uint256 indexed tier, uint256 forDays, address indexed payer);
     event DeactivateTier(uint256 indexed tier);
     event SetTier(
         uint256 indexed tier,
@@ -35,7 +35,7 @@ contract TierRegistryTest is TierRegistryTestSuite {
     //////////////////////////////////////////////////////////////*/
 
     function testVersion() public {
-        assertEq(tierRegistry.VERSION(), "2025.06.01");
+        assertEq(tierRegistry.VERSION(), "2025.06.16");
     }
 
     function testOwner() public {