regression: revert thread messages endpoint usage (#40913)

Author: MartinSchoeler

apps/meteor/client/views/room/contextualBar/Threads/hooks/useThreadMessagesQuery.ts

@@ -1,11 +1,10 @@
 import { isThreadMessage, type IMessage, type IRoom, type IThreadMainMessage, type IThreadMessage } from '@rocket.chat/core-typings';
-import { useEndpoint, useMethod, useStream } from '@rocket.chat/ui-contexts';
+import { useMethod, useStream } from '@rocket.chat/ui-contexts';
 import { useQuery, useQueryClient } from '@tanstack/react-query';
 import { useEffect, useRef } from 'react';
 
 import { onClientMessageReceived } from '../../../../../lib/onClientMessageReceived';
 import { roomsQueryKeys } from '../../../../../lib/queryKeys';
-import { mapMessageFromApi } from '../../../../../lib/utils/mapMessageFromApi';
 import { modifyMessageOnFilesDelete } from '../../../../../lib/utils/modifyMessageOnFilesDelete';
 import {
 	createDeleteCriteria,
@@ -25,11 +24,7 @@ export const useThreadMessagesQuery = (tmid: IThreadMainMessage['_id'], rid?: IR
 
 	const queryClient = useQueryClient();
 	const queryKey = roomsQueryKeys.threadMessages(roomId, tmid);
-	const getThreadMessages = useEndpoint('GET', '/v1/chat.getThreadMessages');
-	// REST has no per-thread read-marker endpoint yet; fall back to the
-	// `readThreads` DDP method so the side effect that DDP getThreadMessages
-	// used to do server-side keeps happening for callers.
-	const readThreads = useMethod('readThreads');
+	const getThreadMessages = useMethod('getThreadMessages');
 
 	const subscribeToRoomMessages = useStream('room-messages');
 	const subscribeToNotifyRoom = useStream('notify-room');
@@ -110,11 +105,10 @@ export const useThreadMessagesQuery = (tmid: IThreadMainMessage['_id'], rid?: IR
 		queryFn: async () => {
 			const cachedMessages = queryClient.getQueryData<IThreadMessage[]>(queryKey) || [];
 
-			const { messages } = await getThreadMessages({ tmid });
-			void Promise.resolve(readThreads(tmid)).catch(() => undefined);
-			const filtered = messages
-				.map((m) => mapMessageFromApi(m))
-				.filter((msg): msg is IThreadMessage => isThreadMessage(msg) && msg.tmid === tmid && msg._id !== tmid && msg._hidden !== true);
+			const messages = await getThreadMessages({ tmid });
+			const filtered = messages.filter(
+				(msg): msg is IThreadMessage => isThreadMessage(msg) && msg.tmid === tmid && msg._id !== tmid && msg._hidden !== true,
+			);
 
 			const sorted = mergeThreadMessages(cachedMessages, filtered);
 			if (unprocessedReadMessagesEvent.current) {