emit deprecation on pyclasses that are affected by the FromPyObject blanket impl

Author: Icxolu

guide/src/class.md

@@ -887,7 +887,7 @@ Classes can also be passed by value if they can be cloned, i.e. they automatical
 ```rust,no_run
 # #![allow(dead_code)]
 # use pyo3::prelude::*;
-#[pyclass]
+#[pyclass(from_py_object)]
 #[derive(Clone)]
 struct MyClass {
     my_field: Box<i32>,

guide/src/faq.md

@@ -88,7 +88,7 @@ You may have a nested struct similar to this:
 
 ```rust,no_run
 # use pyo3::prelude::*;
-#[pyclass]
+#[pyclass(from_py_object)]
 #[derive(Clone)]
 struct Inner {/* fields omitted */}
 

newsfragments/5550.changed.md

@@ -0,0 +1 @@
+deprecate implicit by value extraction of pyclasses

pyo3-macros-backend/src/pyclass.rs

@@ -2517,6 +2517,20 @@ impl<'a> PyClassImplsBuilder<'a> {
             quote! {}
         };
 
+        let deprecation = if self.attr.options.skip_from_py_object.is_none()
+            && self.attr.options.from_py_object.is_none()
+        {
+            quote! {
+                const _: () = {
+                    #[allow(unused_import)]
+                    use #pyo3_path::impl_::pyclass::Probe as _;
+                    #pyo3_path::impl_::deprecated::DeprecatedFromPyObjectBlanket::<{ #pyo3_path::impl_::pyclass::IsClone::<#cls>::VALUE }>::MSG
+                };
+            }
+        } else {
+            TokenStream::new()
+        };
+
         let extract_pyclass_with_clone = if let Some(from_py_object) =
             self.attr.options.from_py_object
         {
@@ -2546,6 +2560,8 @@ impl<'a> PyClassImplsBuilder<'a> {
         };
 
         Ok(quote! {
+            #deprecation
+
             #extract_pyclass_with_clone
 
             #assertions

pytests/src/pyclasses.rs

@@ -4,7 +4,7 @@ use pyo3::exceptions::{PyStopIteration, PyValueError};
 use pyo3::prelude::*;
 use pyo3::types::PyType;
 
-#[pyclass]
+#[pyclass(from_py_object)]
 #[derive(Clone, Default)]
 struct EmptyClass {}
 
@@ -70,7 +70,7 @@ impl PyClassThreadIter {
 }
 
 /// Demonstrates a base class which can operate on the relevant subclass in its constructor.
-#[pyclass(subclass)]
+#[pyclass(subclass, skip_from_py_object)]
 #[derive(Clone, Debug)]
 struct AssertingBaseClass;
 
@@ -104,7 +104,7 @@ impl ClassWithDict {
     }
 }
 
-#[pyclass]
+#[pyclass(skip_from_py_object)]
 #[derive(Clone)]
 struct ClassWithDecorators {
     attr: usize,

src/impl_.rs

@@ -10,6 +10,7 @@ pub mod callback;
 pub mod concat;
 #[cfg(feature = "experimental-async")]
 pub mod coroutine;
+pub mod deprecated;
 pub mod exceptions;
 pub mod extract_argument;
 pub mod freelist;

src/impl_/deprecated.rs

@@ -0,0 +1,13 @@
+pub struct DeprecatedFromPyObjectBlanket<const IS_CLONE: bool> {}
+
+impl DeprecatedFromPyObjectBlanket<true> {
+    #[deprecated(
+        since = "0.28.0",
+        note = "Implicit by value extraction of pyclasses is deprecated. Use `from_py_object` to keep the current behaviour or `skip_from_py_object` to opt-out."
+    )]
+    pub const MSG: () = ();
+}
+
+impl DeprecatedFromPyObjectBlanket<false> {
+    pub const MSG: () = ();
+}

src/impl_/pyclass/probes.rs

@@ -73,6 +73,12 @@ impl<T: super::doc::PyClassNewTextSignature> HasNewTextSignature<T> {
     pub const VALUE: bool = true;
 }
 
+probe!(IsClone);
+
+impl<T: Clone> IsClone<T> {
+    pub const VALUE: bool = true;
+}
+
 #[cfg(test)]
 macro_rules! value_of {
     ($probe:ident, $ty:ty) => {{

src/marker.rs

@@ -1007,7 +1007,7 @@ mod tests {
     fn test_py_run_inserts_globals_2() {
         use std::ffi::CString;
 
-        #[crate::pyclass(crate = "crate")]
+        #[crate::pyclass(crate = "crate", skip_from_py_object)]
         #[derive(Clone)]
         struct CodeRunner {
             code: CString,

src/pycell.rs

@@ -751,7 +751,7 @@ mod tests {
 
     use super::*;
 
-    #[crate::pyclass]
+    #[crate::pyclass(skip_from_py_object)]
     #[pyo3(crate = "crate")]
     #[derive(Copy, Clone, PartialEq, Eq, Debug)]
     struct SomeClass(i32);