Walkthrough

Adds optional embedded mesh storage to MeshCollider and ConvexHullMeshCollider; updates RigidBodySystem to prefer embedded mesh, fall back to entity Object::Mesh, and pass the resolved mesh into mesh-shape creation, with clearer warnings when mesh data is missing.

Changes

Sequence Diagram(s)

sequenceDiagram
  participant RigidBodySystem as RigidBodySystem
  participant Collider as Collider(Component)
  participant Registry as Registry/Object
  participant ShapeFactory as ShapeCreator

  RigidBodySystem->>Collider: Inspect collider (ConvexHull/MeshCollider)
  alt collider has embedded mesh
    Collider-->>RigidBodySystem: return embedded Mesh
  else no embedded mesh
    RigidBodySystem->>Registry: get Object::Mesh component
    alt Object::Mesh present
      Registry-->>RigidBodySystem: return Mesh
    else missing mesh
      Registry-->>RigidBodySystem: none (log warning)
      RigidBodySystem-->>RigidBodySystem: abort shape creation
    end
  end
  RigidBodySystem->>ShapeFactory: CreateMeshShapeFromMesh(resolved Mesh)
  ShapeFactory-->>RigidBodySystem: collision shape

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

• feat(physics): add CapsuleCollider and SphereCollider components #402: Touches CreateShapeFromColliders in RigidBodySystem.cpp (related shape-creation branches).
• refactor: use mesh shape for mesh collider #440: Modifies the same collider components and RigidBodySystem mesh-shape codepaths.
• feat(physics): mesh collider #439: Related changes introducing or extending mesh-collider handling and shape creation.

Suggested labels

enhancement

Suggested reviewers

• Divengerss

Poem

🐰
I hid a mesh within my fur so light,
Vertices tucked away from plain sight,
No need to fetch across the night,
Shapes form quick and hop just right—
Physics hums beneath the moonlight.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

• 📝 Generate docstrings

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch create-mesh-colliders-without-mesh

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

• X
• Mastodon
• Reddit
• LinkedIn

Comment @coderabbitai help to get the list of available commands and usage tips.

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud