Better cleanup, fix issue with noise simulation

Author: pierr3

backend/extern/afv-native

@@ -34,6 +34,6 @@
     "author": "Pierre Ferran",
     "license": "GPL-3.0-only",
     "engines": {
-        "node": ">=20.0.0 <=21.0.0"
+        "node": ">=20.0.0 <=23.0.0"
     }
 }

backend/package-lock.json

@@ -16,7 +16,11 @@ InputHandler::InputHandler()
     timer.start(Poco::TimerCallback<InputHandler>(*this, &InputHandler::onTimer));
 }
 
-InputHandler::~InputHandler() { timer.stop(); }
+InputHandler::~InputHandler()
+{
+    timer.stop();
+    uioHookWrapper_->stop();
+}
 
 void InputHandler::startPttSetup(int pttIndex, bool listenForJoysticks)
 {

backend/package.json

@@ -1,7 +1,7 @@
 #include "Shared.hpp"
 #include "Helpers.hpp"
 
-const semver::version VERSION = semver::version { 1, 3, 2 };
+const semver::version VERSION = semver::version { 1, 3, 3 };
 const std::string CLIENT_NAME = std::string("TrackAudio-") + VERSION.to_string();
 std::unique_ptr<afv_native::api::atcClient> mClient = nullptr;
 

backend/src/InputHandler.cpp

@@ -838,15 +838,24 @@ void SetSession(const Napi::CallbackInfo& info)
 Napi::Boolean Exit(const Napi::CallbackInfo& info)
 {
     PLOGI << "Awaiting to exit TrackAudio...";
+
+    MainThreadShared::mApiServer.reset();
+    MainThreadShared::mRemoteDataHandler.reset();
+    MainThreadShared::inputHandler.reset();
+    MainThreadShared::vuMeterThread.reset();
+
     NapiHelpers::_requestExit.store(true);
     if (mClient->IsVoiceConnected()) {
         PLOGI << "Connection to network detected, forcing disconnect...";
         mClient->Disconnect();
     }
 
-    MainThreadShared::mApiServer.reset();
-    MainThreadShared::mRemoteDataHandler.reset();
-    MainThreadShared::inputHandler.reset();
+    if (mClient->IsAudioRunning()) {
+        PLOGI << "Audio running, stopping...";
+        mClient->StopAudio();
+    }
+
+
 
     mClient.reset();
     PLOGI << "Exiting TrackAudio...";

backend/src/Shared.cpp

@@ -1,7 +1,7 @@
 {
   "name": "trackaudio",
   "productName": "TrackAudio",
-  "version": "1.3.2",
+  "version": "1.3.3",
   "main": "./out/main/index.js",
   "author": "github.com/pierr3",
   "description": "TrackAudio - Audio for VATSIM Client",

backend/src/main.cpp

@@ -431,11 +431,18 @@ app
     console.log(`Error initializing app: ${err.message}`);
   });
 
+app.on('before-quit', () => {
+  // Perform cleanup
+  if (TrackAudioAfv.IsConnected()) {
+    TrackAudioAfv.Disconnect();
+  }
+  TrackAudioAfv.Exit();
+});
+
 // Quit when all windows are closed, except on macOS. There, it's common
 // for applications and their menu bar to stay active until the user quits
 // explicitly with Cmd + Q.
 app.on('window-all-closed', () => {
-  TrackAudioAfv.Exit();
   app.quit();
 });
 
@@ -575,15 +582,12 @@ ipcMain.handle(
   }
 );
 
-ipcMain.handle(
-  'audio-remove-frequency',
-  (_, frequency: number, callsign?: string) => {
-    if (callsign) {
-      TrackAudioAfv.RemoveFrequency(frequency, callsign);
-    }
-    TrackAudioAfv.RemoveFrequency(frequency);
+ipcMain.handle('audio-remove-frequency', (_, frequency: number, callsign?: string) => {
+  if (callsign) {
+    TrackAudioAfv.RemoveFrequency(frequency, callsign);
   }
-);
+  TrackAudioAfv.RemoveFrequency(frequency);
+});
 
 ipcMain.handle(
   'audio-set-frequency-state',