fix(reactions): broadcast reaction updates to all room viewers

nrupard commented

2026-04-29 16:01:21 +02:00

Owner

ReactionBar held an internal use_signal initialized once from the
initial_reactions prop, ignoring later prop changes. The reacter saw
their toggle because the click handler called reactions.set(updated)
locally, but peers did not refresh: the parent updated reactions_map
from the WS event, the prop changed, but the bar's internal signal
stayed stale.

Drop the internal signal and render the prop directly. The hub already
broadcasts ReactionAdded/Removed to every room subscriber including the
sender, so reacter and peers both see the update via the same path.

Co-Authored-By: Claude Opus 4.7 (1M context) noreply@anthropic.com

ReactionBar held an internal use_signal initialized once from the initial_reactions prop, ignoring later prop changes. The reacter saw their toggle because the click handler called reactions.set(updated) locally, but peers did not refresh: the parent updated reactions_map from the WS event, the prop changed, but the bar's internal signal stayed stale. Drop the internal signal and render the prop directly. The hub already broadcasts ReactionAdded/Removed to every room subscriber including the sender, so reacter and peers both see the update via the same path. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

nrupard added 1 commit

2026-04-29 16:01:22 +02:00

fix(reactions): broadcast reaction updates to all room viewers fcfd3a3b60

ReactionBar held an internal use_signal initialized once from the
initial_reactions prop, ignoring later prop changes. The reacter saw
their toggle because the click handler called reactions.set(updated)
locally, but peers did not refresh: the parent updated reactions_map
from the WS event, the prop changed, but the bar's internal signal
stayed stale.

Drop the internal signal and render the prop directly. The hub already
broadcasts ReactionAdded/Removed to every room subscriber including the
sender, so reacter and peers both see the update via the same path.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

nrupard merged commit 5adc3c6745 into main

2026-04-29 16:01:26 +02:00

nrupard referenced this pull request from a commit

2026-04-29 16:01:26 +02:00

Merge pull request 'fix(reactions): broadcast reaction updates to all room viewers' (#22) from fix/reactions-broadcast-to-peers into main

nrupard deleted branch fix/reactions-broadcast-to-peers

2026-04-29 16:01:26 +02:00

Rows
Columns

fix(reactions): broadcast reaction updates to all room viewers #22