rename hooks

Author: jacopo-eth

.gas-snapshot

@@ -1,6 +1,6 @@
 LinearVRGDACorrectnessTest:testParamsSchema() (gas: 9581)
 LinearVRGDACorrectnessTest:testSetup_HookInitialized() (gas: 5671)
-LinearVRGDACorrectnessTest:testSupportsInterface_RegistryPricingStrategy() (gas: 9685)
+LinearVRGDACorrectnessTest:testSupportsInterface_RegistryProductPrice() (gas: 9685)
 LinearVRGDATest:testAlwaystargetPriceInRightConditions(uint256) (runs: 256, μ: 14604, ~: 14369)
 LinearVRGDATest:testParamsSchema() (gas: 9564)
 LinearVRGDATest:testPricingAdjustedByQuantity() (gas: 18788)
@@ -11,7 +11,7 @@ LinearVRGDATest:testProductPriceEth() (gas: 26435)
 LinearVRGDATest:testProductPriceMultiple() (gas: 29695)
 LinearVRGDATest:testSetMultiplePrices() (gas: 171215)
 LinearVRGDATest:testSetup_HookInitialized() (gas: 5715)
-LinearVRGDATest:testSupportsInterface_RegistryPricingStrategy() (gas: 9684)
+LinearVRGDATest:testSupportsInterface_RegistryProductPrice() (gas: 9684)
 LinearVRGDATest:testTargetPrice() (gas: 11418)
 LogisticVRGDATest:testAlwaysTargetPriceInRightConditions(uint256) (runs: 256, μ: 16809, ~: 16994)
 LogisticVRGDATest:testGetTargetSaleTimeDoesNotRevertEarly() (gas: 6630)
@@ -27,7 +27,7 @@ LogisticVRGDATest:testProductPriceEth() (gas: 28425)
 LogisticVRGDATest:testProductPriceMultiple() (gas: 34309)
 LogisticVRGDATest:testSetMultiplePrices() (gas: 181254)
 LogisticVRGDATest:testSetup_HookInitialized() (gas: 5693)
-LogisticVRGDATest:testSupportsInterface_RegistryPricingStrategy() (gas: 9752)
+LogisticVRGDATest:testSupportsInterface_RegistryProductPrice() (gas: 9752)
 LogisticVRGDATest:testTargetPrice() (gas: 13363)
 LogisticVRGDATest:test_RevertOverflow_BeyondLimitTokens(uint256,uint256) (runs: 256, μ: 14963, ~: 14978)
 NFTDiscountTest:testDeploy() (gas: 5246)
@@ -44,4 +44,4 @@ NFTDiscountTest:testSetProductPrice__Edit_Add() (gas: 249809)
 NFTDiscountTest:testSetProductPrice__Edit_Remove() (gas: 230519)
 NFTDiscountTest:testSetProductPrice__MultipleCurrencies() (gas: 198172)
 NFTDiscountTest:testSetup_HookInitialized() (gas: 5715)
-NFTDiscountTest:testSupportsInterface_RegistryPricingStrategy() (gas: 9705)
+NFTDiscountTest:testSupportsInterface_RegistryProductPrice() (gas: 9705)

README.md

@@ -21,8 +21,8 @@ src/
 
 Hooks are built around three main interfaces:
 
-- **[`IOnchainAction`](./src/interfaces/IOnchainAction.sol)**: Execute custom logic during purchases (eligibility checks, rewards, etc.)
-- **[`IPricingStrategy`](./src/interfaces/IPricingStrategy.sol)**: Calculate dynamic prices for products
+- **[`IProductAction`](./src/interfaces/IProductAction.sol)**: Execute custom logic during purchases (eligibility checks, rewards, etc.)
+- **[`IProductPrice`](./src/interfaces/IProductPrice.sol)**: Calculate dynamic prices for products
 - **[`IHookRegistry`](./src/interfaces/IHookRegistry.sol)**: Enable reusable hooks across multiple products with frontend integration
 
 Hooks can be:
@@ -42,13 +42,13 @@ Here's how hooks integrate into the product purchase flow:
        ▼
 ┌─────────────────────┐
 │   Price Fetching    │ ← `productPrice` called here
-│   (before purchase) │   (IPricingStrategy)
+│   (before purchase) │   (IProductPrice)
 └─────────────────────┘
        │
        ▼
 ┌─────────────────────┐
 │  Purchase Execution │ ← `onProductPurchase` called here
-│  (during purchase)  │   (IOnchainAction)
+│  (during purchase)  │   (IProductAction)
 └─────────────────────┘
        │
        ▼
@@ -85,15 +85,15 @@ The base contracts in `src/utils` are designed to be inherited, providing essent
 
 ###  Registry (Reusable):
 
