@bananapus/721-hook-v6 0.0.35 → 0.0.37

package/USER_JOURNEYS.md

@@ -1,98 +1,121 @@
 # User Journeys
 
-## Who This Repo Serves
+## Repo Purpose
 
-- project owners selling or rewarding supporters with tiered NFTs
-- operators managing tier supply, pricing, reserves, and ruleset-aware hook behavior
-- supporters minting or cashing out tier positions through normal Juicebox flows
-- integrators composing custom token URI resolvers on top of the standard 721 hook
+This repo adds tiered NFT logic to Juicebox payment and cash-out flows. It owns tier pricing, reserves, and NFT lifecycle state. It does not own project-specific artwork or game logic.
 
-## Journey 1: Add A Tiered 721 Hook To An Existing Project
+## Primary Actors
 
-**Starting state:** the project already exists in Juicebox and needs tiered NFT issuance without relaunching.
+- projects that want priced NFT tiers in their Juicebox flow
+- operators managing tier configuration and hook permissions
+- holders minting, transferring, and cashing out tiered NFTs
+- auditors reviewing tier accounting, reserve behavior, and deployer wiring
 
-**Success:** a hook clone is deployed, initialized, and attached to the project's ruleset metadata.
+## Key Surfaces
 
-**Flow**
-1. Use `JB721TiersHookDeployer` to clone a hook for the target project.
-2. Define tier config, reserve behavior, token URI resolver, and per-ruleset flags.
-3. Queue or install ruleset metadata that tells the project when the hook should participate in pay and cash-out flows.
-4. Future payments into the project can now mint tiers under the configured constraints.
+- `JB721TiersHook`: runtime 721 hook behavior
+- `JB721TiersHookStore`: tier accounting and supply state
+- `JB721TiersHookDeployer` and `JB721TiersHookProjectDeployer`: wiring surfaces
+- token URI resolver contracts in downstream repos: presentation layer
 
-## Journey 2: Launch A New Project With A 721 Hook Already Wired In
+## Journey 1: Launch A Project With A Tiered NFT Hook
 
-**Starting state:** the product wants its project treasury and NFT logic created in one operation.
+**Actor:** project operator or deployer.
 
-**Success:** the project launches with the hook, terminals, and initial tiers already aligned.
+**Intent:** attach tiered NFT issuance to a project from the start.
 
-**Flow**
-1. Use `JB721TiersHookProjectDeployer` with launch config, rulesets, and hook config.
-2. The deployer launches the project through the core protocol and deploys the hook in the same packaged flow.
-3. The initial ruleset metadata points at the newly created hook so there is no post-launch rewiring gap.
-4. The project starts life as a tiered 721 project instead of becoming one later.
+**Preconditions**
+- the project knows its tier structure and hook expectations
+- the deployer path matches whether the project already exists
 
-## Journey 3: Mint Specific Tiers Through A Payment
+**Main Flow**
+1. Deploy a hook clone or launch a project with the hook already attached.
+2. Configure tier data, hook metadata, and resolver expectations.
+3. Transfer hook ownership into the intended project control surface.
 
-**Starting state:** the project has live tiers and the payer knows which tiers they want.
+**Failure Modes**
+- wrong hook wiring at launch
+- wrong resolver assumptions
+- teams treat deployer convenience as proof that runtime economics are correct
 
-**Success:** the treasury receives funds and the beneficiary receives the intended NFT tiers plus any accompanying project-token behavior.
+**Postconditions**
+- the project has a tiered NFT hook wired into its Juicebox flow
 
-**Flow**
-1. The payer submits a payment with metadata encoding the desired tier selections.
-2. `JB721TiersHook` validates tier availability, quantity rules, discounts, category constraints, and any ruleset flags affecting minting.
-3. `JB721TiersHookStore` updates supply and reserve accounting.
-4. The hook mints the correct NFTs and the underlying terminal completes treasury accounting.
+## Journey 2: Pay And Mint Tiered NFTs
 
-**Failure cases that matter:** sold-out tiers, bad metadata, cross-currency pricing mistakes, paused pay-hook behavior, and split-routing edge cases when part of the payment should bypass normal minting assumptions.
+**Actor:** payer or integration acting for a payer.
 
-## Journey 4: Mint Reserves And Operate Tier Inventory Over Time
+**Intent:** mint NFTs from configured tiers while preserving the project's terminal flow.
 
