fix: change agreements mapping to internal to fix stack-too-deep in coverage

Author: Maikol

packages/horizon/contracts/payments/collectors/RecurringCollector.sol

@@ -42,7 +42,7 @@ contract RecurringCollector is EIP712, GraphDirectory, Authorizable, IRecurringC
     /* solhint-enable gas-small-strings */
 
     /// @notice Tracks agreements
-    mapping(bytes16 agreementId => AgreementData data) public agreements;
+    mapping(bytes16 agreementId => AgreementData data) internal agreements;
 
     /**
      * @notice Constructs a new instance of the RecurringCollector contract.

packages/horizon/package.json

@@ -35,7 +35,7 @@
     "test:deployment": "SECURE_ACCOUNTS_DISABLE_PROVIDER=true hardhat test test/deployment/*.ts",
     "test:integration": "./scripts/integration",
     "test:coverage": "pnpm build && pnpm test:coverage:self",
-    "test:coverage:self": "forge coverage --ir-minimum",
+    "test:coverage:self": "forge coverage",
     "prepublishOnly": "pnpm run build"
   },
   "devDependencies": {

packages/subgraph-service/package.json

@@ -33,7 +33,7 @@
     "test:deployment": "SECURE_ACCOUNTS_DISABLE_PROVIDER=true hardhat test test/deployment/*.ts",
     "test:integration": "./scripts/integration",
     "test:coverage": "pnpm build && pnpm test:coverage:self",
-    "test:coverage:self": "forge coverage --ir-minimum",
+    "test:coverage:self": "forge coverage",
     "prepublishOnly": "pnpm run build"
   },
   "devDependencies": {