Merge pull request 'update' (#2) from Bastion/Bastion:main into main

Reviewed-on: #2

scripts/restodruid.lua

@@ -160,7 +160,6 @@ local DispelTarget = Bastion.UnitManager:CreateCustomUnit('dispel', function(uni
 
             if not unit:IsDead() and Player:CanSee(unit) and
                 unit:GetAuras():HasAnyDispelableAura(NaturesCure) then
-
                 local hp = unit:GetHP()
                 if hp < lowestHP then
                     lowest = unit
@@ -258,14 +257,12 @@ local RejuvUnit = Bastion.UnitManager:CreateCustomUnit('rejuv', function(unit)
                 (
                 not unit:GetAuras():FindMy(Rejuvenation):IsUp() or
                 unit:GetAuras():FindMy(Rejuvenation):GetRemainingTime() <= 3.6) then
-
                 local hp = unit:GetHP()
                 if hp < lowestHP then
                     lowest = unit
                     lowestHP = hp
                 end
             end
-
         end)
 
 
@@ -302,14 +299,12 @@ local SwiftmendUnit = Bastion.UnitManager:CreateCustomUnit('swiftmend', function
                 not unit:GetAuras():FindMy(WildGrowth):IsUp())
                 )
                 ) then
-
                 local hp = unit:GetHP()
                 if hp < lowestHP then
                     lowest = unit
                     lowestHP = hp
                 end
             end
-
         end)
 
 
@@ -351,7 +346,6 @@ local WildGrowthUnit = Bastion.UnitManager:CreateCustomUnit('wildgrowth', functi
                     lowestHP = hp
                 end
             end
-
         end)
 
 
@@ -410,7 +404,6 @@ local RakeTarget = Bastion.UnitManager:CreateCustomUnit('rake', function(unit)
                 unit:GetAuras():FindMy(RakeDebuff):GetRemainingTime() <= 3.6) then
                 rakeTarget = unit
             end
-
         end)
 
 
@@ -443,7 +436,6 @@ local MoonfireTarget = Bastion.UnitManager:CreateCustomUnit('moonfire', function
                 unit:GetAuras():FindMy(MoonfireAura):GetRemainingTime() <= 3.6) then
                 moonfireTarget = unit
             end
-
         end)
 
         if moonfireTarget == nil then
@@ -475,7 +467,6 @@ local SunfireTarget = Bastion.UnitManager:CreateCustomUnit('sunfire', function(u
                 unit:GetAuras():FindMy(SunfireAura):GetRemainingTime() <= 3.6) then
                 sunfireTarget = unit
             end
-
         end)
 
         if sunfireTarget == nil then
@@ -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)
@@ -714,6 +700,13 @@ 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()

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,7 +461,18 @@ 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
             return true
@@ -193,8 +490,20 @@ 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
             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
 
-    local timeto = self:PredictTime(0) - GetTime()
+    if not self.ttd_ticker then
+        self:StartTTDTicker()
+    end
 
-    if timeto ~= timeto or timeto < 0 or timeto == math.huge then
+    -- 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
 
-    return timeto
+    -- 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 self.ttd ~= self.ttd or self.ttd < 0 or self.ttd == math.huge then
+        return 0
+    end
+
+    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,6 +117,20 @@ 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)
 
@@ -228,6 +260,12 @@ 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