-- **`RegistryOnchainAction`**: Base for reusable onchain actions
-- **`RegistryPricingStrategy`**: Base for reusable pricing strategies  
-- **`RegistryPricingStrategyAction`**: Base for reusable pricing + action hooks
+- **`RegistryProductAction`**: Base for reusable onchain actions
+- **`RegistryProductPrice`**: Base for reusable pricing strategies  
+- **`RegistryProductPriceAction`**: Base for reusable pricing + action hooks
 
 ### Product-Specific
 
-- **`OnchainAction`**: Base for product-specific onchain actions
-- **`PricingStrategy`**: Base for product-specific pricing strategies
-- **`PricingStrategyAction`**: Base for product-specific pricing + action hooks
+- **`ProductAction`**: Base for product-specific onchain actions
+- **`ProductPrice`**: Base for product-specific pricing strategies
+- **`ProductPriceAction`**: Base for product-specific pricing + action hooks
 
 ## Quick Start
 
@@ -126,12 +126,12 @@ The script will present you with a list of available contracts to deploy. Select
 
 When writing tests for your hooks, inherit from the appropriate base test contract:
 
-- **`RegistryOnchainActionTest`**: For testing `RegistryOnchainAction` contracts
-- **`RegistryPricingStrategyTest`**: For testing `RegistryPricingStrategy` contracts
-- **`RegistryPricingStrategyActionTest`**: For testing `RegistryPricingStrategyAction` contracts
-- **`OnchainActionTest`**: For testing `OnchainAction` contracts
-- **`PricingStrategyTest`**: For testing `PricingStrategy` contracts
-- **`PricingStrategyActionTest`**: For testing `PricingStrategyAction` contracts
+- **`RegistryProductActionTest`**: For testing `RegistryProductAction` contracts
+- **`RegistryProductPriceTest`**: For testing `RegistryProductPrice` contracts
+- **`RegistryProductPriceActionTest`**: For testing `RegistryProductPriceAction` contracts
+- **`ProductActionTest`**: For testing `ProductAction` contracts
+- **`ProductPriceTest`**: For testing `ProductPrice` contracts
+- **`ProductPriceActionTest`**: For testing `ProductPriceAction` contracts
 
 Inheriting the appropriate test contract for your hook allows you to focus your tests solely on your custom hook logic.
 

src/examples/README.md

@@ -4,9 +4,9 @@ This folder contains product-specific smart contract implementations that demons
 
 ## Key Interfaces
 
-**IPricingStrategy**:
+**IProductPrice**:
 ```solidity
-interface IPricingStrategy {
+interface IProductPrice {
     function productPrice(
         uint256 slicerId,
         uint256 productId,
@@ -18,9 +18,9 @@ interface IPricingStrategy {
 }
 ```
 
-**IOnchainAction**:
+**IProductAction**:
 ```solidity
-interface IOnchainAction {
+interface IProductAction {
     function isPurchaseAllowed(
         uint256 slicerId,
         uint256 productId,
@@ -43,15 +43,15 @@ interface IOnchainAction {
 
 ## Base Contracts
 
-- **OnchainAction**: Add arbitrary requirements and/or custom logic after product purchase.
-- **PricingStrategy**: Customize product pricing logic.
-- **PricingStrategyAction**: Provide functionality of both Onchain Actions and Pricing Strategies
+- **ProductAction**: Add arbitrary requirements and/or custom logic after product purchase.
+- **ProductPrice**: Customize product pricing logic.
+- **ProductPriceAction**: Provide functionality of both Onchain Actions and Pricing Strategies
 
 ## Key Differences from Registry Hooks
 
 Unlike the reusable hooks in `/hooks/`, these examples:
 - Are tailored for specific products/projects
-- Inherit directly from base contracts (`OnchainAction`, `PricingStrategy`)
+- Inherit directly from base contracts (`ProductAction`, `ProductPrice`)
 - Don't implement `IHookRegistry` (not intended for Slice frontend integration)
 - Serve as reference implementations and starting points
 
@@ -68,13 +68,13 @@ Unlike the reusable hooks in `/hooks/`, these examples:
 
 To create a custom product-specific onchain action:
 
-1. **Inherit from OnchainAction**:
+1. **Inherit from ProductAction**:
 ```solidity
-import {OnchainAction, IProductsModule} from "@/utils/OnchainAction.sol";
+import {ProductAction, IProductsModule} from "@/utils/ProductAction.sol";
 
