**fix: add bug reporting to exception handlers**

- Integrated `SharedBugReportService` and `ReportBugAsync` calls across ArchiveService, UpdateService, MainWindow, and related error paths.
- Updated BugReportService payload to match API model (removed extraneous fields).
- Bumped version to 2.6.1.

Author: drpetersonfernandes

BatchConvertToCHD/BatchConvertToCHD.csproj

@@ -8,8 +8,8 @@
     <UseWPF>true</UseWPF>
     <ApplicationIcon>icon\chd.ico</ApplicationIcon>
     <SupportedOSPlatformVersion>7.0</SupportedOSPlatformVersion>
-    <AssemblyVersion>2.6.0</AssemblyVersion>
-    <FileVersion>2.6.0</FileVersion>
+    <AssemblyVersion>2.6.1</AssemblyVersion>
+    <FileVersion>2.6.1</FileVersion>
     <IsPackable>false</IsPackable>
     <NeutralLanguage>en</NeutralLanguage>
     <DebugType>embedded</DebugType>

BatchConvertToCHD/MainWindow.xaml.cs

@@ -321,6 +321,7 @@ private async Task<bool> ValidateExecutableAccessAsync(string exePath, string ex
         {
             LogMessage($"ERROR: Cannot access {exeName}. {ex.Message}");
             ShowError($"Cannot access {exeName}. Check antivirus or permissions.");
+            _ = ReportBugAsync($"Cannot access {exeName}", ex);
             return false;
         }
     }
@@ -418,6 +419,7 @@ private async Task<bool> ValidateChdmanCompatibilityAsync(string chdmanPath, Can
 
             // Other errors are acceptable - at least the exe started or we have a generic error
             LogMessage($"WARNING: Could not validate chdman compatibility: {ex.Message}");
+            _ = ReportBugAsync("Could not validate chdman compatibility", ex);
             return true;
         }
     }
@@ -1423,6 +1425,7 @@ private async Task<bool> ProcessSingleFileForConversionAsync(string chdmanPath,
             catch (Exception ex)
             {
                 LogMessage($"Direct conversion attempt error for {originalName}: {ex.Message}");
+                _ = ReportBugAsync($"Direct conversion attempt error for {originalName}", ex);
             }
 
             // Fallback: If direct conversion failed and we haven't already extracted to temp (i.e. it was a direct file attempt),
@@ -1491,6 +1494,7 @@ private async Task<bool> ProcessSingleFileForConversionAsync(string chdmanPath,
                 catch (Exception ex)
                 {
                     LogMessage($"Retry via temp failed for {originalName}: {ex.Message}");
+                    _ = ReportBugAsync($"Retry via temp failed for {originalName}", ex);
                 }
             }
 
@@ -1543,6 +1547,7 @@ private async Task<bool> ProcessSingleFileForConversionAsync(string chdmanPath,
         catch (Exception ex)
         {
             LogMessage($"Error processing {originalName}: {ex.Message}");
+            _ = ReportBugAsync($"Error processing {originalName}", ex);
             if (!string.IsNullOrEmpty(outputChd)) await TryDeleteFileAsync(outputChd, "failed CHD", CancellationToken.None);
             return false;
         }
@@ -1667,6 +1672,7 @@ private static async Task MoveVerifiedFileAsync(string sourceFile, string target
         {
             // Log error but don't fail the verification
             Debug.WriteLine($"Failed to move file {sourceFile}: {ex.Message}");
+            _ = ReportBugAsync($"Failed to move file {sourceFile}", ex);
         }
     }
 
@@ -2498,6 +2504,7 @@ private async Task DeleteOriginalGameFilesAsync(string inputFile, CancellationTo
         catch (Exception ex)
         {
             LogMessage($"Delete error: {ex.Message}");
+            _ = ReportBugAsync("Delete error", ex);
         }
     }
 

BatchConvertToCHD/Services/ArchiveService.cs

@@ -188,6 +188,19 @@ public ArchiveService(string maxCsoPath, bool isMaxCsoAvailable)
         }
         catch (Exception ex)
         {
+            // Report bug if service is available
+            try
+            {
+                if (App.SharedBugReportService != null)
+                {
+                    _ = App.SharedBugReportService.SendBugReportAsync("Error extracting archive", ex);
+                }
+            }
+            catch
+            {
+                // Ignore errors in bug reporting to avoid infinite loops
+            }
+
             return (false, new List<string>(), tempDirectoryRoot, $"Error extracting archive: {ex.Message}");
         }
     }
@@ -273,6 +286,18 @@ private static void ExtractArchiveWithFallback<TArchive>(
         catch (Exception ex)
         {
             onLog($"Direct extraction failed ({ex.Message}). Attempting fallback with local copy...");
+            // Report bug if service is available
+            try
+            {
+                if (App.SharedBugReportService != null)
+                {
+                    _ = App.SharedBugReportService.SendBugReportAsync("Direct extraction failed", ex);
+                }
+            }
+            catch
+            {
+                // Ignore errors in bug reporting to avoid infinite loops
+            }
         }
 
         if (directExtractionSuccess)

BatchConvertToCHD/Services/BugReportService.cs

@@ -62,28 +62,22 @@ public async Task<bool> SendBugReportAsync(string message, Exception? ex = null)
             // Build the formatted message with all details
             var formattedMessage = BuildFormattedReport(message, ex);
 
-            // Get environment details
+            // Get environment details for version field
             var envDetails = GetEnvironmentDetails();
 
             // Get exception stack trace
             var stackTrace = GetExceptionStackTrace(ex);
 
             // Create the request payload matching the API's BugReportRequest model
+            // Only include fields defined in BugReportRequest: Message, ApplicationName, Version, UserInfo, Environment, StackTrace
             var requestPayload = new
             {
                 message = formattedMessage,
                 applicationName = _applicationName,
                 version = envDetails.ApplicationVersion,
                 userInfo = Environment.UserName,
                 environment = "Production",
-                stackTrace,
-                osVersion = envDetails.OsVersion,
-                architecture = envDetails.Architecture,
-                bitness = envDetails.Bitness,
-                windowsVersion = envDetails.WindowsVersion,
-                processorCount = envDetails.ProcessorCount,
-                baseDirectory = envDetails.BaseDirectory,
-                tempPath = envDetails.TempPath
+                stackTrace
             };
 
             // Create JSON content

BatchConvertToCHD/Services/UpdateService.cs

@@ -116,6 +116,7 @@ await Application.Current.Dispatcher.InvokeAsync(() =>
                         catch (Exception urlEx)
                         {
                             onLog($"Failed to open browser: {urlEx.Message}");
+                            _ = onBugReport("Failed to open browser", urlEx);
 
                             // Copy URL to clipboard
                             try
@@ -125,6 +126,7 @@ await Application.Current.Dispatcher.InvokeAsync(() =>
                             catch (Exception clipboardEx)
                             {
                                 onLog($"Failed to copy URL to clipboard: {clipboardEx.Message}");
+                                _ = onBugReport("Failed to copy URL to clipboard", clipboardEx);
                             }
 
                             // Show URL in message box so user can manually access it