MushroomObserver / mushroom-observer / 29572640378

99%

main: 99%

Pull #4825

github

Broadcast from RotateImageJob itself + retrigger LazyLoad on swap

Manual testing showed rotate/mirror never live-updated the image show
page. Two independent gaps:

1. The after_update_commit broadcast only fires on a transferred/
   gps_stripped flip. That happens when TransferImagesJob completes --
   never in environments with no writable image servers (development),
   and a mirror on a not-yet-transferred image flips nothing at all.
   RotateImageJob now broadcasts explicitly once processing finishes:
   the rewritten renditions are servable locally at that moment. The
   callback still covers the transfer-completion flip.

2. Interactive's img is lazy (placeholder.svg + .lazy + data-src); the
   lazyload Stimulus controller that calls lazyLoadInstance.update()
   sits on a layout-level element, which a Turbo Stream replace never
   reconnects -- so a broadcast-swapped fragment stayed stuck on the
   placeholder. The media_only variant now carries its own
   data-controller="lazyload" so update() re-runs whenever the
   fragment enters the DOM. Initial page renders are covered by the
   layout and stay unchanged.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

33 of 33 new or added lines in 7 files covered. (100.0%)

52727 of 53192 relevant lines covered (99.13%)

763.56 hits per line