**refactor: add XML docs, improve exception handling and archive extraction**

- Added comprehensive XML documentation to `GameFileParser`, `ArchiveService`, `UpdateService`, `FileItem`, and new `IExternalProcessService`.
- Standardized exception rethrow using `ExceptionDispatchInfo.Capture().Throw()` in parsers.
- Refactored archive extraction with `ExtractArchiveWithFallback` and `ExtractArchiveEntries` to eliminate duplication.
- Switched to shared `SocketsHttpHandler` in `BugReportService` and `UpdateService` for connection pooling.
- Updated `FileItem` property setters with early returns and minor cleanups.

Author: drpetersonfernandes

BatchConvertToCHD/AboutWindow.xaml.cs

@@ -5,8 +5,14 @@
 
 namespace BatchConvertToCHD;
 
+/// <summary>
+/// About window displaying application version and information.
+/// </summary>
 public partial class AboutWindow
 {
+    /// <summary>
+    /// Initializes a new instance of the <see cref="AboutWindow"/> class.
+    /// </summary>
     public AboutWindow()
     {
         InitializeComponent();

BatchConvertToCHD/App.xaml

@@ -306,23 +306,6 @@
             </Setter>
         </Style>
 
-        <!-- Modern Log TextBox Style -->
-        <Style x:Key="ModernLogTextBox" TargetType="TextBox" BasedOn="{StaticResource {x:Type TextBox}}">
-            <Setter Property="FontFamily" Value="Consolas"/>
-            <Setter Property="FontSize" Value="12"/>
-            <Setter Property="Background" Value="{StaticResource TerminalBackgroundBrush}"/>
-            <Setter Property="Foreground" Value="{StaticResource TerminalForegroundBrush}"/>
-            <Setter Property="BorderThickness" Value="0"/>
-            <Setter Property="Padding" Value="10"/>
-            <Setter Property="AcceptsReturn" Value="True"/>
-            <Setter Property="VerticalScrollBarVisibility" Value="Auto"/>
-            <Setter Property="HorizontalScrollBarVisibility" Value="Auto"/>
-            <Setter Property="IsReadOnly" Value="True"/>
-            <Setter Property="TextAlignment" Value="Left"/>
-            <Setter Property="TextWrapping" Value="NoWrap"/>
-            <Setter Property="CaretBrush" Value="{StaticResource TerminalForegroundBrush}"/>
-        </Style>
-
         <!-- Default StatusBar Style -->
         <Style TargetType="StatusBar">
             <Setter Property="Background" Value="{StaticResource StatusBarBackgroundBrush}"/>

BatchConvertToCHD/App.xaml.cs

@@ -5,6 +5,9 @@
 
 namespace BatchConvertToCHD;
 
+/// <summary>
+/// Application class for BatchConvertToCHD. Handles startup, exception handling, and service initialization.
+/// </summary>
 public partial class App : IDisposable
 {
     private BugReportService? _bugReportService;
@@ -15,6 +18,10 @@ public partial class App : IDisposable
     /// </summary>
     public static BugReportService? SharedBugReportService { get; private set; }
 
+    /// <summary>
+    /// Initializes a new instance of the <see cref="App"/> class.
+    /// Sets up services, exception handling, and event handlers.
+    /// </summary>
     public App()
     {
         // Initialize services
@@ -95,12 +102,9 @@ private static void DeleteOldDllFiles()
 
     private void App_Exit(object sender, ExitEventArgs e)
     {
-        // Dispose services and clear references to prevent double disposal
-        _bugReportService?.Dispose();
+        // Clear references (HttpClient instances are static and reused across the app lifetime)
         _bugReportService = null;
         SharedBugReportService = null;
-
-        _statsService?.Dispose();
         _statsService = null;
 
         // Unregister static event handlers to prevent memory leaks
@@ -152,12 +156,9 @@ private void ReportException(Exception exception, string source)
     public void Dispose()
     {
         // Cleanup is primarily handled in App_Exit. This method provides a safety net
-        // for explicit disposal scenarios and prevents double disposal.
-        _bugReportService?.Dispose();
+        // for explicit disposal scenarios. HttpClient instances are static and reused.
         _bugReportService = null;
         SharedBugReportService = null;
-
-        _statsService?.Dispose();
         _statsService = null;
 
         // Unregister static event handlers to prevent them from firing after disposal

BatchConvertToCHD/MainWindow.xaml

@@ -101,7 +101,7 @@
                                     <StackPanel>
                                         <CheckBox x:Name="SearchSubfoldersConversionCheckBox" Content="Search subfolders (recursive search)" Margin="0,2" HorizontalAlignment="Left" ToolTip="Recursively search for files in all subfolders"
                                                   Checked="SearchSubfoldersCheckBox_Changed" Unchecked="SearchSubfoldersCheckBox_Changed" />
-                                        <CheckBox x:Name="DeleteOriginalsCheckBox" Content="Delete originals after a successfull conversion" Margin="0,2" HorizontalAlignment="Left" VerticalAlignment="Top" ToolTip="Automatically delete source files once they are successfully converted to CHD" />
+                                        <CheckBox x:Name="DeleteOriginalsCheckBox" Content="Delete originals after a successful conversion" Margin="0,2" HorizontalAlignment="Left" VerticalAlignment="Top" ToolTip="Automatically delete source files once they are successfully converted to CHD" />
                                         <CheckBox x:Name="ProcessSmallerFirstCheckBox" Content="Process smaller files first" Margin="0,2" HorizontalAlignment="Left" ToolTip="Sort files by size and process the smallest ones first" />
                                         <StackPanel Orientation="Horizontal" Margin="0,2">
                                             <CheckBox x:Name="ForceCreateCdCheckBox" Content="Force CD" Margin="0,0,10,0" Checked="ForceCreateCdCheckBox_Checked" ToolTip="Force CHDMAN to use the 'createcd' command" />
@@ -238,7 +238,7 @@
                                         <StackPanel>
                                             <CheckBox x:Name="SearchSubfoldersExtractionCheckBox" Content="Search subfolders (recursive search)" Margin="0,2" HorizontalAlignment="Left" ToolTip="Recursively search for files in all subfolders"
                                                       Checked="SearchSubfoldersCheckBox_Changed" Unchecked="SearchSubfoldersCheckBox_Changed" />
-                                            <CheckBox x:Name="DeleteOriginalChdCheckBox" Content="Delete original CHD after a successfull extraction" Margin="0,2" HorizontalAlignment="Left" VerticalAlignment="Top" ToolTip="Delete original CHD after a successfull extraction" />
+                                            <CheckBox x:Name="DeleteOriginalChdCheckBox" Content="Delete original CHD after a successful extraction" Margin="0,2" HorizontalAlignment="Left" VerticalAlignment="Top" ToolTip="Delete original CHD after a successfull extraction" />
                                         </StackPanel>
                                     </GroupBox>
                                 </Grid>