ArcUI_Options.lua

-- ===================================================================
-- ArcUI_Options.lua  
-- Main Options registration for Arc UI
-- v3.4.2: Fixed OpenOptions nil error and added AceDB error handling
-- ===================================================================

local ADDON, ns = ...
ns.Options = ns.Options or {}

local AceConfig = LibStub("AceConfig-3.0")
local AceConfigDialog = LibStub("AceConfigDialog-3.0")
local AceDB = LibStub("AceDB-3.0")
local LSM = LibStub("LibSharedMedia-3.0", true)

local function GetGlobalFontValues()
  local fonts = { ["Friz Quadrata TT"] = "Friz Quadrata TT" }
  if LSM then
    for _, name in pairs(LSM:List("font")) do
      fonts[name] = name
    end
  end
  return fonts
end

local function GetGlobalStatusBarTextureValues()
  local textures = { ["Blizzard"] = "Blizzard", ["Smooth"] = "Smooth" }
  if LSM then
    for _, name in pairs(LSM:List("statusbar")) do
      textures[name] = name
    end
  end
  return textures
end

-- Profile browser collapsed state (defaults closed)
local profileBrowserCollapsed = true

-- ===================================================================
-- RELIABLE PANEL OPEN/CLOSE DETECTION
-- Hook AceConfigDialog:Open and :Close directly - much more reliable
-- than trying to hook frame OnShow/OnHide
-- ===================================================================
if not AceConfigDialog._arcUIHooked then
    AceConfigDialog._arcUIHooked = true
    
    -- Hook Close
    hooksecurefunc(AceConfigDialog, "Close", function(self, appName)
        if appName == "ArcUI" then
            ns._arcUIOptionsOpen = false
            -- Re-evaluate aura textures (hide previews, disable movers)
            if ns.Textures and ns.Textures.OnOptionsClosed then
                ns.Textures.OnOptionsClosed()
            end
            -- Make cooldown bars click-through again
            if ns.CooldownBars and ns.CooldownBars.RefreshMouseInteractivity then
                ns.CooldownBars.RefreshMouseInteractivity()
            end
            -- Fire registered panel callbacks (ArcAurasCooldown, SpellUsability, etc.)
            if ns.CDMShared and ns.CDMShared.FirePanelCallbacks then
                ns.CDMShared.FirePanelCallbacks(false)
            end
            if ns.CDMGroups and ns.CDMGroups.DynamicLayout and ns.CDMGroups.DynamicLayout.OnOptionsPanelClosed then
                ns.CDMGroups.DynamicLayout.OnOptionsPanelClosed()
            end
            -- IMMEDIATE: All panel-close logic (reflow, click-through, visuals)
            if ns.CDMGroups and ns.CDMGroups.OnArcUIPanelChanged then
                ns.CDMGroups.OnArcUIPanelChanged(false)
            end
        end
    end)
    
    -- Hook Open (backup - we also call directly in OpenOptions)
    hooksecurefunc(AceConfigDialog, "Open", function(self, appName)
        if appName == "ArcUI" then
            local wasOpen = ns._arcUIOptionsOpen
            ns._arcUIOptionsOpen = true
            -- Only fire layout/restore callbacks on actual open (closed → open).
            -- NotifyChange causes AceConfig to call Open again while already open,
            -- which would re-run Layout+RestoreIconsToSavedPositions on every UI rebuild.
            if not wasOpen then
                -- Fire registered panel callbacks (ArcAurasCooldown, SpellUsability, etc.)
                if ns.CDMShared and ns.CDMShared.FirePanelCallbacks then
                    ns.CDMShared.FirePanelCallbacks(true)
                end
                if ns.CDMGroups and ns.CDMGroups.DynamicLayout and ns.CDMGroups.DynamicLayout.OnOptionsPanelOpened then
                    ns.CDMGroups.DynamicLayout.OnOptionsPanelOpened()
                end
                -- IMMEDIATE: All panel-open logic (borders, scan, drag, visuals)
                if ns.CDMGroups and ns.CDMGroups.OnArcUIPanelChanged then
                    ns.CDMGroups.OnArcUIPanelChanged(true)
                end
            end
        end
    end)
end

-- ===================================================================
-- ADDON INFO
-- ===================================================================
-- Get version from TOC file (auto-updates when TOC changes)
local function GetAddonVersion()
  if C_AddOns and C_AddOns.GetAddOnMetadata then
    return C_AddOns.GetAddOnMetadata(ADDON, "Version") or "Unknown"
  elseif GetAddOnMetadata then
    return GetAddOnMetadata(ADDON, "Version") or "Unknown"
  end
  return "Unknown"
end

ns.AddonInfo = {
  Version = GetAddonVersion(),
  Discord = "https://discord.gg/yMZmnFjUTd",
  Author = "Arc",
}

-- ===================================================================
-- DISCORD LINK BUTTON (must be defined before OpenOptions)
-- ===================================================================
local function CreateDiscordLink(parentFrame)
  if parentFrame._arcUIDiscordLink then return end
  
  local container = CreateFrame("Frame", nil, parentFrame)
  container:SetSize(200, 20)
  container:SetPoint("TOPRIGHT", parentFrame, "TOPRIGHT", -10, -8)
  
  local label = container:CreateFontString(nil, "OVERLAY", "GameFontNormal")
  label:SetPoint("RIGHT", container, "RIGHT", 0, 0)
  label:SetText("|cff5865F2Discord:|r |cff7289DA" .. ns.AddonInfo.Discord .. "|r")
  
  local link = CreateFrame("EditBox", nil, container)
  link:SetSize(200, 20)
  link:SetPoint("RIGHT", container, "RIGHT", 0, 0)
  link:SetFontObject(GameFontNormal)
  link:SetAutoFocus(false)
  link:EnableMouse(true)
  link:SetText(ns.AddonInfo.Discord)
  link:SetCursorPosition(0)
  link:Hide()
  
  container:EnableMouse(true)
  container:SetScript("OnEnter", function(self)
    label:SetText("|cff5865F2Discord:|r |cffffffffClick to copy|r")
    GameTooltip:SetOwner(self, "ANCHOR_BOTTOM")
    GameTooltip:AddLine("Click to copy Discord link", 1, 1, 1)
    GameTooltip:Show()
  end)
  container:SetScript("OnLeave", function(self)
    label:SetText("|cff5865F2Discord:|r |cff7289DA" .. ns.AddonInfo.Discord .. "|r")
    GameTooltip:Hide()
  end)
  container:SetScript("OnMouseDown", function(self)
    link:Show()
    link:SetFocus()
    link:HighlightText()
    label:Hide()
  end)
  
  link:SetScript("OnEditFocusLost", function(self)
    self:Hide()
    label:Show()
  end)
  link:SetScript("OnEscapePressed", function(self)
    self:ClearFocus()
  end)
  link:SetScript("OnEnterPressed", function(self)
    self:ClearFocus()
  end)
  
  parentFrame._arcUIDiscordLink = container
end

-- ===================================================================
-- EARLY OPENOPTIONS DEFINITION (before PLAYER_LOGIN)
-- This ensures /arcui always has a function to call, even if DB fails
-- ===================================================================
local optionsRegistered = false

ns.API = ns.API or {}

