Keep in mind that Sathorvarr and the dragon Kalecgos switch back and forth between neutral and hostile as you get teleported up and down the realms. You can verify this by checking the combat log src/dst flags.
I use an ancient and simple addon called "Aggro" to show when I get aggro myself. All it does is scan party/raid targets, check if "player" or "pet" are targetted, and pop a big "AGGRO" in the middle of the screen.
This addon shows correctly when I become Sathrovarr's target, Grid through LibBanzai never did. The code it uses is very similar to LibBanzai:
if (UnitCanAttack(unit,"player") and UnitIsUnit(unit.."target","player")) then
Jerry said in the Grid thread that the return value of UnitExists or UnitCanAttack might be bugged for this mob, so maybe the order of parameters in UnitCanAttack matters in this case? The line above uses mob->player, whereas LibBanzai uses player->mob. It shouldn't make a difference, but maybe it's an obscure bug just for Sathrovarr.
I actually meant to verify this this week, but I forgot before raid time, if I manage to remember before next week's Kalecgos I'll post here the results.
I'm fairly certain it's a bug with Sathrovarr. When Sathrovarr is targetting a player tank, if I target Sathrovarr and type"
/dump UnitCanAttack("target", "targettarget")
...the return value is nil. However, I just looked at the Banzai code, and it seems to be using the opposite order for unit tokens, so I guess I'll have to wait until next week to try:
/dump UnitCanAttack("targettarget", "target")
Interesting. I've been annoyed by this since first Sunwell raid and it looks like you've found a solution here. Can any of you based on the information posted here sum up what I need to change for it to hopefully work? I know it's only in theory, but drycoding often seems to do the trick just fine anyway.