-**Starting state:** the collection is live and the owner needs to manage what exists for future minters versus reserve recipients.
+**Preconditions**
+- the project has active tiers
+- payment metadata correctly names the intended tiers
 
-**Success:** reserves, new tiers, removed tiers, and editable fields evolve without corrupting ownership or supply history.
+**Main Flow**
+1. A payment reaches the hook through the terminal.
+2. The hook decodes tier selection and records mint state in the store.
+3. NFTs mint, reserve implications update, and any split routing is applied.
 
-**Flow**
-1. Authorized operators use the hook's tier-management surfaces to add, remove, or adjust tiers.
-2. Reserve minting uses the configured reserve frequency and reserve beneficiary settings instead of ad hoc treasury actions.
-3. The store keeps historical tier state coherent so existing token IDs continue to resolve correctly.
-4. Downstream products such as Croptop, Banny, and Revnets can continue assuming stable tier semantics.
+**Failure Modes**
+- malformed metadata
+- currency mismatch or missing pricing support
+- splits or discounts behave differently than the integration expected
 
-## Journey 5: Let Holders Cash Out Tier Positions
+**Postconditions**
+- the payer or beneficiary receives the intended NFT tiers and tier state updates
 
-**Starting state:** a holder owns one or more NFTs from a hook-enabled project and the active ruleset allows surplus-backed exits.
+## Journey 3: Mint Or Release Reserve NFTs
 
-**Success:** the holder burns the intended tier exposure and receives the correct reclaim value through the core terminal.
+**Actor:** reserve beneficiary, operator, or any caller using the reserve path.
 
-**Flow**
-1. The holder calls the project's cash-out path on the terminal.
-2. The hook participates in the cash-out calculation so tier-specific weight and store state are reflected correctly.
-3. The terminal settles reclaim value and the NFT position is burned or otherwise marked as consumed by the exit path.
+**Intent:** realize pending reserves under the configured reserve rules.
 
-**Edge conditions that change user experience:** ERC-20 cash-out surfaces, tier splits, reserve accounting drift, broken downstream terminals, and projects that use the hook for metadata only versus economically binding flows.
+**Preconditions**
+- the relevant tiers have reserve logic enabled
+- the ruleset does not pause pending reserve minting
 
-## Journey 6: Compose A Custom Product On Top Of The Standard Hook
+**Main Flow**
+1. Eligible reserve amounts accumulate as mint activity happens.
+2. A caller triggers reserve minting for pending tiers.
+3. The store moves reserve state forward and NFTs mint to the configured reserve beneficiary.
 
-**Starting state:** the team wants product-specific rendering or business rules but does not want to fork tier issuance.
+**Failure Modes**
+- teams misunderstand that reserve minting timing is not owner-exclusive
+- reserve assumptions drift from actual tier settings
 
-**Success:** the product resolver or wrapper composes with the hook instead of reimplementing pricing, tier accounting, and treasury logic.
+**Postconditions**
+- pending reserves mint according to tier configuration
 
-**Flow**
-1. Plug a custom resolver into the hook for metadata and product-specific presentation.
-2. Keep collection-specific behavior in the downstream repo while leaving pay, reserve, and cash-out semantics in this repo.
-3. Audit hook-store interactions here first, then audit the downstream resolver or wrapper.
+## Journey 4: Cash Out Tiered NFTs
 
-## Journey 7: Mint NFTs To The Correct Beneficiary During Cross-Chain Payments
+**Actor:** NFT holder.
 
-**Starting state:** a sucker pays the project on behalf of a remote user via `payRemote`, and the 721 hook needs to mint NFTs and accrue credits to the real user instead of the sucker contract.
+**Intent:** redeem tiered NFT exposure through the terminal cash-out path.
 
-**Success:** NFTs mint to and pay credits accrue to the real remote user.
+**Preconditions**
+- the holder owns valid NFTs
+- the hook is active for the cash-out path
 
