IBBoardThe PR author made changes! So at least it's not a "slop and forget" change.
However, all of their responses have that horrific obsequious tone of LLMs.
And one of my comments was "this call to openFile() seems to render the file, which would be inefficient for multiple files - can you just add it to the database instead?" and their response was "Good point, I've fixed it to be less complicated and more efficient"…
…and then they simplified the OTHER function (which did have more complex async for unexplained reasons)… and it STILL uses the function that I said was inefficient 🙄
Meanwhile, the comments on "this change seems unrelated" got "you're right, that was included by accident, I've reverted it".
I'm calling slop, and I hate the state of the world.