chore: stop importing action manager as global (#29766)

apps/meteor/app/ui-message/client/ActionManager.js

@@ -1,18 +1,17 @@
 import { UIKitIncomingInteractionType } from '@rocket.chat/apps-engine/definition/uikit';
-import { Meteor } from 'meteor/meteor';
 import { Random } from '@rocket.chat/random';
 import { Emitter } from '@rocket.chat/emitter';
 import { UIKitInteractionTypes } from '@rocket.chat/core-typings';
+import { lazy } from 'react';
 
-import Notifications from '../../notifications/client/lib/Notifications';
-import { CachedCollectionManager } from '../../ui-cached-collection/client';
-import { t } from '../../utils/client';
+import { t } from '../../utils/lib/i18n';
 import * as banners from '../../../client/lib/banners';
 import { dispatchToastMessage } from '../../../client/lib/toast';
-import { imperativeModal } from '../../../client/lib/imperativeModal';
-import UiKitModal from '../../../client/views/modal/uikit/UiKitModal';
 import { sdk } from '../../utils/client/lib/SDKClient';
 import { router } from '../../../client/providers/RouterProvider';
+import { imperativeModal } from '../../../client/lib/imperativeModal';
+
+const UiKitModal = lazy(() => import('../../../client/views/modal/uikit/UiKitModal'));
 
 const events = new Emitter();
 
@@ -45,7 +44,7 @@ export const generateTriggerId = (appId) => {
 	return triggerId;
 };
 
-const handlePayloadUserInteraction = (type, { /* appId,*/ triggerId, ...data }) => {
+export const handlePayloadUserInteraction = (type, { /* appId,*/ triggerId, ...data }) => {
 	if (!triggersId.has(triggerId)) {
 		return;
 	}
@@ -256,11 +255,3 @@ export const getUserInteractionPayloadByViewId = (viewId) => {
 
 	return instance.payload;
 };
-
-Meteor.startup(() =>
-	CachedCollectionManager.onLogin(() =>
-		Notifications.onUser('uiInteraction', ({ type, ...data }) => {
-			handlePayloadUserInteraction(type, data);
-		}),
-	),
-);

apps/meteor/app/ui-utils/client/lib/MessageAction.ts

@@ -66,7 +66,7 @@ export type MessageActionConfig = {
 			chat,
 			autoTranslateOptions,
 		}: {
-			message?: IMessage & Partial<ITranslatedMessage>;
+			message: IMessage & Partial<ITranslatedMessage>;
 			tabbar: ToolboxContextValue;
 			room?: IRoom;
 			chat: ContextType<typeof ChatContext>;

apps/meteor/app/ui-utils/client/lib/messageBox.ts

@@ -3,7 +3,7 @@ import type { TranslationKey } from '@rocket.chat/ui-contexts';
 
 import type { ChatAPI } from '../../../../client/lib/chats/ChatAPI';
 
-type MessageBoxAction = {
+export type MessageBoxAction = {
 	label: TranslationKey;
 	id: string;
 	icon?: string;

apps/meteor/client/UIKit/hooks/useUIKitHandleAction.tsx

@@ -5,17 +5,19 @@ import { UIKitIncomingInteractionContainerType } from '@rocket.chat/apps-engine/
 // import React, { Context, FC, useMemo } from 'react';
 import type { UiKitPayload, UIKitActionEvent } from '@rocket.chat/core-typings';
 import { useMutableCallback } from '@rocket.chat/fuselage-hooks';
+
+import { useUiKitActionManager } from '../../hooks/useUiKitActionManager';
 // import { UIKitIncomingInteractionContainerType } from '@rocket.chat/apps-engine/definition/uikit/UIKitIncomingInteractionContainer';
 
 // import { useEndpoint } from '@rocket.chat/ui-contexts';
-import * as ActionManager from '../../../app/ui-message/client/ActionManager';
 
-const useUIKitHandleAction = <S extends UiKitPayload>(state: S): ((event: UIKitActionEvent) => Promise<void>) =>
-	useMutableCallback(async ({ blockId, value, appId, actionId }) => {
+const useUIKitHandleAction = <S extends UiKitPayload>(state: S): ((event: UIKitActionEvent) => Promise<void>) => {
+	const actionManager = useUiKitActionManager();
+	return useMutableCallback(async ({ blockId, value, appId, actionId }) => {
 		if (!appId) {
 			throw new Error('useUIKitHandleAction - invalid appId');
 		}
-		return ActionManager.triggerBlockAction({
+		return actionManager.triggerBlockAction({
 			container: {
 				type: UIKitIncomingInteractionContainerType.VIEW,
 				id: state.viewId || state.appId,
@@ -26,5 +28,6 @@ const useUIKitHandleAction = <S extends UiKitPayload>(state: S): ((event: UIKitA
 			blockId,
 		});
 	});
+};
 
 export { useUIKitHandleAction };

apps/meteor/client/UIKit/hooks/useUIKitHandleClose.tsx

@@ -6,28 +6,30 @@ import type { UIKitInteractionType } from '@rocket.chat/apps-engine/definition/u
 import type { UiKitPayload } from '@rocket.chat/core-typings';
 import { useMutableCallback } from '@rocket.chat/fuselage-hooks';
 import { useToastMessageDispatch } from '@rocket.chat/ui-contexts';
+
+import { useUiKitActionManager } from '../../hooks/useUiKitActionManager';
 // import { UIKitIncomingInteractionContainerType } from '@rocket.chat/apps-engine/definition/uikit/UIKitIncomingInteractionContainer';
 
 // import { useEndpoint } from '@rocket.chat/ui-contexts';
 
-import * as ActionManager from '../../../app/ui-message/client/ActionManager';
-
 // eslint-disable-next-line @typescript-eslint/no-unused-vars
 const emptyFn = (_error: any, _result: UIKitInteractionType | void): void => undefined;
 
 const useUIKitHandleClose = <S extends UiKitPayload>(state: S, fn = emptyFn): (() => Promise<void | UIKitInteractionType>) => {
+	const actionManager = useUiKitActionManager();
 	const dispatchToastMessage = useToastMessageDispatch();
 	return useMutableCallback(() =>
-		ActionManager.triggerCancel({
-			appId: state.appId,
-			viewId: state.viewId,
-			view: {
-				...state,
-				id: state.viewId,
-				// state: groupStateByBlockId(values),
-			},
-			isCleared: true,
-		})
+		actionManager
+			.triggerCancel({
+				appId: state.appId,
+				viewId: state.viewId,
+				view: {
+					...state,
+					id: state.viewId,
+					// state: groupStateByBlockId(values),
+				},
+				isCleared: true,
+			})
 			.then((result) => fn(undefined, result))
 			.catch((error) => {
 				dispatchToastMessage({ type: 'error', message: error });

apps/meteor/client/UIKit/hooks/useUIKitStateManager.tsx

@@ -3,9 +3,10 @@ import { isErrorType } from '@rocket.chat/core-typings';
 import { useSafely } from '@rocket.chat/fuselage-hooks';
 import { useEffect, useState } from 'react';
 
-import * as ActionManager from '../../../app/ui-message/client/ActionManager';
+import { useUiKitActionManager } from '../../hooks/useUiKitActionManager';
 
 const useUIKitStateManager = <S extends UiKitPayload>(initialState: S): S => {
+	const actionManager = useUiKitActionManager();
 	const [state, setState] = useSafely(useState(initialState));
 
 	const { viewId } = state;
@@ -22,10 +23,10 @@ const useUIKitStateManager = <S extends UiKitPayload>(initialState: S): S => {
 			setState(rest as any);
 		};
 
-		ActionManager.on(viewId, handleUpdate);
+		actionManager.on(viewId, handleUpdate);
 
 		return (): void => {
-			ActionManager.off(viewId, handleUpdate);
+			actionManager.off(viewId, handleUpdate);
 		};
 	}, [setState, viewId]);