Thanks for your amazing work, chip, that's really good news! As a "rule of thumb" I feel that every effect that has a duration and is communicated by changing the visual appearance of your character (glowing hands, heads, etc.) should be communicated via EVENT_EFFECT_CHANGED for the sake of visibility in large scale combat situations. If you could add the fields
sourceName and
sourceUnitId or at least
combatUnitType to EVENT_EFFECT_CHANGED it would make it absolutely perfect
Just out of curiosity: What type of buffs/debuffs/combat vvents shall not be visible to the client and why?