fix: Move mixxing dependency logic to main process

electron/main.ts

@@ -38,6 +38,8 @@ import {createMenu, getDockMenu} from './menu';
 import initKubeconfig from './src/initKubeconfig';
 import terminal from '../cli/terminal';
 import {downloadPlugin} from './pluginService';
+import {AlertEnum, AlertType} from '@models/alert';
+import {setAlert} from '@redux/reducers/alert';
 
 Object.assign(console, ElectronLog.functions);
 autoUpdater.logger = console;
@@ -88,13 +90,6 @@ ipcMain.on('run-kustomize', (event, folder: string) => {
   }
 });
 
-ipcMain.on('check-missing-dependency', event => {
-  const missingDependecies = checkMissingDependencies(APP_DEPENDENCIES);
-  if (missingDependecies.length > 0) {
-    event.sender.send('missing-dependency-result', {dependencies: missingDependecies});
-  }
-});
-
 ipcMain.handle('select-file', async (event, options: any) => {
   const browserWindow = BrowserWindow.fromId(event.sender.id);
   let dialogOptions: Electron.OpenDialogSyncOptions = {};
@@ -246,17 +241,19 @@ export const createWindow = (givenPath?: string) => {
     mainStore.dispatch(updateNewVersion({code: NewVersionCode.Downloaded, data: null}));
   });
 
-  const missingDependecies = checkMissingDependencies(APP_DEPENDENCIES);
-
-  if (missingDependecies.length > 0) {
-    win.webContents.on('did-finish-load', () => {
-      win.webContents.send('missing-dependency-result', {dependencies: missingDependecies});
-    });
-  }
-
   win.webContents.on('did-finish-load', async () => {
     await checkNewVersion(true);
     initKubeconfig(mainStore, userHomeDir);
+    const missingDependecies = checkMissingDependencies(APP_DEPENDENCIES);
+
+    if (missingDependecies.length > 0) {
+      const alert: AlertType = {
+        type: AlertEnum.Warning,
+        title: 'Missing dependency',
+        message: `${missingDependecies.toString()} must be installed for all Monokle functionality to be available`,
+      };
+      mainStore.dispatch(setAlert(alert));
+    }
     win.webContents.send('executed-from', {path: givenPath});
   });
 

src/App.tsx

@@ -1,4 +1,4 @@
-import React, {useCallback, useEffect} from 'react';
+import React from 'react';
 import 'antd/dist/antd.less';
 import {Layout} from '@atoms';
 import {
@@ -15,39 +15,15 @@ import {
 } from '@organisms';
 import {Size} from '@models/window';
 import {useWindowSize} from '@utils/hooks';
-import {useAppDispatch} from '@redux/hooks';
-import {ipcRenderer} from 'electron';
-import {setAlert} from '@redux/reducers/alert';
-import {AlertEnum, AlertType} from '@models/alert';
 import ValidationErrorsModal from '@components/molecules/ValidationErrorsModal';
 import UpdateModal from '@components/organisms/UpdateModal';
 import AppContext from './AppContext';
 
 const App = () => {
-  const dispatch = useAppDispatch();
   const size: Size = useWindowSize();
 
   const mainHeight = `${size.height}px`;
 
-  const onMissingDependencyResult = useCallback(
-    (_, {dependencies}) => {
-      const alert: AlertType = {
-        type: AlertEnum.Warning,
-        title: 'Missing dependency',
-        message: `${dependencies.toString()} must be installed for all Monokle functionality to be available`,
-      };
-      dispatch(setAlert(alert));
-    },
-    [dispatch]
-  );
-
-  useEffect(() => {
-    ipcRenderer.on('missing-dependency-result', onMissingDependencyResult);
-    return () => {
-      ipcRenderer.removeListener('missing-dependency-result', onMissingDependencyResult);
-    };
-  }, [onMissingDependencyResult]);
-
   return (
     <AppContext.Provider value={{windowSize: size}}>
       <div style={{overflowY: 'hidden'}}>

src/redux/thunks/previewCluster.ts

@@ -55,11 +55,9 @@ const previewClusterHandler = async (configPath: string, thunkAPI: any) => {
               type: AlertEnum.Warning,
             };
 
-            console.log('previewResult1', previewResult);
             return previewResult;
           }
         }
-        console.log('previewResult2', previewResult);
         return previewResult;
       },
       reason => {