> Yeah the financial value of a codebase is in successfully solving a problem

It doesn't even need do it successfully, plenty of slow buggy codebases that are raking in huge amounts of money every day.

I knew it was bad when I started getting tickets to "improve the skeleton loader" because its displayed for so long the project owner had time to contemplate changes to it.

code quality helps you get to the final successful solution, keep it there and iterate on it faster and with less bugs. It's not just pretty indentation

but yeah sure no value