Before You Begin

• I confirm that I have downloaded the latest version of the addon.
• I am not playing on a private server.
• I checked for an existing, open ticket for this issue and was not able to find one.
• I edited the title of this issue (above) so that it describes the issue I am reporting.
• I am reporting an issue with the default priority included with the specialization (imported or edited priorities are not supported).

Spec

Mage - Fire

Describe the Issue

As stated in various guides, including this https://www.icy-veins.com/wow/fire-mage-pve-dps-rotation-cooldowns-abilities?area=area_2 the wrist should be used after consuming at least 2 fire blast, but by the way it is currently suggested it is not refreshing anything.
Another issue with fire is when combustion is coming back. As shown in the snapshot, even when combustion is at around 20s to come back, the addon only recommends fireball while in this case we could use fireblasts in the meantime. Usually the pooling phase starts at around 10s from combustion

How to Reproduce

1 enter game as fire mage
2 test on dummies
3 execute rotation

Snapshot (Link)

wrist: https://pastebin.com/Ed7bPrLG comb filler: https://pastes.io/comb-82275

Raidbots Sim Report (Link)

https://www.raidbots.com/simbot/report/xk1uXC14GnxKddctJYAG2J

Additional Information

No response

Contact Information

yokaii__

Also found another issue when we have shifting power coming up but the addon doesn't recommend any fireblast or phoenix flames that will get refreshed by it anyway https://pastes.io/time-11054

The sims skip over hyperthread wrists for the generic "use item" entry but we don't, we'll have to fix that to solve it.

The sims skip over hyperthread wrists for the generic "use item" entry but we don't, we'll have to fix that to solve it.

I don't have the item, so it's hard for me to do testing. More likely than not, this is an issue that the spell ID is set in BfA/Items.lua, which might override how items are loaded.

Revising the entry as follows may work:

    -- Hyperthread Wristwraps
    all:RegisterAbility( "hyperthread_wristwraps", {
        -- id = 300142,
        cast = 0,
        cooldown = 120,
        gcd = "off",
        equipped = "hyperthread_wristwraps",

        item = 168989,
        toggle = "cooldowns",

        handler = function ()
            -- Gain 5 seconds of CD for the last 3 spells.
            for i = 1, 3 do
                local ability = prev[i].spell

                if ability and ability ~= "no_action" then
                    gainChargeTime( ability, 5 )
                end
            end
        end,

        copy = { 300142, "hyperthread_wristwraps_300142" }
    } )

If that doesn't fix it, I'd copy the handler and re-implement the item so it is exactly consistent with other modern trinkets and see if that works.

It might also be useful to see what is shown from:

• /dump Hekili.Scripts.PackInfo.Fire.items
• /dump Hekili.Class.abilities.hyperthread_wristwraps.item