fixed node-pty conpty assertion failure - fixes #10618

Author: Eugeny

app/package.json

@@ -38,7 +38,7 @@
   "optionalDependencies": {
     "@tabby-gang/windows-blurbehind": "^3.1.0",
     "macos-native-processlist": "^2.1.0",
-    "patch-package": "^6.5.0",
+    "patch-package": "^8",
     "serialport": "11.0.1",
     "serialport-binding-webserialapi": "^1.0.3",
     "windows-native-registry": "^3.2.1",

app/patches/node-pty+1.1.0-beta9.patch

@@ -0,0 +1,29 @@
+diff --git a/node_modules/node-pty/src/win/conpty.cc b/node_modules/node-pty/src/win/conpty.cc
+index c41796c..08c6439 100644
+--- a/node_modules/node-pty/src/win/conpty.cc
++++ b/node_modules/node-pty/src/win/conpty.cc
+@@ -107,9 +107,22 @@ void SetupExitCallback(Napi::Env env, Napi::Function cb, pty_baton* baton) {
+     CloseHandle(baton->hOut);
+ 
+     auto status = tsfn.BlockingCall(exit_event, callback); // In main thread
+-    assert(status == napi_ok);
++    switch (status) {
++      case napi_closing:
++        break;
+ 
+-    tsfn.Release();
++      case napi_queue_full:
++        Napi::Error::Fatal("SetupExitCallback", "Queue was full");
++
++      case napi_ok:
++        if (tsfn.Release() != napi_ok) {
++          Napi::Error::Fatal("SetupExitCallback", "ThreadSafeFunction.Release() failed");
++        }
++        break;
++
++      default:
++        Napi::Error::Fatal("SetupExitCallback", "ThreadSafeFunction.BlockingCall() failed");
++    }
+   });
+ }
+