-- Define OpenOptions early - will be replaced with full version after registration
ns.API.OpenOptions = function()
  if not optionsRegistered then
    print("|cff00ccffArc UI|r Options not ready yet. Try again in a moment.")
    print("|cff00ccffArc UI|r If this persists, type: /arcui reset-db")
    return
  end
  
  if InCombatLockdown() then
    ns._arcPendingOptionsOpen = true
    print("|cff00ccffArc UI|r Options will open when combat ends.")
    return
  end
  
  ns._arcPendingOptionsOpen = nil
  ns._arcUIOptionsOpen = true  -- Flag for Resources module to detect options are open

  -- Restore saved position/size into AceConfig's own status table BEFORE Open
  -- AceConfig reads status.top/left/width/height inside Open via SetStatusTable
  do
    local globalDB = ns.API.GetGlobalDB and ns.API.GetGlobalDB()
    local pos  = globalDB and globalDB.optionsPanelPos
    local size = globalDB and globalDB.optionsPanelSize
    if pos or size then
      local status = AceConfigDialog:GetStatusTable("ArcUI")
      if status then
        if pos  then status.top  = pos.top;  status.left   = pos.left   end
        if size then status.width = size.width; status.height = size.height end
        -- Clamp to screen so saved positions from different resolutions don't go offscreen
        local sw, sh = GetScreenWidth(), GetScreenHeight()
        local w = status.width  or 900
        local h = status.height or 700
        if status.top  then status.top  = math.max(h,       math.min(status.top,  sh)) end
        if status.left then status.left = math.max(0,       math.min(status.left, sw - w)) end
        -- Also clamp size to screen
        if w > sw     then status.width  = sw      end
        if h > sh - 50 then status.height = sh - 50 end
      end
    end
  end

  AceConfigDialog:Open("ArcUI")
  -- CDM_Shared's ACD:Open posthook sets ns.optionsPanelOpen and fires all callbacks

  -- First time these options are opened on a build that HAS a tour, ask once
  -- whether they want it. Deferred a frame so it lands over a drawn panel; the
  -- answer (either way) is remembered per patch, so it never asks twice.
  if ns.Tour and ns.Tour.OfferIfNew then
    C_Timer.After(0.35, function() ns.Tour.OfferIfNew() end)
  end

  -- Refresh resource bars immediately so they show despite talent/spec/combat conditions
  if ns.Resources and ns.Resources.RefreshAllBars then
    ns.Resources.RefreshAllBars()
  end

  -- Refresh aura textures so they preview (and become draggable) while open
  if ns.Textures and ns.Textures.RefreshAll then
    ns.Textures.RefreshAll()
  end

  -- Cooldown bars become draggable while open; click-through when closed
  if ns.CooldownBars and ns.CooldownBars.RefreshMouseInteractivity then
    ns.CooldownBars.RefreshMouseInteractivity()
  end

  -- Show "Hidden by Bar" overlays on CDM icons that are being hidden
  C_Timer.After(0.1, function()
    if ns.API.ShowHiddenByBarOverlays then
      ns.API.ShowHiddenByBarOverlays()
    end
  end)
  
  C_Timer.After(0.05, function()
    local widget = AceConfigDialog.OpenFrames["ArcUI"]
    if widget and widget.frame then
      local actualFrame = widget.frame
      local globalDB = ns.API.GetGlobalDB and ns.API.GetGlobalDB()
      local alpha = globalDB and globalDB.menuBackgroundAlpha or 1.0
      
      -- Create solid background
      if not actualFrame._arcUISolidBgFrame then
        actualFrame._arcUISolidBgFrame = CreateFrame("Frame", nil, actualFrame)
        actualFrame._arcUISolidBgFrame:SetPoint("TOPLEFT", actualFrame, "TOPLEFT", 8, -8)
        actualFrame._arcUISolidBgFrame:SetPoint("BOTTOMRIGHT", actualFrame, "BOTTOMRIGHT", -8, 8)
        actualFrame._arcUISolidBgFrame:SetFrameLevel(math.max(1, actualFrame:GetFrameLevel() - 1))
        
        actualFrame._arcUISolidBgFrame.tex = actualFrame._arcUISolidBgFrame:CreateTexture(nil, "BACKGROUND")
        actualFrame._arcUISolidBgFrame.tex:SetAllPoints()
        actualFrame._arcUISolidBgFrame.tex:SetColorTexture(0.02, 0.02, 0.02, 1)
      end
      
      actualFrame._arcUISolidBgFrame:SetAlpha(alpha)
      actualFrame._arcUISolidBgFrame:Show()

      -- Position/size restored via AceConfig status table before Open — nothing to do here

      -- Stretch the drag area across the full top of the frame.
      -- AceGUIContainer-Frame: titletext is a child of the title Frame (the drag handle).
      -- widget.titletext is exposed, so :GetParent() gives us the title frame directly.
      -- We clear its points and anchor it across the full frame top so the whole
      -- header bar is draggable, not just the narrow title texture.
      if not actualFrame._arcUITitleStretched then
        actualFrame._arcUITitleStretched = true
        local widget = AceConfigDialog.OpenFrames["ArcUI"]
        if widget and widget.titletext then
          local titleFrame = widget.titletext:GetParent()
          if titleFrame and titleFrame ~= actualFrame then
            titleFrame:ClearAllPoints()
            titleFrame:SetPoint("TOPLEFT",  actualFrame, "TOPLEFT",  0,  0)
            titleFrame:SetPoint("TOPRIGHT", actualFrame, "TOPRIGHT", 0,  0)
            titleFrame:SetHeight(28)
          end
        end
      end
      
      -- Create Discord link at top right (or show existing one)
      CreateDiscordLink(actualFrame)
      if actualFrame._arcUIDiscordLink then
        actualFrame._arcUIDiscordLink:Show()
      end
      
      -- Hook OnHide for cleanup (visual elements, drag mode, etc)
      -- NOTE: OnOptionsPanelClosed is called via AceConfigDialog:Close hook, not here
      if not actualFrame._arcUIOnHideHooked then
        actualFrame._arcUIOnHideHooked = true
        local originalOnHide = actualFrame:GetScript("OnHide")
        actualFrame:SetScript("OnHide", function(self, ...)
          if originalOnHide then originalOnHide(self, ...) end
          
          -- Clear options open flag (backup - Close hook also does this)
          ns._arcUIOptionsOpen = false
          -- Backup: re-evaluate textures + cooldown bar click-through
          if ns.Textures and ns.Textures.OnOptionsClosed then
              ns.Textures.OnOptionsClosed()
          end
          if ns.CooldownBars and ns.CooldownBars.RefreshMouseInteractivity then
              ns.CooldownBars.RefreshMouseInteractivity()
          end
          -- Fire registered panel callbacks (backup path)
          if ns.CDMShared and ns.CDMShared.FirePanelCallbacks then
              ns.CDMShared.FirePanelCallbacks(false)
          end
          
          -- BACKUP: Run panel-close logic if Close hook didn't fire
          -- (e.g. Escape key, other addons closing the frame)
          if ns.CDMGroups and ns.CDMGroups.OnArcUIPanelChanged then
              ns.CDMGroups.OnArcUIPanelChanged(false)
          end
          
          -- Hide "Hidden by Bar" overlays
          if ns.API.HideHiddenByBarOverlays then
            ns.API.HideHiddenByBarOverlays()
          end
          
          -- Save panel position and size via AceConfig status table (kept up to date by AceGUI on drag/resize)
          do
            local globalDB = ns.API.GetGlobalDB and ns.API.GetGlobalDB()
            if globalDB then
              local status = AceConfigDialog:GetStatusTable("ArcUI")
              if status then
                globalDB.optionsPanelPos  = { top = status.top, left = status.left }
                globalDB.optionsPanelSize = { width = status.width, height = status.height }
              end
            end
          end

          -- CRITICAL: Hide Discord link when panel closes
          -- AceConfigDialog reuses frame objects, so our Discord link would
          -- appear on other addons' config panels if we don't hide it
          if self._arcUIDiscordLink then
            self._arcUIDiscordLink:Hide()
          end
          if self._arcUISolidBgFrame then
            self._arcUISolidBgFrame:Hide()
          end
          
          if ns.Display and ns.Display.HideDeleteButtons then
            ns.Display.HideDeleteButtons()
          end
          if ns.Resources and ns.Resources.HideDeleteButtons then
            ns.Resources.HideDeleteButtons()
          end
          if ns.CDMEnhance and ns.CDMEnhance.SetUnlocked then
            ns.CDMEnhance.SetUnlocked(false)
          end
          if ns.CDMGroups and ns.CDMGroups.SetDragMode then
            ns.CDMGroups.SetDragMode(false)
          end
          
          C_Timer.After(0.1, function()
            if ns.API.ValidateAllBarTracking then
              ns.API.ValidateAllBarTracking()
            end
            -- Refresh resource bars to re-apply visibility rules now that options panel is closed
            -- (bars with unmet talent conditions or wrong spec should now be hidden again)
            if ns.Resources and ns.Resources.RefreshAllBars then
              ns.Resources.RefreshAllBars()
            end
          end)
        end)
      end
    end
    
    if ns.API.ValidateAllBarTracking then
      ns.API.ValidateAllBarTracking()
    end
  end)
