Add missing lifetime replacement for generic param decls on derive(IntoFuture) impl block (#337)

Author: Veetaha

bon-macros/src/builder/builder_gen/builder_derives/into_future.rs

@@ -163,6 +163,10 @@ impl BuilderGenCtx {
             original_lifetimes: &original_lifetimes,
         };
 
+        for decl in &mut new_generics_decl {
+            replace_lifetimes.visit_generic_param_mut(decl);
+        }
+
         let mut new_where_clause = where_clause.clone();
 
         if let Some(where_clause) = &mut new_where_clause {

bon/tests/integration/builder/attr_into_future.rs

@@ -151,6 +151,30 @@ mod tests {
 
             let &Dummy = assert_send(builder).await;
         }
+
+        #[tokio::test]
+        async fn complex_generics() {
+            struct Dummy;
+
+            #[builder(derive(IntoFuture(Box)))]
+            async fn sut<'a: 'b, 'b, T: Sync + 'a + 'b, U: Sync + 'a, const N: usize>(
+                x1: &'a T,
+                x2: &'b U,
+            ) -> (&'a T, &'b U, usize) {
+                async {}.await;
+                (x1, x2, N)
+            }
+
+            // Store the dummy struct in local variables to make sure no `'static`
+            // lifetime promotion happens
+            let local_x1 = Dummy;
+            let local_x2 = Dummy;
+
+            let builder = sut::<_, _, 42>().x1(&local_x1).x2(&local_x2);
+
+            let (&Dummy, &Dummy, usize) = assert_send(builder).await;
+            assert_eq!(usize, 42);
+        }
     }
 
     mod test_method {
@@ -253,5 +277,32 @@ mod tests {
 
             let _: &Dummy = assert_send(builder).await;
         }
+
+        #[tokio::test]
+        async fn complex_generics() {
+            struct Dummy;
+
+            #[bon]
+            impl Dummy {
+                #[builder(derive(IntoFuture(Box)))]
+                async fn sut<'a: 'b, 'b, T: Sync + 'a + 'b, U: Sync + 'a, const N: usize>(
+                    x1: &'a T,
+                    x2: &'b U,
+                ) -> (&'a T, &'b U, usize) {
+                    async {}.await;
+                    (x1, x2, N)
+                }
+            }
+
+            // Store the dummy struct in local variables to make sure no `'static`
+            // lifetime promotion happens
+            let local_x1 = Dummy;
+            let local_x2 = Dummy;
+
+            let builder = Dummy::sut::<_, _, 42>().x1(&local_x1).x2(&local_x2);
+
+            let (&Dummy, &Dummy, usize) = assert_send(builder).await;
+            assert_eq!(usize, 42);
+        }
     }
 }