fix(textParse): use mention id from message parameters if available

Signed-off-by: Maksim Sukharev <[email protected]>

Author: Antreesy

src/stores/__tests__/chatExtras.spec.js

@@ -128,8 +128,8 @@ describe('chatExtrasStore', () => {
 		it('should render mentions properly when editing message', () => {
 			// Arrange
 			const parameters = {
-				'mention-call1': { type: 'call', name: 'Conversation101' },
-				'mention-user1': { type: 'user', name: 'Alice Joel', id: 'alice' },
+				'mention-call1': { type: 'call', name: 'Conversation101', 'mention-id': 'all' },
+				'mention-user1': { type: 'user', name: 'Alice Joel', id: 'alice', 'mention-id': 'alice' },
 			}
 			// Act
 			chatExtrasStore.setChatEditInput({
@@ -138,7 +138,7 @@ describe('chatExtrasStore', () => {
 				parameters
 			})
 			// Assert
-			expect(chatExtrasStore.getChatEditInput('token-1')).toBe('Hello @all and @alice')
+			expect(chatExtrasStore.getChatEditInput('token-1')).toBe('Hello @"all" and @"alice"')
 		})
 
 		it('should store chat input without escaping special symbols', () => {

src/types/index.ts

@@ -127,7 +127,7 @@ export type importEmailsParams = Required<operations['room-import-emails-as-part
 export type importEmailsResponse = ApiResponse<operations['room-import-emails-as-participants']['responses'][200]['content']['application/json']>
 
 // Chats
-export type Mention = RichObject<'server'|'call-type'|'icon-url'>
+export type Mention = RichObject<'server'|'call-type'|'icon-url'> & { 'mention-id'?: string }
 export type File = RichObject<'size'|'path'|'link'|'mimetype'|'preview-available'> & {
 	'etag': string,
 	'permissions': string,

src/utils/__tests__/textParse.spec.js

@@ -10,6 +10,39 @@ jest.mock('@nextcloud/router', () => ({
 
 describe('textParse', () => {
 	describe('parseMentions', () => {
+		it('replaces mentions correctly if mention-id is available', () => {
+			const input = 'test {mention-call1} test {mention-user1} test {mention-group1} test {mention-federated-user1}'
+			const output = 'test @"all" test @"alice" test @"group/talk" test @"federated_user/[email protected]"'
+			const parameters = {
+				'mention-call1': {
+					id: 'room-id',
+					name: 'Room Display Name',
+					type: 'call',
+					'mention-id': 'all',
+				},
+				'mention-user1': {
+					id: 'alice',
+					name: 'Just Alice',
+					type: 'user',
+					'mention-id': 'alice',
+				},
+				'mention-group1': {
+					id: 'talk',
+					name: 'Talk Group',
+					type: 'user-group',
+					'mention-id': 'group/talk',
+				},
+				'mention-federated-user1': {
+					id: 'alice',
+					name: 'Feder Alice',
+					type: 'user',
+					server: 'https://server2.com',
+					'mention-id': 'federated_user/[email protected]',
+				}
+			}
+			expect(parseMentions(input, parameters)).toBe(output)
+		})
+
 		it('replaces {mention-call} correctly', () => {
 			const input = 'test {mention-call1}'
 			const output = 'test @all'

src/utils/textParse.ts

@@ -19,7 +19,9 @@ function parseMentions(text: string, parameters: ChatMessage['messageParameters'
 		const value: Mention = parameters[key] as Mention
 		let mention = ''
 
-		if (key.startsWith('mention-call') && value.type === MENTION.TYPE.CALL) {
+		if (value['mention-id']) {
+			mention = `@"${value['mention-id']}"`
+		} else if (key.startsWith('mention-call') && value.type === MENTION.TYPE.CALL) {
 			mention = '@all'
 		} else if (key.startsWith('mention-federated-user')
 			&& [MENTION.TYPE.USER, MENTION.TYPE.FEDERATED_USER].includes(value.type)) {