When auditing any software system, it’s important to remember that there is another trail of history that can’t be checked into Git: the context in which the system was built. Maybe the person working on it was inexperienced, but maybe it was fit for purpose once, but then the purpose changed. Or there was a need to get it out quickly and a proper solution couldn’t be done in time. Or priorities shifted and people needed to be redeployed to other things.

Whatever the reason is, I doubt the reason was that the original implementor was intentionally careless. So go easy on yourself/others when you see something strange and say out loud, “why did I/they do it like that?”