@ensuro/core 1.2.7 → 2.0.0-beta1

package/build/interfaces/IAssetManager.sol/IAssetManager.json

@@ -1,31 +0,0 @@
-{
-  "_format": "hh-sol-artifact-1",
-  "contractName": "IAssetManager",
-  "sourceName": "interfaces/IAssetManager.sol",
-  "abi": [
-    {
-      "inputs": [],
-      "name": "deinvestAll",
-      "outputs": [],
-      "stateMutability": "nonpayable",
-      "type": "function"
-    },
-    {
-      "inputs": [
-        {
-          "internalType": "uint256",
-          "name": "paymentAmount",
-          "type": "uint256"
-        }
-      ],
-      "name": "refillWallet",
-      "outputs": [],
-      "stateMutability": "nonpayable",
-      "type": "function"
-    }
-  ],
-  "bytecode": "0x",
-  "deployedBytecode": "0x",
-  "linkReferences": {},
-  "deployedLinkReferences": {}
-}

package/build/interfaces/IExchange.sol/IPriceOracle.json

@@ -1,30 +0,0 @@
-{
-  "_format": "hh-sol-artifact-1",
-  "contractName": "IPriceOracle",
-  "sourceName": "interfaces/IExchange.sol",
-  "abi": [
-    {
-      "inputs": [
-        {
-          "internalType": "address",
-          "name": "asset",
-          "type": "address"
-        }
-      ],
-      "name": "getAssetPrice",
-      "outputs": [
-        {
-          "internalType": "uint256",
-          "name": "",
-          "type": "uint256"
-        }
-      ],
-      "stateMutability": "view",
-      "type": "function"
-    }
-  ],
-  "bytecode": "0x",
-  "deployedBytecode": "0x",
-  "linkReferences": {},
-  "deployedLinkReferences": {}
-}

package/contracts/AaveAssetManager.sol

