fix(chat): exclude contact cards from FilePreviewsWrapper

- contact card shares use the same 'file' key as regular files, but rendered with ContactCard.vue
- they are not excluded from richParameters and rendered via NcRichText

Assisted-by: ClaudeCode:claude-sonnet-5
Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>

Author: Antreesy

src/components/MessagesList/MessagesGroup/Message/MessageItem.spec.js

@@ -18,6 +18,7 @@ import MessageQuote from '../../../MessageQuote.vue'
 import CallButton from '../../../TopBar/CallButton.vue'
 import MessageButtonsBar from './MessageButtonsBar/MessageButtonsBar.vue'
 import MessageItem from './MessageItem.vue'
+import ContactCard from './MessagePart/ContactCard.vue'
 import DeckCard from './MessagePart/DeckCard.vue'
 import DefaultParameter from './MessagePart/DefaultParameter.vue'
 import FilePreview from './MessagePart/FilePreview.vue'
@@ -408,6 +409,26 @@ describe('MessageItem.vue', () => {
 				expect(wrapper.findComponent(NcRichText).exists()).toBe(false)
 			})
 
+			test('renders contact card (text/vcard) via NcRichText', () => {
+				const params = {
+					file: {
+						id: '123',
+						type: 'file',
+						mimetype: 'text/vcard',
+						name: 'John Doe.vcf',
+						link: 'https://example.com/John%20Doe.vcf',
+					},
+				}
+				messageProps.message.message = '{file}'
+				messageProps.message.messageParameters = params
+				store.dispatch('processMessage', { token: TOKEN, message: messageProps.message })
+				const wrapper = mountMessage(messageProps)
+
+				expect(wrapper.findComponent(NcRichText).exists()).toBe(true)
+				expect(wrapper.findComponent(ContactCard).exists()).toBe(true)
+				expect(wrapper.findComponent(FilePreviewsWrapper).exists()).toBe(false)
+			})
+
 			test('renders deck cards', () => {
 				const params = {
 					actor: {

src/components/MessagesList/MessagesGroup/Message/MessageItem.vue

@@ -113,6 +113,7 @@ import { hasTalkFeature } from '../../../../services/CapabilitiesManager.ts'
 import { EventBus } from '../../../../services/EventBus.ts'
 import { useActorStore } from '../../../../stores/actor.ts'
 import { useChatExtrasStore } from '../../../../stores/chatExtras.ts'
+import { isFilePreviewParameter } from '../../../../utils/message.ts'
 
 const LocationCard = defineAsyncComponent(() => import('./MessagePart/LocationCard.vue'))
 
@@ -229,7 +230,7 @@ export default {
 							token: this.message.token,
 						},
 					}
-				} else if (type === 'file' && mimetype !== 'text/vcard') {
+				} else if (isFilePreviewParameter(p, this.message.messageParameters[p])) {
 					// File previews are rendered by FilePreviewsWrapper, skip from richParameters
 					return
 				} else if (type === SHARED_ITEM.OBJECT_TYPE.DECK_CARD) {

src/components/MessagesList/MessagesGroup/Message/MessagePart/FilePreviewsWrapper.vue

@@ -9,13 +9,13 @@ import type { ChatMessage } from '../../../../../types/index.ts'
 import { computed } from 'vue'
 import FilePreview from './FilePreview.vue'
 import { getItemTypeFromMessage } from '../../../../../utils/getItemTypeFromMessage.ts'
-import { getFileKeys } from '../../../../../utils/message.ts'
+import { getFilePreviewKeys } from '../../../../../utils/message.ts'
 
 const props = defineProps<{
 	message: ChatMessage
 }>()
 
-const fileKeys = computed(() => getFileKeys(props.message))
+const fileKeys = computed(() => getFilePreviewKeys(props.message))
 
 /**
  * Get referenceId of a file parameter to look up a local preview (if available).

src/components/MessagesList/MessagesGroup/Message/MessagePart/MessageBody.vue

@@ -66,11 +66,11 @@
 			<MessageQuote v-if="showQuote" :message="message.parent" />
 
 			<!-- File previews, rendered as a block on top of the (optional) caption -->
-			<FilePreviewsWrapper v-if="isFileShare" :message="message" />
+			<FilePreviewsWrapper v-if="hasFilePreviews" :message="message" />
 
 			<!-- Message content / text -->
 			<NcRichText
-				v-if="!isFileShareWithoutCaption"
+				v-if="!isFileShareWithoutCaption || !hasFilePreviews"
 				:text="renderedMessage"
 				:arguments="richParameters"
 				:class="{ 'single-emoji': isSingleEmoji }"
@@ -218,6 +218,7 @@ import { useChatExtrasStore } from '../../../../../stores/chatExtras.ts'
 import { usePollsStore } from '../../../../../stores/polls.ts'
 import { useUploadStore } from '../../../../../stores/upload.ts'
 import { formatDateTime } from '../../../../../utils/formattedTime.ts'
+import { getFileKeys, getFilePreviewKeys, isFilePreviewParameter } from '../../../../../utils/message.ts'
 import { parseMentions, parseSpecialSymbols } from '../../../../../utils/textParse.ts'
 
 // Regular expression to check for Unicode emojis in message text
@@ -327,13 +328,22 @@ export default {
 	},
 
 	computed: {
+		hasFilePreviews() {
+			return getFilePreviewKeys(this.message).length > 0
+		},
+
 		showQuote() {
 			return !!this.message.parent && this.message.parent.id !== this.threadId
 		},
 
 		renderedMessage() {
 			// File previews are rendered separately, as a block on top of this text (see FilePreviewsWrapper)
-			if (this.isLocationMessageWithName) {
+			if (this.isFileShare && !this.isFileShareWithoutCaption) {
+				// Contact cards with mimetype 'text/vcard' are rendered here.
+				// In case of caption present, placeholder should be put before it.
+				const vcardKey = getFileKeys(this.message).find((key) => !isFilePreviewParameter(key, this.message.messageParameters[key]))
+				return vcardKey ? `{${vcardKey}}\n\n${this.message.message}` : this.message.message
+			} else if (this.isLocationMessageWithName) {
 				return this.message.message + '\n\n' + this.message.messageParameters.object.name
 			} else {
 				return this.message.message

src/utils/message.ts

@@ -197,6 +197,27 @@ export function getFileKeys(message: Pick<ChatMessage, 'messageParameters'>) {
 	return Object.keys(message.messageParameters ?? {}).filter((key) => FILE_KEY_REGEX.test(key))
 }
 
+/**
+ * Checks whether a message parameter should be rendered as a preview in a FilePreviewsWrapper
+ * (Contact cards with mimetype 'text/vcard' are rendered separately)
+ *
+ * @param key key of the message parameter ('file', 'file-1', …)
+ * @param parameter the message parameter itself
+ * @param parameter.mimetype the parameter's mimetype, if any
+ */
+export function isFilePreviewParameter(key: string, parameter: { mimetype?: string }): boolean {
+	return FILE_KEY_REGEX.test(key) && parameter.mimetype !== 'text/vcard'
+}
+
+/**
+ * Returns keys of files shared with the message to be rendered with a FilePreviewsWrapper
+ *
+ * @param message Chat message (or an object with messageParameters)
+ */
+export function getFilePreviewKeys(message: Pick<ChatMessage, 'messageParameters'>) {
+	return getFileKeys(message).filter((key) => isFilePreviewParameter(key, message.messageParameters[key]))
+}
+
 /**
  * Returns whether the given message shares a file (has a rich object parameter with a key starting with 'file').
  *