Reconsider internal placement of logic to choose between CI and pre-commit #980

This might be in the wrong layer of abstraction (currently in the core layer).

I suspect it would be better to move this logic into the CI configuration getter - if pre-commit is being used, we can short circuit from within these functions

This helps work toward the core layer being standardized across tools