@@ -1,298 +0,0 @@
-// SPDX-License-Identifier: Apache-2.0
-pragma solidity ^0.8.0;
-
-import {AddressUpgradeable} from "@openzeppelin/contracts-upgradeable/utils/AddressUpgradeable.sol";
-import {IERC20Metadata} from "@openzeppelin/contracts/token/ERC20/extensions/IERC20Metadata.sol";
-import {BaseAssetManager} from "./BaseAssetManager.sol";
-import {SafeERC20} from "@openzeppelin/contracts/token/ERC20/utils/SafeERC20.sol";
-import {IPolicyPool} from "../interfaces/IPolicyPool.sol";
-import {IPolicyPoolConfig} from "../interfaces/IPolicyPoolConfig.sol";
-import {IExchange} from "../interfaces/IExchange.sol";
-import {WadRayMath} from "./WadRayMath.sol";
-import {ILendingPoolAddressesProvider} from "@aave/protocol-v2/contracts/interfaces/ILendingPoolAddressesProvider.sol";
-import {ILendingPool} from "@aave/protocol-v2/contracts/interfaces/ILendingPool.sol";
-import {IAToken} from "@aave/protocol-v2/contracts/interfaces/IAToken.sol";
-import {AaveProtocolDataProvider} from "@aave/protocol-v2/contracts/misc/AaveProtocolDataProvider.sol";
-
-/**
- * @title AssetManager that reinvests the capital in AAVE
- * @dev Deposits and withdraw from AAVE, also converts the rewards.
- *      Invest into AAVE AToken for the underlying asset (ex. USDC) getting lending interests.
- *      When needs to deinvest, first deinvest from AAVE, but also can liquidate the rewards
- *      (AAVE in mainnet or MATIC in Polygon) using a DEX.
- *      Above a given threshold, the rewards are claimed. Also, above a given threshold they are also reinvested
- *      to accrue additional interests and rewards.
- *      An authorized user (SWAP_REWARDS_ROLE) can force the swap of the rewards for the pool's currency.
- * @custom:security-contact security@ensuro.co
- * @author Ensuro
- */
-contract AaveAssetManager is BaseAssetManager {
-  using SafeERC20 for IERC20Metadata;
-  using WadRayMath for uint256;
-  using AddressUpgradeable for address;
-
-  bytes32 public constant SWAP_REWARDS_ROLE = keccak256("SWAP_REWARDS_ROLE");
-
-  /// @custom:oz-upgrades-unsafe-allow state-variable-immutable
-  ILendingPoolAddressesProvider internal immutable _aaveAddrProv;
-  /// @custom:oz-upgrades-unsafe-allow state-variable-immutable
-  IAToken internal immutable _aToken;
-  /// @custom:oz-upgrades-unsafe-allow state-variable-immutable
-  IAToken internal immutable _rewardAToken;
-  /// @custom:oz-upgrades-unsafe-allow state-variable-immutable
-  IERC20Metadata internal immutable _rewardToken;
-  /// @custom:oz-upgrades-unsafe-allow state-variable-immutable
-  uint256 internal _claimRewardsMin; // Minimum amount of rewards accumulated to claim
-  uint256 internal _reinvestRewardsMin; // Minimum amount of rewards to reinvest into AAVE
-
-  bytes32 internal constant DATA_PROVIDER_ID =
-    0x0100000000000000000000000000000000000000000000000000000000000000;
-
-  uint256 internal constant SECONDS_PER_YEAR = 365 days;
-
-  event RewardSwapped(uint256 rewardIn, uint256 currencyOut);
-
-  /// @custom:oz-upgrades-unsafe-allow constructor
-  constructor(IPolicyPool policyPool_, ILendingPoolAddressesProvider aaveAddrProv_)
-    BaseAssetManager(policyPool_)
-  {
-    _aaveAddrProv = aaveAddrProv_;
-    AaveProtocolDataProvider dataProvider = AaveProtocolDataProvider(
-      aaveAddrProv_.getAddress(DATA_PROVIDER_ID)
-    );
-    (address aToken_, , ) = dataProvider.getReserveTokensAddresses(address(policyPool_.currency()));
-    _aToken = IAToken(aToken_);
-    address rewardToken_ = IAToken(aToken_).getIncentivesController().REWARD_TOKEN();
-    _rewardToken = IERC20Metadata(rewardToken_);
-    (address rewardAToken_, , ) = dataProvider.getReserveTokensAddresses(address(rewardToken_));
-    _rewardAToken = IAToken(rewardAToken_);
-  }
-
-  function initialize(
-    uint256 liquidityMin_,
-    uint256 liquidityMiddle_,
-    uint256 liquidityMax_,
-    uint256 claimRewardsMin_,
-    uint256 reinvestRewardsMin_
-  ) public initializer {
-    __BaseAssetManager_init(liquidityMin_, liquidityMiddle_, liquidityMax_);
-    __AaveAssetManager_init(claimRewardsMin_, reinvestRewardsMin_);
-  }
-
-  // solhint-disable-next-line func-name-mixedcase
-  function __AaveAssetManager_init(uint256 claimRewardsMin_, uint256 reinvestRewardsMin_)
-    internal
-    initializer
-  {
-    _claimRewardsMin = claimRewardsMin_;
-    _reinvestRewardsMin = reinvestRewardsMin_;
-  }
-
-  function getInvestmentValue() public view override returns (uint256) {
-    uint256 balance = aToken().balanceOf(address(this));
-    uint256 rewardBalance = rewardToken().balanceOf(address(this)) +
-      rewardAToken().balanceOf(address(this));
-    // Don't count unclaimedRewards as part of investmentValue to save gas and because if doing that will
-    // also need to claim rewards as part of _deinvest process
-    return balance + _rewardToCurrency(rewardBalance);
-  }
-
-  function unclaimedRewards() public view returns (uint256) {
-    // Also add unclaimed rewards
-    IAToken atk = aToken();
-    address[] memory atks = new address[](2);
-    atks[0] = address(atk);
-    atks[1] = address(rewardAToken());
-    return atk.getIncentivesController().getRewardsBalance(atks, address(this));
-  }
-
-  function _claimRewards(bool ignoreMin) internal returns (uint256) {
-    if (ignoreMin || unclaimedRewards() > _claimRewardsMin) {
-      IAToken atk = aToken();
-      address[] memory atks = new address[](2);
-      atks[0] = address(atk);
-      atks[1] = address(rewardAToken());
-      return atk.getIncentivesController().claimRewards(atks, type(uint256).max, address(this));
-    } else {
-      return 0;
-    }
-  }
-
-  function lendingPool() public view returns (ILendingPool) {
-    return ILendingPool(_aaveAddrProv.getLendingPool());
-  }
-
-  function _aaveDataProvider() internal view returns (AaveProtocolDataProvider) {
-    return AaveProtocolDataProvider(_aaveAddrProv.getAddress(DATA_PROVIDER_ID));
-  }
-
-  function _exchange() internal view returns (IExchange) {
-    return policyPool().config().exchange();
-  }
-
-  function aToken() public view returns (IAToken) {
-    return _aToken;
-  }
-
-  function rewardToken() public view returns (IERC20Metadata) {
-    return _rewardToken;
-  }
-
-  function rewardAToken() public view returns (IAToken) {
-    return _rewardAToken;
-  }
-
-  function _exchangePath() internal view returns (address[] memory) {
-    address[] memory path = new address[](2);
-    path[0] = address(rewardToken());
-    path[1] = address(currency());
-    return path;
-  }
-
-  function _rewardToCurrency(uint256 amount) internal view returns (uint256) {
-    return _exchange().convert(address(rewardToken()), address(currency()), amount);
-  }
-
-  function reinvestRewardToken() public {
-    ILendingPool lendingPool_ = lendingPool();
-    IERC20Metadata token = rewardToken();
-    uint256 rewardBalance = token.balanceOf(address(this));
-    if (rewardBalance <= _reinvestRewardsMin) return;
-    token.approve(address(lendingPool_), rewardBalance);
-    lendingPool_.deposit(address(token), rewardBalance, address(this), 0);
-  }
-
-  function _swapRewards(uint256 amount, address outAddr) internal returns (uint256, uint256) {
-    IERC20Metadata rw = rewardToken();
-    uint256 swapIn = rw.balanceOf(address(this));
-    if (swapIn < amount) {
-      uint256 toWithdraw = amount - swapIn;
-      if (rewardAToken().balanceOf(address(this)) < toWithdraw) {
-        toWithdraw = type(uint256).max; // if not enought withdraw all
-      }
-      swapIn += lendingPool().withdraw(address(rw), toWithdraw, address(this));
-    } else {
-      swapIn = amount;
-    }
-    address swapRouter = _exchange().getSwapRouter();
-    rw.approve(swapRouter, swapIn);
-    bytes memory swapCall = _exchange().sell(
-      address(rw),
-      address(currency()),
-      swapIn,
-      outAddr,
-      block.timestamp
-    );
-
-    bytes memory response = swapRouter.functionCall(swapCall, "Swap operation failed");
-    uint256 swapOut = _exchange().decodeSwapOut(response);
-    emit RewardSwapped(swapIn, swapOut);
-    return (swapIn, swapOut);
-  }
-
-  function swapRewards(uint256 amount)
-    external
-    onlyPoolRole(SWAP_REWARDS_ROLE)
-    returns (uint256, uint256)
-  {
-    (uint256 swapIn, uint256 swapOut) = _swapRewards(amount, address(this));
-    ILendingPool lendingPool_ = lendingPool();
-    IERC20Metadata token = currency();
-    token.approve(address(lendingPool_), swapOut);
-    lendingPool_.deposit(address(token), swapOut, address(this), 0);
-    return (swapIn, swapOut);
-  }
-
-  function rebalance() public virtual override whenNotPaused {
-    _claimRewards(false);
-    super.rebalance();
-    reinvestRewardToken();
-  }
-
-  function _invest(uint256 amount) internal override {
-    ILendingPool lendingPool_ = lendingPool();
-    IERC20Metadata token = currency();
-    token.safeTransferFrom(address(_policyPool), address(this), amount);
-    token.approve(address(lendingPool_), amount);
-    lendingPool_.deposit(address(token), amount, address(this), 0);
-    super._invest(amount);
-  }
-
-  function _deinvest(uint256 amount) internal override {
-    uint256 remainingAmount = amount;
-    uint256 toWithdraw = amount;
-    if (aToken().balanceOf(address(this)) < toWithdraw) {
-      toWithdraw = type(uint256).max;
-    }
-    remainingAmount -= lendingPool().withdraw(
-      address(currency()),
-      toWithdraw,
-      address(_policyPool) // Withdraw directly to _policyPool
-    );
-    if (remainingAmount > 0) {
-      // In this case, it's safe using getAmountsIn to compute how many rewards are needed to swap
-      // but then, when I do the swap I validate the slippage with the market price (given by AAVE's Oracle)
-      // is acceptable
-      uint256 requiredRewards = _exchange().getAmountIn(
-        address(rewardToken()),
-        address(currency()),
-        remainingAmount
-      );
-      (, uint256 currencyOut) = _swapRewards(requiredRewards, address(_policyPool));
-      if (currencyOut < remainingAmount) {
-        remainingAmount -= currencyOut;
-      } else {
-        remainingAmount = 0;
-      }
-    }
-    super._deinvest(amount - remainingAmount);
-  }
-
-  /**
-   * @dev Deinvest all the assets and return the cash back to the PolicyPool.
-   *      Called from PolicyPool when new asset manager is assigned
-   */
-  function _liquidateAll() internal virtual override {
-    _claimRewards(true);
-    lendingPool().withdraw(
-      address(currency()),
-      type(uint256).max,
-      address(_policyPool) // Withdraw directly to _policyPool
-    );
-    // Withdraw all rewards
-    lendingPool().withdraw(address(_rewardToken), type(uint256).max, address(this));
-    _swapRewards(_rewardToken.balanceOf(address(this)), address(_policyPool));
-  }
-
-  // Contract parameters
-  function claimRewardsMin() external view returns (uint256) {
-    return _claimRewardsMin;
-  }
-
-  function reinvestRewardsMin() external view returns (uint256) {
-    return _reinvestRewardsMin;
-  }
-
-  function setClaimRewardsMin(uint256 newValue) external onlyPoolRole2(LEVEL2_ROLE, LEVEL3_ROLE) {
-    bool tweak = !hasPoolRole(LEVEL2_ROLE);
-    require(
-      !tweak || _isTweakWad(_claimRewardsMin, newValue, 3e26),
-      "Tweak exceeded: claimRewardsMin tweaks only up to 30%"
-    );
-    _claimRewardsMin = newValue;
-    _parameterChanged(IPolicyPoolConfig.GovernanceActions.setClaimRewardsMin, newValue, tweak);
-  }
-
-  function setReinvestRewardsMin(uint256 newValue)
-    external
-    onlyPoolRole2(LEVEL2_ROLE, LEVEL3_ROLE)
-  {
-    bool tweak = !hasPoolRole(LEVEL2_ROLE);
-    require(
-      !tweak || _isTweakWad(_reinvestRewardsMin, newValue, 3e26),
-      "Tweak exceeded: reinvestRewardsMin tweaks only up to 30%"
-    );
-    _reinvestRewardsMin = newValue;
-    _parameterChanged(IPolicyPoolConfig.GovernanceActions.setReinvestRewardsMin, newValue, tweak);
-  }
-}

