Extend ClassicAPI bridge for Extras

api.lua

@@ -44,6 +44,9 @@ API.inventory = type(_G.C_Container) == "table"
 API.containeritems = type(_G.C_Container) == "table"
   and type(_G.C_Container.GetContainerItemID) == "function"
 
+API.containeriteminfo = type(_G.C_Container) == "table"
+  and type(_G.C_Container.GetContainerItemInfo) == "function"
+
 API.items = type(_G.C_Item) == "table"
 API.iteminfo = API.items and type(_G.C_Item.GetItemInfo) == "function"
 API.itemname = API.items and type(_G.C_Item.GetItemNameByID) == "function"
@@ -74,6 +77,13 @@ API.playerstate = type(_G.IsMounted) == "function"
   and type(_G.CancelShapeshiftForm) == "function"
 
 API.unitguid = type(_G.UnitGUID) == "function"
+API.unitrange = type(_G.UnitInRange) == "function"
+
+API.macrospell = type(_G.GetMacroSpell) == "function"
+API.attackverbs = type(_G.StartAttack) == "function"
+  and type(_G.StopAttack) == "function"
+API.focus = type(_G.FocusUnit) == "function"
+  and type(_G.ClearFocus) == "function"
 
 API.chatidentity = type(_G.GetCurrentChatGUID) == "function"
   and type(_G.GetPlayerInfoByGUID) == "function"
@@ -153,6 +163,36 @@ API.GetActionInfo = function(slot)
   end
 end
 
+API.GetMacroSpell = function(slot)
+  if API.macrospell then
+    return _G.GetMacroSpell(slot)
+  end
+end
+
+API.StartAttack = function(unit)
+  if API.attackverbs then
+    return _G.StartAttack(unit)
+  end
+end
+
+API.StopAttack = function()
+  if API.attackverbs then
+    return _G.StopAttack()
+  end
+end
+
+API.FocusUnit = function(unit)
+  if API.focus then
+    return _G.FocusUnit(unit)
+  end
+end
+
+API.ClearFocus = function()
+  if API.focus then
+    return _G.ClearFocus()
+  end
+end
+
 API.IsAutoAttackSpell = function(spellID)
   return API.autoattack and _G.C_Spell.IsAutoAttackSpell(spellID) or false
 end
@@ -191,6 +231,20 @@ API.UnitDebuff = function(unit, index, filter)
   end
 end
 
+-- Normalize the debuff type across ClassicAPI's Classic-Era-shaped
+-- positional return and Vanilla's legacy three-value UnitDebuff return.
+API.GetDebuffType = function(unit, index, filter)
+  if API.aurapositional then
+    local _, _, _, debuffType = _G.C_UnitAuras.UnitDebuff(unit, index, filter)
+    return debuffType
+  end
+
+  if type(_G.UnitDebuff) == "function" then
+    local _, _, debuffType = _G.UnitDebuff(unit, index)
+    return debuffType
+  end
+end
+
 API.GetCurrentChatGUID = function()
   if API.chatidentity then
     return _G.GetCurrentChatGUID()
@@ -273,13 +327,36 @@ end
 
 API.GetItemIDFromLink = GetItemIDFromLink
 
+API.GetContainerItemLink = function(bag, slot)
+  if API.containeriteminfo then
+    local info = _G.C_Container.GetContainerItemInfo(bag, slot)
+    if info and info.hyperlink then
+      return info.hyperlink
+    end
+  end
+
+  return _G.GetContainerItemLink(bag, slot)
+end
+
+API.GetContainerItemStackCount = function(bag, slot)
+  if API.containeriteminfo then
+    local info = _G.C_Container.GetContainerItemInfo(bag, slot)
+    if info and info.stackCount then
+      return info.stackCount
+    end
+  end
+
+  local _, count = _G.GetContainerItemInfo(bag, slot)
+  return count or 0
+end
+
 API.GetContainerItemID = function(bag, slot)
   if API.containeritems then
     local itemID = _G.C_Container.GetContainerItemID(bag, slot)
     if itemID then return itemID end
   end
 
