Changes

scripts/restodruid.lua

@@ -111,380 +111,371 @@ local RakeDebuff = Bastion.SpellBook:GetSpell(155722)
 
 
 local Lowest = Bastion.UnitManager:CreateCustomUnit('lowest', function(unit)
-    local lowest = nil
+        local lowest = nil
-    local lowestHP = math.huge
+        local lowestHP = math.huge
 
-    Bastion.UnitManager:EnumFriends(function(unit)
+        Bastion.UnitManager:EnumFriends(function(unit)
-        if unit:IsDead() then
+            if unit:IsDead() then
-            return false
+                return false
-        end
+            end
 
-        if Player:GetDistance(unit) > 40 then
+            if Player:GetDistance(unit) > 40 then
-            return false
+                return false
-        end
+            end
 
-        if not Player:CanSee(unit) then
+            if not Player:CanSee(unit) then
-            return false
+                return false
-        end
+            end
 
-        local hp = unit:GetHP()
+            local hp = unit:GetHP()
-        if hp < lowestHP then
+            if hp < lowestHP then
-            lowest = unit
+                lowest = unit
-            lowestHP = hp
+                lowestHP = hp
-        end
+            end
-    end)
+        end)
 
-    if not lowest then
+        if not lowest then
-        lowest = Player
+            lowest = Player
-    end
+        end
 
-    return lowest
+        return lowest
-end)
+    end)
 
 local DispelTarget = Bastion.UnitManager:CreateCustomUnit('dispel', function(unit)
-    local lowest = nil
+        local lowest = nil
-    local lowestHP = math.huge
+        local lowestHP = math.huge
-
-    Bastion.UnitManager:EnumFriends(function(unit)
-        if unit:IsDead() then
-            return false
-        end
 
-        if not Player:CanSee(unit) then
+        Bastion.UnitManager:EnumFriends(function(unit)
-            return false
+            if unit:IsDead() then
-        end
+                return false
+            end
 
-        if Player:GetDistance(unit) > 40 then
+            if not Player:CanSee(unit) then
-            return false
+                return false
-        end
+            end
 
-        if not unit:IsDead() and Player:CanSee(unit) and
+            if Player:GetDistance(unit) > 40 then
-            unit:GetAuras():HasAnyDispelableAura(NaturesCure) then
+                return false
+            end
 
-            local hp = unit:GetHP()
+            if not unit:IsDead() and Player:CanSee(unit) and
-            if hp < lowestHP then
+                unit:GetAuras():HasAnyDispelableAura(NaturesCure) then
-                lowest = unit
+                local hp = unit:GetHP()
-                lowestHP = hp
+                if hp < lowestHP then
+                    lowest = unit
+                    lowestHP = hp
+                end
             end
+        end)
+
+        if lowest == nil then
+            lowest = None
         end
+
+        return lowest
     end)
 
-    if lowest == nil then
+local PurgeTarget = Bastion.UnitManager:CreateCustomUnit('purge', function(unit)
-        lowest = None
+        local purge = nil
-    end
 
-    return lowest
+        Bastion.UnitManager:EnumEnemies(function(unit)
-end)
+            if unit:IsDead() then
+                return false
+            end
 
-local PurgeTarget = Bastion.UnitManager:CreateCustomUnit('purge', function(unit)
+            if not Player:CanSee(unit) then
-    local purge = nil
+                return false
+            end
 
-    Bastion.UnitManager:EnumEnemies(function(unit)
+            if Player:GetDistance(unit) > 40 then
-        if unit:IsDead() then
+                return false
-            return false
+            end
-        end
 
-        if not Player:CanSee(unit) then
+            if not unit:IsDead() and Player:CanSee(unit) and
-            return false
+                unit:GetAuras():HasAnyStealableAura() then
-        end
+                purge = unit
+                return true
+            end
+        end)
 
-        if Player:GetDistance(unit) > 40 then
+        if purge == nil then
-            return false
+            purge = None
         end
 
-        if not unit:IsDead() and Player:CanSee(unit) and
+        return purge
-            unit:GetAuras():HasAnyStealableAura() then
-            purge = unit
-            return true
-        end
     end)
 
-    if purge == nil then
+local Tank = Bastion.UnitManager:CreateCustomUnit('tank', function(unit)
-        purge = None
+        local tank = nil
-    end
 
-    return purge
+        Bastion.UnitManager:EnumFriends(function(unit)
-end)
+            if Player:GetDistance(unit) > 40 then
+                return false
+            end
 
-local Tank = Bastion.UnitManager:CreateCustomUnit('tank', function(unit)
+            if not Player:CanSee(unit) then
-    local tank = nil
+                return false
+            end
 
-    Bastion.UnitManager:EnumFriends(function(unit)
+            if unit:IsDead() then
-        if Player:GetDistance(unit) > 40 then
+                return false
-            return false
+            end
-        end
 
-        if not Player:CanSee(unit) then
+            if unit:IsTank() then
-            return false
+                tank = unit
-        end
+                return true
+            end
 
-        if unit:IsDead() then
             return false
-        end
+        end)
 
-        if unit:IsTank() then
+        if tank == nil then
-            tank = unit
+            tank = Player
-            return true
         end
 
-        return false
+        return tank
     end)
 
-    if tank == nil then
-        tank = Player
-    end
-
-    return tank
-end)
-
 local RejuvUnit = Bastion.UnitManager:CreateCustomUnit('rejuv', function(unit)
-    local lowest = nil
+        local lowest = nil
-    local lowestHP = math.huge
+        local lowestHP = math.huge
 
-    Bastion.UnitManager:EnumFriends(function(unit)
+        Bastion.UnitManager:EnumFriends(function(unit)
-        if unit:IsDead() then
+            if unit:IsDead() then
-            return false
+                return false
-        end
+            end
 
-        if not Player:CanSee(unit) then
+            if not Player:CanSee(unit) then
-            return false
+                return false
-        end
+            end
 
-        if Player:GetDistance(unit) > 40 then
+            if Player:GetDistance(unit) > 40 then
-            return false
+                return false
-        end
+            end
 
-        if not unit:IsDead() and Player:CanSee(unit) and
+            if not unit:IsDead() and Player:CanSee(unit) and
-            (
+                (
-            not unit:GetAuras():FindMy(Rejuvenation):IsUp() or
+                not unit:GetAuras():FindMy(Rejuvenation):IsUp() or
                 unit:GetAuras():FindMy(Rejuvenation):GetRemainingTime() <= 3.6) then
-
+                local hp = unit:GetHP()
-            local hp = unit:GetHP()
+                if hp < lowestHP then
-            if hp < lowestHP then
+                    lowest = unit
-                lowest = unit
+                    lowestHP = hp
-                lowestHP = hp
+                end
             end
-        end
+        end)
-
-    end)
 
 
-    if lowest == nil then
+        if lowest == nil then
-        lowest = Player
+            lowest = Player
-    end
+        end
 
-    return lowest
+        return lowest
-end)
+    end)
 
 local SwiftmendUnit = Bastion.UnitManager:CreateCustomUnit('swiftmend', function(unit)
-    local lowest = nil
+        local lowest = nil
-    local lowestHP = math.huge
+        local lowestHP = math.huge
 
-    Bastion.UnitManager:EnumFriends(function(unit)
+        Bastion.UnitManager:EnumFriends(function(unit)
-        if unit:IsDead() then
+            if unit:IsDead() then
-            return false
+                return false
-        end
+            end
 
-        if not Player:CanSee(unit) then
+            if not Player:CanSee(unit) then
-            return false
+                return false
-        end
+            end
 
-        if Player:GetDistance(unit) > 40 then
+            if Player:GetDistance(unit) > 40 then
-            return false
+                return false
-        end
+            end
 
-        if (
+            if (
-            Player:CanSee(unit) and (
+                Player:CanSee(unit) and (
                 (unit:GetAuras():FindMy(Regrowth):IsUp())
-                    or
+                or
-                    (
+                (
-                    unit:GetAuras():FindMy(Rejuvenation):IsUp() and
+                unit:GetAuras():FindMy(Rejuvenation):IsUp() and
-                        not unit:GetAuras():FindMy(WildGrowth):IsUp())
+                not unit:GetAuras():FindMy(WildGrowth):IsUp())
                 )
-            ) then
+                ) then
-
+                local hp = unit:GetHP()
-            local hp = unit:GetHP()
+                if hp < lowestHP then
-            if hp < lowestHP then
+                    lowest = unit
-                lowest = unit
+                    lowestHP = hp
-                lowestHP = hp
+                end
             end
-        end
+        end)
-
-    end)
 
 
-    if lowest == nil then
+        if lowest == nil then
-        lowest = None
+            lowest = None
-    end
+        end
 
-    return lowest
+        return lowest
-end)
+    end)
 
 local WildGrowthUnit = Bastion.UnitManager:CreateCustomUnit('wildgrowth', function(unit)
-    local lowest = nil
+        local lowest = nil
-    local lowestHP = math.huge
+        local lowestHP = math.huge
 
-    Bastion.UnitManager:EnumFriends(function(unit)
+        Bastion.UnitManager:EnumFriends(function(unit)
-        if unit:IsDead() then
+            if unit:IsDead() then
-            return false
+                return false
-        end
+            end
 
-        if not Player:CanSee(unit) then
+            if not Player:CanSee(unit) then
-            return false
+                return false
-        end
+            end
 
-        if Player:GetDistance(unit) > 40 then
+            if Player:GetDistance(unit) > 40 then
-            return false
+                return false
-        end
+            end
 
-        if Player:CanSee(unit) and (
+            if Player:CanSee(unit) and (
-            (
+                (
                 Player:GetAuras():FindMy(SoulOfTheForest):IsUp() and
-                    (
+                (
-                    Player:GetAuras():FindMy(SoulOfTheForest):GetRemainingTime() <= 5 or
+                Player:GetAuras():FindMy(SoulOfTheForest):GetRemainingTime() <= 5 or
-                        unit:GetPartyHPAround(30, 90) >= 2)) or
+                unit:GetPartyHPAround(30, 90) >= 2)) or
                 (unit:GetPartyHPAround(30, 90) >= 3 or unit:GetPartyHPAround(30, 85) >= 2))
-        then
+            then
-            local hp = unit:GetHP()
+                local hp = unit:GetHP()
-            if hp < lowestHP then
+                if hp < lowestHP then
-                lowest = unit
+                    lowest = unit
-                lowestHP = hp
+                    lowestHP = hp
+                end
             end
+        end)
+
+
+        if lowest == nil then
+            lowest = None
         end
 
+        return lowest
     end)
 
+local Explosive = Bastion.UnitManager:CreateCustomUnit('explosive', function(unit)
+        local explosive = nil
 
-    if lowest == nil then
+        Bastion.ObjectManager.explosives:each(function(unit)
-        lowest = None
+            if unit:IsDead() then
-    end
+                return false
-
+            end
-    return lowest
-end)
 
-local Explosive = Bastion.UnitManager:CreateCustomUnit('explosive', function(unit)
+            if not Player:CanSee(unit) then
-    local explosive = nil
+                return false
+            end
 
-    Bastion.ObjectManager.explosives:each(function(unit)
+            if Player:GetDistance(unit) <= 40 then
-        if unit:IsDead() then
+                explosive = unit
-            return false
+                return true
-        end
+            end
+        end)
 
-        if not Player:CanSee(unit) then
+        if explosive == nil then
-            return false
+            explosive = None
         end
 
-        if Player:GetDistance(unit) <= 40 then
+        return explosive
-            explosive = unit
-            return true
-        end
     end)
 
-    if explosive == nil then
-        explosive = None
-    end
-
-    return explosive
-end)
-
 local RakeTarget = Bastion.UnitManager:CreateCustomUnit('rake', function(unit)
-    local rakeTarget = nil
+        local rakeTarget = nil
 
-    Bastion.UnitManager:EnumEnemies(function(unit)
+        Bastion.UnitManager:EnumEnemies(function(unit)
-        if unit:IsDead() then
+            if unit:IsDead() then
-            return false
+                return false
-        end
+            end
 
-        if not Player:CanSee(unit) then
+            if not Player:CanSee(unit) then
-            return false
+                return false
-        end
+            end
 
-        if Player:GetDistance(unit) > 40 then
+            if Player:GetDistance(unit) > 40 then
-            return false
+                return false
-        end
+            end
 
-        if not unit:IsDead() and Player:CanSee(unit) and unit:InCombatOdds() > 80 and unit:InMelee(Player) and
+            if not unit:IsDead() and Player:CanSee(unit) and unit:InCombatOdds() > 80 and unit:InMelee(Player) and
-            Player:IsFacing(unit) and
+                Player:IsFacing(unit) and
-            (
+                (
-            not unit:GetAuras():FindMy(RakeDebuff):IsUp() or
+                not unit:GetAuras():FindMy(RakeDebuff):IsUp() or
                 unit:GetAuras():FindMy(RakeDebuff):GetRemainingTime() <= 3.6) then
-            rakeTarget = unit
+                rakeTarget = unit
-        end
+            end
-
+        end)
-    end)
 
 
-    if rakeTarget == nil then
+        if rakeTarget == nil then
-        rakeTarget = None
+            rakeTarget = None
-    end
+        end
 
-    return rakeTarget
+        return rakeTarget
-end)
+    end)
 
 local MoonfireTarget = Bastion.UnitManager:CreateCustomUnit('moonfire', function(unit)
-    local moonfireTarget = nil
+        local moonfireTarget = nil
 
-    Bastion.UnitManager:EnumEnemies(function(unit)
+        Bastion.UnitManager:EnumEnemies(function(unit)
-        if unit:IsDead() then
+            if unit:IsDead() then
-            return false
+                return false
-        end
+            end
 
-        if not Player:CanSee(unit) then
+            if not Player:CanSee(unit) then
-            return false
+                return false
-        end
+            end
 
-        if Player:GetDistance(unit) > 40 then
+            if Player:GetDistance(unit) > 40 then
-            return false
+                return false
-        end
+            end
 
-        if not unit:IsDead() and Player:CanSee(unit) and unit:InCombatOdds() > 80 and
+            if not unit:IsDead() and Player:CanSee(unit) and unit:InCombatOdds() > 80 and
-            (
+                (
-            not unit:GetAuras():FindMy(MoonfireAura):IsUp() or
+                not unit:GetAuras():FindMy(MoonfireAura):IsUp() or
                 unit:GetAuras():FindMy(MoonfireAura):GetRemainingTime() <= 3.6) then
-            moonfireTarget = unit
+                moonfireTarget = unit
+            end
+        end)
+
+        if moonfireTarget == nil then
+            moonfireTarget = None
         end
 
+        return moonfireTarget
     end)
 
-    if moonfireTarget == nil then
-        moonfireTarget = None
-    end
-
-    return moonfireTarget
-end)
-
 local SunfireTarget = Bastion.UnitManager:CreateCustomUnit('sunfire', function(unit)
-    local sunfireTarget = nil
+        local sunfireTarget = nil
 
-    Bastion.UnitManager:EnumEnemies(function(unit)
+        Bastion.UnitManager:EnumEnemies(function(unit)
-        if unit:IsDead() then
+            if unit:IsDead() then
-            return false
+                return false
-        end
+            end
 
-        if not Player:CanSee(unit) then
+            if not Player:CanSee(unit) then
-            return false
+                return false
-        end
+            end
 
-        if Player:GetDistance(unit) > 40 then
+            if Player:GetDistance(unit) > 40 then
-            return false
+                return false
-        end
+            end
 
-        if not unit:IsDead() and Player:CanSee(unit) and unit:InCombatOdds() > 80 and
+            if not unit:IsDead() and Player:CanSee(unit) and unit:InCombatOdds() > 80 and
-            (
+                (
-            not unit:GetAuras():FindMy(SunfireAura):IsUp() or
+                not unit:GetAuras():FindMy(SunfireAura):IsUp() or
                 unit:GetAuras():FindMy(SunfireAura):GetRemainingTime() <= 3.6) then
-            sunfireTarget = unit
+                sunfireTarget = unit
+            end
+        end)
+
+        if sunfireTarget == nil then
+            sunfireTarget = None
         end
 
+        return sunfireTarget
     end)
 
-    if sunfireTarget == nil then
-        sunfireTarget = None
-    end
-
-    return sunfireTarget
-end)
-
 local RestoCommands = Bastion.Command:New('resto')
 
 local PLACE_EFFLO = false
@@ -502,7 +493,7 @@ DamageAPL:AddSpell(
         return RakeTarget:Exists() and self:IsKnownAndUsable() and not Player:IsCastingOrChanneling() and
             (
             not RakeTarget:GetAuras():FindMy(RakeDebuff):IsUp() or
-                RakeTarget:GetAuras():FindMy(RakeDebuff):GetRemainingTime() <= 3.6)
+            RakeTarget:GetAuras():FindMy(RakeDebuff):GetRemainingTime() <= 3.6)
     end):SetTarget(RakeTarget)
 )
 
@@ -520,11 +511,6 @@ DamageAPL:AddSpell(
     end):SetTarget(Target)
 )
 
-DefaultAPL:AddSpell(
-    Moonfire:CastableIf(function(self)
-        return Explosive:Exists() and self:IsKnownAndUsable() and not Player:IsCastingOrChanneling()
-    end):SetTarget(Explosive)
-)
 
 DefaultAPL:AddSpell(
     Efflorescence:CastableIf(function(self)
@@ -545,7 +531,7 @@ end)
 DefaultAPL:AddAction(
     'cat_form_shift',
     function()
-        if (IsShiftKeyDown() or not Player:IsAffectingCombat()) and not Player:IsMounted() and
+        if (IsShiftKeyDown()) and not Player:IsMounted() and
             not Player:GetAuras():FindMy(CatForm):IsUp() and
             not Player:IsCastingOrChanneling() then
             CatForm:Cast(Player)
@@ -613,9 +599,9 @@ DefaultAPL:AddSpell(
             and Player:CanSee(SwiftmendUnit) and
             (
             SwiftmendUnit:GetHP() <= 60 or
-                (
+            (
-                Lowest:GetPartyHPAround(30, 90) >= 3 or Lowest:GetPartyHPAround(30, 85) >= 2
+            Lowest:GetPartyHPAround(30, 90) >= 3 or Lowest:GetPartyHPAround(30, 85) >= 2
-                )
+            )
             )
     end):SetTarget(SwiftmendUnit)
 )
@@ -626,8 +612,8 @@ DefaultAPL:AddSpell(
             and Player:CanSee(WildGrowthUnit) and
             (
             Player:GetAuras():FindMy(SoulOfTheForest):IsUp()
-                or
+            or
-                (WildGrowthUnit:GetPartyHPAround(30, 90) >= 3 or WildGrowthUnit:GetPartyHPAround(30, 85) >= 2)
+            (WildGrowthUnit:GetPartyHPAround(30, 90) >= 3 or WildGrowthUnit:GetPartyHPAround(30, 85) >= 2)
             ) and
             not Player:IsMoving()
     end):SetTarget(WildGrowthUnit)
@@ -647,7 +633,7 @@ DefaultAPL:AddSpell(
             and Player:CanSee(Lowest) and Lowest:GetHP() < 70 and
             (
             NaturesSwiftness:GetTimeSinceLastCast() < 2 or Player:GetAuras():FindMy(NaturesSwiftness):IsUp() or
-                NaturesSwiftness:IsKnownAndUsable()) and not Player:IsMoving() and
+            NaturesSwiftness:IsKnownAndUsable()) and not Player:IsMoving() and
             not Player:GetAuras():FindMy(SoulOfTheForest):IsUp()
     end):SetTarget(Lowest)
 )
@@ -680,7 +666,7 @@ DefaultAPL:AddSpell(
             and
             (
             not Player:GetAuras():FindMy(LifebloomAura):IsUp() or
-                Player:GetAuras():FindMy(LifebloomAura):GetRemainingTime() <= 4.5) and Player:IsAffectingCombat()
+            Player:GetAuras():FindMy(LifebloomAura):GetRemainingTime() <= 4.5) and Player:IsAffectingCombat()
     end):SetTarget(Player)
 )
 
