[auto-fix] Attempt fixes from cargo/clippy

src/calendar.rs

@@ -401,7 +401,7 @@ fn unescape_ical_text(input: &str) -> String {
 }
 
 /// Parse iCalendar content into a vector of (key, value) property pairs.
-/// 
+///
 /// This function uses nom-based parser combinators for better performance
 /// and error handling compared to manual string manipulation.
 #[must_use]
@@ -454,9 +454,9 @@ fn split_ical_blocks(ics: &str) -> Vec<Vec<String>> {
 
 /// Extract VTIMEZONE block from iCalendar content using nom parser.
 #[must_use]
-    fn extract_vtimezone_block(ics: &str) -> Option<String> {
-        ical_parser::parse_vtimezone_block(ics).ok().flatten()
-    }
+fn extract_vtimezone_block(ics: &str) -> Option<String> {
+    ical_parser::parse_vtimezone_block(ics).ok().flatten()
+}
 
 fn parse_categories_value(value: &str) -> Vec<String> {
     value
@@ -523,9 +523,9 @@ fn format_ical_datetime_with_timezone(
 /// Parse duration in ISO 8601 format using nom parser.
 /// Returns the number of minutes, where negative values represent time before an event
 /// (e.g., reminders). A negative sign in the input (-PT15M) returns a negative number (-15).
-    fn parse_duration_minutes(trigger: &str) -> Option<i32> {
-        ical_parser::parse_ical_duration_minutes(trigger).ok()
-    }
+fn parse_duration_minutes(trigger: &str) -> Option<i32> {
+    ical_parser::parse_ical_duration_minutes(trigger).ok()
+}
 
 fn fold_ics_line(line: &str) -> String {
     const MAX_LINE_LEN: usize = 75;
@@ -1281,10 +1281,10 @@ pub fn parse_eas_sync_mutations(xml: &str) -> Result<Vec<EasSyncMutation>> {
                     };
                     match stack.last().map(|v| v.as_slice()) {
                         Some(b"ClientId") => current.client_id = Some(value),
-                        Some(b"ServerId") => {
-                            if !stack.iter().any(|v| v.as_slice() == b"Exception") {
-                                current.server_id = Some(value);
-                            }
+                        Some(b"ServerId")
+                            if !stack.iter().any(|v| v.as_slice() == b"Exception") =>
+                        {
+                            current.server_id = Some(value);
                         }
                         Some(b"InstanceId") => {
                             current.instance_id = parse_datetime(&value);

src/config.rs

@@ -44,7 +44,7 @@ impl Config {
         if self.bind.is_empty() {
             return Err(anyhow::anyhow!("Config: 'bind' address is required"));
         }
-        
+
         if !self.bind.contains(':') {
             return Err(anyhow::anyhow!(
                 "Config: 'bind' must be in format 'host:port'"
@@ -55,11 +55,15 @@ impl Config {
         validate_url(&self.worker_url, "worker_url")?;
 
         if self.worker_secret.expose_secret().len() < 16 {
-            return Err(anyhow::anyhow!("Config: 'worker_secret' must be at least 16 characters"));
+            return Err(anyhow::anyhow!(
+                "Config: 'worker_secret' must be at least 16 characters"
+            ));
         }
 
         if self.hmac_secret.expose_secret().len() < 32 {
-            return Err(anyhow::anyhow!("Config: 'hmac_secret' must be at least 32 characters"));
+            return Err(anyhow::anyhow!(
+                "Config: 'hmac_secret' must be at least 32 characters"
+            ));
         }
 
         if !self.gateway_host.is_empty() && self.gateway_host.contains("://") {

src/eas.rs

@@ -100,7 +100,8 @@ struct EasRequest {
 struct CommandGrammar {
     namespace: &'static str,
     required_tags: &'static [&'static str],
-    #[allow(dead_code)] // Documents allowed optional tags for protocol completeness; validation not currently required
+    #[allow(dead_code)]
+    // Documents allowed optional tags for protocol completeness; validation not currently required
     optional_tags: &'static [&'static str],
 }
 
@@ -299,10 +300,8 @@ fn validate_payload(command: &str, xml: &str) -> Result<(), &'static str> {
                 return Err("Add requires ClientId");
             }
         }
-        "meetingresponse" => {
-            if extract_first_tag_text(xml, b"UserResponse").is_none() {
-                return Err("MeetingResponse requires UserResponse");
-            }
+        "meetingresponse" if extract_first_tag_text(xml, b"UserResponse").is_none() => {
+            return Err("MeetingResponse requires UserResponse");
         }
         _ => {}
     }
@@ -1465,7 +1464,7 @@ async fn handle_resolve_recipients(
     let freebusy_results = join_all(freebusy_futures).await;
 
     let mut recipient_xml = String::new();
-    for (recipient, freebusy) in recipients.iter().zip(freebusy_results.into_iter()) {
+    for (recipient, freebusy) in recipients.iter().zip(freebusy_results) {
         let avail_xml = if availability_window.is_some() {
             format!(
                 "<Availability><Status>1</Status><MergedFreeBusy>{}</MergedFreeBusy></Availability>",

src/error.rs

@@ -2,7 +2,7 @@
 use thiserror::Error;
 
 /// Gateway error types for protocol and application errors.
-/// 
+///
 /// This enum is marked as `#[non_exhaustive]` to allow adding new error
 /// variants without breaking changes to downstream code.
 #[non_exhaustive]

src/ews.rs

@@ -181,12 +181,12 @@ fn validate_schema(action: &EwsAction, xml: &str) -> Result<(), &'static str> {
     if !xml.contains(EWS_MSG_NS) && !xml.contains("xmlns:m=") {
         return Err("Missing EWS messages namespace");
     }
-    
+
     // Use const fn for compile-time optimization
     if action.requires_mime_validation() && xml.contains("IncludeMimeContent") {
         return Err("This operation does not support IncludeMimeContent");
     }
-    
+
     Ok(())
 }
 

src/ews_update.rs

@@ -2,7 +2,7 @@
 use crate::calendar::{
     CalendarItem, extract_ews_field, extract_ews_fields, parse_ews_attendees, parse_ews_recurrence,
 };
-use crate::util::{xml_escape_text, xml_escape_attr};
+use crate::util::{xml_escape_attr, xml_escape_text};
 use quick_xml::Reader;
 use quick_xml::events::{BytesEnd, BytesStart, Event};