end

-- ===================================================================
-- MODULE MASTER SWITCHES (Settings > Modules)
-- Peripheral modules can be turned off entirely: runtime gated AND their
-- options tab hidden until re-enabled here. Account-wide. ONE flag per
-- module, never two: Kick Assist and Loot Planner route to the master
-- flags those modules already own; Castbar and Pings use
-- ns.db.global.moduleToggles (nil = enabled, so current installs keep
-- everything on).
-- ===================================================================
function ns.API.IsModuleEnabled(key)
  if key == "kickassist" then
    local g = ns.db and ns.db.global and ns.db.global.setMyKick
    return not g or g.enabled ~= false
  end
  if key == "lootplanner" then
    -- export absent = the standalone owns it (module file early-returns);
    -- the tab then shows the dormant notice, so treat as enabled
    if ns.BonusRollIsEnabled then return ns.BonusRollIsEnabled() and true or false end
    return true
  end
  local t = ns.db and ns.db.global and ns.db.global.moduleToggles
  return not t or t[key] ~= false
end

function ns.API.SetModuleEnabled(key, v)
  v = v and true or false
  if key == "kickassist" then
    if ns.SetMyKick and ns.SetMyKick.SetEnabled then ns.SetMyKick.SetEnabled(v) end
    return
  end
  if key == "lootplanner" then
    if ns.BonusRollSetEnabled then ns.BonusRollSetEnabled(v) end
    return
  end
  if ns.db and ns.db.global then
    ns.db.global.moduleToggles = ns.db.global.moduleToggles or {}
    ns.db.global.moduleToggles[key] = v
  end
  -- live apply: modules re-evaluate their own gates
  if key == "castbar" then
    if ns.Castbar and ns.Castbar.ApplyAppearance then ns.Castbar.ApplyAppearance() end
    if ns.FocusCastbar and ns.FocusCastbar.ApplyAppearance then ns.FocusCastbar.ApplyAppearance() end
    if ns.TargetCastbar and ns.TargetCastbar.ApplyAppearance then ns.TargetCastbar.ApplyAppearance() end
  elseif key == "pings" then
    if ns.PingsOnModuleToggled then ns.PingsOnModuleToggled(v) end
  end
end

