Major project refactor

Author: ramsafin

.clang-format

@@ -1,12 +1,37 @@
+# .clang-format — Based on the Google C++ Style Guide
+
 BasedOnStyle: Google
+
+# Indentation & wrapping
 IndentWidth: 2
+TabWidth: 2
+UseTab: Never
 ColumnLimit: 120
-AccessModifierOffset: -2
-AllowShortFunctionsOnASingleLine: Inline
-SortIncludes: true
-IncludeBlocks: Preserve
-PointerAlignment: Left
-DerivePointerAlignment: false
+
+# Brace and paren placement
+BreakBeforeBraces: Attach
 SpaceBeforeParens: ControlStatements
-BreakBeforeBraces: Allman  # Or 'Attach' if you prefer brace on same line
+
+# Pointer alignment
+PointerAlignment: Right
+DerivePointerAlignment: false
+
+# Includes
+SortIncludes: true
+IncludeBlocks: Regroup
+
+# Alignment tweaks
+AlignConsecutiveDeclarations: true
+AlignConsecutiveAssignments:  true
+
+# Disable column-alignment of trailing comments
+AlignTrailingComments: false
+
+# Allow short inline functions on one line
+AllowShortFunctionsOnASingleLine: Inline
+
+# Trailing comments
 SpacesBeforeTrailingComments: 2
+
+# Maximum empty lines
+MaxEmptyLinesToKeep: 1

.clang-tidy

@@ -1,26 +1,47 @@
 Checks: >
+  google-*,
   bugprone-*,
   clang-analyzer-*,
-  modernize-*,
+  clang-analyzer-security.*,
+  cppcoreguidelines-*,
   performance-*,
   readability-*,
-  cppcoreguidelines-*,
+  modernize-*,
+  -google-build-using-namespace,
+  -google-readability-function-size,
   -modernize-use-trailing-return-type,
   -cppcoreguidelines-pro-type-vararg,
   -cppcoreguidelines-owning-memory
 
 UseColor: true
-WarningsAsErrors: '*'
-HeaderFilterRegex: '^(include|src|tests|benchmarks)/'
 FormatStyle: file
+WarningsAsErrors: '*'
+
 CheckOptions:
-  - key: modernize-use-nullptr.NullMacros
-    value: 'NULL'
+  # Naming conventions (match .clang-format indent/specs)
   - key: readability-identifier-naming.VariableCase
+    value: lower_case
+  - key: readability-identifier-naming.FunctionCase
     value: camelBack
   - key: readability-identifier-naming.ClassCase
     value: CamelCase
   - key: readability-identifier-naming.ConstantCase
     value: UPPER_CASE
-  - key: readability-identifier-naming.ParameterCase
-    value: camelBack
+  - key: readability-identifier-naming.MemberCase
+    value: lower_case
+  - key: readability-identifier-naming.PrivateMemberPrefix
+    value: "m_"
+
+  # Modernize
+  - key: modernize-use-nullptr.NullMacros
+    value: 'NULL'
+  - key: modernize-use-using.CheckAliases
+    value: 'true'
+
+  # Performance
+  - key: performance-unnecessary-value-param.AllowedTypes
+    value: 'std::string,std::vector'
+
+  # Readability
+  - key: readability-braces-around-statements.ShortStatementLines
+    value: '1'

.vscode/extensions.json

@@ -7,4 +7,4 @@
     "matepek.vscode-catch2-test-adapter",
     "llvm-vs-code-extensions.vscode-clangd",
   ]
-}
+}

.vscode/settings.json

@@ -31,4 +31,4 @@
     "editor.defaultFormatter": "llvm-vs-code-extensions.vscode-clangd",
     "editor.formatOnSave": true,
   }
-}
+}

CMakeLists.txt

@@ -1,76 +1,62 @@
 cmake_minimum_required(VERSION 3.23 FATAL_ERROR)
-project(modern_cpp_project LANGUAGES CXX VERSION 0.1.0)
+project(modern_cpp LANGUAGES CXX VERSION 0.1.0)
 
-# === C++ Standard ===
+list(APPEND CMAKE_MODULE_PATH "${CMAKE_SOURCE_DIR}/cmake")
+
+# === Project Configuration ===
 set(CMAKE_CXX_STANDARD 20)
+set(CMAKE_CXX_EXTENSIONS OFF)
 set(CMAKE_CXX_STANDARD_REQUIRED ON)
 
-# === Compile Commands ===
-set(CMAKE_EXPORT_COMPILE_COMMANDS ON)
-
-# === Custom Build Types ===
-if(CMAKE_CONFIGURATION_TYPES)
-  foreach(build_type IN ITEMS Sanitize Coverage)
-    list(APPEND CMAKE_CONFIGURATION_TYPES ${build_type})
-  endforeach()
-  list(REMOVE_DUPLICATES CMAKE_CONFIGURATION_TYPES)
-  set(CMAKE_CONFIGURATION_TYPES "${CMAKE_CONFIGURATION_TYPES}" CACHE STRING "" FORCE)
-endif()
+# === Options ===
+include(Options)
 
