eleven / messenger Public

web: fix reactions never sending — myName() is not in scope (#519)

The encrypted-reactions bridge (a18577a) built the reaction payload with
name: myName(), but myName only exists as a property on the setup bridge
object, not as a function in shell.js module scope. Every reaction tap threw
ReferenceError before ws.send, so nothing went on the wire — and since the
throw was outside the try and toggleReaction's .then has no .catch, the
failure was silent apart from an unhandled-rejection line.

Stamp the sender as session.name, exactly like every other outbound payload
in shell.js.
Co-authored-by: Claude <noreply@anthropic.com>
James Adam ca9d641118289eb679206bc7c81c780d28db0181 parent d7f9636
1 file changed, +1 −1
  • web/static/shell.js +1 −1
diff --git a/web/static/shell.js b/web/static/shell.js
index b294577..eb9ec26 100644
--- a/web/static/shell.js
+++ b/web/static/shell.js
@@ -3105,7 +3105,7 @@ setChatAppHost({
targetSid,
emoji,
on,
- name: myName(),
+ name: session.name || "",
uid: session.uid,
ts: Date.now(),
};