@bananapus/721-hook-v6 0.0.1 → 0.0.2

package/test/unit/tierSplitRouting_Unit.t.sol

@@ -0,0 +1,275 @@
+// SPDX-License-Identifier: MIT
+pragma solidity 0.8.23;
+
+import "../utils/UnitTestSetup.sol";
+import {IJB721TiersHookStore} from "../../src/interfaces/IJB721TiersHookStore.sol";
+import {JBSplit} from "@bananapus/core-v6/src/structs/JBSplit.sol";
+import {IJBSplitHook} from "@bananapus/core-v6/src/interfaces/IJBSplitHook.sol";
+import {IJBSplits} from "@bananapus/core-v6/src/interfaces/IJBSplits.sol";
+import {IJBController} from "@bananapus/core-v6/src/interfaces/IJBController.sol";
+
+contract Test_TierSplitRouting is UnitTestSetup {
+    using stdStorage for StdStorage;
+
+    address alice = makeAddr("alice");
+    address bob = makeAddr("bob");
+    address mockSplits = makeAddr("mockSplits");
+
+    function setUp() public override {
+        super.setUp();
+        vm.etch(mockSplits, new bytes(0x69));
+    }
+
+    // Helper: build a tier config with splits.
+    function _tierConfigWithSplit(
+        uint104 price,
+        uint32 splitPercent
+    )
+        internal
+        pure
+        returns (JB721TierConfig memory config)
+    {
+        config.price = price;
+        config.initialSupply = uint32(100);
+        config.category = uint24(1);
+        config.encodedIPFSUri = bytes32(uint256(0x1234));
+        config.splitPercent = splitPercent;
+    }
+
+    // Helper: build payer metadata for tier IDs.
+    function _buildPayerMetadata(
+        address hookAddress,
+        uint16[] memory tierIdsToMint
+    )
+        internal
+        view
+        returns (bytes memory)
+    {
+        bytes[] memory data = new bytes[](1);
+        data[0] = abi.encode(false, tierIdsToMint);
+        bytes4[] memory ids = new bytes4[](1);
+        ids[0] = metadataHelper.getId("pay", hookAddress);
+        return metadataHelper.createMetadata(ids, data);
+    }
+
+    // ──────────────────────────────────────────────
+    // Test: beforePayRecordedWith calculates split amount
+    // ──────────────────────────────────────────────
+
+    function test_beforePayRecorded_calculatesSplitAmount() public {
+        // Create hook with a default tier (no splits).
+        ForTest_JB721TiersHook testHook = _initializeForTestHook(0);
+        IJB721TiersHookStore hookStore = testHook.STORE();
+
+        // Add a tier with 50% split directly to the hook's store.
+        JB721TierConfig[] memory tierConfigs = new JB721TierConfig[](1);
+        tierConfigs[0] = _tierConfigWithSplit(1 ether, 500_000_000); // 50%
+        vm.prank(address(testHook));
+        uint256[] memory tierIds = hookStore.recordAddTiers(tierConfigs);
+
+        // Build payer metadata requesting that tier.
+        uint16[] memory mintIds = new uint16[](1);
+        mintIds[0] = uint16(tierIds[0]);
+        bytes memory payerMetadata = _buildPayerMetadata(address(testHook), mintIds);
+
+        JBBeforePayRecordedContext memory context = JBBeforePayRecordedContext({
+            terminal: mockTerminalAddress,
+            payer: beneficiary,
+            amount: JBTokenAmount({
+                token: JBConstants.NATIVE_TOKEN,
+                value: 1 ether,
+                decimals: 18,
+                currency: uint32(uint160(JBConstants.NATIVE_TOKEN))
+            }),
+            projectId: projectId,
+            rulesetId: 0,
+            beneficiary: beneficiary,
+            weight: 10e18,
+            reservedPercent: 5000,
+            metadata: payerMetadata
+        });
+
+        (uint256 weight, JBPayHookSpecification[] memory specs) = testHook.beforePayRecordedWith(context);
+
+        // Weight unchanged.
+        assertEq(weight, 10e18);
+        // Hook spec should forward 50% of 1 ETH = 0.5 ETH.
+        assertEq(specs.length, 1);
+        assertEq(specs[0].amount, 0.5 ether);
+    }
+
+    // ──────────────────────────────────────────────
+    // Test: no splitPercent means no forwarded amount
+    // ──────────────────────────────────────────────
+
+    function test_beforePayRecorded_noSplitPercent_noForwardedAmount() public {
+        ForTest_JB721TiersHook testHook = _initializeForTestHook(0);
+        IJB721TiersHookStore hookStore = testHook.STORE();
+
+        // Add a tier with 0% split.
+        JB721TierConfig[] memory tierConfigs = new JB721TierConfig[](1);
+        tierConfigs[0] = _tierConfigWithSplit(1 ether, 0);
+        vm.prank(address(testHook));
+        uint256[] memory tierIds = hookStore.recordAddTiers(tierConfigs);
+
+        uint16[] memory mintIds = new uint16[](1);
+        mintIds[0] = uint16(tierIds[0]);
+        bytes memory payerMetadata = _buildPayerMetadata(address(testHook), mintIds);
+
+        JBBeforePayRecordedContext memory context = JBBeforePayRecordedContext({
+            terminal: mockTerminalAddress,
+            payer: beneficiary,
+            amount: JBTokenAmount({
+                token: JBConstants.NATIVE_TOKEN,
+                value: 1 ether,
+                decimals: 18,
+                currency: uint32(uint160(JBConstants.NATIVE_TOKEN))
+            }),
+            projectId: projectId,
+            rulesetId: 0,
+            beneficiary: beneficiary,
+            weight: 10e18,
+            reservedPercent: 5000,
+            metadata: payerMetadata
+        });
+
+        (, JBPayHookSpecification[] memory specs) = testHook.beforePayRecordedWith(context);
+
+        // No split amount forwarded.
+        assertEq(specs[0].amount, 0);
+    }
+
+    // ──────────────────────────────────────────────
+    // Test: multiple tiers with different split percents
+    // ──────────────────────────────────────────────
+
+    function test_beforePayRecorded_multipleTiersDifferentSplitPercents() public {
+        ForTest_JB721TiersHook testHook = _initializeForTestHook(0);
+        IJB721TiersHookStore hookStore = testHook.STORE();
+
+        // Tier 1: 1 ETH, 30% split. Tier 2: 2 ETH, 100% split.
+        JB721TierConfig[] memory tierConfigs = new JB721TierConfig[](2);
+        tierConfigs[0] = _tierConfigWithSplit(1 ether, 300_000_000);
+        tierConfigs[0].category = 1;
+        tierConfigs[1] = _tierConfigWithSplit(2 ether, 1_000_000_000);
+        tierConfigs[1].category = 2;
+        vm.prank(address(testHook));
+        uint256[] memory tierIds = hookStore.recordAddTiers(tierConfigs);
+
+        uint16[] memory mintIds = new uint16[](2);
+        mintIds[0] = uint16(tierIds[0]);
+        mintIds[1] = uint16(tierIds[1]);
+        bytes memory payerMetadata = _buildPayerMetadata(address(testHook), mintIds);
+
+        JBBeforePayRecordedContext memory context = JBBeforePayRecordedContext({
+            terminal: mockTerminalAddress,
+            payer: beneficiary,
+            amount: JBTokenAmount({
+                token: JBConstants.NATIVE_TOKEN,
+                value: 3 ether,
+                decimals: 18,
+                currency: uint32(uint160(JBConstants.NATIVE_TOKEN))
+            }),
+            projectId: projectId,
+            rulesetId: 0,
+            beneficiary: beneficiary,
+            weight: 10e18,
+            reservedPercent: 5000,
+            metadata: payerMetadata
+        });
+
+        (, JBPayHookSpecification[] memory specs) = testHook.beforePayRecordedWith(context);
+
+        // Total split = 1 ETH * 30% + 2 ETH * 100% = 0.3 + 2.0 = 2.3 ETH.
+        assertEq(specs[0].amount, 2.3 ether);
+    }
+
+    // ──────────────────────────────────────────────
+    // Test: afterPayRecordedWith distributes to split beneficiary
+    // ──────────────────────────────────────────────
+
+    function test_afterPayRecorded_distributesToBeneficiary() public {
+        ForTest_JB721TiersHook testHook = _initializeForTestHook(0);
+        IJB721TiersHookStore hookStore = testHook.STORE();
+
+        // Add a tier with 50% split.
+        JB721TierConfig[] memory tierConfigs = new JB721TierConfig[](1);
+        tierConfigs[0] = _tierConfigWithSplit(1 ether, 500_000_000);
+        vm.prank(address(testHook));
+        uint256[] memory tierIds = hookStore.recordAddTiers(tierConfigs);
+
+        // Mock directory checks.
+        mockAndExpect(
+            address(mockJBDirectory),
+            abi.encodeWithSelector(IJBDirectory.isTerminalOf.selector, projectId, mockTerminalAddress),
+            abi.encode(true)
+        );
+        mockAndExpect(
+            address(mockJBDirectory),
+            abi.encodeWithSelector(IJBDirectory.controllerOf.selector, projectId),
+            abi.encode(mockJBController)
+        );
+        mockAndExpect(mockJBController, abi.encodeWithSelector(IJBController.SPLITS.selector), abi.encode(mockSplits));
+
+        // Mock splits: alice gets 100%.
+        JBSplit[] memory splits = new JBSplit[](1);
+        splits[0] = JBSplit({
+            percent: uint32(JBConstants.SPLITS_TOTAL_PERCENT),
+            projectId: 0,
+            beneficiary: payable(alice),
+            preferAddToBalance: false,
+            lockedUntil: 0,
+            hook: IJBSplitHook(address(0))
+        });
+
+        uint256 groupId = uint256(uint160(address(testHook))) | (uint256(tierIds[0]) << 160);
+        mockAndExpect(
+            mockSplits, abi.encodeWithSelector(IJBSplits.splitsOf.selector, projectId, 0, groupId), abi.encode(splits)
+        );
+
+        // Build payer metadata.
+        uint16[] memory mintIds = new uint16[](1);
+        mintIds[0] = uint16(tierIds[0]);
+        bytes memory payerMetadata = _buildPayerMetadata(address(testHook), mintIds);
+
+        // Build hook metadata (per-tier split breakdown from beforePayRecordedWith).
+        uint16[] memory splitTierIds = new uint16[](1);
+        splitTierIds[0] = uint16(tierIds[0]);
+        uint256[] memory splitAmounts = new uint256[](1);
+        splitAmounts[0] = 0.5 ether;
+
+        JBAfterPayRecordedContext memory payContext = JBAfterPayRecordedContext({
+            payer: beneficiary,
+            projectId: projectId,
+            rulesetId: 0,
+            amount: JBTokenAmount({
+                token: JBConstants.NATIVE_TOKEN,
+                value: 1 ether,
+                decimals: 18,
+                currency: uint32(uint160(JBConstants.NATIVE_TOKEN))
+            }),
+            forwardedAmount: JBTokenAmount({
+                token: JBConstants.NATIVE_TOKEN,
+                value: 0.5 ether,
+                decimals: 18,
+                currency: uint32(uint160(JBConstants.NATIVE_TOKEN))
+            }),
+            weight: 10e18,
+            newlyIssuedTokenCount: 0,
+            beneficiary: beneficiary,
+            hookMetadata: abi.encode(splitTierIds, splitAmounts),
+            payerMetadata: payerMetadata
+        });
+
+        uint256 aliceBalanceBefore = alice.balance;
+
+        vm.deal(mockTerminalAddress, 1 ether);
+        vm.prank(mockTerminalAddress);
+        testHook.afterPayRecordedWith{value: 0.5 ether}(payContext);
+
+        // Alice should have received 0.5 ETH.
+        assertEq(alice.balance - aliceBalanceBefore, 0.5 ether);
+        // NFT should have been minted.
+        assertEq(testHook.balanceOf(beneficiary), 1);
+    }
+}

