@zoralabs/coins 2.2.1 → 2.3.0

package/src/hook-registry/ZoraHookRegistry.sol

@@ -23,6 +23,10 @@ contract ZoraHookRegistry is IZoraHookRegistry, MultiOwnable {
     /// @dev The tag for each hook
     mapping(address hook => string tag) internal hookTags;
 
+    /// @notice Constructor for deployment pathway
+    /// @dev This contract needs to be atomically initialized, otherwise it is unsafe to deploy.
+    /// @dev Initial owners need to be automatically set by the deployer contract.
+    /// @dev The initial owners is not a part of the constructor() to allow for multichain deterministic addresses.
     constructor() {}
 
     /// @notice Initializes the registry with initial owners

package/src/hooks/ZoraV4CoinHook.sol

@@ -31,13 +31,14 @@ import {IUpgradeableV4Hook} from "../interfaces/IUpgradeableV4Hook.sol";
 import {IHooksUpgradeGate} from "../interfaces/IHooksUpgradeGate.sol";
 import {MultiOwnable} from "../utils/MultiOwnable.sol";
 import {ERC165} from "@openzeppelin/contracts/utils/introspection/ERC165.sol";
-import {IUpgradeableDestinationV4Hook} from "../interfaces/IUpgradeableV4Hook.sol";
+import {IUpgradeableDestinationV4Hook, IUpgradeableDestinationV4HookWithUpdateableFee} from "../interfaces/IUpgradeableV4Hook.sol";
 import {IHooks} from "@uniswap/v4-core/src/interfaces/IHooks.sol";
 import {BurnedPosition} from "../interfaces/IUpgradeableV4Hook.sol";
 import {LiquidityAmounts} from "../utils/uniswap/LiquidityAmounts.sol";
 import {TickMath} from "../utils/uniswap/TickMath.sol";
 import {ContractVersionBase, IVersionedContract} from "../version/ContractVersionBase.sol";
 import {IHasCoinType} from "../interfaces/ICoin.sol";
+import {CoinConstants} from "../libs/CoinConstants.sol";
 
 /// @title ZoraV4CoinHook
 /// @notice Uniswap V4 hook that automatically handles fee collection and reward distributions on every swap,
@@ -49,7 +50,14 @@ import {IHasCoinType} from "../interfaces/ICoin.sol";
 ///      2. Swaps collected fees to the backing currency through multi-hop paths
 ///      3. Distributes converted fees as rewards
 /// @author oveddan
-contract ZoraV4CoinHook is BaseHook, ContractVersionBase, IZoraV4CoinHook, ERC165, IUpgradeableDestinationV4Hook {
+contract ZoraV4CoinHook is
+    BaseHook,
+    ContractVersionBase,
+    IZoraV4CoinHook,
+    ERC165,
+    IUpgradeableDestinationV4Hook,
+    IUpgradeableDestinationV4HookWithUpdateableFee
+{
     using BalanceDeltaLibrary for BalanceDelta;
 
     /// @notice Mapping of trusted message senders - these are addresses that are trusted to provide a
@@ -130,6 +138,7 @@ contract ZoraV4CoinHook is BaseHook, ContractVersionBase, IZoraV4CoinHook, ERC16
         return
             super.supportsInterface(interfaceId) ||
             interfaceId == type(IUpgradeableDestinationV4Hook).interfaceId ||
+            interfaceId == type(IUpgradeableDestinationV4HookWithUpdateableFee).interfaceId ||
             interfaceId == type(IVersionedContract).interfaceId;
     }
 
@@ -137,10 +146,18 @@ contract ZoraV4CoinHook is BaseHook, ContractVersionBase, IZoraV4CoinHook, ERC16
     /// @param coin The coin address.
     /// @param key The pool key for the coin.
     /// @return positions The contract-created liquidity positions the positions for the coin's pool.
-    function _generatePositions(ICoin coin, PoolKey memory key) internal view returns (LpPosition[] memory positions) {
+    function _generatePositions(ICoin coin, PoolKey memory key) internal view returns (LpPosition[] memory) {
         bool isCoinToken0 = Currency.unwrap(key.currency0) == address(coin);
 
-        positions = CoinDopplerMultiCurve.calculatePositions(isCoinToken0, coin.getPoolConfiguration(), coin.totalSupplyForPositions());
+        LpPosition[] memory calculatedPositions = CoinDopplerMultiCurve.calculatePositions(
+            isCoinToken0,
+            coin.getPoolConfiguration(),
+            coin.totalSupplyForPositions()
+        );
+
+        // sometimes the calculated positions have liquidity added in duplicated positions.   So here we dedupe them
+        // to save on gas in future swaps.
+        return V4Liquidity.dedupePositions(calculatedPositions);
     }
 
     /// @notice Internal fn called when a pool is initialized.
@@ -168,17 +185,50 @@ contract ZoraV4CoinHook is BaseHook, ContractVersionBase, IZoraV4CoinHook, ERC16
     }
 
     /// @inheritdoc IUpgradeableDestinationV4Hook
+    /// @dev left for backward compatibility for migrating from hooks that dont support
+    /// updating the fee
     function initializeFromMigration(
         PoolKey calldata poolKey,
         address coin,
         uint160 sqrtPriceX96,
         BurnedPosition[] calldata migratedLiquidity,
-        bytes calldata
+        bytes calldata additionalData
     ) external {
+        // keep the existing fee and tick spacing.
+        uint24 fee = poolKey.fee;
+        int24 tickSpacing = poolKey.tickSpacing;
+
+        _initializeFromMigration(poolKey, coin, sqrtPriceX96, migratedLiquidity, additionalData, fee, tickSpacing);
+    }
+
+    /// @inheritdoc IUpgradeableDestinationV4HookWithUpdateableFee
+    function initializeFromMigrationWithUpdateableFee(
+        PoolKey calldata poolKey,
+        address coin,
+        uint160 sqrtPriceX96,
+        BurnedPosition[] calldata migratedLiquidity,
+        bytes calldata additionalData
+    ) external returns (uint24 fee, int24 tickSpacing) {
+        // update the fee to the current one.
+        fee = CoinConstants.LP_FEE_V4;
+        tickSpacing = CoinConstants.TICK_SPACING;
+
+        _initializeFromMigration(poolKey, coin, sqrtPriceX96, migratedLiquidity, additionalData, fee, tickSpacing);
+    }
+
+    function _initializeFromMigration(
+        PoolKey calldata poolKey,
+        address coin,
+        uint160 sqrtPriceX96,
+        BurnedPosition[] calldata migratedLiquidity,
+        bytes calldata,
+        uint24 fee,
+        int24 tickSpacing
+    ) internal {
         address oldHook = msg.sender;
         address newHook = address(this);
 
-        // Verify that the caller (new hook) is authorized to perform this migration
+        // Verify that the caller (old hook) is authorized to perform this migration
         // Only registered upgrade paths in the upgrade gate are allowed to migrate liquidity
         if (!upgradeGate.isRegisteredUpgradePath(oldHook, newHook)) {
             revert IUpgradeableV4Hook.UpgradePathNotRegistered(oldHook, newHook);
@@ -189,8 +239,8 @@ contract ZoraV4CoinHook is BaseHook, ContractVersionBase, IZoraV4CoinHook, ERC16
         PoolKey memory newKey = PoolKey({
             currency0: poolKey.currency0,
             currency1: poolKey.currency1,
-            fee: poolKey.fee,
-            tickSpacing: poolKey.tickSpacing,
+            fee: fee,
+            tickSpacing: tickSpacing,
             hooks: IHooks(newHook)
         });
 
@@ -343,7 +393,7 @@ contract ZoraV4CoinHook is BaseHook, ContractVersionBase, IZoraV4CoinHook, ERC16
         return CoinRewardsV4.getCoinType(coin);
     }
 
-    /// @notice Internal fn called when the PoolManager is unlocked.  Used to mint initial liquidity positions.
+    /// @notice Internal fn called when the PoolManager is unlocked. Used to mint initial liquidity positions and burn positions during migration.
     function unlockCallback(bytes calldata data) external onlyPoolManager returns (bytes memory) {
         return V4Liquidity.handleCallback(poolManager, data);
     }
@@ -376,7 +426,14 @@ contract ZoraV4CoinHook is BaseHook, ContractVersionBase, IZoraV4CoinHook, ERC16
         }
 
         newPoolKey = V4Liquidity.lockAndMigrate(poolManager, poolKey, poolCoin.positions, poolCoin.coin, newHook, additionalData);
+
+        // Delete the old pool key mapping to prevent future operations on the migrated pool
+        delete poolCoins[poolKeyHash];
     }
 
+    /// @notice Receives ETH from the pool manager for ETH-backed coins during fee collection.
+    /// @dev Only required for coins using ETH as backing currency (currency = address(0)).
+    ///      Restricted to onlyPoolManager to prevent ETH from getting stuck in the contract.
+    ///      Unused for ERC20-backed coins.
     receive() external payable onlyPoolManager {}
 }

