Fix: Add more error handling

Co-Authored-By: SamsidParty <[email protected]>

Author: SamarthMP

Docs/Screenshot2.pdn

@@ -3,7 +3,7 @@
   <!--Package created by MSIX Packaging Tool version: 1.2023.807.0-->
   <!--Use CN=68C2D20A-96CA-43CC-A323-A549C2786CDA For Prod-->
   <!--Use CN=SamsidParty For Beta-->
-  <Identity Name="55968SamsidGameStudios.TopNotify" Publisher="CN=68C2D20A-96CA-43CC-A323-A549C2786CDA" Version="3.0.0.0" ProcessorArchitecture="x64" />
+  <Identity Name="55968SamsidGameStudios.TopNotify" Publisher="CN=68C2D20A-96CA-43CC-A323-A549C2786CDA" Version="3.0.1.0" ProcessorArchitecture="x64" />
   <Properties>
     <DisplayName>TopNotify</DisplayName>
     <PublisherDisplayName>SamsidParty</PublisherDisplayName>

Docs/Screenshot2.png

@@ -19,30 +19,56 @@ public class Program
 
         public static Daemon.Daemon Background;
         public static AppManager GUI;
+        public static IEnumerable<Process> ValidTopNotifyInstances;
+
+        public static bool IsDaemonRunning => ValidTopNotifyInstances.Where((p) => {
+            try
+            {
+                string commandLine;
+                ProcessCommandLine.Retrieve(p, out commandLine, ProcessCommandLine.Parameter.CommandLine);
+                return !commandLine.ToLower().Contains("--settings");
+            }
+            catch { }
+            return false;
+        }).Any();
+
+        public static bool IsGUIRunning => ValidTopNotifyInstances.Where((p) => {
+            try
+            {
+                string commandLine;
+                ProcessCommandLine.Retrieve(p, out commandLine, ProcessCommandLine.Parameter.CommandLine);
+                return commandLine.ToLower().Contains("--settings");
+            }
+            catch { }
+            return false;
+        }).Any();
 
         [STAThread]
         public static void Main(string[] args)
         {
+            AppDomain.CurrentDomain.UnhandledException += (object sender, UnhandledExceptionEventArgs e) =>
+            {
+                NotificationTester.MessageBox("Something went wrong with TopNotify", "Unfortunately, TopNotify has crashed. Details: " + e.ExceptionObject.ToString());
+            };
+
             //By Default, The App Will Be Launched In Daemon Mode
             //Daemon Mode Is A Background Process That Handles Changing The Position Of Notifications
             //If The "--settings" Arg Is Used, Then The App Will Launch In Settings Mode
             //Settings Mode Shows A GUI That Can Be Used To Configure The App
             //These Mode Switches Ensure All Functions Of The App Use The Same Executable
 
             //Find Other Instances Of TopNotify
-            var validTopNotifyInstances = Process.GetProcessesByName("TopNotify").Where((p) => !p.HasExited && p.Id != Process.GetCurrentProcess().Id);
-
-            var isGUIRunning = validTopNotifyInstances.Where((p) => {
-                string commandLine;
-                ProcessCommandLine.Retrieve(p, out commandLine, ProcessCommandLine.Parameter.CommandLine);
-                return commandLine.ToLower().Contains("--settings");
-            }).Any();
-
-            var isDaemonRunning = validTopNotifyInstances.Where((p) => {
-                string commandLine;
-                ProcessCommandLine.Retrieve(p, out commandLine, ProcessCommandLine.Parameter.CommandLine);
-                return !commandLine.ToLower().Contains("--settings");
-            }).Any();
+            ValidTopNotifyInstances = Process.GetProcessesByName("TopNotify").Where((p) => {
+                try
+                {
+                    return !p.HasExited && p.Id != Process.GetCurrentProcess().Id;
+                }
+                catch { }
+                return false;
+            });
+
+            var isGUIRunning = IsGUIRunning;
+            var isDaemonRunning = IsDaemonRunning;
 
             #if !GUI_DEBUG
             if (!args.Contains("--settings") && isDaemonRunning && !isGUIRunning)