EngineSquared
diff --git a/.clang-tidy b/.clang-tidy
@@ -0,0 +1,26 @@
+---
+
+# bugprone-exception-escape, bugprone-throwing-static-initialization, bugprone-easily-swappable-parameters should be removed but require to modify a lot of stuff in the project
+Checks: "-*, bugprone-*, -bugprone-reserved-identifier,
+  -bugprone-exception-escape, -bugprone-easily-swappable-parameters,
+  -bugprone-throwing-static-initialization"
+
+WarningsAsErrors: "*"
+HeaderFileExtensions:
+  - ""
+  - h
+  - hh
+  - hpp
+  - hxx
+  - ipp
+ImplementationFileExtensions:
+  - c
+  - cc
+  - cpp
+  - cxx
+HeaderFilterRegex: "^src/(?!.*/webgpu\\.hpp$).*"
+FormatStyle: file
+User: miouzora
+CheckOptions: {}
+SystemHeaders: false
+...
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -4,19 +4,23 @@ on:
   push:
   pull_request:
   release:
-    types: [published]
+    types: [ published ]
 
 concurrency:
-  group: ${{ github.workflow }}-${{ github.ref }}-${{ github.base_ref }}-${{ github.head_ref }}
+  group: ${{ github.workflow }}-${{ github.ref }}-${{ github.base_ref }}-${{
+    github.head_ref }}
   cancel-in-progress: true
 
 env:
-  UNWANTED_REGEX: '^(?!.*tests\/).*gc(no|da|ov)$|(.*\.(a|o|so|lib))$|(.*~)$|^(#.*#)$|^tmp\/.*|.*\/tmp\/.*'
+  UNWANTED_REGEX: "^(?!.*tests\\/).*gc(no|da|ov)$|(.*\\.(a|o|so|lib))$|(.*~)$|^(#\
+    .*#)$|^tmp\\/.*|.*\\/tmp\\/.*"
 
 jobs:
   check_repository_cleanliness:
-    if: ${{ github.event_name != 'pull_request' || (!startsWith(github.head_ref, 'ga-ignore-') && github.head_ref != 'gh-pages') }}
-    name: Checks if the repository is clean and void of any unwanted files (temp files, binary files, etc.)
+    if: ${{ github.event_name != 'pull_request' || (!startsWith(github.head_ref,
+      'ga-ignore-') && github.head_ref != 'gh-pages') }}
+    name: Checks if the repository is clean and void of any unwanted files (temp
+      files, binary files, etc.)
     runs-on: ubuntu-latest
     steps:
       - name: Checkout repository
@@ -66,9 +70,37 @@ jobs:
         run: |
           xmake format -c
 
+  tidy:
+    name: Run clang-tidy
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v6
+        with:
+          fetch-depth: 0
+
+      - name: Install clang-tidy
+        run: |
+          sudo apt update
+          wget https://apt.llvm.org/llvm.sh
+          chmod +x llvm.sh
+          sudo ./llvm.sh 22
+          sudo apt install -y clang-tidy
+
+      - name: Install xmake
+        uses: xmake-io/github-action-setup-xmake@fadadea1162ec75ce1541d5bb68226fb147c221e
+        with:
+          xmake-version: latest
+          actions-cache-folder: ".xmake-cache"
+          actions-cache-key: "clang-tidy"
+          build-cache: true
+          build-cache-key: "clang-tidy"
+      - name: Run clang-tidy
+        run: |
+          xmake check -y clang.tidy --configfile=./.clang-tidy
+
   check_program_compilation:
     name: Build and verify binaries
-    needs: [check_repository_cleanliness, lint_code]
+    needs: [ check_repository_cleanliness, lint_code ]
     strategy:
       matrix:
         include:
@@ -105,7 +137,7 @@ jobs:
 
   run_tests:
     name: Run tests
-    needs: [check_repository_cleanliness, lint_code]
+    needs: [ check_repository_cleanliness, lint_code ]
     strategy:
       matrix:
         include:
@@ -163,7 +195,7 @@ jobs:
 
   check_examples:
     name: Build and test examples
-    needs: [check_repository_cleanliness, lint_code]
+    needs: [ check_repository_cleanliness, lint_code ]
     strategy:
       matrix:
         include:

