enh: Ask user: Capture error in Sentry when app crashes (#184)

* enh: Ask user: Capture error in Sentry when app crashes * Reverting testing changes * Small update * Removing saving portion of settings * Reverting UI change * Reverting changes * Reverting UI change --------- Co-authored-by: Oto Ciulis <[email protected]>
Comfy-Org · Nov 8, 2024 · bcf2877 · bcf2877
1 parent f6dbbb5
commit bcf2877
Showing 1 changed file with 9 additions and 4 deletions.
diff --git a/src/main.ts b/src/main.ts
@@ -102,13 +102,18 @@ if (!gotTheLock) {
   Sentry.init({
     dsn: SENTRY_URL_ENDPOINT,
     autoSessionTracking: false,
-    beforeSend(event, hint) {
-      if (event.extra?.comfyUIExecutionError) {
+    async beforeSend(event, hint) {
+      if (event.extra?.comfyUIExecutionError || comfySettings.sendCrashStatistics) {
         return event;
       }
 
-      //TODO (use default pop up behavior).
-      return event;
+      const { response } = await dialog.showMessageBox({
+        title: 'Send Crash Statistics',
+        message: `Would you like to send crash statistics to the team?`,
+        buttons: ['Always send crash reports', 'Do not send crash report'],
+      });
+
+      return response === 0 ? event : null;
     },
     integrations: [
       Sentry.childProcessIntegration({