Comment by kgeist
14 hours ago
Interesting case:
- a project manager vibe-coded the change without thinking it through at all
- the PR was reviewed by an LLM
- an actual engineer gave LGTM without really reviewing the changes, trusting the LLM
Did I get this right?
>a project manager vibe-coded the change without thinking it through at all
The PMs vibe-coding and having no idea what they're doing isn't even the main issue (although it is pretty bad).
The main issue is: how are the actual engineers supposed to "review" the slop? They probably report to the same PM or are at below in the org chart and might be evaluated by them. Not just at MS, but any company.
Such a conflict of interest would be detrimental to quality anywhere. You wouldn't build a bridge like this, nor should you software.
The revert commit appears to have also been done by copilot
The LLM actually points out the problem tho
Maybe the engineer's LLM agent's summary of the GitHub LLM bot's review omitted that warning.
Maybe the GitHub comments didn’t properly load due to the weekly partial outage.
You can't make this shit up.