diff --git a/src/engine/src/Id.hpp b/src/engine/src/Id.hpp
@@ -19,12 +19,6 @@ template <typename TDerived, typename TValue> struct BasicId {
     /// @brief The underlying value of the ID.
     TValue value;
 
-    /// @brief Constructor for BasicId. Allows implicit construction from the underlying value type, defaulting to the
-    ///     null value.
-    /// @param value_ The value to initialize the ID with. Defaults to the null value defined by the derived class.
-    /// @return An instance of the derived ID type initialized with the given value.
-    constexpr explicit(false) BasicId(TValue value_ = TDerived::NullValue());
-
     /// @brief Get the null value for this ID type. This should be defined by the derived class to specify what
     ///     constitutes a null/invalid ID.
     /// @return The value representing a null/invalid ID for this type.
@@ -33,6 +27,14 @@ template <typename TDerived, typename TValue> struct BasicId {
     /// @brief Checks if the ID is null/invalid by comparing its value to the null value defined by the derived class.
     /// @return True if the ID is null/invalid, false otherwise.
     constexpr bool IsNull(void) const;
+
+  private:
+    /// @brief Constructor for BasicId. Allows implicit construction from the underlying value type, defaulting to the
+    ///     null value.
+    /// @param value_ The value to initialize the ID with. Defaults to the null value defined by the derived class.
+    /// @return An instance of the derived ID type initialized with the given value.
+    constexpr explicit(false) BasicId(TValue value_ = TDerived::NullValue());
+    friend TDerived;
 };
 
 /// @struct Id

diff --git a/src/engine/src/entity/EntityId.hpp b/src/engine/src/entity/EntityId.hpp
@@ -25,10 +25,6 @@ struct EntityId : Id {
     /// @param id The Engine::Id to convert.
     constexpr EntityId(Id id);
 
-    /// @brief Implicit conversion operator to the underlying ValueType. This allows for seamless use of EntityId where
-    ///     a ValueType is expected.
-    constexpr operator ValueType() const;
-
     /// @brief Returns a null EntityId. A null EntityId is an EntityId that does not correspond to any valid entity in
     ///     the registry.
     /// @return A null EntityId.

diff --git a/src/engine/src/entity/EntityId.ipp b/src/engine/src/entity/EntityId.ipp
@@ -5,8 +5,6 @@ constexpr EntityId::EntityId(ValueType v) : Id{v} {}
 
 constexpr EntityId::EntityId(Id id) : Id{id} {}
 
-constexpr EntityId::operator ValueType() const { return value; }
-
 constexpr EntityId EntityId::Null() { return EntityId{NullValue()}; }
 
 inline bool EntityId::IsValid(const Engine::Core &core) const { return core.IsEntityValid(value); }

diff --git a/src/engine/src/scheduler/FixedTimeUpdate.cpp b/src/engine/src/scheduler/FixedTimeUpdate.cpp
@@ -15,7 +15,7 @@ void Engine::Scheduler::FixedTimeUpdate::RunSystems()
 {
     _bufferedTime += this->_core.GetResource<Engine::Resource::Time>()._elapsedTime;
     auto ticks = static_cast<unsigned int>(_bufferedTime / _tickRate);
-    _bufferedTime -= ticks * _tickRate;
+    _bufferedTime -= static_cast<float>(ticks) * _tickRate;
 
     for (unsigned int i = 0; i < ticks; i++)
     {

diff --git a/src/engine/src/scheduler/RelativeTimeUpdate.cpp b/src/engine/src/scheduler/RelativeTimeUpdate.cpp
@@ -18,8 +18,8 @@ void Engine::Scheduler::RelativeTimeUpdate::RunSystems()
 {
     _bufferedTime += this->_core.GetResource<Engine::Resource::Time>()._elapsedTime;
     auto ticks = static_cast<unsigned int>(_bufferedTime / _tickRate);
-    float remainder = _bufferedTime - ticks * _tickRate;
-    _bufferedTime -= ticks * _tickRate;
+    float remainder = _bufferedTime - static_cast<float>(ticks) * _tickRate;
+    _bufferedTime -= static_cast<float>(ticks) * _tickRate;
 
     for (unsigned int i = 0; i < ticks; i++)
     {

diff --git a/src/plugin/default-pipeline/src/system/GPUComponentManagement/OnMaterialCreation.cpp b/src/plugin/default-pipeline/src/system/GPUComponentManagement/OnMaterialCreation.cpp
@@ -22,6 +22,13 @@ void DefaultPipeline::System::OnMaterialCreation(Engine::Core &core, Engine::Ent
     auto &textureContainer = core.GetResource<Graphic::Resource::TextureContainer>();
     auto &samplerContainer = core.GetResource<Graphic::Resource::SamplerContainer>();
     const auto &context = core.GetResource<Graphic::Resource::Context>();
+    const auto &device = context.deviceContext.GetDevice();
+
+    if (!device.has_value())
+    {
+        Log::Error("DefaultPipeline::System::OnMaterialCreation: device is empty");
+        return;
+    }
 
     auto &GPUMaterial = entity.AddComponent<Component::GPUMaterial>();
 
@@ -46,7 +53,7 @@ void DefaultPipeline::System::OnMaterialCreation(Engine::Core &core, Engine::Ent
 
     if (!samplerContainer.Contains(samplerId))
     {
-        Graphic::Resource::Sampler sampler{context.deviceContext.GetDevice().value()};
+        Graphic::Resource::Sampler sampler{device.value()};
         samplerContainer.Add(samplerId, std::move(sampler));
     }
     GPUMaterial.sampler = samplerId;

diff --git a/src/plugin/default-pipeline/src/system/initialization/CreateLights.cpp b/src/plugin/default-pipeline/src/system/initialization/CreateLights.cpp
@@ -14,6 +14,13 @@ static void CreateDirectionalLightsShadowTextures(Engine::Core &core)
     auto &textureContainer = core.GetResource<Graphic::Resource::TextureContainer>();
     auto &samplerContainer = core.GetResource<Graphic::Resource::SamplerContainer>();
     const auto &context = core.GetResource<Graphic::Resource::Context>();
+    const auto &device = context.deviceContext.GetDevice();
+
+    if (!device.has_value())
+    {
+        Log::Error("CreateLights::CreateDirectionalLightsShadowTextures: device is empty");
+        return;
+    }
 
     wgpu::TextureDescriptor textureDescriptor(wgpu::Default);
     textureDescriptor.label = wgpu::StringView("DIRECTIONAL_LIGHTS_SHADOW_TEXTURE");
@@ -34,7 +41,7 @@ static void CreateDirectionalLightsShadowTextures(Engine::Core &core)
     samplerDescriptor.addressModeV = wgpu::AddressMode::ClampToEdge;
     samplerDescriptor.minFilter = wgpu::FilterMode::Linear;
     samplerDescriptor.magFilter = wgpu::FilterMode::Linear;
-    Graphic::Resource::Sampler shadowSampler(context.deviceContext.GetDevice().value(), samplerDescriptor);
+    Graphic::Resource::Sampler shadowSampler(device.value(), samplerDescriptor);
     samplerContainer.Add(Utils::DIRECTIONAL_LIGHTS_SHADOW_SAMPLER_ID, std::move(shadowSampler));
 }
 

diff --git a/src/plugin/graphic/src/system/initialization/ConfigureSurface.cpp b/src/plugin/graphic/src/system/initialization/ConfigureSurface.cpp
@@ -24,9 +24,16 @@ void Graphic::System::ConfigureSurface(Engine::Core &core)
     if (config.height == 0)
         config.height = 1;
 
+    auto &optDevice = context.deviceContext.GetDevice();
+    if (!context.surface.has_value() || !context.surface.value().capabilities.has_value() || !optDevice.has_value() ||
+        !context.surface.value().value.has_value())
+    {
+        return;
+    }
+
     config.usage = wgpu::TextureUsage::RenderAttachment;
     config.format = context.surface->capabilities->formats[0];
-    config.device = context.deviceContext.GetDevice().value();
+    config.device = optDevice.value();
     config.presentMode = wgpu::PresentMode::Fifo;
     config.alphaMode = wgpu::CompositeAlphaMode::Auto;
 

diff --git a/src/plugin/graphic/src/system/initialization/CreateAdapter.cpp b/src/plugin/graphic/src/system/initialization/CreateAdapter.cpp
@@ -21,6 +21,8 @@ static std::optional<wgpu::BackendType> getBackendType(const wgpu::Adapter &adap
 
 static std::optional<wgpu::Adapter> findVulkanAdapter(const Graphic::Resource::Context &context)
 {
+    if (!context.instance.has_value())
+        return std::nullopt;
     wgpu::InstanceEnumerateAdapterOptions enumOpts(wgpu::Default);
     size_t count = context.instance->enumerateAdapters(enumOpts, nullptr);
     if (count == 0)
@@ -67,6 +69,12 @@ void Graphic::System::CreateAdapter(Engine::Core &core)
     auto &context = core.GetResource<Resource::Context>();
     const auto &graphicSettings = core.GetResource<Resource::GraphicSettings>();
 
+    if (!context.instance.has_value())
+    {
+        Log::Error("Graphic::System::CreateAdapter: context.instance has no value");
+        return;
+    }
+
     wgpu::RequestAdapterOptions adapterOpts(wgpu::Default);
 
     if (graphicSettings.GetPowerPreference() == Resource::PowerPreference::LowPower)
@@ -78,7 +86,7 @@ void Graphic::System::CreateAdapter(Engine::Core &core)
         adapterOpts.powerPreference = wgpu::PowerPreference::HighPerformance;
     }
 
-    if (context.surface.has_value())
+    if (context.surface.has_value() && context.surface->value.has_value())
         adapterOpts.compatibleSurface = context.surface->value.value();
 
     wgpu::Adapter adapter = context.instance->requestAdapter(adapterOpts);

diff --git a/src/plugin/graphic/src/system/initialization/CreateDefaultSampler.cpp b/src/plugin/graphic/src/system/initialization/CreateDefaultSampler.cpp
@@ -22,8 +22,13 @@ void CreateDefaultSampler(Engine::Core &core)
     samplerDesc.addressModeV = wgpu::AddressMode::ClampToEdge;
     samplerDesc.addressModeW = wgpu::AddressMode::ClampToEdge;
 
-    Resource::Sampler defaultSampler(context.deviceContext.GetDevice().value(), samplerDesc);
-
+    const auto &optDevice = context.deviceContext.GetDevice();
+    if (!optDevice.has_value())
+    {
+        Log::Error("Graphic::System::CreateDefaultSampler: Graphic device not found");
+        return;
+    }
+    Resource::Sampler defaultSampler(optDevice.value(), samplerDesc);
     samplerContainer.Add(Utils::DEFAULT_SAMPLER_ID, std::move(defaultSampler));
 }
 } // namespace Graphic::System
diff --git a/src/plugin/graphic/src/system/initialization/CreateDevice.cpp b/src/plugin/graphic/src/system/initialization/CreateDevice.cpp
@@ -12,7 +12,6 @@ static void SetupDeviceDescriptor(wgpu::DeviceDescriptor &deviceDesc, Graphic::R
     deviceDesc.requiredLimits = dynamic_cast<wgpu::Limits *>(&settings.GetWantedLimits());
     deviceDesc.defaultQueue.nextInChain = nullptr;
     deviceDesc.defaultQueue.label = wgpu::StringView("The default queue");
-    deviceDesc.deviceLostCallbackInfo = {};
     deviceDesc.deviceLostCallbackInfo.nextInChain = nullptr;
     deviceDesc.deviceLostCallbackInfo.mode = wgpu::CallbackMode::AllowProcessEvents;
     deviceDesc.deviceLostCallbackInfo.callback = [](WGPUDevice const *device, WGPUDeviceLostReason reason,
@@ -21,7 +20,6 @@ static void SetupDeviceDescriptor(wgpu::DeviceDescriptor &deviceDesc, Graphic::R
         Log::Error(fmt::format("Device lost: reason {:x} ({})", static_cast<uint32_t>(reason),
                                std::string(message.data, message.length)));
     };
-    deviceDesc.uncapturedErrorCallbackInfo = {};
     deviceDesc.uncapturedErrorCallbackInfo.nextInChain = nullptr;
     deviceDesc.uncapturedErrorCallbackInfo.callback = settings.GetOnErrorCallback();
 }

diff --git a/src/plugin/graphic/src/system/initialization/CreateQueue.cpp b/src/plugin/graphic/src/system/initialization/CreateQueue.cpp
@@ -8,8 +8,12 @@ void Graphic::System::CreateQueue(Engine::Core &core)
 {
     auto &context = core.GetResource<Resource::Context>();
     const auto &settings = core.GetResource<Resource::GraphicSettings>();
+    auto &device = context.deviceContext.GetDevice();
 
-    auto queue = context.deviceContext.GetDevice()->getQueue();
+    if (!device.has_value())
+        return;
+
+    auto queue = device->getQueue();
 
     if (queue == nullptr)
         throw Exception::QueueCreationError("Could not create WebGPU queue");

diff --git a/src/plugin/graphic/src/system/initialization/CreateSurface.cpp b/src/plugin/graphic/src/system/initialization/CreateSurface.cpp
@@ -10,16 +10,21 @@ void Graphic::System::CreateSurface(Engine::Core &core)
 {
     auto &context = core.GetResource<Resource::Context>();
     auto &graphicSettings = core.GetResource<Resource::GraphicSettings>();
-    auto &instance = context.instance.value();
+    auto &instance = context.instance;
 
+    if (!instance.has_value())
+    {
+        Log::Error("Graphic::System::CreateSurface: context.instance has no value");
+        return;
+    }
     wgpu::Surface surface = nullptr;
 
     if (graphicSettings.GetWindowSystem() == Resource::WindowSystem::None)
         return;
     else if (graphicSettings.GetWindowSystem() == Resource::WindowSystem::GLFW)
     {
         auto glfwWindow = core.GetResource<Window::Resource::Window>().GetGLFWWindow();
-        surface = glfwCreateWindowWGPUSurface(instance, glfwWindow);
+        surface = glfwCreateWindowWGPUSurface(instance.value(), glfwWindow);
     }
     else
     {

diff --git a/src/plugin/graphic/src/system/initialization/ReleaseAdapter.cpp b/src/plugin/graphic/src/system/initialization/ReleaseAdapter.cpp
@@ -6,6 +6,9 @@
 void Graphic::System::ReleaseAdapter(Engine::Core &core)
 {
     auto &context = core.GetResource<Resource::Context>();
-    context.adapter->release();
-    context.adapter.reset();
+    if (context.adapter.has_value())
+    {
+        context.adapter->release();
+        context.adapter.reset();
+    }
 }
diff --git a/src/plugin/graphic/src/system/initialization/RequestCapabilities.cpp b/src/plugin/graphic/src/system/initialization/RequestCapabilities.cpp
@@ -8,7 +8,7 @@ void RequestCapabilities(Engine::Core &core)
 {
     auto &context = core.GetResource<Resource::Context>();
 
-    if (context.surface == std::nullopt)
+    if (context.surface == std::nullopt || !context.adapter.has_value())
         return;
     if (context.surface->updateCapabilities(context.adapter.value()) == wgpu::Status::Error)
     {

diff --git a/src/plugin/graphic/src/system/initialization/SetupQueue.cpp b/src/plugin/graphic/src/system/initialization/SetupQueue.cpp
@@ -11,6 +11,8 @@ void Graphic::System::SetupQueue(Engine::Core &core)
         return;
 
     auto &context = core.GetResource<Resource::Context>();
+    if (!context.queue.has_value())
+        return;
 
     auto onQueueWorkDone = [](WGPUQueueWorkDoneStatus status, [[maybe_unused]] WGPU_NULLABLE void *userdata1,
                               [[maybe_unused]] WGPU_NULLABLE void *userdata2) {

diff --git a/src/plugin/graphic/src/system/presentation/Present.cpp b/src/plugin/graphic/src/system/presentation/Present.cpp
@@ -7,7 +7,7 @@ void Graphic::System::Present(Engine::Core &core)
 {
     auto &context = core.GetResource<Resource::Context>();
 
-    if (!context.surface.has_value() || !context.surface->configured)
+    if (!context.surface.has_value() || !context.surface->value.has_value() || !context.surface->configured)
         return;
 
     context.surface->value->present();