Refactor Azgalor Rain of Fire handling for melee DPS and Doomguard targeting

[Copilot]

Azgalor melee behavior was checking Rain of Fire state in multiple places and could continue Doomguard DPS handling while a melee bot was standing in fire. This change centralizes the Rain of Fire occupancy check and makes melee bots swap to a lesser doomguard while escaping, only falling back to movement away from Azgalor when no doomguard is available.

• Shared Rain of Fire state

  • Added a helper for “bot is inside active Azgalor Rain of Fire”.
  • Moved active-fire pruning/lookup into shared helper logic instead of duplicating it across triggers and action code.

• Trigger behavior

  • AzgalorBossCastsRainOfFireOnMeleeTrigger now keys off the shared helper.
  • AzgalorDoomguardsContinueToSpawnTrigger now stays inactive for melee DPS while they are inside active Rain of Fire, so fire escape behavior wins over Doomguard reprioritization.

• Melee escape action

  • AzgalorMeleeGetOutOfFireAction now behaves as an attack-capable action.
  • When a lesser doomguard is available, melee switches to it immediately while handling Rain of Fire.
  • If no lesser doomguard is found, the action keeps the previous fallback of moving away from Azgalor.

• Rain of Fire tracking

  • Tightened AzgalorRainOfFireScript usage to match the shared helper path and keep Rain of Fire position tracking as the single source of truth.

if (Unit* doomguard = AI_VALUE2(Unit*, "find target", "lesser doomguard"))
{
    if (bot->GetVictim() != doomguard)
        return Attack(doomguard);

    return true;
}

return MoveAway(azgalor, 5.0f);