Pushed a follow-up commit (0f75d518) that makes this branch compile and adds two fixes:

Build fix — duplicate definitions. The reaction/mute widgets were defined both in accord_home_message_row.dart and in the dedicated part files (accord_home_mute_button.dart / accord_home_reactions.dart) that refactor #108 had already extracted, causing duplicate_definition errors (_MuteButton, _MuteButtonState, _NotifAction, _ReactButton, _ReactionPill, _ReactorsDialog). Removed the stale copies from accord_home_message_row.dart and ported the fixes into the canonical part files.

Bug fixes ported into the canonical files:

• Mute state never reflected: listMutes returns objects {user_id, channel_id, created_at}, but the client stringified each whole map (e.toString()) instead of reading channel_id, so the muted check was always false. Now parses channel_id.
• Custom-emoji reactions/management showed broken images: the client built <cdn>/emojis/<id>.png, but the server serves <cdn>/emojis/<space_id>/<id>.<ext>. Now uses the server-provided image_url (resolved via the space emoji catalog for reactions), matching the emoji picker / message-markup pattern.

flutter analyze --no-fatal-infos is clean and all 186 tests pass.

Review findings and fixes (commit 298bd3b)

Findings ranked by severity:

🔴 CRITICAL — Duplicate class definitions (compile error)

_ReactButton, _ReactionPill, and _ReactorsDialog were added to accord_home_message_row.dart by this PR, but identical/older definitions of all three already existed in accord_home_reactions.dart. Both files are part of 'accord_home.dart', so the same class name in two part files is a compile-time duplicate-definition error — the app would not build.

Fix: Moved all reaction/notification widgets to accord_home_reactions.dart (their natural home) and removed the duplicate definitions from accord_home_message_row.dart. The improved _ReactionPill with imageUrl (from this PR) replaced the old cdnUrl-based version in the reactions file.

🟠 HIGH — Wrong docstring on _MuteButton

The doc-comment on _MuteButton began with:

/// Opens the full emoji picker to add a reaction to the message.
/// A bell toggle in the channel header …

The first line describes _ReactButton, not _MuteButton — a stale line accidentally left from a copy-paste. Fixed by removing it.

🟡 MEDIUM — accord_home_message_row.dart over 1000 lines

After the PR the file reached 1234 lines. Resolved as a side-effect of fix #1 (moving the widgets to the reactions file brings it back to 837 lines).

🟡 MEDIUM — Double accordAvatarUrl() call per reactor

In _ReactorsDialogState.build(), accordAvatarUrl(user, cdnUrl) was called twice per user — once for the null check and once to get the actual value. Fixed by extracting a _ReactorTile widget that stores the URL in a local variable.

🟢 LOW — Redundant emojiUrl alias in _ReactionPill

final emojiUrl = imageUrl;  // pointless rename

emojiUrl was used in exactly the same way imageUrl would be. Fixed by removing the alias and using imageUrl! directly.

🟢 LOW — Line too long in _reactionEmojiUrl

final id = rawId ?? parseEmojiToken(reaction.emoji['name']?.toString() ?? '').id;

Wrapped to two lines to stay within 80 columns.

🔵 TEST COVERAGE — New test file added

test/features/messaging/emoji_utils_test.dart — covers:

• parseEmojiToken: name/id splitting, colon-wrapped shortcodes, non-numeric suffixes, empty input
• resolveEmojiGlyph: known shortcode → glyph, passthrough unicode, colon-stripped lookup, unknown fallback
• Mute-list channel_id extraction logic (mirrors the parsing in channel_context_menu.dart and _MuteButtonState._load)

Generated by Claude Code