@bananapus/core-v6 0.0.15 → 0.0.17

package/test/regression/WeightCacheBoundary.t.sol

@@ -0,0 +1,291 @@
+// SPDX-License-Identifier: MIT
+pragma solidity >=0.8.6;
+
+import {TestBaseWorkflow} from "../helpers/TestBaseWorkflow.sol";
+import {IJBController} from "../../src/interfaces/IJBController.sol";
+import {IJBRulesets} from "../../src/interfaces/IJBRulesets.sol";
+import {IJBRulesetApprovalHook} from "../../src/interfaces/IJBRulesetApprovalHook.sol";
+import {JBConstants} from "../../src/libraries/JBConstants.sol";
+import {JBRulesetMetadata} from "../../src/structs/JBRulesetMetadata.sol";
+import {JBRulesetConfig} from "../../src/structs/JBRulesetConfig.sol";
+import {JBRuleset} from "../../src/structs/JBRuleset.sol";
+import {JBSplitGroup} from "../../src/structs/JBSplitGroup.sol";
+import {JBFundAccessLimitGroup} from "../../src/structs/JBFundAccessLimitGroup.sol";
+import {JBTerminalConfig} from "../../src/structs/JBTerminalConfig.sol";
+import {JBRulesets} from "../../src/JBRulesets.sol";
+
+import {mulDiv} from "@prb/math/src/Common.sol";
+
+/// @notice Deterministic (non-fuzz) tests for the JBRulesets weight cache at the 20,000 iteration boundary.
+/// The contract reverts with `JBRulesets_WeightCacheRequired` when the cycle gap exceeds the threshold
+/// and the cache has not been populated. These tests verify:
+///   1. Exactly 20,000 cycles work without cache.
+///   2. 20,001 cycles revert without cache.
+///   3. Progressive `updateRulesetWeightCache()` calls unblock the revert.
+///   4. The cached weight equals the iteratively computed weight.
+contract WeightCacheBoundary_Local is TestBaseWorkflow {
+    uint256 private constant _THRESHOLD = 20_000;
+
+    /// @dev 1-second duration makes each cycle = 1 second, so we can hit the boundary fast.
+    uint32 private constant _DURATION = 1;
+
+    /// @dev Tiny weight cut (0.1%) so the weight doesn't decay to zero over 20k cycles.
+    uint32 private constant _WEIGHT_CUT_PERCENT = 1_000_000; // 0.1% of MAX_WEIGHT_CUT_PERCENT (1e9)
+
+    uint112 private constant _INITIAL_WEIGHT = 1000e18;
+
+    IJBController private _controller;
+    IJBRulesets private _rulesets;
+    address private _projectOwner;
+
+    JBRulesetMetadata private _metadata;
+
+    function setUp() public override {
+        super.setUp();
+
+        _projectOwner = multisig();
+        _rulesets = jbRulesets();
+        _controller = jbController();
+
+        _metadata = JBRulesetMetadata({
+            reservedPercent: 0,
+            cashOutTaxRate: 0,
+            baseCurrency: uint32(uint160(JBConstants.NATIVE_TOKEN)),
+            pausePay: false,
+            pauseCreditTransfers: false,
+            allowOwnerMinting: false,
+            allowSetCustomToken: false,
+            allowTerminalMigration: false,
+            allowSetTerminals: false,
+            ownerMustSendPayouts: false,
+            allowSetController: false,
+            allowAddAccountingContext: true,
+            allowAddPriceFeed: false,
+            holdFees: false,
+            useTotalSurplusForCashOuts: true,
+            useDataHookForPay: false,
+            useDataHookForCashOut: false,
+            dataHook: address(0),
+            metadata: 0
+        });
+    }
+
+    /// @dev Helper: launch a project with a 1-second duration and the given weight cut percent.
+    function _launchProject() internal returns (uint256 projectId) {
+        JBRulesetConfig[] memory configs = new JBRulesetConfig[](1);
+        configs[0].mustStartAtOrAfter = 0;
+        configs[0].duration = _DURATION;
+        configs[0].weight = _INITIAL_WEIGHT;
+        configs[0].weightCutPercent = _WEIGHT_CUT_PERCENT;
+        configs[0].approvalHook = IJBRulesetApprovalHook(address(0));
+        configs[0].metadata = _metadata;
+        configs[0].splitGroups = new JBSplitGroup[](0);
+        configs[0].fundAccessLimitGroups = new JBFundAccessLimitGroup[](0);
+
+        projectId = _controller.launchProjectFor({
+            owner: _projectOwner,
+            projectUri: "weightCacheBoundary",
+            rulesetConfigurations: configs,
+            terminalConfigurations: new JBTerminalConfig[](0),
+            memo: ""
+        });
+    }
+
+    /// @dev Compute expected weight by iterating the cut percent `n` times on the initial weight.
+    function _expectedWeight(uint256 n) internal pure returns (uint256 weight) {
+        weight = _INITIAL_WEIGHT;
+        for (uint256 i; i < n; i++) {
+            weight = mulDiv(
+                weight, JBConstants.MAX_WEIGHT_CUT_PERCENT - _WEIGHT_CUT_PERCENT, JBConstants.MAX_WEIGHT_CUT_PERCENT
+            );
+            if (weight == 0) break;
+        }
+    }
+
+    // ═══════════════════════════════════════════════════════════════════
+    //  Test 1: Exactly 20,000 cycles works without cache
+    // ═══════════════════════════════════════════════════════════════════
+
+    function test_exactlyAtThreshold_noCache_succeeds() public {
+        uint256 projectId = _launchProject();
+
+        // Warp forward exactly 20,000 durations.
+        vm.warp(block.timestamp + _DURATION * _THRESHOLD);
+
+        // currentOf should succeed — still within the iteration limit.
+        JBRuleset memory ruleset = _rulesets.currentOf(projectId);
+
+        // Verify weight matches the iterative calculation.
+        uint256 expected = _expectedWeight(_THRESHOLD);
+        assertEq(ruleset.weight, expected, "Weight at exactly 20,000 cycles should match iterative calculation");
+        assertTrue(ruleset.weight > 0, "Weight should not have decayed to zero at 20k cycles with 0.1% cut");
+    }
+
+    // ═══════════════════════════════════════════════════════════════════
+    //  Test 2: 20,001 cycles reverts without cache
+    // ═══════════════════════════════════════════════════════════════════
+
+    function test_aboveThreshold_noCache_reverts() public {
+        uint256 projectId = _launchProject();
+
+        // Warp forward 20,001 durations.
+        vm.warp(block.timestamp + _DURATION * (_THRESHOLD + 1));
+
+        // currentOf should revert because the iteration count exceeds the threshold.
+        vm.expectRevert(abi.encodeWithSelector(JBRulesets.JBRulesets_WeightCacheRequired.selector, projectId));
+        _rulesets.currentOf(projectId);
+    }
+
+    // ═══════════════════════════════════════════════════════════════════
+    //  Test 3: Single cache call unblocks 20,001 cycles
+    // ═══════════════════════════════════════════════════════════════════
+
+    function test_aboveThreshold_singleCacheCall_succeeds() public {
+        uint256 projectId = _launchProject();
+        uint256 rulesetId = _rulesets.latestRulesetIdOf(projectId);
+
+        // Warp forward 20,001 durations.
+        vm.warp(block.timestamp + _DURATION * (_THRESHOLD + 1));
+
+        // Without cache, this reverts.
+        vm.expectRevert(abi.encodeWithSelector(JBRulesets.JBRulesets_WeightCacheRequired.selector, projectId));
+        _rulesets.currentOf(projectId);
+
+        // Populate the cache.
+        _rulesets.updateRulesetWeightCache(projectId, rulesetId);
+
+        // Now currentOf should succeed.
+        JBRuleset memory ruleset = _rulesets.currentOf(projectId);
+
+        // Verify weight matches iterative calculation.
+        uint256 expected = _expectedWeight(_THRESHOLD + 1);
+        assertEq(ruleset.weight, expected, "Weight at 20,001 cycles should match after single cache call");
+    }
+
+    // ═══════════════════════════════════════════════════════════════════
+    //  Test 4: Progressive caching for large gaps (40,001 cycles)
+    // ═══════════════════════════════════════════════════════════════════
+
+    function test_largeGap_progressiveCaching_succeeds() public {
+        uint256 projectId = _launchProject();
+        uint256 rulesetId = _rulesets.latestRulesetIdOf(projectId);
+
+        // Warp forward 40,001 durations (needs 2 cache calls).
+        vm.warp(block.timestamp + _DURATION * (_THRESHOLD * 2 + 1));
+
+        // Without cache, reverts.
+        vm.expectRevert(abi.encodeWithSelector(JBRulesets.JBRulesets_WeightCacheRequired.selector, projectId));
+        _rulesets.currentOf(projectId);
+
+        // First cache call: advances cache by up to 20,000 iterations.
+        _rulesets.updateRulesetWeightCache(projectId, rulesetId);
+
+        // Still reverts — we need another cache call to cover the remaining gap.
+        vm.expectRevert(abi.encodeWithSelector(JBRulesets.JBRulesets_WeightCacheRequired.selector, projectId));
+        _rulesets.currentOf(projectId);
+
+        // Second cache call: covers the rest.
+        _rulesets.updateRulesetWeightCache(projectId, rulesetId);
+
+        // Now it should succeed.
+        JBRuleset memory ruleset = _rulesets.currentOf(projectId);
+
+        // Verify weight matches iterative calculation.
+        uint256 expected = _expectedWeight(_THRESHOLD * 2 + 1);
+        assertEq(ruleset.weight, expected, "Weight at 40,001 cycles should match after two cache calls");
+    }
+
+    // ═══════════════════════════════════════════════════════════════════
+    //  Test 5: Cache produces same weight as direct iteration
+    // ═══════════════════════════════════════════════════════════════════
+
+    function test_cachedWeight_matchesDirect_atBoundary() public {
+        // Launch two identical projects.
+        uint256 projectA = _launchProject();
+        uint256 projectB = _launchProject();
+
+        uint256 rulesetIdB = _rulesets.latestRulesetIdOf(projectB);
+
+        // Warp to exactly 20,000 cycles (within threshold for both).
+        vm.warp(block.timestamp + _DURATION * _THRESHOLD);
+
+        // Project A: no cache, direct iteration.
+        JBRuleset memory rulesetA = _rulesets.currentOf(projectA);
+
+        // Project B: populate cache first, then query.
+        _rulesets.updateRulesetWeightCache(projectB, rulesetIdB);
+        JBRuleset memory rulesetB = _rulesets.currentOf(projectB);
+
+        // Both should produce identical weights.
+        assertEq(rulesetA.weight, rulesetB.weight, "Cached and uncached weights should be identical at threshold");
+    }
+
+    // ═══════════════════════════════════════════════════════════════════
+    //  Test 6: Zero weightCutPercent — cache is no-op
+    // ═══════════════════════════════════════════════════════════════════
+
+    function test_zeroWeightCut_cacheNoOp() public {
+        // Launch a project with zero weight cut.
+        JBRulesetConfig[] memory configs = new JBRulesetConfig[](1);
+        configs[0].mustStartAtOrAfter = 0;
+        configs[0].duration = _DURATION;
+        configs[0].weight = _INITIAL_WEIGHT;
+        configs[0].weightCutPercent = 0; // No decay
+        configs[0].approvalHook = IJBRulesetApprovalHook(address(0));
+        configs[0].metadata = _metadata;
+        configs[0].splitGroups = new JBSplitGroup[](0);
+        configs[0].fundAccessLimitGroups = new JBFundAccessLimitGroup[](0);
+
+        uint256 projectId = _controller.launchProjectFor({
+            owner: _projectOwner,
+            projectUri: "zeroCut",
+            rulesetConfigurations: configs,
+            terminalConfigurations: new JBTerminalConfig[](0),
+            memo: ""
+        });
+
+        // Warp far beyond the threshold — since weightCutPercent = 0, no iterations needed.
+        vm.warp(block.timestamp + _DURATION * (_THRESHOLD * 5));
+
+        // Should succeed without any cache because the weight cut loop is skipped.
+        JBRuleset memory ruleset = _rulesets.currentOf(projectId);
+        assertEq(ruleset.weight, _INITIAL_WEIGHT, "Weight should remain unchanged with zero cut percent");
+    }
+
+    // ═══════════════════════════════════════════════════════════════════
+    //  Test 7: Cache with zero duration — updateRulesetWeightCache is no-op
+    // ═══════════════════════════════════════════════════════════════════
+
+    function test_zeroDuration_cacheNoOp() public {
+        // Launch a project with zero duration.
+        JBRulesetConfig[] memory configs = new JBRulesetConfig[](1);
+        configs[0].mustStartAtOrAfter = 0;
+        configs[0].duration = 0; // Never expires
+        configs[0].weight = _INITIAL_WEIGHT;
+        configs[0].weightCutPercent = _WEIGHT_CUT_PERCENT;
+        configs[0].approvalHook = IJBRulesetApprovalHook(address(0));
+        configs[0].metadata = _metadata;
+        configs[0].splitGroups = new JBSplitGroup[](0);
+        configs[0].fundAccessLimitGroups = new JBFundAccessLimitGroup[](0);
+
+        uint256 projectId = _controller.launchProjectFor({
+            owner: _projectOwner,
+            projectUri: "zeroDuration",
+            rulesetConfigurations: configs,
+            terminalConfigurations: new JBTerminalConfig[](0),
+            memo: ""
+        });
+
+        uint256 rulesetId = _rulesets.latestRulesetIdOf(projectId);
+
+        // Warp far ahead.
+        vm.warp(block.timestamp + 1_000_000);
+
+        // updateRulesetWeightCache should be a no-op (returns without caching).
+        _rulesets.updateRulesetWeightCache(projectId, rulesetId);
+
+        // currentOf should succeed — zero duration means the same ruleset is always current.
+        JBRuleset memory ruleset = _rulesets.currentOf(projectId);
+        assertEq(ruleset.weight, _INITIAL_WEIGHT, "Weight should remain unchanged with zero duration");
+    }
+}

