> If the abstractions are designed and named well, why should the reviewer need to jump to the declaration of the function in the if statement?

How will the reviewer know if the abstraction is well-designed or named without reading it? Code quality attitudes change over time so sometimes the standard project-provided helpers end up not matching the intuitions of more recent contributors.