-# === Project Options ===
-option(ENABLE_WARNINGS          "Enable compiler warnings"          ON)
-option(ENABLE_STRICT_WARNINGS   "Treat warnings as errors"          ON)
+# === Flags ===
+include(CompilerFlags)
 
-option(ENABLE_SANITIZERS        "Enable sanitizer instrumentation"  OFF)
+# === Search All Source Files ===
+file(GLOB_RECURSE CORE_SOURCES
+  CONFIGURE_DEPENDS
+  src/*.cpp
+  src/*.hpp
+  include/*.hpp
+)
 
-option(ENABLE_TESTING           "Build unit tests"                  ON)
-option(ENABLE_BENCHMARKS        "Build benchmarks"                  ON)
-option(ENABLE_COVERAGE          "Enable coverage instrumentation"   OFF)
+set(ALL_SOURCE_FILES ${CORE_SOURCES})
 
-option(BUILD_APP                "Build the demo app"                ON)
-option(BUILD_DOCS               "Build Doxygen documentation"       ON)
-
-option(ENABLE_CPPCHECK          "Enable cppcheck analysis target"   ON)
-option(ENABLE_CLANG_TIDY        "Enable clang-tidy analysis target" ON)
-option(ENABLE_CLANG_FORMAT      "Enable clang-format target"        ON)
-
-# === Normalize Build Type (e.g., for string comparison) ===
-if(CMAKE_BUILD_TYPE)
-  string(TOLOWER "${CMAKE_BUILD_TYPE}" BUILD_TYPE_LOWER)
-else()
-  set(BUILD_TYPE_LOWER "release")
+if(ENABLE_TESTING)
+  file(GLOB_RECURSE TEST_SOURCES
+    CONFIGURE_DEPENDS
+    tests/*.cpp
+  )
+  list(APPEND ALL_SOURCE_FILES ${TEST_SOURCES})
 endif()
 
-# === Auto-toggle Options for Special Build Types ===
-if(BUILD_TYPE_LOWER STREQUAL "sanitize")
-  set(ENABLE_SANITIZERS ON CACHE BOOL "Enable sanitizers" FORCE)
+if(ENABLE_BENCHMARKS)
+  file(GLOB_RECURSE BENCH_SOURCES
+    CONFIGURE_DEPENDS
+    benchmarks/*.cpp
+  )
+  list(APPEND ALL_SOURCE_FILES ${BENCH_SOURCES})
 endif()
 
-if(BUILD_TYPE_LOWER STREQUAL "coverage")
-  set(ENABLE_COVERAGE ON CACHE BOOL "Enable coverage" FORCE)
-  set(BUILD_APP OFF CACHE BOOL "Disable building app in coverage builds" FORCE)
-  set(ENABLE_BENCHMARKS OFF CACHE BOOL "Disable benchmarks in coverage builds" FORCE)
+if(BUILD_APP)
+  file(GLOB_RECURSE APP_SOURCES
+    CONFIGURE_DEPENDS
+    app/*.cpp
+  )
+  list(APPEND ALL_SOURCE_FILES ${APP_SOURCES})
 endif()
 
-# === CMake Module Path ===
-list(APPEND CMAKE_MODULE_PATH "${CMAKE_SOURCE_DIR}/cmake")
-
-# === Core Configuration ===
-include(options/Warnings)
-include(options/Sanitizers)
+# === Helpers ===
+include(Helpers)
 
-# === Optional Tooling ===
-include(tools/Coverage)
-include(tools/Cppcheck)
-include(tools/ClangTidy)
-include(tools/ClangFormat)
+# === Core Library ===
+add_subdirectory(src)
 
-# === Core library ===
-add_subdirectory(src)  # modern_cpp_project::math
-
-# === Application ===
+# === App ===
 if(BUILD_APP)
-  add_subdirectory(app)  # modern_cpp_project_app
+  add_subdirectory(app)
 endif()
 
 # === Unit Tests ===
@@ -84,8 +70,14 @@ if(ENABLE_BENCHMARKS)
   add_subdirectory(benchmarks)
 endif()
 
-# === Documentation ===
-include(tools/Doxygen)
+# === Developer Tooling (formatters, linters, etc.) ===
+include(Tooling)
+
+# === Coverage ===
+include(Coverage)
 
-# === Installation ===
+# === Installation & Export ===
 include(install/InstallConfig)
+
+# === Packaging ===
+include(CPackConfig)