Skip calls to hooks when msg.sender is the hook contract

.forge-snapshots/addLiquidity with empty hook.snap

@@ -1 +1 @@
-312292
+312421

.forge-snapshots/addLiquidity with native token.snap

@@ -1 +1 @@
-192474
+192592

.forge-snapshots/addLiquidity.snap

@@ -1 +1 @@
-192452
+192570

.forge-snapshots/before swap hook, already cached dynamic fee.snap

@@ -1 +1 @@
-184412
+184495

.forge-snapshots/cached dynamic fee, no hooks.snap

@@ -1 +1 @@
-138439
+138491

.forge-snapshots/donate gas with 1 token.snap

@@ -1 +1 @@
-131592
+131644

.forge-snapshots/donate gas with 2 tokens.snap

@@ -1 +1 @@
-176613
+176665

.forge-snapshots/initialize.snap

@@ -1 +1 @@
-51767
+51819

.forge-snapshots/poolManager bytecode size.snap

@@ -1 +1 @@
-23279
+23491

.forge-snapshots/removeLiquidity with empty hook.snap

@@ -1 +1 @@
-99307
+99425

.forge-snapshots/removeLiquidity with native token.snap

@@ -1 +1 @@
-200680
+200798

.forge-snapshots/removeLiquidity.snap

@@ -1 +1 @@
-196957
+197075

.forge-snapshots/simple swap with native.snap

@@ -1 +1 @@
-187012
+187064

.forge-snapshots/simple swap.snap

@@ -1 +1 @@
-195549
+195601

.forge-snapshots/swap against liquidity with native token.snap

@@ -1 +1 @@
-117675
+117727

.forge-snapshots/swap against liquidity.snap

@@ -1 +1 @@
-105139
+105191

.forge-snapshots/swap burn 6909 for input.snap

@@ -1 +1 @@
-125350
+125402

.forge-snapshots/swap burn native 6909 for input.snap

@@ -1 +1 @@
-121303
+121355

.forge-snapshots/swap mint native output as 6909.snap

@@ -1 +1 @@
-189538
+189590

.forge-snapshots/swap mint output as 6909.snap

@@ -1 +1 @@
-206343
+206395

.forge-snapshots/swap with dynamic fee.snap

@@ -1 +1 @@
-183513
+183596

.forge-snapshots/swap with hooks.snap

@@ -1 +1 @@
-105117
+105169

.forge-snapshots/update dynamic fee in before swap.snap

@@ -1 +1 @@
-190217
+190300

src/libraries/Hooks.sol