@@ -690,7 +676,7 @@ DefaultAPL:AddSpell(
             and
             (
             not Tank:GetAuras():FindMy(LifebloomAura):IsUp() or
-                Tank:GetAuras():FindMy(LifebloomAura):GetRemainingTime() <= 4.5) and Tank:IsAffectingCombat()
+            Tank:GetAuras():FindMy(LifebloomAura):GetRemainingTime() <= 4.5) and Tank:IsAffectingCombat()
     end):SetTarget(Tank)
 )
 
@@ -700,7 +686,7 @@ DefaultAPL:AddSpell(
             and Player:CanSee(Lowest) and
             (
             not Player:GetAuras():FindMy(Regrowth):IsUp() and Lowest:GetHP() < 70 or
-                (Lowest:GetHP() <= 85 and Player:GetAuras():FindMy(ClearCasting):IsUp())) and
+            (Lowest:GetHP() <= 85 and Player:GetAuras():FindMy(ClearCasting):IsUp())) and
             not Player:GetAuras():FindMy(SoulOfTheForest):IsUp() and
             not Player:IsMoving()
     end):SetTarget(Lowest)
@@ -714,13 +700,20 @@ DefaultAPL:AddSpell(
     end):SetTarget(RejuvUnit)
 )
 
+
+DefaultAPL:AddSpell(
+    Moonfire:CastableIf(function(self)
+        return Explosive:Exists() and self:IsKnownAndUsable() and not Player:IsCastingOrChanneling()
+    end):SetTarget(Explosive)
+)
+
 DefaultAPL:AddSpell(
     Sunfire:CastableIf(function(self)
         return SunfireTarget:Exists() and self:IsKnownAndUsable() and not Player:IsCastingOrChanneling()
             and Player:CanSee(SunfireTarget) and
             (
             not SunfireTarget:GetAuras():FindMy(SunfireAura):IsUp() or
-                SunfireTarget:GetAuras():FindMy(SunfireAura):GetRemainingTime() <= 5.4) and
+            SunfireTarget:GetAuras():FindMy(SunfireAura):GetRemainingTime() <= 5.4) and
             SunfireTarget:IsHostile() and
             SunfireTarget:IsAffectingCombat() and Player:GetPP() >= 25
     end):SetTarget(SunfireTarget)
