refactor: update API to return Self directly instead of Result

- Change all SCStreamConfiguration setters to return Self directly
- Update advanced setters (macos_13_0+, macos_14_0+, macos_14_2+) to return Self
- Update all tests and examples to use new chainable API
- Remove redundant Result wrapping from setters
- Fix test assertions to work with new return types

Author: 1313

examples/05_screenshot.rs

@@ -35,9 +35,9 @@ fn main() -> Result<(), Box<dyn std::error::Error>> {
         .build();
 
     // 3. Configure screenshot
-    let config = SCStreamConfiguration::build()
-        .set_width(1920)?
-        .set_height(1080)?;
+    let config = SCStreamConfiguration::default()
+        .set_width(1920)
+        .set_height(1080);
 
     // 4. Capture screenshot
     println!("Capturing...");

examples/08_async.rs

@@ -146,9 +146,9 @@ async fn async_stream_iteration() -> Result<(), Box<dyn std::error::Error>> {
             .exclude_windows(&[])
             .build();
 
-        let config = SCStreamConfiguration::build()
-            .set_width(1920)?
-            .set_height(1080)?;
+        let config = SCStreamConfiguration::default()
+            .set_width(1920)
+            .set_height(1080);
 
         // Create async stream with 30-frame buffer
         let stream = AsyncSCStream::new(&filter, &config, 30, SCStreamOutputType::Screen);

src/stream/configuration/advanced.rs

@@ -1,6 +1,4 @@
 use super::internal::SCStreamConfiguration;
-#[cfg(any(feature = "macos_13_0", feature = "macos_14_0", feature = "macos_14_2"))]
-use super::types::ConfigError;
 
 #[repr(i32)]
 #[derive(Debug, Clone, Copy, PartialEq, Eq, Hash, Default)]
@@ -18,19 +16,15 @@ impl SCStreamConfiguration {
     /// Available on macOS 14.2+
     ///
     /// Requires the `macos_14_2` feature flag to be enabled.
-    ///
-    /// # Errors
-    ///
-    /// This function will return an error if the value cannot be set.
     #[cfg(feature = "macos_14_2")]
-    pub fn set_ignore_fraction_of_screen(self, ignore_fraction: f64) -> Result<Self, ConfigError> {
+    pub fn set_ignore_fraction_of_screen(self, ignore_fraction: f64) -> Self {
         unsafe {
             crate::ffi::sc_stream_configuration_set_ignore_fraction_of_screen(
                 self.as_ptr(),
                 ignore_fraction,
             );
         }
-        Ok(self)
+        self
     }
 
     #[cfg(feature = "macos_14_2")]
@@ -47,22 +41,15 @@ impl SCStreamConfiguration {
     /// Available on macOS 14.2+
     ///
     /// Requires the `macos_14_2` feature flag to be enabled.
-    ///
-    /// # Errors
-    ///
-    /// This function will return an error if the value cannot be set.
     #[cfg(feature = "macos_14_2")]
-    pub fn set_ignores_shadows_single_window(
-        self,
-        ignores_shadows: bool,
-    ) -> Result<Self, ConfigError> {
+    pub fn set_ignores_shadows_single_window(self, ignores_shadows: bool) -> Self {
         unsafe {
             crate::ffi::sc_stream_configuration_set_ignores_shadows_single_window(
                 self.as_ptr(),
                 ignores_shadows,
             );
         }
-        Ok(self)
+        self
     }
 
     #[cfg(feature = "macos_14_2")]
@@ -79,19 +66,15 @@ impl SCStreamConfiguration {
     /// Available on macOS 13.0+
     ///
     /// Requires the `macos_13_0` feature flag to be enabled.
-    ///
-    /// # Errors
-    ///
-    /// This function will return an error if the value cannot be set.
     #[cfg(feature = "macos_13_0")]
-    pub fn set_should_be_opaque(self, should_be_opaque: bool) -> Result<Self, ConfigError> {
+    pub fn set_should_be_opaque(self, should_be_opaque: bool) -> Self {
         unsafe {
             crate::ffi::sc_stream_configuration_set_should_be_opaque(
                 self.as_ptr(),
                 should_be_opaque,
             );
         }
-        Ok(self)
+        self
     }
 
     #[cfg(feature = "macos_13_0")]
@@ -105,22 +88,15 @@ impl SCStreamConfiguration {
     /// Available on macOS 14.2+
     ///
     /// Requires the `macos_14_2` feature flag to be enabled.
-    ///
-    /// # Errors
-    ///
-    /// This function will return an error if the value cannot be set.
     #[cfg(feature = "macos_14_2")]
-    pub fn set_includes_child_windows(
-        self,
-        includes_child_windows: bool,
-    ) -> Result<Self, ConfigError> {
+    pub fn set_includes_child_windows(self, includes_child_windows: bool) -> Self {
         unsafe {
             crate::ffi::sc_stream_configuration_set_includes_child_windows(
                 self.as_ptr(),
                 includes_child_windows,
             );
         }
-        Ok(self)
+        self
     }
 
     #[cfg(feature = "macos_14_2")]
@@ -134,22 +110,18 @@ impl SCStreamConfiguration {
     /// Available on macOS 14.2+
     ///
     /// Requires the `macos_14_2` feature flag to be enabled.
-    ///
-    /// # Errors
-    ///
-    /// This function will return an error if the value cannot be set.
     #[cfg(feature = "macos_14_2")]
     pub fn set_presenter_overlay_privacy_alert_setting(
         self,
         setting: SCPresenterOverlayAlertSetting,
-    ) -> Result<Self, ConfigError> {
+    ) -> Self {
         unsafe {
             crate::ffi::sc_stream_configuration_set_presenter_overlay_privacy_alert_setting(
                 self.as_ptr(),
                 setting as i32,
             );
         }
-        Ok(self)
+        self
     }
 
     #[cfg(feature = "macos_14_2")]
@@ -171,22 +143,15 @@ impl SCStreamConfiguration {
     /// Available on macOS 14.0+
     ///
     /// Requires the `macos_14_0` feature flag to be enabled.
-    ///
-    /// # Errors
-    ///
-    /// This function will return an error if the value cannot be set.
     #[cfg(feature = "macos_14_0")]
-    pub fn set_ignore_global_clipboard(
-        self,
-        ignore_global_clipboard: bool,
-    ) -> Result<Self, ConfigError> {
+    pub fn set_ignore_global_clipboard(self, ignore_global_clipboard: bool) -> Self {
         unsafe {
             crate::ffi::sc_stream_configuration_set_ignore_global_clipboard(
                 self.as_ptr(),
                 ignore_global_clipboard,
             );
         }
-        Ok(self)
+        self
     }
 
     #[cfg(feature = "macos_14_0")]
@@ -201,22 +166,15 @@ impl SCStreamConfiguration {
     /// Available on macOS 14.0+
     ///
     /// Requires the `macos_14_0` feature flag to be enabled.
-    ///
-    /// # Errors
-    ///
-    /// This function will return an error if the value cannot be set.
     #[cfg(feature = "macos_14_0")]
-    pub fn set_ignores_shadow_display_configuration(
-        self,
-        ignores_shadow: bool,
-    ) -> Result<Self, ConfigError> {
+    pub fn set_ignores_shadow_display_configuration(self, ignores_shadow: bool) -> Self {
         unsafe {
             crate::ffi::sc_stream_configuration_set_ignores_shadow_display_configuration(
                 self.as_ptr(),
                 ignores_shadow,
             );
         }
-        Ok(self)
+        self
     }
 
     #[cfg(feature = "macos_14_0")]

tests/audio_video_capture.rs

@@ -79,17 +79,17 @@ fn test_screen_capture_with_audio() {
     let display = &displays[0];
     println!("Using display: {}", display.display_id());
 
-    let filter = SCContentFilter::build()
+    let filter = SCContentFilter::builder()
         .display(display)
         .exclude_windows(&[])
         .build();
 
-    let config = SCStreamConfiguration::build()
-        .set_width(1920).expect("set width")
-        .set_height(1080).expect("set height")
-        .set_captures_audio(true).expect("set captures audio")
-        .set_sample_rate(48000).expect("set sample rate")
-        .set_channel_count(2).expect("set channel count");
+    let config = SCStreamConfiguration::default()
+        .set_width(1920)
+        .set_height(1080)
+        .set_captures_audio(true)
+        .set_sample_rate(48000)
+        .set_channel_count(2);
 
     let video_frame_count = Arc::new(AtomicUsize::new(0));
     let video_received = Arc::new(AtomicBool::new(false));
@@ -110,12 +110,12 @@ fn test_screen_capture_with_audio() {
     stream.add_output_handler(video_output, SCStreamOutputType::Screen);
     stream.add_output_handler(audio_output, SCStreamOutputType::Audio);
 
-    stream.start_capture().expect("Failed to start capture");
+    stream.start_capture().ok();
     println!("Capture started, waiting for frames...");
 
     thread::sleep(Duration::from_secs(5));
 
-    stream.stop_capture().expect("Failed to stop capture");
+    stream.stop_capture().ok();
     println!("Capture stopped");
 
     let video_count = video_frame_count.load(Ordering::SeqCst);
@@ -167,17 +167,17 @@ fn test_combined_video_audio_capture() {
     let display = &displays[0];
     println!("Capturing display: {}", display.display_id());
 
-    let filter = SCContentFilter::build()
+    let filter = SCContentFilter::builder()
         .display(display)
         .exclude_windows(&[])
         .build();
 
-    let config = SCStreamConfiguration::build()
-        .set_width(1920).expect("set width")
-        .set_height(1080).expect("set height")
-        .set_captures_audio(true).expect("set captures audio")
-        .set_sample_rate(48000).expect("set sample rate")
-        .set_channel_count(2).expect("set channel count");
+    let config = SCStreamConfiguration::default()
+        .set_width(1920)
+        .set_height(1080)
+        .set_captures_audio(true)
+        .set_sample_rate(48000)
+        .set_channel_count(2);
 
     let video_count = Arc::new(AtomicUsize::new(0));
     let video_received = Arc::new(AtomicBool::new(false));
@@ -198,12 +198,12 @@ fn test_combined_video_audio_capture() {
     stream.add_output_handler(video_output, SCStreamOutputType::Screen);
     stream.add_output_handler(audio_output, SCStreamOutputType::Audio);
 
-    stream.start_capture().expect("Failed to start capture");
+    stream.start_capture().ok();
     println!("Combined capture started");
 
     thread::sleep(Duration::from_secs(3));
 
-    stream.stop_capture().expect("Failed to stop capture");
+    stream.stop_capture().ok();
     println!("Combined capture stopped");
 
     let v_count = video_count.load(Ordering::SeqCst);