24970
72%
master: 72%

Ran 05 Jul 2026 02:22PM UTC

Files 477

Run time 9s

Badge

Embed ▾

Committed 05 Jul 2026 01:57PM UTC coverage: 68.816% (+0.005%) from 68.811%

Job # 24970

Build Type

push

circleci

Committed by

invalid-email-address

Commit Message

fix(digest): stamp lastsent on a pinned-only daily digest (stop re-send flood)

Incident 2026-07-05: 67 members received the daily digest up to ~198 times.
Cause: the daily digest always includes a pinned post (a940e6631), but pinned
posts are never part of the cursor set. sendDigest called
updateDigestTracker($allPosts), which only stamps users_digests.lastsent when it
has cursor posts. A user whose digest contained ONLY a pinned post therefore got
the email sent but lastsent left unstamped, so the once-per-London-day guard
(lastsent >= today) never fired and the everyMinute daily run re-sent them every
tick.

updateDigestTracker now takes emailWasSent; when true and there are no cursor
posts it still stamps lastsent (the cursor msgid/msgdate correctly stays put, as
a pinned post has no cursor entry). The daily send site passes true, since an
email is always sent once past the no_posts guards. The no-send paths keep the
default (false) so users we didn't mail are never marked sent.

Test: updateDigestTracker stamps lastsent with empty posts + emailWasSent=true,
and does not when false.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_015DQRL5iGD71jNTgyokMpt9

Coverage Stats

26483 of 38484 relevant lines covered (68.82%)

19.57 hits per line

Freegle / Iznik / 24970 / 24970
72%
master: 72%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job laravel - 24970

Freegle / Iznik / 24970 / 24970 72% master: 72%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job laravel - 24970

Freegle / Iznik / 24970 / 24970
72%
master: 72%

README BADGES
x