package/contracts/BaseAssetManager.sol

@@ -1,301 +0,0 @@
-// SPDX-License-Identifier: Apache-2.0
-pragma solidity ^0.8.0;
-
-import {WadRayMath} from "./WadRayMath.sol";
-import {IERC20} from "@openzeppelin/contracts/token/ERC20/IERC20.sol";
-import {IPolicyPool} from "../interfaces/IPolicyPool.sol";
-import {PolicyPoolComponent} from "./PolicyPoolComponent.sol";
-import {IAssetManager} from "../interfaces/IAssetManager.sol";
-import {IEToken} from "../interfaces/IEToken.sol";
-import {IPolicyPoolConfig} from "../interfaces/IPolicyPoolConfig.sol";
-import {Policy} from "./Policy.sol";
-import {Math} from "@openzeppelin/contracts/utils/math/Math.sol";
-
-/**
- * @title Ensuro Asset Manager base contract
- * @dev Base class for asset managers that reinvest the assets of the protocol to get additional returns.
- *      The specific asset management strategy needs to be implemented by child contracts.
- *      Settings liquidityMin, liquidityMiddle, liquidityMax are the thresholds used to define how much liquidity
- *      to keep in the PolicyPool and when to invest/deinvest. Every invest/deinvest operation tries to leave the
- *      PolicyPool at liquidityMiddle.
- * @custom:security-contact security@ensuro.co
- * @author Ensuro
- */
-abstract contract BaseAssetManager is IAssetManager, PolicyPoolComponent {
-  using WadRayMath for uint256;
-
-  uint256 internal _liquidityMin;
-  uint256 internal _liquidityMiddle;
-  uint256 internal _liquidityMax;
-
-  uint256 internal _lastInvestmentValue;
-
-  event MoneyInvested(uint256 amount);
-  event MoneyDeinvested(uint256 amount);
-  event EarningsDistributed(bool positive, uint256 amount);
-
-  /// @custom:oz-upgrades-unsafe-allow constructor
-  // solhint-disable-next-line no-empty-blocks
-  constructor(IPolicyPool policyPool_) PolicyPoolComponent(policyPool_) {}
-
-  /**
-   * @dev Initializes the asset manager
-   * @param liquidityMin_ Minimal liquidity to keep in pool's wallet
-   * @param liquidityMiddle_ Target liquidity when doing rebalance
-   * @param liquidityMax_ Maximum liquidity to keep in pool's wallet
-   */
-  // solhint-disable-next-line func-name-mixedcase
-  function __BaseAssetManager_init(
-    uint256 liquidityMin_,
-    uint256 liquidityMiddle_,
-    uint256 liquidityMax_
-  ) internal initializer {
-    __PolicyPoolComponent_init();
-    __BaseAssetManager_init_unchained(liquidityMin_, liquidityMiddle_, liquidityMax_);
-  }
-
-  // solhint-disable-next-line func-name-mixedcase
-  function __BaseAssetManager_init_unchained(
-    uint256 liquidityMin_,
-    uint256 liquidityMiddle_,
-    uint256 liquidityMax_
-  ) internal initializer {
-    /*
-    _lastInvestmentValue = 0;
-    */
-    _liquidityMin = liquidityMin_;
-    _liquidityMiddle = liquidityMiddle_;
-    _liquidityMax = liquidityMax_;
-    _validateParameters();
-  }
-
-  function _validateParameters() internal view override {
-    require(
-      _liquidityMin <= _liquidityMiddle && _liquidityMiddle <= _liquidityMax,
-      "Validation: Liquidity limits are invalid"
-    );
-  }
-
-  /**
-   * @dev Returns the total amount that is available to invest by the asset manager
-   */
-  function totalInvestable() public view returns (uint256) {
-    uint256 poolInvestable = _policyPool.getInvestable();
-    uint256 etksInvestable = 0;
-    for (uint256 i = 0; i < _policyPool.getETokenCount(); i++) {
-      IEToken etk = _policyPool.getETokenAt(i);
-      etksInvestable += etk.getInvestable();
-    }
-    return poolInvestable + etksInvestable;
-  }
-
-  /**
-   * @dev Calculates asset earnings and distributes them updating accounting in PolicyPool and eTokens
-   */
-  function distributeEarnings() public virtual whenNotPaused {
-    // TODO Check: Anyone can call this funcion. This could be a potencial surface of flash loan attack?
-    _distributeEarnings(getInvestmentValue());
-  }
-
-  function _distributeEarnings(uint256 investmentValue) internal virtual whenNotPaused {
-    bool positive;
-    uint256 earnings;
-    if (investmentValue > _lastInvestmentValue) {
-      earnings = investmentValue - _lastInvestmentValue;
-      positive = true;
-    } else if (investmentValue < _lastInvestmentValue) {
-      earnings = _lastInvestmentValue - investmentValue;
-      positive = false;
-    } else {
-      return; // No earnings
-    }
-
-    uint256 totalInv = totalInvestable();
-
-    uint256 remaining = earnings;
-    for (uint256 i = 0; i < _policyPool.getETokenCount(); i++) {
-      IEToken etk = _policyPool.getETokenAt(i);
-      uint256 aux = earnings.wadMul(etk.getInvestable().wadDiv(totalInv));
-      if (aux > 0) {
-        etk.assetEarnings(aux, positive);
-        remaining -= aux;
-      }
-    }
-
-    if (remaining > 0) _policyPool.assetEarnings(remaining, positive);
-
-    _lastInvestmentValue = investmentValue;
-    emit EarningsDistributed(positive, earnings);
-  }
-
-  /**
-   * @dev Returns the current value of the investment portfolio
-   */
-  function getInvestmentValue() public view virtual returns (uint256);
-
-  /**
-   * @dev Rebalances cash between PolicyPool wallet and
-   */
-  function rebalance() public virtual whenNotPaused {
-    // TODO Check: Anyone can call this funcion. This could be a potencial surface of flash loan attack?
-    uint256 poolCash = currency().balanceOf(address(_policyPool));
-    if (poolCash > _liquidityMax) {
-      _invest(poolCash - _liquidityMiddle);
-    } else if (poolCash < _liquidityMin) {
-      uint256 deinvestAmount = Math.min(getInvestmentValue(), _liquidityMiddle - poolCash);
-      if (deinvestAmount > 0) {
-        _deinvest(deinvestAmount);
-      }
-    }
-  }
-
-  /**
-   * @dev Function to be called automatically by a crontask - Distributes and rebalances
-   */
-  function checkpoint() external {
-    distributeEarnings();
-    rebalance();
-  }
-
-  /**
-   * @dev This is called from PolicyPool when doesn't have enought money for payment.
-   *      After the call, there should be enought money in PolicyPool.currency().balanceOf(this) to
-   *      do the payment
-   * @param paymentAmount The amount of the payment
-   */
-  function refillWallet(uint256 paymentAmount) external override onlyPolicyPool {
-    uint256 poolCash = currency().balanceOf(address(_policyPool));
-    require(poolCash < paymentAmount, "No need to refill the wallet for this payment");
-    uint256 investmentValue = getInvestmentValue();
-    // try to leave the pool balance at liquidity_middle after the payment
-    uint256 deinvest = paymentAmount + _liquidityMiddle - poolCash;
-    if (deinvest > investmentValue) deinvest = investmentValue;
-    _deinvest(deinvest);
-  }
-
-  function _invest(uint256 amount) internal virtual {
-    _lastInvestmentValue += amount;
-    emit MoneyInvested(amount);
-    // must be reimplemented do the actual cash movement
-  }
-
-  function _deinvest(uint256 amount) internal virtual {
-    _lastInvestmentValue -= Math.min(_lastInvestmentValue, amount);
-    emit MoneyDeinvested(amount);
-    // must be reimplemented do the actual cash movement
-  }
-
-  /**
-   * @dev Deinvest all the assets and return the cash back to the PolicyPool.
-   *      Called from PolicyPool when new asset manager is assigned
-   */
-  function deinvestAll() external virtual override onlyPolicyPool {
-    uint256 poolBalanceBefore = currency().balanceOf(address(_policyPool));
-    _liquidateAll();
-    uint256 poolBalanceAfter = currency().balanceOf(address(_policyPool));
-    _distributeEarnings(poolBalanceAfter - poolBalanceBefore);
-    _lastInvestmentValue = 0;
-    emit MoneyDeinvested(poolBalanceAfter - poolBalanceBefore);
-  }
-
-  function _liquidateAll() internal virtual;
-
-  function liquidityMin() external view returns (uint256) {
-    return _liquidityMin;
-  }
-
-  function liquidityMiddle() external view returns (uint256) {
-    return _liquidityMiddle;
-  }
-
-  function liquidityMax() external view returns (uint256) {
-    return _liquidityMax;
-  }
-
-  function setLiquidityMin(uint256 newValue) external onlyPoolRole2(LEVEL2_ROLE, LEVEL3_ROLE) {
-    bool tweak = !hasPoolRole(LEVEL2_ROLE);
-    require(
-      !tweak || _isTweakRay(_liquidityMin, newValue, 3e26),
-      "Tweak exceeded: liquidityMin tweaks only up to 30%"
-    );
-    _liquidityMin = newValue;
-    _parameterChanged(IPolicyPoolConfig.GovernanceActions.setLiquidityMin, newValue, tweak);
-  }
-
-  function setLiquidityMiddle(uint256 newValue) external onlyPoolRole2(LEVEL2_ROLE, LEVEL3_ROLE) {
-    bool tweak = !hasPoolRole(LEVEL2_ROLE);
-    require(
-      !tweak || _isTweakRay(_liquidityMiddle, newValue, 3e26),
-      "Tweak exceeded: liquidityMiddle tweaks only up to 30%"
-    );
-    _liquidityMiddle = newValue;
-    _parameterChanged(IPolicyPoolConfig.GovernanceActions.setLiquidityMiddle, newValue, tweak);
-  }
-
-  function setLiquidityMax(uint256 newValue) external onlyPoolRole2(LEVEL2_ROLE, LEVEL3_ROLE) {
-    bool tweak = !hasPoolRole(LEVEL2_ROLE);
-    require(
-      !tweak || _isTweakRay(_liquidityMax, newValue, 3e26),
-      "Tweak exceeded: liquidityMax tweaks only up to 30%"
-    );
-    _liquidityMax = newValue;
-    _parameterChanged(IPolicyPoolConfig.GovernanceActions.setLiquidityMax, newValue, tweak);
-  }
-
-  function setLiquidityMultiple(
-    uint256 min,
-    uint256 middle,
-    uint256 max
-  ) external onlyPoolRole2(LEVEL2_ROLE, LEVEL3_ROLE) {
-    bool setMin = min != type(uint256).max;
-    bool setMiddle = middle != type(uint256).max;
-    bool setMax = max != type(uint256).max;
-    bool tweak = !hasPoolRole(LEVEL2_ROLE);
-    // First set all - Then call _parameterChanged
-    if (setMin) {
-      require(
-        !tweak || _isTweakRay(_liquidityMin, min, 3e26),
-        "Tweak exceeded: liquidityMin tweaks only up to 30%"
-      );
-      _liquidityMin = min;
-    }
-    if (setMiddle) {
-      require(
-        !tweak || _isTweakRay(_liquidityMiddle, middle, 3e26),
-        "Tweak exceeded: liquidityMiddle tweaks only up to 30%"
-      );
-      _liquidityMiddle = middle;
-    }
-    if (setMax) {
-      require(
-        !tweak || _isTweakRay(_liquidityMax, max, 3e26),
-        "Tweak exceeded: liquidityMax tweaks only up to 30%"
-      );
-      _liquidityMax = max;
-    }
-    if (setMin) _parameterChanged(IPolicyPoolConfig.GovernanceActions.setLiquidityMin, min, tweak);
-    if (setMiddle)
-      _parameterChanged(IPolicyPoolConfig.GovernanceActions.setLiquidityMiddle, middle, tweak);
-    if (setMax) _parameterChanged(IPolicyPoolConfig.GovernanceActions.setLiquidityMax, max, tweak);
-  }
-
-  function _setLiquidityMiddle(uint256 newValue) internal {
-    bool tweak = !hasPoolRole(LEVEL2_ROLE);
-    require(
-      !tweak || _isTweakRay(_liquidityMiddle, newValue, 3e26),
-      "Tweak exceeded: liquidityMiddle tweaks only up to 30%"
-    );
-    _liquidityMiddle = newValue;
-    _parameterChanged(IPolicyPoolConfig.GovernanceActions.setLiquidityMiddle, newValue, tweak);
-  }
-
-  function _setLiquidityMax(uint256 newValue) internal {
-    bool tweak = !hasPoolRole(LEVEL2_ROLE);
-    require(
-      !tweak || _isTweakRay(_liquidityMax, newValue, 3e26),
-      "Tweak exceeded: liquidityMax tweaks only up to 30%"
-    );
-    _liquidityMax = newValue;
-    _parameterChanged(IPolicyPoolConfig.GovernanceActions.setLiquidityMax, newValue, tweak);
-  }
-}