From 2130be3d8ce4420e09808773b01dca1881f7d97e Mon Sep 17 00:00:00 2001
From: dlamanna <phuze3@gmail.com>
Date: Fri, 6 Mar 2026 22:37:30 -0800
Subject: [PATCH 1/2] Add Zygor Guides item score integration

Adds a SupportedAddOn integration for ZygorGuidesViewer that overrides
DialogueUI's upgrade detection and tooltip display with Zygor's
stat-weight-based item scoring system.

- Override IsItemAnUpgrade_External to use Zygor's Upgrades:IsUpgrade()
  for more accurate upgrade arrows on quest reward items
- Override ProcessItemExternal on SharedTooltip to show Zygor ItemScore
  upgrade/downgrade percentages per equipment slot
- Handle deferred item data by signaling "not ready" so DialogueUI retries
- Follow the same AddSupportedAddOn pattern used by Pawn integration

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
---
 Code/SupportedAddOns/SupportedAddOns.xml |   1 +
 Code/SupportedAddOns/Zygor.lua           | 121 +++++++++++++++++++++++
 2 files changed, 122 insertions(+)
 create mode 100644 Code/SupportedAddOns/Zygor.lua

diff --git a/Code/SupportedAddOns/SupportedAddOns.xml b/Code/SupportedAddOns/SupportedAddOns.xml
index 6d4dfa0..7e8865a 100644
--- a/Code/SupportedAddOns/SupportedAddOns.xml
+++ b/Code/SupportedAddOns/SupportedAddOns.xml
@@ -7,6 +7,7 @@
     <Script file="Wholly.lua"/>
     <Script file="Roleplay.lua"/>
     <Script file="Pawn.lua"/>
+    <Script file="Zygor.lua"/>
     <Script file="InteractiveWormholes.lua"/>
     <Script file="Rematch.lua"/>
     <Script file="BtWQuests.lua"/>
diff --git a/Code/SupportedAddOns/Zygor.lua b/Code/SupportedAddOns/Zygor.lua
new file mode 100644
index 0000000..f8d218b
--- /dev/null
+++ b/Code/SupportedAddOns/Zygor.lua
@@ -0,0 +1,121 @@
+-- Zygor Guides Item Score Integration
+local _, addon = ...
+local API = addon.API;
+
+
+do
+    local ADDON_NAME = "ZygorGuidesViewer";
+
+    local requiredMethods = {
+        "ZGV";
+    };
+
+    local function OnAddOnLoaded()
+        local ZGV = _G.ZGV;
+        if not (ZGV and ZGV.ItemScore and ZGV.ItemScore.Upgrades) then return end;
+
+        local ItemScore = ZGV.ItemScore;
+        local Upgrades = ItemScore.Upgrades;
+        local tooltip = addon.SharedTooltip;
+        local floor = math.floor;
+
+        local function get_change(old, new)
+            if old and old > 0 then
+                return floor(((new * 100 / old) - 100) * 100) / 100
+            else
+                return 100
+            end
+        end
+
+        -- Override upgrade detection for reward item arrows
+        API.IsItemAnUpgrade_External = function(itemLink)
+            if not ItemScore.ActiveRuleSet then
+                return API.IsItemAnUpgrade(itemLink)
+            end
+            local isUpgrade, slot, change, score, comment = Upgrades:IsUpgrade(itemLink);
+            if comment == "not scored" or comment == "no link" then
+                return nil, false
+            end
+            return isUpgrade, true
+        end
+
+        -- Append a single slot's upgrade/downgrade line to the tooltip.
+        local function addSlotLine(self, slotLabel, futurePrefix, stripped, equipped, score)
+            if equipped and (equipped.score or score) then
+                if stripped ~= equipped.itemlink then
+                    local change;
+                    if equipped.score and ItemScore:IsValidItem(equipped.itemlink) then
+                        change = get_change(equipped.artifactscore or equipped.score, score);
+                    else
+                        change = 100;
+                    end
+                    local pct = (change > 999) and "999+" or tostring(change);
+                    if change > 0 then
+                        self:AddLeftLine("  " .. slotLabel .. futurePrefix .. "Upgrade: +" .. pct .. "%", 0, 1, 0, false, nil, 2);
+                    elseif change < 0 then
+                        self:AddLeftLine("  " .. slotLabel .. futurePrefix .. "Downgrade: " .. pct .. "%", 1, 0, 0, false, nil, 2);
+                    else
+                        self:AddLeftLine("  " .. slotLabel .. "No change", 0.6, 0.6, 0.6, false, nil, 2);
+                    end
+                else
+                    self:AddLeftLine("  " .. slotLabel .. "Equipped", 0.6, 0.6, 0.6, false, nil, 2);
+                end
+            elseif (score or 0) > 0 then
+                self:AddLeftLine("  " .. slotLabel .. futurePrefix .. "Upgrade: +100%", 0, 1, 0, false, nil, 2);
+            end
+        end
+
+        -- Add Zygor score info to reward item tooltip
+        function tooltip:ProcessItemExternal(itemLink)
+            if not ItemScore.ActiveRuleSet then return end;
+
+            local item = ItemScore:GetItemDetails(itemLink, "temporary");
+            if not item then return end;
+            if item.type == "INVTYPE_NON_EQUIP_IGNORE" then return end;
+
+            local score, success = ItemScore:GetItemScore(item.itemlink);
+            if not success then return end;
+
+            local valid, validfuture, final = ItemScore:IsValidItem(item.itemlink, "future");
+            if not final then return end;
+
+            local spec = ItemScore.ActiveRuleSet.specname;
+            local specText = spec and (" (" .. spec .. ")") or "";
+
+            self:AddBlankLine();
+            self:AddLeftLine("Zygor ItemScore" .. specText .. ":", 254/255, 97/255, 0, true);
+
+            if valid or validfuture then
+                local futurePrefix = (not valid and validfuture) and "Future " or "";
+                local slot_1, slot_2 = item.slot, item.slot_2;
+                local hasTwoSlots = slot_2 ~= nil;
+                local stripped = item.itemlink;
+
+                addSlotLine(self,
+                    hasTwoSlots and "Slot 1: " or "",
+                    futurePrefix, stripped,
+                    slot_1 and Upgrades.EquippedItems[slot_1],
+                    score);
+
+                if hasTwoSlots then
+                    addSlotLine(self,
+                        "Slot 2: ",
+                        futurePrefix, stripped,
+                        Upgrades.EquippedItems[slot_2],
+                        score);
+                end
+            else
+                if item.type == "INVTYPE_TRINKET" then
+                    self:AddLeftLine("  Trinkets are not part of the scoring system.", 0.6, 0.6, 0.6, true, nil, 2);
+                else
+                    local specName = ItemScore.playerspecName or (ItemScore.ActiveRuleSet and ItemScore.ActiveRuleSet.specname) or "current spec";
+                    self:AddLeftLine("  Not valid for " .. specName, 1, 0, 0, false, nil, 2);
+                end
+            end
+
+            self:Show();
+        end
+    end
+
+    addon.AddSupportedAddOn(ADDON_NAME, OnAddOnLoaded, requiredMethods);
+end
\ No newline at end of file

