There is no zero address/value check for the immutable variables (_rewardsVault, _startDropPerSecond,_dropDecreaseDuration,_baseLockBonusRatio, _minLockBonusRatio, _maxLockBonusRatio ) inside the constructor function at HolyPaladinToken.sol
Contract is not pausable. Instead of Emergency condition which downgrades users' positions, OpenZeppelin's pausable.solcan be utilized.
safeApprove. Linkimmutablevariables (_rewardsVault,_startDropPerSecond,_dropDecreaseDuration,_baseLockBonusRatio,_minLockBonusRatio,_maxLockBonusRatio) inside theconstructorfunction atHolyPaladinToken.solpausable. Instead of Emergency condition which downgrades users' positions, OpenZeppelin'spausable.solcan be utilized.