Freegle / Iznik / 25289

73%

Build Type

push

circleci

Committed by invalid-email-address

Commit Message

fix(browse): drain the unread count through normal browsing + the button

Two small client fixes for "the unseen count won't reset and the Mark seen
button comes and goes".

1. messageStore.markSeen() refreshed the badge with fetchCount() and NO
   arguments, so it recomputed the count for the DEFAULT view (nearby,
   unlimited) instead of the member's actual view. For a "mygroups" member -
   or anyone with the distance slider set - clicking Mark seen repainted the
   badge with a different view's number instead of zero. Pass the member's
   browseView + browseMaxDistance, mirroring nearbyStore.fetchMessages.

2. The browse feed de-duplicates a poster's crosspost/repost copies to one
   card (useMessageDedup), but the server counts every copy as its own unseen
   post. Viewing the shown card only marked its own copy seen, so the hidden
   copies kept the count above zero and normal browsing could never drain it
   (measured: ~3.8k duplicate copies live, ~14% of open posts). MessageList
   now maps each card to the copies collapsed under it (same dedupKey) and,
   on a card view, marks those hidden copies seen too via a lightweight
   markSeenSiblings() (writes only still-unseen copies, no count refetch -
   the badge refreshes on the next scroll/poll like any per-card view).

dedupKey is exported so the sibling map reuses the exact feed grouping.
Tests cover both fixes and the dedupKey grouping. Not run locally.

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

Coverage Stats

12756 of 16824 branches covered (75.82%)

Branch coverage included in aggregate %.

88 of 93 new or added lines in 3 files covered. (94.62%)

1 existing line in 1 file now uncovered.

133488 of 184867 relevant lines covered (72.21%)

38.06 hits per line