wulkano#129 cleanup before quitting while recording

main/aperture.ts

@@ -217,6 +217,32 @@ export const stopRecording = async () => {
   }
 };
 
+export const stopRecordingWithNoEdit = async () => {
+  // Ensure we only stop recording once
+  if (!past) {
+    return;
+  }
+
+  console.log(`Stopped recording after ${(Date.now() - past) / 1000}s`);
+  past = undefined;
+
+  try {
+    await aperture.stopRecording();
+  } catch (error) {
+    track('recording/quit/error');
+    showError(error as any, {title: 'Recording error', plugin: undefined});
+    cleanup();
+    return;
+  }
+
+  try {
+    cleanup();
+  } finally {
+    track('recording/quit');
+    stopCurrentRecording(recordingName);
+  }
+};
+
 export const pauseRecording = async () => {
   // Ensure we only pause if there's a recording in progress and if it's currently not paused
   const isPaused = await aperture.isPaused();

main/index.ts

@@ -23,11 +23,14 @@ import {setupRemoteStates} from './remote-states';
 import {setUpExportsListeners} from './export';
 import {windowManager} from './windows/manager';
 import {setupProtocol} from './utils/protocol';
+import {stopRecordingWithNoEdit} from './aperture';
 
 const prepareNext = require('electron-next');
 
 const filesToOpen: string[] = [];
 
+let onExitCleanupComplete = false;
+
 app.commandLine.appendSwitch('--enable-features', 'OverlayScrollbar');
 
 app.on('open-file', (event, path) => {
@@ -133,6 +136,12 @@ app.on('will-finish-launching', () => {
   });
 });
 
-app.on('quit', () => {
-  cleanPastRecordings();
+app.on('before-quit', async (event: any) => {
+  if (!onExitCleanupComplete) {
+    event.preventDefault();
+    await stopRecordingWithNoEdit();
+    cleanPastRecordings();
+    onExitCleanupComplete = true;
+    app.quit();
+  }
 });