
When weird code needs to explain itself
From No Compromises by Joel Clermont and Aaron Saray
March 28, 2026 · 11 min · Episode 149
About this episode
The episode discusses the importance of context in code reviews and how to properly document workarounds in software development.
Have you ever looked at a colleague's code and thought, "This is clearly wrong," only to find out it was actually a well-reasoned workaround for a tricky bug? In the latest episode of the No Compromises podcast, we discuss what happened when Aaron reviewed Joel's code and couldn't make sense of a pattern spread across multiple Livewire components. The code wasn't bad, it was solving a real UX flicker bug in an older version of Mary UI. But without context, it looked like a mistake and nearly got rejected. The fix wasn't just refactoring; it was giving the workaround a proper home: a trait with a descriptive name, clear method names, and thorough documentation explaining the bug, the reason for the pattern, and when it can eventually be removed. We also talk about why "the explanation is in the PR note" isn't good enough, how AI coding agents can unknowingly propagate patterns they don't understand, and why strange code deserves to look strange, on purpose. Explore Mastering Laravel resources to deepen your understanding of patterns like these. 00:00 The confusing code review that started this 01:15 Flagging the unclear pattern across components 03:54 The Mary UI toast flicker bug…
People in this episode
Hosts: Joel Clermont, Aaron Saray
Topics covered
- code review
- UX design
- software development
- documentation
- Laravel
- programming patterns
Keywords
- code review
- workaround
- documentation
- UX flicker bug
- Laravel
- Livewire
- software patterns
Mentioned in this episode
Organizations: Mary UI, Livewire
More episodes of No Compromises
- How much logic is too much logic in a PHP enum · June 6, 2026 · 13 min
- Your codebase is not a museum for old code · May 23, 2026 · 14 min
- Do you actually own the code you ship? · May 9, 2026 · 15 min
- Do you actually need a multi-tenancy package? · April 25, 2026 · 14 min
- Local and production should match even for Laravel tools · April 11, 2026 · 13 min
- Are you testing your app or just the framework? · March 14, 2026 · 13 min
Explore listener stats, chart rankings, contacts and more on the No Compromises podcast page.