deny large stack frames

async-stripe-webhook/src/lib.rs

@@ -1,3 +1,4 @@
+#![deny(clippy::large_stack_frames)]
 #![allow(clippy::large_enum_variant)]
 #![recursion_limit = "256"]
 //! This crate provides Rust bindings for handling [Stripe webhook events](https://stripe.com/docs/webhooks).

generated/async-stripe-billing/src/mod.rs

@@ -1,4 +1,5 @@
 #![recursion_limit = "256"]
+#![deny(clippy::large_stack_frames)]
 #![allow(clippy::large_enum_variant)]
 #![allow(rustdoc::broken_intra_doc_links)]
 #![allow(rustdoc::invalid_html_tags)]

generated/async-stripe-checkout/src/mod.rs

@@ -1,4 +1,5 @@
 #![recursion_limit = "256"]
+#![deny(clippy::large_stack_frames)]
 #![allow(clippy::large_enum_variant)]
 #![allow(rustdoc::broken_intra_doc_links)]
 #![allow(rustdoc::invalid_html_tags)]

generated/async-stripe-connect/src/mod.rs

@@ -1,4 +1,5 @@
 #![recursion_limit = "256"]
+#![deny(clippy::large_stack_frames)]
 #![allow(clippy::large_enum_variant)]
 #![allow(rustdoc::broken_intra_doc_links)]
 #![allow(rustdoc::invalid_html_tags)]

generated/async-stripe-core/src/mod.rs

@@ -1,4 +1,5 @@
 #![recursion_limit = "256"]
+#![deny(clippy::large_stack_frames)]
 #![allow(clippy::large_enum_variant)]
 #![allow(rustdoc::broken_intra_doc_links)]
 #![allow(rustdoc::invalid_html_tags)]

generated/async-stripe-fraud/src/mod.rs

@@ -1,4 +1,5 @@
 #![recursion_limit = "256"]
+#![deny(clippy::large_stack_frames)]
 #![allow(clippy::large_enum_variant)]
 #![allow(rustdoc::broken_intra_doc_links)]
 #![allow(rustdoc::invalid_html_tags)]

generated/async-stripe-issuing/src/mod.rs

@@ -1,4 +1,5 @@
 #![recursion_limit = "256"]
+#![deny(clippy::large_stack_frames)]
 #![allow(clippy::large_enum_variant)]
 #![allow(rustdoc::broken_intra_doc_links)]
 #![allow(rustdoc::invalid_html_tags)]

generated/async-stripe-misc/src/mod.rs

@@ -1,4 +1,5 @@
 #![recursion_limit = "256"]
+#![deny(clippy::large_stack_frames)]
 #![allow(clippy::large_enum_variant)]
 #![allow(rustdoc::broken_intra_doc_links)]
 #![allow(rustdoc::invalid_html_tags)]

generated/async-stripe-payment/src/mod.rs

@@ -1,4 +1,5 @@
 #![recursion_limit = "256"]
+#![deny(clippy::large_stack_frames)]
 #![allow(clippy::large_enum_variant)]
 #![allow(rustdoc::broken_intra_doc_links)]
 #![allow(rustdoc::invalid_html_tags)]

generated/async-stripe-product/src/mod.rs

@@ -1,4 +1,5 @@
 #![recursion_limit = "256"]
+#![deny(clippy::large_stack_frames)]
 #![allow(clippy::large_enum_variant)]
 #![allow(rustdoc::broken_intra_doc_links)]
 #![allow(rustdoc::invalid_html_tags)]

generated/async-stripe-shared/src/mod.rs

@@ -1,4 +1,5 @@
 #![recursion_limit = "256"]
+#![deny(clippy::large_stack_frames)]
 #![allow(clippy::large_enum_variant)]
 #![allow(rustdoc::broken_intra_doc_links)]
 #![allow(rustdoc::invalid_html_tags)]

generated/async-stripe-terminal/src/mod.rs

@@ -1,4 +1,5 @@
 #![recursion_limit = "256"]
+#![deny(clippy::large_stack_frames)]
 #![allow(clippy::large_enum_variant)]
 #![allow(rustdoc::broken_intra_doc_links)]
 #![allow(rustdoc::invalid_html_tags)]

generated/async-stripe-treasury/src/mod.rs

@@ -1,4 +1,5 @@
 #![recursion_limit = "256"]
+#![deny(clippy::large_stack_frames)]
 #![allow(clippy::large_enum_variant)]
 #![allow(rustdoc::broken_intra_doc_links)]
 #![allow(rustdoc::invalid_html_tags)]

openapi/src/codegen.rs

@@ -111,6 +111,7 @@ impl CodeGen {
             let mod_rs = formatdoc! {
                 r#"
             #![recursion_limit = "256"]
+            #![deny(clippy::large_stack_frames)]
             #![allow(clippy::large_enum_variant)]
             #![allow(rustdoc::broken_intra_doc_links)]
             #![allow(rustdoc::invalid_html_tags)]

openapi/src/main.rs

@@ -104,6 +104,10 @@ fn main() -> Result<()> {
 
     if !args.dry_run {
         info!("Formatting generated files");
+
+        // in a rather bizarre turn of events, we actually need to run fmt _twice_
+        // to get it 'settled'. otherwise it will fail formatting checks
+        let _out = fmt_cmd.output()?;
         let out = fmt_cmd.output()?;
         if !out.status.success() {
             bail!("Rustfmt failed with outputs {out:?}");

openapi/src/webhook.rs

@@ -74,7 +74,7 @@ fn write_event_object(components: &Components, out_path: &Path) -> anyhow::Resul
         let evt_type = &webhook_obj.event_type;
         let _ = writeln!(
             match_inner,
-            r#""{evt_type}" => Self::{ident}(FromValueOpt::from_value(data)?),"#
+            r#"if typ == "{evt_type}" {{ return FromValueOpt::from_value(data).map(Self::{ident}) }}"#
         );
     }
     let _ = writedoc! {enum_body, r#"
@@ -93,17 +93,16 @@ fn write_event_object(components: &Components, out_path: &Path) -> anyhow::Resul
     /// The event data for a webhook event.
     pub enum EventObject {{
         {enum_body}
-    }} 
+    }}
     "#};
 
     let _ = writedoc! {out, r#"
     impl EventObject {{
         pub(crate) fn from_raw_data(typ: &str, data: miniserde::json::Value) -> Option<Self> {{
             use stripe_types::miniserde_helpers::FromValueOpt;
-            Some(match typ {{
-                {match_inner}
-                _ => Self::Unknown(data),
-            }})
+            {match_inner}
+
+            Some(Self::Unknown(data))
         }}
     }}
     "#};