UniswapHookEvents.sol

[QUESTION] Hook calculation

Uniswap V4 hook documentation seems unclear as to whether HookSwap should contain the Uniswap swap volume also. We recommend that this is confirmed.

[INFO] Optimisation of UV4 hook event emits

Whilst reviewing the contract, it seems that it may be beneficial to utilise the Notifier contract to emit the events, rather than having an immutable upgrade to the PositionManager.

The Notifier could be triggered before the transient storage is reset in the PositionManager, thus resulting in a trigger that has context of swap amounts. This also means that it could be added to the existing PositionManager.