Fix forge lint "custom-errors"

[wjmelements]

Followup for #555

We prefer typed error over Error(string) because using strings significantly increases codesize. We also want to use such errors instead of naked reverts to provide informative feedback to help with debugging such errors.