GPT-5 finding
BaseHook inherits ImmutableState but does not pass required PoolManager constructor argument, causing compilation/deployment failure
highbuild-releasehigh
- src/base/BaseHook.sol
ImmutableState’s constructor requires an IPoolManager address. BaseHook defines a constructor but does not pass the required parameter to the ImmutableState base constructor. This will not compile or, if made to compile via testing tricks, would set poolManager to zero and make onlyPoolManager unusable at runtime.
Recommendation
Update BaseHook’s constructor to accept and forward an IPoolManager to ImmutableState, e.g.: constructor(IPoolManager _poolManager) ImmutableState(_poolManager) { validateHookAddress(this); }