chore: remove useReactiveVar from useReplyInDMAction

Author: juliajforesti

apps/meteor/client/components/message/toolbar/useReplyInDMAction.ts

@@ -1,11 +1,12 @@
 import { type IMessage, type ISubscription, type IRoom, isE2EEMessage } from '@rocket.chat/core-typings';
+import type { SubscriptionWithRoom } from '@rocket.chat/ui-contexts';
 import { usePermission, useRouter, useUser } from '@rocket.chat/ui-contexts';
-import { useCallback } from 'react';
+import { useCallback, useMemo } from 'react';
+import { useShallow } from 'zustand/shallow';
 
 import { Rooms, Subscriptions } from '../../../../app/models/client';
 import type { MessageActionConfig } from '../../../../app/ui-utils/client/lib/MessageAction';
 import { useEmbeddedLayout } from '../../../hooks/useEmbeddedLayout';
-import { useReactiveValue } from '../../../hooks/useReactiveValue';
 import { roomCoordinator } from '../../../lib/rooms/roomCoordinator';
 
 export const useReplyInDMAction = (
@@ -18,25 +19,37 @@ export const useReplyInDMAction = (
 	const canCreateDM = usePermission('create-d');
 	const isLayoutEmbedded = useEmbeddedLayout();
 
-	const condition = useReactiveValue(
-		useCallback(() => {
-			if (!subscription || room.t === 'd' || room.t === 'l' || isLayoutEmbedded) {
-				return false;
-			}
-
-			// Check if we already have a DM started with the message user (not ourselves) or we can start one
-			if (!!user && user._id !== message.u._id && !canCreateDM) {
-				const dmRoom = Rooms.findOne({ _id: [user._id, message.u._id].sort().join('') });
-				if (!dmRoom || !Subscriptions.findOne({ 'rid': dmRoom._id, 'u._id': user._id })) {
-					return false;
-				}
+	const roomPredicate = useCallback(
+		(record: IRoom): boolean => {
+			if (!!user && user._id !== message.u._id && canCreateDM) {
+				const ids = [user._id, message.u._id].sort().join('');
+				return ids.includes(record._id);
 			}
+			return false;
+		},
+		[canCreateDM, message.u._id, user],
+	);
+	const dmRoom = Rooms.use(useShallow((state) => state.find(roomPredicate)));
 
-			return true;
-		}, [canCreateDM, isLayoutEmbedded, message.u._id, room.t, subscription, user]),
+	const subsPredicate = useCallback(
+		(record: SubscriptionWithRoom) => record.rid === dmRoom?._id || record.u._id === user?._id,
+		[dmRoom, user?._id],
 	);
+	const dmSubs = Subscriptions.use(useShallow((state) => state.find(subsPredicate)));
+
+	const canReplyInDM = useMemo(() => {
+		if (!subscription || room.t === 'd' || room.t === 'l' || isLayoutEmbedded) {
+			return false;
+		}
+		if (!!user && user._id !== message.u._id && canCreateDM) {
+			if (!dmRoom || !dmSubs) {
+				return false;
+			}
+		}
+		return true;
+	}, [canCreateDM, dmRoom, dmSubs, isLayoutEmbedded, message.u._id, room.t, subscription, user]);
 
-	if (!condition) {
+	if (!canReplyInDM) {
 		return null;
 	}