-contract MyProductAction is OnchainAction {
+contract MyProductAction is ProductAction {
     constructor(IProductsModule productsModule, uint256 slicerId)
-        OnchainAction(productsModule, slicerId) {}
+        ProductAction(productsModule, slicerId) {}
 }
 ```
 
@@ -108,13 +108,13 @@ function isPurchaseAllowed(
 
 To create a custom product-specific pricing strategy:
 
-1. **Inherit from PricingStrategy**:
+1. **Inherit from ProductPrice**:
 ```solidity
-import {PricingStrategy, IProductsModule} from "@/utils/PricingStrategy.sol";
+import {ProductPrice, IProductsModule} from "@/utils/ProductPrice.sol";
 
-contract MyProductAction is PricingStrategy {
+contract MyProductAction is ProductPrice {
     constructor(IProductsModule productsModule, uint256 slicerId)
-        PricingStrategy(productsModule, slicerId) {}
+        ProductPrice(productsModule, slicerId) {}
 }
 ```
 

src/examples/actions/BaseCafe_2.sol

@@ -1,14 +1,14 @@
 // SPDX-License-Identifier: MIT
 pragma solidity ^0.8.20;
 
-import {IProductsModule, OnchainAction} from "@/utils/OnchainAction.sol";
+import {IProductsModule, ProductAction} from "@/utils/ProductAction.sol";
 
 /**
  * @title BaseCafe
  * @notice Onchain action that mints an NFT to the buyer on every purchase.
  * @author Slice <jacopo.eth>
  */
-contract BaseCafe is OnchainAction {
+contract BaseCafe is ProductAction {
     /*//////////////////////////////////////////////////////////////
         IMMUTABLE STORAGE
     //////////////////////////////////////////////////////////////*/
@@ -21,15 +21,15 @@ contract BaseCafe is OnchainAction {
     //////////////////////////////////////////////////////////////*/
 
     constructor(IProductsModule productsModuleAddress, uint256 slicerId)
-        OnchainAction(productsModuleAddress, slicerId)
+        ProductAction(productsModuleAddress, slicerId)
     {}
 
     /*//////////////////////////////////////////////////////////////
         FUNCTIONS
     //////////////////////////////////////////////////////////////*/
 
     /**
-     * @inheritdoc OnchainAction
+     * @inheritdoc ProductAction
      * @notice Mint `quantity` NFTs to `account` on purchase
      */
     function _onProductPurchase(uint256, uint256, address buyer, uint256 quantity, bytes memory, bytes memory)

src/examples/actions/BaseGirlsScout.sol

@@ -1,7 +1,7 @@
 // SPDX-License-Identifier: MIT
 pragma solidity ^0.8.20;
 
-import {IProductsModule, OnchainAction} from "@/utils/OnchainAction.sol";
+import {IProductsModule, ProductAction} from "@/utils/ProductAction.sol";
 import {Ownable} from "@openzeppelin-4.8.0/access/Ownable.sol";
 import {IERC1155} from "@openzeppelin-4.8.0/interfaces/IERC1155.sol";
 
@@ -10,7 +10,7 @@ import {IERC1155} from "@openzeppelin-4.8.0/interfaces/IERC1155.sol";
  * @notice Onchain action that mints Base Girls Scout NFTs to the buyer on every purchase.
  * @author Slice <jacopo.eth>
  */
-contract BaseGirlsScout is OnchainAction, Ownable {
+contract BaseGirlsScout is ProductAction, Ownable {
     /*//////////////////////////////////////////////////////////////
         IMMUTABLE STORAGE
     //////////////////////////////////////////////////////////////*/
@@ -29,7 +29,7 @@ contract BaseGirlsScout is OnchainAction, Ownable {
     //////////////////////////////////////////////////////////////*/
 
     constructor(IProductsModule productsModuleAddress, uint256 slicerId)
-        OnchainAction(productsModuleAddress, slicerId)
+        ProductAction(productsModuleAddress, slicerId)
         Ownable()
     {
         allowedSlicerIds[2217] = true;
@@ -41,7 +41,7 @@ contract BaseGirlsScout is OnchainAction, Ownable {
     //////////////////////////////////////////////////////////////*/
 
     /**
-     * @inheritdoc OnchainAction
+     * @inheritdoc ProductAction
      * @notice Mint `quantity` NFTs to `account` on purchase
      */
     function _onProductPurchase(uint256, uint256, address buyer, uint256 quantity, bytes memory, bytes memory)

src/hooks/actions/Allowlisted/Allowlisted.sol

@@ -4,18 +4,18 @@ pragma solidity ^0.8.20;
 import {MerkleProof} from "@openzeppelin-4.8.0/utils/cryptography/MerkleProof.sol";
 import {
     IProductsModule,
-    RegistryOnchainAction,
+    RegistryProductAction,
     HookRegistry,
-    IOnchainAction,
+    IProductAction,
     IHookRegistry
-} from "@/utils/RegistryOnchainAction.sol";
+} from "@/utils/RegistryProductAction.sol";
 
 /**
  * @title   Allowlisted
  * @notice  Onchain action registry for allowlist requirement.
  * @author  Slice <jacopo.eth>
  */
-contract Allowlisted is RegistryOnchainAction {
+contract Allowlisted is RegistryProductAction {
     /*//////////////////////////////////////////////////////////////
         MUTABLE STORAGE
     //////////////////////////////////////////////////////////////*/
@@ -26,14 +26,14 @@ contract Allowlisted is RegistryOnchainAction {
         CONSTRUCTOR
     //////////////////////////////////////////////////////////////*/
 
-    constructor(IProductsModule productsModuleAddress) RegistryOnchainAction(productsModuleAddress) {}
+    constructor(IProductsModule productsModuleAddress) RegistryProductAction(productsModuleAddress) {}
 
     /*//////////////////////////////////////////////////////////////
         CONFIGURATION
     //////////////////////////////////////////////////////////////*/
 
     /**
-     * @inheritdoc IOnchainAction
+     * @inheritdoc IProductAction
      * @dev Checks if the account is in the allowlist.
      */
     function isPurchaseAllowed(

src/hooks/actions/ERC20Gated/ERC20Gated.sol

@@ -4,18 +4,18 @@ pragma solidity ^0.8.20;
 import {ERC20Gate} from "./types/ERC20Gate.sol";
 import {
     IProductsModule,
-    RegistryOnchainAction,
+    RegistryProductAction,
     HookRegistry,
-    IOnchainAction,
+    IProductAction,
     IHookRegistry
-} from "@/utils/RegistryOnchainAction.sol";
+} from "@/utils/RegistryProductAction.sol";
 
 /**
  * @title   ERC20Gated
  * @notice  Onchain action registry for ERC20 gating.
  * @author  Slice <jacopo.eth>
  */
-contract ERC20Gated is RegistryOnchainAction {
+contract ERC20Gated is RegistryProductAction {
     /*//////////////////////////////////////////////////////////////
         MUTABLE STORAGE
     //////////////////////////////////////////////////////////////*/
@@ -26,14 +26,14 @@ contract ERC20Gated is RegistryOnchainAction {
         CONSTRUCTOR
     //////////////////////////////////////////////////////////////*/
 
-    constructor(IProductsModule productsModuleAddress) RegistryOnchainAction(productsModuleAddress) {}
+    constructor(IProductsModule productsModuleAddress) RegistryProductAction(productsModuleAddress) {}
 
     /*//////////////////////////////////////////////////////////////
         CONFIGURATION
     //////////////////////////////////////////////////////////////*/
 
     /**
-     * @inheritdoc IOnchainAction
+     * @inheritdoc IProductAction
      * @dev Checks if `account` owns the required amount of all ERC20 tokens.
      */
     function isPurchaseAllowed(

src/hooks/actions/ERC20Mint/ERC20Mint.sol

@@ -3,11 +3,11 @@ pragma solidity ^0.8.20;
 
 import {
     IProductsModule,
-    RegistryOnchainAction,
+    RegistryProductAction,
     HookRegistry,
-    IOnchainAction,
+    IProductAction,
     IHookRegistry
-} from "@/utils/RegistryOnchainAction.sol";
+} from "@/utils/RegistryProductAction.sol";
 import {ERC20Data} from "./types/ERC20Data.sol";
 import {ERC20Mint_BaseToken} from "./utils/ERC20Mint_BaseToken.sol";
 
@@ -17,7 +17,7 @@ import {ERC20Mint_BaseToken} from "./utils/ERC20Mint_BaseToken.sol";
  * @dev     If `revertOnMaxSupplyReached` is set to true, reverts when max supply is exceeded.
  * @author  Slice <jacopo.eth>
  */
-contract ERC20Mint is RegistryOnchainAction {
+contract ERC20Mint is RegistryProductAction {
     /*//////////////////////////////////////////////////////////////
         ERRORS
     //////////////////////////////////////////////////////////////*/
@@ -34,14 +34,14 @@ contract ERC20Mint is RegistryOnchainAction {
         CONSTRUCTOR
     //////////////////////////////////////////////////////////////*/
 
-    constructor(IProductsModule productsModuleAddress) RegistryOnchainAction(productsModuleAddress) {}
+    constructor(IProductsModule productsModuleAddress) RegistryProductAction(productsModuleAddress) {}
 
     /*//////////////////////////////////////////////////////////////
         CONFIGURATION
     //////////////////////////////////////////////////////////////*/
 
     /**
-     * @inheritdoc IOnchainAction
+     * @inheritdoc IProductAction
      * @dev If `revertOnMaxSupplyReached` is set to true, returns false when max supply is exceeded.
      */
     function isPurchaseAllowed(
@@ -63,7 +63,7 @@ contract ERC20Mint is RegistryOnchainAction {
     }
 
     /**
-     * @inheritdoc RegistryOnchainAction
+     * @inheritdoc RegistryProductAction
      * @notice Mint tokens to the buyer.
      * @dev If `revertOnMaxSupplyReached` is set to true, reverts when max supply is exceeded.
      */