Multibot Options Frame Ace3 Migrated

Author: Wishmaster117

Core/MultiBotDebug.lua

@@ -0,0 +1,66 @@
+-- MultiBotDebug.lua
+-- Temporary migration debug helpers.
+-- Keep this file during ACE3 migration; remove once migration diagnostics are no longer needed.
+
+MultiBot.Debug = MultiBot.Debug or {}
+
+local function EmitDebugMessage(message, colorHex)
+  local text = message
+  if colorHex and colorHex ~= "" then
+    text = "|cff" .. colorHex .. message .. "|r"
+  end
+
+  if DEFAULT_CHAT_FRAME and DEFAULT_CHAT_FRAME.AddMessage then
+    DEFAULT_CHAT_FRAME:AddMessage(text)
+  elseif type(print) == "function" then
+    print(message)
+  end
+end
+
+function MultiBot.Debug.Print(message, colorHex)
+  EmitDebugMessage(tostring(message), colorHex)
+end
+
+function MultiBot.Debug.Once(key, message, colorHex)
+  if type(key) ~= "string" or key == "" then
+    MultiBot.Debug.Print(message, colorHex)
+    return
+  end
+
+  MultiBot._debugOnceFlags = MultiBot._debugOnceFlags or {}
+  if MultiBot._debugOnceFlags[key] then
+    return
+  end
+
+  MultiBot._debugOnceFlags[key] = true
+  MultiBot.Debug.Print(message, colorHex)
+end
+
+function MultiBot.Debug.OptionsPath(path, detail)
+  local message = string.format("MultiBot Options: using %s path", tostring(path))
+  if detail and detail ~= "" then
+    message = message .. string.format(" (%s)", detail)
+  end
+  MultiBot.Debug.Once("options.path", message, "33ff99")
+end
+
+function MultiBot.Debug.AceGUILoadState(reason)
+  local hasLibStub = type(LibStub) == "table"
+  local aceMinor = nil
+  local aceLoaded = false
+
+  if hasLibStub and type(LibStub.minors) == "table" then
+    aceMinor = LibStub.minors["AceGUI-3.0"]
+    aceLoaded = LibStub.libs and LibStub.libs["AceGUI-3.0"] ~= nil
+  end
+
+  local message = string.format(
+    "MultiBot Options: AceGUI debug => reason=%s, LibStub=%s, minor=%s, loaded=%s",
+    tostring(reason),
+    tostring(hasLibStub),
+    tostring(aceMinor),
+    tostring(aceLoaded)
+  )
+
+  MultiBot.Debug.Once("options.acegui.load", message, "ffff00")
+end

MultiBot.toc

@@ -18,7 +18,7 @@ Libs\AceConsole-3.0\AceConsole-3.0.lua
 Libs\AceDB-3.0\AceDB-3.0.lua
 Libs\AceLocale-3.0\AceLocale-3.0.lua
 Libs\AceTimer-3.0\AceTimer-3.0.lua
-Libs\AceGUI-3.0\AceGUI-3.0.lua
+Libs\AceGUI-3.0\AceGUI-3.0.xml
 Libs\LibDBIcon-1.0\LibDBIcon-1.0.lua
 Libs\libdatabroker-1-1\LibDataBroker-1.1.lua
 Libs\LibDropdown-1.0\LibDropdown-1.0.lua
@@ -27,6 +27,7 @@ Libs\LibDropdown-1.0\LibDropdown-1.0.lua
 Core\MultiBot.lua
 Core\MultiBotConfig.lua
 Core\MultiBotLocale.lua
+Core\MultiBotDebug.lua
 UI\MultiBotOptions.lua
 Core\MultiBotThrottle.lua
 

ROADMAP.md

@@ -20,8 +20,8 @@
   - AceDB bootstrap/runtime migration is now complete for supported SavedVariables paths; one-way versioned legacy cutovers are in place with guarded legacy creation and post-migration cleanup to avoid stale duplicate persistence.
 - **Milestone 7 (Minimap/options integration):** Completed.
   - Minimap hide/angle, global frame strata, options timers/throttle, Spec dropdown positions, Hunter/Shaman quick-bar positions, Hunter pet stance state and Shaman totem choice state now run through AceDB-backed helpers with one-way versioned legacy cutover and guarded legacy fallback (no legacy table creation on pure read paths).
-- **Milestone 8 (AceGUI UI refactor):** Planned (next).
-  - Replace legacy frame construction screen-by-screen with AceGUI containers/widgets while preserving behavior and slash/open flows.
+- **Milestone 8 (AceGUI UI refactor):** In progress.
+  - `UI/MultiBotOptions.lua` panel content has been migrated to AceGUI widgets while preserving category registration and slash/open flows; remaining screens continue screen-by-screen.
  - **Milestone 9 (Localization and text pipeline):** Completed.
   - Core locale loader + per-locale payload files are integrated (`Core/MultiBotLocale.lua`, `Locales/MultiBotAceLocale-*.lua`).
   - `Core/MultiBotInit.lua`, `Features/MultiBotRaidus.lua`, `Core/MultiBotEvery.lua`, `Core/MultiBotEngine.lua`, `Core/MultiBotHandler.lua`, `Strategies/MultiBotDruid.lua`, `Strategies/MultiBotPaladin.lua`, `Strategies/MultiBotMage.lua`, `Strategies/MultiBotWarlock.lua`, `Strategies/MultiBotPriest.lua`, `Strategies/MultiBotShaman.lua`, `Strategies/MultiBotHunter.lua`, `Strategies/MultiBotRogue.lua`, `Strategies/MultiBotDeathKnight.lua`, and `Strategies/MultiBotWarrior.lua` migration sweeps are completed for legacy `MultiBot.tips.*` runtime reads.