chore: Replace useSafeRefCallback from ui-client with fuselage-hooks' version

apps/meteor/client/components/message/content/attachments/file/hooks/useReloadOnError.tsx

@@ -1,5 +1,4 @@
-import { useEffectEvent } from '@rocket.chat/fuselage-hooks';
-import { useSafeRefCallback } from '@rocket.chat/ui-client';
+import { useEffectEvent, useSafeRefCallback } from '@rocket.chat/fuselage-hooks';
 import { useCallback, useRef, useState } from 'react';
 
 const events = ['error', 'stalled', 'play'];
@@ -135,11 +134,7 @@ export const useReloadOnError = (url: string, type: 'video' | 'audio') => {
 
 	const mediaRefCallback = useSafeRefCallback(
 		useCallback(
-			(node: HTMLAudioElement | null) => {
-				if (!node) {
-					return;
-				}
-
+			(node: HTMLAudioElement) => {
 				events.forEach((event) => {
 					node.addEventListener(event, handleMediaURLRecovery);
 				});

apps/meteor/client/views/room/MessageList/hooks/useJumpToMessage.ts

@@ -1,6 +1,5 @@
 import type { IMessage } from '@rocket.chat/core-typings';
-import { useMergedRefs } from '@rocket.chat/fuselage-hooks';
-import { useSafeRefCallback } from '@rocket.chat/ui-client';
+import { useMergedRefs, useSafeRefCallback } from '@rocket.chat/fuselage-hooks';
 import { useRouter } from '@rocket.chat/ui-contexts';
 import { useCallback, useRef } from 'react';
 
@@ -49,12 +48,8 @@ export const useJumpToMessage = (messageId: IMessage['_id']) => {
 
 	const ref = useSafeRefCallback(
 		useCallback(
-			(node: HTMLElement | null) => {
-				if (!node || !scroll) {
-					return;
-				}
-
-				if (!listRef) {
+			(node: HTMLElement) => {
+				if (!listRef || !scroll) {
 					return;
 				}
 

apps/meteor/client/views/room/body/hooks/useGetMore.ts

@@ -1,4 +1,4 @@
-import { useSafeRefCallback } from '@rocket.chat/ui-client';
+import { useSafeRefCallback } from '@rocket.chat/fuselage-hooks';
 import { useSearchParameter } from '@rocket.chat/ui-contexts';
 import type { MutableRefObject } from 'react';
 import { useCallback, useEffect, useRef } from 'react';
@@ -19,10 +19,7 @@ export const useGetMore = (rid: string, atBottomRef: MutableRefObject<boolean>)
 
 	const ref = useSafeRefCallback(
 		useCallback(
-			(element: HTMLElement | null) => {
-				if (!element) {
-					return;
-				}
+			(element: HTMLElement) => {
 				const checkPositionAndGetMore = withThrottling({ wait: 100 })(async () => {
 					if (!element.isConnected) {
 						return;

apps/meteor/client/views/room/body/hooks/useListIsAtBottom.ts

@@ -1,5 +1,4 @@
-import { useMergedRefs } from '@rocket.chat/fuselage-hooks';
-import { useSafeRefCallback } from '@rocket.chat/ui-client';
+import { useMergedRefs, useSafeRefCallback } from '@rocket.chat/fuselage-hooks';
 import type { MutableRefObject } from 'react';
 import { useCallback, useRef } from 'react';
 
@@ -35,11 +34,7 @@ export const useListIsAtBottom = () => {
 
 	const ref = useSafeRefCallback(
 		useCallback(
-			(node: HTMLElement | null) => {
-				if (!node) {
-					return;
-				}
-
+			(node: HTMLElement) => {
 				const messageList = node.querySelector('ul');
 
 				if (!messageList) {

apps/meteor/client/views/room/body/hooks/useRestoreScrollPosition.ts

@@ -1,4 +1,4 @@
-import { useSafeRefCallback } from '@rocket.chat/ui-client';
+import { useSafeRefCallback } from '@rocket.chat/fuselage-hooks';
 import { useCallback, useRef } from 'react';
 
 import { isAtBottom } from '../../../../../app/ui/client/views/app/lib/scrolling';
@@ -9,10 +9,7 @@ export function useRestoreScrollPosition(rid: string, wait = 100) {
 	const jumpToRef = useRef<HTMLElement>(undefined);
 	const ref = useSafeRefCallback(
 		useCallback(
-			(node: HTMLElement | null) => {
-				if (!node) {
-					return;
-				}
+			(node: HTMLElement) => {
 				const store = RoomManager.getStore(rid);
 				if (store?.atBottom) {
 					node.scrollTop = node.scrollHeight;

apps/meteor/client/views/room/composer/RoomComposer/hooks/useAutoGrow.ts

@@ -1,4 +1,4 @@
-import { useSafeRefCallback } from '@rocket.chat/ui-client';
+import { useSafeRefCallback } from '@rocket.chat/fuselage-hooks';
 import type { CSSProperties, MutableRefObject, RefCallback } from 'react';
 import { useCallback } from 'react';
 
@@ -19,10 +19,6 @@ export const useAutoGrow = (
 	const autoGrowRef = useSafeRefCallback(
 		useCallback(
 			(node: HTMLTextAreaElement) => {
-				if (!node) {
-					return;
-				}
-
 				ref.current = node;
 
 				const resize = () => {

apps/meteor/client/views/room/composer/messageBox/MessageBox.tsx

@@ -1,7 +1,6 @@
 /* eslint-disable complexity */
 import { isRoomFederated, isRoomNativeFederated, type IMessage, type ISubscription } from '@rocket.chat/core-typings';
-import { useContentBoxSize, useEffectEvent } from '@rocket.chat/fuselage-hooks';
-import { useSafeRefCallback } from '@rocket.chat/ui-client';
+import { useContentBoxSize, useEffectEvent, useSafeRefCallback } from '@rocket.chat/fuselage-hooks';
 import {
 	MessageComposerAction,
 	MessageComposerToolbarActions,
@@ -358,9 +357,6 @@ const MessageBox = ({
 	const keyDownHandlerCallbackRef = useSafeRefCallback(
 		useCallback(
 			(node: HTMLTextAreaElement) => {
-				if (node === null) {
-					return;
-				}
 				const eventHandler = (e: KeyboardEvent) => keyboardEventHandler(e);
 				node.addEventListener('keydown', eventHandler);
 

apps/meteor/client/views/room/hooks/useIsVisible.ts

@@ -1,17 +1,12 @@
-import { useDebouncedState, useSafely } from '@rocket.chat/fuselage-hooks';
-import { useSafeRefCallback } from '@rocket.chat/ui-client';
+import { useDebouncedState, useSafely, useSafeRefCallback } from '@rocket.chat/fuselage-hooks';
 import { useCallback } from 'react';
 
 export const useIsVisible = () => {
 	const [menuVisibility, setMenuVisibility] = useSafely(useDebouncedState(!!window.DISABLE_ANIMATION, 100));
 
 	const callbackRef = useSafeRefCallback(
 		useCallback(
-			(node: HTMLElement | null) => {
-				if (!node) {
-					return;
-				}
-
+			(node: HTMLElement) => {
 				const observer = new IntersectionObserver((entries) => {
 					entries.forEach((entry) => {
 						setMenuVisibility(entry.isIntersecting);

packages/ui-client/src/hooks/index.ts

@@ -6,6 +6,5 @@ export * from './useFeaturePreview';
 export * from './useFeaturePreviewList';
 export * from './useLicense';
 export * from './usePreferenceFeaturePreviewList';
-export * from './useSafeRefCallback';
 export * from './useUserDisplayName';
 export * from './useValidatePassword';

packages/ui-voip/src/context/useMediaStream.ts

@@ -1,5 +1,5 @@
+import { useSafeRefCallback } from '@rocket.chat/fuselage-hooks';
 import type { MediaSignalingSession } from '@rocket.chat/media-signaling';
-import { useSafeRefCallback } from '@rocket.chat/ui-client';
 import { useCallback, useRef } from 'react';
 
 const getRemoteStream = (instance?: MediaSignalingSession) => {
@@ -29,12 +29,7 @@ const useMediaStream = (
 	return [
 		useSafeRefCallback(
 			useCallback(
-				(node) => {
-					// TODO remove node check when useSafeRefCallback is updated from fuselage.
-					if (!node) {
-						return;
-					}
-
+				(node: HTMLAudioElement) => {
 					actualRef.current = node;
 
 					if (!remoteStream) {

packages/ui-voip/src/hooks/VoipPopupDraggable/DraggableCore.ts

@@ -1,6 +1,6 @@
 import { Emitter } from '@rocket.chat/emitter';
 import type { OffCallbackHandler } from '@rocket.chat/emitter';
-import { useSafeRefCallback } from '@rocket.chat/ui-client';
+import { useSafeRefCallback } from '@rocket.chat/fuselage-hooks';
 import { useCallback, useRef, useState } from 'react';
 
 const GRAB_DOM_EVENTS = ['pointerdown'] as const;
@@ -531,11 +531,7 @@ export const useDraggable = () => {
 
 	const handleElementCallbackRef = useSafeRefCallback(
 		useCallback(
-			(node: HTMLElement | null) => {
-				if (!node) {
-					return;
-				}
-
+			(node: HTMLElement) => {
 				return handleElement.element.setElement(node);
 			},
 			[handleElement],
@@ -544,11 +540,7 @@ export const useDraggable = () => {
 
 	const draggableCallbackRef = useSafeRefCallback(
 		useCallback(
-			(node: HTMLElement | null) => {
-				if (!node) {
-					return;
-				}
-
+			(node: HTMLElement) => {
 				const offMove = draggableElement.onMove(() => {
 					restorePositionRef.current = node.getBoundingClientRect();
 				});
@@ -570,11 +562,7 @@ export const useDraggable = () => {
 
 	const boundingCallbackRef = useSafeRefCallback(
 		useCallback(
-			(node: HTMLElement | null) => {
-				if (!node) {
-					return;
-				}
-
+			(node: HTMLElement) => {
 				return boundingElement.element.setElement(node);
 			},
 			[boundingElement],