fix(restart): optional stopping miners for restart application (#1060)

Added a flag for terminating processes before restarting the app. We
don't want to kill processes during auto-updates, but in other cases we
want to kill them and prevent from hanging processes and killing them
after the restart.

Co-authored-by: Brian Pearce <[email protected]>

src-tauri/src/main.rs

@@ -468,11 +468,15 @@ async fn set_monerod_config(
 
 #[tauri::command]
 async fn restart_application(
+    should_stop_miners: bool,
     _window: tauri::Window,
-    _state: tauri::State<'_, UniverseAppState>,
+    state: tauri::State<'_, UniverseAppState>,
     app: tauri::AppHandle,
 ) -> Result<(), String> {
-    // This restart doesn't need to shutdown all the miners
+    if should_stop_miners {
+        stop_all_miners(state.inner().clone(), 5).await?;
+    }
+
     app.restart();
     Ok(())
 }

src/components/dialogs/RestartDialog.tsx

@@ -29,7 +29,7 @@ export default function RestartDialog() {
     const handleRestart = useCallback(async () => {
         try {
             console.info('Restarting application.');
-            await invoke('restart_application');
+            await invoke('restart_application', { shouldStopMiners: true });
         } catch (error) {
             Sentry.captureException(error);
             console.error('Restart error: ', error);

src/containers/floating/ExternalDependenciesDialog/ExternalDependenciesDialog.tsx

@@ -28,7 +28,7 @@ export const ExternalDependenciesDialog = () => {
     const handleRestart = useCallback(async () => {
         try {
             setIsRestarting(true);
-            await invoke('restart_application');
+            await invoke('restart_application', { shouldStopMiners: true });
         } catch (e) {
             Sentry.captureException(e);
             console.error('Error restarting application:', e);

src/hooks/useUpdateStatus.ts

@@ -77,7 +77,7 @@ export const useHandleUpdate = () => {
         console.info('Installing latest version of Tari Universe');
         try {
             console.info('Restarting application after update');
-            await invoke('restart_application');
+            await invoke('restart_application', { shouldStopMiners: false });
         } catch (e) {
             Sentry.captureException(e);
             console.error('Relaunch error', e);

src/types/invoke.ts

@@ -59,7 +59,7 @@ declare module '@tauri-apps/api/tauri' {
     ): Promise<void>;
     function invoke(param: 'set_cpu_mining_enabled', payload: { enabled: boolean }): Promise<void>;
     function invoke(param: 'exit_application'): Promise<string>;
-    function invoke(param: 'restart_application'): Promise<string>;
+    function invoke(param: 'restart_application', payload: { shouldStopMiners: boolean }): Promise<string>;
     function invoke(param: 'set_use_tor', payload: { useTor: boolean }): Promise<void>;
     function invoke(param: 'get_transaction_history'): Promise<TransactionInfo[]>;
     function invoke(param: 'import_seed_words', payload: { seedWords: string[] }): Promise<void>;