chore: resolve issues per comments

src/interfaces/IQuoter.sol

@@ -5,12 +5,14 @@ pragma solidity ^0.8.24;
 import {Currency} from "pancake-v4-core/src/types/Currency.sol";
 import {PathKey} from "../libraries/PathKey.sol";
 import {PoolKey} from "pancake-v4-core/src/types/PoolKey.sol";
+import {PoolId} from "pancake-v4-core/src/types/PoolId.sol";
 
 /// @title IQuoter Interface
 /// @notice Supports quoting the delta amounts from exact input or exact output swaps.
 /// @dev These functions are not marked view because they rely on calling non-view functions and reverting
 /// to compute the result. They are also not gas efficient and should not be called on-chain.
 interface IQuoter {
+    error NotEnoughLiquidity(PoolId poolId);
     error NotSelf();
     error UnexpectedCallSuccess();
 

src/pool-bin/lens/BinQuoter.sol

@@ -157,11 +157,12 @@ contract BinQuoter is BaseV4Quoter, IBinQuoter {
     {
         deltas = poolManager.swap(poolKey, zeroForOne, amountSpecified, hookData);
 
-        // TODO: confirm we don't need this check in the bin quoter since BinPool will revert if there is not enough liquidity
-        // Check that the pool was not illiquid.
-        // int128 amountSpecifiedActual = (zeroForOne == (amountSpecified < 0)) ? deltas.amount0() : deltas.amount1();
-        // if (amountSpecifiedActual != amountSpecified) {
-        //     revert NotEnoughLiquidity(poolKey.toId());
-        // }
+        /// @dev Check that the pool was not illiquid
+        /// even BinPool will emit BinPool__OutOfLiquidity when the pool is illiquid
+        /// We still need to apply the check in case hook contract manipulates the delta
+        int128 amountSpecifiedActual = (zeroForOne == (amountSpecified < 0)) ? deltas.amount0() : deltas.amount1();
+        if (amountSpecifiedActual != amountSpecified) {
+            revert NotEnoughLiquidity(poolKey.toId());
+        }
     }
 }

src/pool-cl/interfaces/ICLQuoter.sol

@@ -2,16 +2,13 @@
 pragma solidity ^0.8.24;
 
 import {IQuoter} from "../../interfaces/IQuoter.sol";
-import {PoolId} from "pancake-v4-core/src/types/PoolId.sol";
 
 /// @title ICLQuoter Interface
 /// @notice Supports quoting the delta amounts for exact input or exact output swaps.
 /// @notice For each pool also tells you the estimation of gas cost for the swap.
 /// @dev These functions are not marked view because they rely on calling non-view functions and reverting
 /// to compute the result. They are also not gas efficient and should not be called on-chain.
 interface ICLQuoter is IQuoter {
-    error NotEnoughLiquidity(PoolId poolId);
-
     /// @notice Returns the delta amounts for a given exact input swap of a single pool
     /// @param params The params for the quote, encoded as `QuoteExactSingleParams`
     /// poolKey The key for identifying a V4 pool

test/pool-cl/CLQuoter.t.sol

@@ -273,7 +273,7 @@ contract CLQuoterTest is Test, Deployers {
         vm.expectRevert(
             abi.encodeWithSelector(
                 QuoterRevert.UnexpectedRevertBytes.selector,
-                abi.encodeWithSelector(ICLQuoter.NotEnoughLiquidity.selector, key01.toId())
+                abi.encodeWithSelector(IQuoter.NotEnoughLiquidity.selector, key01.toId())
             )
         );
         quoter.quoteExactOutputSingle(