package/src/abstract/JB721Hook.sol

@@ -1,279 +0,0 @@
-// SPDX-License-Identifier: MIT
-pragma solidity 0.8.23;
-
-import {IJBCashOutHook} from "@bananapus/core-v6/src/interfaces/IJBCashOutHook.sol";
-import {IJBDirectory} from "@bananapus/core-v6/src/interfaces/IJBDirectory.sol";
-import {IJBPayHook} from "@bananapus/core-v6/src/interfaces/IJBPayHook.sol";
-import {IJBRulesetDataHook} from "@bananapus/core-v6/src/interfaces/IJBRulesetDataHook.sol";
-import {IJBTerminal} from "@bananapus/core-v6/src/interfaces/IJBTerminal.sol";
-import {JBConstants} from "@bananapus/core-v6/src/libraries/JBConstants.sol";
-import {JBMetadataResolver} from "@bananapus/core-v6/src/libraries/JBMetadataResolver.sol";
-import {JBAfterPayRecordedContext} from "@bananapus/core-v6/src/structs/JBAfterPayRecordedContext.sol";
-import {JBAfterCashOutRecordedContext} from "@bananapus/core-v6/src/structs/JBAfterCashOutRecordedContext.sol";
-import {JBBeforePayRecordedContext} from "@bananapus/core-v6/src/structs/JBBeforePayRecordedContext.sol";
-import {JBBeforeCashOutRecordedContext} from "@bananapus/core-v6/src/structs/JBBeforeCashOutRecordedContext.sol";
-import {JBCashOutHookSpecification} from "@bananapus/core-v6/src/structs/JBCashOutHookSpecification.sol";
-import {JBPayHookSpecification} from "@bananapus/core-v6/src/structs/JBPayHookSpecification.sol";
-import {JBRuleset} from "@bananapus/core-v6/src/structs/JBRuleset.sol";
-import {IERC2981} from "@openzeppelin/contracts/interfaces/IERC2981.sol";
-import {IERC165} from "@openzeppelin/contracts/utils/introspection/IERC165.sol";
-import {mulDiv} from "@prb/math/src/Common.sol";
-
-import {ERC721} from "./ERC721.sol";
-import {IJB721Hook} from "../interfaces/IJB721Hook.sol";
-
-/// @title JB721Hook
-/// @notice When a project which uses this hook is paid, this hook may mint NFTs to the payer, depending on this hook's
-/// setup, the amount paid, and information specified by the payer. The project's owner can enable NFT cash outs.
-/// through this hook, allowing the NFT holders to burn their NFTs to reclaim funds from the project (in proportion to
-/// the NFT's price).
-abstract contract JB721Hook is ERC721, IJB721Hook {
-    //*********************************************************************//
-    // --------------------------- custom errors ------------------------- //
-    //*********************************************************************//
-
-    error JB721Hook_InvalidCashOut();
-    error JB721Hook_InvalidPay();
-    error JB721Hook_UnauthorizedToken(uint256 tokenId, address holder);
-    error JB721Hook_UnexpectedTokenCashedOut();
-
-    //*********************************************************************//
-    // --------------- public immutable stored properties ---------------- //
-    //*********************************************************************//
-
-    /// @notice The directory of terminals and controllers for projects.
-    IJBDirectory public immutable override DIRECTORY;
-
-    /// @notice The ID used when parsing metadata.
-    address public immutable override METADATA_ID_TARGET;
-
-    //*********************************************************************//
-    // -------------------- public stored properties --------------------- //
-    //*********************************************************************//
-
-    /// @notice The ID of the project that this contract is associated with.
-    uint256 public override PROJECT_ID;
-
-    //*********************************************************************//
-    // -------------------------- constructor ---------------------------- //
-    //*********************************************************************//
-
-    /// @param directory A directory of terminals and controllers for projects.
-    constructor(IJBDirectory directory) {
-        DIRECTORY = directory;
-        // Store the address of the original hook deploy. Clones will each use the address of the instance they're based
-        // on.
-        METADATA_ID_TARGET = address(this);
-    }
-
-    //*********************************************************************//
-    // ------------------------- external views -------------------------- //
-    //*********************************************************************//
-
-    /// @notice The data calculated before a payment is recorded in the terminal store. This data is provided to the
-    /// terminal's `pay(...)` transaction.
-    /// @dev Sets this contract as the pay hook. Part of `IJBRulesetDataHook`.
-    /// @param context The payment context passed to this contract by the `pay(...)` function.
-    /// @return weight The new `weight` to use, overriding the ruleset's `weight`.
-    /// @return hookSpecifications The amount and data to send to pay hooks (this contract) instead of adding to the
-    /// terminal's balance.
-    function beforePayRecordedWith(JBBeforePayRecordedContext calldata context)
-        public
-        view
-        virtual
-        override
-        returns (uint256 weight, JBPayHookSpecification[] memory hookSpecifications)
-    {
-        // Forward the received weight and memo, and use this contract as the only pay hook.
-        weight = context.weight;
-        hookSpecifications = new JBPayHookSpecification[](1);
-        hookSpecifications[0] = JBPayHookSpecification({hook: this, amount: 0, metadata: bytes("")});
-    }
-
-    /// @notice The data calculated before a cash out is recorded in the terminal store. This data is provided to the
-    /// terminal's `cashOutTokensOf(...)` transaction.
-    /// @dev Sets this contract as the cash out hook. Part of `IJBRulesetDataHook`.
-    /// @dev This function is used for NFT cash outs, and will only be called if the project's ruleset has
-    /// `useDataHookForCashOut` set to `true`.
-    /// @param context The cash out context passed to this contract by the `cashOutTokensOf(...)` function.
-    /// @return cashOutTaxRate The cash out tax rate influencing the reclaim amount.
-    /// @return cashOutCount The amount of tokens that should be considered cashed out.
-    /// @return totalSupply The total amount of tokens that are considered to be existing.
-    /// @return hookSpecifications The amount and data to send to cash out hooks (this contract) instead of returning to
-    /// the beneficiary.
-    function beforeCashOutRecordedWith(JBBeforeCashOutRecordedContext calldata context)
-        public
-        view
-        virtual
-        override
-        returns (
-            uint256 cashOutTaxRate,
-            uint256 cashOutCount,
-            uint256 totalSupply,
-            JBCashOutHookSpecification[] memory hookSpecifications
-        )
-    {
-        // Make sure (fungible) project tokens aren't also being cashed out.
-        if (context.cashOutCount > 0) revert JB721Hook_UnexpectedTokenCashedOut();
-
-        // Fetch the cash out hook metadata using the corresponding metadata ID.
-        (bool metadataExists, bytes memory metadata) = JBMetadataResolver.getDataFor({
-            id: JBMetadataResolver.getId({purpose: "cashOut", target: METADATA_ID_TARGET}), metadata: context.metadata
-        });
-
-        // Use this contract as the only cash out hook.
-        hookSpecifications = new JBCashOutHookSpecification[](1);
-        hookSpecifications[0] = JBCashOutHookSpecification({hook: this, amount: 0, metadata: bytes("")});
-
-        uint256[] memory decodedTokenIds;
-
-        // Decode the metadata.
-        if (metadataExists) decodedTokenIds = abi.decode(metadata, (uint256[]));
-
-        // Use the cash out weight of the provided 721s.
-        cashOutCount = cashOutWeightOf({tokenIds: decodedTokenIds, context: context});
-
-        // Use the total cash out weight of the 721s.
-        totalSupply = totalCashOutWeight(context);
-
-        // Use the cash out tax rate from the context.
-        cashOutTaxRate = context.cashOutTaxRate;
-    }
-
-    /// @notice Required by the IJBRulesetDataHook interfaces. Return false to not leak any permissions.
-    function hasMintPermissionFor(uint256, JBRuleset memory, address) external pure returns (bool) {
-        return false;
-    }
-
-    //*********************************************************************//
-    // -------------------------- public views --------------------------- //
-    //*********************************************************************//
-
-    /// @notice Returns the cumulative cash out weight of the specified token IDs relative to the
-    /// `totalCashOutWeight`.
-    /// @param tokenIds The NFT token IDs to calculate the cumulative cash out weight of.
-    /// @param context The cash out context passed to this contract by the `cashOutTokensOf(...)` function.
-    /// @return The cumulative cash out weight of the specified token IDs.
-    function cashOutWeightOf(
-        uint256[] memory tokenIds,
-        JBBeforeCashOutRecordedContext calldata context
-    )
-        public
-        view
-        virtual
-        returns (uint256)
-    {
-        tokenIds; // Prevents unused var compiler and natspec complaints.
-        context; // Prevents unused var compiler and natspec complaints.
-        return 0;
-    }
-
-    /// @notice Indicates if this contract adheres to the specified interface.
-    /// @dev See {IERC165-supportsInterface}.
-    /// @param interfaceId The ID of the interface to check for adherence to.
-    function supportsInterface(bytes4 interfaceId) public view virtual override(ERC721, IERC165) returns (bool) {
-        return interfaceId == type(IJB721Hook).interfaceId || interfaceId == type(IJBRulesetDataHook).interfaceId
-            || interfaceId == type(IJBPayHook).interfaceId || interfaceId == type(IJBCashOutHook).interfaceId
-            || interfaceId == type(IERC2981).interfaceId || super.supportsInterface(interfaceId);
-    }
-
-    /// @notice Calculates the cumulative cash out weight of all NFT token IDs.
-    /// @param context The cash out context passed to this contract by the `cashOutTokensOf(...)` function.
-    /// @return The total cumulative cash out weight of all NFT token IDs.
-    function totalCashOutWeight(JBBeforeCashOutRecordedContext calldata context) public view virtual returns (uint256) {
-        context; // Prevents unused var compiler and natspec complaints.
-        return 0;
-    }
-
-    //*********************************************************************//
-    // ---------------------- external transactions ---------------------- //
-    //*********************************************************************//
-
-    /// @notice Mints one or more NFTs to the `context.beneficiary` upon payment if conditions are met. Part of
-    /// `IJBPayHook`.
-    /// @dev Reverts if the calling contract is not one of the project's terminals.
-    /// @param context The payment context passed in by the terminal.
-    // slither-disable-next-line locked-ether
-    function afterPayRecordedWith(JBAfterPayRecordedContext calldata context) external payable virtual override {
-        uint256 projectId = PROJECT_ID;
-
-        // Make sure the caller is a terminal of the project, and that the call is being made on behalf of an
-        // interaction with the correct project.
-        if (
-            msg.value != 0 || !DIRECTORY.isTerminalOf({projectId: projectId, terminal: IJBTerminal(msg.sender)})
-                || context.projectId != projectId
-        ) revert JB721Hook_InvalidPay();
-
-        // Process the payment.
-        _processPayment(context);
-    }
-
-    /// @notice Burns the specified NFTs upon token holder cash out, reclaiming funds from the project's balance for
-    /// `context.beneficiary`. Part of `IJBCashOutHook`.
-    /// @dev Reverts if the calling contract is not one of the project's terminals.
-    /// @param context The cash out context passed in by the terminal.
-    // slither-disable-next-line locked-ether
-    function afterCashOutRecordedWith(JBAfterCashOutRecordedContext calldata context)
-        external
-        payable
-        virtual
-        override
-    {
-        // Keep a reference to the project ID.
-        uint256 projectId = PROJECT_ID;
-
-        // Make sure the caller is a terminal of the project, and that the call is being made on behalf of an
-        // interaction with the correct project.
-        if (
-            msg.value != 0 || !DIRECTORY.isTerminalOf({projectId: projectId, terminal: IJBTerminal(msg.sender)})
-                || context.projectId != projectId
-        ) revert JB721Hook_InvalidCashOut();
-
-        // Fetch the cash out hook metadata using the corresponding metadata ID.
-        (bool metadataExists, bytes memory metadata) = JBMetadataResolver.getDataFor({
-            id: JBMetadataResolver.getId({purpose: "cashOut", target: METADATA_ID_TARGET}),
-            metadata: context.cashOutMetadata
-        });
-
-        uint256[] memory decodedTokenIds;
-
-        // Decode the metadata.
-        if (metadataExists) decodedTokenIds = abi.decode(metadata, (uint256[]));
-
-        // Iterate through the NFTs, burning them if the owner is correct.
-        for (uint256 i; i < decodedTokenIds.length; i++) {
-            // Set the current NFT's token ID.
-            uint256 tokenId = decodedTokenIds[i];
-
-            // Make sure the token's owner is correct.
-            if (_ownerOf(tokenId) != context.holder) revert JB721Hook_UnauthorizedToken(tokenId, context.holder);
-
-            // Burn the token.
-            _burn(tokenId);
-        }
-
-        // Call the hook.
-        _didBurn(decodedTokenIds);
-    }
-
-    //*********************************************************************//
-    // ---------------------- internal transactions ---------------------- //
-    //*********************************************************************//
-
-    /// @notice Initializes the contract by associating it with a project and adding ERC721 details.
-    /// @param projectId The ID of the project that this contract is associated with.
-    /// @param name The name of the NFT collection.
-    /// @param symbol The symbol representing the NFT collection.
-    function _initialize(uint256 projectId, string memory name, string memory symbol) internal {
-        ERC721._initialize({name_: name, symbol_: symbol});
-        PROJECT_ID = projectId;
-    }
-
-    /// @notice Executes after NFTs have been burned via cash out.
-    /// @param tokenIds The token IDs of the NFTs that were burned.
-    function _didBurn(uint256[] memory tokenIds) internal virtual;
-
-    /// @notice Process a received payment.
-    /// @param context The payment context passed in by the terminal.
-    function _processPayment(JBAfterPayRecordedContext calldata context) internal virtual;
-}

package/src/interfaces/IJB721Hook.sol

@@ -1,21 +0,0 @@
-// SPDX-License-Identifier: MIT
-pragma solidity ^0.8.0;
-
-import {IJBCashOutHook} from "@bananapus/core-v6/src/interfaces/IJBCashOutHook.sol";
-import {IJBDirectory} from "@bananapus/core-v6/src/interfaces/IJBDirectory.sol";
-import {IJBPayHook} from "@bananapus/core-v6/src/interfaces/IJBPayHook.sol";
-import {IJBRulesetDataHook} from "@bananapus/core-v6/src/interfaces/IJBRulesetDataHook.sol";
-
-interface IJB721Hook is IJBRulesetDataHook, IJBPayHook, IJBCashOutHook {
-    /// @notice The directory of terminals and controllers for projects.
-    /// @return The directory contract.
-    function DIRECTORY() external view returns (IJBDirectory);
-
-    /// @notice The ID used when parsing metadata.
-    /// @return The address of the metadata ID target.
-    function METADATA_ID_TARGET() external view returns (address);
-
-    /// @notice The ID of the project that this contract is associated with.
-    /// @return The project ID.
-    function PROJECT_ID() external view returns (uint256);
-}