package/test/units/static/JBChainlinkV3PriceFeed/TestPriceFeed.sol

@@ -1,7 +1,6 @@
 // SPDX-License-Identifier: MIT
 pragma solidity 0.8.26;
 
-import /* {*} from */ "../../../helpers/TestBaseWorkflow.sol";
 import {JBTest} from "../../../helpers/JBTest.sol";
 import {JBChainlinkV3PriceFeed} from "../../../../src/JBChainlinkV3PriceFeed.sol";
 import {AggregatorV3Interface} from "@chainlink/contracts/src/v0.8/shared/interfaces/AggregatorV3Interface.sol";

package/test/units/static/JBController/JBControllerSetup.sol

@@ -1,7 +1,16 @@
 // SPDX-License-Identifier: MIT
 pragma solidity 0.8.26;
 
-import /* {*} from */ "../../../helpers/TestBaseWorkflow.sol";
+import {JBController} from "../../../../src/JBController.sol";
+import {IJBController} from "../../../../src/interfaces/IJBController.sol";
+import {IJBDirectory} from "../../../../src/interfaces/IJBDirectory.sol";
+import {IJBFundAccessLimits} from "../../../../src/interfaces/IJBFundAccessLimits.sol";
+import {IJBPermissions} from "../../../../src/interfaces/IJBPermissions.sol";
+import {IJBPrices} from "../../../../src/interfaces/IJBPrices.sol";
+import {IJBProjects} from "../../../../src/interfaces/IJBProjects.sol";
+import {IJBRulesets} from "../../../../src/interfaces/IJBRulesets.sol";
+import {IJBSplits} from "../../../../src/interfaces/IJBSplits.sol";
+import {IJBTokens} from "../../../../src/interfaces/IJBTokens.sol";
 import {JBTest} from "../../../helpers/JBTest.sol";
 
 /*

package/test/units/static/JBController/TestBurnTokensOf.sol

@@ -1,7 +1,14 @@
 // SPDX-License-Identifier: MIT
 pragma solidity 0.8.26;
 
-import /* {*} from */ "../../../helpers/TestBaseWorkflow.sol";
+import {JBController} from "../../../../src/JBController.sol";
+import {JBPermissioned} from "../../../../src/abstract/JBPermissioned.sol";
+import {IJBController} from "../../../../src/interfaces/IJBController.sol";
+import {IJBDirectory} from "../../../../src/interfaces/IJBDirectory.sol";
+import {IJBPermissions} from "../../../../src/interfaces/IJBPermissions.sol";
+import {IJBTerminal} from "../../../../src/interfaces/IJBTerminal.sol";
+import {IJBTokens} from "../../../../src/interfaces/IJBTokens.sol";
+import {JBPermissionIds} from "@bananapus/permission-ids-v6/src/JBPermissionIds.sol";
 import {JBControllerSetup} from "./JBControllerSetup.sol";
 
 contract TestBurnTokensOf_Local is JBControllerSetup {

package/test/units/static/JBController/TestClaimTokensFor.sol

@@ -1,7 +1,10 @@
 // SPDX-License-Identifier: MIT
 pragma solidity 0.8.26;
 
-import /* {*} from */ "../../../helpers/TestBaseWorkflow.sol";
+import {JBPermissioned} from "../../../../src/abstract/JBPermissioned.sol";
+import {IJBPermissions} from "../../../../src/interfaces/IJBPermissions.sol";
+import {IJBTokens} from "../../../../src/interfaces/IJBTokens.sol";
+import {JBPermissionIds} from "@bananapus/permission-ids-v6/src/JBPermissionIds.sol";
 import {JBControllerSetup} from "./JBControllerSetup.sol";
 
 contract TestClaimTokensFor_Local is JBControllerSetup {

package/test/units/static/JBController/TestDeployErc20For.sol

@@ -1,7 +1,12 @@
 // SPDX-License-Identifier: MIT
 pragma solidity 0.8.26;
 
-import /* {*} from */ "../../../helpers/TestBaseWorkflow.sol";
+import {JBPermissioned} from "../../../../src/abstract/JBPermissioned.sol";
+import {IJBPermissions} from "../../../../src/interfaces/IJBPermissions.sol";
+import {IJBToken} from "../../../../src/interfaces/IJBToken.sol";
+import {IJBTokens} from "../../../../src/interfaces/IJBTokens.sol";
+import {JBPermissionIds} from "@bananapus/permission-ids-v6/src/JBPermissionIds.sol";
+import {IERC721} from "@openzeppelin/contracts/token/ERC721/IERC721.sol";
 import {JBControllerSetup} from "./JBControllerSetup.sol";
 
 contract TestDeployERC20For_Local is JBControllerSetup {
@@ -34,6 +39,7 @@ contract TestDeployERC20For_Local is JBControllerSetup {
     function test_WhenCallerIsPermissionedAndSaltIsNonZero() external {
         // it will deploy ERC20 and return IJBToken
 
+        // forge-lint: disable-next-line(unsafe-typecast)
         bytes32 _nonZeroSalt = bytes32("somesalt");
         bytes32 _encodedSalt = keccak256(abi.encodePacked(address(this), _nonZeroSalt));
 

package/test/units/static/JBController/TestLaunchProjectFor.sol

@@ -1,7 +1,27 @@
 // SPDX-License-Identifier: MIT
 pragma solidity 0.8.26;
 
-import /* {*} from */ "../../../helpers/TestBaseWorkflow.sol";
+import {JBController} from "../../../../src/JBController.sol";
+import {IJBController} from "../../../../src/interfaces/IJBController.sol";
+import {IJBDirectory} from "../../../../src/interfaces/IJBDirectory.sol";
+import {IJBFundAccessLimits} from "../../../../src/interfaces/IJBFundAccessLimits.sol";
+import {IJBProjects} from "../../../../src/interfaces/IJBProjects.sol";
+import {IJBRulesetApprovalHook} from "../../../../src/interfaces/IJBRulesetApprovalHook.sol";
+import {IJBRulesets} from "../../../../src/interfaces/IJBRulesets.sol";
+import {IJBSplitHook} from "../../../../src/interfaces/IJBSplitHook.sol";
+import {IJBSplits} from "../../../../src/interfaces/IJBSplits.sol";
+import {IJBTerminal} from "../../../../src/interfaces/IJBTerminal.sol";
+import {JBConstants} from "../../../../src/libraries/JBConstants.sol";
+import {JBRulesetMetadataResolver} from "../../../../src/libraries/JBRulesetMetadataResolver.sol";
+import {JBAccountingContext} from "../../../../src/structs/JBAccountingContext.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 {IERC165} from "@openzeppelin/contracts/utils/introspection/ERC165.sol";
 import {JBControllerSetup} from "./JBControllerSetup.sol";
 
 contract TestLaunchProjectFor_Local is JBControllerSetup {

package/test/units/static/JBController/TestLaunchRulesetsFor.sol

@@ -1,7 +1,27 @@
 // SPDX-License-Identifier: MIT
 pragma solidity 0.8.26;
 
-import /* {*} from */ "../../../helpers/TestBaseWorkflow.sol";
+import {JBController} from "../../../../src/JBController.sol";
+import {JBPermissioned} from "../../../../src/abstract/JBPermissioned.sol";
+import {IJBController} from "../../../../src/interfaces/IJBController.sol";
+import {IJBDirectory} from "../../../../src/interfaces/IJBDirectory.sol";
+import {IJBFundAccessLimits} from "../../../../src/interfaces/IJBFundAccessLimits.sol";
+import {IJBPermissions} from "../../../../src/interfaces/IJBPermissions.sol";
+import {IJBRulesetApprovalHook} from "../../../../src/interfaces/IJBRulesetApprovalHook.sol";
+import {IJBRulesets} from "../../../../src/interfaces/IJBRulesets.sol";
+import {IJBSplits} from "../../../../src/interfaces/IJBSplits.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 {JBPermissionIds} from "@bananapus/permission-ids-v6/src/JBPermissionIds.sol";
+import {IERC721} from "@openzeppelin/contracts/token/ERC721/IERC721.sol";
+import {IERC165} from "@openzeppelin/contracts/utils/introspection/ERC165.sol";
 import {JBControllerSetup} from "./JBControllerSetup.sol";
 
 contract TestLaunchRulesetsFor_Local is JBControllerSetup {

package/test/units/static/JBController/TestMigrateController.sol

@@ -1,7 +1,16 @@
 // SPDX-License-Identifier: MIT
 pragma solidity 0.8.26;
 
-import /* {*} from */ "../../../helpers/TestBaseWorkflow.sol";
+import {StdStorage, stdStorage} from "forge-std/StdStorage.sol";
+import {JBController} from "../../../../src/JBController.sol";
+import {IJBMigratable} from "../../../../src/interfaces/IJBMigratable.sol";
+import {IJBRulesetApprovalHook} from "../../../../src/interfaces/IJBRulesetApprovalHook.sol";
+import {IJBRulesets} from "../../../../src/interfaces/IJBRulesets.sol";
+import {JBConstants} from "../../../../src/libraries/JBConstants.sol";
+import {JBRulesetMetadataResolver} from "../../../../src/libraries/JBRulesetMetadataResolver.sol";
+import {JBRuleset} from "../../../../src/structs/JBRuleset.sol";
+import {JBRulesetMetadata} from "../../../../src/structs/JBRulesetMetadata.sol";
+import {IERC721} from "@openzeppelin/contracts/token/ERC721/IERC721.sol";
 import {JBControllerSetup} from "./JBControllerSetup.sol";
 
 contract TestMigrateController_Local is JBControllerSetup {

package/test/units/static/JBController/TestMintTokensOfUnits.sol

@@ -1,7 +1,16 @@
 // SPDX-License-Identifier: MIT
 pragma solidity 0.8.26;
 
-import /* {*} from */ "../../../helpers/TestBaseWorkflow.sol";
+import {JBController} from "../../../../src/JBController.sol";
+import {IJBDirectory} from "../../../../src/interfaces/IJBDirectory.sol";
+import {IJBRulesetApprovalHook} from "../../../../src/interfaces/IJBRulesetApprovalHook.sol";
+import {IJBRulesets} from "../../../../src/interfaces/IJBRulesets.sol";
+import {IJBTerminal} from "../../../../src/interfaces/IJBTerminal.sol";
+import {JBConstants} from "../../../../src/libraries/JBConstants.sol";
+import {JBRulesetMetadataResolver} from "../../../../src/libraries/JBRulesetMetadataResolver.sol";
+import {JBRuleset} from "../../../../src/structs/JBRuleset.sol";
+import {JBRulesetMetadata} from "../../../../src/structs/JBRulesetMetadata.sol";
+import {IERC721} from "@openzeppelin/contracts/token/ERC721/IERC721.sol";
 import {JBControllerSetup} from "./JBControllerSetup.sol";
 
 contract TestMintTokensOfUnits_Local is JBControllerSetup {

package/test/units/static/JBController/TestPayReservedTokenToTerminal.sol

@@ -1,7 +1,10 @@
 // SPDX-License-Identifier: MIT
 pragma solidity 0.8.26;
 
-import /* {*} from */ "../../../helpers/TestBaseWorkflow.sol";
+import {JBController} from "../../../../src/JBController.sol";
+import {IJBTerminal} from "../../../../src/interfaces/IJBTerminal.sol";
+import {IJBToken} from "../../../../src/interfaces/IJBToken.sol";
+import {IERC20} from "@openzeppelin/contracts/token/ERC20/IERC20.sol";
 import {JBControllerSetup} from "./JBControllerSetup.sol";
 
 contract TestExecutePayReservedTokenToTerminal_Local is JBControllerSetup {

package/test/units/static/JBController/TestReceiveMigrationFrom.sol

@@ -1,7 +1,11 @@
 // SPDX-License-Identifier: MIT
 pragma solidity 0.8.26;
 
-import /* {*} from */ "../../../helpers/TestBaseWorkflow.sol";
+import {JBController} from "../../../../src/JBController.sol";
+import {IJBController} from "../../../../src/interfaces/IJBController.sol";
+import {IJBMigratable} from "../../../../src/interfaces/IJBMigratable.sol";
+import {IJBProjectUriRegistry} from "../../../../src/interfaces/IJBProjectUriRegistry.sol";
+import {IERC165} from "@openzeppelin/contracts/utils/introspection/ERC165.sol";
 import {JBControllerSetup} from "./JBControllerSetup.sol";
 
 contract TestReceiveMigrationFrom_Local is JBControllerSetup {

package/test/units/static/JBController/TestRulesetViews.sol

@@ -1,7 +1,13 @@
 // SPDX-License-Identifier: MIT
 pragma solidity 0.8.26;
 
-import /* {*} from */ "../../../helpers/TestBaseWorkflow.sol";
+import {JBApprovalStatus} from "../../../../src/enums/JBApprovalStatus.sol";
+import {IJBRulesetApprovalHook} from "../../../../src/interfaces/IJBRulesetApprovalHook.sol";
+import {IJBRulesets} from "../../../../src/interfaces/IJBRulesets.sol";
+import {JBRulesetMetadataResolver} from "../../../../src/libraries/JBRulesetMetadataResolver.sol";
+import {JBRuleset} from "../../../../src/structs/JBRuleset.sol";
+import {JBRulesetMetadata} from "../../../../src/structs/JBRulesetMetadata.sol";
+import {JBRulesetWithMetadata} from "../../../../src/structs/JBRulesetWithMetadata.sol";
 import {JBControllerSetup} from "./JBControllerSetup.sol";
 
 /**

package/test/units/static/JBController/TestSendReservedTokensToSplitsOf.sol

@@ -1,7 +1,27 @@
 // SPDX-License-Identifier: MIT
 pragma solidity 0.8.26;
 
-import /* {*} from */ "../../../helpers/TestBaseWorkflow.sol";
+import {StdStorage, stdStorage} from "forge-std/StdStorage.sol";
+import {JBController} from "../../../../src/JBController.sol";
+import {IJBController} from "../../../../src/interfaces/IJBController.sol";
+import {IJBDirectory} from "../../../../src/interfaces/IJBDirectory.sol";
+import {IJBRulesetApprovalHook} from "../../../../src/interfaces/IJBRulesetApprovalHook.sol";
+import {IJBRulesets} from "../../../../src/interfaces/IJBRulesets.sol";
+import {IJBSplitHook} from "../../../../src/interfaces/IJBSplitHook.sol";
+import {IJBSplits} from "../../../../src/interfaces/IJBSplits.sol";
+import {IJBToken} from "../../../../src/interfaces/IJBToken.sol";
+import {IJBTokens} from "../../../../src/interfaces/IJBTokens.sol";
+import {JBConstants} from "../../../../src/libraries/JBConstants.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 {JBSplit} from "../../../../src/structs/JBSplit.sol";
+import {JBSplitGroup} from "../../../../src/structs/JBSplitGroup.sol";
+import {JBSplitHookContext} from "../../../../src/structs/JBSplitHookContext.sol";
+import {IERC20} from "@openzeppelin/contracts/token/ERC20/IERC20.sol";
+import {IERC721} from "@openzeppelin/contracts/token/ERC721/IERC721.sol";
 import {JBControllerSetup} from "./JBControllerSetup.sol";
 
 contract TestSendReservedTokensToSplitsOf_Local is JBControllerSetup {

package/test/units/static/JBController/TestSetSplitGroupsOf.sol

@@ -1,7 +1,12 @@
 // SPDX-License-Identifier: MIT
 pragma solidity 0.8.26;
 
-import /* {*} from */ "../../../helpers/TestBaseWorkflow.sol";
+import {JBPermissioned} from "../../../../src/abstract/JBPermissioned.sol";
+import {IJBPermissions} from "../../../../src/interfaces/IJBPermissions.sol";
+import {IJBSplits} from "../../../../src/interfaces/IJBSplits.sol";
+import {JBSplitGroup} from "../../../../src/structs/JBSplitGroup.sol";
+import {JBPermissionIds} from "@bananapus/permission-ids-v6/src/JBPermissionIds.sol";
+import {IERC721} from "@openzeppelin/contracts/token/ERC721/IERC721.sol";
 import {JBControllerSetup} from "./JBControllerSetup.sol";
 
 contract TestSetSplitGroupsOf_Local is JBControllerSetup {

package/test/units/static/JBController/TestSetTokenFor.sol

@@ -1,7 +1,19 @@
 // SPDX-License-Identifier: MIT
 pragma solidity 0.8.26;
 
-import /* {*} from */ "../../../helpers/TestBaseWorkflow.sol";
+import {JBController} from "../../../../src/JBController.sol";
+import {JBPermissioned} from "../../../../src/abstract/JBPermissioned.sol";
+import {IJBPermissions} from "../../../../src/interfaces/IJBPermissions.sol";
+import {IJBRulesetApprovalHook} from "../../../../src/interfaces/IJBRulesetApprovalHook.sol";
+import {IJBRulesets} from "../../../../src/interfaces/IJBRulesets.sol";
+import {IJBToken} from "../../../../src/interfaces/IJBToken.sol";
+import {IJBTokens} from "../../../../src/interfaces/IJBTokens.sol";
+import {JBConstants} from "../../../../src/libraries/JBConstants.sol";
+import {JBRulesetMetadataResolver} from "../../../../src/libraries/JBRulesetMetadataResolver.sol";
+import {JBRuleset} from "../../../../src/structs/JBRuleset.sol";
+import {JBRulesetMetadata} from "../../../../src/structs/JBRulesetMetadata.sol";
+import {JBPermissionIds} from "@bananapus/permission-ids-v6/src/JBPermissionIds.sol";
+import {IERC721} from "@openzeppelin/contracts/token/ERC721/IERC721.sol";
 import {JBControllerSetup} from "./JBControllerSetup.sol";
 
 contract TestSetTokenFor_Local is JBControllerSetup {

package/test/units/static/JBController/TestSetUriOf.sol

@@ -1,7 +1,11 @@
 // SPDX-License-Identifier: MIT
 pragma solidity 0.8.26;
 
-import /* {*} from */ "../../../helpers/TestBaseWorkflow.sol";
+import {JBPermissioned} from "../../../../src/abstract/JBPermissioned.sol";
+import {IJBController} from "../../../../src/interfaces/IJBController.sol";
+import {IJBPermissions} from "../../../../src/interfaces/IJBPermissions.sol";
+import {JBPermissionIds} from "@bananapus/permission-ids-v6/src/JBPermissionIds.sol";
+import {IERC721} from "@openzeppelin/contracts/token/ERC721/IERC721.sol";
 import {JBControllerSetup} from "./JBControllerSetup.sol";
 
 contract TestSetUriOf_Local is JBControllerSetup {

package/test/units/static/JBController/TestTransferCreditsFrom.sol

@@ -1,7 +1,17 @@
 // SPDX-License-Identifier: MIT
 pragma solidity 0.8.26;
 
-import /* {*} from */ "../../../helpers/TestBaseWorkflow.sol";
+import {JBController} from "../../../../src/JBController.sol";
+import {JBPermissioned} from "../../../../src/abstract/JBPermissioned.sol";
+import {IJBPermissions} from "../../../../src/interfaces/IJBPermissions.sol";
+import {IJBRulesetApprovalHook} from "../../../../src/interfaces/IJBRulesetApprovalHook.sol";
+import {IJBRulesets} from "../../../../src/interfaces/IJBRulesets.sol";
+import {IJBTokens} from "../../../../src/interfaces/IJBTokens.sol";
+import {JBConstants} from "../../../../src/libraries/JBConstants.sol";
+import {JBRulesetMetadataResolver} from "../../../../src/libraries/JBRulesetMetadataResolver.sol";
+import {JBRuleset} from "../../../../src/structs/JBRuleset.sol";
+import {JBRulesetMetadata} from "../../../../src/structs/JBRulesetMetadata.sol";
+import {JBPermissionIds} from "@bananapus/permission-ids-v6/src/JBPermissionIds.sol";
 import {JBControllerSetup} from "./JBControllerSetup.sol";
 
 contract TestTransferCreditsFrom_Local is JBControllerSetup {