When weird code needs to explain itself

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

Explore listener stats, chart rankings, contacts and more on the No Compromises podcast page.