-- ===================================================================
-- MAIN OPTIONS TABLE
-- ===================================================================
local function GetOptionsTable()
  local optionsTable = {
    type = "group",
    name = "Arc UI",
    childGroups = "tab",
    args = {
      icons = {
        type = "group",
        name = "Icons",
        order = 1,
        childGroups = "tab",
        args = {
          groups = (function()
            local tbl = ns.GetCDMGroupsOptionsTable and ns.GetCDMGroupsOptionsTable() or {
              type = "group",
              name = "Groups",
              args = { loading = { type = "description", name = "Loading...", order = 1 } }
            }
            tbl.name = "Groups"
            tbl.order = 1
            return tbl
          end)(),
          
          cdmIcons = (function()
            local tbl = ns.GetCDMIconsOptionsTable and ns.GetCDMIconsOptionsTable() or {
              type = "group",
              name = "Icon Catalog",
              args = { loading = { type = "description", name = "Loading...", order = 1 } }
            }
            tbl.name = "Icon Catalog"
            tbl.order = 2
            return tbl
          end)(),
          
          defaults = {
            type = "group",
            name = "Globals",
            order = 5,
            childGroups = "tab",
            args = {
              auraDefaults = (function()
                local tbl = ns.GetCDMGlobalAuraDefaultsOptionsTable and ns.GetCDMGlobalAuraDefaultsOptionsTable() or {
                  type = "group",
                  name = "Aura Globals",
                  args = { loading = { type = "description", name = "Loading...", order = 1 } }
                }
                tbl.name = "Aura Globals"
                tbl.order = 1
                return tbl
              end)(),
              
              cooldownDefaults = (function()
                local tbl = ns.GetCDMGlobalCooldownDefaultsOptionsTable and ns.GetCDMGlobalCooldownDefaultsOptionsTable() or {
                  type = "group",
                  name = "Cooldown Globals",
                  args = { loading = { type = "description", name = "Loading...", order = 1 } }
                }
                tbl.name = "Cooldown Globals"
                tbl.order = 2
                return tbl
              end)(),
            },
          },
          
          -- Extras tab (Keybind Display, Assisted Combat Highlight, Button Press Highlight)
          extras = (function()
            local tbl = ns.GetCDMUtilitiesOptionsTable and ns.GetCDMUtilitiesOptionsTable() or {
              type = "group",
              name = "Extras",
              args = { loading = { type = "description", name = "Loading...", order = 1 } }
            }
            tbl.name = "Extras"
            tbl.order = 6
            return tbl
          end)(),
          
          -- (Arc Icons + Custom Icons tabs RETIRED 2026-08-09: the Icon
          -- Catalog absorbed everything — Add popup, filters, Load
          -- Conditions, Arc Icon Settings, mounted timer editor, auto-track
          -- config, bulk management. The FILES stay loaded: the catalog
          -- consumes ns.GetCustomIconsOptionsTable for the editor mount and
          -- the ns.ArcAurasOptions exports/popup.)
          
          -- Profiles tab: Arc Manager profile selector + Profile Browser combined
          profiles = (function()
            local profileMgr = ns.GetCDMProfileManagerOnlyOptionsTable and ns.GetCDMProfileManagerOnlyOptionsTable() or { args = {} }
            local profileBrowser = ns.GetCDMProfileBrowserOptionsTable and ns.GetCDMProfileBrowserOptionsTable() or { args = {} }

            -- Merge browser args under a collapsible header, re-keyed to avoid conflicts
            local mergedArgs = {}
            for k, v in pairs(profileMgr.args or {}) do
              mergedArgs[k] = v
            end

            -- Browser section header with collapsible toggle
            mergedArgs["browserSectionHeader"] = {
              type = "toggle",
              name = "|cffffd100Profile Browser|r",
              desc = "Browse, rename, or delete profiles across all characters and specs",
              dialogControl = "CollapsibleHeader",
              order = 50,
              width = "full",
              get = function() return not profileBrowserCollapsed end,
              set = function(_, v) profileBrowserCollapsed = not v end,
            }
            mergedArgs["browserSectionDesc"] = {
              type = "description",
              name = "|cffaaaaaaBrowse, rename, or delete profiles across all characters and specs on this account.|r",
              order = 51,
              fontSize = "small",
              hidden = function() return profileBrowserCollapsed end,
            }
            -- Inline the browser args at order 52+ sorted by original order
            local browserOrder = 52
            local sortedBrowser = {}
            for k, v in pairs(profileBrowser.args or {}) do
              table.insert(sortedBrowser, { key = k, val = v, ord = v.order or 999 })
            end
            table.sort(sortedBrowser, function(a, b) return a.ord < b.ord end)
            for _, item in ipairs(sortedBrowser) do
              local entry = {}
              for ek, ev in pairs(item.val) do entry[ek] = ev end
              entry.order = browserOrder
              browserOrder = browserOrder + 1
              local origHidden = entry.hidden
              entry.hidden = function()
                if profileBrowserCollapsed then return true end
                if origHidden then return origHidden() end
                return false
              end
              mergedArgs["browser_" .. item.key] = entry
            end

            return {
              type = "group",
              name = "Profiles",
              order = 7,
              args = mergedArgs,
            }
          end)(),

          sharing = (function()
            if ns.CDMSharedProfiles and ns.CDMSharedProfiles.GetOptionsTable then
              local tbl = ns.CDMSharedProfiles.GetOptionsTable()
              tbl.name = "Account Sharing"
              tbl.order = 8
              return tbl
            end
            return {
              type = "group",
              name = "Account Sharing",
              order = 8,
              args = { loading = { type = "description", name = "Loading...", order = 1 } },
            }
          end)(),
        },
      },
      
      -- ═══════════════════════════════════════════════════════════════
      -- AURAS: buff/debuff tracking. One catalog (Bars) where you pick an
      -- aura and create a bar or a texture, plus the shared Appearance.
      -- ═══════════════════════════════════════════════════════════════
      auras = {
        type = "group",
        name = "Buffs/Debuffs",
        order = 2,
        childGroups = "tab",
        args = {
          auraBars = (function()
            local tbl = ns.TrackingOptions and ns.TrackingOptions.GetBuffDebuffSetupTable() or {
              type = "group",
              name = "Catalog",
              args = { loading = { type = "description", name = "Loading...", order = 1 } }
            }
            tbl.name = "Catalog"
            tbl.order = 1
            return tbl
          end)(),

          textures = (function()
            local tbl = ns.GetTexturesOptionsTable and ns.GetTexturesOptionsTable() or {
              type = "group",
              name = "Textures",
              args = { loading = { type = "description", name = "Loading...", order = 1 } }
            }
            tbl.name = "Textures"
            tbl.order = 2
            return tbl
          end)(),

          appearance = (function()
            local tbl = ns.AppearanceOptions and ns.AppearanceOptions.GetOptionsTable() or {
              type = "group",
              name = "Appearance",
              args = { loading = { type = "description", name = "Loading...", order = 1 } }
            }
            tbl.name = "Appearance"
            tbl.order = 3
            return tbl
          end)(),

        },
      },

      -- ═══════════════════════════════════════════════════════════════
      -- COOLDOWNS: spell cooldown / charge / custom-timer bars + castbar.
      -- ═══════════════════════════════════════════════════════════════
      cooldowns = {
        type = "group",
        name = "Cooldowns",
        order = 3,
        childGroups = "tab",
        args = {
          cooldownBars = (function()
            local tbl = ns.CooldownBarOptions and ns.CooldownBarOptions.GetOptionsTable() or {
              type = "group",
              name = "Cooldown Bars",
              args = { loading = { type = "description", name = "Loading...", order = 1 } }
            }
            tbl.name = "Cooldown Bars"
            tbl.order = 1
            return tbl
          end)(),

          timerBars = (function()
            local tbl = ns.TimerBarOptions and ns.TimerBarOptions.GetOptionsTable() or {
              type = "group",
              name = "Custom Bars",
              args = { loading = { type = "description", name = "Loading...", order = 1 } }
            }
            tbl.name = "Custom Bars"
            tbl.order = 2
            return tbl
          end)(),

          cooldownReminder = (function()
            local tbl = ns.GetCooldownReminderOptionsTable and ns.GetCooldownReminderOptionsTable() or {
              type = "group",
              name = "Cooldown Reminder",
              args = { loading = { type = "description", name = "Loading...", order = 1 } }
            }
            tbl.name = "Cooldown Reminder"
            tbl.order = 3
            return tbl
          end)(),

          appearance = (function()
            local tbl = ns.AppearanceOptions and ns.AppearanceOptions.GetOptionsTable() or {
              type = "group",
              name = "Appearance",
              args = { loading = { type = "description", name = "Loading...", order = 1 } }
            }
            tbl.name = "Appearance"
            tbl.order = 4
            return tbl
          end)(),
        },
      },

      resources = {
        type = "group",
        name = "Resources",
        order = 4,
        childGroups = "tab",
        args = {
          setup = (function()
            local tbl = ns.TrackingOptions and ns.TrackingOptions.GetResourceSetupTable() or {
              type = "group",
              name = "Setup",
              args = { loading = { type = "description", name = "Loading...", order = 1 } }
            }
            tbl.name = "Setup"
            tbl.order = 1
            return tbl
          end)(),
          
          appearance = (function()
            local tbl = ns.AppearanceOptions and ns.AppearanceOptions.GetOptionsTable() or {
              type = "group",
              name = "Appearance",
              args = { loading = { type = "description", name = "Loading...", order = 1 } }
            }
            tbl.name = "Appearance"
            tbl.order = 2
            return tbl
          end)(),
        },
      },
      
      -- ═══════════════════════════════════════════════════════════════
      -- IMPORT / EXPORT
      -- Tabs: CDM Export | Display Export | Master Export | Import (unified)
      -- ═══════════════════════════════════════════════════════════════
      importExport = {
        type = "group",
        name = "Import/Export",
        order = 7,
        childGroups = "tab",
        args = {
          cdmExport = (function()
            local tbl = ns.GetCDMExportOnlyOptionsTable and ns.GetCDMExportOnlyOptionsTable() or {
              type = "group",
              name = "Icon Manager Export",
              args = { loading = { type = "description", name = "Loading...", order = 1 } }
            }
            tbl.name = "Icon Manager Export"
            tbl.order = 1
            return tbl
          end)(),

          barsExport = (function()
            local tbl = ns.GetBarsExportOnlyOptionsTable and ns.GetBarsExportOnlyOptionsTable() or {
              type = "group",
              name = "Display Export",
              args = { loading = { type = "description", name = "Loading...", order = 1 } }
            }
            -- "Display Export": bars AND textures (and the castbar) ride one string
            tbl.name = "Display Export"
            tbl.order = 2
            return tbl
          end)(),

          masterExport = (function()
            local tbl = ns.GetCDMMasterExportOptionsTable and ns.GetCDMMasterExportOptionsTable() or {
              type = "group",
              name = "Master Export",
              args = { loading = { type = "description", name = "Loading...", order = 1 } }
            }
            tbl.name = "Master Export"
            tbl.order = 3
            return tbl
          end)(),

          crExport = (function()
            local tbl = ns.GetCRExportOnlyOptionsTable and ns.GetCRExportOnlyOptionsTable() or {
              type = "group",
              name = "Cooldown Reminder Export",
              args = { loading = { type = "description", name = "Loading...", order = 1 } }
            }
            tbl.name = "Cooldown Reminder Export"
            tbl.order = 3.5
            return tbl
          end)(),

          castbarExport = (function()
            local tbl = ns.GetCastbarExportOnlyOptionsTable and ns.GetCastbarExportOnlyOptionsTable() or {
              type = "group",
              name = "Castbar Export",
              args = { loading = { type = "description", name = "Loading...", order = 1 } }
            }
            tbl.name = "Castbar Export"
            tbl.order = 3.7
            return tbl
          end)(),

          unifiedImport = (function()
            local tbl = ns.GetUnifiedImportExportOptionsTable and ns.GetUnifiedImportExportOptionsTable() or {
              type = "group",
              name = "Import",
              args = { loading = { type = "description", name = "Loading...", order = 1 } }
            }
            tbl.name = "Import"
            tbl.order = 4
            return tbl
          end)(),
        },
      },

      migration = (function()
        local tbl = ns.GetMigrationOptionsTable and ns.GetMigrationOptionsTable() or {
          type = "group",
          name = "Migration",
          args = { loading = { type = "description", name = "Loading...", order = 1 } }
        }
        tbl.name  = "Migration"
        tbl.order = 8
        return tbl
      end)(),

      castbar = {
        type        = "group",
        name        = "Castbar",
        order       = 5,
        childGroups = "tab",
        hidden      = function() return not ns.API.IsModuleEnabled("castbar") end,
        args        = {
          playerCastbar = (function()
            local tbl = ns.CastbarOptions and ns.CastbarOptions.GetOptionsTable() or {
              type = "group",
              name = "Player Castbar",
              args = { loading = { type = "description", name = "Loading...", order = 1 } }
            }
            tbl.name  = "Player Castbar"
            tbl.order = 1
            return tbl
          end)(),
          focusCastbar = (function()
            local tbl = ns.FocusCastbarOptions and ns.FocusCastbarOptions.GetOptionsTable() or {
              type = "group",
              name = "Focus Castbar",
              args = { loading = { type = "description", name = "Loading...", order = 1 } }
            }
            tbl.name  = "Focus Castbar"
            tbl.order = 2
            return tbl
          end)(),
          targetCastbar = (function()
            local tbl = ns.TargetCastbarOptions and ns.TargetCastbarOptions.GetOptionsTable() or {
              type = "group",
              name = "Target Castbar",
              args = { loading = { type = "description", name = "Loading...", order = 1 } }
            }
            tbl.name  = "Target Castbar"
            tbl.order = 3
            return tbl
          end)(),
        },
      },

      setMyKick = (function()
        local tbl = ns.GetSetMyKickOptionsTable and ns.GetSetMyKickOptionsTable() or {
          type = "group",
          name = "Kick Assist",
          args = { loading = { type = "description", name = "Loading...", order = 1 } }
        }
        tbl.name  = "Kick Assist"
        tbl.order = 6
        tbl.hidden = function() return not ns.API.IsModuleEnabled("kickassist") end
        return tbl
      end)(),

      pings = (function()
        local tbl = ns.GetPingsOptionsTable and ns.GetPingsOptionsTable() or {
          type = "group",
          name = "Pings",
          args = { loading = { type = "description", name = "Loading...", order = 1 } }
        }
        tbl.name  = "Pings"
        tbl.order = 6.5
        tbl.hidden = function() return not ns.API.IsModuleEnabled("pings") end
        return tbl
      end)(),

      bonusroll = (function()
        -- the module early-returns when the standalone Arc Loot Planner
        -- addon (or its old Arc Bonus Roll name) is installed, so a missing
        -- table here means "standalone owns it"
        local tbl = ns.GetBonusRollOptionsTable and ns.GetBonusRollOptionsTable() or {
          type = "group",
          name = "Loot Planner",
          args = { standalone = { type = "description", order = 1, fontSize = "medium",
            name = "The standalone Arc Loot Planner addon is installed and active - it owns this feature, and ArcUI's built-in copy is dormant. Configure it with /alp." } }
        }
        tbl.name  = "Loot Planner"
        tbl.order = 6.6
        tbl.hidden = function() return not ns.API.IsModuleEnabled("lootplanner") end
        return tbl
      end)(),

      settings = {
        type = "group",
        name = "Settings",
        order = 9,
        args = {
          modulesHeader = {
            type = "header",
            name = "Modules",
            order = 0.1,
          },
          modulesDesc = {
            type = "description",
            order = 0.2,
            name = "Master switches for ArcUI's side modules. Turning one OFF disables the whole module and hides its tab until you re-enable it here.",
          },
          moduleCastbar = {
            type = "toggle", order = 0.3, width = 0.9,
            name = "Castbar",
            desc = "The player, focus, and target castbars. OFF hides all three bars, releases the Blizzard castbar, and hides the Castbar tab.",
            get = function() return ns.API.IsModuleEnabled("castbar") end,
            set = function(_, v) ns.API.SetModuleEnabled("castbar", v) end,
          },
          moduleKickAssist = {
            type = "toggle", order = 0.4, width = 0.9,
            name = "Kick Assist",
            desc = "The interrupt-marker helper. Same switch as the one on its own tab and in its popup window.",
            get = function() return ns.API.IsModuleEnabled("kickassist") end,
            set = function(_, v) ns.API.SetModuleEnabled("kickassist", v) end,
          },
          modulePings = {
            type = "toggle", order = 0.5, width = 0.9,
            name = "Pings",
            desc = "The ping feed window and ping keybinds. OFF stops the feed, disables every ping keybind entry, and hides the Pings tab.",
            get = function() return ns.API.IsModuleEnabled("pings") end,
            set = function(_, v) ns.API.SetModuleEnabled("pings", v) end,
          },
          moduleLootPlanner = {
            type = "toggle", order = 0.6, width = 0.9,
            name = "Loot Planner",
            desc = "The bonus roll planner. Same switch as the one on its own tab. When the standalone Arc Loot Planner addon is installed it owns the feature and this toggle does nothing.",
            get = function() return ns.API.IsModuleEnabled("lootplanner") end,
            set = function(_, v) ns.API.SetModuleEnabled("lootplanner", v) end,
          },
          menuHeader = {
            type = "header",
            name = "Background",
            order = 1
          },
          menuBackgroundAlpha = {
            type = "range",
            name = "Menu Background Solidity",
            desc = "Control how solid/opaque the options panel background is (0 = see-through, 1 = fully solid dark background)",
            order = 2,
            min = 0,
            max = 1,
            step = 0.05,
            isPercent = true,
            width = 1.5,
            get = function()
              local globalDB = ns.API.GetGlobalDB and ns.API.GetGlobalDB()
              return globalDB and globalDB.menuBackgroundAlpha or 1.0
            end,
            set = function(_, val)
              local globalDB = ns.API.GetGlobalDB and ns.API.GetGlobalDB()
              if globalDB then
                globalDB.menuBackgroundAlpha = val
                local widget = AceConfigDialog.OpenFrames["ArcUI"]
                if widget and widget.frame and widget.frame._arcUISolidBgFrame then
                  widget.frame._arcUISolidBgFrame:SetAlpha(val)
                end
              end
            end,
          },
          
          fontTextureHeader = {
            type = "header",
            name = "Global Font & Texture",
            order = 3,
          },
          fontTextureDesc = {
            type = "description",
            name = "|cffaaaaaaPick a font and statusbar texture once, then push them onto everything: every aura, resource, cooldown, and timer bar, both castbars, and every CDM icon group's cooldown/stack text, custom labels, and keybind text -- no need to set it in each group individually. This updates everything in place; anything can still be customized afterward.|r",
            order = 4,
            fontSize = "medium",
          },
          globalFont = {
            type = "select",
            dialogControl = "LSM30_Font",
            name = "Font",
            order = 5,
            width = 1.2,
            values = GetGlobalFontValues,
            get = function()
              local g = ns.API.GetGlobalDB and ns.API.GetGlobalDB()
              return g and g.globalFont or "Friz Quadrata TT"
            end,
            set = function(_, val)
              local g = ns.API.GetGlobalDB and ns.API.GetGlobalDB()
              if g then g.globalFont = val end
            end,
          },
          globalBarTexture = {
            type = "select",
            dialogControl = "ArcUI_TexturePicker",
            name = "Statusbar Texture",
            order = 6,
            width = 1.2,
            values = GetGlobalStatusBarTextureValues,
            get = function()
              local g = ns.API.GetGlobalDB and ns.API.GetGlobalDB()
              return g and g.globalBarTexture or "Blizzard"
            end,
            set = function(_, val)
              local g = ns.API.GetGlobalDB and ns.API.GetGlobalDB()
              if g then g.globalBarTexture = val end
            end,
          },
          globalFontTextureApply = {
            type = "execute",
            name = "Apply to All Bars",
            desc = "Push the font and texture above onto every existing aura, resource, cooldown, and timer bar, the player and focus castbars, and the font onto every CDM icon group's cooldown/stack text, custom labels, and keybind text.",
            order = 7,
            width = 1.5,
            confirm = true,
            confirmText = "This will overwrite the font and statusbar texture on every bar, castbar, and icon group you have configured. Continue?",
            func = function()
              local g = ns.API.GetGlobalDB and ns.API.GetGlobalDB()
              local font = g and g.globalFont or "Friz Quadrata TT"
              local texture = g and g.globalBarTexture or "Blizzard"
              if ns.API.ApplyGlobalFontTexture then
                ns.API.ApplyGlobalFontTexture(font, texture)
              end
            end,
          },

          minimapHeader = {
            type = "header",
            name = "Minimap",
            order = 10
          },
          minimapButton = {
            type = "toggle",
            name = "Show Minimap Button",
            desc = "Toggle the minimap button visibility",
            order = 11,
            width = 1.5,
            get = function()
              local globalDB = ns.API.GetGlobalDB and ns.API.GetGlobalDB()
              return globalDB and not globalDB.minimap.hide
            end,
            set = function(_, val)
              local globalDB = ns.API.GetGlobalDB and ns.API.GetGlobalDB()
              if globalDB then
                globalDB.minimap.hide = not val
                if val then
                  ns.API.ShowMinimapButton()
                else
                  ns.API.HideMinimapButton()
                end
              end
            end,
          },
          
          changelogHeader = {
            type = "header",
            name = "Changelog",
            order = 20
          },
          changelogShow = {
            type = "toggle",
            name = "Show Changelog on Update",
            desc = "Automatically pop up the 'What's New' window once after each ArcUI update.",
            order = 21,
            width = 1.8,
            get = function()
              local g = ns.API.GetGlobalDB and ns.API.GetGlobalDB()
              return not (g and g.changelog and g.changelog.disabled)
            end,
            set = function(_, val)
              local g = ns.API.GetGlobalDB and ns.API.GetGlobalDB()
              if g then
                g.changelog = g.changelog or {}
                g.changelog.disabled = not val
              end
            end,
          },
          changelogView = {
            type = "execute",
            name = "View Changelog",
            desc = "Open the 'What's New' window now.",
            order = 22,
            width = 1.2,
            func = function()
              if ns.Changelog and ns.Changelog.Show then ns.Changelog.Show() end
            end,
          },

          troubleshootingHeader = {
            type = "header",
            name = "Troubleshooting",
            order = 80,
          },
          troubleshootingDesc = {
            type = "description",
            name = "|cffaaaaaaDebug messages for bug reports. Leave these OFF for normal play - they flood the chat window with internal traces. Changes take effect immediately and persist until turned off.|r",
            order = 81,
            fontSize = "medium",
          },
          debugVerbose = {
            type = "toggle",
            name = "Verbose Debug Messages",
            desc = "Developer traces from the bar engine, catalog scans, and icon styling. Enable when asked to during a bug report, then copy the chat output.",
            order = 82,
            width = 1.5,
            get = function()
              local g = ns.API.GetGlobalDB and ns.API.GetGlobalDB()
              return (g and g.troubleshooting and g.troubleshooting.devMode) or false
            end,
            set = function(_, v)
              local g = ns.API.GetGlobalDB and ns.API.GetGlobalDB()
              if not g then return end
              g.troubleshooting = g.troubleshooting or {}
              g.troubleshooting.devMode = v or nil
              ns.devMode = v
              _G.ARCUI_DEBUG = v or nil
            end,
          },
          debugFrameController = {
            type = "toggle",
            name = "Frame Controller Trace",
            desc = "Traces CDM frame assignment, group reconciliation, and rebuild handling (the /arcuifc system). Very chatty during spec changes and reloads.",
            order = 83,
            width = 1.5,
            get = function()
              local g = ns.API.GetGlobalDB and ns.API.GetGlobalDB()
              return (g and g.troubleshooting and g.troubleshooting.fcDebug) or false
            end,
            set = function(_, v)
              local g = ns.API.GetGlobalDB and ns.API.GetGlobalDB()
              if not g then return end
              g.troubleshooting = g.troubleshooting or {}
              g.troubleshooting.fcDebug = v or nil
              _G.ARCUI_FC_DEBUG = v
            end,
          },
          debugCDMStyling = {
            type = "toggle",
            name = "CDM Styling Debug Log",
            desc = "Records CDM icon styling diagnostics used by /arcdebug reports.",
            order = 84,
            width = 1.5,
            get = function()
              local g = ns.API.GetGlobalDB and ns.API.GetGlobalDB()
              return (g and g.troubleshooting and g.troubleshooting.enhanceDebug) or false
            end,
            set = function(_, v)
              local g = ns.API.GetGlobalDB and ns.API.GetGlobalDB()
              if not g then return end
              g.troubleshooting = g.troubleshooting or {}
              g.troubleshooting.enhanceDebug = v or nil
              _G.ArcUI_CDMEnhance_Debug = v or nil
            end,
          },
          aboutHeader = {
            type = "header",
            name = "About",
            order = 90
          },
          version = {
            type = "input",
            name = "Version",
            order = 91,
            width = 1.0,
            dialogControl = "SFX-Info",
            get = function() return ns.AddonInfo.Version end,
            set = function() end,
          },
          author = {
            type = "input",
            name = "Author",
            order = 92,
            width = 1.0,
            dialogControl = "SFX-Info",
            get = function() return ns.AddonInfo.Author end,
            set = function() end,
          },
        },
      },
    },
  }
  
  return optionsTable
