@bananapus/core-v6 0.0.14 → 0.0.16

package/test/{AuditExploits.t.sol → CoreExploitTests.t.sol}

@@ -1,7 +1,32 @@
 // SPDX-License-Identifier: MIT
 pragma solidity ^0.8.6;
 
-import /* {*} from */ "./helpers/TestBaseWorkflow.sol";
+import {TestBaseWorkflow} from "./helpers/TestBaseWorkflow.sol";
+import {MetadataResolverHelper} from "./helpers/MetadataResolverHelper.sol";
+import {MockERC20} from "./mock/MockERC20.sol";
+import {JBMultiTerminal} from "../src/JBMultiTerminal.sol";
+import {JBTokens} from "../src/JBTokens.sol";
+import {JBApprovalStatus} from "../src/enums/JBApprovalStatus.sol";
+import {IJBController} from "../src/interfaces/IJBController.sol";
+import {IJBMultiTerminal} from "../src/interfaces/IJBMultiTerminal.sol";
+import {IJBPayHook} from "../src/interfaces/IJBPayHook.sol";
+import {IJBRulesetApprovalHook} from "../src/interfaces/IJBRulesetApprovalHook.sol";
+import {IJBRulesetDataHook} from "../src/interfaces/IJBRulesetDataHook.sol";
+import {IJBTerminal} from "../src/interfaces/IJBTerminal.sol";
+import {IJBToken} from "../src/interfaces/IJBToken.sol";
+import {JBConstants} from "../src/libraries/JBConstants.sol";
+import {JBFees} from "../src/libraries/JBFees.sol";
+import {JBAccountingContext} from "../src/structs/JBAccountingContext.sol";
+import {JBCurrencyAmount} from "../src/structs/JBCurrencyAmount.sol";
+import {JBFee} from "../src/structs/JBFee.sol";
+import {JBFundAccessLimitGroup} from "../src/structs/JBFundAccessLimitGroup.sol";
+import {JBPayHookSpecification} from "../src/structs/JBPayHookSpecification.sol";
+import {JBRuleset} from "../src/structs/JBRuleset.sol";
+import {JBRulesetConfig} from "../src/structs/JBRulesetConfig.sol";
+import {JBRulesetMetadata} from "../src/structs/JBRulesetMetadata.sol";
+import {JBSplitGroup} from "../src/structs/JBSplitGroup.sol";
+import {JBTerminalConfig} from "../src/structs/JBTerminalConfig.sol";
+import {ERC165, IERC165} from "@openzeppelin/contracts/utils/introspection/ERC165.sol";
 import {JBCashOuts} from "../src/libraries/JBCashOuts.sol";
 
 /// @notice Tests for edge case scenarios in Juicebox V5.
@@ -682,7 +707,7 @@ contract EdgeCases_Local is TestBaseWorkflow {
         // Cash out all remaining tokens.
         uint256 tokenBalance = _tokens.totalBalanceOf(_beneficiary, projectId);
         vm.prank(_beneficiary);
-        uint256 reclaimAmount = _terminal.cashOutTokensOf({
+        _terminal.cashOutTokensOf({
             holder: _beneficiary,
             projectId: projectId,
             cashOutCount: tokenBalance,
@@ -1227,7 +1252,9 @@ contract EdgeCases_Local is TestBaseWorkflow {
         uint256 hugeBorrow = uint256(type(uint112).max) + 500 ether;
 
         // Silent truncation — Solidity 0.8.x does NOT revert on explicit downcasts!
+        // forge-lint: disable-next-line(unsafe-typecast)
         uint112 truncatedCollateral = uint112(hugeCollateral);
+        // forge-lint: disable-next-line(unsafe-typecast)
         uint112 truncatedBorrow = uint112(hugeBorrow);
 
         // The truncated values are dramatically smaller.
@@ -1451,8 +1478,6 @@ contract EdgeCases_Local is TestBaseWorkflow {
         vm.warp(block.timestamp + 2_419_200 + 1); // 28 days + 1 second
 
         // Record terminal ETH before processing.
-        uint256 terminalBalanceBefore = address(_terminal).balance;
-
         // Process both held fees.
         _terminal.processHeldFeesOf(projectId, JBConstants.NATIVE_TOKEN, 100);
 
@@ -2052,6 +2077,7 @@ contract EdgeCases_Local is TestBaseWorkflow {
 
     /// @notice Helper: launches a project with only MockERC20 (6 decimals, simulating USDC on Tempo).
     ///         No NATIVE_TOKEN accounting context is registered.
+    // forge-lint: disable-next-line(mixed-case-function)
     function _launchProjectERC20(uint16 cashOutTaxRate, uint16 reservedPercent) internal returns (uint256 projectId) {
         MockERC20 token = usdcToken();
 
@@ -2121,6 +2147,7 @@ contract EdgeCases_Local is TestBaseWorkflow {
     }
 
     /// @notice Helper: launches an ERC-20-only project with payout limits.
+    // forge-lint: disable-next-line(mixed-case-function)
     function _launchProjectERC20WithPayoutLimit(
         uint16 cashOutTaxRate,
         uint224 payoutLimit
@@ -2634,8 +2661,11 @@ contract ReentrancyAttacker {
 
 /// @notice Malicious fee terminal that reenters processHeldFeesOf during pay().
 contract ReentrantFeeTerminal is ERC165 {
+    // forge-lint: disable-next-line(screaming-snake-case-immutable)
     IJBMultiTerminal public immutable victim;
+    // forge-lint: disable-next-line(screaming-snake-case-immutable)
     uint256 public immutable targetProjectId;
+    // forge-lint: disable-next-line(screaming-snake-case-immutable)
     address public immutable targetToken;
     uint256 public payCallCount;
 
@@ -2690,6 +2720,7 @@ contract ReentrantFeeTerminal is ERC165 {
 
 /// @notice Malicious approval hook that always reverts.
 contract MaliciousApprovalHook is ERC165, IJBRulesetApprovalHook {
+    // forge-lint: disable-next-line(mixed-case-function)
     function DURATION() external pure override returns (uint256) {
         return 1 days;
     }

package/test/EconomicSimulation.t.sol

@@ -1,8 +1,16 @@
 // SPDX-License-Identifier: MIT
 pragma solidity ^0.8.6;
 
-import "forge-std/StdInvariant.sol";
-import /* {*} from */ "./helpers/TestBaseWorkflow.sol";
+import {StdInvariant} from "forge-std/StdInvariant.sol";
+import {TestBaseWorkflow} from "./helpers/TestBaseWorkflow.sol";
+import {IJBRulesetApprovalHook} from "../src/interfaces/IJBRulesetApprovalHook.sol";
+import {JBRulesetMetadataResolver} from "../src/libraries/JBRulesetMetadataResolver.sol";
+import {JBFundAccessLimitGroup} from "../src/structs/JBFundAccessLimitGroup.sol";
+import {JBRuleset} from "../src/structs/JBRuleset.sol";
+import {JBRulesetConfig} from "../src/structs/JBRulesetConfig.sol";
+import {JBRulesetMetadata} from "../src/structs/JBRulesetMetadata.sol";
+import {JBSplitGroup} from "../src/structs/JBSplitGroup.sol";
+import {JBTerminalConfig} from "../src/structs/JBTerminalConfig.sol";
 import {EconomicHandler} from "./invariants/handlers/EconomicHandler.sol";
 import {JBAccountingContext} from "../src/structs/JBAccountingContext.sol";
 import {JBConstants} from "../src/libraries/JBConstants.sol";

package/test/EntryPointPermutations.t.sol

@@ -1,10 +1,19 @@
 // SPDX-License-Identifier: MIT
 pragma solidity ^0.8.6;
 
-import /* {*} from */ "./helpers/TestBaseWorkflow.sol";
+import {TestBaseWorkflow} from "./helpers/TestBaseWorkflow.sol";
+import {IJBRulesetApprovalHook} from "../src/interfaces/IJBRulesetApprovalHook.sol";
+import {JBRulesetMetadataResolver} from "../src/libraries/JBRulesetMetadataResolver.sol";
+import {JBCurrencyAmount} from "../src/structs/JBCurrencyAmount.sol";
+import {JBFee} from "../src/structs/JBFee.sol";
+import {JBFundAccessLimitGroup} from "../src/structs/JBFundAccessLimitGroup.sol";
+import {JBRuleset} from "../src/structs/JBRuleset.sol";
+import {JBRulesetConfig} from "../src/structs/JBRulesetConfig.sol";
+import {JBRulesetMetadata} from "../src/structs/JBRulesetMetadata.sol";
+import {JBSplitGroup} from "../src/structs/JBSplitGroup.sol";
+import {JBTerminalConfig} from "../src/structs/JBTerminalConfig.sol";
 import {JBAccountingContext} from "../src/structs/JBAccountingContext.sol";
 import {JBConstants} from "../src/libraries/JBConstants.sol";
-import {JBFees} from "../src/libraries/JBFees.sol";
 
 /// @title EntryPointPermutations
 /// @notice Systematically tests every JBMultiTerminal external function with edge-case parameters
@@ -452,7 +461,7 @@ contract EntryPointPermutations_Local is TestBaseWorkflow {
         );
 
         // Held fees should be reduced (partially or fully returned)
-        JBFee[] memory feesAfter = jbMultiTerminal().heldFeesOf(projectStandard, JBConstants.NATIVE_TOKEN, 100);
+        jbMultiTerminal().heldFeesOf(projectStandard, JBConstants.NATIVE_TOKEN, 100);
 
         if (feesBefore.length > 0) {
             // Either fewer fees or smaller amounts
@@ -593,7 +602,9 @@ contract EntryPointPermutations_Local is TestBaseWorkflow {
             if (payoutLimit > 0) {
                 payoutLimits = new JBCurrencyAmount[](1);
                 payoutLimits[0] = JBCurrencyAmount({
-                    amount: uint224(payoutLimit), currency: uint32(uint160(JBConstants.NATIVE_TOKEN))
+                    // forge-lint: disable-next-line(unsafe-typecast)
+                    amount: uint224(payoutLimit),
+                    currency: uint32(uint160(JBConstants.NATIVE_TOKEN))
                 });
             } else {
                 payoutLimits = new JBCurrencyAmount[](0);
@@ -602,7 +613,9 @@ contract EntryPointPermutations_Local is TestBaseWorkflow {
             if (surplusAllowance > 0) {
                 surplusAllowances = new JBCurrencyAmount[](1);
                 surplusAllowances[0] = JBCurrencyAmount({
-                    amount: uint224(surplusAllowance), currency: uint32(uint160(JBConstants.NATIVE_TOKEN))
+                    // forge-lint: disable-next-line(unsafe-typecast)
+                    amount: uint224(surplusAllowance),
+                    currency: uint32(uint160(JBConstants.NATIVE_TOKEN))
                 });
             } else {
                 surplusAllowances = new JBCurrencyAmount[](0);

package/test/FlashLoanAttacks.t.sol

@@ -1,7 +1,17 @@
 // SPDX-License-Identifier: MIT
 pragma solidity ^0.8.6;
 
-import /* {*} from */ "./helpers/TestBaseWorkflow.sol";
+import {TestBaseWorkflow} from "./helpers/TestBaseWorkflow.sol";
+import {IJBRulesetApprovalHook} from "../src/interfaces/IJBRulesetApprovalHook.sol";
+import {JBConstants} from "../src/libraries/JBConstants.sol";
+import {JBRulesetMetadataResolver} from "../src/libraries/JBRulesetMetadataResolver.sol";
+import {JBCurrencyAmount} from "../src/structs/JBCurrencyAmount.sol";
+import {JBFundAccessLimitGroup} from "../src/structs/JBFundAccessLimitGroup.sol";
+import {JBRuleset} from "../src/structs/JBRuleset.sol";
+import {JBRulesetConfig} from "../src/structs/JBRulesetConfig.sol";
+import {JBRulesetMetadata} from "../src/structs/JBRulesetMetadata.sol";
+import {JBSplitGroup} from "../src/structs/JBSplitGroup.sol";
+import {JBTerminalConfig} from "../src/structs/JBTerminalConfig.sol";
 import {JBAccountingContext} from "../src/structs/JBAccountingContext.sol";
 
 /// @notice Tests that flash-loan style atomic pay+cashOut attacks cannot extract profit.
@@ -254,7 +264,6 @@ contract FlashLoanAttacks_Local is TestBaseWorkflow {
 
         // Both should get proportional shares (with cashOutTax reducing it)
         // Key check: they should get roughly equal amounts since they have equal tokens
-        uint256 diff = aliceReclaim > bobReclaim ? aliceReclaim - bobReclaim : bobReclaim - aliceReclaim;
         // Alice cashes out first, so she gets slightly more due to reduced supply.
         // But the proportional split should be reasonable.
         assertTrue(aliceReclaim > 0, "Alice should get some reclaim");
@@ -549,6 +558,7 @@ contract FlashLoanAttacks_Local is TestBaseWorkflow {
 
         // Attacker front-runs: pays right before payout
         address attacker = address(0xA77AC0);
+        // forge-lint: disable-next-line(mixed-case-variable)
         uint256 attackerInitialETH = 10 ether;
         vm.deal(attacker, attackerInitialETH);
         vm.prank(attacker);

package/test/PermissionEscalation.t.sol

@@ -1,7 +1,20 @@
 // SPDX-License-Identifier: MIT
 pragma solidity ^0.8.6;
 
-import /* {*} from */ "./helpers/TestBaseWorkflow.sol";
+import {TestBaseWorkflow} from "./helpers/TestBaseWorkflow.sol";
+import {JBPermissions} from "../src/JBPermissions.sol";
+import {IJBRulesetApprovalHook} from "../src/interfaces/IJBRulesetApprovalHook.sol";
+import {JBConstants} from "../src/libraries/JBConstants.sol";
+import {JBRulesetMetadataResolver} from "../src/libraries/JBRulesetMetadataResolver.sol";
+import {JBCurrencyAmount} from "../src/structs/JBCurrencyAmount.sol";
+import {JBFundAccessLimitGroup} from "../src/structs/JBFundAccessLimitGroup.sol";
+import {JBPermissionsData} from "../src/structs/JBPermissionsData.sol";
+import {JBRuleset} from "../src/structs/JBRuleset.sol";
+import {JBRulesetConfig} from "../src/structs/JBRulesetConfig.sol";
+import {JBRulesetMetadata} from "../src/structs/JBRulesetMetadata.sol";
+import {JBSplitGroup} from "../src/structs/JBSplitGroup.sol";
+import {JBTerminalConfig} from "../src/structs/JBTerminalConfig.sol";
+import {JBPermissionIds} from "@bananapus/permission-ids-v6/src/JBPermissionIds.sol";
 import {JBAccountingContext} from "../src/structs/JBAccountingContext.sol";
 
 /// @notice Tests for permission system correctness: ROOT restrictions, wildcard, boundary IDs, escalation prevention.
@@ -38,6 +51,7 @@ contract PermissionEscalation_Local is TestBaseWorkflow {
         jbPermissions()
             .setPermissionsFor(
                 projectOwner,
+                // forge-lint: disable-next-line(unsafe-typecast)
                 JBPermissionsData({operator: alice, projectId: uint64(projectId2), permissionIds: rootPerms})
             );
     }
@@ -175,13 +189,17 @@ contract PermissionEscalation_Local is TestBaseWorkflow {
                 JBPermissions.JBPermissions_Unauthorized.selector,
                 projectOwner,
                 alice,
+                // forge-lint: disable-next-line(unsafe-typecast)
                 uint64(projectId2),
                 JBPermissionIds.ROOT
             )
         );
         jbPermissions()
             .setPermissionsFor(
-                projectOwner, JBPermissionsData({operator: bob, projectId: uint64(projectId2), permissionIds: perms})
+                // forge-lint: disable-next-line(unsafe-typecast)
+                projectOwner,
+                // forge-lint: disable-next-line(unsafe-typecast)
+                JBPermissionsData({operator: bob, projectId: uint64(projectId2), permissionIds: perms})
             );
     }
 
@@ -197,7 +215,10 @@ contract PermissionEscalation_Local is TestBaseWorkflow {
         vm.prank(alice);
         jbPermissions()
             .setPermissionsFor(
-                projectOwner, JBPermissionsData({operator: bob, projectId: uint64(projectId2), permissionIds: perms})
+                // forge-lint: disable-next-line(unsafe-typecast)
+                projectOwner,
+                // forge-lint: disable-next-line(unsafe-typecast)
+                JBPermissionsData({operator: bob, projectId: uint64(projectId2), permissionIds: perms})
             );
 
         // Verify Bob now has CASH_OUT_TOKENS
@@ -263,7 +284,10 @@ contract PermissionEscalation_Local is TestBaseWorkflow {
         vm.prank(projectOwner);
         jbPermissions()
             .setPermissionsFor(
-                projectOwner, JBPermissionsData({operator: bob, projectId: uint64(projectId2), permissionIds: perms})
+                // forge-lint: disable-next-line(unsafe-typecast)
+                projectOwner,
+                // forge-lint: disable-next-line(unsafe-typecast)
+                JBPermissionsData({operator: bob, projectId: uint64(projectId2), permissionIds: perms})
             );
 
         // Verify Bob has it
@@ -287,6 +311,7 @@ contract PermissionEscalation_Local is TestBaseWorkflow {
         jbPermissions()
             .setPermissionsFor(
                 projectOwner,
+                // forge-lint: disable-next-line(unsafe-typecast)
                 JBPermissionsData({operator: bob, projectId: uint64(projectId2), permissionIds: emptyPerms})
             );
 
@@ -317,7 +342,10 @@ contract PermissionEscalation_Local is TestBaseWorkflow {
         vm.prank(projectOwner);
         jbPermissions()
             .setPermissionsFor(
-                projectOwner, JBPermissionsData({operator: bob, projectId: uint64(projectId2), permissionIds: perms})
+                // forge-lint: disable-next-line(unsafe-typecast)
+                projectOwner,
+                // forge-lint: disable-next-line(unsafe-typecast)
+                JBPermissionsData({operator: bob, projectId: uint64(projectId2), permissionIds: perms})
             );
 
         bool hasPerm = jbPermissions()
@@ -357,7 +385,10 @@ contract PermissionEscalation_Local is TestBaseWorkflow {
         vm.expectRevert(abi.encodeWithSelector(JBPermissions.JBPermissions_NoZeroPermission.selector));
         jbPermissions()
             .setPermissionsFor(
-                projectOwner, JBPermissionsData({operator: bob, projectId: uint64(projectId2), permissionIds: perms})
+                // forge-lint: disable-next-line(unsafe-typecast)
+                projectOwner,
+                // forge-lint: disable-next-line(unsafe-typecast)
+                JBPermissionsData({operator: bob, projectId: uint64(projectId2), permissionIds: perms})
             );
     }
 
@@ -468,7 +499,10 @@ contract PermissionEscalation_Local is TestBaseWorkflow {
         vm.prank(projectOwner);
         jbPermissions()
             .setPermissionsFor(
-                projectOwner, JBPermissionsData({operator: bob, projectId: uint64(pid), permissionIds: perms})
+                // forge-lint: disable-next-line(unsafe-typecast)
+                projectOwner,
+                // forge-lint: disable-next-line(unsafe-typecast)
+                JBPermissionsData({operator: bob, projectId: uint64(pid), permissionIds: perms})
             );
 
         // Charlie (no permission) tries to send payouts — should fail because ownerMustSendPayouts
@@ -500,7 +534,7 @@ contract PermissionEscalation_Local is TestBaseWorkflow {
     //  Test 10: ROOT on project 2 has no power over project 3
     // ═══════════════════════════════════════════════════════════════════
 
-    function test_permission_rootOnProject_doesNotAffectOtherProject() public {
+    function test_permission_rootOnProject_doesNotAffectOtherProject() public view {
         // Alice has ROOT on project 2 (from setUp)
         // Alice should NOT have any permissions on project 3
         bool hasPermOnProject3 = jbPermissions()
@@ -539,7 +573,10 @@ contract PermissionEscalation_Local is TestBaseWorkflow {
         vm.prank(projectOwner);
         jbPermissions()
             .setPermissionsFor(
-                projectOwner, JBPermissionsData({operator: bob, projectId: uint64(projectId2), permissionIds: perms1})
+                // forge-lint: disable-next-line(unsafe-typecast)
+                projectOwner,
+                // forge-lint: disable-next-line(unsafe-typecast)
+                JBPermissionsData({operator: bob, projectId: uint64(projectId2), permissionIds: perms1})
             );
 
         assertTrue(
@@ -574,7 +611,10 @@ contract PermissionEscalation_Local is TestBaseWorkflow {
         vm.prank(projectOwner);
         jbPermissions()
             .setPermissionsFor(
-                projectOwner, JBPermissionsData({operator: bob, projectId: uint64(projectId2), permissionIds: perms2})
+                // forge-lint: disable-next-line(unsafe-typecast)
+                projectOwner,
+                // forge-lint: disable-next-line(unsafe-typecast)
+                JBPermissionsData({operator: bob, projectId: uint64(projectId2), permissionIds: perms2})
             );
 
         // Bob should only have MINT_TOKENS, not the previous ones
@@ -628,7 +668,10 @@ contract PermissionEscalation_Local is TestBaseWorkflow {
         vm.prank(projectOwner);
         jbPermissions()
             .setPermissionsFor(
-                projectOwner, JBPermissionsData({operator: bob, projectId: uint64(projectId2), permissionIds: perms})
+                // forge-lint: disable-next-line(unsafe-typecast)
+                projectOwner,
+                // forge-lint: disable-next-line(unsafe-typecast)
+                JBPermissionsData({operator: bob, projectId: uint64(projectId2), permissionIds: perms})
             );
 
         // Call from trusted forwarder with Bob appended as sender
@@ -638,17 +681,6 @@ contract PermissionEscalation_Local is TestBaseWorkflow {
         // Verify the forwarder is set up correctly
         assertTrue(forwarder != address(0), "Trusted forwarder should be set");
 
-        // Construct the calldata with Bob's address appended (ERC2771 pattern)
-        bytes memory callData = abi.encodeWithSelector(
-            IJBPermissions.hasPermission.selector,
-            bob,
-            projectOwner,
-            projectId2,
-            JBPermissionIds.CASH_OUT_TOKENS,
-            false,
-            false
-        );
-
         // Direct call should return true
         bool hasPerm = jbPermissions()
             .hasPermission({

package/test/RulesetTransitions.t.sol

@@ -1,7 +1,18 @@
 // SPDX-License-Identifier: MIT
 pragma solidity ^0.8.6;
 
-import /* {*} from */ "./helpers/TestBaseWorkflow.sol";
+import {TestBaseWorkflow} from "./helpers/TestBaseWorkflow.sol";
+import {JBApprovalStatus} from "../src/enums/JBApprovalStatus.sol";
+import {IJBRulesetApprovalHook} from "../src/interfaces/IJBRulesetApprovalHook.sol";
+import {JBConstants} from "../src/libraries/JBConstants.sol";
+import {JBRulesetMetadataResolver} from "../src/libraries/JBRulesetMetadataResolver.sol";
+import {JBCurrencyAmount} from "../src/structs/JBCurrencyAmount.sol";
+import {JBFundAccessLimitGroup} from "../src/structs/JBFundAccessLimitGroup.sol";
+import {JBRuleset} from "../src/structs/JBRuleset.sol";
+import {JBRulesetConfig} from "../src/structs/JBRulesetConfig.sol";
+import {JBRulesetMetadata} from "../src/structs/JBRulesetMetadata.sol";
+import {JBSplitGroup} from "../src/structs/JBSplitGroup.sol";
+import {JBTerminalConfig} from "../src/structs/JBTerminalConfig.sol";
 import {JBAccountingContext} from "../src/structs/JBAccountingContext.sol";
 import {ERC165, IERC165} from "@openzeppelin/contracts/utils/introspection/ERC165.sol";
 
@@ -255,6 +266,7 @@ contract RulesetTransitions_Local is TestBaseWorkflow {
         uint256[] memory tokensPerCycle = new uint256[](10);
         for (uint256 i = 0; i < 10; i++) {
             if (i > 0) vm.warp(block.timestamp + 1 days + 1);
+            // forge-lint: disable-next-line(unsafe-typecast)
             tokensPerCycle[i] = _payProject(pid, address(uint160(0x6000 + i)), 1 ether);
         }
 
@@ -660,6 +672,7 @@ contract RulesetTransitions_Local is TestBaseWorkflow {
 
 /// @notice Approval hook that approves after a configurable delay.
 contract DelayedApprovalHook is ERC165, IJBRulesetApprovalHook {
+    // forge-lint: disable-next-line(screaming-snake-case-immutable)
     uint256 public immutable approvalDelay;
 
     constructor(uint256 _approvalDelay) {
@@ -685,6 +698,7 @@ contract DelayedApprovalHook is ERC165, IJBRulesetApprovalHook {
 
 /// @notice Approval hook that always rejects.
 contract AlwaysRejectingHook is ERC165, IJBRulesetApprovalHook {
+    // forge-lint: disable-next-line(mixed-case-function)
     function DURATION() external pure override returns (uint256) {
         return 0;
     }

package/test/SplitLoopTests.t.sol

@@ -1,7 +1,22 @@
 // SPDX-License-Identifier: MIT
 pragma solidity ^0.8.6;
 
-import /* {*} from */ "./helpers/TestBaseWorkflow.sol";
+import {TestBaseWorkflow} from "./helpers/TestBaseWorkflow.sol";
+import {IJBMultiTerminal} from "../src/interfaces/IJBMultiTerminal.sol";
+import {IJBRulesetApprovalHook} from "../src/interfaces/IJBRulesetApprovalHook.sol";
+import {IJBSplitHook} from "../src/interfaces/IJBSplitHook.sol";
+import {IJBTokens} from "../src/interfaces/IJBTokens.sol";
+import {JBConstants} from "../src/libraries/JBConstants.sol";
+import {JBRulesetMetadataResolver} from "../src/libraries/JBRulesetMetadataResolver.sol";
+import {JBCurrencyAmount} from "../src/structs/JBCurrencyAmount.sol";
+import {JBFundAccessLimitGroup} from "../src/structs/JBFundAccessLimitGroup.sol";
+import {JBRuleset} from "../src/structs/JBRuleset.sol";
+import {JBRulesetConfig} from "../src/structs/JBRulesetConfig.sol";
+import {JBRulesetMetadata} from "../src/structs/JBRulesetMetadata.sol";
+import {JBSplit} from "../src/structs/JBSplit.sol";
+import {JBSplitGroup} from "../src/structs/JBSplitGroup.sol";
+import {JBSplitHookContext} from "../src/structs/JBSplitHookContext.sol";
+import {JBTerminalConfig} from "../src/structs/JBTerminalConfig.sol";
 import {JBAccountingContext} from "../src/structs/JBAccountingContext.sol";
 import {ERC165, IERC165} from "@openzeppelin/contracts/utils/introspection/ERC165.sol";
 
@@ -120,7 +135,8 @@ contract SplitLoopTests_Local is TestBaseWorkflow {
 
         JBCurrencyAmount[] memory payoutLimits = new JBCurrencyAmount[](1);
         payoutLimits[0] =
-            JBCurrencyAmount({amount: uint224(payoutLimit), currency: uint32(uint160(JBConstants.NATIVE_TOKEN))});
+        // forge-lint: disable-next-line(unsafe-typecast)
+        JBCurrencyAmount({amount: uint224(payoutLimit), currency: uint32(uint160(JBConstants.NATIVE_TOKEN))});
         JBFundAccessLimitGroup[] memory fundAccessLimitGroups = new JBFundAccessLimitGroup[](1);
         fundAccessLimitGroups[0] = JBFundAccessLimitGroup({
             terminal: address(jbMultiTerminal()),
@@ -186,12 +202,14 @@ contract SplitLoopTests_Local is TestBaseWorkflow {
         projectA = _launchProjectWithSplitsAndPayoutLimit(dummySplits, 10 ether);
 
         // Now create B with split to A
+        // forge-lint: disable-next-line(unsafe-typecast)
         splitsB[0].projectId = uint64(projectA);
         projectB = _launchProjectWithSplitsAndPayoutLimit(splitsB, 10 ether);
 
         // Update A's splits to point to B
         (JBRuleset memory rulesetA,) = jbController().currentRulesetOf(projectA);
         JBSplitGroup[] memory newSplitGroupsA = new JBSplitGroup[](1);
+        // forge-lint: disable-next-line(unsafe-typecast)
         splitsA[0].projectId = uint64(projectB);
         newSplitGroupsA[0] = JBSplitGroup({groupId: uint256(uint160(JBConstants.NATIVE_TOKEN)), splits: splitsA});
         vm.prank(projectOwner);
@@ -372,6 +390,7 @@ contract SplitLoopTests_Local is TestBaseWorkflow {
         uint256 pid = _launchProjectWithSplitsAndPayoutLimit(new JBSplit[](0), 10 ether);
 
         // Update splits to point to self
+        // forge-lint: disable-next-line(unsafe-typecast)
         splits[0].projectId = uint64(pid);
         (JBRuleset memory ruleset,) = jbController().currentRulesetOf(pid);
         JBSplitGroup[] memory newSplitGroups = new JBSplitGroup[](1);
@@ -404,12 +423,14 @@ contract SplitLoopTests_Local is TestBaseWorkflow {
         uint256 numSplits = 200;
         JBSplit[] memory splits = new JBSplit[](numSplits);
 
+        // forge-lint: disable-next-line(unsafe-typecast)
         uint32 perSplitPercent = uint32(JBConstants.SPLITS_TOTAL_PERCENT / numSplits);
 
         for (uint256 i = 0; i < numSplits; i++) {
             splits[i] = JBSplit({
                 percent: perSplitPercent,
                 projectId: 0,
+                // forge-lint: disable-next-line(unsafe-typecast)
                 beneficiary: payable(address(uint160(0x3000 + i))),
                 preferAddToBalance: false,
                 lockedUntil: 0,
@@ -452,6 +473,7 @@ contract SplitLoopTests_Local is TestBaseWorkflow {
             splits[i] = JBSplit({
                 percent: oneThird,
                 projectId: 0,
+                // forge-lint: disable-next-line(unsafe-typecast)
                 beneficiary: payable(address(uint160(0x4000 + i))),
                 preferAddToBalance: false,
                 lockedUntil: 0,
@@ -463,10 +485,8 @@ contract SplitLoopTests_Local is TestBaseWorkflow {
 
         _payProject(pid, address(0xBA1E), 10 ether);
 
-        uint256 totalBefore = address(jbMultiTerminal()).balance;
-
         vm.prank(projectOwner);
-        uint256 netLeftover = jbMultiTerminal()
+        jbMultiTerminal()
             .sendPayoutsOf({
                 projectId: pid,
                 token: JBConstants.NATIVE_TOKEN,
@@ -478,6 +498,7 @@ contract SplitLoopTests_Local is TestBaseWorkflow {
         // Check no rounding loss greater than 3 wei (one per split)
         uint256 totalPaidToSplits = 0;
         for (uint256 i = 0; i < 3; i++) {
+            // forge-lint: disable-next-line(unsafe-typecast)
             totalPaidToSplits += address(uint160(0x4000 + i)).balance;
         }
 

package/test/TestAccessToFunds.sol

@@ -1,8 +1,23 @@
 // SPDX-License-Identifier: MIT
 pragma solidity >=0.8.6;
 
-import /* {*} from */ "./helpers/TestBaseWorkflow.sol";
-import "@openzeppelin/contracts/utils/Address.sol";
+import {TestBaseWorkflow} from "./helpers/TestBaseWorkflow.sol";
+import {MockERC20} from "./mock/MockERC20.sol";
+import {JBTerminalStore} from "../src/JBTerminalStore.sol";
+import {IJBController} from "../src/interfaces/IJBController.sol";
+import {IJBMultiTerminal} from "../src/interfaces/IJBMultiTerminal.sol";
+import {IJBPrices} from "../src/interfaces/IJBPrices.sol";
+import {IJBRulesetApprovalHook} from "../src/interfaces/IJBRulesetApprovalHook.sol";
+import {IJBTokens} from "../src/interfaces/IJBTokens.sol";
+import {JBConstants} from "../src/libraries/JBConstants.sol";
+import {JBAccountingContext} from "../src/structs/JBAccountingContext.sol";
+import {JBCurrencyAmount} from "../src/structs/JBCurrencyAmount.sol";
+import {JBFundAccessLimitGroup} from "../src/structs/JBFundAccessLimitGroup.sol";
+import {JBRulesetConfig} from "../src/structs/JBRulesetConfig.sol";
+import {JBRulesetMetadata} from "../src/structs/JBRulesetMetadata.sol";
+import {JBSplitGroup} from "../src/structs/JBSplitGroup.sol";
+import {JBTerminalConfig} from "../src/structs/JBTerminalConfig.sol";
+import {mulDiv} from "@prb/math/src/Common.sol";
 import {MockPriceFeed} from "./mock/MockPriceFeed.sol";
 import {MaliciousAllowanceBeneficiary, MaliciousPayoutBeneficiary} from "./mock/MockMaliciousBeneficiary.sol";
 

package/test/TestCashOut.sol

@@ -1,8 +1,21 @@
 // SPDX-License-Identifier: MIT
 pragma solidity ^0.8.6;
 
-import /* {*} from */ "./helpers/TestBaseWorkflow.sol";
-import {JBCashOuts} from "../src/libraries/JBCashOuts.sol";
+import {TestBaseWorkflow} from "./helpers/TestBaseWorkflow.sol";
+import {JBTokens} from "../src/JBTokens.sol";
+import {IJBController} from "../src/interfaces/IJBController.sol";
+import {IJBMultiTerminal} from "../src/interfaces/IJBMultiTerminal.sol";
+import {IJBRulesetApprovalHook} from "../src/interfaces/IJBRulesetApprovalHook.sol";
+import {IJBTerminal} from "../src/interfaces/IJBTerminal.sol";
+import {JBConstants} from "../src/libraries/JBConstants.sol";
+import {JBAccountingContext} from "../src/structs/JBAccountingContext.sol";
+import {JBFundAccessLimitGroup} from "../src/structs/JBFundAccessLimitGroup.sol";
+import {JBRulesetConfig} from "../src/structs/JBRulesetConfig.sol";
+import {JBRulesetMetadata} from "../src/structs/JBRulesetMetadata.sol";
+import {JBSplitGroup} from "../src/structs/JBSplitGroup.sol";
+import {JBTerminalConfig} from "../src/structs/JBTerminalConfig.sol";
+import {mulDiv} from "@prb/math/src/Common.sol";
+import {mul as UD60x18mul, unwrap as UD60x18unwrap, wrap as UD60x18wrap} from "@prb/math/src/UD60x18.sol";
 
 // Projects can issue a token, be paid to receieve claimed tokens,  burn some of the claimed tokens, cash out the rest
 // of

package/test/TestCashOutCountFor.sol

@@ -1,8 +1,7 @@
 // SPDX-License-Identifier: MIT
 pragma solidity ^0.8.17;
 
-import "forge-std/Test.sol";
-import {mulDiv} from "@prb/math/src/Common.sol";
+import {Test} from "forge-std/Test.sol";
 import {JBCashOuts} from "../src/libraries/JBCashOuts.sol";
 import {JBConstants} from "../src/libraries/JBConstants.sol";