Skip to content

Archimonde: interrupt Air Burst target casts before spread movement - #15

Merged
brighton-chi merged 1 commit into
the-labfrom
copilot/increase-archimonde-spread-sophistication
Apr 21, 2026
Merged

Archimonde: interrupt Air Burst target casts before spread movement#15
brighton-chi merged 1 commit into
the-labfrom
copilot/increase-archimonde-spread-sophistication

Conversation

Copilot AI commented Apr 21, 2026

Copy link
Copy Markdown

Archimonde Air Burst spread handling was keyed to the main tank target check, so the actual Air Burst target could continue casting and delay movement. This update makes the targeted bot react immediately by interrupting and creating distance from the main tank.

  • Targeting logic correction

    • In ArchimondeSpreadToAvoidAirBurstAction::Execute, the Air Burst check now matches spell->m_targets.GetUnitTarget() against bot (the current bot), not the main tank.
  • Immediate movement readiness

    • When the targeted bot is inside the unsafe range to main tank, it now calls RequestSpellInterrupt() before MoveAway(...) so repositioning is not blocked by an active cast.
  • Behavioral outcome

    • Air Burst avoidance is now executed by the actual targeted bot with lower reaction latency and cleaner separation from the tank stack.
if (mainTank && spell->m_targets.GetUnitTarget() == bot)
{
    float currentDistance = bot->GetDistance2d(mainTank);
    constexpr float safeDistance = 14.0f;
    if (currentDistance < safeDistance)
    {
        botAI->RequestSpellInterrupt();
        return MoveAway(mainTank, safeDistance - currentDistance);
    }
}

@brighton-chi
brighton-chi marked this pull request as ready for review April 21, 2026 22:16
@brighton-chi
brighton-chi merged commit 148eb91 into the-lab Apr 21, 2026
1 check failed
@brighton-chi
brighton-chi deleted the copilot/increase-archimonde-spread-sophistication branch April 21, 2026 22:16
brighton-chi added a commit that referenced this pull request Apr 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants