**fix: enhance bug reporting with structured payload and improve app shutdown**

- Refactored `BugReportService` to send detailed structured fields (env, exception, stack) matching API model.
- Updated `MainWindow` closing logic for better cancellation handling and forced shutdown.
- Converted event handlers to async naming convention and centralized log/terminal styles in App.xaml.
- Bumped SharpCompress to v0.47.4.

Author: drpetersonfernandes

BatchConvertToCHD/App.xaml

@@ -579,5 +579,49 @@
             <Setter Property="Padding" Value="10"/>
         </Style>
 
+        <!-- ==================== LOG TEXTBOX STYLES ==================== -->
+        <!-- Terminal TextBox Style (reused across windows) -->
+        <Style x:Key="TerminalLogTextBox" TargetType="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}"/>
+            <Setter Property="VerticalAlignment" Value="Stretch"/>
+            <Setter Property="VerticalContentAlignment" Value="Top"/>
+        </Style>
+
+        <!-- Terminal TextBox with Glow Effect -->
+        <Style x:Key="TerminalLogTextBoxWithGlow" TargetType="TextBox" BasedOn="{StaticResource TerminalLogTextBox}">
+            <Setter Property="Effect">
+                <Setter.Value>
+                    <DropShadowEffect ShadowDepth="0" Color="#39FF14" BlurRadius="2" Opacity="0.4"/>
+                </Setter.Value>
+            </Setter>
+        </Style>
+
+        <!-- ==================== BUTTON CONTENT COLORS ==================== -->
+        <!-- Dark text color for buttons with light backgrounds -->
+        <SolidColorBrush x:Key="ButtonTextDarkBrush" Color="#121518"/>
+
+        <!-- ==================== TABCONTROL STYLE ==================== -->
+        <Style TargetType="TabControl">
+            <Setter Property="Background" Value="Transparent"/>
+            <Setter Property="BorderThickness" Value="0"/>
+        </Style>
+
+        <!-- ==================== GRIDSPLITTER STYLE ==================== -->
+        <Style TargetType="GridSplitter">
+            <Setter Property="Background" Value="Transparent"/>
+        </Style>
+
     </Application.Resources>
 </Application>

BatchConvertToCHD/App.xaml.cs

@@ -34,6 +34,10 @@ public App()
 
     protected override void OnStartup(StartupEventArgs e)
     {
+        // Set shutdown mode to close the application when the main window closes
+        // This ensures the app fully terminates when the user closes the window
+        ShutdownMode = ShutdownMode.OnMainWindowClose;
+
         // Delete old 7z DLL files if they exist
         DeleteOldDllFiles();
 

BatchConvertToCHD/AppConfig.cs

@@ -19,7 +19,4 @@ internal static class AppConfig
 
     // Timeout configurations
     public const int MaxConversionTimeoutHours = 4;
-    public const int ProcessKillTimeoutSeconds = 5;
-    public const int FileAccessTimeoutSeconds = 5;
-    public const int ValidationTimeoutSeconds = 10;
 }

BatchConvertToCHD/MainWindow.xaml

@@ -30,15 +30,15 @@
                         <Button.Content>
                             <StackPanel Orientation="Horizontal">
                                 <Image Source="pack://application:,,,/images/about.png" Width="16" Height="16" Margin="0,0,5,0" />
-                                <TextBlock Text="About" Foreground="#121518" VerticalAlignment="Center" />
+                                <TextBlock Text="About" Foreground="{StaticResource ButtonTextDarkBrush}" VerticalAlignment="Center" />
                             </StackPanel>
                         </Button.Content>
                     </Button>
                     <Button Click="ExitMenuItem_Click" Style="{StaticResource ActionButtonStyle}" Background="{StaticResource ExitButtonBrush}" ToolTip="Close the application">
                         <Button.Content>
                             <StackPanel Orientation="Horizontal">
                                 <Image Source="pack://application:,,,/images/exit.png" Width="16" Height="16" Margin="0,0,5,0" />
-                                <TextBlock Text="Exit" Foreground="#121518" VerticalAlignment="Center" />
+                                <TextBlock Text="Exit" Foreground="{StaticResource ButtonTextDarkBrush}" VerticalAlignment="Center" />
                             </StackPanel>
                         </Button.Content>
                     </Button>
