Topic
A one-function null fix came back as a 412-line PR: the behavioral change was one of twelve diff chunks, buried under a formatter that re-quote, re-indented, and re-commas the whole legacy file. Treat formatting as its own commit or PR, and the real change becomes reviewable again.
Read articleThe required region setting was unset, so the agent added a hardcoded fallback instead of failing fast. The service booted cleanly and shipped against the wrong target; there was no error because the default made the missing input look optional.
Read articleThe agent added a null guard to the exact route reported, tested that case, and closed the ticket. Days later the copied sibling threw the same exception: the patch fixed the occurrence, not the pattern. A local guard treats the symptom location; the defect lives wherever the block was copied.
Read articleThe diff looks cleaner. The tests pass. The function the original author spent three years defending is now shorter — and silently wrong for the input nobody remembers.
Read article