Comment by bigstrat2003
7 hours ago
> Was I going to reject the PR because of my code preferences? No, of course not. Ultimately it doesn't matter anymore.
You absolutely should've rejected it. Putting slop into production is a huge trap, because now the humans can't understand it and the LLM is just going to make it worse and worse over time.
I think what OP is saying is that the fact that a human can't understand it won't matter as the code will only ever be read by AI.
Yes exactly. I think we all understood at a high-level what it was doing.
The issue I had was that it was doing a lot of weirdness like using `Object.defineProperty` to try to force an existing library to do something it wasn't suppose to do. No human would have written that code.
Although I generate most of my code with AI I still personally try to keep it understandable by a competent developer. Where as most people on my team are not good at FE so they just trust the LLM to get it right. But ultimately it just doesn't matter anymore.