-  return GetItemIDFromLink(_G.GetContainerItemLink(bag, slot))
+  return GetItemIDFromLink(API.GetContainerItemLink(bag, slot))
 end
 
 API.GetInventoryItemID = function(unit, slot)
@@ -380,3 +457,29 @@ API.UnitGUID = function(unit)
     return _G.UnitGUID(unit)
   end
 end
+
+-- Prefer ClassicAPI's reach-aware 40-yard UnitInRange. The wrapper handles
+-- the player's own frame explicitly and preserves a Vanilla interaction check
+-- only as a centralized compatibility fallback.
+API.UnitInRange = function(unit)
+  if not unit then return false, false end
+
+  if type(_G.UnitIsUnit) == "function" and _G.UnitIsUnit(unit, "player") then
+    return true, true
+  end
+
+  if API.unitrange then
+    local inRange, checked = _G.UnitInRange(unit)
+    if checked then
+      return inRange and true or false, true
+    end
+  end
+
+  if type(_G.UnitExists) == "function"
+    and type(_G.CheckInteractDistance) == "function"
+    and _G.UnitExists(unit) then
+    return _G.CheckInteractDistance(unit, 4) and true or false, true
+  end
+
+  return false, false
+end

helpers.lua

@@ -359,21 +359,14 @@ ShaguTweaks.GetItemCount = function(itemName)
   local count = 0
   for bag = 4, 0, -1 do
     for slot = 1, GetContainerNumSlots(bag) do
-      local _, itemCount = GetContainerItemInfo(bag, slot)
+      local itemCount = API and API.GetContainerItemStackCount
-      if itemCount then
+        and API.GetContainerItemStackCount(bag, slot) or 0
-        local itemID
-        if API and API.GetContainerItemID then
-          itemID = API.GetContainerItemID(bag, slot)
-        else
-          itemID = LegacyItemIDFromLink(GetContainerItemLink(bag, slot))
-        end
 
-        local queryName
+      if itemCount > 0 then
-        if itemID and API and API.GetItemNameByID then
+        local itemID = API and API.GetContainerItemID
-          queryName = API.GetItemNameByID(itemID)
+          and API.GetContainerItemID(bag, slot)
-        elseif itemID then
+        local queryName = itemID and API and API.GetItemNameByID
-          queryName = GetItemInfo(itemID)
+          and API.GetItemNameByID(itemID)
-        end
 
         if queryName == itemName then
           count = count + itemCount
@@ -401,10 +394,11 @@ local function FindOwnedItemLinkByName(name)
   -- works for Turtle WoW custom item IDs beyond vanilla's original range.
   for bag = 0, 4 do
     for slot = 1, GetContainerNumSlots(bag) do
-      local link = GetContainerItemLink(bag, slot)
+      local link = API and API.GetContainerItemLink
+        and API.GetContainerItemLink(bag, slot)
       if link then
-        local itemID = API and API.GetContainerItemID and API.GetContainerItemID(bag, slot)
+        local itemID = API and API.GetContainerItemID
-          or LegacyItemIDFromLink(link)
+          and API.GetContainerItemID(bag, slot)
         local itemName = itemID and API and API.GetItemNameByID and API.GetItemNameByID(itemID)
         if not itemName then
           local _, _, linkedName = string.find(link, "%[(.-)%]")
@@ -443,8 +437,15 @@ ShaguTweaks.GetItemLinkByName = function(name)
 
   -- Vanilla has no generic name -> itemID resolver. Keep the original base-game
   -- fallback for non-owned items, but custom owned items no longer depend on it.
+  local API = ShaguTweaks.API
   for itemID = 1, 25818 do
-    local itemName, itemLink, itemQuality = GetItemInfo(itemID)
+    local itemName, itemLink, itemQuality
+    if API and API.GetItemInfo then
+      itemName, itemLink, itemQuality = API.GetItemInfo(itemID)
+    else
+      itemName, itemLink, itemQuality = GetItemInfo(itemID)
+    end
+
     if itemName and itemName == name then
       local _, _, _, hex = GetItemQualityColor(tonumber(itemQuality))
       local hyperLink = hex.. "|H".. itemLink .."|h["..itemName.."]|h" .. FONT_COLOR_CODE_CLOSE