@bananapus/core-v6 0.0.15 → 0.0.16

package/src/structs/JBRulesetMetadata.sol

@@ -33,6 +33,7 @@ pragma solidity ^0.8.0;
 /// @custom:member dataHook The data hook to use during this ruleset.
 /// @custom:member metadata Metadata of the metadata, only the 14 least significant bits can be used, the 2 most
 /// significant bits are disregarded.
+// forge-lint: disable-next-line(pascal-case-struct)
 struct JBRulesetMetadata {
     uint16 reservedPercent;
     uint16 cashOutTaxRate;

package/src/structs/JBRulesetWeightCache.sol

@@ -3,6 +3,7 @@ pragma solidity ^0.8.0;
 
 /// @custom:member weight The cached weight value.
 /// @custom:member weightCutMultiple The weight cut multiple that produces the given weight.
+// forge-lint: disable-next-line(pascal-case-struct)
 struct JBRulesetWeightCache {
     uint112 weight;
     uint168 weightCutMultiple;

package/src/structs/JBRulesetWithMetadata.sol

@@ -6,6 +6,7 @@ import {JBRulesetMetadata} from "./JBRulesetMetadata.sol";
 
 /// @custom:member ruleset The ruleset.
 /// @custom:member metadata The ruleset's metadata.
+// forge-lint: disable-next-line(pascal-case-struct)
 struct JBRulesetWithMetadata {
     JBRuleset ruleset;
     JBRulesetMetadata metadata;

package/src/structs/JBSingleAllowance.sol

@@ -7,6 +7,7 @@ pragma solidity ^0.8.0;
 /// @custom:member nonce An incrementing value indexed per owner,token,and spender for each signature.
 /// @custom:member signature The signature over the permit data. Supports EOA signatures, compact signatures defined by
 /// EIP-2098, and contract signatures defined by EIP-1271.
+// forge-lint: disable-next-line(pascal-case-struct)
 struct JBSingleAllowance {
     uint256 sigDeadline;
     uint160 amount;

package/src/structs/JBSplit.sol

@@ -27,6 +27,7 @@ import {IJBSplitHook} from "./../interfaces/IJBSplitHook.sol";
 /// increased while a split is locked. If `lockedUntil` is zero, this split can be changed at any time.
 /// @custom:member hook A contract which will receive this split's tokens and properties, and can define custom
 /// behavior.
+// forge-lint: disable-next-line(pascal-case-struct)
 struct JBSplit {
     uint32 percent;
     uint64 projectId;

package/src/structs/JBSplitGroup.sol

@@ -6,6 +6,7 @@ import {JBSplit} from "./JBSplit.sol";
 /// @custom:member groupId An identifier for the group. By convention, this ID is `uint256(uint160(tokenAddress))` for
 /// payouts and `1` for reserved tokens.
 /// @custom:member splits The splits in the group.
+// forge-lint: disable-next-line(pascal-case-struct)
 struct JBSplitGroup {
     uint256 groupId;
     JBSplit[] splits;

package/src/structs/JBSplitHookContext.sol

@@ -10,6 +10,7 @@ import {JBSplit} from "./JBSplit.sol";
 /// @custom:member groupId The group the split belongs to. By convention, this ID is `uint256(uint160(tokenAddress))`
 /// for payouts and `1` for reserved tokens.
 /// @custom:member split The split which specified the hook.
+// forge-lint: disable-next-line(pascal-case-struct)
 struct JBSplitHookContext {
     address token;
     uint256 amount;

package/src/structs/JBTerminalConfig.sol

@@ -6,6 +6,7 @@ import {IJBTerminal} from "./../interfaces/IJBTerminal.sol";
 
 /// @custom:member terminal The terminal to configure.
 /// @custom:member accountingContextsToAccept The accounting contexts to accept from the terminal.
+// forge-lint: disable-next-line(pascal-case-struct)
 struct JBTerminalConfig {
     IJBTerminal terminal;
     JBAccountingContext[] accountingContextsToAccept;

package/src/structs/JBTokenAmount.sol

@@ -6,6 +6,7 @@ pragma solidity ^0.8.0;
 /// @custom:member currency The currency. By convention, this is `uint32(uint160(tokenAddress))` for tokens, or a
 /// constant ID from e.g. `JBCurrencyIds` for other currencies.
 /// @custom:member value The amount of tokens that was paid, as a fixed point number.
+// forge-lint: disable-next-line(pascal-case-struct)
 struct JBTokenAmount {
     address token;
     uint8 decimals;

package/test/ComprehensiveInvariant.t.sol

@@ -1,8 +1,21 @@
 // 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 {IJBSplitHook} from "../src/interfaces/IJBSplitHook.sol";
+import {IJBToken} from "../src/interfaces/IJBToken.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 {JBTerminalConfig} from "../src/structs/JBTerminalConfig.sol";
 import {ComprehensiveHandler} from "./invariants/handlers/ComprehensiveHandler.sol";
 import {JBAccountingContext} from "../src/structs/JBAccountingContext.sol";
 

package/test/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.
@@ -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.
@@ -2050,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();
 
@@ -2119,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
@@ -2632,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;
 
@@ -2688,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,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 {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";
 
@@ -592,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);
@@ -601,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.
@@ -548,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
@@ -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

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,
@@ -476,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,7 +1,21 @@
 // SPDX-License-Identifier: MIT
 pragma solidity ^0.8.6;
 
-import /* {*} from */ "./helpers/TestBaseWorkflow.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,7 +1,7 @@
 // SPDX-License-Identifier: MIT
 pragma solidity ^0.8.17;
 
-import "forge-std/Test.sol";
+import {Test} from "forge-std/Test.sol";
 import {JBCashOuts} from "../src/libraries/JBCashOuts.sol";
 import {JBConstants} from "../src/libraries/JBConstants.sol";