@@ -61,7 +61,7 @@
                     <ColumnDefinition Width="*" MinWidth="300" />
                 </Grid.ColumnDefinitions>
 
-                <TabControl Grid.Column="0" x:Name="MainTabControl" SelectionChanged="MainTabControl_SelectionChanged" Background="Transparent" BorderThickness="0">
+                <TabControl Grid.Column="0" x:Name="MainTabControl" SelectionChanged="MainTabControl_SelectionChanged">
                     <TabItem Header="Convert to CHD" x:Name="ConvertTab" ToolTip="Batch convert various disc image formats to CHD">
                         <Border Background="{StaticResource CardBackgroundBrush}" BorderBrush="{StaticResource BorderBrush}" BorderThickness="1" CornerRadius="0,0,6,6" Padding="15">
                             <Grid>
@@ -123,7 +123,7 @@
                                     <Button Content="Deselect All" Click="DeselectAllConversion_Click" Style="{StaticResource ActionButtonStyle}" Margin="5,0" />
                                 </StackPanel>
 
-                                <Button Grid.Row="6" x:Name="StartConversionButton" Content="Start Conversion" Click="StartConversionButton_Click" Style="{StaticResource StartButtonStyle}" HorizontalAlignment="Stretch" ToolTip="Begin the batch conversion process" />
+                                <Button Grid.Row="6" x:Name="StartConversionButton" Content="Start Conversion" Click="StartConversionButton_ClickAsync" Style="{StaticResource StartButtonStyle}" HorizontalAlignment="Stretch" ToolTip="Begin the batch conversion process" />
                             </Grid>
                         </Border>
                     </TabItem>
@@ -173,7 +173,7 @@
                                     <Button Content="Deselect All" Click="DeselectAllVerification_Click" Style="{StaticResource ActionButtonStyle}" Margin="5,0" />
                                 </StackPanel>
 
-                                <Button Grid.Row="5" x:Name="StartVerificationButton" Content="Start Verification" Click="StartVerificationButton_Click" Style="{StaticResource StartButtonStyle}" HorizontalAlignment="Stretch" ToolTip="Begin the batch verification process" />
+                                <Button Grid.Row="5" x:Name="StartVerificationButton" Content="Start Verification" Click="StartVerificationButton_ClickAsync" Style="{StaticResource StartButtonStyle}" HorizontalAlignment="Stretch" ToolTip="Begin the batch verification process" />
                             </Grid>
                         </Border>
                     </TabItem>
@@ -256,36 +256,18 @@
                                     <Button Content="Deselect All" Click="DeselectAllExtraction_Click" Style="{StaticResource ActionButtonStyle}" Margin="5,0" />
                                 </StackPanel>
 
-                                <Button Grid.Row="6" x:Name="StartExtractionButton" Content="Start Extraction" Click="StartExtractionButton_Click" Style="{StaticResource StartButtonStyle}" HorizontalAlignment="Stretch" ToolTip="Begin the batch extraction process" />
+                                <Button Grid.Row="6" x:Name="StartExtractionButton" Content="Start Extraction" Click="StartExtractionButton_ClickAsync" Style="{StaticResource StartButtonStyle}" HorizontalAlignment="Stretch" ToolTip="Begin the batch extraction process" />
                             </Grid>
                         </Border>
                     </TabItem>
                 </TabControl>
 
-                <GridSplitter Grid.Column="1" Width="10" HorizontalAlignment="Stretch" Background="Transparent" />
+                <GridSplitter Grid.Column="1" Width="10" HorizontalAlignment="Stretch" />
 
                 <Border Grid.Column="2" BorderBrush="{StaticResource TerminalBorderBrush}" BorderThickness="2" CornerRadius="4" Background="{StaticResource TerminalBackgroundBrush}">
                     <TextBox x:Name="LogViewer"
