Implement #7718 (Exposing hitboxes and their bounds)

engine/Sandbox.Engine/Scene/Components/Game/Hitbox.cs

@@ -23,7 +23,7 @@ internal Hitbox( GameObject gameObject, BoneCollection.Bone bone, ITagSet tagSet
 	public BoneCollection.Bone Bone { get; private set; }
 	public ITagSet Tags { get; private set; }
 	public PhysicsBody Body { get; set; }
-	internal BBox Bounds { get; set; }
+	public BBox Bounds { get; set; }
 
 	public void Dispose()
 	{

engine/Sandbox.Engine/Scene/Components/Game/ModelHitboxes.cs

@@ -12,42 +12,38 @@ public sealed class ModelHitboxes : Component, Component.ExecuteInEditor
 {
 	HitboxSystem system;
 
-	SkinnedModelRenderer _renderer;
-
 	/// <summary>
 	/// The target SkinnedModelRenderer that holds the model/skeleton you want to 
 	/// take the hitboxes from.
 	/// </summary>
 	[Property]
 	public SkinnedModelRenderer Renderer
 	{
-		get => _renderer;
+		get;
 		set
 		{
-			if ( _renderer == value ) return;
+			if ( field == value ) return;
 
 			Clear();
 
-			_renderer = value;
+			field = value;
 
 			AddFrom( Renderer );
 		}
 	}
 
-	GameObject _target;
-
 	/// <summary>
 	/// The target GameObject to report in trace hits. If this is unset we'll defaault to the gameobject on which this component is.
 	/// </summary>
 	[Property]
 	public GameObject Target
 	{
-		get => _target;
+		get;
 		set
 		{
-			if ( _target == value ) return;
+			if ( field == value ) return;
 
-			_target = value;
+			field = value;
 
 			Rebuild();
 		}
@@ -73,10 +69,16 @@ protected override void OnDestroy()
 		Clear();
 	}
 
+	/// <summary>
+	/// Invoked when the hitboxes have been rebuilt.
+	/// </summary>
+	public Action HitboxesRebuilt;
+
 	public void Rebuild()
 	{
 		Clear();
 		AddFrom( Renderer );
+		HitboxesRebuilt?.Invoke();
 	}
 
 	void Clear()
@@ -165,11 +167,58 @@ public void UpdatePositions()
 
 	internal readonly List<Hitbox> Hitboxes = new();
 
+	/// <summary>
+	/// Adds a hitbox to the models hitbox list
+	/// </summary>
+	/// <param name="hitbox"></param>
 	public void AddHitbox( Hitbox hitbox )
 	{
 		Hitboxes.Add( hitbox );
 	}
 
+	/// <summary>
+	/// Removes a hitbox from the models hitbox list.
+	/// </summary>
+	/// <param name="hitbox"></param>
+	public void RemoveHitbox( Hitbox hitbox )
+	{
+		Hitboxes.Remove( hitbox );
+	}
+
+	/// <summary>
+	/// Returns every hitbox on this model.
+	/// </summary>
+	/// <returns></returns>
+	public IReadOnlyList<Hitbox> GetHitboxes() => Hitboxes;
+
+	/// <summary>
+	/// Retrieves the hitbox associated with the bone index.
+	/// </summary>
+	/// <param name="boneIndex">The bone index to retrieve.</param>
+	/// <returns>null if no matching hitbox is found.</returns>
+	public Hitbox GetHitbox( int boneIndex ) => Hitboxes.FirstOrDefault( x => x.Bone.Index == boneIndex );
+
+	/// <summary>
+	/// Retrieves the hitbox associated with the specified bone.
+	/// </summary>
+	/// <param name="bone">The bone for which to find the corresponding hitbox.</param>
+	/// <returns>null if no matching hitbox is found.</returns>
+	public Hitbox GetHitbox( BoneCollection.Bone bone ) => Hitboxes.FirstOrDefault( x => x.Bone == bone );
+
+	/// <summary>
+	/// Retrieves the hitbox associated with the specified bone name.
+	/// </summary>
+	/// <param name="boneName">The name of the bone for which to retrieve the hitbox.</param>
+	/// <returns>null if no matching hitbox is found.</returns>
+	public Hitbox GetHitbox( string boneName ) => Hitboxes.FirstOrDefault( x => x.Bone.Name == boneName );
+
+	/// <summary>
+	/// Retrieves the hitbox associated with the specified physics body.
+	/// </summary>
+	/// <param name="physicsBody">The physics body for which to find the corresponding hitbox.</param>
+	/// <returns>null if no matching hitbox is found.</returns>
+	public Hitbox GetHitbox( PhysicsBody physicsBody ) => Hitboxes.FirstOrDefault( x => x.Body == physicsBody );
+
 	/// <summary>
 	/// The gameobject tags have changed, update collision tags on the target objects
 	/// </summary>

engine/Sandbox.Engine/Scene/Components/Render/ModelRenderer.cs

@@ -196,7 +196,10 @@ internal void OnModelReloaded()
 		OnModelChanged();
 	}
 
-	internal Action ModelChanged;
+	/// <summary>
+	/// Invoked when <see cref="Model"/> has been changed.
+	/// </summary>
+	public Action ModelChanged;
 
 	internal void OnModelChanged()
 	{