@@ -732,7 +725,7 @@ DefaultAPL:AddSpell(
             and Player:CanSee(MoonfireTarget) and
             (
             not MoonfireTarget:GetAuras():FindMy(MoonfireAura):IsUp() or
-                MoonfireTarget:GetAuras():FindMy(MoonfireAura):GetRemainingTime() <= 5.4) and
+            MoonfireTarget:GetAuras():FindMy(MoonfireAura):GetRemainingTime() <= 5.4) and
             MoonfireTarget:IsHostile() and
             MoonfireTarget:IsAffectingCombat() and Player:GetPP() >= 25
     end):SetTarget(MoonfireTarget)

src/APL/APL.lua

@@ -82,6 +82,25 @@ end
 
 -- Execute
 function APLActor:Execute()
+    -- If the actor is a sequencer we don't want to continue executing the APL if the sequencer is not finished
+    if self:GetActor().sequencer then
+        if self:GetActor().condition and self:GetActor().condition() and not self:GetActor().sequencer:Finished() then
+            print("Execute?")
+            self:GetActor().sequencer:Execute()
+            return true
+        end
+
+        if not self:GetActor().condition and not self:GetActor().sequencer:Finished() then
+            print("Execute?")
+            self:GetActor().sequencer:Execute()
+            return true
+        end
+
+        -- Check if the sequencer can be reset and reset it if it can
+        if self:GetActor().sequencer:ShouldReset() then
+            self:GetActor().sequencer:Reset()
+        end
+    end
     if self:GetActor().apl then
         if self:GetActor().condition and self:GetActor().condition() then
             -- print("Bastion: APL:Execute: Executing sub APL " .. self:GetActor().apl.name)
@@ -116,6 +135,7 @@ function APLActor:Execute()
         -- print("Bastion: APL:Execute: Setting variable " .. self:GetActor().variable)
         self:GetActor()._apl.variables[self:GetActor().variable] = self:GetActor().cb(self:GetActor()._apl)
     end
+    return false
 end
 
 -- has traits
@@ -229,13 +249,29 @@ end
 function APL:Execute()
     for _, actor in ipairs(self.apl) do
         if actor:HasTraits() and actor:Evaluate() then
-            actor:Execute()
+            if actor:Execute() then
+                print("BREAQK", actor)
+                break
+            end
         else
-            actor:Execute()
+            if actor:Execute() then
+                print("BREAQK", actor)
+                break
+            end
         end
     end
 end
 
+-- Add a Sequencer to the APL
+---@param sequencer Sequencer
+---@param condition fun(...):boolean
+---@return APLActor
+function APL:AddSequence(sequencer, condition)
+    local actor = APLActor:New({ sequencer = sequencer, condition = condition })
+    table.insert(self.apl, actor)
+    return actor
+end
+
 -- tostring
 ---@return string
 function APL:__tostring()

src/MythicPlusUtils/MythicPlusUtils.lua

@@ -18,79 +18,365 @@ function MythicPlusUtils:New()
 
     self.random = math.random(1000000, 9999999)
     self.kickList = {
-        -- Algeth'ar Academy
+        -- Ruby life pools
-        [396812] = { true, true }, -- https://www.wowhead.com/spell=396812/mystic-blast
+        [372735] = { -- Techtonic Slam
-        [388392] = { true, true }, -- https://www.wowhead.com/spell=388392/monotonous-lecture
+            [187969] = {
-        [388863] = { true, true }, -- https://www.wowhead.com/spell=388863/mana-void
+                false, true, true -- Kick, Stun, Disorient
-        [388862] = { true, true }, -- https://www.wowhead.com/spell=388862/surge
+            }
-        [377389] = { true, true }, -- https://www.wowhead.com/spell=377389/call-of-the-flock
+        },
-        [388623] = { true, true }, -- https://www.wowhead.com/spell=388623/branch-out
+        [384933] = { -- Ice Shield
-        [396640] = { true, true }, -- https://www.wowhead.com/spell=396640/healing-touch
+            [188067] = {
-        [387975] = { true, true }, -- https://www.wowhead.com/spell=387975/arcane-missiles
+                true, true, true
-        [387843] = { true, true }, -- https://www.wowhead.com/spell=387843/astral-bomb
+            }
-        -- Court of Stars
+        },
-        [211401] = { true, true }, -- https://www.wowhead.com/spell=211401/drifting-embers
+        [372749] = { -- Ice Shield
-        [207980] = { true, true }, -- https://www.wowhead.com/spell=207980/disintegration-beam
+            [188067] = {
-        [208165] = { true, true }, -- https://www.wowhead.com/spell=208165/withering-soul
+                true, true, true
-        [207881] = { true, true }, -- https://www.wowhead.com/spell=207881/infernal-eruption
+            }
-        [209413] = { true, false }, -- https://www.wowhead.com/spell=209413/suppress
+        },
-        [209485] = { true, true }, -- https://www.wowhead.com/spell=209485/drain-magic
+        [372743] = { -- Ice Shield
-        [209410] = { true, true }, -- https://www.wowhead.com/spell=209410/nightfall-orb
+            [188067] = {
-        [211470] = { true, true }, -- https://www.wowhead.com/spell=211470/bewitch
+                true, true, true
-        [225100] = { true, false }, -- https://www.wowhead.com/spell=225100/charging-station
+            }
-        [211299] = { true, false }, -- https://www.wowhead.com/spell=211299/searing-glare
+        },
-        -- Halls of Valor
+        [371984] = {
-        [198595] = { true, true }, -- https://www.wowhead.com/spell=198595/thunderous-bolt
+            [188067] = {
-        [198959] = { true, true }, -- https://www.wowhead.com/spell=198959/etch
+                true, true, true
-        [215433] = { true, true }, -- https://www.wowhead.com/spell=215433/holy-radiance
+            }
-        [192288] = { true, true }, -- https://www.wowhead.com/spell=192288/searing-light
+        },
-        [199726] = { true, true }, -- https://www.wowhead.com/spell=199726/unruly-yell
+        [373680] = {
-        [198750] = { true, true }, -- https://www.wowhead.com/spell=198750/surge
+            [188252] = {
-        [198934] = { true, true }, -- https://www.wowhead.com/spell=198934/rune-of-healing
+                true, false, false
-        [192563] = { true, true }, -- https://www.wowhead.com/spell=192563/cleansing-flames
+            }
-        -- Ruby Life Pools
+        },
-        [372749] = { true, true }, -- https://www.wowhead.com/spell=372749/ice-shield
+        [373688] = {
-        -- [373803] = { true, true }, -- https://www.wowhead.com/spell=373803/cold-claws
+            [188252] = {
-        [373017] = { true, true }, -- https://www.wowhead.com/spell=373017/roaring-blaze
+                true, false, false
-        [392398] = { true, true }, -- https://www.wowhead.com/spell=392398/crackling-detonation
+            }
-        [392451] = { true, true }, -- https://www.wowhead.com/spell=392451/flashfire
+        },
-        [385310] = { true, true }, -- https://www.wowhead.com/spell=385310/lightning-bolt
+        [385310] = {
-        [384194] = { true, true }, -- https://www.wowhead.com/spell=384194/cinderbolt
+            [195119] = {
-        [373680] = { true, false }, -- https://www.wowhead.com/spell=373680/frost-overload
+                true, false, false
-        -- Shadowmoon Burial Grounds
+            }
-        [152818] = { true, true }, -- https://www.wowhead.com/spell=152818/shadow-mend
+        },
-        [156776] = { true, true }, -- https://www.wowhead.com/spell=156776/rending-voidlash
+        [384194] = {
-        [156722] = { true, true }, -- https://www.wowhead.com/spell=156722/void-bolt
+            [190207] = {
-        [398206] = { true, true }, -- https://www.wowhead.com/spell=398206/death-blast
+                true, true, true
-        [153524] = { true, true }, -- https://www.wowhead.com/spell=153524/plague-spit
+            }
-        [156718] = { true, true }, -- https://www.wowhead.com/spell=156718/necrotic-burst
+        },
-        -- Temple of the Jade Serpent
+        [384197] = {
-        [397888] = { true, false }, -- https://www.wowhead.com/spell=397888/hydrolance
+            [190207] = {
-        [395859] = { true, true }, -- https://www.wowhead.com/spell=395859/haunting-scream
+                true, true, true
-        [396073] = { true, true }, -- https://www.wowhead.com/spell=396073/cat-nap
+            }
-        [397914] = { true, true }, -- https://www.wowhead.com/spell=397914/defiling-mist
+        },
-        [397899] = { false, true }, -- https://www.wowhead.com/spell=397899/leg-sweep
+        [373017] = {
-        [397801] = { true, false }, -- https://www.wowhead.com/spell=397801/hydrolance
+            [189886] = {
-        [395872] = { true, true }, -- https://www.wowhead.com/spell=395872/sleepy-soliloquy
+                true, false, false
-        -- The Azure Vault
+            }
-        [375602] = { true, true }, -- https://www.wowhead.com/spell=375602/erratic-growth
+        },
-        [387564] = { true, true }, -- https://www.wowhead.com/spell=387564/mystic-vapors
+        [392576] = {
-        [373932] = { true, true }, -- https://www.wowhead.com/spell=373932/illusionary-bolt
+            [198047] = {
-        [386546] = { true, true }, -- https://www.wowhead.com/spell=386546/waking-bane
+                true, false, false
-        [389804] = { true, true }, -- https://www.wowhead.com/spell=389804/heavy-tome
+            }
-        [377488] = { true, true }, -- https://www.wowhead.com/spell=377488/icy-bindings
+        },
-        [377503] = { true, true }, -- https://www.wowhead.com/spell=377503/condensed-frost
+        [392451] = {
-        [375596] = { true, true }, -- https://www.wowhead.com/spell=375596/erratic-growth
+            [197985] = {
-        -- NO
+                true, true, false,
-        [384365] = { true, true }, -- https://www.wowhead.com/spell=384365/disruptive-shout
+            }
-        [386012] = { true, false }, -- https://www.wowhead.com/spell=386012/stormbolt
+        },
-        [386024] = { true, true }, -- https://www.wowhead.com/spell=386024/tempest
+        [392452] = {
-        [387411] = { true, true }, -- https://www.wowhead.com/spell=387411/death-bolt-volley
+            [197985] = {
-        [387606] = { true, true }, -- https://www.wowhead.com/spell=387606/dominate
+                true, true, false,
-        [373395] = { true, true }, -- https://www.wowhead.com/spell=373395/bloodcurdling-shout
+            }
-        [376725] = { true, true }, -- https://www.wowhead.com/spell=376725/storm-bolt
+        },
-        [384808] = { true, true }, -- https://www.wowhead.com/spell=384808/guardian-wind
+        -- Nokhud
-        -- [363607] = {true, true}, -- https://www.wowhead.com/spell=363607/domination-bolt
+        [383823] = {
+            [192796] = {
+                false, true, true
+            }
+        },
+        [384492] = {
+            [192794] = {
+                false, true, true
+            }
+        },
+        [384365] = {
+            [192800] = {
+                true, false, false
+            },
+            [191847] = {
+                true, false, false
+            }
+        },
+        [386012] = {
+            [194317] = {
+                true, false, false
+            },
+            [195265] = {
+                true, false, false
+            },
+            [194315] = {
+                true, false, false
+            },
+            [194316] = {
+                true, false, false
+            }
+
+        },
+        [386028] = {
+            [195696] = {
+                true, false, false
+            }
+        },
+        [386024] = {
+            [194894] = {
+                true, true, true
+            }
+        },
+        [386025] = {
+            [194894] = {
+                true, true, true
+            }
+        },
+        [387629] = {
+            [195876] = {
+                false, true, true
+            }
+        },
+        [387608] = {
+            [195842] = {
+                false, true, true
+            }
+        },
+        [387611] = {
+            [195842] = {
+                false, true, true
+            }
+        },
+        [387440] = {
+            [195878] = {
+                false, true, true
+            }
+        },
+        [373395] = {
+            [199717] = {
+                true, false, false
+            }
+        },
+        [376725] = {
+            [190294] = {
+                true, true, true
+            },
+        },
+        [370764] = {
+            [187160] = {
+                false, true, true
+            },
+            [196116] = {
+                false, true, true
+            },
+        },
+        [387564] = {
+            [196102] = {
+                true, true, true
+            }
+        },
+        [375596] = {
+            [196115] = {
+                true, false, false
+            },
+            [191164] = {
+                true, false, false
+            },
+
+        },
+        [386549] = {
+            [186741] = {
+                true, true, true
+            }
+        },
+        [386546] = {
+            [186741] = {
+                true, true, true
+            }
+        },
+        [389804] = {
+            [187154] = {
+                true, false, false
+            }
+        },
+        [377488] = {
+            [187155] = {
+                true, true, true
+            }
+        },
+        [377105] = {
+            [190510] = {
+                false, true, true
+            }
+        },
+        [373932] = {
+            [190187] = {
+                true, false, false
+            }
+        },
+        -- AA
+        [387910] = {
+            [196200] = {
+                false, true, true
+            }
+        },
+        [387975] = {
+            [196202] = {
+                true, true, true
+            }
+        },
+        [388863] = {
+            [196045] = {
+                true, true, true
+            }
+        },
+        [388392] = {
+            [196044] = {
+                true, true, true
+            }
+        },
+        [396812] = {
+            [196576] = {
+                true, true, true
+            }
+        },
+        [377389] = {
+            [192333] = {
+                true, false, false
+            }
+        },
+        [397888] = {
+            [200126] = {
+                true, true, true
+            }
+        },
+        [397801] = {
+            [56448] = {
+                true, false, false
+            }
+        },
+        [395859] = {
+            [59555] = {
+                true, true, true
+            }
+        },
+        [395872] = {
+            [59546] = {
+                true, false, false
+            }
+        },
+        [396018] = {
+            [59552] = {
+                true, false, false
+            }
+        },
+        [396073] = {
+            [59544] = {
+                true, true, false
+            }
+        },
+        [397899] = {
+            [200131] = {
+                false, true, true
+            }
+        },
+        [397914] = {
+            [200137] = {
+                true, true, true
+            }
+        },
+        -- sbg
+        [152818] = {
+            [75713] = {
+                true, true, false
+            }
+        },
+        [398154] = {
+            [75451] = {
+                false, true, true
+            }
+        },
+        [156776] = {
+            [76446] = {
+                true, true, true
+            }
+        },
+        [156772] = {
+            [77700] = {
+                true, false, false
+            }
+        },
+        [153524] = {
+            [75459] = {
+                true, true, true
+            }
+        },
+        [156718] = {
+            [76104] = {
+                true, false, false
+            }
+        },
+        [225100] = {
+            [104270] = {
+                true, false, false
+            }
+        },
+        [210261] = {
+            [104251] = {
+                true, true, true
+            }
+        },
+        [209027] = {
+            [104246] = {
+                false, true, true
+            }
+        },
+        [212031] = {
+            [105705] = {
+                false, true, false
+            }
+        },
+        [212784] = {
+            [105715] = {
+                false, true, false
+            }
+        },
+        [198585] = {
+            [95842] = {
+                true, true, true
+            }
+        },
+        [198959] = {
+            [96664] = {
+                true, true, true
+            }
+        },
+        [215433] = {
+            [95834] = {
+                true, true, true
+            }
+        },
+        [199210] = {
+            [96640] = {
+                false, true, true
+            }
+        },
+        [199090] = {
+            [96611] = {
+                false, true, true
+            }
+        },
+        [185425] = {
+            [96677] = {
+                false, true, false
+            }
+        },
     }
 
     Bastion.EventManager:RegisterWoWEvent('UNIT_AURA', function(unit, auras)
@@ -175,9 +461,21 @@ function MythicPlusUtils:CastingCriticalKick(unit, percent)
 
     if castingSpell then
         local spellID = castingSpell:GetID()
-        local isKick = unpack(self.kickList[spellID] or { false, false })
+        local kickEntry = self.kickList[spellID]
+        if not kickEntry then
+            return false
+        end
+
+        local npcTraits = kickEntry[unit:GetID()]
+
+        if not npcTraits then
+            return false
+        end
+
+        local isKick, isStun, isDisorient = unpack(npcTraits)
 
         if isKick and unit:IsInterruptibleAt(percent) then
+            print("I want to kick", unit:GetName(), unit:GetID() .. "'s " .. spellID)
             return true
         end
     end
@@ -193,8 +491,21 @@ function MythicPlusUtils:CastingCriticalStun(unit, percent)
 
     if castingSpell then
         local spellID = castingSpell:GetID()
-        local _, isStun = unpack(self.kickList[spellID] or { false, false })
+        local kickEntry = self.kickList[spellID]
-        if isStun and unit:IsInterruptibleAt(percent, true) then
+        if not kickEntry then
+            return false
+        end
+
+        local npcTraits = kickEntry[unit:GetID()]
+
+        if not npcTraits then
+            return false
+        end
+
+        local isKick, isStun, isDisorient = unpack(npcTraits)
+
+        if (isStun or isDisorient) and not isKick and unit:IsInterruptibleAt(percent, true) then
+            print("I want to stun", unit:GetName(), unit:GetID() .. "'s " .. spellID)
             return true
         end
     end

src/Sequencer/Sequencer.lua

@@ -0,0 +1,87 @@
+-- Create a sequencer class that takes a table of actions and executes them in order
+
+---@class Sequencer
+local Sequencer = {}
+Sequencer.__index = Sequencer
+
+-- Constructor
+---@param actions table
+---@return Sequencer
+function Sequencer:New(actions, resetCondition)
+    local self = setmetatable({}, Sequencer)
+
+    self.actions = actions
+    self.index = 1
+
+    self.resetCondition = resetCondition
+
+    return self
+end
+
+-- Should we reset the sequencer
+---@return boolean
+function Sequencer:ShouldReset()
+    if self.resetCondition then
+        return self.resetCondition()
+    end
+
+    return false
+end
+
+-- Should we abort the sequencer
+---@return boolean
+function Sequencer:ShouldAbort()
+    if self.abortCondition then
+        return self.abortCondition()
+    end
+
+    return false
+end
+
+-- Execute the next action in the sequence if it doesn't return true we need to try it again
+---@return boolean
+function Sequencer:Next()
+    if self:Finished() then
+        print("Its finished?")
+        return false
+    end
+
+    local action = self.actions[self.index]
+    print("Attempting action: " .. self.index .. "")
+    if action(self) then
+        self.index = self.index + 1
+        return true
+    end
+
+    return false
+end
+
+-- Reset the sequencer
+---@return nil
+function Sequencer:Reset()
+    self.index = 1
+end
+
+function Sequencer:Execute()
+    if self:Next() then
+        return true
+    end
+
+    return false
+end
+
+function Sequencer:Finished()
+    return self.index > #self.actions
+end
+
+function Sequencer:Abort()
+    self.index = #self.actions + 1
+end
+
+-- tostring
+---@return string
+function Sequencer:__tostring()
+    return "Bastion.__Sequencer"
+end
+
+return Sequencer

src/Spell/Spell.lua

@@ -7,9 +7,9 @@ local Spell = {
     PreCastFunc = false,
     OnCastFunc = false,
     PostCastFunc = false,
-    lastCastAttempt = 0,
+    lastCastAttempt = false,
     wasLooking = false,
-    lastCastAt = 0,
+    lastCastAt = false,
     conditions = {},
     target = false,
 }
@@ -119,6 +119,13 @@ function Spell:OnCooldown()
     return self:GetCooldownRemaining() > 0
 end
 
+-- Clear castable function
+---@return Spell
+function Spell:ClearCastableFunction()
+    self.CastableIfFunc = false
+    return self
+end
+
 -- Cast the spell
 ---@param unit Unit
 ---@param condition string
@@ -163,6 +170,42 @@ function Spell:Cast(unit, condition)
     return true
 end
 
+-- ForceCast the spell
+---@param unit Unit
+---@param condition string
+---@return boolean
+function Spell:ForceCast(unit)
+    -- Call pre cast function
+    -- if self:GetPreCastFunction() then
+    --     self:GetPreCastFunction()(self)
+    -- end
+
+    -- Check if the mouse was looking
+    self.wasLooking = IsMouselooking()
+
+    -- if unit:GetOMToken() contains 'nameplate' then we need to use Object wrapper to cast
+    local u = unit:GetOMToken()
+    if type(u) == "string" and string.find(u, 'nameplate') then
+        u = Object(u)
+    end
+
+    -- Cast the spell
+    CastSpellByName(self:GetName(), u)
+    SpellCancelQueuedSpell()
+
+    Bastion:Debug("Casting", self)
+
+    -- Set the last cast time
+    self.lastCastAttempt = GetTime()
+
+    -- -- Call post cast function
+    -- if self:GetOnCastFunction() then
+    --     self:GetOnCastFunction()(self)
+    -- end
+
+    return true
+end
+
 -- Get post cast func
 ---@return fun(self:Spell)
 function Spell:GetPostCastFunction()
@@ -187,7 +230,7 @@ end
 ---@return boolean
 function Spell:IsUsable()
     local usable, noMana = IsUsableSpell(self:GetID())
-    return usable or usableExcludes[self:GetID()]
+    return usable or usableExcludes[self:GetID()] and not noMana
 end
 
 -- Check if the spell is castable
@@ -321,6 +364,10 @@ function Spell:GetCharges()
     return GetSpellCharges(self:GetID())
 end
 
+function Spell:GetMaxCharges()
+    return select(2, GetSpellCharges(self:GetID()))
+end
+
 -- Get the spells charges
 ---@return number
 function Spell:GetChargesFractional()

src/Unit/Unit.lua

@@ -13,6 +13,9 @@ local Unit = {
     last_off_attack = 0,
     last_main_attack = 0,
     last_combat_time = 0,
+    ttd_ticker = 0,
+    ttd = 0,
+    id = false,
 }
 
 function Unit:__index(k)
@@ -449,6 +452,7 @@ end
 
 -- Check if unit is interruptible
 ---@param percent number
+---@param ignoreInterruptible boolean
 ---@return boolean
 function Unit:IsInterruptibleAt(percent, ignoreInterruptible)
     if not ignoreInterruptible and not self:IsInterruptible() then
@@ -626,6 +630,13 @@ function Unit:IsBehind(unit)
     return math.abs(angle) > 90
 end
 
+-- IsInfront
+---@param unit Unit
+---@return boolean
+function Unit:IsInfront(unit)
+    return not self:IsBehind(unit)
+end
+
 ---@return number
 function Unit:GetMeleeBoost()
     if IsPlayerSpell(196924) then
@@ -669,7 +680,9 @@ end
 -- Get object id
 ---@return number
 function Unit:GetID()
-    return ObjectID(self:GetOMToken())
+    if self.id then return self.id end
+    self.id = ObjectID(self:GetOMToken())
+    return self.id
 end
 
 -- In party
@@ -714,7 +727,7 @@ function Unit:PredictHealth(time)
     local x = {}
     local y = {}
 
-    if #self.regression_history > 20 then
+    if #self.regression_history > 60 then
         table.remove(self.regression_history, 1)
     end
 
@@ -737,7 +750,7 @@ function Unit:PredictTime(percent)
     local x = {}
     local y = {}
 
-    if #self.regression_history > 20 then
+    if #self.regression_history > 60 then
         table.remove(self.regression_history, 1)
     end
 
@@ -753,21 +766,50 @@ function Unit:PredictTime(percent)
     return (percent - intercept) / slope
 end
 
+-- Start time to die ticker
+function Unit:StartTTDTicker()
+    if self.ttd_ticker then
+        return
+    end
+
+    self.ttd_ticker = C_Timer.NewTicker(0.5, function()
+            local timeto = self:PredictTime(0) - GetTime()
+            self.ttd = timeto
+        end)
+end
+
 -- Time until death
 ---@return number
 function Unit:TimeToDie()
     if self:IsDead() then
         self.regression_history = {}
+        if self.ttd_ticker then
+            self.ttd_ticker:Cancel()
+            self.ttd_ticker = nil
+        end
+        return 0
+    end
+
+    if not self.ttd_ticker then
+        self:StartTTDTicker()
+    end
+
+    -- If there's not enough data to make a prediction return 0 unless the unit has more than 5 million health
+    if #self.regression_history < 5 and self:GetMaxHealth() < 5000000 then
         return 0
     end
 
-    local timeto = self:PredictTime(0) - GetTime()
+    -- if the unit has more than 5 million health but there's not enough data to make a prediction we can assume there's roughly 250000 damage per second and estimate the time to die
+    if #self.regression_history < 5 and self:GetMaxHealth() > 5000000 then
+        return self:GetMaxHealth() /
+            250000 -- 250000 is an estimate of the average damage per second a well geared group will average
+    end
 
-    if timeto ~= timeto or timeto < 0 or timeto == math.huge then
+    if self.ttd ~= self.ttd or self.ttd < 0 or self.ttd == math.huge then
         return 0
     end
 
-    return timeto
+    return self.ttd
 end
 
 -- Set combat time if affecting combat and return the difference between now and the last time
@@ -820,7 +862,12 @@ function Unit:GetMaxGCD()
         haste = 50
     end
 
-    return 1.5 / (1 + haste / 100)
+    -- if the unit uses focus their gcd is 1.0 seconds not 1.5
+    local base = 1.5
+    if self:GetPowerType() == 3 then
+        base = 1.0
+    end
+    return base / (1 + haste / 100)
 end
 
 -- IsStealthed

src/_bastion.lua

@@ -18,6 +18,8 @@ Bastion.List = Bastion.require("List")
 Bastion.NotificationsList, Bastion.Notification = Bastion.require("NotificationsList")
 ---@type Vector3
 Bastion.Vector3 = Bastion.require("Vector3")
+---@type Sequencer
+Bastion.Sequencer = Bastion.require("Sequencer")
 ---@type Command
 Bastion.Command = Bastion.require("Command")
 ---@type Cache
@@ -85,8 +87,24 @@ Bastion.EventManager:RegisterWoWEvent("UNIT_SPELLCAST_SUCCEEDED", function(...)
     end
 end)
 
+local pguid = UnitGUID("player")
+local missed = {}
+
 Bastion.EventManager:RegisterWoWEvent("COMBAT_LOG_EVENT_UNFILTERED", function()
-    local _, subtype, _, sourceGUID, sourceName, _, _, destGUID, destName, destFlags, _, spellID, spellName, _, amount, interrupt, a, b, c, d, offhand, multistrike = CombatLogGetCurrentEventInfo()
+    local args = { CombatLogGetCurrentEventInfo() }
+
+    local subEvent = args[2]
+    local sourceGUID = args[4]
+    local destGUID = args[8]
+    local spellID = args[12]
+
+    -- if sourceGUID == pguid then
+    --     local args = { CombatLogGetCurrentEventInfo() }
+
+    --     for i = 1, #args do
+    --         Log(tostring(args[i]))
+    --     end
+    -- end
 
     local u = Bastion.UnitManager[sourceGUID]
     local u2 = Bastion.UnitManager[destGUID]
@@ -99,23 +117,37 @@ Bastion.EventManager:RegisterWoWEvent("COMBAT_LOG_EVENT_UNFILTERED", function()
 
     if u2 then
         u2:SetLastCombatTime(t)
+
+        if subEvent == "SPELL_MISSED" and sourceGUID == pguid and spellID == 408 then
+            local missType = args[15]
+
+            if missType == "IMMUNE" then
+                local castingSpell = u:GetCastingOrChannelingSpell()
+
+                if castingSpell then
+                    if not missed[castingSpell:GetID()] then
+                        missed[castingSpell:GetID()] = true
+                    end
+                end
+            end
+        end
     end
 end)
 
 Bastion.Ticker = C_Timer.NewTicker(0.1, function()
-    if not Bastion.CombatTimer:IsRunning() and UnitAffectingCombat("player") then
+        if not Bastion.CombatTimer:IsRunning() and UnitAffectingCombat("player") then
-        Bastion.CombatTimer:Start()
+            Bastion.CombatTimer:Start()
-    elseif Bastion.CombatTimer:IsRunning() and not UnitAffectingCombat("player") then
+        elseif Bastion.CombatTimer:IsRunning() and not UnitAffectingCombat("player") then
-        Bastion.CombatTimer:Reset()
+            Bastion.CombatTimer:Reset()
-    end
+        end
 
-    if Bastion.Enabled then
+        if Bastion.Enabled then
-        Bastion.ObjectManager:Refresh()
+            Bastion.ObjectManager:Refresh()
-        for i = 1, #Bastion.modules do
+            for i = 1, #Bastion.modules do
-            Bastion.modules[i]:Tick()
+                Bastion.modules[i]:Tick()
+            end
         end
-    end
+    end)
-end)
 
 function Bastion:Register(module)
     table.insert(Bastion.modules, module)
@@ -228,11 +260,17 @@ Command:Register('mplus', 'Toggle m+ module on/off', function(args)
     Bastion:Print("casts")
 end)
 
+Command:Register('missed', 'Dump the list of immune kidney shot spells', function()
+    for k, v in pairs(missed) do
+        Bastion:Print(k)
+    end
+end)
+
 local files = ListFiles("scripts/bastion/scripts")
 
 for i = 1, #files do
     local file = files[i]
-    if file:sub(-4) == ".lua" or file:sub(-5) == '.luac' then
+    if file:sub( -4) == ".lua" or file:sub( -5) == '.luac' then
         Tinkr:require("scripts/bastion/scripts/" .. file:sub(1, -5), Bastion)
     end
 end