Merge pull request #335 from cosmic-utils/feat/comprehensive-bug-report

feat: include full insights, settings, and libcamera metadata in bug report

Author: FreddyFunk

src/app/handlers/system.rs

@@ -414,24 +414,35 @@ impl AppModel {
         Task::none()
     }
 
-    pub(crate) fn handle_generate_bug_report(&self) -> Task<cosmic::Action<Message>> {
+    pub(crate) fn handle_generate_bug_report(&mut self) -> Task<cosmic::Action<Message>> {
         info!("Generating bug report...");
 
+        // Ensure insights are up-to-date (normally only refreshed when the
+        // Insights drawer is open, so they may be stale/empty here).
+        let _ = self.handle_update_insights_metrics();
+
         let video_devices = self.available_cameras.clone();
         let audio_devices = self.available_audio_devices.clone();
         let video_encoders = self.available_video_encoders.clone();
-        let selected_encoder_index = self.current_video_encoder_index;
         let save_folder_name = self.config.save_folder_name.clone();
+        let insights = self.insights.clone();
+        let config = self.config.clone();
+        let snapshot = crate::bug_report::AppStateSnapshot {
+            current_camera_index: self.current_camera_index,
+            current_audio_device_index: self.current_audio_device_index,
+            current_video_encoder_index: self.current_video_encoder_index,
+        };
 
         Task::perform(
             async move {
                 crate::bug_report::BugReportGenerator::generate(
                     &video_devices,
                     &audio_devices,
                     &video_encoders,
-                    selected_encoder_index,
-                    None,
                     &save_folder_name,
+                    &insights,
+                    &config,
+                    &snapshot,
                 )
                 .await
                 .map(|p| p.display().to_string())
@@ -825,6 +836,7 @@ impl AppModel {
                 self.insights.meta_lens_position = meta.lens_position;
                 self.insights.meta_lux = meta.lux;
                 self.insights.meta_focus_fom = meta.focus_fom;
+                self.insights.frame_metadata = Some(meta.clone());
             }
         }
     }

src/app/insights/types.rs

@@ -77,6 +77,9 @@ pub struct InsightsState {
     pub meta_focus_fom: Option<i32>,
     /// Whether libcamera metadata is present at all
     pub has_libcamera_metadata: bool,
+    /// Full libcamera FrameMetadata snapshot (includes fields not shown in UI:
+    /// sensor_timestamp, colour_correction_matrix, af/ae/awb state)
+    pub frame_metadata: Option<crate::backends::camera::types::FrameMetadata>,
 
     // Recording pipeline info (populated while recording)
     /// Recording pipeline diagnostics (None when not recording)

src/app/mod.rs

@@ -38,7 +38,7 @@ pub mod frame_processor;
 mod gallery_primitive;
 mod gallery_widget;
 mod handlers;
-mod insights;
+pub mod insights;
 mod motor_picker;
 pub mod qr_overlay;
 pub mod settings;