Tags literals in code and ITaggable.

[Sl4vP0weR]

Replace tags literals like this:

if (lastDamage.HasTag("vehicle"))

to something like this with constants:

if (lastDamage.HasTag(BaseEntityTags.Vehicle))

Add ITaggable interface with bool HasTag(string tagName), void SetTag(string tagName) methods.
First thing that came in mind.
Or maybe we can add TagStorage class that would look like this:

public class TagStorage
{
    internal HashSet<string> Tags;
    public bool Has(string tagName)
    {
        if (Tags is null) return false;
        return Tags.Contains(tagName);
    }
    public void Set(string tagName)
    {
        Tags ??= new();
        if (Tags.Contains(tagName)) return;
        Tags.Add(tagName);
    }
    public void Clear() => Tags.Clear();
}

And will be used in other types like this:

class Entity
{
    public readonly TagStorage Tags = new();
}

Seems like simple things, but should be a good addition.

[darkhog]

I think this is overengineering things and making the API unnecessarily complex.

[Sl4vP0weR]

It's default OOP, I don't see any complexity in this.