crawlberg 1.0.4 → 1.0.6

data/ext/crawlberg_rb/src/lib.rs

@@ -1,6 +1,4 @@
 // This file is auto-generated by alef. DO NOT EDIT.
-// alef:hash:e0600612767fdd3d51e9128afdb0ffcd1fb490e9942f3839a5120ab7a4e3b658
-// Re-generate with: alef generate
 #![allow(dead_code, unused_imports, unused_variables)]
 #![allow(
     clippy::too_many_arguments,
@@ -2040,6 +2038,7 @@ impl CrawlResult {
     }
 
     fn unique_normalized_urls(&self) -> usize {
+        #[allow(clippy::needless_update)]
         let core_self = crawlberg::CrawlResult {
             pages: self.pages.clone().into_iter().map(Into::into).collect(),
 
@@ -3982,7 +3981,6 @@ impl CrawlEngineHandle {
         };
         let ruby = unsafe { Ruby::get_unchecked() };
         if ruby.block_given() {
-            // Drive the stream synchronously, yielding each chunk to the block.
             iterator.each()?;
             Ok(ruby.qnil().as_value())
         } else {
@@ -4017,7 +4015,6 @@ impl CrawlEngineHandle {
         };
         let ruby = unsafe { Ruby::get_unchecked() };
         if ruby.block_given() {
-            // Drive the stream synchronously, yielding each chunk to the block.
             iterator.each()?;
             Ok(ruby.qnil().as_value())
         } else {
@@ -4451,8 +4448,6 @@ impl magnus::IntoValue for BrowserMode {
 impl magnus::TryConvert for BrowserMode {
     fn try_convert(val: magnus::Value) -> Result<Self, magnus::Error> {
         let s: String = magnus::TryConvert::try_convert(val)?;
-        // Accept the serde wire name (snake_case), the PascalCase Rust variant name,
-        // and a lowercase fallback so fixtures written in any of those styles work.
         match s.as_str() {
             "auto" | "Auto" => Ok(BrowserMode::Auto),
             "always" | "Always" => Ok(BrowserMode::Always),
@@ -4497,8 +4492,6 @@ impl magnus::IntoValue for BrowserWait {
 impl magnus::TryConvert for BrowserWait {
     fn try_convert(val: magnus::Value) -> Result<Self, magnus::Error> {
         let s: String = magnus::TryConvert::try_convert(val)?;
-        // Accept the serde wire name (snake_case), the PascalCase Rust variant name,
-        // and a lowercase fallback so fixtures written in any of those styles work.
         match s.as_str() {
             "network_idle" | "NetworkIdle" => Ok(BrowserWait::NetworkIdle),
             "selector" | "Selector" => Ok(BrowserWait::Selector),
@@ -4540,8 +4533,6 @@ impl magnus::IntoValue for BrowserBackend {
 impl magnus::TryConvert for BrowserBackend {
     fn try_convert(val: magnus::Value) -> Result<Self, magnus::Error> {
         let s: String = magnus::TryConvert::try_convert(val)?;
-        // Accept the serde wire name (snake_case), the PascalCase Rust variant name,
-        // and a lowercase fallback so fixtures written in any of those styles work.
         match s.as_str() {
             "chromiumoxide" | "Chromiumoxide" => Ok(BrowserBackend::Chromiumoxide),
             "native" | "Native" => Ok(BrowserBackend::Native),
@@ -4587,8 +4578,6 @@ impl magnus::IntoValue for AuthConfig {
 
 impl magnus::TryConvert for AuthConfig {
     fn try_convert(val: magnus::Value) -> Result<Self, magnus::Error> {
-        // For data enums with fields (e.g., PageAction), try to deserialize from JSON first.
-        // For unit enums or when passed as a string, fall back to string-based conversion.
         let json_str: String = if let Ok(s) = <String as magnus::TryConvert>::try_convert(val) {
             s
         } else {
@@ -4599,19 +4588,12 @@ impl magnus::TryConvert for AuthConfig {
                 )
             })?
         };
-        // Try deserializing as JSON first (handles JSON strings like "\"markdown\"" or "{\"click\":{\"selector\":\"...\"}}\"")
-        // For internally-tagged enums, a bare variant string is wrapped as {"<tag>": value}.
-        // If that fails, try treating it as a plain string value and wrap in quotes
-        // If both fail, try as Custom variant (for untagged enum support)
         serde_json::from_str(&json_str)
             .or_else(|_| serde_json::from_value(serde_json::json!({ "type": json_str })))
             .or_else(|_| serde_json::from_str(&format!("\"{json_str}\"")))
-            .or_else(|_| {
-                // Try as a JSON string for Custom variant (untagged enums accept any remaining value)
-                match serde_json::to_value(&json_str) {
-                    Ok(val) => serde_json::from_value(val),
-                    Err(e) => Err(e),
-                }
+            .or_else(|_| match serde_json::to_value(&json_str) {
+                Ok(val) => serde_json::from_value(val),
+                Err(e) => Err(e),
             })
             .map_err(|e| magnus::Error::new(unsafe { Ruby::get_unchecked() }.exception_type_error(), e.to_string()))
     }
@@ -4651,8 +4633,6 @@ impl magnus::IntoValue for LinkType {
 impl magnus::TryConvert for LinkType {
     fn try_convert(val: magnus::Value) -> Result<Self, magnus::Error> {
         let s: String = magnus::TryConvert::try_convert(val)?;
-        // Accept the serde wire name (snake_case), the PascalCase Rust variant name,
-        // and a lowercase fallback so fixtures written in any of those styles work.
         match s.as_str() {
             "internal" | "Internal" => Ok(LinkType::Internal),
             "external" | "External" => Ok(LinkType::External),
@@ -4701,8 +4681,6 @@ impl magnus::IntoValue for ImageSource {
 impl magnus::TryConvert for ImageSource {
     fn try_convert(val: magnus::Value) -> Result<Self, magnus::Error> {
         let s: String = magnus::TryConvert::try_convert(val)?;
-        // Accept the serde wire name (snake_case), the PascalCase Rust variant name,
-        // and a lowercase fallback so fixtures written in any of those styles work.
         match s.as_str() {
             "img" | "Img" => Ok(ImageSource::Img),
             "picture_source" | "PictureSource" => Ok(ImageSource::PictureSource),
@@ -4747,8 +4725,6 @@ impl magnus::IntoValue for FeedType {
 impl magnus::TryConvert for FeedType {
     fn try_convert(val: magnus::Value) -> Result<Self, magnus::Error> {
         let s: String = magnus::TryConvert::try_convert(val)?;
-        // Accept the serde wire name (snake_case), the PascalCase Rust variant name,
-        // and a lowercase fallback so fixtures written in any of those styles work.
         match s.as_str() {
             "rss" | "Rss" => Ok(FeedType::Rss),
             "atom" | "Atom" => Ok(FeedType::Atom),
@@ -4806,8 +4782,6 @@ impl magnus::IntoValue for AssetCategory {
 impl magnus::TryConvert for AssetCategory {
     fn try_convert(val: magnus::Value) -> Result<Self, magnus::Error> {
         let s: String = magnus::TryConvert::try_convert(val)?;
-        // Accept the serde wire name (snake_case), the PascalCase Rust variant name,
-        // and a lowercase fallback so fixtures written in any of those styles work.
         match s.as_str() {
             "document" | "Document" => Ok(AssetCategory::Document),
             "image" | "Image" => Ok(AssetCategory::Image),
@@ -4858,8 +4832,6 @@ impl magnus::IntoValue for CrawlEvent {
 
 impl magnus::TryConvert for CrawlEvent {
     fn try_convert(val: magnus::Value) -> Result<Self, magnus::Error> {
-        // For data enums with fields (e.g., PageAction), try to deserialize from JSON first.
-        // For unit enums or when passed as a string, fall back to string-based conversion.
         let json_str: String = if let Ok(s) = <String as magnus::TryConvert>::try_convert(val) {
             s
         } else {
@@ -4870,19 +4842,12 @@ impl magnus::TryConvert for CrawlEvent {
                 )
             })?
         };
-        // Try deserializing as JSON first (handles JSON strings like "\"markdown\"" or "{\"click\":{\"selector\":\"...\"}}\"")
-        // For internally-tagged enums, a bare variant string is wrapped as {"<tag>": value}.
-        // If that fails, try treating it as a plain string value and wrap in quotes
-        // If both fail, try as Custom variant (for untagged enum support)
         serde_json::from_str(&json_str)
             .or_else(|_| serde_json::from_value(serde_json::json!({ "type": json_str })))
             .or_else(|_| serde_json::from_str(&format!("\"{json_str}\"")))
-            .or_else(|_| {
-                // Try as a JSON string for Custom variant (untagged enums accept any remaining value)
-                match serde_json::to_value(&json_str) {
-                    Ok(val) => serde_json::from_value(val),
-                    Err(e) => Err(e),
-                }
+            .or_else(|_| match serde_json::to_value(&json_str) {
+                Ok(val) => serde_json::from_value(val),
+                Err(e) => Err(e),
             })
             .map_err(|e| magnus::Error::new(unsafe { Ruby::get_unchecked() }.exception_type_error(), e.to_string()))
     }
@@ -4943,8 +4908,6 @@ impl magnus::IntoValue for PageAction {
 
 impl magnus::TryConvert for PageAction {
     fn try_convert(val: magnus::Value) -> Result<Self, magnus::Error> {
-        // For data enums with fields (e.g., PageAction), try to deserialize from JSON first.
-        // For unit enums or when passed as a string, fall back to string-based conversion.
         let json_str: String = if let Ok(s) = <String as magnus::TryConvert>::try_convert(val) {
             s
         } else {
@@ -4955,19 +4918,12 @@ impl magnus::TryConvert for PageAction {
                 )
             })?
         };
-        // Try deserializing as JSON first (handles JSON strings like "\"markdown\"" or "{\"click\":{\"selector\":\"...\"}}\"")
-        // For internally-tagged enums, a bare variant string is wrapped as {"<tag>": value}.
-        // If that fails, try treating it as a plain string value and wrap in quotes
-        // If both fail, try as Custom variant (for untagged enum support)
         serde_json::from_str(&json_str)
             .or_else(|_| serde_json::from_value(serde_json::json!({ "type": json_str })))
             .or_else(|_| serde_json::from_str(&format!("\"{json_str}\"")))
-            .or_else(|_| {
-                // Try as a JSON string for Custom variant (untagged enums accept any remaining value)
-                match serde_json::to_value(&json_str) {
-                    Ok(val) => serde_json::from_value(val),
-                    Err(e) => Err(e),
-                }
+            .or_else(|_| match serde_json::to_value(&json_str) {
+                Ok(val) => serde_json::from_value(val),
+                Err(e) => Err(e),
             })
             .map_err(|e| magnus::Error::new(unsafe { Ruby::get_unchecked() }.exception_type_error(), e.to_string()))
     }
@@ -5003,8 +4959,6 @@ impl magnus::IntoValue for ScrollDirection {
 impl magnus::TryConvert for ScrollDirection {
     fn try_convert(val: magnus::Value) -> Result<Self, magnus::Error> {
         let s: String = magnus::TryConvert::try_convert(val)?;
-        // Accept the serde wire name (snake_case), the PascalCase Rust variant name,
-        // and a lowercase fallback so fixtures written in any of those styles work.
         match s.as_str() {
             "up" | "Up" => Ok(ScrollDirection::Up),
             "down" | "Down" => Ok(ScrollDirection::Down),
@@ -5277,6 +5231,7 @@ fn batch_crawl_stream(engine: CrawlEngineHandle, req: BatchCrawlStreamRequest) -
     engine.batch_crawl_stream(req)
 }
 
+#[allow(clippy::needless_update)]
 #[allow(clippy::redundant_closure, clippy::useless_conversion)]
 impl From<ExtractionMeta> for crawlberg::ExtractionMeta {
     fn from(val: ExtractionMeta) -> Self {
@@ -5286,6 +5241,7 @@ impl From<ExtractionMeta> for crawlberg::ExtractionMeta {
             completion_tokens: val.completion_tokens,
             model: val.model,
             chunks_processed: val.chunks_processed,
+            ..Default::default()
         }
     }
 }
@@ -5303,6 +5259,7 @@ impl From<crawlberg::ExtractionMeta> for ExtractionMeta {
     }
 }
 
+#[allow(clippy::needless_update)]
 #[allow(clippy::redundant_closure, clippy::useless_conversion)]
 impl From<ProxyConfig> for crawlberg::ProxyConfig {
     fn from(val: ProxyConfig) -> Self {
@@ -5310,6 +5267,7 @@ impl From<ProxyConfig> for crawlberg::ProxyConfig {
             url: val.url,
             username: val.username,
             password: val.password,
+            ..Default::default()
         }
     }
 }
@@ -5325,6 +5283,7 @@ impl From<crawlberg::ProxyConfig> for ProxyConfig {
     }
 }
 
+#[allow(clippy::needless_update)]
 #[allow(clippy::redundant_closure, clippy::useless_conversion)]
 impl From<ContentConfig> for crawlberg::ContentConfig {
     fn from(val: ContentConfig) -> Self {
@@ -5341,6 +5300,7 @@ impl From<ContentConfig> for crawlberg::ContentConfig {
             wrap: val.wrap,
             wrap_width: val.wrap_width,
             include_document_structure: val.include_document_structure,
+            ..Default::default()
         }
     }
 }
@@ -5365,6 +5325,7 @@ impl From<crawlberg::ContentConfig> for ContentConfig {
     }
 }
 
+#[allow(clippy::needless_update)]
 #[allow(clippy::redundant_closure, clippy::useless_conversion)]
 impl From<BrowserConfig> for crawlberg::BrowserConfig {
     fn from(val: BrowserConfig) -> Self {
@@ -5382,6 +5343,7 @@ impl From<BrowserConfig> for crawlberg::BrowserConfig {
             robots_user_agent: val.robots_user_agent,
             capture_network_events: val.capture_network_events,
             session_affinity: val.session_affinity,
+            ..Default::default()
         }
     }
 }
@@ -5511,6 +5473,7 @@ impl From<crawlberg::CrawlConfig> for CrawlConfig {
     }
 }
 
+#[allow(clippy::needless_update)]
 #[allow(clippy::redundant_closure, clippy::useless_conversion)]
 impl From<BrowserExtras> for crawlberg::BrowserExtras {
     fn from(val: BrowserExtras) -> Self {
@@ -5518,6 +5481,7 @@ impl From<BrowserExtras> for crawlberg::BrowserExtras {
             eval_result: val.eval_result.as_ref().and_then(|s| serde_json::from_str(s).ok()),
             network_events: val.network_events.into_iter().map(Into::into).collect(),
             cookies: val.cookies.into_iter().map(Into::into).collect(),
+            ..Default::default()
         }
     }
 }
@@ -5587,6 +5551,7 @@ impl From<crawlberg::InteractionResult> for InteractionResult {
     }
 }
 
+#[allow(clippy::needless_update)]
 #[allow(clippy::redundant_closure, clippy::useless_conversion)]
 impl From<ActionResult> for crawlberg::ActionResult {
     fn from(val: ActionResult) -> Self {
@@ -5596,6 +5561,7 @@ impl From<ActionResult> for crawlberg::ActionResult {
             success: val.success,
             data: val.data.as_ref().and_then(|s| serde_json::from_str(s).ok()),
             error: val.error,
+            ..Default::default()
         }
     }
 }
@@ -5687,6 +5653,7 @@ impl From<crawlberg::ScrapeResult> for ScrapeResult {
     }
 }
 
+#[allow(clippy::needless_update)]
 #[allow(clippy::redundant_closure, clippy::useless_conversion)]
 impl From<CrawlPageResult> for crawlberg::CrawlPageResult {
     fn from(val: CrawlPageResult) -> Self {
@@ -5712,6 +5679,7 @@ impl From<CrawlPageResult> for crawlberg::CrawlPageResult {
             extraction_meta: val.extraction_meta.map(Into::into),
             downloaded_document: val.downloaded_document.map(Into::into),
             browser_used: val.browser_used,
+            ..Default::default()
         }
     }
 }
@@ -5779,6 +5747,7 @@ impl From<crawlberg::CrawlResult> for CrawlResult {
     }
 }
 
+#[allow(clippy::needless_update)]
 #[allow(clippy::redundant_closure, clippy::useless_conversion)]
 impl From<SitemapUrl> for crawlberg::SitemapUrl {
     fn from(val: SitemapUrl) -> Self {
@@ -5787,6 +5756,7 @@ impl From<SitemapUrl> for crawlberg::SitemapUrl {
             lastmod: val.lastmod,
             changefreq: val.changefreq,
             priority: val.priority,
+            ..Default::default()
         }
     }
 }
@@ -5803,11 +5773,13 @@ impl From<crawlberg::SitemapUrl> for SitemapUrl {
     }
 }
 
+#[allow(clippy::needless_update)]
 #[allow(clippy::redundant_closure, clippy::useless_conversion)]
 impl From<MapResult> for crawlberg::MapResult {
     fn from(val: MapResult) -> Self {
         Self {
             urls: val.urls.into_iter().map(Into::into).collect(),
+            ..Default::default()
         }
     }
 }
@@ -5821,6 +5793,7 @@ impl From<crawlberg::MapResult> for MapResult {
     }
 }
 
+#[allow(clippy::needless_update)]
 #[allow(clippy::redundant_closure, clippy::useless_conversion)]
 impl From<MarkdownResult> for crawlberg::MarkdownResult {
     fn from(val: MarkdownResult) -> Self {
@@ -5838,6 +5811,7 @@ impl From<MarkdownResult> for crawlberg::MarkdownResult {
             warnings: val.warnings.into_iter().collect(),
             citations: val.citations,
             fit_content: val.fit_content,
+            ..Default::default()
         }
     }
 }
@@ -5856,6 +5830,7 @@ impl From<crawlberg::MarkdownResult> for MarkdownResult {
     }
 }
 
+#[allow(clippy::needless_update)]
 #[allow(clippy::redundant_closure, clippy::useless_conversion)]
 impl From<LinkInfo> for crawlberg::LinkInfo {
     fn from(val: LinkInfo) -> Self {
@@ -5865,6 +5840,7 @@ impl From<LinkInfo> for crawlberg::LinkInfo {
             link_type: val.link_type.into(),
             rel: val.rel,
             nofollow: val.nofollow,
+            ..Default::default()
         }
     }
 }
@@ -5882,6 +5858,7 @@ impl From<crawlberg::LinkInfo> for LinkInfo {
     }
 }
 
+#[allow(clippy::needless_update)]
 #[allow(clippy::redundant_closure, clippy::useless_conversion)]
 impl From<ImageInfo> for crawlberg::ImageInfo {
     fn from(val: ImageInfo) -> Self {
@@ -5891,6 +5868,7 @@ impl From<ImageInfo> for crawlberg::ImageInfo {
             width: val.width,
             height: val.height,
             source: val.source.into(),
+            ..Default::default()
         }
     }
 }
@@ -5908,6 +5886,7 @@ impl From<crawlberg::ImageInfo> for ImageInfo {
     }
 }
 
+#[allow(clippy::needless_update)]
 #[allow(clippy::redundant_closure, clippy::useless_conversion)]
 impl From<FeedInfo> for crawlberg::FeedInfo {
     fn from(val: FeedInfo) -> Self {
@@ -5915,6 +5894,7 @@ impl From<FeedInfo> for crawlberg::FeedInfo {
             url: val.url,
             title: val.title,
             feed_type: val.feed_type.into(),
+            ..Default::default()
         }
     }
 }
@@ -5930,6 +5910,7 @@ impl From<crawlberg::FeedInfo> for FeedInfo {
     }
 }
 
+#[allow(clippy::needless_update)]
 #[allow(clippy::redundant_closure, clippy::useless_conversion)]
 impl From<JsonLdEntry> for crawlberg::JsonLdEntry {
     fn from(val: JsonLdEntry) -> Self {
@@ -5937,6 +5918,7 @@ impl From<JsonLdEntry> for crawlberg::JsonLdEntry {
             schema_type: val.schema_type,
             name: val.name,
             raw: val.raw,
+            ..Default::default()
         }
     }
 }
@@ -5952,6 +5934,7 @@ impl From<crawlberg::JsonLdEntry> for JsonLdEntry {
     }
 }
 
+#[allow(clippy::needless_update)]
 #[allow(clippy::redundant_closure, clippy::useless_conversion)]
 impl From<CookieInfo> for crawlberg::CookieInfo {
     fn from(val: CookieInfo) -> Self {
@@ -5960,6 +5943,7 @@ impl From<CookieInfo> for crawlberg::CookieInfo {
             value: val.value,
             domain: val.domain,
             path: val.path,
+            ..Default::default()
         }
     }
 }
@@ -5976,6 +5960,7 @@ impl From<crawlberg::CookieInfo> for CookieInfo {
     }
 }
 
+#[allow(clippy::needless_update)]
 #[allow(clippy::redundant_closure, clippy::useless_conversion)]
 impl From<DownloadedAsset> for crawlberg::DownloadedAsset {
     fn from(val: DownloadedAsset) -> Self {
@@ -5986,6 +5971,7 @@ impl From<DownloadedAsset> for crawlberg::DownloadedAsset {
             size: val.size,
             asset_category: val.asset_category.into(),
             html_tag: val.html_tag,
+            ..Default::default()
         }
     }
 }
@@ -6004,6 +5990,7 @@ impl From<crawlberg::DownloadedAsset> for DownloadedAsset {
     }
 }
 
+#[allow(clippy::needless_update)]
 #[allow(clippy::redundant_closure, clippy::useless_conversion)]
 impl From<ArticleMetadata> for crawlberg::ArticleMetadata {
     fn from(val: ArticleMetadata) -> Self {
@@ -6013,6 +6000,7 @@ impl From<ArticleMetadata> for crawlberg::ArticleMetadata {
             author: val.author,
             section: val.section,
             tags: val.tags.into_iter().collect(),
+            ..Default::default()
         }
     }
 }
@@ -6030,12 +6018,14 @@ impl From<crawlberg::ArticleMetadata> for ArticleMetadata {
     }
 }
 
+#[allow(clippy::needless_update)]
 #[allow(clippy::redundant_closure, clippy::useless_conversion)]
 impl From<HreflangEntry> for crawlberg::HreflangEntry {
     fn from(val: HreflangEntry) -> Self {
         Self {
             lang: val.lang,
             url: val.url,
+            ..Default::default()
         }
     }
 }
@@ -6050,6 +6040,7 @@ impl From<crawlberg::HreflangEntry> for HreflangEntry {
     }
 }
 
+#[allow(clippy::needless_update)]
 #[allow(clippy::redundant_closure, clippy::useless_conversion)]
 impl From<FaviconInfo> for crawlberg::FaviconInfo {
     fn from(val: FaviconInfo) -> Self {
@@ -6058,6 +6049,7 @@ impl From<FaviconInfo> for crawlberg::FaviconInfo {
             rel: val.rel,
             sizes: val.sizes,
             mime_type: val.mime_type,
+            ..Default::default()
         }
     }
 }
@@ -6074,12 +6066,14 @@ impl From<crawlberg::FaviconInfo> for FaviconInfo {
     }
 }
 
+#[allow(clippy::needless_update)]
 #[allow(clippy::redundant_closure, clippy::useless_conversion)]
 impl From<HeadingInfo> for crawlberg::HeadingInfo {
     fn from(val: HeadingInfo) -> Self {
         Self {
             level: val.level,
             text: val.text,
+            ..Default::default()
         }
     }
 }
@@ -6094,6 +6088,7 @@ impl From<crawlberg::HeadingInfo> for HeadingInfo {
     }
 }
 
+#[allow(clippy::needless_update)]
 #[allow(clippy::redundant_closure, clippy::useless_conversion)]
 impl From<ResponseMeta> for crawlberg::ResponseMeta {
     fn from(val: ResponseMeta) -> Self {
@@ -6105,6 +6100,7 @@ impl From<ResponseMeta> for crawlberg::ResponseMeta {
             x_powered_by: val.x_powered_by,
             content_language: val.content_language,
             content_encoding: val.content_encoding,
+            ..Default::default()
         }
     }
 }
@@ -6124,6 +6120,7 @@ impl From<crawlberg::ResponseMeta> for ResponseMeta {
     }
 }
 
+#[allow(clippy::needless_update)]
 #[allow(clippy::redundant_closure, clippy::useless_conversion)]
 impl From<PageMetadata> for crawlberg::PageMetadata {
     fn from(val: PageMetadata) -> Self {
@@ -6171,6 +6168,7 @@ impl From<PageMetadata> for crawlberg::PageMetadata {
             favicons: val.favicons.map(|v| v.into_iter().map(Into::into).collect()),
             headings: val.headings.map(|v| v.into_iter().map(Into::into).collect()),
             word_count: val.word_count,
+            ..Default::default()
         }
     }
 }
@@ -6226,10 +6224,14 @@ impl From<crawlberg::PageMetadata> for PageMetadata {
     }
 }
 
+#[allow(clippy::needless_update)]
 #[allow(clippy::redundant_closure, clippy::useless_conversion)]
 impl From<CrawlStreamRequest> for crawlberg::CrawlStreamRequest {
     fn from(val: CrawlStreamRequest) -> Self {
-        Self { url: val.url }
+        Self {
+            url: val.url,
+            ..Default::default()
+        }
     }
 }
 
@@ -6242,11 +6244,13 @@ impl From<crawlberg::CrawlStreamRequest> for CrawlStreamRequest {
     }
 }
 
+#[allow(clippy::needless_update)]
 #[allow(clippy::redundant_closure, clippy::useless_conversion)]
 impl From<BatchCrawlStreamRequest> for crawlberg::BatchCrawlStreamRequest {
     fn from(val: BatchCrawlStreamRequest) -> Self {
         Self {
             urls: val.urls.into_iter().collect(),
+            ..Default::default()
         }
     }
 }
@@ -6260,12 +6264,14 @@ impl From<crawlberg::BatchCrawlStreamRequest> for BatchCrawlStreamRequest {
     }
 }
 
+#[allow(clippy::needless_update)]
 #[allow(clippy::redundant_closure, clippy::useless_conversion)]
 impl From<CitationResult> for crawlberg::CitationResult {
     fn from(val: CitationResult) -> Self {
         Self {
             content: val.content,
             references: val.references.into_iter().map(Into::into).collect(),
+            ..Default::default()
         }
     }
 }
@@ -6280,6 +6286,7 @@ impl From<crawlberg::CitationResult> for CitationResult {
     }
 }
 
+#[allow(clippy::needless_update)]
 #[allow(clippy::redundant_closure, clippy::useless_conversion)]
 impl From<CitationReference> for crawlberg::CitationReference {
     fn from(val: CitationReference) -> Self {
@@ -6287,6 +6294,7 @@ impl From<CitationReference> for crawlberg::CitationReference {
             index: val.index,
             url: val.url,
             text: val.text,
+            ..Default::default()
         }
     }
 }
@@ -6302,6 +6310,7 @@ impl From<crawlberg::CitationReference> for CitationReference {
     }
 }
 
+#[allow(clippy::needless_update)]
 #[allow(clippy::redundant_closure, clippy::useless_conversion)]
 impl From<BatchScrapeResult> for crawlberg::BatchScrapeResult {
     fn from(val: BatchScrapeResult) -> Self {
@@ -6309,6 +6318,7 @@ impl From<BatchScrapeResult> for crawlberg::BatchScrapeResult {
             url: val.url,
             result: val.result.map(Into::into),
             error: val.error,
+            ..Default::default()
         }
     }
 }
@@ -6324,6 +6334,7 @@ impl From<crawlberg::BatchScrapeResult> for BatchScrapeResult {
     }
 }
 
+#[allow(clippy::needless_update)]
 #[allow(clippy::redundant_closure, clippy::useless_conversion)]
 impl From<BatchCrawlResult> for crawlberg::BatchCrawlResult {
     fn from(val: BatchCrawlResult) -> Self {
@@ -6331,6 +6342,7 @@ impl From<BatchCrawlResult> for crawlberg::BatchCrawlResult {
             url: val.url,
             result: val.result.map(Into::into),
             error: val.error,
+            ..Default::default()
         }
     }
 }
@@ -6346,6 +6358,7 @@ impl From<crawlberg::BatchCrawlResult> for BatchCrawlResult {
     }
 }
 
+#[allow(clippy::needless_update)]
 #[allow(clippy::redundant_closure, clippy::useless_conversion)]
 impl From<BatchScrapeResults> for crawlberg::BatchScrapeResults {
     fn from(val: BatchScrapeResults) -> Self {
@@ -6354,6 +6367,7 @@ impl From<BatchScrapeResults> for crawlberg::BatchScrapeResults {
             total_count: val.total_count,
             completed_count: val.completed_count,
             failed_count: val.failed_count,
+            ..Default::default()
         }
     }
 }
@@ -6370,6 +6384,7 @@ impl From<crawlberg::BatchScrapeResults> for BatchScrapeResults {
     }
 }
 
+#[allow(clippy::needless_update)]
 #[allow(clippy::redundant_closure, clippy::useless_conversion)]
 impl From<BatchCrawlResults> for crawlberg::BatchCrawlResults {
     fn from(val: BatchCrawlResults) -> Self {
@@ -6378,6 +6393,7 @@ impl From<BatchCrawlResults> for crawlberg::BatchCrawlResults {
             total_count: val.total_count,
             completed_count: val.completed_count,
             failed_count: val.failed_count,
+            ..Default::default()
         }
     }
 }
@@ -6766,7 +6782,6 @@ impl CrawlStreamIterator {
             }
             Some(Err(e)) => Err(Error::new(ruby.exception_runtime_error(), e.to_string())),
             None => {
-                // Drop the stream to release any resources.
                 let inner = self.inner.clone();
                 let runtime = self.runtime.clone();
                 runtime.block_on(async move {
@@ -6784,8 +6799,6 @@ impl CrawlStreamIterator {
         use magnus::value::ReprValue;
         let ruby = unsafe { Ruby::get_unchecked() };
         if !ruby.block_given() {
-            // Without a block, return an Enumerator over `each` so the caller can
-            // call `.to_a`, `.lazy`, etc.
             let self_val: magnus::Value = self.clone().into_value_with(&ruby);
             let enumerator = self_val.enumeratorize(ruby.to_symbol("each"), ());
             return Ok(enumerator.as_value());
@@ -6848,7 +6861,6 @@ impl BatchCrawlStreamIterator {
             }
             Some(Err(e)) => Err(Error::new(ruby.exception_runtime_error(), e.to_string())),
             None => {
-                // Drop the stream to release any resources.
                 let inner = self.inner.clone();
                 let runtime = self.runtime.clone();
                 runtime.block_on(async move {
@@ -6866,8 +6878,6 @@ impl BatchCrawlStreamIterator {
         use magnus::value::ReprValue;
         let ruby = unsafe { Ruby::get_unchecked() };
         if !ruby.block_given() {
-            // Without a block, return an Enumerator over `each` so the caller can
-            // call `.to_a`, `.lazy`, etc.
             let self_val: magnus::Value = self.clone().into_value_with(&ruby);
             let enumerator = self_val.enumeratorize(ruby.to_symbol("each"), ());
             return Ok(enumerator.as_value());
@@ -6887,7 +6897,6 @@ impl BatchCrawlStreamIterator {
 fn ruby_init(ruby: &Ruby) -> Result<(), Error> {
     let module = ruby.define_module("Crawlberg")?;
 
-    // Ensure JSON library is loaded for Hash#to_json
     let _ = ruby.eval::<magnus::Value>("require \"json\"");
 
     let class = module.define_class("ExtractionMeta", ruby.class_object())?;