-                             AcceptsReturn="True"
-                             VerticalScrollBarVisibility="Auto"
-                             HorizontalScrollBarVisibility="Auto"
-                             VerticalAlignment="Stretch"
-                             VerticalContentAlignment="Top"
-                             IsReadOnly="True"
-                             Padding="10"
-                             FontFamily="Consolas"
-                             FontSize="12"
-                             Background="{StaticResource TerminalBackgroundBrush}"
-                             Foreground="{StaticResource TerminalForegroundBrush}"
-                             BorderThickness="0"
-                             TextAlignment="Left"
-                             TextWrapping="NoWrap"
-                             CaretBrush="{StaticResource TerminalForegroundBrush}"
-                             ToolTip="Application activity log">
-                        <TextBox.Effect>
-                            <DropShadowEffect ShadowDepth="0" Color="#39FF14" BlurRadius="2" Opacity="0.4" />
-                        </TextBox.Effect>
-                    </TextBox>
+                             Style="{StaticResource TerminalLogTextBoxWithGlow}"
+                             ToolTip="Application activity log" />
                 </Border>
             </Grid>
 

BatchConvertToCHD/MainWindow.xaml.cs

@@ -92,13 +92,13 @@ public MainWindow()
         LogEnvironmentDetails();
 
         // Defer heavy initialization until after window is shown
-        Loaded += MainWindow_Loaded;
+        Loaded += MainWindow_LoadedAsync;
 
         // Hide speed display initially until we know counters are available
         SpeedStatCard.Visibility = Visibility.Collapsed;
     }
 
-    private async void MainWindow_Loaded(object sender, RoutedEventArgs e)
+    private async void MainWindow_LoadedAsync(object sender, RoutedEventArgs e)
     {
         try
         {
@@ -414,8 +414,30 @@ private void MainTabControl_SelectionChanged(object sender, SelectionChangedEven
 
     private void Window_Closing(object sender, CancelEventArgs e)
     {
-        _cts.Cancel();
+        // Check if any operation is currently running (buttons disabled = operation in progress)
+        bool isOperationRunning = !StartConversionButton.IsEnabled || !StartVerificationButton.IsEnabled || !StartExtractionButton.IsEnabled;
+
+        if (isOperationRunning)
+        {
+            if (!_cts.IsCancellationRequested)
+            {
+                // First close attempt - cancel operations but keep window open momentarily
+                _cts.Cancel();
+                LogMessage("Cancelling operations before closing...");
+                UpdateStatusBarMessage("Cancelling...");
+                e.Cancel = true;
+                return;
+            }
+            // else: Cancellation already requested, allow close but force termination
+            // because background tasks might be stuck
+        }
+
+        // Perform cleanup of resources
         Dispose();
+
+        // Ensure the application shuts down completely
+        // This is necessary because background tasks/threads might keep the app alive
+        Application.Current.Shutdown();
     }
 
     private void LogMessage(string message)
@@ -486,7 +508,7 @@ private void BrowseExtractionOutputButton_Click(object sender, RoutedEventArgs e
         HandleFolderBrowse(ExtractionOutputFolderTextBox, "Extraction output");
     }
 
-    private async void StartExtractionButton_Click(object sender, RoutedEventArgs e)
+    private async void StartExtractionButton_ClickAsync(object sender, RoutedEventArgs e)
     {
         try
         {
@@ -796,7 +818,7 @@ private void DeselectAllExtraction_Click(object sender, RoutedEventArgs e)
         }
     }
 
-    private async void StartConversionButton_Click(object sender, RoutedEventArgs e)
+    private async void StartConversionButton_ClickAsync(object sender, RoutedEventArgs e)
     {
         try
         {
@@ -870,7 +892,7 @@ await PerformBatchConversionAsync(Path.Combine(AppDomain.CurrentDomain.BaseDirec
         }
     }
 
-    private async void StartVerificationButton_Click(object sender, RoutedEventArgs e)
+    private async void StartVerificationButton_ClickAsync(object sender, RoutedEventArgs e)
     {
         try
         {
@@ -2270,6 +2292,8 @@ private void ResetSpeedCounters()
 
     private void ExitMenuItem_Click(object sender, RoutedEventArgs e)
     {
+        // Ensure the window close process is initiated
+        // The Window_Closing event will handle proper cleanup and shutdown
         Close();
     }