Skip to content

PLAT-39442 connector: declare the reactions Reddit accepts - #14

Merged
Fizzadar merged 2 commits into
mainfrom
nick/plat-39442-allowed-reactions
Oct 29, 2026
Merged

Fizzadar merged 2 commits into
mainfrom
nick/plat-39442-allowed-reactions

Conversation

@Fizzadar

Copy link
Copy Markdown
Contributor

Reddit only accepts its own reactions, but rooms didn't advertise which, so clients offered emoji like ❤️ and 🎉 that the bridge then rejected. Room features now list the 32 emoji that map to Reddit reactions (capabilities v12, so existing rooms get the update), and skin-toned variants such as 👍🏼 resolve to the base reaction. Desktop and Android show a bounded picker when the list is set, so Reddit's 14 reactions without an emoji alias can only be added by tapping an existing one.

🤖 Generated with Claude Code

Reddit only takes its own reactions, but rooms didn't say which, so clients
offered emoji the bridge then rejected. Skin-toned variants of aliased emoji
now resolve too.

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

indent Bot commented Oct 29, 2026 •

Copy link
Copy Markdown

Issues

No issues found.

CI Checks

All CI checks passed on 0144af7.

Review agents

Select any unchecked box below to run or rerun that agent.

Passed (1)
  • Indent Review Agent · No bugs found in the reaction allowlist change.
Full results

Indent Review Agent

  • Summary: No bugs found in the reaction allowlist change.
  • Last ran on commit: 0144af73
  • Latest result
    {
      "summary": "No bugs found in the reaction allowlist change.",
      "findings": []
    }

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

Copy link
Copy Markdown
Contributor Author

Live-tested on a self-hosted bridge (build dbba83e = main ce81b9c + #14 at 0144af7 + #15) with real Reddit accounts:

  • existing DM portals got com.beeper.reddit.capabilities.v12.dm.blocking with the 32 allowed_reactions at startup (v11, no list, on main); DM-specific levels unchanged
  • 👍, 👍🏼, 👍️ and 😂 from Beeper arrive natively as Reddit's thumbs_up/joy (👍🏼 was rejected on main); ❤️ still rejected as intended; removing a reaction removes it natively
  • incoming native reactions with and without an emoji alias (joy, upvote) unchanged; downtime catch-up and no echo unchanged

Known client gap: Android's canSendReaction ignores custom_emoji_reactions once a list is set, so Android can't +1 an existing Reddit image reaction until that's fixed client-side.

@Fizzadar
Fizzadar merged commit 82e94b9 into main Oct 29, 2026
11 checks passed
@Fizzadar
Fizzadar deleted the nick/plat-39442-allowed-reactions branch October 29, 2026 14:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant