Dots that are reapplied have their aura deactivated despite having ticks remaining

[BadJukeBox]

Dots that rollover deactivate early.

To Reproduce
On the feature/frost-mage while writing the code for living bomb exploding when reapplied with 1 tick left, I noticed that the amount of ticks for the new dot was not always 5. After doing some debugging I found that the aura for the dot is deactivating early regardless of ticks left. I'm trying to to figure out why this happens but if anyone with more experience can track it down, would be wonderful!

Here is the rotation being used. You'll notice in output after a few rounds of reapplying it alternates between 5 and 4 ticks (base is 4, updated is always 5 on beta), which is attributed to the aura deactivating.

[InDebt]

if result.Landed() {
	activeLbs := len(activeLivingBombs)
	if activeLbs >= maxLivingBombs {
		bombExplode = false
		activeLivingBombs[activeLbs-1].Deactivate(sim)
		if activeLbs != 0 {
			activeLivingBombs = activeLivingBombs[:1]
		}
		bombExplode = true
	}
	if spell.Dot(target).RemainingTicks() == 1 {
		livingBombExplosionSpell.Cast(sim, target)
	}
	spell.Dot(target).Apply(sim)
	activeLivingBombs = append(activeLivingBombs, mage.LivingBomb.Dot(target))
	sort.Slice(activeLivingBombs, func(i, j int) bool {
		return activeLivingBombs[i].Duration < activeLivingBombs[j].Duration
	})
}

The aura logic is working fine.
The activeLbs code branch is disabeling your Living Bomb. The code does just insert new Living bombs based on aura duration and does not handle clipping. Because when you clip it, it get's never disabled but is added to the list of active living bombs.

So you should be able to rewrite this as

if !spell.Dot(target).IsActive() {
    activeLivingBombs = append(activeLivingBombs, mage.LivingBomb.Dot(target))
}

spell.Dot(target).Apply(sim)

And then it should work as you expect