Contracts: use require with custom errors outside KlerosCore

[jaybuidl]

Currently KlerosCore is using custom errors without require to avoid hitting the contract size limit.

Custom errors were not adopted in other parts of the code partly due to the inverted logic. But with the recent support for require with custom errors, the logic can remain the same.

Solidity feature

[Pranay-Bhilare]

Hi, I would like to contribute on addressing this issue, which recommends using require(condition, CustomError()) for better readability and gas efficiency. I have a few clarification points before proceeding:

1. The repository currently uses Solidity 0.8.24, as seen in the contracts and hardhat.config.ts file. Should we upgrade to "0.8.29" according to the parent issue, (or a compatible version) to support this feature?
2. The Hardhat configuration has the viaIR flag commented out. Is the project configured to use the Yul IR pipeline, or should we explicitly enable it? As in the docs it is given that require with custom error can only be used with YUL IR pipeline :

Note that currently, using custom errors with require is only supported by the IR pipeline, i.e. compilation via Yul. For the legacy pipeline, please use the if (!condition) revert CustomError(); pattern instead.

3. Are there any potential compatibility or testing concerns for these updates that I should be aware of?

Looking forward to your guidance as I would like to contribute on this issue.
Thank you!