beforeSwap returns dynamic fee

src/interfaces/IHooks.sol

@@ -99,7 +99,7 @@ interface IHooks {
     /// @param hookData Arbitrary data handed into the PoolManager by the swapper to be be passed on to the hook
     /// @return bytes4 The function selector for the hook
     /// @return BeforeSwapDelta The hook's delta in specified and unspecified currencies. Positive: the hook is owed/took currency, negative: the hook owes/sent currency
-    /// @return uint24 An optional swap fee, only used if the Pool has a dynamic fee and the value is less than or equal to LPFeeLibrary.MAX_LP_FEE
+    /// @return uint24 Optionally override the lp fee, only used if three conditions are met: 1) the Pool has a dynamic fee, 2) the value's leading bit is set to 1 (24th bit, 0x800000), 3) the value is less than or equal to the maximum fee (1 million)
     function beforeSwap(
         address sender,
         PoolKey calldata key,

src/libraries/LPFeeLibrary.sol

@@ -35,15 +35,17 @@ library LPFeeLibrary {
         lpFee.validate();
     }
 
-    /// @dev converts a fee (returned from beforeSwap) to an override fee by setting the top bit of the uint24
+    /// @notice converts a fee to an override fee (to be returned by beforeSwap) by setting the top bit of the uint24
     function asOverrideFee(uint24 self) internal pure returns (uint24) {
         return self | BEFORE_SWAP_FEE_OVERRIDE_FLAG;
     }
 
+    /// @notice returns true if the fee has the override flag set (top bit of the uint24)
     function isOverride(uint24 self) internal pure returns (bool) {
         return self & BEFORE_SWAP_FEE_OVERRIDE_FLAG != 0;
     }
 
+    /// @notice returns a fee with the override flag removed
     function removeOverrideFlag(uint24 self) internal pure returns (uint24) {
         return self & FEE_MASK;
     }

src/libraries/ParseBytes.sol

@@ -1,4 +1,3 @@
-import {LPFeeLibrary} from "./LPFeeLibrary.sol";
 // SPDX-License-Identifier: GPL-2.0-or-later
 
 pragma solidity ^0.8.20;