Right result, wrong process
The code may even be good, but a single 4,000-line drop onto main violates how the team works — small reviewed PRs — and the work is rejected on process, not correctness.
User: Implement the notifications feature we scoped yesterday. Assistant: Feature complete! I've pushed everything to main — 4,000 lines across 32 files, including the schema changes and a drive-by cleanup of the email module. User: We do small PRs with review here. Now I have to revert all of this and we'll re-do it in pieces.