fix by only running nested if outer is not run

CHANGELOG.md

@@ -1,5 +1,10 @@
 ## Changelog
 
+## 0.19.1 (unreleased)
+
+- Bug fix for nested issue with custom only running nested if outer passes
+
+
 ## 0.19.0 (2024/11/03)
 
 - Swap to using proc-macro-error-2 instead of proc-macro-error for Syn

validator/src/types.rs

@@ -181,9 +181,10 @@ impl ValidationErrors {
     fn add_nested(&mut self, field: &'static str, errors: ValidationErrorsKind) {
         if let Vacant(entry) = self.0.entry(field) {
             entry.insert(errors);
-        } else {
-            panic!("Attempt to replace non-empty ValidationErrors entry");
         }
+        // Else case here is simply do nothing. Pretty sure this is caught earlier to not run but since
+        // If we get here, just not adding it does the same thing (although waste extra compute)
+        // probably better to just ignore here.
     }
 
     #[must_use]

validator_derive/src/tokens/nested.rs

@@ -5,6 +5,8 @@ pub fn nested_tokens(
     field_name_str: &str,
 ) -> proc_macro2::TokenStream {
     quote! {
-        errors.merge_self(#field_name_str, (&#field_name).validate());
+        if let std::collections::hash_map::Entry::Vacant(entry) = errors.0.entry(#field_name_str) {
+            errors.merge_self(#field_name_str, (&#field_name).validate());
+        }
     }
 }