Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@src/engine/src/entity/EntityId.hpp`:
- Around line 113-116: The shared map temporaryComponent must be protected
against concurrent access: add an inline static mutex (e.g.,
temporaryComponentMutex) alongside temporaryComponent and take a lock
(std::lock_guard or std::unique_lock) whenever AddTemporaryComponent and
RemoveTemporaryComponents access or mutate the map (and any other access sites);
use a shared_mutex if you want concurrent reads and exclusive writes, and ensure
all reads are also properly synchronized to prevent data races and corruption.

In `@src/engine/src/entity/EntityId.ipp`:
- Line 12: The three non-template out-of-class definitions need the inline
keyword to avoid ODR violations when EntityId.ipp is included in multiple
translation units: mark bool EntityId::IsValid(const Engine::Core &core) const,
void EntityId::RemoveTemporaryComponents(Core &core), and std::size_t
std::hash<Engine::EntityId>::operator()(const Engine::EntityId &entityId) const
noexcept as inline; update their declarations/definitions in EntityId.ipp
accordingly so they can be duplicated across TUs without linker errors.

In `@src/engine/src/scheduler/FixedTimeUpdate.cpp`:
- Around line 6-13: Validate tickRate is strictly positive to avoid
divide-by-zero: update the FixedTimeUpdate constructor and SetTickRate(float
tickRate) to check tickRate > 0 and reject invalid values (e.g., throw
std::invalid_argument with a clear message or assert) instead of assigning
zero/negative to _tickRate; keep GetTickRate() unchanged and ensure callers like
RunSystems() rely on this invariant so division by _tickRate is safe.

In `@src/engine/src/scheduler/RelativeTimeUpdate.cpp`:
- Around line 6-14: Validate and guard the tickRate value in
Engine::Scheduler::RelativeTimeUpdate: in the constructor
RelativeTimeUpdate(Core &core, float tickRate) and in SetTargetTickRate(float
tickRate) ensure tickRate is > 0 before assigning to _tickRate (either throw
std::invalid_argument or clamp to a small positive epsilon and/or log), so that
_tickRate is never zero or negative when used as a divisor in RunSystems(); keep
GetTargetTickRate unchanged.

In `@src/engine/src/system/System.hpp`:
- Around line 34-36: Update the GetSystems() doxygen comment to match the
function signature: change the return description that currently mentions
"vector" to "std::list" (or "list") so the doc for const
std::list<std::unique_ptr<SystemBase>>& GetSystems() correctly reflects the
returned container type; locate the comment above the GetSystems() declaration
in System.hpp and update the `@return` text accordingly.
- Line 54: SystemContainer::GetSystems() and SystemContainer::DeleteSystem() are
defined non-inline in System.ipp which is included into System.hpp causing
ODR/linker issues; fix by either marking both member functions as inline in
their definitions in System.ipp or by moving their full definitions out of
System.ipp into a dedicated System.cpp (and keep only declarations in
System.hpp), ensuring the declarations in System.hpp correspond to the
moved/inline definitions.

In `@src/engine/xmake.lua`:
- Around line 20-29: The xmake.lua header file globs omit entity implementation
files: add_headerfiles currently includes "src/(entity/*.hpp)" but not
"src/(entity/*.ipp)", so consumers that rely on Entity.hpp including an .ipp
will break; update the add_headerfiles calls to also include
"src/(entity/*.ipp)" (matching the pattern used for scheduler/system) so that
entity implementation .ipp files are exported alongside the .hpp headers.

---

Nitpick comments:
In `@src/engine/src/scheduler/SchedulerContainer.hpp`:
- Line 27: The declaration of the exception specification for the overridden
what() method uses the deprecated throw() spec; update the
SchedulerContainer::what() declaration (and its corresponding definition) to use
noexcept instead of throw(), e.g. change "const char *what() const throw()
override;" to use noexcept so the signature matches modern
std::exception::what() and keeps the override valid.

Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@src/engine/src/entity/EntityId.ipp`:
- Around line 45-53: The shared static map temporaryComponent is accessed
without synchronization in EntityId::AddTemporaryComponent and where it's
iterated/cleared (lines ~86-91), creating TOCTOU and concurrent-modification
races; fix by introducing a dedicated mutex (e.g., temporaryComponentMutex) and
acquire a lock (std::lock_guard or std::scoped_lock) around all accesses and
mutations of temporaryComponent, and replace the separate contains-then-assign
pattern in AddTemporaryComponent with an atomic insert/emplace that uses the
insertion result while still holding the lock to avoid race windows.

Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@src/engine/src/entity/Entity.cpp`:
- Line 15: The method Entity::RemoveTemporaryComponents currently calls
EntityId::RemoveTemporaryComponents(core) directly which bypasses the instance's
context; change it to delegate via the wrapped id and context by invoking
_entityId.RemoveTemporaryComponents(GetCore()) so it uses this Entity's stored
_entityId and GetCore() rather than an arbitrary Core passed by callers.
- Around line 17-19: The equality operators on Entity currently only compare
_entityId.value; update both Entity::operator==(const Entity &rhs) and
Entity::operator==(const EntityId &rhs) to include Core identity by delegating
to EntityId's own operator== (or by comparing both _core and _entityId) instead
of only comparing _entityId.value — i.e., use _entityId == rhs._entityId (or
_entityId == rhs) and ensure the comparison accounts for the Core
pointer/identifier so entities from different cores are not considered equal.