feat(chat): Add the mention id to parameters for easier editing

[nickvergessen]

🛠️ API Checklist

🚧 Tasks

• ...

🏁 Checklist

• ⛑️ Tests (unit and/or integration) are included or not possible
• 📘 API documentation in docs/ has been updated or is not required
• 🔖 Capability is added or not needed

[SystemKeeper]

Is there any reason to not always wrap a mention in quotes? Currently we only do it if there's a "/" or whitespace, but does it really matter?

[nickvergessen]

Is there any reason to not always wrap a mention in quotes? Currently we only do it if there's a "/" or whitespace, but does it really matter?

IIRC it works always so its save to do it always, but e.g. on our instance since we use "first name", I always type it directly manually without waiting for the autocomplete

[nickvergessen]

/backport to stable31

[nickvergessen]

when proxying such a message I guess the server should extend the attribute if it was missing?

[SystemKeeper]

when proxying such a message I guess the server should extend the attribute if it was missing?

Would be nice, if it’s not too much work. Then it consistent across all clients.

[nickvergessen]

Can't add it to OpenAPI docs as the type then conflicts with the RichObject Definitions and Psalm complains

[nickvergessen]

/backport to stable31

[SystemKeeper]

Can we backport to stable30 and stable29 as well, since we introduced message editing with 19/29? This would allow for more consistent editing and testing.

[SystemKeeper]

/backport to stable30

[SystemKeeper]

/backport to stable29