From 3e33b3afe3006fd84caf62df794ad86057a3afb2 Mon Sep 17 00:00:00 2001
From: dlamanna <phuze3@gmail.com>
Date: Sun, 8 Mar 2026 03:23:45 -0700
Subject: [PATCH 2/2] feat(zygor): add ilvl-based trinket fallback for unscored
 trinkets

Zygor does not score trinkets, so they previously showed no upgrade
arrow and a gray "not part of scoring system" tooltip message. Now
when Zygor returns "not scored" for a trinket, we fall back to
item-level comparison against both equipped trinket slots.

- Upgrade arrow: delegates to built-in API.IsItemAnUpgrade for trinkets
- Tooltip: shows per-slot ilvl deltas (Trinket 1/2) with color-coded
  upgrade/downgrade/no-change lines and empty-slot handling
- Fails soft if ilvl data isn't available yet

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
---
 Code/SupportedAddOns/Zygor.lua | 35 +++++++++++++++++++++++++++++++++-
 1 file changed, 34 insertions(+), 1 deletion(-)

diff --git a/Code/SupportedAddOns/Zygor.lua b/Code/SupportedAddOns/Zygor.lua
index f8d218b..8268884 100644
--- a/Code/SupportedAddOns/Zygor.lua
+++ b/Code/SupportedAddOns/Zygor.lua
@@ -34,6 +34,13 @@ do
             end
             local isUpgrade, slot, change, score, comment = Upgrades:IsUpgrade(itemLink);
             if comment == "not scored" or comment == "no link" then
+                -- For unscored trinkets, fall back to ilvl-based comparison.
+                -- Zygor does not score trinkets, but ilvl is a useful proxy.
+                -- This does not imply full stat/effect evaluation.
+                local itemDetails = ItemScore:GetItemDetails(itemLink, "temporary");
+                if itemDetails and itemDetails.type == "INVTYPE_TRINKET" then
+                    return API.IsItemAnUpgrade(itemLink)
+                end
                 return nil, false
             end
             return isUpgrade, true
@@ -106,7 +113,33 @@ do
                 end
             else
                 if item.type == "INVTYPE_TRINKET" then
-                    self:AddLeftLine("  Trinkets are not part of the scoring system.", 0.6, 0.6, 0.6, true, nil, 2);
+                    -- Zygor does not score trinkets; fall back to item level comparison.
+                    -- This does not imply full stat/effect evaluation.
+                    local newIlvl = API.GetItemLevel(itemLink);
+                    if newIlvl and newIlvl > 0 then
+                        self:AddLeftLine("  Trinket comparison (item level):", 0.6, 0.6, 0.6, true, nil, 2);
+                        for i, slotID in ipairs({13, 14}) do
+                            local label = "  Trinket " .. i .. ": ";
+                            local equippedLink = GetInventoryItemLink("player", slotID);
+                            if equippedLink then
+                                local equippedIlvl = API.GetItemLevel(equippedLink);
+                                if equippedIlvl and equippedIlvl > 0 then
+                                    local delta = newIlvl - equippedIlvl;
+                                    if delta > 0 then
+                                        self:AddLeftLine(label .. "Upgrade: +" .. delta .. " ilvl", 0, 1, 0, false, nil, 2);
+                                    elseif delta < 0 then
+                                        self:AddLeftLine(label .. "Downgrade: " .. delta .. " ilvl", 1, 0, 0, false, nil, 2);
+                                    else
+                                        self:AddLeftLine(label .. "No change", 0.6, 0.6, 0.6, false, nil, 2);
+                                    end
+                                end
+                            else
+                                self:AddLeftLine(label .. "Upgrade: empty slot", 0, 1, 0, false, nil, 2);
+                            end
+                        end
+                    else
+                        self:AddLeftLine("  Trinkets are not part of the scoring system.", 0.6, 0.6, 0.6, true, nil, 2);
+                    end
                 else
                     local specName = ItemScore.playerspecName or (ItemScore.ActiveRuleSet and ItemScore.ActiveRuleSet.specname) or "current spec";
                     self:AddLeftLine("  Not valid for " .. specName, 1, 0, 0, false, nil, 2);