[NaySurGithub]

Problem

On a Hive hub, every game hologram switches to Build Battle a few seconds after joining. For example, the Sky Solo NPC shows "Build Battle" above it.

Cause

The holograms (hivehub:game_hologram) pick their name texture with query.mark_variant, and index 0 is Build Battle. The server sends the right MARK_VARIANT in add_entity. Its later player-count updates (set_entity_data, every ~5 s) only carry VARIANT.

ApplyMetadata then called:

tracker.SetVariant(rid, std::max(variant, 0), std::max(mark, 0), std::max(color, 0));

Every field the packet left out was still at its -1 sentinel, so max(-1, 0) wrote 0 into markVariant and color. Metadata updates are partial: a field that is not sent keeps its value.

Change

• EntityTracker::SetVariant takes std::optional<int> for variant, mark variant and color, and only writes the fields that are set.
• ApplyMetadata collects the three as optionals and still clamps the values it actually received to >= 0.

Testing

• macOS arm64 Release build.
• Not tested in game yet: on a Hive hub, the game holograms should keep their own titles after the first player-count update.