-**Flow**
-1. The sucker calls `terminal.pay()` with itself as both payer and beneficiary, embedding the real user's address in the `JB_RELAY_BENEFICIARY` metadata key.
-2. `_mintAndUpdateCredits` detects that `payer == beneficiary` and finds relay-beneficiary metadata.
-3. All NFT minting and credit accounting uses the resolved relay beneficiary instead of the sucker address.
+**Main Flow**
+1. The holder requests a cash out with NFT-specific metadata.
+2. The hook burns the selected NFTs and records the burn in the store.
+3. The terminal completes reclaim logic using the hook-aware cash-out surface.
+
+**Failure Modes**
+- integrations mix fungible-token and NFT cash-out assumptions
+- pending reserves or discounts are misunderstood in value calculations
+- token IDs are invalid or already burned
+
+**Postconditions**
+- NFTs burn and reclaim value follows the intended tier model
+
+## Trust Boundaries
+
+- this repo trusts core terminals, directory checks, and pricing surfaces from `nana-core-v6`
+- metadata resolvers are outside this repo but still affect user-visible trust
+- the store is the main source of truth for tier lifecycle state
 
 ## Hand-Offs
 
-- Use [nana-core-v6](../nana-core-v6/USER_JOURNEYS.md) for the treasury, ruleset, and permission surfaces the hook plugs into.
-- Use [banny-retail-v6](../banny-retail-v6/USER_JOURNEYS.md), [croptop-core-v6](../croptop-core-v6/USER_JOURNEYS.md), and [revnet-core-v6](../revnet-core-v6/USER_JOURNEYS.md) for product layers built on this hook.
+- Use [nana-core-v6](../nana-core-v6/USER_JOURNEYS.md) for the underlying terminal and accounting behavior.
+- Use the downstream resolver repo when the question is about project-specific metadata or rendering.

package/foundry.toml

@@ -13,6 +13,8 @@ runs = 1024
 depth = 100
 fail_on_revert = false
 
