fix: General fixes

Author: caiopiccirillo

examples/example.cpp

@@ -17,19 +17,19 @@ int main()
         auto& config = app.GetConfig();
 
         // Type-safe value retrieval
-        std::string db_url = config.GetSetting(example::AppConfigName::DatabaseUrl).Value<std::string>();
+        auto db_url = config.GetSetting(example::AppConfigName::DatabaseUrl).Value<std::string>();
         std::cout << "Database URL: " << db_url << '\n';
 
-        int max_conn = config.GetSetting(example::AppConfigName::MaxConnections).Value<int>();
+        auto max_conn = config.GetSetting(example::AppConfigName::MaxConnections).Value<int>();
         std::cout << "Max Connections: " << max_conn << '\n';
 
-        bool logging_enabled = config.GetSetting(example::AppConfigName::EnableLogging).Value<bool>();
+        auto logging_enabled = config.GetSetting(example::AppConfigName::EnableLogging).Value<bool>();
         std::cout << "Logging Enabled: " << (logging_enabled ? "Yes" : "No") << '\n';
 
-        int retry_count = config.GetSetting(example::AppConfigName::RetryCount).Value<int>();
+        auto retry_count = config.GetSetting(example::AppConfigName::RetryCount).Value<int>();
         std::cout << "Retry Count: " << retry_count << '\n';
 
-        std::string log_level = config.GetSetting(example::AppConfigName::LogLevel).Value<std::string>();
+        auto log_level = config.GetSetting(example::AppConfigName::LogLevel).Value<std::string>();
         std::cout << "Log Level: " << log_level << '\n';
 
         // Update a setting

src/GenericConfiguration.h

@@ -19,7 +19,7 @@ namespace config {
  * @brief Type trait to check if a serializer derives from IConfigurationSerializer
  */
 template <typename E, typename Serializer>
-struct is_valid_serializer : std::is_base_of<IConfigurationSerializer<E>, Serializer> { };
+struct is_valid_serializer : std::is_base_of<IConfigurationSerializer<E>, Serializer> { }; // NOLINT(readability-identifier-naming)
 
 /**
  * @brief Generic configuration implementation

src/JsonSerializer.h

@@ -7,6 +7,7 @@
 #include <nlohmann/json_fwd.hpp>
 #include <optional>
 #include <ostream>
+#include <stdexcept>
 #include <string>
 #include <type_traits>
 #include <unordered_map>

src/Setting.h

@@ -5,11 +5,12 @@
 #include <stdexcept>
 #include <string>
 #include <type_traits>
+#include <typeinfo>
+#include <utility>
 #include <variant>
 
 namespace config {
 
-// Legacy variant type for backward compatibility
 using SettingValueType = std::variant<int, float, double, std::string, bool>;
 
 // Forward declaration of Setting
@@ -37,7 +38,7 @@ using default_setting_type_t = typename default_setting_type<E, SettingVariant,
  * @brief Compile-time check if a type is valid for an enum value based on defaults
  */
 template <typename E, typename SettingVariant, E EnumValue, typename T>
-inline constexpr bool is_valid_type_v = std::is_same_v<default_setting_type_t<E, SettingVariant, EnumValue>, T>;
+inline constexpr bool is_valid_type_v = std::is_same_v<default_setting_type_t<E, SettingVariant, EnumValue>, T>; // NOLINT(readability-identifier-naming)
 
 /**
  * @brief Type trait to associate enum values with their types at compile time
@@ -48,7 +49,7 @@ inline constexpr bool is_valid_type_v = std::is_same_v<default_setting_type_t<E,
  * @tparam Enable SFINAE enabler
  */
 template <typename E, E EnumValue, typename Enable = void>
-struct setting_type_trait {
+struct setting_type_trait { // NOLINT(readability-identifier-naming)
     // Default implementation provides a compile error
     static_assert(sizeof(E) == 0,
                   "You must specialize setting_type_trait for your enum values");
@@ -73,7 +74,7 @@ using setting_type = typename setting_type_trait<E, EnumValue>::type;
  * @tparam T The type to check against
  */
 template <typename E, E EnumValue, typename T>
-inline constexpr bool is_correct_type_v = std::is_same_v<setting_type<E, EnumValue>, T>;
+inline constexpr bool is_correct_type_v = std::is_same_v<setting_type<E, EnumValue>, T>; // NOLINT(readability-identifier-naming)
 
 /**
  * @brief Generic Setting class that is strongly typed