@@ -107,7 +107,7 @@ library Hooks {
     function beforeInitialize(IHooks self, PoolKey memory key, uint160 sqrtPriceX96, bytes calldata hookData)
         internal
     {
-        if (self.hasPermission(BEFORE_INITIALIZE_FLAG)) {
+        if (self.hasPermission(BEFORE_INITIALIZE_FLAG) && (msg.sender != address(self))) {
             self.callHook(
                 abi.encodeWithSelector(IHooks.beforeInitialize.selector, msg.sender, key, sqrtPriceX96, hookData)
             );
@@ -118,7 +118,7 @@ library Hooks {
     function afterInitialize(IHooks self, PoolKey memory key, uint160 sqrtPriceX96, int24 tick, bytes calldata hookData)
         internal
     {
-        if (self.hasPermission(AFTER_INITIALIZE_FLAG)) {
+        if (self.hasPermission(AFTER_INITIALIZE_FLAG) && (msg.sender != address(self))) {
             self.callHook(
                 abi.encodeWithSelector(IHooks.afterInitialize.selector, msg.sender, key, sqrtPriceX96, tick, hookData)
             );
@@ -132,12 +132,16 @@ library Hooks {
         IPoolManager.ModifyLiquidityParams memory params,
         bytes calldata hookData
     ) internal {
-        if (params.liquidityDelta > 0 && key.hooks.hasPermission(BEFORE_ADD_LIQUIDITY_FLAG)) {
-            self.callHook(abi.encodeWithSelector(IHooks.beforeAddLiquidity.selector, msg.sender, key, params, hookData));
-        } else if (params.liquidityDelta <= 0 && key.hooks.hasPermission(BEFORE_REMOVE_LIQUIDITY_FLAG)) {
-            self.callHook(
-                abi.encodeWithSelector(IHooks.beforeRemoveLiquidity.selector, msg.sender, key, params, hookData)
-            );
+        if ((msg.sender != address(self))) {
+            if (params.liquidityDelta > 0 && key.hooks.hasPermission(BEFORE_ADD_LIQUIDITY_FLAG)) {
+                self.callHook(
+                    abi.encodeWithSelector(IHooks.beforeAddLiquidity.selector, msg.sender, key, params, hookData)
+                );
+            } else if (params.liquidityDelta <= 0 && key.hooks.hasPermission(BEFORE_REMOVE_LIQUIDITY_FLAG)) {
+                self.callHook(
+                    abi.encodeWithSelector(IHooks.beforeRemoveLiquidity.selector, msg.sender, key, params, hookData)
+                );
+            }
         }
     }
 
@@ -149,22 +153,32 @@ library Hooks {
         BalanceDelta delta,
         bytes calldata hookData
     ) internal {
-        if (params.liquidityDelta > 0 && key.hooks.hasPermission(AFTER_ADD_LIQUIDITY_FLAG)) {
-            self.callHook(
-                abi.encodeWithSelector(IHooks.afterAddLiquidity.selector, msg.sender, key, params, delta, hookData)
-            );
-        } else if (params.liquidityDelta <= 0 && key.hooks.hasPermission(AFTER_REMOVE_LIQUIDITY_FLAG)) {
-            self.callHook(
-                abi.encodeWithSelector(IHooks.afterRemoveLiquidity.selector, msg.sender, key, params, delta, hookData)
-            );
+        if ((msg.sender != address(self))) {
+            if (
+                params.liquidityDelta > 0 && key.hooks.hasPermission(AFTER_ADD_LIQUIDITY_FLAG)
+                    && (msg.sender != address(self))
+            ) {
+                self.callHook(
+                    abi.encodeWithSelector(IHooks.afterAddLiquidity.selector, msg.sender, key, params, delta, hookData)
+                );
+            } else if (
+                params.liquidityDelta <= 0 && key.hooks.hasPermission(AFTER_REMOVE_LIQUIDITY_FLAG)
+                    && (msg.sender != address(self))
+            ) {
+                self.callHook(
+                    abi.encodeWithSelector(
+                        IHooks.afterRemoveLiquidity.selector, msg.sender, key, params, delta, hookData
+                    )
+                );
+            }
         }
     }
 
     /// @notice calls beforeSwap hook if permissioned and validates return value
     function beforeSwap(IHooks self, PoolKey memory key, IPoolManager.SwapParams memory params, bytes calldata hookData)
         internal
     {
-        if (key.hooks.hasPermission(BEFORE_SWAP_FLAG)) {
+        if (key.hooks.hasPermission(BEFORE_SWAP_FLAG) && (msg.sender != address(self))) {
             self.callHook(abi.encodeWithSelector(IHooks.beforeSwap.selector, msg.sender, key, params, hookData));
         }
     }
@@ -177,7 +191,7 @@ library Hooks {
         BalanceDelta delta,
         bytes calldata hookData
     ) internal {
-        if (key.hooks.hasPermission(AFTER_SWAP_FLAG)) {
+        if (key.hooks.hasPermission(AFTER_SWAP_FLAG) && (msg.sender != address(self))) {
             self.callHook(abi.encodeWithSelector(IHooks.afterSwap.selector, msg.sender, key, params, delta, hookData));
         }
     }
@@ -186,7 +200,7 @@ library Hooks {
     function beforeDonate(IHooks self, PoolKey memory key, uint256 amount0, uint256 amount1, bytes calldata hookData)
         internal
     {
-        if (key.hooks.hasPermission(BEFORE_DONATE_FLAG)) {
+        if (key.hooks.hasPermission(BEFORE_DONATE_FLAG) && (msg.sender != address(self))) {
             self.callHook(
                 abi.encodeWithSelector(IHooks.beforeDonate.selector, msg.sender, key, amount0, amount1, hookData)
             );
@@ -197,7 +211,7 @@ library Hooks {
     function afterDonate(IHooks self, PoolKey memory key, uint256 amount0, uint256 amount1, bytes calldata hookData)
         internal
     {
-        if (key.hooks.hasPermission(AFTER_DONATE_FLAG)) {
+        if (key.hooks.hasPermission(AFTER_DONATE_FLAG) && (msg.sender != address(self))) {
             self.callHook(
                 abi.encodeWithSelector(IHooks.afterDonate.selector, msg.sender, key, amount0, amount1, hookData)
             );