In `@src/plugin/object/src/utils/MeshSimplifier.cpp`:
- Around line 497-509: The loop in MeshSimplifier.cpp that computes cell
coordinates for each vertex (using variables cx, cy, cz in the for loop
iterating over vertices) casts floor(...) results to uint32_t which can UB if
the float is slightly negative; modify the computation to avoid casting negative
floats by clamping the float result to >= 0.0f before static_cast (or switch to
signed ints and clamp after), and apply the same fix to the identical pattern
later in the file (the block around the second occurrence at the lines mentioned
in the review). Ensure you still clamp to cellsX-1 / cellsY-1 / cellsZ-1 after
conversion.
- Around line 391-428: The SimplificationResult bestResult is returned
uninitialized if maxIterations == 0; update the function to either early-handle
the zero-iteration case or fully initialize bestResult before the loop: set
bestResult.originalVertexCount to mesh.GetVertices().size(), set
bestResult.simplifiedVertexCount to that same value (or to 0/appropriate
default), set bestResult.wasSimplified to false, and initialize
bestResult.vertexMap to an empty map/vector. Ensure these initializations occur
before the for-loop (or return a valid result immediately when maxIterations ==
0) so callers of SimplifyMesh/SimplificationResult get a well-defined object.

In `@src/plugin/object/src/utils/MeshSimplifier.hpp`:
- Around line 68-69: The field preserveBoundaries on MeshSimplifier is declared
but never used; either remove it or implement boundary-preservation checks in
the simplification routines. Locate the MeshSimplifier class and either delete
the preserveBoundaries member and any builder/exposure methods, or thread its
value into the simplifier logic: check preserveBoundaries inside SimplifyMesh,
SimplifyMeshToTarget, SimplifyMeshByVoxelization and DeduplicateVertices and
modify edge/collapse selection to skip collapsing or altering edges that are
boundary edges (edges with only one adjacent face). Ensure the builder methods
remain consistent with the chosen approach (remove builder when removing the
field, or keep and honor the flag when implementing).

In `@src/plugin/physics/src/component/SoftBody.hpp`:
- Around line 272-275: The comment on SoftBody::attachedEntityId references a
nonexistent AttachToEntity() method; update the comment or add the method.
Either implement a SoftBody::AttachToEntity(Engine::EntityId) method (or
similarly named attach function) that sets attachedEntityId and documents
intended behavior, or change the inline comment for attachedEntityId to refer to
the actual API used to attach (e.g., set attachedEntityId directly or call the
existing attach function name) so the doc matches the code.

In `@src/plugin/physics/src/system/SoftBodySystem.cpp`:
- Around line 709-727: The per-tick Log::Warn calls inside the
SyncSoftBodyVertices loop (the block that checks ConvexHullMeshCollider and
MeshCollider and currently calls Log::Warn "SoftBody: Using ... mesh as geometry
for SoftBody" or "SoftBody: No Mesh component found...") will spam logs; either
change these Log::Warn calls to Log::Debug, or add a one-time flag (e.g., a bool
on the SoftBody component or an internal map keyed by entity) that records that
the warning has been emitted and only logs on the first occurrence, setting the
flag after emitting; update the calls referring to ConvexHullMeshCollider,
MeshCollider and the "No Mesh" path accordingly and ensure the continue behavior
remains unchanged.
- Around line 58-100: The implementation of SelectKinematicVerticesByY does not
match its name/docs — it selects vertices via a uniform stride instead of by
lowest Y; either restore Y-based selection or rename and update docs. To fix: if
you want Y-based behavior, sort or index the input vertices by glm::vec3.y
(preserving original indices), select the lowest N indices (compute count using
std::ceil with std::clamp like currently done) and return those indices; update
variables like kinematicIndices accordingly and keep the 0%/100% short-circuits.
Alternatively, if you prefer uniform selection, rename
SelectKinematicVerticesByY to SelectKinematicVerticesUniform (and update the
function comment to describe uniform/stride selection) and adjust any callers to
use the new name.
- Around line 494-514: The code reads softBody.attachedEntityId.value() and
calls registry.try_get<...>(attachedEntity) without verifying the entity exists;
update the SoftBodySystem code to first check registry.valid(attachedEntity) (or
equivalent) after obtaining attachedEntity and before any try_get calls, and if
the entity is not valid log a warning (similar to the existing message) and skip
the attachment logic so attachedBodyID is not set from an invalid entity.
- Line 842: Remove the unused local variable by deleting the line that assigns
auto &bodyInterface = physicsManager.GetBodyInterface(); inside
UpdateKinematicVertices; if any later code intended to use the BodyInterface,
instead access it where needed via physicsManager.GetBodyInterface() inline or
introduce a used variable with a clear name, but otherwise remove the unused
declaration to eliminate the warning.
- Around line 858-871: Update the lock ordering to consistently acquire the
parent body lock before the soft-body lock to avoid ABBA inversion between
UpdateKinematicVertices and SyncSoftBodyVertices; in SyncSoftBodyVertices
reorder the BodyLockRead for the parent (using internal.attachedToBodyID) to be
taken before the soft-body BodyLockRead (using internal.bodyID), preserve the
Succeeded checks and early-continue behavior, and ensure you keep the same lock
types (read vs read) and variable names so the locking semantics match
UpdateKinematicVertices exactly.