package/src/interfaces/IUpgradeableV4Hook.sol

@@ -50,3 +50,21 @@ interface IUpgradeableDestinationV4Hook {
         bytes calldata additionalData
     ) external;
 }
+
+interface IUpgradeableDestinationV4HookWithUpdateableFee {
+    /// @notice Initialize after migration from old hook
+    /// @param poolKey The pool key being migrated
+    /// @param coin The coin address
+    /// @param sqrtPriceX96 The current sqrt price
+    /// @param migratedLiquidity The migrated liquidity
+    /// @param additionalData Additional data for initialization
+    /// @return fee The new fee for the migrated liquidity
+    /// @return tickSpacing The new tick spacing for the migrated liquidity
+    function initializeFromMigrationWithUpdateableFee(
+        PoolKey calldata poolKey,
+        address coin,
+        uint160 sqrtPriceX96,
+        BurnedPosition[] calldata migratedLiquidity,
+        bytes calldata additionalData
+    ) external returns (uint24 fee, int24 tickSpacing);
+}

package/src/interfaces/IZoraFactory.sol

@@ -97,7 +97,20 @@ interface IZoraFactory is IDeployedCoinVersionLookup {
     /// @notice Thrwon when an invalid config version is provided
     error InvalidConfig();
 
-    /// @notice Creates a new creator coin contract
+    /// @dev Deprecated: use `deployCreatorCoin` instead that has a salt and post-deploy hook specified
+    function deployCreatorCoin(
+        address payoutRecipient,
+        address[] memory owners,
+        string memory uri,
+        string memory name,
+        string memory symbol,
+        bytes memory poolConfig,
+        address platformReferrer,
+        bytes32 coinSalt
+    ) external returns (address);
+
+    /// @notice Creates a new creator coin contract with an optional hook that runs after the coin is deployed.
+    /// Enables buying initial supply by supporting ETH transfers to the post-deploy hook.
     /// @param payoutRecipient The recipient of creator reward payouts; this can be updated by an owner
     /// @param owners The list of addresses that will be able to manage the coin's payout address and metadata uri
     /// @param uri The coin metadata uri
@@ -105,7 +118,11 @@ interface IZoraFactory is IDeployedCoinVersionLookup {
     /// @param symbol The symbol of the coin
     /// @param poolConfig The config parameters for the coin's pool
     /// @param platformReferrer The address of the platform referrer
+    /// @param postDeployHook The address of the hook to run after the coin is deployed
+    /// @param postDeployHookData The data to pass to the hook
     /// @param coinSalt The salt used to deploy the coin
+    /// @return coin The address of the deployed creator coin
+    /// @return postDeployHookDataOut The data returned by the hook
     function deployCreatorCoin(
         address payoutRecipient,
         address[] memory owners,
@@ -114,8 +131,10 @@ interface IZoraFactory is IDeployedCoinVersionLookup {
         string memory symbol,
         bytes memory poolConfig,
         address platformReferrer,
+        address postDeployHook,
+        bytes calldata postDeployHookData,
         bytes32 coinSalt
-    ) external returns (address);
+    ) external payable returns (address coin, bytes memory postDeployHookDataOut);
 
     /// @notice Creates a new coin contract with an optional hook that runs after the coin is deployed.
     /// Requires a salt to be specified, which enabled the coin to be deployed deterministically, and at

package/src/libs/CoinConstants.sol

@@ -8,21 +8,64 @@
 pragma solidity ^0.8.23;
 
 library CoinConstants {
+    /// @dev Constant used to increase precision during calculations
+    uint256 internal constant WAD = 1e18;
+
     /// @notice The maximum total supply
     /// @dev Set to 1 billion coins with 18 decimals
-    uint256 public constant MAX_TOTAL_SUPPLY = 1_000_000_000e18;
+    uint256 internal constant MAX_TOTAL_SUPPLY = 1_000_000_000e18;
+
+    /// @notice The total supply for creator coins (same as MAX_TOTAL_SUPPLY)
+    /// @dev 1 billion coins
+    uint256 internal constant TOTAL_SUPPLY = 1_000_000_000e18;
 
-    /// @notice The number of coins allocated to the liquidity pool
+    /// @notice The number of coins allocated to the liquidity pool for content coins
     /// @dev 990 million coins
-    uint256 public constant POOL_LAUNCH_SUPPLY = 990_000_000e18;
+    uint256 internal constant CONTENT_COIN_MARKET_SUPPLY = 990_000_000e18;
+
+    /// @notice The number of coins allocated to the liquidity pool for creator coins
+    /// @dev 500 million coins
+    uint256 internal constant CREATOR_COIN_MARKET_SUPPLY = 500_000_000e18;
 
-    /// @notice The number of coins rewarded to the creator
+    /// @notice The number of coins rewarded to the creator for content coins on launch
     /// @dev 10 million coins
-    uint256 public constant CREATOR_LAUNCH_REWARD = 10_000_000e18;
+    uint256 internal constant CONTENT_COIN_INITIAL_CREATOR_SUPPLY = TOTAL_SUPPLY - CONTENT_COIN_MARKET_SUPPLY;
+
+    /// @notice Creator coin vesting supply for creator
+    /// @dev 500 million coins
+    uint256 internal constant CREATOR_COIN_CREATOR_VESTING_SUPPLY = TOTAL_SUPPLY - CREATOR_COIN_MARKET_SUPPLY;
+
+    /// @notice Creator coin vesting duration
+    /// @dev 5 years with leap years accounted for
+    uint256 internal constant CREATOR_VESTING_DURATION = (5 * 365.25 days);
+
+    /// @notice The backing currency for creator coins
+    /// @dev ETH backing currency address
+    address internal constant CREATOR_COIN_CURRENCY = 0x1111111111166b7FE7bd91427724B487980aFc69;
+
+    /// @notice The LP fee
+    /// @dev 10000 basis points = 1%
+    uint24 internal constant LP_FEE_V4 = 10_000;
+
+    /// @notice The spacing for 1% pools
+    /// @dev 200 ticks
+    int24 internal constant TICK_SPACING = 200;
+
+    // Creator gets 62.5% of market rewards (0.50% of total 1% fee)
+    // Market rewards = 80% of total fee (0.80% of 1%)
+    uint256 internal constant CREATOR_REWARD_BPS = 6250;
+
+    // Platform referrer gets 25% of market rewards (0.20% of total 1% fee)
+    uint256 internal constant CREATE_REFERRAL_REWARD_BPS = 2500;
+
+    // Trade referrer gets 5% of market rewards (0.04% of total 1% fee)
+    uint256 internal constant TRADE_REFERRAL_REWARD_BPS = 500;
+
+    // Doppler gets 1.25% of market rewards (0.01% of total 1% fee)
+    uint256 internal constant DOPPLER_REWARD_BPS = 125;
 
-    /// @notice The minimum order size allowed for trades
-    /// @dev Set to 0.0000001 ETH to prevent dust transactions
-    uint256 public constant MIN_ORDER_SIZE = 0.0000001 ether;
+    // LPs get 20% of total fee (0.20% of 1%)
+    uint256 internal constant LP_REWARD_BPS = 2000;
 
     int24 internal constant DEFAULT_DISCOVERY_TICK_LOWER = -777000;
     int24 internal constant DEFAULT_DISCOVERY_TICK_UPPER = 222000;

package/src/libs/CoinDopplerMultiCurve.sol

@@ -10,7 +10,7 @@ pragma solidity ^0.8.23;
 import {PoolConfiguration} from "../interfaces/ICoin.sol";
 import {CoinConfigurationVersions} from "./CoinConfigurationVersions.sol";
 import {LpPosition} from "../types/LpPosition.sol";
-import {MarketConstants} from "./MarketConstants.sol";
+import {CoinConstants} from "./CoinConstants.sol";
 import {FullMath} from "../utils/uniswap/FullMath.sol";
 import {TickMath} from "../utils/uniswap/TickMath.sol";
 import {IDopplerErrors} from "../interfaces/IDopplerErrors.sol";
@@ -56,8 +56,8 @@ library CoinDopplerMultiCurve {
         uint256 totalDiscoverySupplyShare;
         uint256 totalDiscoveryPositions;
 
-        int24 boundryTickLower = DopplerMath.alignTickToTickSpacing(isCoinToken0, TickMath.MAX_TICK, MarketConstants.TICK_SPACING);
-        int24 boundryTickUpper = DopplerMath.alignTickToTickSpacing(isCoinToken0, TickMath.MIN_TICK, MarketConstants.TICK_SPACING);
+        int24 boundryTickLower = DopplerMath.alignTickToTickSpacing(isCoinToken0, TickMath.MAX_TICK, CoinConstants.TICK_SPACING);
+        int24 boundryTickUpper = DopplerMath.alignTickToTickSpacing(isCoinToken0, TickMath.MIN_TICK, CoinConstants.TICK_SPACING);
 
         // For each curve:
         for (uint256 i; i < numCurves; i++) {
@@ -69,8 +69,8 @@ library CoinDopplerMultiCurve {
             totalDiscoveryPositions += numDiscoveryPositions_[i];
             totalDiscoverySupplyShare += maxDiscoverySupplyShare_[i];
 
-            int24 currentTickLower = DopplerMath.alignTickToTickSpacing(isCoinToken0, tickLower_[i], MarketConstants.TICK_SPACING);
-            int24 currentTickUpper = DopplerMath.alignTickToTickSpacing(isCoinToken0, tickUpper_[i], MarketConstants.TICK_SPACING);
+            int24 currentTickLower = DopplerMath.alignTickToTickSpacing(isCoinToken0, tickLower_[i], CoinConstants.TICK_SPACING);
+            int24 currentTickUpper = DopplerMath.alignTickToTickSpacing(isCoinToken0, tickUpper_[i], CoinConstants.TICK_SPACING);
 
             require(currentTickLower < currentTickUpper, ConfigTickLowerMustBeLessThanTickUpper());
 
@@ -84,15 +84,15 @@ library CoinDopplerMultiCurve {
 
         require(boundryTickLower < boundryTickUpper, InvalidTickRangeMisordered(boundryTickLower, boundryTickUpper));
         require(totalDiscoveryPositions > 1 && totalDiscoveryPositions <= 200, IDopplerErrors.NumDiscoveryPositionsOutOfRange());
-        require(totalDiscoverySupplyShare < MarketConstants.WAD, IDopplerErrors.MaxShareToBeSoldExceeded(totalDiscoverySupplyShare, MarketConstants.WAD));
+        require(totalDiscoverySupplyShare < CoinConstants.WAD, IDopplerErrors.MaxShareToBeSoldExceeded(totalDiscoverySupplyShare, CoinConstants.WAD));
 
         sqrtPriceX96 = TickMath.getSqrtPriceAtTick(isCoinToken0 ? boundryTickLower : boundryTickUpper);
 
         poolConfiguration = PoolConfiguration({
             version: CoinConfigurationVersions.DOPPLER_MULTICURVE_UNI_V4_POOL_VERSION,
             numPositions: uint16(totalDiscoveryPositions + 1), // Add one for the final tail position
-            fee: MarketConstants.LP_FEE_V4,
-            tickSpacing: MarketConstants.TICK_SPACING,
+            fee: CoinConstants.LP_FEE_V4,
+            tickSpacing: CoinConstants.TICK_SPACING,
             numDiscoveryPositions: numDiscoveryPositions_,
             tickLower: tickLower_,
             tickUpper: tickUpper_,
@@ -113,12 +113,12 @@ library CoinDopplerMultiCurve {
         uint256 numCurves = poolConfiguration.tickLower.length;
 
         for (uint256 i; i < numCurves; i++) {
-            uint256 curveSupply = FullMath.mulDiv(totalSupply, poolConfiguration.maxDiscoverySupplyShare[i], MarketConstants.WAD);
+            uint256 curveSupply = FullMath.mulDiv(totalSupply, poolConfiguration.maxDiscoverySupplyShare[i], CoinConstants.WAD);
 
             (positions, curveSupply) = DopplerMath.calculateLogNormalDistribution(
                 poolConfiguration.tickLower[i],
                 poolConfiguration.tickUpper[i],
-                MarketConstants.TICK_SPACING,
+                CoinConstants.TICK_SPACING,
                 isCoinToken0,
                 curveSupply,
                 poolConfiguration.numDiscoveryPositions[i],
@@ -138,7 +138,7 @@ library CoinDopplerMultiCurve {
             poolConfiguration.tickUpper[numCurves - 1],
             isCoinToken0,
             tailSupply,
-            MarketConstants.TICK_SPACING
+            CoinConstants.TICK_SPACING
         );
     }
 }

package/src/libs/CoinRewardsV4.sol

@@ -33,26 +33,11 @@ import {ICreatorCoinHook} from "../interfaces/ICreatorCoinHook.sol";
 import {IHasCoinType} from "../interfaces/ICoin.sol";
 import {IERC165} from "@openzeppelin/contracts/utils/introspection/IERC165.sol";
 import {ICreatorCoin} from "../interfaces/ICreatorCoin.sol";
+import {CoinConstants} from "./CoinConstants.sol";
 
 library CoinRewardsV4 {
     using SafeERC20 for IERC20;
 
-    // Creator gets 62.5% of market rewards (0.50% of total 1% fee)
-    // Market rewards = 80% of total fee (0.80% of 1%)
-    uint256 public constant CREATOR_REWARD_BPS = 6250;
-
-    // Platform referrer gets 25% of market rewards (0.20% of total 1% fee)
-    uint256 public constant CREATE_REFERRAL_REWARD_BPS = 2500;
-
-    // Trade referrer gets 5% of market rewards (0.04% of total 1% fee)
-    uint256 public constant TRADE_REFERRAL_REWARD_BPS = 500;
-
-    // Doppler gets 1.25% of market rewards (0.01% of total 1% fee)
-    uint256 public constant DOPPLER_REWARD_BPS = 125;
-
-    // LPs get 20% of total fee (0.20% of 1%)
-    uint256 public constant LP_REWARD_BPS = 2000;
-
     function getTradeReferral(bytes calldata hookData) internal pure returns (address) {
         return hookData.length >= 20 ? abi.decode(hookData, (address)) : address(0);
     }
@@ -83,13 +68,16 @@ library CoinRewardsV4 {
 
     /// @dev Computes the LP reward and remaining amount for market rewards from the total amount
     function computeLpReward(uint128 totalBackingAmount) internal pure returns (uint128 lpRewardAmount) {
-        lpRewardAmount = uint128(calculateReward(uint256(totalBackingAmount), LP_REWARD_BPS));
+        lpRewardAmount = uint128(calculateReward(uint256(totalBackingAmount), CoinConstants.LP_REWARD_BPS));
     }
 
     function convertDeltaToPositiveUint128(int256 delta) internal pure returns (uint128) {
         if (delta < 0) {
             revert SafeCast.SafeCastOverflow();
         }
+        if (delta > int256(uint256(type(uint128).max))) {
+            revert SafeCast.SafeCastOverflow();
+        }
         return uint128(uint256(delta));
     }
 
@@ -181,6 +169,7 @@ library CoinRewardsV4 {
     /// @param fees The total amount of fees collected to be distributed
     /// @param coin The coin contract instance that implements IHasRewardsRecipients to get recipient addresses
     /// @param tradeReferrer The address of the trade referrer who should receive trade referral rewards (can be zero address)
+    /// @param coinType The type of coin (Creator or Content) which affects reward distribution percentages
     function distributeMarketRewards(
         Currency currency,
         uint128 fees,
@@ -258,26 +247,30 @@ library CoinRewardsV4 {
     ) internal returns (MarketRewards memory rewards) {
         rewards = _computeMarketRewards(fee, tradeReferral != address(0), platformReferrer != address(0));
 
-        if (platformReferrer != address(0)) {
-            _transferCurrency(currency, rewards.platformReferrerAmount, platformReferrer);
-        }
-        if (tradeReferral != address(0)) {
-            _transferCurrency(currency, rewards.tradeReferrerAmount, tradeReferral);
-        }
-        _transferCurrency(currency, rewards.creatorAmount, payoutRecipient);
-        _transferCurrency(currency, rewards.dopplerAmount, doppler);
-        _transferCurrency(currency, rewards.protocolAmount, protocolRewardRecipient);
+        // Notes on ETH transfer fallback behavior:
+        // - If the platform referrer is immutable; if it is set to an address that cannot receive ETH, it can brick swaps on the coin, as they would revert.
+        // - Both the creator recipient and trade referral are changeable, the former via updating the payout recipient on the coin, and the latter via updating the trade referrer argument when doing a swap;
+        // therefore we do not need to worry about permanently bricking swaps, and don't need a backup recipient.
+        _transferCurrency(currency, rewards.platformReferrerAmount, platformReferrer, protocolRewardRecipient);
+        _transferCurrency(currency, rewards.tradeReferrerAmount, tradeReferral, address(0));
+        _transferCurrency(currency, rewards.creatorAmount, payoutRecipient, address(0));
+        _transferCurrency(currency, rewards.dopplerAmount, doppler, protocolRewardRecipient);
+        _transferCurrency(currency, rewards.protocolAmount, protocolRewardRecipient, address(0));
     }
 
-    function _transferCurrency(Currency currency, uint256 amount, address to) internal {
-        if (amount == 0) {
+    function _transferCurrency(Currency currency, uint256 amount, address to, address backupRecipient) internal {
+        if (amount == 0 || to == address(0)) {
             return;
         }
 
         if (currency.isAddressZero()) {
             (bool success, ) = payable(to).call{value: amount}("");
             if (!success) {
-                revert ICoin.EthTransferFailed();
+                if (backupRecipient == address(0)) {
+                    revert ICoin.EthTransferFailed();
+                } else {
+                    _transferCurrency(currency, amount, backupRecipient, address(0));
+                }
             }
         } else {
             IERC20(Currency.unwrap(currency)).safeTransfer(to, amount);
@@ -290,10 +283,10 @@ library CoinRewardsV4 {
         }
 
         uint256 totalAmount = uint256(fee);
-        rewards.platformReferrerAmount = hasCreateReferral ? calculateReward(totalAmount, CREATE_REFERRAL_REWARD_BPS) : 0;
-        rewards.tradeReferrerAmount = hasTradeReferral ? calculateReward(totalAmount, TRADE_REFERRAL_REWARD_BPS) : 0;
-        rewards.creatorAmount = calculateReward(totalAmount, CREATOR_REWARD_BPS);
-        rewards.dopplerAmount = calculateReward(totalAmount, DOPPLER_REWARD_BPS);
+        rewards.platformReferrerAmount = hasCreateReferral ? calculateReward(totalAmount, CoinConstants.CREATE_REFERRAL_REWARD_BPS) : 0;
+        rewards.tradeReferrerAmount = hasTradeReferral ? calculateReward(totalAmount, CoinConstants.TRADE_REFERRAL_REWARD_BPS) : 0;
+        rewards.creatorAmount = calculateReward(totalAmount, CoinConstants.CREATOR_REWARD_BPS);
+        rewards.dopplerAmount = calculateReward(totalAmount, CoinConstants.DOPPLER_REWARD_BPS);
         rewards.protocolAmount = totalAmount - rewards.platformReferrerAmount - rewards.tradeReferrerAmount - rewards.creatorAmount - rewards.dopplerAmount;
     }
 

package/src/libs/CoinSetup.sol

@@ -11,12 +11,11 @@ import {PoolConfigurationV4} from "../interfaces/ICoin.sol";
 import {CoinConfigurationVersions} from "./CoinConfigurationVersions.sol";
 import {ICoin} from "../interfaces/ICoin.sol";
 import {CoinCommon} from "./CoinCommon.sol";
-import {MarketConstants} from "./MarketConstants.sol";
+import {CoinConstants} from "./CoinConstants.sol";
 import {TickMath} from "../utils/uniswap/TickMath.sol";
 import {IPoolManager, PoolKey, Currency, IHooks} from "@uniswap/v4-core/src/interfaces/IPoolManager.sol";
 import {Actions} from "@uniswap/v4-periphery/src/libraries/Actions.sol";
 import {IPositionManager} from "@uniswap/v4-periphery/src/interfaces/IPositionManager.sol";
-import {MarketConstants} from "./MarketConstants.sol";
 import {LpPosition} from "../types/LpPosition.sol";
 import {CoinDopplerMultiCurve, PoolConfiguration} from "./CoinDopplerMultiCurve.sol";
 
@@ -37,13 +36,7 @@ library CoinSetup {
         Currency currency0 = isCoinToken0 ? Currency.wrap(coin) : Currency.wrap(currency);
         Currency currency1 = isCoinToken0 ? Currency.wrap(currency) : Currency.wrap(coin);
 
-        poolKey = PoolKey({
-            currency0: currency0,
-            currency1: currency1,
-            fee: MarketConstants.LP_FEE_V4,
-            tickSpacing: MarketConstants.TICK_SPACING,
-            hooks: hooks
-        });
+        poolKey = PoolKey({currency0: currency0, currency1: currency1, fee: CoinConstants.LP_FEE_V4, tickSpacing: CoinConstants.TICK_SPACING, hooks: hooks});
     }
 
     function setupPoolWithVersion(

package/src/libs/DopplerMath.sol

@@ -13,7 +13,7 @@ import {FullMath} from "../utils/uniswap/FullMath.sol";
 import {SqrtPriceMath} from "../utils/uniswap/SqrtPriceMath.sol";
 import {LiquidityAmounts} from "../utils/uniswap/LiquidityAmounts.sol";
 import {LpPosition} from "../types/LpPosition.sol";
-import {MarketConstants} from "./MarketConstants.sol";
+import {CoinConstants} from "./CoinConstants.sol";
 
 /// @author Whetstone Research
 /// @notice Calculates liquidity provisioning with Uniswap v3
@@ -54,7 +54,7 @@ library DopplerMath {
         int24 spread = tickUpper - tickLower;
 
         uint160 farSqrtPriceX96 = TickMath.getSqrtPriceAtTick(farTick);
-        uint256 amountPerPosition = FullMath.mulDiv(discoverySupply, MarketConstants.WAD, totalPositions * MarketConstants.WAD);
+        uint256 amountPerPosition = FullMath.mulDiv(discoverySupply, CoinConstants.WAD, totalPositions * CoinConstants.WAD);
         uint256 totalAssetsSold;
 
         for (uint256 i; i < totalPositions; i++) {