Im leaving some feedback thats minimal and non-actionable, add read-only record fields to the per-spell C_DamageMeter struct (DamageMeterCombatSpell), for example maxCrit, maxHit, critCount, and the same for the healing DamageMeterType.
three hard rules:
-
out-of-combat only (never feeds a live decision),
-
aggregate max, not a per-event stream (a max is not a replay-able timeline, cannot be stitched into a reconstructed log),
-
own character only (no raid-wide reads, no parse overlays).
C_DamageMeter already surfaces a per-hit max today, the source-level maxAmount. This is the same nature of data one level finer.
Full Write up: CoU feedback: revive harmless personal-best crit trackers via read-only C_DamageMeter fields