end

-- ===================================================================
-- OPTIONS REGISTRATION
-- ===================================================================
local function RegisterOptions()
  -- skipValidation=true (3rd arg, AceConfigRegistry direct): every edit in an
  -- open panel triggers NotifyChange → re-fetch → full recursive VALIDATION of
  -- the entire tree on top of the rebuild. The lib documents skipValidation as
  -- "primarily useful for extremely huge options, with a noticeable slowdown"
  -- — which is exactly this tree. (AceConfig:RegisterOptionsTable's 3rd arg is
  -- a slash command, not skipValidation, hence the direct registry call.)
  LibStub("AceConfigRegistry-3.0"):RegisterOptionsTable("ArcUI", GetOptionsTable, true)
  AceConfigDialog:SetDefaultSize("ArcUI", 900, 700)
  optionsRegistered = true
end

-- ===================================================================
-- DATABASE RESET FUNCTION
-- ===================================================================
local function ResetDatabase()
  -- Clear the corrupted SavedVariables
  ArcUIDB = nil
  ArcUI_CDMEnhance_Debug = nil
  
  print("|cff00ccffArc UI|r Database has been reset. Please |cffff0000/reload|r to complete the reset.")
  print("|cff00ccffArc UI|r Your settings will be restored to defaults.")
end

