feat/read-receipts #8

Merged

nrupard merged 14 commits from feat/read-receipts into main

2026-04-13 20:43:22 +02:00

nrupard commented

2026-04-13 18:06:31 +02:00

Owner

No description provided.

nrupard added 12 commits

2026-04-13 18:06:31 +02:00

docs(spec): add read-receipts design and update TODO ba22f22a8b

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

docs(plan): add read-receipts implementation plan 04f058eb8f

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

feat(migrations): add dm_read_state for read receipts e421e16681

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

feat(auth): add read_receipts_enabled user column 632a58a810

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

feat(db): add dm read-state upsert and unread counts a982fe146e

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

feat(ws): add DmRead event 7926f98db1

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

feat(server_fns): add mark_dm_read, peer read state, and unread counts 32d6d808ed

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

feat(server_fns): add set_read_receipts_enabled 0c481e2537

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

feat(dm): mark read on visibility and render Seen label d2c1ec410e

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

feat(sidebar): show DM unread count badges 3bc45aacae

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

feat(sidebar): add read-receipts toggle in user menu c74ba952db

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

fix(dm): notify recipient sidebar of new messages e5792f53b5

broadcast_to_room only reaches WS subscribers of that room. DM recipients
who haven't opened the thread are not subscribed, so their sidebar never
saw new DMs. Also broadcast NewMessage directly to each non-sender member.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

nrupard added 2 commits

2026-04-13 20:41:35 +02:00

fix(dm): ignore own DmRead events in peer read tracker c71d70a7a4

The server broadcasts DmRead to the whole room, so the sender received
their own mark-read back and overwrote peer_last_read_id with their own
read position (which points at a peer message id). The "Seen" label
then jumped to a stale older own-message, or disappeared if no own
message fell ≤ that id. Filter on user_id != me.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

fix(dm): persist Seen label across refresh 9910a4fb00

Previously peer read state was copied from the server_future into a
signal via use_effect, then rendered from the signal. On refresh the
effect's set-signal path left a window where the render saw None and
no Seen appeared until a WS DmRead arrived (e.g. after sending a new
message). Read peer_state directly at render, with WS updates tracked
in an override signal that wins when present.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>