[pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

Author: pre-commit-ci[bot]

.augment/rules/build-all.md

@@ -23,4 +23,4 @@ Build the complete project from scratch and systematically fix all encountered i
 
 8. **Testing**: If applicable, run any existing tests to verify the build produces a functional application
 
-Please provide detailed feedback on each issue encountered and the steps taken to resolve it, so I can track progress and understand the solutions implemented.
+Please provide detailed feedback on each issue encountered and the steps taken to resolve it, so I can track progress and understand the solutions implemented.

.augment/rules/build-examples-fix.md

@@ -14,4 +14,4 @@ Build all examples in the project completely and fix any issues encountered duri
 Please provide a summary of:
 - Which examples were built
 - What issues were encountered and how they were resolved
-- Verification that all functionality is working properly
+- Verification that all functionality is working properly

WARP.md

@@ -23,7 +23,7 @@ atom/<module>/
 The build system enforces a strict dependency hierarchy defined in `cmake/module_dependencies.cmake`:
 
 - **Foundation**: `atom-error` (base, no dependencies)
-- **Core**: `atom-log` → `atom-meta`/`atom-utils` 
+- **Core**: `atom-log` → `atom-meta`/`atom-utils`
 - **Specialized**: `atom-web`, `atom-async`, `atom-system`, etc.
 
 Build order: `atom-error` → `atom-log` → `atom-meta`/`atom-utils` → specialized modules
@@ -158,7 +158,7 @@ Use `atom-log` structured logging instead of `std::cout`
 
 ### Naming Conventions (per STYLE_OF_CODE.md)
 - **Variables/Functions**: camelCase
-- **Classes/Namespaces**: PascalCase 
+- **Classes/Namespaces**: PascalCase
 - **Constants**: UPPER_SNAKE_CASE
 - **Files**: lower_snake_case.[cpp|hpp]
 - **Class members**: m_prefix for private variables

atom/meta/awaitable.hpp

@@ -18,21 +18,21 @@ template <typename F, typename... Args>
 class SimpleAwaitable {
 public:
     using result_type = std::invoke_result_t<F, Args...>;
-    
+
     SimpleAwaitable(F&& func, Args&&... args)
         : func_(std::forward<F>(func)), args_(std::forward<Args>(args)...) {}
-    
+
     bool await_ready() const noexcept { return false; }
-    
+
     void await_suspend(std::coroutine_handle<> handle) const noexcept {
         // For this simple implementation, we just resume immediately
         handle.resume();
     }
-    
+
     result_type await_resume() {
         return std::apply(func_, args_);
     }
-    
+
 private:
     F func_;
     std::tuple<Args...> args_;

cmake/FindDependencies.cmake

@@ -22,7 +22,7 @@ function(atom_find_dependency dep_name)
 
     string(TOUPPER ${dep_name} DEP_UPPER)
     set(found_var "${DEP_UPPER}_FOUND")
-    
+
     # Skip if already found
     if(${found_var})
         return()
@@ -57,9 +57,9 @@ function(atom_find_dependency dep_name)
 
     # Method 3: Manual search for header-only libraries
     if(NOT ${found_var} AND AFD_PATHS)
-        find_path(${DEP_UPPER}_INCLUDE_DIR 
+        find_path(${DEP_UPPER}_INCLUDE_DIR
             NAMES ${AFD_PATHS}
-            PATHS 
+            PATHS
                 /usr/include
                 /usr/local/include
                 /mingw64/include
@@ -95,26 +95,26 @@ endfunction()
 # Function to setup a dependency target
 function(atom_setup_dependency_target dep_name target_name)
     string(TOUPPER ${dep_name} DEP_UPPER)
-    
+
     if(NOT TARGET ${target_name})
         # Create imported target if it doesn't exist
         if(${DEP_UPPER}_FOUND)
             add_library(${target_name} INTERFACE IMPORTED)
-            
+
             # Set include directories
             if(${DEP_UPPER}_INCLUDE_DIRS)
                 target_include_directories(${target_name} INTERFACE ${${DEP_UPPER}_INCLUDE_DIRS})
             elseif(${DEP_UPPER}_INCLUDE_DIR)
                 target_include_directories(${target_name} INTERFACE ${${DEP_UPPER}_INCLUDE_DIR})
             endif()
-            
+
             # Set libraries
             if(${DEP_UPPER}_LIBRARIES)
                 target_link_libraries(${target_name} INTERFACE ${${DEP_UPPER}_LIBRARIES})
             elseif(${dep_name}_LIBRARIES)
                 target_link_libraries(${target_name} INTERFACE ${${dep_name}_LIBRARIES})
             endif()
-            
+
             # Set compile flags
             if(${DEP_UPPER}_CFLAGS_OTHER)
                 target_compile_options(${target_name} INTERFACE ${${DEP_UPPER}_CFLAGS_OTHER})
@@ -130,7 +130,7 @@ endfunction()
 # OpenSSL - Always required
 atom_find_dependency(OpenSSL REQUIRED)
 
-# ZLIB - Always required  
+# ZLIB - Always required
 atom_find_dependency(ZLIB REQUIRED)
 
 # SQLite3 - Core database functionality
@@ -184,19 +184,19 @@ endif()
 
 if(ATOM_USE_BOOST)
     set(BOOST_COMPONENTS)
-    
+
     if(ATOM_USE_BOOST_LOCKFREE)
         list(APPEND BOOST_COMPONENTS atomic thread)
     endif()
-    
+
     if(ATOM_USE_BOOST_GRAPH)
         list(APPEND BOOST_COMPONENTS graph)
     endif()
-    
+
     if(ATOM_USE_BOOST_CONTAINER)
         list(APPEND BOOST_COMPONENTS container)
     endif()
-    
+
     if(BOOST_COMPONENTS)
         atom_find_dependency(Boost QUIET VERSION 1.74 COMPONENTS ${BOOST_COMPONENTS})
     else()

cmake/ModuleDependencies.cmake

@@ -9,7 +9,7 @@ function(atom_setup_module_dependencies module_name)
     cmake_parse_arguments(AMD "${options}" "${oneValueArgs}" "${multiValueArgs}" ${ARGN})
 
     string(TOUPPER ${module_name} MODULE_UPPER)
-    
+
     # Link required dependencies
     foreach(dep ${AMD_REQUIRED_DEPS})
         if(TARGET ${dep})
@@ -18,7 +18,7 @@ function(atom_setup_module_dependencies module_name)
             message(WARNING "Required dependency ${dep} not found for module ${module_name}")
         endif()
     endforeach()
-    
+
     # Link optional dependencies
     foreach(dep ${AMD_OPTIONAL_DEPS})
         if(TARGET ${dep})
@@ -28,7 +28,7 @@ function(atom_setup_module_dependencies module_name)
             message(STATUS "Optional dependency ${dep} not available for ${module_name}")
         endif()
     endforeach()
-    
+
     # Link system libraries
     foreach(lib ${AMD_SYSTEM_LIBS})
         target_link_libraries(${module_name} PUBLIC ${lib})
@@ -41,11 +41,11 @@ function(atom_setup_standard_dependencies module_name)
     if(TARGET atom-error)
         target_link_libraries(${module_name} PUBLIC atom-error)
     endif()
-    
+
     # Most modules need threading
     find_package(Threads REQUIRED)
     target_link_libraries(${module_name} PUBLIC Threads::Threads)
-    
+
     # Platform-specific libraries
     if(WIN32)
         # Windows-specific libraries that many modules need
@@ -67,15 +67,15 @@ endfunction()
 # Function to setup logging dependencies
 function(atom_setup_logging_deps module_name)
     # Try to find loguru
-    find_library(LOGURU_LIBRARY 
+    find_library(LOGURU_LIBRARY
         NAMES loguru
-        PATHS 
+        PATHS
             /usr/lib
             /usr/local/lib
             /mingw64/lib
             ${CMAKE_PREFIX_PATH}/lib
     )
-    
+
     if(LOGURU_LIBRARY)
         target_link_libraries(${module_name} PUBLIC ${LOGURU_LIBRARY})
         message(STATUS "Loguru linked to ${module_name}")
@@ -96,7 +96,7 @@ function(atom_setup_xml_deps module_name)
             return()
         endif()
     endif()
-    
+
     # Fallback to find_package
     find_package(tinyxml2 QUIET)
     if(tinyxml2_FOUND)
@@ -112,7 +112,7 @@ function(atom_setup_compression_deps module_name)
     if(ZLIB_FOUND)
         target_link_libraries(${module_name} PUBLIC ZLIB::ZLIB)
     endif()
-    
+
     # Try to find additional compression libraries
     find_package(PkgConfig QUIET)
     if(PkgConfig_FOUND)
@@ -142,12 +142,12 @@ function(atom_setup_networking_deps module_name)
         target_compile_definitions(${module_name} PRIVATE ASIO_STANDALONE)
         message(STATUS "Asio linked to ${module_name}")
     endif()
-    
+
     if(OpenSSL_FOUND)
         target_link_libraries(${module_name} PUBLIC OpenSSL::SSL OpenSSL::Crypto)
         message(STATUS "OpenSSL linked to ${module_name}")
     endif()
-    
+
     # Platform-specific networking libraries
     if(WIN32)
         target_link_libraries(${module_name} PUBLIC ws2_32 wsock32 iphlpapi)
@@ -194,20 +194,20 @@ endfunction()
 # Macro to simplify common module setup
 macro(atom_configure_module module_name)
     atom_setup_standard_dependencies(${module_name})
-    
+
     # Set common compile features
     target_compile_features(${module_name} PUBLIC cxx_std_20)
-    
+
     # Set common compile options
     if(MSVC)
         target_compile_options(${module_name} PRIVATE /W4)
     else()
         target_compile_options(${module_name} PRIVATE -Wall -Wextra -Wpedantic)
     endif()
-    
+
     # Set common include directories
-    target_include_directories(${module_name} 
-        PUBLIC 
+    target_include_directories(${module_name}
+        PUBLIC
             $<BUILD_INTERFACE:${CMAKE_CURRENT_SOURCE_DIR}>
             $<INSTALL_INTERFACE:include>
     )

docs/ARCHITECTURAL_IMPROVEMENTS.md

@@ -179,7 +179,7 @@ atom_setup_networking_deps(${module_name})
 
 ### Core Dependencies ✅
 - **OpenSSL**: Cryptographic operations - Working
-- **ZLIB**: Compression - Working  
+- **ZLIB**: Compression - Working
 - **SQLite3**: Database operations - Working
 - **fmt**: String formatting - Working
 - **Asio**: Networking (standalone) - Working

docs/ARCHITECTURE_ANALYSIS.md

@@ -94,7 +94,7 @@ atom-system (system operations)
 
 #### Missing Example Files
 - **Issue**: Multiple example files referenced but not present
-- **Locations**: 
+- **Locations**:
   - `example/image/io/` (4 missing files)
   - `example/image/processing/` (4 missing files)
   - `example/image/formats/` (5 missing files)
@@ -132,7 +132,7 @@ atom-system (system operations)
 
 #### Inconsistent Package Finding
 - **Issue**: Different modules use different approaches for finding dependencies
-- **Examples**: 
+- **Examples**:
   - Some use `find_package()`
   - Others use `pkg_check_modules()`
   - Inconsistent error handling

example/components/type_conversion_example.cpp

@@ -32,14 +32,14 @@ class TypeConverter {
         static TypeConverter instance;
         return instance;
     }
-    
+
     struct Statistics {
         uint64_t totalConversions = 0;
         uint64_t successfulConversions = 0;
         uint64_t failedConversions = 0;
         uint64_t registeredConverters = 0;
     };
-    
+
     Statistics getStatistics() const {
         return Statistics{};
     }
@@ -68,14 +68,14 @@ struct PlayerData {
         return id == other.id && name == other.name && score == other.score &&
                active == other.active;
     }
-    
+
     bool operator<(const PlayerData& other) const {
         if (id != other.id) return id < other.id;
         if (name != other.name) return name < other.name;
         if (score != other.score) return score < other.score;
         return active < other.active;
     }
-    
+
     bool operator>(const PlayerData& other) const {
         return other < *this;
     }
@@ -103,12 +103,12 @@ class Point2D {
         return std::abs(x_ - other.x_) < epsilon &&
                std::abs(y_ - other.y_) < epsilon;
     }
-    
+
     bool operator<(const Point2D& other) const {
         if (x_ != other.x_) return x_ < other.x_;
         return y_ < other.y_;
     }
-    
+
     bool operator>(const Point2D& other) const {
         return other < *this;
     }
@@ -135,18 +135,18 @@ int main() {
     try {
         // Create a simple component to demonstrate basic functionality
         auto component = std::make_shared<TypeConversionComponent>("TypeConversionDemo");
-        
+
         std::cout << "\n1. Component created successfully" << std::endl;
         std::cout << "2. TypeConverter stub is functional" << std::endl;
         std::cout << "3. Custom types have comparison operators" << std::endl;
-        
+
         // Test basic functionality
         PlayerData player(1, "TestPlayer", 100.0, true);
         Point2D point(3.14, 2.71);
-        
+
         std::cout << "4. PlayerData: " << player.toString() << std::endl;
         std::cout << "5. Point2D: " << point.toString() << std::endl;
-        
+
         // Test TypeConverter stub
         auto& converter = TypeConverter::instance();
         auto stats = converter.getStatistics();

example/extra/asio/mqtt_client.cpp

@@ -79,7 +79,7 @@ class Client {
     void async_connect(const std::string& host, uint16_t port, const ConnectionOptions& options) {
         std::cout << "Connecting to " << host << ":" << port << " (stub)" << std::endl;
         std::cout << "  Client ID: " << options.client_id << std::endl;
-        
+
         // Simulate connection
         std::thread([this]() {
             std::this_thread::sleep_for(100ms);
@@ -91,7 +91,7 @@ class Client {
 
     void async_subscribe(const std::string& topic, QoS qos) {
         std::cout << "Subscribing to topic: " << topic << " with QoS " << static_cast<int>(qos) << " (stub)" << std::endl;
-        
+
         // Simulate receiving messages
         std::thread([this, topic]() {
             std::this_thread::sleep_for(200ms);