-- ===================================================================
-- SLASH COMMANDS (defined early so they always work)
-- ===================================================================
SLASH_ARCBARS1 = "/arcbars"
SLASH_ARCBARS2 = "/ab"
SLASH_ARCBARS3 = "/arcui"
SLASH_ARCBARS4 = "/aui"

SLASH_ARCCDM1 = "/cdm"
SlashCmdList["ARCCDM"] = function()
  local frame = _G["CooldownViewerSettings"]
  if frame and frame.Show then
    frame:Show()
    frame:Raise()
  end
end
SlashCmdList["ARCBARS"] = function(msg)
  msg = msg:lower():trim()
  
  -- Database reset command (always available, even if options fail)
  if msg == "reset-db" or msg == "resetdb" then
    StaticPopupDialogs["ARCUI_RESET_DB"] = {
      text = "This will reset ALL Arc UI settings to defaults.\n\nAre you sure?",
      button1 = "Yes, Reset",
      button2 = "Cancel",
      OnAccept = function()
        ResetDatabase()
      end,
      timeout = 0,
      whileDead = true,
      hideOnEscape = true,
      preferredIndex = 3,
    }
    StaticPopup_Show("ARCUI_RESET_DB")
    return
  end
  
  if msg == "" or msg == "options" or msg == "config" then
    ns.API.OpenOptions()
  elseif msg == "scan" then
    local buffs, err = ns.API.ScanAvailableBuffs()
    if buffs then
      print("|cff00ccffArc UI|r Found " .. #buffs .. " buff(s)/debuff(s). Open options to configure.")
    else
      print("|cff00ccffArc UI|r Error: " .. (err or "Unknown"))
    end
  elseif msg == "unlock" or msg == "drag" then
    if ns.CDMGroups and ns.CDMGroups.ToggleDragMode then
      ns.CDMGroups.ToggleDragMode()
      if ns.CDMGroups.dragModeEnabled then
        print("|cff00ccffArc UI|r Edit mode |cff00ff00ENABLED|r - drag icons to reposition")
      else
        print("|cff00ccffArc UI|r Edit mode |cffff0000DISABLED|r")
      end
    elseif ns.CDMEnhance and ns.CDMEnhance.ToggleUnlock then
      ns.CDMEnhance.ToggleUnlock()
    else
      print("|cff00ccffArc UI|r CDM module not loaded yet")
    end
  elseif msg == "layout" then
    if ns.LayoutEditor and ns.LayoutEditor.Toggle then
      ns.LayoutEditor.Toggle()
    end
  elseif msg == "recenter" then
    -- Clear saved position so panel opens at default center next time
    local globalDB = ns.API.GetGlobalDB and ns.API.GetGlobalDB()
    if globalDB then
      globalDB.optionsPanelPos  = nil
      globalDB.optionsPanelSize = nil
    end
    local status = AceConfigDialog:GetStatusTable("ArcUI")
    if status then
      local sw, sh = GetScreenWidth(), GetScreenHeight()
      local w, h = 900, 700
      status.top  = sh / 2 + h / 2
      status.left = sw / 2 - w / 2
      status.width  = w
      status.height = h
    end
    -- Reopen at new position if already open
    local widget = AceConfigDialog.OpenFrames and AceConfigDialog.OpenFrames["ArcUI"]
    if widget and widget.frame and widget.frame:IsShown() then
      AceConfigDialog:Close("ArcUI")
      ns.API.OpenOptions()
    end
    print("|cff00ccffArc UI|r Options panel recentered")
  elseif msg == "reset" then
    local db = ns.API.GetDB()
    if db then
      for i = 1, 30 do
        if db.bars and db.bars[i] then
          db.bars[i].display.barPosition = {point="CENTER", relPoint="CENTER", x=0, y=200-(i-1)*30}
          db.bars[i].display.textPosition = {point="CENTER", relPoint="CENTER", x=0, y=230-(i-1)*30}
        end
        if db.resourceBars and db.resourceBars[i] then
          db.resourceBars[i].display.barPosition = {point="CENTER", relPoint="CENTER", x=0, y=-100-(i-1)*35}
          db.resourceBars[i].display.textPosition = {point="CENTER", relPoint="CENTER", x=0, y=-70-(i-1)*35}
        end
      end
      for i = 1, 30 do
        if ns.Display and ns.Display.ApplyAppearance then
          ns.Display.ApplyAppearance(i)
        end
        if ns.Resources and ns.Resources.ApplyAppearance then
          ns.Resources.ApplyAppearance(i)
        end
      end
      print("|cff00ccffArc UI|r Positions reset")
    end
  elseif msg == "minimap" then
    local globalDB = ns.API.GetGlobalDB()
    if globalDB then
      globalDB.minimap.hide = not globalDB.minimap.hide
      if globalDB.minimap.hide then
        ns.API.HideMinimapButton()
        print("|cff00ccffArc UI|r Minimap button hidden")
      else
        ns.API.ShowMinimapButton()
        print("|cff00ccffArc UI|r Minimap button shown")
      end
    end
  elseif msg == "export" then
    -- Quick export shortcut
    if optionsRegistered then
      ns.API.OpenOptions()
      C_Timer.After(0.1, function()
        AceConfigDialog:SelectGroup("ArcUI", "icons", "profiles")
      end)
    else
      print("|cff00ccffArc UI|r Options not ready yet.")
    end
  elseif msg == "import" then
    -- Quick import shortcut
    if optionsRegistered then
      ns.API.OpenOptions()
      C_Timer.After(0.1, function()
        AceConfigDialog:SelectGroup("ArcUI", "icons", "profiles")
      end)
    else
      print("|cff00ccffArc UI|r Options not ready yet.")
    end
  elseif msg == "help" then
    print("|cff00ccffArc UI|r Commands:")
    print("  /arcui - Open options")
    print("  /arcui scan - Scan for buffs/debuffs")
    print("  /arcui drag - Toggle icon group editing")
    print("  /arcui recenter - Move options panel back to center of screen")
    print("  /arcui reset - Reset bar positions")
    print("  /arcui minimap - Toggle minimap button")
    print("  /arcui export - Open import/export panel")
    print("  /arcui import - Open import/export panel")
    print("  /arcui reset-db - |cffff0000Reset ALL settings to defaults|r")
  else
    print("|cff00ccffArc UI|r Unknown command. Use /arcui help")
  end
end

-- ===================================================================
-- /arcdebug  — full system overview
-- ===================================================================
SLASH_ARCDEBUG1 = "/arcdebug"
SlashCmdList["ARCDEBUG"] = function()
  local ver = C_AddOns.GetAddOnMetadata(ADDON, "Version") or "?"
  local db  = ns.db and ns.db.char

  local sep = "|cff444444" .. string.rep("-", 44) .. "|r"
  local function yn(v)  return v and "|cff00ff00yes|r" or "|cffff5555no|r" end
  local function hi(s)  return "|cffffd100" .. tostring(s) .. "|r" end
  local function dim(s) return "|cff888888" .. tostring(s) .. "|r" end

  print(sep)
  print("|cff00ccffArcUI|r v" .. hi(ver) .. "   " .. dim(date("%H:%M:%S")) .. "   combat=" .. yn(InCombatLockdown()))
  print(sep)

  -- Castbar
  if ns.Castbar and ns.Castbar.GetStatus then
    local s = ns.Castbar.GetStatus()
    local detail = ""
    if s.enabled and s.castActive then
      local stages = s.castEmpowerStages > 0 and (" x" .. s.castEmpowerStages) or ""
      detail = "  " .. dim("[" .. s.castKind .. stages .. "]")
    end
    print("  Castbar       " .. yn(s.enabled) .. detail)
    if s.enabled then
      local flags = {}
      if s.hideChannels    then flags[#flags+1] = "hide-channels" end
      if s.hideOutOfCombat then flags[#flags+1] = "hide-ooc"      end
      if #flags > 0 then print("    " .. dim(table.concat(flags, "  "))) end
    end
  else
    print("  Castbar       " .. dim("module not loaded"))
  end

  -- Aura bars — tracking.enabled is the active flag (same logic as GetActiveBars)
  do
    local activeCount = ns.API.GetActiveBars and #ns.API.GetActiveBars() or 0
    local total = 0
    if db and db.bars then
      for i = 1, 500 do if db.bars[i] then total = total + 1 end end
    end
    print("  Aura Bars     " .. hi(activeCount) .. " enabled / " .. dim(total .. " configured"))
  end

  -- Resource bars — tracking.enabled (same logic as GetActiveResourceBars)
  do
    local activeCount = ns.API.GetActiveResourceBars and #ns.API.GetActiveResourceBars() or 0
    local total = 0
    if db and db.resourceBars then
      for i = 1, 50 do if db.resourceBars[i] then total = total + 1 end end
    end
    print("  Resource Bars " .. hi(activeCount) .. " enabled / " .. dim(total .. " configured"))
  end

  -- Cooldown bars (nested: cooldownBarConfigs[spellID][barType])
  do
    local cdTotal = 0
    if db and db.cooldownBarConfigs then
      for _, configs in pairs(db.cooldownBarConfigs) do
        for _, cfg in pairs(configs) do
          if type(cfg) == "table" then cdTotal = cdTotal + 1 end
        end
      end
    end
    print("  Cooldown Bars " .. hi(cdTotal) .. " configured")
  end

  -- Timer bars
  do
    local total = 0
    if db and db.timerBarConfigs then
      for _, cfg in pairs(db.timerBarConfigs) do
        if type(cfg) == "table" then total = total + 1 end
      end
    end
    print("  Timer Bars    " .. hi(total) .. " configured")
  end

  -- Cooldown Reminder
  do
    local crDB = db and db.cooldownReminder
    print("  CD Reminder   " .. yn(crDB and crDB.enabled))
  end

  print(sep)

  -- CDM Enhancement
  if ns.CDMShared then
    print("  CDM Enhance   " .. yn(ns.CDMShared.IsCDMStylingEnabled()))
  else
    print("  CDM Enhance   " .. dim("module not loaded"))
  end

  -- CDM Groups
  if ns.CDMGroups then
    local groupCount, freeCount = 0, 0
    if ns.CDMGroups.groups then
      for _ in pairs(ns.CDMGroups.groups) do groupCount = groupCount + 1 end
    end
    if ns.CDMGroups.freeIcons then
      for _ in pairs(ns.CDMGroups.freeIcons) do freeCount = freeCount + 1 end
    end
    print("  CDM Groups    " .. hi(groupCount) .. " groups / " .. hi(freeCount) .. " free icons")
  else
    print("  CDM Groups    " .. dim("module not loaded"))
  end

  -- Arc Auras
  if ns.ArcAuras then
    local arcCount = 0
    local frames = ns.ArcAuras.frames or {}
    for _ in pairs(frames) do arcCount = arcCount + 1 end
    print("  Arc Auras     " .. hi(arcCount) .. " tracked")
  else
    print("  Arc Auras     " .. dim("module not loaded"))
  end

  print(sep)
  print(dim("  /arcui  /arccastdebug  /arcmasque  /arcrepair"))
  print(sep)
end

-- ===================================================================
-- MAIN INITIALIZATION
-- ===================================================================
local initFrame = CreateFrame("Frame")
initFrame:RegisterEvent("PLAYER_LOGIN")
initFrame:RegisterEvent("PLAYER_REGEN_DISABLED")
initFrame:RegisterEvent("PLAYER_REGEN_ENABLED")

initFrame:SetScript("OnEvent", function(self, event)
  if event == "PLAYER_REGEN_DISABLED" then
    -- Close options panel when entering combat
    local widget = AceConfigDialog.OpenFrames["ArcUI"]
    if widget then
      AceConfigDialog:Close("ArcUI")
      print("|cff00ccffArc UI|r Options closed - entering combat.")
    end
    return
  end
  
  if event == "PLAYER_REGEN_ENABLED" then
    -- Open options panel if user tried to open during combat
    if ns._arcPendingOptionsOpen then
      ns._arcPendingOptionsOpen = nil
      C_Timer.After(0.1, function()
        if not InCombatLockdown() then
          ns.API.OpenOptions()
        end
      end)
    end
    return
  end
  
  if event == "PLAYER_LOGIN" then
    -- ═══════════════════════════════════════════════════════════════════
    -- ACEDB INITIALIZATION
    -- pcall is banned in ArcUI, so instead of catching an AceDB failure we
    -- pre-sanitize the raw SavedVariable: a corrupted, non-table ArcUIDB is the
    -- one thing that makes AceDB:New throw. Reset it to nil and let AceDB rebuild
    -- from defaults. Valid saved data is left untouched.
    -- ═══════════════════════════════════════════════════════════════════
    if ArcUIDB ~= nil and type(ArcUIDB) ~= "table" then
      print("|cff00ccffArc UI|r |cffff0000WARNING:|r Saved settings were corrupted and have been reset to defaults.")
      print("|cff00ccffArc UI|r Type |cffff0000/arcui reset-db|r if problems persist.")
      ArcUIDB = nil
    end
    ns.db = AceDB:New("ArcUIDB", ns.DB_DEFAULTS, true)

    -- ═══════════════════════════════════════════════════════════════════
    -- CLEANUP: Remove empty/unconfigured bar configs to reduce memory
    -- Replaces old sparse array hole-filling which was adding bloat
    -- ═══════════════════════════════════════════════════════════════════
    if ns.DataRepair and ns.DataRepair.RunAutoCleanup then
      ns.DataRepair.RunAutoCleanup()
    end

    -- ENGINE PREBUILD -- must run SYNCHRONOUSLY here, inside the load window.
    -- 12.1 blocks aura-slot creation from addon stacks whenever auras are
    -- secret (all combat, instances 24/7), and the normal bar init runs from a
    -- timer well after this point -- so reloading during a pull used to leave
    -- every engine-driven bar empty for the rest of the fight. Creating the
    -- slots here makes every later pass a filter retarget, which stays legal.
    if ns.Display and ns.Display.EnginePrebuild then
      ns.Display.EnginePrebuild()
    end

    C_Timer.After(0.1, function()
      RegisterOptions()
      
      if ns.Options.InitMinimapButton then
        ns.Options.InitMinimapButton()
      end
      if ns.Display and ns.Display.Init then
        ns.Display.Init()
      end
      if ns.Resources and ns.Resources.Init then
        ns.Resources.Init()
      end
      if ns.CooldownBars and ns.CooldownBars.Init then
        ns.CooldownBars.Init()
      end
      if ns.CustomTracking and ns.CustomTracking.Init then
        ns.CustomTracking.Init()
      end
      if ns.Castbar and ns.Castbar.Init then
        ns.Castbar.Init()
      end
      if ns.FocusCastbar and ns.FocusCastbar.Init then
        ns.FocusCastbar.Init()
      end
      if ns.TargetCastbar and ns.TargetCastbar.Init then
        ns.TargetCastbar.Init()
      end
      if ns.SetMyKick and ns.SetMyKick.Init then
        ns.SetMyKick.Init()
      end
      if ns.Textures and ns.Textures.Init then
        ns.Textures.Init()
      end

      -- Re-apply persisted Troubleshooting debug toggles (Settings tab).
      -- Runs after every module has loaded, so it wins over file-load resets
      -- (CDMEnhance sets its debug SavedVariable false at file scope).
      do
        local g = ns.API.GetGlobalDB and ns.API.GetGlobalDB()
        local ts = g and g.troubleshooting
        if ts then
          if ts.devMode then
            ns.devMode = true
            _G.ARCUI_DEBUG = true
          end
          if ts.fcDebug then _G.ARCUI_FC_DEBUG = true end
          if ts.enhanceDebug then _G.ArcUI_CDMEnhance_Debug = true end
        end
      end

      print("|cff00ccffArc UI|r v" .. ns.AddonInfo.Version .. " loaded - /arcui")
    end)
  end
end)