fix

Author: Wandalen

module/core/former_meta/src/derive_former/former_enum.rs

@@ -118,7 +118,7 @@
 #![allow(unused_variables)] // Temporary for placeholder handlers
 
 
-use macro_tools::{Result, generic_params::GenericsRef, syn, proc_macro2, diag};
+use macro_tools::{Result, generic_params::GenericsRef, syn, proc_macro2};
 use macro_tools::quote::{format_ident, quote};
 use macro_tools::proc_macro2::TokenStream;
 use super::struct_attrs::ItemAttributes; // Corrected import

module/core/former_meta/src/derive_former/former_enum/unit_variant_handler.rs

@@ -118,7 +118,7 @@
 //! - **Unit Semantics**: Maintains proper Rust unit variant semantics and syntax
 
 use super::*;
-use macro_tools::{Result, quote::quote, syn_err};
+use macro_tools::{Result, quote::quote};
 use crate::derive_former::raw_identifier_utils::variant_to_method_name;
 use crate::derive_former::attribute_validation::{validate_variant_attributes, get_field_count, get_variant_type};
 

module/core/former_meta/src/derive_former/raw_identifier_utils.rs

@@ -82,6 +82,7 @@ fn is_rust_keyword(s: &str) -> bool {
 /// 
 /// This is similar to `ident::ident_maybe_raw` but specifically designed for
 /// parameter name generation in constructor contexts.
+#[allow(dead_code)]
 pub fn field_to_param_name(field_ident: &syn::Ident) -> syn::Ident {
     ident::ident_maybe_raw(field_ident)
 }
@@ -105,6 +106,7 @@ pub fn strip_raw_prefix_for_compound_ident(ident: &syn::Ident) -> String {
 }
 
 /// Creates a constructor name from a struct/enum name, handling raw identifiers.
+#[allow(dead_code)]
 pub fn type_to_constructor_name(type_ident: &syn::Ident) -> syn::Ident {
     let type_str = type_ident.to_string();
 
@@ -132,7 +134,7 @@ pub fn type_to_constructor_name(type_ident: &syn::Ident) -> syn::Ident {
 #[cfg(test)]
 mod tests {
     use super::*;
-    use quote::format_ident;
+    use macro_tools::quote::format_ident;
 
     #[test]
     fn test_variant_to_method_name_normal() {

module/core/former_meta/src/derive_former/trait_detection.rs

@@ -26,6 +26,7 @@ use macro_tools::{ syn, quote::quote, proc_macro2 };
 ///     fn has_former() -> bool { true }
 /// }
 /// ```
+#[allow(dead_code)]
 pub fn generate_former_trait_detector() -> proc_macro2::TokenStream {
     quote! {
         // Compile-time trait detection helper
@@ -46,6 +47,7 @@ pub fn generate_former_trait_detector() -> proc_macro2::TokenStream {
 /// Generates code to check if a type implements Former at compile-time.
 /// 
 /// Returns a boolean expression that evaluates to true if the type implements Former.
+#[allow(dead_code)]
 pub fn generate_former_check(field_type: &syn::Type) -> proc_macro2::TokenStream {
     quote! {
         <() as __FormerDetector<#field_type>>::HAS_FORMER
@@ -58,6 +60,7 @@ pub fn generate_former_check(field_type: &syn::Type) -> proc_macro2::TokenStream
 /// This allows handlers to automatically select the best approach:
 /// - If type implements Former: Use subform delegation  
 /// - If type doesn't implement Former: Use scalar/direct approach
+#[allow(dead_code)]
 pub fn generate_smart_routing(
     field_type: &syn::Type,
     subform_approach: proc_macro2::TokenStream,
@@ -76,6 +79,7 @@ pub fn generate_smart_routing(
 
 /// Generates a const assertion that can be used to provide better error messages
 /// when trait requirements aren't met.
+#[allow(dead_code)]
 pub fn generate_former_assertion(field_type: &syn::Type, _context: &str) -> proc_macro2::TokenStream {
     quote! {
         const _: fn() = || {
@@ -89,6 +93,7 @@ pub fn generate_former_assertion(field_type: &syn::Type, _context: &str) -> proc
 
 /// Configuration for smart routing behavior
 #[derive(Debug, Clone)]
+#[allow(dead_code)]
 pub struct SmartRoutingConfig {
     /// Whether to prefer subform approach when Former is detected
     pub prefer_subform: bool,
@@ -109,6 +114,7 @@ impl Default for SmartRoutingConfig {
 }
 
 /// Advanced smart routing with configuration options
+#[allow(dead_code)]
 pub fn generate_configurable_smart_routing(
     field_type: &syn::Type,
     subform_approach: proc_macro2::TokenStream,

module/core/test_tools/src/lib.rs

@@ -133,6 +133,11 @@ pub use standalone::*;
 #[cfg(not(all(feature = "standalone_build", not(feature = "normal_build"))))]
 pub use ::{error_tools, collection_tools, impls_index, mem_tools, typing_tools, diagnostics_tools};
 
+#[cfg(feature = "enabled")]
+#[cfg(not(feature = "doctest"))]
+#[cfg(not(all(feature = "standalone_build", not(feature = "normal_build"))))]
+pub use error_tools::error;
+
 #[cfg(feature = "enabled")]
 #[cfg(not(feature = "doctest"))]
 #[cfg(all(feature = "standalone_build", not(feature = "normal_build")))]
@@ -164,6 +169,7 @@ pub mod own {
 
   #[doc(inline)]
   pub use {
+    error_tools::{debug_assert_id, debug_assert_identical, debug_assert_ni, debug_assert_not_identical, ErrWith},
     collection_tools::orphan::*, impls_index::orphan::*, mem_tools::orphan::*, typing_tools::orphan::*,
     diagnostics_tools::orphan::*,
   };
@@ -198,6 +204,7 @@ pub mod exposed {
 
   #[doc(inline)]
   pub use {
+    error_tools::{debug_assert_id, debug_assert_identical, debug_assert_ni, debug_assert_not_identical, ErrWith},
     collection_tools::exposed::*, impls_index::exposed::*, mem_tools::exposed::*, typing_tools::exposed::*,
     diagnostics_tools::exposed::*,
   };
@@ -217,6 +224,7 @@ pub mod prelude {
 
   #[doc(inline)]
   pub use {
+    error_tools::{debug_assert_id, debug_assert_identical, debug_assert_ni, debug_assert_not_identical, ErrWith},
     collection_tools::prelude::*, impls_index::prelude::*, mem_tools::prelude::*, typing_tools::prelude::*,
     diagnostics_tools::prelude::*,
   };