Fix read marker triggering room-wide notifications on alias mention

[Santhoshkumar044]

Summary

Fixes bug where mentioning a room's alias (e.g., #room:server.com) incorrectly triggers @room-like notifications for all members instead of treating it as a regular link.

Problem

When a message contained the current room's alias, the getMentions() function incorrectly set "room": true in the m.mentions field. This caused all members with "Contains @room: Notify Loud" enabled to receive notifications, even when the room alias was mentioned as a regular link and not as an @room mention.

Root cause: Both @room mentions (via autocomplete) and manually typed room aliases resolve to the same room alias ID, making them indistinguishable when only checking the node.id.

Solution

Modified the getMentions() function to distinguish between:

• @room mentions → Display name doesn't start with # → Sets room: true ✓
• Room alias mentions → Display name starts with # → Treated as regular link ✓

The fix checks the node.name (display text) instead of just the node.id to determine intent.

Changes

• Updated getMentions() in src/app/components/editor/util.ts (or your file path)
• Added condition: if (!node.name.startsWith('#')) before setting mentionData.room = true
• This ensures only actual @room mentions trigger room-wide notifications

Testing

Tested with 3 users in a room with alias #test:matrix.org:

Before fix:

• ❌ @user:server.com welcome to #test:matrix.org → All members notified
• ❌ Check #test:matrix.org → All members notified

After fix:

• ✅ @room hello → All members notified
• ✅ @user:server.com welcome to #test:matrix.org → Only @user notified
• ✅ Check #test:matrix.org → No one notified

Issue Number

Fixes #2533

[github-actions]

All contributors have signed the CLA ✍️ ✅
_{Posted by the CLA Assistant Lite bot.}

[Santhoshkumar044]

I have read the CLA Document and I hereby sign the CLA

[github-actions]

Preview: https://2562--pr-cinny.netlify.app
⚠️ Exercise caution. Use test accounts. ⚠️

[Santhoshkumar044]

Hi @ajbura
I have addressed the requested changes and updated it, The latest Netlify preview reflects the fixes.
Ready for re-review.Thanks!