derive debug only in dev mode (#204)

* derive debug only in dev mode

* one more...

Author: samuelcolvin

src/errors/kinds.rs

@@ -10,7 +10,8 @@ use super::PydanticValueError;
 /// * the variables in the message need to match the enum struct
 /// * you need to add an entry to the `render` enum to render the error message as a template
 /// * you need to add an entry to the `py_dict` enum to generate `ctx` for error messages
-#[derive(Debug, Display, EnumMessage, Clone)]
+#[derive(Display, EnumMessage, Clone)]
+#[cfg_attr(debug_assertions, derive(Debug))]
 #[strum(serialize_all = "snake_case")]
 pub enum ErrorKind {
     #[strum(message = "Invalid input")]

src/errors/line_error.rs

@@ -10,7 +10,7 @@ use super::validation_exception::{pretty_py_line_errors, PyLineError};
 
 pub type ValResult<'a, T> = Result<T, ValError<'a>>;
 
-#[derive(Debug)]
+#[cfg_attr(debug_assertions, derive(Debug))]
 pub enum ValError<'a> {
     LineErrors(Vec<ValLineError<'a>>),
     InternalErr(PyErr),
@@ -65,7 +65,7 @@ pub fn pretty_line_errors(py: Python, line_errors: Vec<ValLineError>) -> String
 /// A `ValLineError` is a single error that occurred during validation which is converted to a `PyLineError`
 /// to eventually form a `ValidationError`.
 /// I don't like the name `ValLineError`, but it's the best I could come up with (for now).
-#[derive(Debug)]
+#[cfg_attr(debug_assertions, derive(Debug))]
 pub struct ValLineError<'a> {
     pub kind: ErrorKind,
     // location is reversed so that adding an "outer" location item is pushing, it's reversed before showing to the user
@@ -112,7 +112,7 @@ impl<'a> ValLineError<'a> {
     }
 }
 
-#[derive(Debug)]
+#[cfg_attr(debug_assertions, derive(Debug))]
 pub enum InputValue<'a> {
     PyAny(&'a PyAny),
     JsonInput(&'a JsonInput),

src/errors/location.rs

@@ -5,7 +5,8 @@ use pyo3::types::PyList;
 
 /// Used to store individual items of the error location, e.g. a string for key/field names
 /// or a number for array indices.
-#[derive(Debug, Clone)]
+#[derive(Clone)]
+#[cfg_attr(debug_assertions, derive(Debug))]
 pub enum LocItem {
     /// string type key, used to identify items from a dict or anything that implements `__getitem__`
     S(String),
@@ -55,7 +56,8 @@ impl ToPyObject for LocItem {
 /// Note: location in List is stored in **REVERSE** so adding an "outer" item to location involves
 /// pushing to the vec which is faster than inserting and shifting everything along.
 /// Then when "using" location in `Display` and `ToPyObject` order has to be reversed
-#[derive(Debug, Clone)]
+#[derive(Clone)]
+#[cfg_attr(debug_assertions, derive(Debug))]
 pub enum Location {
     // no location, avoid creating an unnecessary vec
     Empty,

src/errors/validation_exception.rs

@@ -13,7 +13,8 @@ use super::location::Location;
 use super::ValError;
 
 #[pyclass(extends=PyValueError, module="pydantic_core._pydantic_core")]
-#[derive(Debug, Clone)]
+#[derive(Clone)]
+#[cfg_attr(debug_assertions, derive(Debug))]
 pub struct ValidationError {
     line_errors: Vec<PyLineError>,
     title: PyObject,
@@ -110,7 +111,8 @@ pub fn pretty_py_line_errors<'a>(py: Python, line_errors_iter: impl Iterator<Ite
 
 /// `PyLineError` are the public version of `ValLineError`, as help and used in `ValidationError`s
 #[pyclass]
-#[derive(Debug, Clone)]
+#[derive(Clone)]
+#[cfg_attr(debug_assertions, derive(Debug))]
 pub struct PyLineError {
     kind: ErrorKind,
     location: Location,

src/errors/value_exception.rs

@@ -7,7 +7,8 @@ use crate::input::Input;
 use super::{ErrorKind, ValError};
 
 #[pyclass(extends=PyValueError, module="pydantic_core._pydantic_core")]
-#[derive(Debug, Clone)]
+#[derive(Clone)]
+#[cfg_attr(debug_assertions, derive(Debug))]
 pub struct PydanticValueError {
     kind: String,
     message_template: String,

src/input/datetime.rs

@@ -8,6 +8,7 @@ use crate::errors::{ErrorKind, ValError, ValResult};
 
 use super::Input;
 
+#[cfg_attr(debug_assertions, derive(Debug))]
 pub enum EitherDate<'a> {
     Raw(Date),
     Py(&'a PyDate),
@@ -53,6 +54,7 @@ impl<'a> EitherDate<'a> {
     }
 }
 
+#[cfg_attr(debug_assertions, derive(Debug))]
 pub enum EitherTime<'a> {
     Raw(Time),
     Py(&'a PyTime),
@@ -70,7 +72,7 @@ impl<'a> From<&'a PyTime> for EitherTime<'a> {
     }
 }
 
-#[derive(Debug, Clone)]
+#[cfg_attr(debug_assertions, derive(Debug))]
 pub enum EitherTimedelta<'a> {
     Raw(Duration),
     Py(&'a PyDelta),
@@ -168,6 +170,7 @@ impl<'a> EitherTime<'a> {
     }
 }
 
+#[cfg_attr(debug_assertions, derive(Debug))]
 pub enum EitherDateTime<'a> {
     Raw(DateTime),
     Py(&'a PyDateTime),
@@ -386,7 +389,8 @@ pub fn float_as_duration(total_seconds: f64) -> Duration {
 }
 
 #[pyclass(module = "pydantic_core._pydantic_core", extends = PyTzInfo)]
-#[derive(Debug, Clone)]
+#[derive(Clone)]
+#[cfg_attr(debug_assertions, derive(Debug))]
 struct TzInfo {
     seconds: i32,
 }

src/input/return_enums.rs

@@ -13,7 +13,7 @@ use super::Input;
 /// Container for all the "list-like" types which can be converted to each other in lax mode.
 /// This cannot be called `GenericSequence` (as it previously was) or `GenericIterable` since it's
 /// members don't match python's definition of `Sequence` or `Iterable`.
-#[derive(Debug)]
+#[cfg_attr(debug_assertions, derive(Debug))]
 pub enum GenericListLike<'a> {
     List(&'a PyList),
     Tuple(&'a PyTuple),
@@ -165,7 +165,7 @@ impl<'a> GenericListLike<'a> {
     }
 }
 
-#[derive(Debug)]
+#[cfg_attr(debug_assertions, derive(Debug))]
 pub enum GenericMapping<'a> {
     PyDict(&'a PyDict),
     PyGetAttr(&'a PyAny),
@@ -176,7 +176,7 @@ derive_from!(GenericMapping, PyDict, PyDict);
 derive_from!(GenericMapping, PyGetAttr, PyAny);
 derive_from!(GenericMapping, JsonObject, JsonObject);
 
-#[derive(Debug)]
+#[cfg_attr(debug_assertions, derive(Debug))]
 pub struct PyArgs<'a> {
     pub args: Option<&'a PyTuple>,
     pub kwargs: Option<&'a PyDict>,
@@ -188,7 +188,7 @@ impl<'a> PyArgs<'a> {
     }
 }
 
-#[derive(Debug)]
+#[cfg_attr(debug_assertions, derive(Debug))]
 pub struct JsonArgs<'a> {
     pub args: Option<&'a [JsonInput]>,
     pub kwargs: Option<&'a JsonObject>,
@@ -200,7 +200,7 @@ impl<'a> JsonArgs<'a> {
     }
 }
 
-#[derive(Debug)]
+#[cfg_attr(debug_assertions, derive(Debug))]
 pub enum GenericArguments<'a> {
     Py(PyArgs<'a>),
     Json(JsonArgs<'a>),
@@ -218,7 +218,7 @@ impl<'a> From<JsonArgs<'a>> for GenericArguments<'a> {
     }
 }
 
-#[derive(Debug)]
+#[cfg_attr(debug_assertions, derive(Debug))]
 pub enum EitherString<'a> {
     Cow(Cow<'a, str>),
     Py(&'a PyString),
@@ -264,7 +264,7 @@ impl<'a> IntoPy<PyObject> for EitherString<'a> {
     }
 }
 
-#[derive(Debug)]
+#[cfg_attr(debug_assertions, derive(Debug))]
 pub enum EitherBytes<'a> {
     Cow(Cow<'a, [u8]>),
     Py(&'a PyBytes),