+[lint]
+exclude_lints = ["pascal-case-struct", "mixed-case-variable"]
 [fmt]
 number_underscore = "thousands"
 multiline_func_header = "all"

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bananapus/721-hook-v6",
-  "version": "0.0.35",
+  "version": "0.0.37",
   "license": "MIT",
   "repository": {
     "type": "git",

package/references/operations.md

@@ -9,7 +9,9 @@
 ## Change Checklist
 
 - If you edit hook initialization, verify deployer config structs and project-launch helpers still encode the same assumptions.
-- If you edit tier config or metadata behavior, inspect [`src/structs/`](../src/structs/) and the corresponding interfaces in [`src/interfaces/`](../src/interfaces/).
+- If you edit tier config or metadata behavior, inspect the corresponding structs and interfaces in `src/structs/` and `src/interfaces/`.
+- If you edit reserve behavior, verify pending reserve counts, default reserve beneficiary semantics, and cash-out denominator effects together.
+- If you edit discount behavior, verify mint price and cash-out weight separately. They are intentionally not the same quantity.
 - If you touch permissions, verify the caller path and permission constants still line up with the downstream ecosystem package that defines them.
 - If you touch URI behavior, confirm whether the issue belongs in this repo or in a downstream resolver contract that the hook calls.
 
@@ -24,5 +26,7 @@
 
 - [`test/Fork.t.sol`](../test/Fork.t.sol) for live-integration assumptions.
 - [`test/TestAuditGaps.sol`](../test/TestAuditGaps.sol) for known edge cases the repo authors considered worth pinning down.
-- [`test/unit/`](../test/unit/) when you need a narrow function-level proof before editing a broad runtime path.
-- [`script/helpers/`](../script/helpers/) when a deployment or launch question is really about config assembly rather than contract behavior.
+- [`test/TestCheckpoints.t.sol`](../test/TestCheckpoints.t.sol) when you need a narrow function-level proof before editing a broad runtime path.
+- [`test/invariants/TierLifecycleInvariant.t.sol`](../test/invariants/TierLifecycleInvariant.t.sol) and [`test/invariants/TieredHookStoreInvariant.t.sol`](../test/invariants/TieredHookStoreInvariant.t.sol) when a local patch may have broken store-level relationships.
+- [`test/audit/CodexRetroactiveReserveBeneficiaryDilution.t.sol`](../test/audit/CodexRetroactiveReserveBeneficiaryDilution.t.sol) when reserve-beneficiary or pending-reserve behavior changes.
+- [`script/Deploy.s.sol`](../script/Deploy.s.sol) when a deployment or launch question is really about config assembly rather than contract behavior.

package/references/runtime.md

@@ -22,11 +22,12 @@
 - Discount behavior: price discounts affect mint eligibility but cash-out weight still tracks the original tier price. Do not conflate the two.
 - Voting units: verify whether a tier uses explicit voting units or falls back to price-based voting power before changing governance-facing math.
 - Tier removal and cleanup: removing tiers is not the same as cleaning the sorted tier list. Storage cleanup behavior matters.
+- Default reserve beneficiary changes: they affect which tiers count pending reserves unless a tier-specific beneficiary overrides it. That is an economic change, not just an admin update.
 
 ## Tests To Trust First
 
-- [`test/invariants/`](../test/invariants/) for broad accounting invariants.
-- [`test/E2E/`](../test/E2E/) for launch and end-to-end payment flows.
-- [`test/regression/`](../test/regression/) for previously broken edge cases.
+- [`test/Fork.t.sol`](../test/Fork.t.sol) for launch and live integration flows.
 - [`test/TestVotingUnitsLifecycle.t.sol`](../test/TestVotingUnitsLifecycle.t.sol) for voting-unit lifecycle behavior.
-- [`test/TestSafeTransferReentrancy.t.sol`](../test/TestSafeTransferReentrancy.t.sol) and [`test/721HookAttacks.t.sol`](../test/721HookAttacks.t.sol) for reentrancy and attack-surface checks.
+- [`test/TestCheckpoints.t.sol`](../test/TestCheckpoints.t.sol) for checkpoint/module behavior.
+- [`test/invariants/TierLifecycleInvariant.t.sol`](../test/invariants/TierLifecycleInvariant.t.sol) and [`test/invariants/TieredHookStoreInvariant.t.sol`](../test/invariants/TieredHookStoreInvariant.t.sol) for store-level lifecycle invariants.
+- [`test/TestSafeTransferReentrancy.t.sol`](../test/TestSafeTransferReentrancy.t.sol), [`test/721HookAttacks.t.sol`](../test/721HookAttacks.t.sol), [`test/audit/CodexRetroactiveReserveBeneficiaryDilution.t.sol`](../test/audit/CodexRetroactiveReserveBeneficiaryDilution.t.sol), and [`test/TestAuditGaps.sol`](../test/TestAuditGaps.sol) for reentrancy and attack-surface checks.

package/src/JB721CheckpointsDeployer.sol

@@ -37,6 +37,8 @@ contract JB721CheckpointsDeployer is IJB721CheckpointsDeployer {
     /// @param store The store that holds tier data for the hook's NFTs.
     /// @return module The newly deployed and initialized checkpoint module.
     function deploy(address hook, IJB721TiersHookStore store) external override returns (IJB721Checkpoints module) {
+        if (msg.sender != hook) revert JB721CheckpointsDeployer_Unauthorized();
+
         module = IJB721Checkpoints(
             LibClone.cloneDeterministic({implementation: IMPLEMENTATION, salt: bytes32(uint256(uint160(hook)))})
         );

package/src/JB721TiersHook.sol

@@ -457,9 +457,7 @@ contract JB721TiersHook is JBOwnable, ERC2771Context, JB721Hook, IJB721TiersHook
         string calldata baseUri,
         string calldata contractUri,
         IJB721TokenUriResolver tokenUriResolver,
-        // forge-lint: disable-next-line(mixed-case-variable)
         uint256 encodedIPFSUriTierId,
-        // forge-lint: disable-next-line(mixed-case-variable)
         bytes32 encodedIPFSUri
     )
         external

package/src/JB721TiersHookProjectDeployer.sol

@@ -124,7 +124,6 @@ contract JB721TiersHookProjectDeployer is ERC2771Context, JBPermissioned, IJB721
         returns (uint256 rulesetId, IJB721TiersHook hook)
     {
         // Get the project's projects contract.
-        // forge-lint: disable-next-line(mixed-case-variable)
         IJBProjects PROJECTS = DIRECTORY.PROJECTS();
 
         // Enforce permissions.

package/src/JB721TiersHookStore.sol

@@ -66,7 +66,6 @@ contract JB721TiersHookStore is IJB721TiersHookStore {
     /// @custom:param hook The 721 contract that the tier belongs to.
     /// @custom:param tierId The ID of the tier to get the encoded IPFS URI of.
     /// @custom:returns The encoded IPFS URI.
-    // forge-lint: disable-next-line(mixed-case-variable)
     mapping(address hook => mapping(uint256 tierId => bytes32)) public override encodedIPFSUriOf;
 
     /// @notice Returns the largest tier ID currently used on the provided 721 contract.
@@ -1344,7 +1343,7 @@ contract JB721TiersHookStore is IJB721TiersHookStore {
     /// @notice Record a new encoded IPFS URI for a tier.
     /// @param tierId The ID of the tier to set the encoded IPFS URI of.
     /// @param encodedIPFSUri The encoded IPFS URI to set for the tier.
-    // forge-lint: disable-next-line(mixed-case-function, mixed-case-variable)
+    // forge-lint: disable-next-line(mixed-case-function)
     function recordSetEncodedIPFSUriOf(uint256 tierId, bytes32 encodedIPFSUri) external override {
         encodedIPFSUriOf[msg.sender][tierId] = encodedIPFSUri;
     }

package/src/abstract/JB721Hook.sol

@@ -49,7 +49,6 @@ abstract contract JB721Hook is ERC721, IJB721Hook {
     //*********************************************************************//
 
     /// @notice The ID of the project that this contract is associated with.
-    // forge-lint: disable-next-line(mixed-case-variable)
     uint256 public override PROJECT_ID;
 
     //*********************************************************************//

package/src/interfaces/IJB721CheckpointsDeployer.sol

@@ -6,6 +6,9 @@ import {IJB721TiersHookStore} from "./IJB721TiersHookStore.sol";
 
 /// @notice Deploys JB721Checkpoints clones for JB721TiersHook instances.
 interface IJB721CheckpointsDeployer {
+    /// @notice Thrown when the caller is not the hook that the checkpoint module is being deployed for.
+    error JB721CheckpointsDeployer_Unauthorized();
+
     /// @notice The implementation contract that clones are based on.
     /// @return The implementation address.
     // forge-lint: disable-next-line(mixed-case-function)

package/src/interfaces/IJB721TiersHook.sol

@@ -234,9 +234,7 @@ interface IJB721TiersHook is IJB721Hook {
         string calldata baseUri,
         string calldata contractUri,
         IJB721TokenUriResolver tokenUriResolver,
-        // forge-lint: disable-next-line(mixed-case-variable)
         uint256 encodedIPFSUriTierId,
-        // forge-lint: disable-next-line(mixed-case-variable)
         bytes32 encodedIPFSUri
     )
         external;

package/src/interfaces/IJB721TiersHookStore.sol

@@ -247,7 +247,7 @@ interface IJB721TiersHookStore {
     /// @notice Record a new encoded IPFS URI for a tier.
     /// @param tierId The ID of the tier to set the encoded IPFS URI of.
     /// @param encodedIPFSUri The encoded IPFS URI to set for the tier.
-    // forge-lint: disable-next-line(mixed-case-function, mixed-case-variable)
+    // forge-lint: disable-next-line(mixed-case-function)
     function recordSetEncodedIPFSUriOf(uint256 tierId, bytes32 encodedIPFSUri) external;
 
     /// @notice Record a newly set token URI resolver.

package/src/libraries/JB721Constants.sol

@@ -8,6 +8,5 @@ library JB721Constants {
     /// @notice The metadata ID used to identify the 721 beneficiary entry in payment metadata.
     /// @dev When a sucker pays on behalf of a remote user, the real user's address is embedded under this key
     /// so NFTs mint to the correct recipient.
-    // forge-lint: disable-next-line(mixed-case-variable)
     bytes4 public constant BENEFICIARY_METADATA_ID = bytes4(keccak256("JB_721_BENEFICIARY"));
 }

package/src/structs/JB721InitTiersConfig.sol

@@ -8,7 +8,6 @@ import {JB721TierConfig} from "./JB721TierConfig.sol";
 /// @custom:member tiers The tiers to initialize the hook with.
 /// @custom:member currency The currency that the tier prices are denoted in. See `JBPrices`.
 /// @custom:member decimals The number of decimals in the fixed point tier prices.
-// forge-lint: disable-next-line(pascal-case-struct)
 struct JB721InitTiersConfig {
     JB721TierConfig[] tiers;
     uint32 currency;

package/src/structs/JB721Tier.sol

@@ -21,7 +21,6 @@ import {JB721TierFlags} from "./JB721TierFlags.sol";
 /// an NFT from this tier is minted. Out of `JBConstants.SPLITS_TOTAL_PERCENT`.
 /// @custom:member resolvedUri A resolved token URI for NFTs in this tier. Only available if the NFT this tier belongs
 /// to has a resolver.
-// forge-lint: disable-next-line(pascal-case-struct)
 struct JB721Tier {
     uint32 id;
     uint104 price;
@@ -30,7 +29,6 @@ struct JB721Tier {
     uint104 votingUnits;
     uint16 reserveFrequency;
     address reserveBeneficiary;
-    // forge-lint: disable-next-line(mixed-case-variable)
     bytes32 encodedIPFSUri;
     uint24 category;
     uint8 discountPercent;

package/src/structs/JB721TierConfig.sol

@@ -23,14 +23,12 @@ import {JB721TierConfigFlags} from "./JB721TierConfigFlags.sol";
 /// an NFT from this tier is minted. Out of `JBConstants.SPLITS_TOTAL_PERCENT`.
 /// @custom:member splits The splits to use for this tier's split group. These define where the split portion of the
 /// tier's price gets routed when an NFT from this tier is minted.
-// forge-lint: disable-next-line(pascal-case-struct)
 struct JB721TierConfig {
     uint104 price;
     uint32 initialSupply;
     uint32 votingUnits;
     uint16 reserveFrequency;
     address reserveBeneficiary;
-    // forge-lint: disable-next-line(mixed-case-variable)
     bytes32 encodedIPFSUri;
     uint24 category;
     uint8 discountPercent;

package/src/structs/JB721TierConfigFlags.sol

@@ -14,7 +14,6 @@ pragma solidity ^0.8.0;
 /// @custom:member cantIncreaseDiscountPercent If the tier cannot have its discount increased.
 /// @custom:member cantBuyWithCredits If true, this tier cannot be purchased using accumulated pay credits. Only fresh
 /// payment value counts toward this tier's price.
-// forge-lint: disable-next-line(pascal-case-struct)
 struct JB721TierConfigFlags {
     bool allowOwnerMint;
     bool useReserveBeneficiaryAsDefault;

package/src/structs/JB721TierFlags.sol

@@ -7,7 +7,6 @@ pragma solidity ^0.8.0;
 /// @custom:member cantBeRemoved A boolean indicating whether attempts to remove this tier will revert.
 /// @custom:member cantIncreaseDiscountPercent If the tier cannot have its discount increased.
 /// @custom:member cantBuyWithCredits If true, this tier cannot be purchased using accumulated pay credits.
-// forge-lint: disable-next-line(pascal-case-struct)
 struct JB721TierFlags {
     bool allowOwnerMint;
     bool transfersPausable;

package/src/structs/JB721TiersHookFlags.sol

@@ -11,7 +11,6 @@ pragma solidity ^0.8.0;
 /// the NFTs being minted will revert.
 /// @custom:member issueTokensForSplits A boolean indicating whether payers receive token credit for the portion of
 /// their payment that is routed to tier splits. When false (default), weight is reduced proportionally.
-// forge-lint: disable-next-line(pascal-case-struct)
 struct JB721TiersHookFlags {
     bool noNewTiersWithReserves;
     bool noNewTiersWithVotes;

package/src/structs/JB721TiersMintReservesConfig.sol

@@ -3,7 +3,6 @@ pragma solidity ^0.8.0;
 
 /// @custom:member tierId The ID of the tier to mint from.
 /// @custom:member count The number of NFTs to mint from that tier.
-// forge-lint: disable-next-line(pascal-case-struct)
 struct JB721TiersMintReservesConfig {
     uint32 tierId;
     uint16 count;

package/src/structs/JB721TiersRulesetMetadata.sol

@@ -6,7 +6,6 @@ pragma solidity ^0.8.0;
 /// @custom:member pauseTransfers A boolean indicating whether NFT transfers are paused during this ruleset.
 /// @custom:member pauseMintPendingReserves A boolean indicating whether pending/outstanding NFT reserves can be minted
 /// during this ruleset.
-// forge-lint: disable-next-line(pascal-case-struct)
 struct JB721TiersRulesetMetadata {
     bool pauseTransfers;
     bool pauseMintPendingReserves;

package/src/structs/JB721TiersSetDiscountPercentConfig.sol

@@ -3,7 +3,6 @@ pragma solidity ^0.8.0;
 
 /// @custom:member tierId The ID of the tier to set the discount percent for.
 /// @custom:member discountPercent The discount percent to set for the tier.
-// forge-lint: disable-next-line(pascal-case-struct)
 struct JB721TiersSetDiscountPercentConfig {
     uint32 tierId;
     uint16 discountPercent;

package/src/structs/JBBitmapWord.sol

@@ -5,7 +5,6 @@ pragma solidity ^0.8.0;
 /// `JBBitmap` matrix is a "word".
 /// @custom:member The information stored at the index.
 /// @custom:member The index.
-// forge-lint: disable-next-line(pascal-case-struct)
 struct JBBitmapWord {
     uint256 currentWord;
     uint256 currentDepth;

package/src/structs/JBDeploy721TiersHookConfig.sol

@@ -12,7 +12,6 @@ import {IJB721TokenUriResolver} from "../interfaces/IJB721TokenUriResolver.sol";
 /// @custom:member contractUri The URI where this contract's metadata can be found.
 /// @custom:member tiersConfig The NFT tiers and pricing config to launch the hook with.
 /// @custom:member flags A set of boolean options to configure the hook with.
-// forge-lint: disable-next-line(pascal-case-struct)
 struct JBDeploy721TiersHookConfig {
     string name;
     string symbol;

package/src/structs/JBLaunchProjectConfig.sol

@@ -10,7 +10,6 @@ import {JBPayDataHookRulesetConfig} from "./JBPayDataHookRulesetConfig.sol";
 /// @custom:member rulesetConfigurations The ruleset configurations to queue.
 /// @custom:member terminalConfigurations The terminal configurations to add for the project.
 /// @custom:member memo A memo to pass along to the emitted event.
-// forge-lint: disable-next-line(pascal-case-struct)
 struct JBLaunchProjectConfig {
     string projectUri;
     JBPayDataHookRulesetConfig[] rulesetConfigurations;

package/src/structs/JBLaunchRulesetsConfig.sol

@@ -9,7 +9,6 @@ import {JBPayDataHookRulesetConfig} from "./JBPayDataHookRulesetConfig.sol";
 /// @custom:member rulesetConfigurations The ruleset configurations to queue.
 /// @custom:member terminalConfigurations The terminal configurations to add for the project.
 /// @custom:member memo A memo to pass along to the emitted event.
-// forge-lint: disable-next-line(pascal-case-struct)
 struct JBLaunchRulesetsConfig {
     uint56 projectId;
     JBPayDataHookRulesetConfig[] rulesetConfigurations;

package/src/structs/JBPayDataHookRulesetConfig.sol

@@ -32,7 +32,6 @@ import {JBPayDataHookRulesetMetadata} from "./JBPayDataHookRulesetMetadata.sol";
 /// its balance in each payment terminal while the ruleset is active. Amounts are fixed point numbers using the same
 /// number of decimals as the corresponding terminal. The `payoutLimit` and `surplusAllowance` parameters must fit in
 /// a `uint232`.
-// forge-lint: disable-next-line(pascal-case-struct)
 struct JBPayDataHookRulesetConfig {
     uint48 mustStartAtOrAfter;
     uint32 duration;

package/src/structs/JBPayDataHookRulesetMetadata.sol

@@ -28,7 +28,6 @@ pragma solidity ^0.8.0;
 /// during
 /// this ruleset.
 /// @custom:member metadata Metadata of the metadata, up to uint8 in size.
-// forge-lint: disable-next-line(pascal-case-struct)
 struct JBPayDataHookRulesetMetadata {
     uint16 reservedPercent;
     uint16 cashOutTaxRate;

package/src/structs/JBQueueRulesetsConfig.sol

@@ -6,7 +6,6 @@ import {JBPayDataHookRulesetConfig} from "./JBPayDataHookRulesetConfig.sol";
 /// @custom:member projectId The ID of the project to queue rulesets for.
 /// @custom:member rulesetConfigurations The ruleset configurations to queue.
 /// @custom:member memo A memo to pass along to the emitted event.
-// forge-lint: disable-next-line(pascal-case-struct)
 struct JBQueueRulesetsConfig {
     uint56 projectId;
     JBPayDataHookRulesetConfig[] rulesetConfigurations;

package/src/structs/JBStored721Tier.sol

@@ -13,7 +13,6 @@ pragma solidity ^0.8.0;
 /// purchased.
 /// @custom:member packedBools Packed boolean flags: allowOwnerMint, transfersPausable, useVotingUnits,
 /// cantBeRemoved, cantIncreaseDiscountPercent, cantBuyWithCredits.
-// forge-lint: disable-next-line(pascal-case-struct)
 struct JBStored721Tier {
     uint104 price;
     uint32 remainingSupply;