Ultimate Profession Master Changelog (formerly TOG Profession Master)

[v1.9.0] (2026-10-24) - Share missing recipes between accounts

New Features

• Missing Recipes import and export (asked for on Discord 2026-10-24 for a player with characters on several accounts who wants every character's missing recipe scrolls in one place to buy at any Auction House; operator: "it needs to be between accounts, so it needs to be exportable, like we did with the copy/paste for the shopping list, but we need it for the missing recipes"). A Share List button on the Missing Recipes tab (My Character view) opens a window holding every character this account knows as text: its own characters' missing scrolls, computed live with the toolbar at its defaults, plus any characters shared in earlier, so a list passes along a chain of accounts. Only scrolls travel; a trainer-only recipe is nothing to buy. Pasting a list there and pressing Import stores each character in it account-wide (TOGPM_Settings global.missingShared), replacing that character's earlier share, so a re-import after they learn a recipe drops it and importing twice changes nothing. This account's own characters in an import are ignored, because their list is live. Shared characters appear in the Character dropdown marked "(shared)", with a Forget button, and All characters lists every scroll any character needs, once each, with "for Name, Name" after it. The skill, spec, "Can learn now" and "Show All" filters do not apply to a shared character, since the other account already applied them and no skill is held for them here. The text is plain UPMMR1:<Name-Realm>=<spell>,<spell>,...;... with no Discord markup character; whitespace a chat client adds is ignored; names are checked (must hold a realm, at most 64 characters, no |) and capped at 50 characters and 3000 recipes each; a recipe this game version does not have is skipped and counted in the window's status line. Kept separate from the Recipes-tab shopping list on purpose: that list holds recipes a character crafts, with reagents, and a scroll to buy has none. Location: GUI/MissingRecipesTab.lua (MakeEntry, ExportMissing, ImportMissing, ForgetShared, BuildFor, OpenShareDialog), TOGProfessionMaster.lua, Locale/enUS.lua. Spec: Tests/missingshare_spec.lua.

Improvements

• One share window for both lists. The copy/paste window the shopping list's Share List opened is now addon.GUI.ShareDialog, used by the Recipes tab and Missing Recipes alike, so the two cannot drift apart. Location: GUI/SharedWidgets.lua, GUI/BrowserTab.lua.

[v1.8.0] (2026-10-21) - Share the shopping list between accounts and friends

New Features

• Shopping list import and export (asked for on Discord 2026-10-21 by a player on three paid accounts who wants one list any of their characters can search the AH with). A Share List button on the Recipes toolbar opens a window holding the list as one block of text, selected and ready to copy. Pasting a shared list there and pressing Import adds it to the current character's list and never replaces it. Each entry remembers which other character wants how many: the shopping list shows "for Name" after the recipe and its tooltip lists everyone with their counts, so a pattern found on the AH can be mailed to the right character. A character's share is replaced on import, never added again, so importing the same list twice or your own list back changes nothing. Your own counts in an imported list are restored (raised to the imported count, never lowered), so a list exported, deleted and imported back on one character comes back whole; the first build dropped them, which the operator caught in game. A list passed through several accounts keeps crediting each item to the character who asked for it. Only the recipe ids and who wants how many travel (operator, 2026-10-21: "we don't need to import/export the recipies, just the items/count on the list. everything needed to repopulate it is in the addon"): the text is plain UPMSL1:<recipe>:<Name-Realm>=<count>,...;..., and the name, icon, crafted item and reagents are rebuilt from the addon's own recipe data with the same calls and client gate the recipe list uses. A recipe this game version does not have is skipped and counted in the window's status line. The text holds no Discord markup character, and line breaks a chat client adds are ignored. Character names and counts from an import are checked and clamped (1-999, at most 64 characters, no |), at most 500 entries. Location: GUI/BrowserTab.lua (ExportShoppingList, ImportShoppingList, ShoppingEntryFor, OpenShareDialog), GUI/SharedWidgets.lua (new addon.GUI.ToolbarButton), Locale/enUS.lua. Spec: Tests/shoppingshare_spec.lua.

[v1.7.0] (2026-10-21) - Renamed to Ultimate Profession Master; supply mail sends from the Mail button and is marked Sent; Guild tab + clicks land; Missing Recipes Sources sort no longer stalls

New Features

• Renamed to Ultimate Profession Master (operator, 2026-10-21: "i want to rebrand TOGPM as Ultimate Profession Master"). Every name a player sees changes: the addon list title in all seven TOCs, the window and sync-log titles, the settings page, the minimap button, chat prefixes and the "[TOGPM]" tooltip and alert tags (now "[UPM]") in all fifteen locales, and the README. /upm is added beside /togpm, which keeps working. The minimap tooltip title, slash-help header, status printouts and the bank dialog's "[UPM]" tag now draw in the brand orange instead of the old TOG purple (operator, in game: "ensure you check tooltips, like the minimap button one that still have old info"). The minimap button and the AddOns-list icon (## IconTexture in all seven TOCs) use the operator's new icons/UPM_MMB_Icon.tga instead of the "TOG PM" art. The minimap button crops that art with iconCoords = { 0.135, 0.825, 0.105, 0.795 } instead of LibDBIcon's default 5% trim, so the coin fills the slot and sits centred (operator, in game: "the rest of the icons aren't squished like mine"; the offsets were nudged to the operator's in-game reports). Location: GUI/MinimapButton.lua. Chat lines start with an orange "UPM:" instead of "TOGProfessionMaster:" (operator, in game: "/upm help" printed the internal name on every line): Ace:Print/Printf in TOGProfessionMaster.lua replace AceConsole's, which prefix with the AceAddon name. The supply-mail messages (GUI/CooldownsTab.lua) say "UPM:" instead of the full name, and /upm help lists every command as /upm, with one line saying /togpm still works. The test env now restores the addon object's Print/Printf on every reset, since three specs replaced them without putting them back. Spec: Tests/chatprefix_spec.lua. Deliberately unchanged, because changing them would lose data or split the guild: the folder and file names (the game keys each player's saved data by folder), the internal addon, locale, options, LibDataBroker and DeltaSync names (guild sync derives its message prefix from the namespace, so a renamed one would not talk to players on older versions), frame global names and the CurseForge slug. The release tag prefix DOES change, to UltimateProfessionMaster-vX.Y.Z from this release on (operator, 2026-10-21: "we need to change the tags to align with the brand"); older TOGProfessionMaster-vX.Y.Z tags stay. The tag becomes the addon's version string, and VersionCheck-1.0 compares only its digit groups, so guild version checks across the two prefixes still order correctly.

• A Sent column on the Cooldowns tab (operator, 2026-10-21: "add a column using the same frameworks as the other columns to annotate who we've sent mats to"). Once a supply mail goes, the row shows how long ago ("5m ago", "3h ago", "2d ago"), with the date and time on hover. It clears by itself when that crafter uses the cooldown, because the record keeps the cooldown's expiry from the moment of sending and drops itself once the expiry jumps forward into the future. A minute of slack keeps it through a resync, since cooldowns travel as time remaining and come back a few seconds off. Every transmute shares one record per crafter, since they share one cooldown. Kept per character in the sender's own saved settings. A mail sent with the mailbox's own Send button counts too. Location: GUI/CooldownsTab.lua (the sent column, CooldownsTab:SentInfo).

• Click Mail again to send the supply mail (operator, same day: "we need to be able to keep clicking the button to also send the mail"). Once the mail is attached and addressed, the next click on the same row's Mail button sends it through the mailbox's own Send handler, which clears any money or COD and sends exactly as the Send button does. A click on another row's Mail still refuses while the mail holds items, and a failed send can be clicked again. The send call is marked restricted in Blizzard's documentation; the operator confirmed in game on 2026-10-21 that the click on our button sends the mail and the Sent column marks the row. If the game ever blocks it (ADDON_ACTION_FORBIDDEN or _BLOCKED naming SendMail), or the mailbox's handler errors, the mail goes back to ready, the mailbox's Send is re-enabled and chat says to use it. The Sent record is only written when, at the moment of sending, the mail is still addressed to the supply mail's recipient and still carries items (a post-hook on SendMailFrame_SendMail, which both Send paths reach), so a different mail sent from the same mailbox does not mark the row. Peer Review on the self-audit. Location: GUI/CooldownsTab.lua.

Bug Fixes

• A supply mail sometimes carried two of a reagent it needed one of (operator, in game 2026-10-21: a Salt Shaker mail with 2 Deeprock Salt). A second click while the first click's split was still landing split again into the same empty bag slot, which stacked the two pieces before the attach picked the slot up. A click while a mail is being built is now ignored, with a "still preparing" line in chat; the operator confirmed the splitting in game the same day. Six specs in Tests/cooldownmail_spec.lua cover the double click, the send on the next click, another row's click, the Send button not ready, a failed send retried, and a mail emptied by hand. The same file could not load on its own, because the Cooldowns tab needs the shared widgets file first, as the addon's load order has it; it now loads it. Location: GUI/CooldownsTab.lua.

• Known By names were cut off in the Professions detail panel (operator, in game 2026-10-21: "you're truncating the names here when we have a lot of space in the row left"). Crafter rows share the reagent rows' columns, so a name stopped where the empty count, [Bank] and [AH] cells began. The name column now spans every following cell that draws nothing for its row, using the column span LibAceGUIWidgets added for this at MINOR 46. The reagent count moved to the last column, so a reagent name ("Refreshing Spring Water", which drew as "Refreshing..." in the same screenshot) also runs over a hidden [Bank]/[AH] and stops at its count. The operator confirmed both the crafter names and the reagent names in game the same day. A row with only one button still cut its name ("Raw Brillian..." before [Bank], with the empty [AH] slot after it), so both buttons now collapse when hidden (RowList col.collapse, also MINOR 46): the visible one sits beside the count and the name spans the rest. With an older LibAceGUIWidgets the names truncate as before. Spec in Tests/browserdetail_spec.lua. Location: GUI/BrowserTab.lua (BrowserTab:BuildDetailList). On a LibAceGUIWidgets below MINOR 46 the list keeps v1.6.1's column order (name, count, [Bank], [AH]), because that RowList ignores span and collapse and the count-last order would have parked the count behind two always-reserved button slots on every reagent row: a visible regression on any released library, not merely no improvement (Peer Review F1 on self-audit 3ea3379a). addon.W_LOADED_MINOR now records the loaded MINOR for features newer than W_MINOR, which stays 36 so an older library still runs the GUI. Two specs pin both orders by handing the list each MINOR; the RowList loaded offline is the working copy at 46, so the older library's own drawing is not exercised. The span spec now reads the public RowList:RowPlan that LibAceGUIWidgets added at MINOR 47 for it, instead of the private _rowPlan, which had already been renamed once under it (Peer Review F4). Location: TOGProfessionMaster.lua, GUI/BrowserTab.lua (BrowserTab.DETAIL_SPAN_MINOR).

• /upm settings printed the command list instead of opening the settings. The settings tooltip in every locale has named this command, but it was never in the dispatcher's table, so it fell through to help. It now opens the settings panel and is listed in /upm help (new SlashHelpSettings string in all fifteen locales). Found while switching the hints to /upm. Spec in Tests/slash_spec.lua. Location: TOGProfessionMaster.lua.

• Hints and usage lines still named /togpm after the rebrand: the cross-guild pull description, the minimap-hidden message (which also drew the command in the old TOG purple, now the brand orange) and the settings tooltip in every locale, and the usage lines for pullroster, dumprecipe, dumpprice, itemgaps, whyvisible and the blocked profession-open message. All say /upm now (Peer Review F3). Location: Locale/*.lua, TOGProfessionMaster.lua, Modules/Crafting/CraftingEngine.lua.

• Hiding the minimap button gave no hint how to bring it back. The "Minimap button hidden. Use /upm minimap to restore." line was translated in every locale but no code printed it. The Settings toggle now prints it when the button is hidden, and not when it is shown. Spec in Tests/settings_spec.lua. That file's Settings re-layout spec also failed when run on its own (6 timers where it expects 2): Ace3's Slider constructor schedules its own font workaround for each slider it builds new (Ace3/AceGUI-3.0/widgets/AceGUIWidget-Slider.lua:276), and a lone run has no pooled sliders yet. It now counts only timers scheduled from GUI/Settings.lua. Location: GUI/Settings.lua.

• Clicking + on a Guild tab profession often did nothing (operator, in game 2026-10-21: "the click isn't working"). The Guild tab renders every kind of data a guild sync carries, so the window tore it down and rebuilt it every few seconds. Each teardown hides the list frame and shows it again, and a hide between the mouse-down and the mouse-up cancels the click. A guild-data refresh now recounts and re-hands the tree to the list in place, the same path a filter change already took. The full redraw remains for the one change that needs it, the empty-guild message giving way to the list. The cause is read from the code and has not been confirmed in game (not verified): if a click still fails, it is not this. Three specs in Tests/guildtab_spec.lua, one of which watches the list frame for a hide. The same file's scale-change spec only passed in a whole-suite run, because an earlier file had created the main window table; it now supplies its own. Location: GUI/GuildTab.lua (GuildTab:RefreshInPlace), GUI/MainWindow.lua.

• Sorting Missing Recipes by Sources looked up every row's places in one frame. A row with no recipe-database sources gets its Sources text from the item library's place list for the scroll, which decodes every spawn and names each zone. On Retail that is nearly every row, and the sort asked for all of them at once (self-audit cd8c79f4, item 1). The sort now spends at most 4 ms on those lookups, sorts with what is ready, and finishes the rest one slice per frame, re-sorting and redrawing once when every key is known. The continuation drops itself if the list is rebuilt or the player sorts by another column. Not measured against Retail data: the fix bounds the work per frame regardless of its size. The place rows are also cached per scroll for the session (keyed by the item library object and the faction), because the rows are rebuilt on every search keystroke and sync, and without the cache every later sort redid every lookup (Peer Review). Five specs in Tests/missingrecipes_spec.lua pin the slice, the single redraw, the single continuation, the give-up and the cache. Location: GUI/MissingRecipesTab.lua.

• The Professions tab offered the classic trainer-rank tier filter on Retail, where it cannot classify anything. Retail measures each expansion line on its own scale (Khaz Algar Blacksmithing runs 1-100), while the bands are the classic 75-point ranks up to Zen Master's 600, so most bands matched no Retail recipe (self-audit cd8c79f4, item 5). Retail now draws no tier filter; every classic flavour keeps it unchanged. Traced on the way: the recipe gate's expansion number is harmless on Retail, because the only field it compares against is absent from every flavour's data, not only Retail's (Peer Review). That check is now marked defensive in the code, and two stale comments that called it the primary gate, and the skill cap a gate, are rewritten. Location: GUI/BrowserTab.lua (the filter moved into BrowserTab:AddTierFilter), Tests/toolbar_spec.lua.

• Three places showed an item as a number or "?" while the item library knew its name. The Missing Recipes tooltip for an uncached scroll and the name the [Bank] dialog opens with went straight to Item #<id>, and the Crafting tab's reagent list drew "?" for a reagent the modern trade-skill API handed over without a name. A new addon:ItemDisplayName gives the one bare-id chain (client name, then the item library, then the placeholder), and the Crafting reagent column now goes through addon:ResolveReagentName. The Crafting reagent hover, [Bank] and [AH] read the healed name and link too: the heal writes itemLink while the trade-skill reagent tables say link, so the healed link was never read (Peer Review). The guild bank request dialog's own placeholder now comes from the same helper. Filed in self-audit b37931c3 on 2026-09-11 and not done until now. Location: TOGProfessionMaster.lua, GUI/MissingRecipesTab.lua, GUI/CraftingTab.lua, Compat.lua, Tests/reagentname_spec.lua, Tests/craftingtab_controls_spec.lua.

[v1.6.1] (2026-10-20) - Every tab is the same size; the Craft button is a plain button for trade skills; the close path runs once

Bug Fixes

• The Cooldowns and Guild tabs were smaller than the rest, so switching to or from them made the window jump (operator, in game 2026-10-20). Those two tabs were locked at 720x500, while Crafting and the Profit Planner need at least 820 and 920 wide, and the other tabs shared a resizable size of their own. Every tab now uses one shared size, resizable, whose floor is the largest minimum any tab declares (920x540 today), so a tab switch never moves the window. Pinned by a new spec in Tests/mainwindow_spec.lua. Two existing size specs there were moved to the new floor, because the change made them fail (run 20261020-223619.605, red) rather than because they were wrong: the "restores a saved size that fits" case now saves 960 wide instead of 800 (below the new floor), with a new spec that a smaller saved size is raised to the floor; and the uncapped shopping-list control uses ten recipes instead of six, since 30 rows now fit in the taller tab. Green after the edit on run 20261020-223845.589. Location: GUI/MainWindow.lua, GUI/CooldownsTab.lua, GUI/GuildTab.lua.

• Clicking the main window's title bar closed the window (operator, in game 2026-10-19). The temporary chat trace never caught it: the one trace captured (on WoW Forever) was an ordinary Close-button click. No cause was identified. The operator reports it no longer happens in Classic Era (2026-10-20). The temporary trace is removed. Location: GUI/MainWindow.lua, Modules/Crafting/CraftingEngine.lua.

• Closing the window from code ran its close-down twice. Close() releases the AceGUI frame, and AceGUI hides it before clearing its callbacks, so the frame's own close callback re-entered the release mid-way and the bottom-row cleanup, the clamp restore, the Craft button re-check and the Crafting engine's window-closed handler all ran a second time. On a close from code the second run also undid the clamp restore, so the pooled frame always went back to AceGUI unclamped whatever its earlier setting (the X button ran the cleanup once and restored it correctly). The release path now returns at once when AceGUI is already releasing the widget. Found by Peer Review by reading. In Tests/mainwindow_spec.lua, two specs reproduce the bug and were seen failing before the fix (one close for Close(); a frame clamped before opening is still clamped after Close()), and one is a regression guard that already passed (one close for a direct hide). Location: GUI/MainWindow.lua.

• The Craft button showed greyed and took no clicks for a trade skill (in game 2026-10-18), while the debug line reported it enabled and above the window. The cause in the secure button was not found. Trade skills no longer use it: Craft is now a plain button inside the detail panel, built like Craft Max (which kept working), and crafts the selected recipe at the stepper's quantity through the same engine call. A trade-skill craft needs only the click, so nothing secure is required. The secure button now shows for Enchanting alone, where the enchant goes through a secure /cast (whether DoCraft itself is protected is not verified), and lost the trade-skill crafting it no longer used. An enchant with no name to cast, which got no button at all, now gets the plain button labelled Enchant, which opens Blizzard's Craft window. Craft, Craft Max and the queue's Craft Next and Craft All are disabled in combat, matching the Enchant button, which is hidden then. An enchant picked during combat is what the Enchant button casts once combat ends; it used to keep the one picked before. Craft Next and Craft All also come back after combat when the selected recipe left the list during it; they used to stay greyed until the queue next changed. Not yet seen in game. Location: GUI/CraftingTab.lua, Modules/Crafting/CraftingEngine.lua.

• The Craft button stayed on screen after TOGPM's window closed, and looked dimmed behind the window while it was open (in game 2026-10-18). The button is a secure frame kept outside the window on purpose, so it has to be told when to hide. It now shows only while the window is open on the Crafting tab, and closing the window re-checks it. Why the panel's own hide did not already hide it was not found; the window check is a guard, not the diagnosis. The placement watcher now also re-places the button when the panel changes strata. Whether that caused the dimmed look is not established: the button is disabled when nothing can be made, which looks much the same. The window's own level is part of the placement check as well, and the button is lifted above whichever of the panel and the window is higher. Each placement now prints its strata, level, enabled state, alpha and mouse state under /togpm debug. Uses only GetFrameStrata, a base frame method; checked against the Classic Era client source, not the other flavours' trees. Location: GUI/CraftingTab.lua, GUI/MainWindow.lua.

Improvements

• One recipe-id mapping instead of four copies. Turning a recipe id read from the client (the crafted item's id on Vanilla) into the spell id the shipped data and the guild data key on was written out separately in the scanner's merge, the crafter-set receive, the crafting engine and the Crafting tab's Profit Planner jump, with slightly different guards and nothing checking they agreed. All four now call addon:CanonicalRecipeId, pinned by Tests/canonicalrecipeid_spec.lua. The Crafting tab's copy used to try the crafted-item map even for a profession with no shipped data; that map is built from the shipped data, so it was always empty there and nothing a player sees changes. Location: TOGProfessionMaster.lua, Scanner.lua, Modules/Crafting/CraftingEngine.lua, GUI/CraftingTab.lua.

[v1.6.0] (2026-10-17) - Allied guild tags; Reagent Tracker scale, opacity and background; cross-realm supply mail

New Features

• Allied guild members carry a short guild tag after their name, like Ahmbrai <Other>, so it is clear at a glance who is in which guild (operator, 2026-10-17: "The Old Gods (TOG) and The Other Gods (TOG) show you the problem"). The tag is worked out automatically from the first word that tells the guild names apart once filler words ("the", "of") are dropped, cut to five letters, and never shared by two guilds -- so The Old Gods and The Other Gods read <Old> and <Other>. Settings -> Cross-Guild has an "Allied guild tags" box (Guild Name = Tag, one per line) to pick your own, and lists the tags in use. Home-guild members carry no tag. The tags are LibGuildRoster's (MINOR 35, GuildRoster 1.4.0), at the operator's direction ("it should feed that to every addon that consumes a sister guild"; inbox 49703e36): the library derives them and stores your own tags account-wide under the home guild, so every TOG addon shows one guild under one tag. TOGPM's readers (GetGuildAbbrev, GetCharGuildAbbrev, the Settings box) are thin wrappers over GetGuildTag / GetCharGuildTag / SetGuildTag, the name columns repaint on the library's OnGuildTagsChanged, and MigrateGuildTagsToLibrary (from OnRosterReady) moves any tags a working copy kept in profile.guildAbbrevs into the library once and deletes them. Under an older GuildRoster there are no tags and the Settings box is greyed out. Typing a tag another allied guild already shows keeps it (your choice) but says so in chat, naming that guild, from SetGuildTag's second return (Peer Review G1, inbox 2899de7d) -- once, when that tag is changed, not on every later save, and in the form the library stored (trimmed, at most 12 characters). TOGPM's own derivation is gone; its spec fixtures now run against the library's. Specs in Tests/sisterguild_spec.lua. Location: TOGProfessionMaster.lua, GUI/Settings.lua, Locale/enUS.lua.
• A supply mail the game refuses now says why in chat. After the Cooldowns tab fills in a mail, the game's own mail error (and a failed send) is repeated in chat with the recipient's name and the error's ERR_MAIL_ name until the mail goes or the mailbox closes. Only mail errors are repeated -- the first cut also blamed unrelated ones such as "Out of range." on the mail (Peer Review F1, inbox 60941923). Added for the operator's report of a mail to an allied-guild member that loops on the "might be someone you don't know" confirm (cause not yet known; GuildRoster asked to check its mailbox autocomplete, inbox 9aa8c30f). Spec in Tests/cooldownmail_spec.lua. Location: GUI/CooldownsTab.lua.
• Scale and opacity sliders for the Reagent Tracker (the shopping-list window opened by right-clicking the minimap button). Settings -> Display now has "Reagent Tracker scale" (50-150%) and "Reagent Tracker opacity" (0-100%) right below the main window's pair, and both apply live. Changing the scale keeps the window's top-left corner on the same spot on screen. Saved per profile as trackerScale / trackerOpacity. Specs in Tests/reagenttracker_spec.lua. Scale checked in game on Classic Era.
• The Reagent Tracker has a dark background, and the opacity slider sets only that. It had none, so the list read as see-through over bright ground. A black panel (50% at 100% opacity) now sits behind the rows; the slider fades just the panel, never the text, icons or buttons, and 0% removes it, matching the old look. The first cut faded the whole window, text included. Not yet checked in game. Location: GUI/ReagentTracker.lua, GUI/Settings.lua, Locale/enUS.lua.

Bug Fixes

• The Professions tab kept showing the recipe list from login until a /reload -- a guildmate learning a recipe, or someone leaving the guild, never reached it. CallbackHandler keeps one handler per (event, self) and a second registration under the same self replaces the first (Ace3 CallbackHandler-1.0.lua:126-129). BrowserTab registered its GUILD_DATA_UPDATED re-warm as addon:RegisterCallback(...) at load, and MainWindow registered its refresh the same way at OnEnable, so MainWindow's replaced BrowserTab's and the cache built 8 s after login was served all session (Peer Review H1, inbox 95fc14a4). Every addon-event subscription now registers as its own module (BrowserTab, MainWindow, CooldownsTab, addon.GUI, SyncLog, the Sync Log window's listener), and TOGPM's seven LibGuildRoster subscriptions register as the addon rather than as the library, a slot another addon making the same mistake would have shared (Peer Review M1). Specs in Tests/callbackkeys_spec.lua. Its first seven cases (the GUILD_DATA_UPDATED one and the six AH / SYNC key checks) were red against the old code (run 20261017-173040.786-2899038007, 0 of 7 passing); the seven LibGuildRoster key cases were added after the fix and have only been run green. All 14 pass (run 20261017-174130.037-3435744109). Location: GUI/BrowserTab.lua, GUI/MainWindow.lua, GUI/CooldownsTab.lua, GUI/SharedWidgets.lua, GUI/Settings.lua, Modules/SyncLog.lua, TOGProfessionMaster.lua.

• The crafter-online alert named the two characters the wrong way round. When an alt of a shopping-list crafter logged in, chat read "Crafter is online (alt of Alt)" instead of "Alt is online (alt of Crafter)" -- the two names were passed to the message in swapped order. Specs (the first for this alert) in Tests/crafteralert_spec.lua. Location: TOGProfessionMaster.lua.

• Right-clicking a crafter on the Professions tab whispered the display text, not the character -- the crafter list carried no character key, so the whisper fell back to the shown name, which can read "Alt (Main)". Each entry now carries its key -- the online alt's when an alt stands in for an offline crafter, so the whisper reaches someone who is on. Typing in the search box also no longer matches the allied-guild tag's colour code ("cff", "9d"). Specs in Tests/browserlist_spec.lua. Location: GUI/BrowserTab.lua.

• Cooldown supply mail to a crafter on another realm went to the wrong person (operator, 2026-10-16). The mail button cut the realm off the stored Name-Realm key before filling the To: box, and a bare name is delivered to that name on the sender's own realm. The recipient now keeps the realm when it is not the sender's, and stays a bare name on the same realm; the greeting still uses the bare name. Spec in Tests/cooldownmail_spec.lua. Not yet checked in game. Location: GUI/CooldownsTab.lua.

• The Reagent Tracker showed a scroll bar and one row too few (operator, 2026-10-15: "why did you add a scroll bar to the tracker?"). The list sizes its host to exactly the rows it holds, then counts the rows that fit by flooring the host's height; a height returned a hair short (likeliest at a tracker scale other than 100%, not verified in game) floored to one row fewer. The host now carries one unit of slack, and the window is sized from the list's own height so the slack never shows. LibAceGUIWidgets is told (inbox 62eb5c52) and RowList tolerates it itself from MINOR 44, so the slack is applied only under an older LibAceGUIWidgets. Spec in Tests/reagenttracker_spec.lua. Location: GUI/ReagentTracker.lua.

• Clicking a recipe on the Professions tab did nothing while TradeSkillMaster's Crafting window was open -- the recipe went into TSM's search box and our detail panel stayed on "Select a recipe" (operator, 2026-10-15, with a screenshot of TSM's search reading the clicked recipe). TSM replaces HandleModifiedItemClick with orig(link) or hookFunc(link) (LibTSMWoW Source/API/Item.lua:158-166), so its hook runs on every click and answers "consumed" whenever its search box takes the link, modifier or not. ItemLink.Click handed every click to that router, so a plain left click looked consumed and ClickRow returned before drawing. It now asks the router only when a modifier Blizzard's own router acts on is held (CHATLINK and DRESSUP everywhere, plus EXPANDITEM, which on Retail opens the profession window for profession gear), so nothing changes without TSM. Every item link click in the addon goes through this one function, so all of them are fixed. Specs in Tests/itemlink_spec.lua. Not yet checked in game. Location: GUI/SharedWidgets.lua.

• The shopping list on the Professions tab could still draw as an empty box with a scrollbar (Discord, 2026-10-13, one recipe on the list). v1.3.1 gave the section 2 px of slack against the client's whole-pixel snapping. That figure was a guess and was never confirmed in game, and at a large UI scale several snapped edges can take more than 2 px off. The list then floored one row to zero. The slack is now half a scale-1.0 row (SL_SLACK = 7), which is well past what pixel snapping should cost and still less than a row at scale 0.5 or above. That snapping is the whole cause is read from the code, not measured in the client. A new spec in Tests/browserdetail_spec.lua takes 4 px off the content and requires the row to draw. It was run with 2 px and fails, and only that spec fails. Verified in game on WoW Forever with one item (operator, 2026-10-13: "it looks good in forever with 1 item"). Not seen on the other flavours or with the scale change, and the poster's addon version is not known. Location: GUI/BrowserTab.lua.

• The shopping list did not re-fit when the shared text scale changed (Peer Review on the above). The section was sized once, at draw time. Another addon's Text Size slider (TOG Bank's) raising the LibAceGUIWidgets scale grew each row but not the section, so one item could floor to zero rows again until the tab was redrawn. The tab now registers a scale listener and refits the section while it is showing. The listener takes the tab as its owner argument, so the weak-keyed listener table does not keep a reloaded tab alive. The section's row height now comes from the scale itself rather than the list, so it does not depend on which listener runs first. The unused first SetHeight in Draw, a second formula for the same height, is removed. Spec in Tests/mainwindow_spec.lua (scale 1.0 to 2.0). Location: GUI/BrowserTab.lua.

• TOGPM handed three kinds of damaged widget back to the shared AceGUI pool (operator, 2026-10-13: "i think we may have some tabs not recycling widgets properly, and they are getting corrupted widgets from teh pool"). A read of every GUI file found three.

  • Every tab draw set the main TabGroup's LayoutFinished to nil. TabGroup copies that method onto each instance, so this deleted it from the pooled widget. Once the window closed, the next addon to get that TabGroup could not auto-size it. It is now SetAutoAdjustHeight(false) at creation, and AceGUI's Release clears that.
  • The shopping-list section's reference outlived the widget. The TOG Bank watcher's timer could refill a section already back in the pool, parking our list inside another addon's box. That box's release would later hide our list while it was showing in our window. The reference is now cleared by a release hook, which also covers the scale refit above.
  • Column headers turned word wrap off on a pooled label and never turned it back on. AceGUI's Label does not reset that, so the next owner's text was cut to one line. That included our own Missing Recipes empty-state message. A release hook turns it back on.
  • Also: the Professions tab's three next-frame redraws and the shopping-list refresh only checked that the tab had ever been drawn. They now check that the window is open on that tab (BrowserTab:IsShowing), so a close in the same frame cannot draw the tab into a TabGroup already back in the pool.

  Three specs in Tests/mainwindow_spec.lua. Each would fail against the old code (read, not run). CLAUDE.md said the TabGroup is not pool-recycled. That was wrong, and the rule now says how to undo raw changes. Whether these caused the Discord empty box is not shown. Location: GUI/MainWindow.lua, GUI/BrowserTab.lua, GUI/SharedWidgets.lua.

Improvements

• Removed: the Scan AH buttons on the Professions, Cooldowns, Missing Recipes and Crafting tabs (operator on Discord, 2026-10-17: "remove the scan AH button from TOGPM."; inbox 546bcca7). ItemDB's own button on the Auction House scans the whole house in one pass, and every [AH] button and price in TOGPM already reads that scan. The per-tab button looked items up by name in the game's item cache and silently skipped any it had not seen, which is why Missing Recipes missed patterns that were listed (Discord, GUNNEY, inbox efaad501). addon.GUI.MakeScanAHButton and CraftingTab:ScanAHItems are gone. What the button's callbacks also did -- repaint the visible tab's [AH] buttons when the AH opens, closes or a scan lands -- is now addon.GUI.SetAHRefresh(tab, fn), registered once per tab and run only for the active tab of an open window. The Crafting list is a row taller without the button, so craftscroll_spec scrolls to 35 rather than the list's end. Specs in Tests/sharedwidgets_spec.lua. The 12 specs that tested only the button are deleted (operator approval, 2026-10-17: "yes, delete the old tests and leftover test strings"), and so are the button's 11 locale keys (BrowserScanAH*, CooldownsScanAHDesc, MissingScanAH*, CraftScanAH*) from all 15 locale files. The six addon.AH scan functions only the button called (StartScan, StartFullScan, CancelScan, IsScanning, IsFullScanning, GetScanProgress) are deleted with their five specs in Tests/pricefacade_spec.lua, on Peer Review's finding (inbox 0b5694c8) that they had no production caller left. Location: GUI/SharedWidgets.lua, GUI/BrowserTab.lua, GUI/CooldownsTab.lua, GUI/MissingRecipesTab.lua, GUI/CraftingTab.lua, Modules/AHScanner.lua, Locale/*.lua.
• Less empty space above the shopping list — the shopping-list box is an AceGUI InlineGroup with no title, but an InlineGroup draws its border 17 px below its top for a title regardless. The Professions tab now lifts the border to the top of the group and puts AceGUI's own anchor back when the pooled group is released (a keyed OnWidgetRelease hook). The section's height sum drops the strip with it: the chrome is now SL_CHROME (23 px), one exported constant the specs read instead of a hard-coded 40. A spec in Tests/mainwindow_spec.lua checks the border is lifted while showing and back at -17 after the window closes. Location: GUI/BrowserTab.lua.
• A click spec through the row's real mouse handler — the recipe-list click spec called onRowClick directly, skipping the library's dispatch. A new spec drives the row frame's own OnMouseDown and checks the detail panel shows the recipe, not the placeholder. Location: Tests/browservirtual_spec.lua.
• The settings file is fully covered by the offline suite (Peer Review thread bf2313d2, item F4). New specs drive the purge buttons and their confirm dialogs, the alert options, the cross-guild options and the sync-log window, which no spec reached before. GUI/Settings.lua is at 881/881 executable lines, and the whole suite passes (1907, 0 failed). Location: Tests/settings_spec.lua.
• A report line for the toolbar "anchor family connection" error (seen in game when opening the main window from the minimap; cause not yet found, Peer Review thread 329b6368). Before a toolbar control's slot is laid out, the toolbar now checks whether that slot is already reachable through the toolbar's own anchors. The first time it is, it prints the chain of widgets once per session, so the next report names the stale widget. A loop is only one candidate: warcraft.wiki.gg names anchoring to the screen after a frame is gone instead. So if a control comes out of AceGUI's layout anchored to nothing, the toolbar also prints (once) every point of that control, of the toolbar's first control and of the toolbar, with each target and parent. AceGUI already runs its layout under its own error catcher, which hands the error to BugSack and carries on, so this changes nothing about what is drawn or logged; it only adds the chat line. Specs in Tests/toolbar_spec.lua drive the real AceGUI layout with a SetPoint that raises and one that does nothing. The test harness is pinned at baca63c, whose frames refuse an anchor loop with the client's message, so the loop spec now checks that AceGUI's layout hit that error and that both report lines print once. A control AceGUI leaves unplaced on purpose is skipped: any but the first child of a Fill layout, and one added after a full-height child in a Flow layout. Every other toolbar case now fails if it prints either report, so an ordinary draw can never show a player a false "please report this line". Location: GUI/SharedWidgets.lua.
• Spec files no longer leave Settings open or stubbed timers behind. Tests/settingsbleed_spec.lua closed Settings by calling the toggle, which opened it for every example that had not; it now closes through the dialog and moves the clock. Tests/browserlink_spec.lua, Tests/craftsuppress_spec.lua and Tests/pettraining_spec.lua now reinstall the base globals in teardown, so the next file does not load against their C_Timer / GetTime stubs.

[v1.5.0] (2026-10-13) - Guild tab: search and filter by crafting skill; the Reagent Tracker stays open and minimises

New Features

• Guild tab filter strip, built from LibAceGUIWidgets. The library's search box and two of its number steppers (Min skill / Max skill) sit above the list in toolbar slots.
  • The search matches every label on the page: a profession whose name matches keeps everyone in it, a specialization whose name matches keeps all its members, and otherwise only characters whose name matches are kept (case-insensitive, realm ignored; "You" finds your own characters; an account-mate's name finds its main, whose row reads "Alt (Main)"). It is applied in GuildTab:BuildTree, which recounts from what it keeps. Branches open when a search starts and close when it is cleared, never on a refresh in between.
  • The skill steppers filter on CRAFTING skill in the profession being counted, and run from 1 to addon.SKILL_CAP (300 Classic Era, 375 TBC, 450 Wrath, 525 Cata, 600 MoP; on Retail the highest recorded skillMax, else 100). The full range is no limit, and a bound filters only once moved off its end. A character with no recorded skill in a profession is hidden from it while the range is narrowed. The range is applied per profession in BuildCounts, so headcounts read "how many matching". Empty rows are left out while filtering.
  • The list re-lays its rows when its scroll box changes size. Before, the library re-laid only on its outer frame's resize; in game every row was 40px wide and the profession names drew as ".." while the scrollbar sat at full width, so the last layout read the scroll box before it had its width (inferred from the screenshot, not instrumented). LibAceGUIWidgets MINOR 42 fixes it in the library (contract 0f350cea), so the hook is installed only on an older copy (GuildTab:NeedsRelayoutHook).
  • The character counts form one 80px column, centred under the "Characters" header, which is itself centred over that column; the space right of it is left free for later columns. The list's valueColumn option is LibAceGUIWidgets MINOR 42 (contract a79b0269); an older copy ignores it and keeps the counts at the row's right edge. The library scales that column by its shared scale (another addon can set it), so the headers are sized through GuildTab:ColumnWidths with W:ScaledSize, and a scale change redraws the window while the Guild tab shows.
  • The steppers are parked across draws by the new addon.GUI.ToolbarStepper (GUI/SharedWidgets.lua), like the other toolbar controls, so a guild sync redrawing the tab mid-typing keeps the box and its number.
  • A debounced search that fires after a tab switch is dropped (per-draw token cleared on release), so it cannot write the Guild count into another tab's status bar. The tab's help tooltip describes the strip. Location: GUI/GuildTab.lua, GUI/SharedWidgets.lua, specs in Tests/guildtab_spec.lua.
• The Reagent Tracker stays up like Questie's tracker, and minimises. It is no longer in UISpecialFrames, so Escape does not close it. Whether it is open and whether it is minimised are saved per character beside its position (RT:State(), db.char.frames.reagentTracker), and PLAYER_ENTERING_WORLD reopens it (RT:Restore) after a logout or /reload. A new -/+ button left of the close button folds it to its title bar. Dragging now writes the position fields into that record instead of replacing it, so the flags survive a move. The window is always anchored by its top-left corner (RT:PinTop, on build and after every drag): anchored by its centre, as the default spot and the game's own drag anchor could be, minimising pulled the title bar down toward that centre. A position saved as a centre anchor by an older build moves its top once, by half the difference between the old height and 200, on the first open. SetDontSavePosition(true) keeps the client's own layout cache from competing with that saved anchor, and the window is clamped to the screen. Location: GUI/ReagentTracker.lua, specs in Tests/reagenttracker_spec.lua.

Bug Fixes

• Every icon in the Reagent Tracker was greyed out. The icon column returned select(10, GetItemInfo(id)), which is the icon AND every value after it; the RowList reads a second return as "desaturate", and the sell price came second. RT:IconFor now returns one value. The other icon columns were checked: each cuts to one value with and ... or or a plain field. Location: GUI/ReagentTracker.lua.

• The Guild tab's column headers were drawn over the first rows of the list. The list's group was added to the tab's Flow layout while the header group above it was still empty, so it was placed at the very top with the full height; filling the headers afterwards never moved it. Draw now lays the tab out again once the top group has its real height. Location: GUI/GuildTab.lua.

[v1.4.0] (2026-10-11) - Item tooltips: an on/off setting for every line, no duplicate sell price, no doubled crafter names

New Features

• Every line TOGPM adds to an item tooltip has its own on/off setting. Settings > Item tooltip gains five toggles: recipe difficulty, recipe sources, your characters who haven't learned the recipe, vendor buy price and vendor sell price. Crafters and IDs already had one each. The vendor rows had no setting at all; they were driven by one hidden key, tooltipVendorPrices, which addon:MigrateVendorPriceToggles carries onto both new keys once (off stays off) and then deletes -- at login and again on every profile switch or copy, since a login only reaches the active profile. A row switched off is not priced at all. The recipe-details mode still decides whether the recipe block may appear at all, so its three part toggles are greyed out under "Never". A part switched off is dropped before the block's "is there anything to show" test, so a block left with nothing still writes no heading. Location: GUI/Settings.lua, GUI/SharedWidgets.lua, TOGProfessionMaster.lua, Locale/enUS.lua.

Bug Fixes

• The crafters line and the IDs checkbox read a missing setting the opposite way from each other. Tooltip.lua treated a missing tooltipShowCrafters as off although crafters has shipped on since v1.0.7 and its checkbox read it as on; the IDs checkbox read a missing tooltipShowIds as on although IDs ship off. Every reader now takes the shipped default. A spec that asserted the old silent default was rewritten to the shipped one. Location: Tooltip.lua, GUI/Settings.lua.

• The Item tooltip header shared order 19.5 with the cooldown reminder field, a tie between the two. The header is now 19.505. Location: GUI/Settings.lua.

• The vendor block repeated the game's own sell price. Classic's client prints "Sell Price:" on item tooltips while a vendor window is open (its OnTooltipAddMoney script, classic_era GameTooltip.lua:297), and Retail's tooltip data has a SellPrice line type for the same thing (TooltipInfoSharedDocumentation.lua:45; when Retail emits it was not checked). Our "Vendor Sell" row then showed the same number a second time. The hooks now notice the game's line and our sell row stands down; the buy row, which the game never shows, stays. Whether the Classic OnTooltipAddMoney script fires before or after our item hook on every client has not been checked in game: where it fires after, the duplicate remains as before. Location: Tooltip.lua, GUI/SharedWidgets.lua.

• A crafter who makes an item through two professions was listed twice ("Bob, Bob"). The crafters line deduplicated per character AND profession, and seven Vanilla items come from two (Gold Bar is Mining's Smelt Gold and Alchemy's Transmute Iron to Gold). Deduplicated per character now. Location: Tooltip.lua.

• The crafters line and the IDs line scanned every shipped recipe on every item hover (~1,565 on Vanilla, twice with the IDs line on). They read the cached item-to-recipe index the recipe block already used, filtered to crafted items so a teaching scroll is never credited to the recipe's crafters. Each item's recipes are sorted, so "the first recipe" the recipe block and the IDs line describe no longer depends on table layout; the sort falls back to comparing as text, so a non-numeric key cannot make it raise. Location: Tooltip.lua.

• The legacy OnTooltipSetItem hook skipped any item while the previous item's flag was still set. It tested the flag for truthiness instead of comparing it to the hovered item, the same defect the Show fallback had already fixed for itself. The vendor block's per-hover guard had the same shape and now compares to the item too. Location: Tooltip.lua, GUI/SharedWidgets.lua.

• A tooltip could carry two TOGPM labels. The crafters and IDs lines open with "[TOGPM]" and the recipe and vendor blocks with their own "TOGPM" heading; the vendor block checked only whether the recipe block had written one. A crafted reagent with a guild crafter and a vendor price showed both. One flag now records that a label was written, and every later block continues under it. Location: Tooltip.lua, GUI/SharedWidgets.lua.

• Per-hover flags could outlive their item where OnTooltipCleared does not run between two hovers, which the Show fallback's own comment says can happen. The recipe block and the brand heading then stayed suppressed for the rest of the session, and the new game-sell flag could hide the sell row on every later item. When a different item arrives while the last one's flag is still set, those flags are now reset as Cleared would have; the game-sell flag holds the item it was set for, and the modern post-call assigns it per item rather than only ever setting it. Raised by Peer Review on the self-audit thread (245309dd). Location: Tooltip.lua, GUI/SharedWidgets.lua.

• The IDs line's crafter count could disagree with the crafters line. It counted the first recipe's raw crafters, including ones the visibility gate hides, so it could read "crafters=1" over an empty crafters line. It now counts the same list the crafters line draws. The two IDs-line builders are one function. Location: Tooltip.lua.

• Tooltip.lua disabled every tooltip hook if AceHook-3.0 was missing, a library the file never calls (its hooks are the frames' own HookScript and hooksecurefunc). The guard is gone. The crafters header's fallback colour was also a stale purple rather than the brand orange. Location: Tooltip.lua.

Improvements

• Tests: specs for each fix above, plus the crafters sort (online first, then name) and the WoW Forever TooltipUtil path, neither of which any spec reached before. Tooltip.lua and GUI/SharedWidgets.lua are at 100% line coverage. Location: Tests/tooltip_spec.lua, Tests/tooltiphooks_spec.lua, Tests/vendorprices_spec.lua, Tests/sharedwidgets_spec.lua.

[v1.3.1] (2026-10-07) - The CurseForge app installs LibDBIcon again; Missing Recipes shows recipe names

Bug Fixes

• The CurseForge app stopped installing LibDBIcon on every client except WoW Forever. The copy of LibDBIcon bundled for the Forever TOC still had the packager's @curseforge-project-slug: libdbicon-1-0@ marker on line 1. The BigWigs packager scans every packaged .lua for that marker and lists the slug as an embedded library for the whole upload, which overrides .pkgmeta's required-dependencies (release.sh 1880-1890). So the upload told CurseForge LibDBIcon was embedded, and the app did not install the standalone addon that the Era, TBC, Wrath, Cata, Mists and Retail TOCs need before they can load. The marker is removed, and Tests/loadorder_spec.lua (PKG-EMBED-001) now fails if either Forever-embedded file carries one. Found by TOGBankClassic's and Peer Review's audits (inbox 5503f91c, 5d7eaf86). Whether the v1.3.0 file on CurseForge shows LibDBIcon as embedded was not checked on the site. Location: Libs/LibDBIcon-1.0/LibDBIcon-1.0.lua.
• Missing Recipes cut recipe names to about ten characters and the window could not be widened (Discord, 2026-10-07: "I cannot read what the recipe is without scrolling over it and there is no way to move columns to the right to read it"). The tab was locked at 720x500 to match Cooldowns, and Sources took a fixed 180 px for a word like "Drop". Recipe is the list's auto-width column, so it got only what was left over. The tab is now resizable and shares the saved size of Professions, Crafting and Profit Planner, and Sources sizes to its widest text (autoFit, minimum 60), so the space goes to Recipe. Three specs in Tests/missingrecipes_spec.lua. Verified in game (operator, 2026-10-07: "the rows are indeed much wider"). Location: GUI/MissingRecipesTab.lua, GUI/MainWindow.lua (comments).
• The shopping list on the Professions tab could draw as an empty box with a scrollbar (operator, 2026-10-07: "i can't see the stuff on my shopping list anymore", one recipe on the list). Since the move to the LibAceGUIWidgets RowList in v1.2.0, the section was sized to exactly its rows plus the InlineGroup's 40 px of chrome. The client snaps anchors to whole pixels, so the row area could come out a fraction short. The RowList floors height / row height, so one row became zero, and a longer list lost its last row. The section now gets 2 px of slack (SL_SLACK). A new spec in Tests/browserdetail_spec.lua takes half a pixel off the content and requires the one row to still draw; it fails without the slack. The diagnosis is read from the code and fits the screenshot, but not yet confirmed in game. Location: GUI/BrowserTab.lua.

[v1.3.0] (2026-10-04) - Retail support; WoW Forever's borrowed ProfessionDB values are shown as unconfirmed

New Features

• Retail (operator, 2026-10-04: "lets add retail support, i belive PDB was updated enough for that"). A new TOGProfessionMaster_Mainline.toc (Interface 120005 / 120007 / 120100) loads the same files with the same dependencies as every other TOC; each required library already lists those interfaces, and LibProfessionDB MINOR 13 ships its own Retail data tree. Retail runs the C_TradeSkillUI scan and craft path built for WoW Forever. addon.isRetail (Interface 100000+, the bound LibProfessionDB uses) makes Jewelcrafting, Inscription and Archaeology available and hides First Aid. Retail has no single expansion skill cap, so addon.SKILL_CAP is nil there and the Guild tab shows the scanned max. Retail recipes carry a per-expansion skillLine, and their skill numbers are on that line's scale: the new addon:GetRecipeSkillRank reads the local character's rank for that line live (C_TradeSkillUI.GetProfessionInfoBySkillLineID) for "Can learn now" and the scroll tooltip's Requires line, and answers "unknown" (no gate) for another character, whose per-line ranks are not synced yet. The profession registry refresh no longer requires GetNumSkillLines, which Retail lacks; GetProfessions serves. wow-version-replication.ps1 now also replicates to _retail_. Loads and scans in game (operator, 2026-10-04: "retail opens now without errors, i was able to populate my retail data in"). Location: TOGProfessionMaster_Mainline.toc, Compat.lua, TOGProfessionMaster.lua, Scanner.lua, GUI/GuildTab.lua, GUI/MissingRecipesTab.lua, GUI/SharedWidgets.lua, Modules/CommTest.lua, Tests/compat_spec.lua, Tests/loadorder_spec.lua.
• Retail: every character's per-expansion skill syncs, so alts are gated too. A profession record may carry lines ([skillLine] = { skillRank, skillMax }), recorded by the owner's window scan from C_TradeSkillUI.GetChildProfessionInfos and sent as an optional l field on the owner's existing professions: leaf, re-minted by the owner only (peers adopt the hash as before). Only a Retail scan writes lines, so a Classic client's payload and minted hash are unchanged, and the v1.2.0 reader ignores l. GetRecipeSkillRank answers another character's synced line rank. Four writers that rebuilt a profession record from scratch now keep its lines. On Retail the profession-level rank has one writer, the profession registry (GetProfessionInfo); the window scan no longer overwrites it with the open line's rank, which would have flipped the record and re-minted the leaf whenever the two differed. Location: Scanner.lua, Modules/HashManager.lua, Compat.lua, Tests/retaillines_spec.lua.

Bug Fixes

• Retail: recipe names had no quality colour, and "Can learn now" could hide every recipe (operator, in game 2026-10-04). Newer Retail recipes ship from ProfessionDB with only their plans' itemId and no craftedItemId (1229652 Blood Knight's Mercy), so the row had no crafted item to colour by; GetRecipeCraftedItemId now falls back to the client's C_TradeSkillUI.GetRecipeSchematic(id, false).outputItemID, cached per recipe (ProfessionDB asked for the field). ItemLink.QualityHex also reads Retail's |cnIQ<n>: link colour and a quality string with or without its |c. And with no profession window open, Retail's GetProfessionInfoBySkillLineID answers skill 0 for a line the character has (/run in game printed 0), which "Can learn now" read as a rank; GetRecipeSkillRank now treats 0 as unknown and falls back to the rank recorded at the last scan. Location: TOGProfessionMaster.lua, GUI/SharedWidgets.lua, Compat.lua, Tests/compat_spec.lua.

• Retail: a scan of one expansion could strip the others' recipes. The merge removed the character from EVERY recipe of the profession before re-adding what the scan listed, and whether Retail's recipe list covers every expansion line or only the open one is not verified. A Retail scan now removes the character only from recipes on the lines it actually covered (the recipe DB's skillLine, else GetTradeSkillLineForRecipe); an unlearned recipe on a scanned line is still removed. A re-scan listing a recipe whose line cannot be placed no longer reports a change and redraws every time. Classic and Forever are unchanged. Location: Scanner.lua, Tests/retaillines_spec.lua.

• WoW Forever: a skill value ProfessionDB borrowed from Vanilla anchored the orange skill-up tier (LibProfessionDB MINOR 13 notice, thread 9ac3ba2a). The tiers are unanchored when difficulty[1] == 1 and requiredSkill is absent OR borrowed (IsBorrowed(id, "requiredSkill")), feature-detected so older ProfessionDB builds behave as before. The recipe id now reaches every FormatSkillTiers call. The Missing Recipes skill sort used the raw first tier, so placeholder tiers sorted as skill 1; it now uses the same learn-skill rule as the "Can learn now" gate. Location: TOGProfessionMaster.lua, Modules/Crafting/CraftingEngine.lua, GUI/CraftingTab.lua, GUI/MissingRecipesTab.lua, GUI/SharedWidgets.lua, Tests/borrowed_spec.lua.

• Missing Recipes said "Unknown" for recipes ItemDB knows where to get (operator, 2026-10-04: "itemdb has a lot of the drop info, and you're showing it as unkown"). The Sources cell read only ProfessionDB's sources, and Retail ships none, so a row whose [Where] listed a quest and a vendor said "Unknown". When ProfessionDB has no sources for a recipe, the cell now names the kinds of place LibItemDB knows for its scroll -- the same rows [Where] opens, with the same faction filter -- mapped to the usual labels (Drop and Boss as Drop, Vendor, Quest, Crafted, Object as Container), and the Sources sort uses the same text. Worked out once per row. Location: GUI/MissingRecipesTab.lua, Tests/missingrecipes_spec.lua.

Improvements

• Borrowed values are shown and marked unconfirmed (operator's direction via ProfessionDB). The Requires line reads "(85, unconfirmed)", and the recipe Sources heading and Missing Recipes source text say "unconfirmed" when ProfessionDB borrowed them. A recipe hidden only by a borrowed "never implemented" flag is no longer hidden. Location: GUI/SharedWidgets.lua, GUI/MissingRecipesTab.lua, Modules/RecipeGate.lua, Locale/enUS.lua, Tests/recipegate_spec.lua.
• A recipe shown only on a borrowed "never implemented" flag says so. Its Professions and Missing Recipes rows and the detail panel header read "Name (unconfirmed)", and its tooltip adds "Unconfirmed: this recipe may not exist in this version of the game." -- outside the recipe-detail block, so the "never" setting cannot hide it. RecipeGate:IsUnconfirmed and the gate read the flag through one helper, so a recipe the gate let through on a borrowed flag is always the one marked. Location: Modules/RecipeGate.lua, GUI/BrowserTab.lua, GUI/MissingRecipesTab.lua, GUI/SharedWidgets.lua, Locale/enUS.lua, Tests/borrowed_spec.lua, Tests/browserdetail_spec.lua.
• The Professions and Guild tabs build their lists with less work. Three openperf_spec budgets failed in the 2026-10-04 run (the All-professions list build 349 ms, the Browser cold draw up to 526 ms, both against 250 ms). Profiled, not guessed: the guild-tag hash ran 541 times per build and is now cached per guild key (registration still runs every call); each crafter's display name and online state is resolved once per build instead of once per recipe; the crafter sorts no longer build a comparator per recipe; the search text is one table.concat; and the Guild tab's counts ask the guild-scope check once per character instead of once per recipe-crafter pair (~37k calls for ~700 characters). Same output, specs unchanged. Location: TOGProfessionMaster.lua, GUI/BrowserTab.lua, GUI/GuildTab.lua.
• The perf budgets measure TOGPM, not the machine. On Windows os.clock is wall time (Microsoft's CRT clock() "doesn't strictly conform to ISO C"), so one sample includes whatever else the box is doing: across 2026-10-04's runs the same Profit Planner draw read 27 to 1089 ms. Tests/wowapi moves from 515873b to 99111aa (WoWAPITesting 953768d: bit 5-12x faster, and wow.bestTime, inbox 0822420a). Every repeatable budget now gates on the fastest of five runs; RebuildOnFirstLoad, which works only on its first call, is timed five times on a fresh copy of the tables it writes. The other login stages mutate the database and still run once. The wire budget times every crafters leaf once and re-times only the worst five times: five passes over every leaf ran the file past the runner's 60 s limit under load. Even best of five cannot beat a machine pinned for the whole run (the Professions draw read 402 ms at 100% CPU, 72 ms on a quieter run), so the budgets moved out of the default suite: Tests/openperf_spec.lua is now Tests/perf/openperf.lua, outside the runner's *_spec.lua pattern, and .writ-suites.json declares it as its own suite to run on purpose (WoWAPITesting's answer, inbox d53bb661). The same file suppresses .busted, which must never be run directly. Location: Tests/openperf_spec.lua, Tests/wowapi.

[v1.2.0] (2026-10-01) - Every window and list moves to LibAceGUIWidgets; WoW Forever: the Crafting tab works, and Forever characters' recipes sync

New Features

• [Where] on Missing Recipes rows: where to buy or farm the pattern (Discord request 2026-09-29, "add integration with questbook to lead you to where patterns are sold/dropped"). A [Where] button on each row whose recipe scroll LibItemDB knows a source for opens ItemDB's "Where to get it" window on that scroll: every vendor, drop and zone, with coordinates. With Questbook installed, clicking a place there guides the player to it -- ItemDB hands the waypoint to Questbook (LibItemDB:WhereTrack), so TOGPM never talks to Questbook and works the same without it. No button for a trainer-taught recipe (no scroll), for a scroll with no known source (WoW Forever's ItemDB ships no places data), or with a LibItemDB older than the window. Whether a row has sources is asked once per row, not on every paint. The Professions tab has no [Where] yet: its rows are recipes someone already knows. Not yet tried in game. Location: GUI/MissingRecipesTab.lua, Locale/enUS.lua, Tests/missingrecipes_spec.lua.
• [Guide] on the Missing Recipes tab and a stop icon on the window: start and cancel Questbook's route from TOGPM (operator, 2026-10-01: "give us the guide route/cancel route inside TOGPM too"). With Questbook installed, a [Guide] button on each row hands the best place for the recipe scroll straight to Questbook -- one in the player's current zone first, then a vendor, then the highest drop chance -- and says in chat where it is guiding to; [Where] still opens the full list to pick another. The window's bottom row carries Questbook's own stop control beside the gear, on every tab (operator: "use the same button/function from questbook"): its red X (ReadyCheck-NotReady), dimmed to 0.4 while Questbook guides to nothing (Questbook's public IsTracking, re-checked four times a second while the window is open, since guiding can start from Questbook itself), with its tooltip, and Questbook's universal stop. A first cut put a "Stop Guide" button on the Missing Recipes toolbar, where it wrapped onto a row of its own and existed on that one tab (in game, 2026-10-01); removed. The route and the stop go through LibItemDB (WhereTrack / WhereStopTracking); TOGPM only checks that Questbook is installed, with the same test LibItemDB makes, and reads IsTracking for the dim. Without Questbook neither control is drawn: the route is Questbook's, and ItemDB's data alone gives the places and coordinates but no arrow. Questbook is added to ## OptionalDeps in all six TOCs. The route was confirmed working in game by the operator (2026-10-01); the stop icon is not yet tried there. Location: GUI/MissingRecipesTab.lua, GUI/MainWindow.lua, Locale/enUS.lua, all six TOCs, .luacheckrc, .luarc.json, Tests/missingrecipes_spec.lua, Tests/toolbar_spec.lua.

Bug Fixes

• The Crafting tab's Enchant button stayed greyed out and could not be clicked, even with the materials in the bags (Discord, 2026-09-29, Classic Hardcore, v1.1.2). Since v1.1.2 the secure Craft/Enchant button lives on its own holder outside the window, placed over the detail panel at the panel's frame level + 10. The window is Toplevel (AceGUIContainer-Frame.lua:194), so any click on it Raises it within FULLSCREEN_DIALOG -- above the button, which is not its descendant. The watcher that keeps the button over the panel compared only position and scale, so it never noticed: the button sat under the panel's backdrop (dimmed) and the panel took its clicks. The watcher's key now includes the panel's frame level, and the button is stacked 100 levels above the panel (the Cooldowns popup's margin) so it also clears the panel's own children. It affected the Craft button for every profession, not only Enchanting. Spec: the button stays above the panel after it is raised. Not yet tried in game. Location: GUI/CraftingTab.lua, Tests/craftingtab_draw_spec.lua.
• The Cooldowns tab's [Bank] button raised "attempt to index local 'anchorBelow' (a number value)" (in game, 2026-09-30). Its click passed select(2, addon.Item.GetInfo(id)) as the LAST argument to addon.Bank.ShowRequestDialog, so every return after the link spilled into the dialog's fourth parameter, anchorBelow, and the dialog indexed a number. The group popup's [Bank] carried the same call. Both now wrap the select in parentheses, which keeps it to one value, and the dialog anchors only to a frame, so a stray value can no longer stop it opening. Spec: the popup's [Bank] hands the dialog exactly three arguments. Location: GUI/CooldownsTab.lua, Compat.lua, Tests/cooldowndraw_spec.lua.
• WoW Forever: clicking the Crafting tab raised ADDON_ACTION_BLOCKED for CastSpellByName() (in-game report, 2026-09-28), and the tab could not read or craft anything there. Engine:OpenProfession opened a profession by casting it; the report's stack is the tab click, a hardware event, so Forever blocks that cast from addon code even on a click, where Classic Era allows it. And every read and craft went through the classic trade-skill globals (GetTradeSkillLine / GetNumTradeSkills / GetTradeSkillInfo / DoTradeSkill), which Forever does not have at all -- not in its source tree, not even as deprecation fallbacks. The engine now has a second path on C_TradeSkillUI, the API Forever's own Professions UI uses, taken only where the classic globals are absent (addon:HasModernTradeSkillAPI()):
  • Open: C_TradeSkillUI.OpenTradeSkill(skillLineID) on the profession's skill line, from the tab click, the dropdown and a Profit Planner jump. warcraft.wiki.gg marks it restricted with the hwevent tag only, so a click may call it. No Open button on Forever (operator, 2026-09-29: "you should not have added the button"). UNRESOLVED: a tab click still raised ADDON_ACTION_BLOCKED for OpenTradeSkill() on some clicks only (in game, 2026-09-29). A secure button and a "use your profession book" prompt were both tried and removed the same day. Every attempt is now recorded (Engine:_LogOpenAttempt: trigger, skill line, the open base and child profession, first call of the session, calls in the same frame, combat, mouse button, whether Blizzard_Professions is loaded, the result, a stack) and paired with any ADDON_ACTION_BLOCKED or ADDON_ACTION_FORBIDDEN, which is announced in chat. /togpm opendebug prints the record. Peer Review's leading hypothesis (thread 4d8158c1): Blizzard's own callers skip OpenTradeSkill when that profession is already open, and the tab does not.
  • Switching professions from the dropdown drew the old one (in game, 2026-09-29: the K window "pops up briefly, a flicker, then closes, and it doesn't change to the tradeskill i select"). A switch while a profession is open lands on TRADE_SKILL_LIST_UPDATE once the new data source is built, not on TRADE_SKILL_SHOW -- which is where Blizzard's own window switches, yielding while IsDataSourceChanging() (Blizzard_ProfessionsFrame.lua :138-161). The engine treated that update as a light refresh, so the tab kept the old profession. It now redraws the whole tab when the update shows a profession other than the one last drawn (Engine._drawnProf), and also listens for TRADE_SKILL_DATA_SOURCE_CHANGED. Every event is logged for /togpm opendebug.
  • The K key could not open Blizzard's window after the tab had opened a profession (in game, 2026-09-29). The tab's claim on the session hid that window on every show. On Forever a show of ProfessionsFrame that does not follow the tab's own open within OWN_OPEN_WINDOW (2 seconds: a margin I chose, not measured; the event log records the real gap) is now the player's choice: the window stays and the claim is dropped, as the WoW UI button does. Classic's frames keep the old rule. Tests/pettraining_spec.lua now loads the real Compat.lua, which the engine asks on every event.
  • The dropdown always landed on the last profession tab (in game, 2026-09-29; the debug log showed every pick ending on Fishing, "Bait and Tackle"). When ProfessionsFrame goes from hidden to shown, each of its profession tabs that is not the open profession casts its own profession spell (Blizzard_ProfessionsTemplates.lua:963-971), each cast opening that profession, so the last tab wins. The takeover HID the frame, so every open from the tab re-showed it and set this off. On Forever the frame is now CLOAKED instead -- alpha 0 with mouse input off on it and every frame inside it, restored on reveal -- and stays shown, so a switch does not re-fire it. Its size is never touched: a first build also scaled it to 1%, and the revealed window came up far narrower than Blizzard's normal one with its Create bar off the edge (in game, compared against an untouched window), and a relayout call to correct that made it worse. The first open, which has to show it, asks for the profession a second time once the cascade is over. Operator's conditions (2026-09-29): K must show Blizzard's window -- the TOGGLEPROFESSIONBOOK keys are override-bound to reveal it while cloaked, since ToggleProfessionsBook would toggle the shown frame closed -- and both windows may be up at once: revealing it (K, or the WoW UI button) leaves TOGPM open. Closing TOGPM while cloaked closes Blizzard's window through its own HideUIPanel, ending the session. Not verified in game: all of it, including whether the second ask goes through inside the same click, and what Escape does with a cloaked frame.
  • Read: the open profession from GetChildProfessionInfo, falling back to GetBaseProfessionInfo when the child id is 0, as Blizzard_Professions.lua:1665 does; learned recipes from GetAllRecipeIDs (falling back to GetFilteredRecipeIDs) plus GetRecipeInfo, grouped under GetCategoryInfo names; difficulty from relativeDifficulty; reagents from GetRecipeSchematic's basic reagent slots with bag counts.
  • Craft: C_TradeSkillUI.CraftRecipe(recipeID, count), as Blizzard_ProfessionsTransaction.lua:352 does. The row index on this path is the recipe id.
  • Other windows: Forever's ProfessionsFrame joins TradeSkillFrame / CraftFrame in the foreign-window suppression and the TOGPM toggle button. Classic Era, TBC, Wrath, Cata and MoP have the classic globals and never take this path. Not verified in game: the cause of the intermittent block, whether Forever has GetAllRecipeIDs, and whether hiding ProfessionsFrame keeps the session open. Location: Modules/Crafting/CraftingEngine.lua, Compat.lua, GUI/CraftingTab.lua.
• WoW Forever: a character's own recipes were never recorded or synced to the guild (known gap since v1.1.2). Scanner:OnTradeSkillEvent returned early without the classic API. New Scanner:ScanModernTradeSkillInto records the learned recipe ids (spell ids, the key addon.recipeDB uses) through the same MergeRecipesIntoGdb, skill rank and cap included. It skips a linked or NPC-crafting session, and it never merges an empty list, which would strip every recipe from the character's crafter set while the data is still loading. Location: Scanner.lua, Compat.lua.
• WoW Forever: switching professions stored one profession's recipes under the others. In game (2026-09-29, debug log) every profession switched to scanned "16 recipes", the Blacksmithing count, and the result was broadcast: the list the client handed back did not follow the open profession. Scanner:ScanModernTradeSkillInto now skips while C_TradeSkillUI.IsDataSourceChanging() (as Blizzard_ProfessionsFrame.lua :142 does) and keeps only recipes that GetTradeSkillLineForRecipe places in the open base profession (ProfessionsUtil.lua:79-82); a recipe it cannot place is kept, as before. A character's wrongly stored recipes are replaced the next time each profession is opened and scanned, since a scan rewrites that character's whole set for the profession. Specs in Tests/scanner_scan_spec.lua. Location: Scanner.lua.
• New specs in Tests/craftsuppress_spec.lua for the Forever path: never opens through OpenTradeSkill and never casts, drops the claim when nothing opens, records each attempt (including one that raises), the list, the reagents, the craft call, and the classic path winning wherever both exist. The spec now loads the real Compat.lua rather than a copy of its check. C_TradeSkillUI is added to .luacheckrc and .luarc.json. The new CraftUnsupportedClient note (English only) is now shown only on a client with neither trade-skill API. The Scanner's Forever scan has no spec yet. Location: Tests/craftsuppress_spec.lua, Locale/enUS.lua.
• Re-ticking every skill tier on the Professions tab never went back to "all tiers". The toggle ended self._selectedTiers = all and nil or sel; in Lua true and nil is nil, so the expression always yields sel, and a player who unticked a tier and ticked it again was left on an explicit set that no longer grew with the client's tiers and persisted that way. Now an if. The same idiom in Modules/CommTest.lua stored the string "nil" as the send error on every successful probe (read only on failure, so never shown); fixed the same way. Found by the new Tests/toolbar_spec.lua. Location: GUI/BrowserTab.lua, Modules/CommTest.lua.
• The Professions tab's crafter column re-joined the whole name list for every name it tried. fitCrafterText now extends the string one name at a time. The cold draw on the reporter-scale database had gone from ~140 ms to 676 ms against a 250 ms budget; it is 197 ms again. Location: GUI/BrowserTab.lua.

Improvements

• The spell helpers now use the namespaced API first on every client. addon.Spell.GetInfo / GetTexture / GetLink tried the bare global first, for a test-only reason: the offline harness had no C_Spell. WoWAPITesting delivered it (a7675ce, with a WoW Forever build), and C_Spell.GetSpellInfo / GetSpellTexture / GetSpellLink / GetSpellCooldown are documented on Classic Era and Classic (Cata/MoP) as well as Forever, so every client now takes the same branch; the bare name is the fallback. GetInfo's second return (rank) is nil on this path; no caller reads it. addon.Spell.IsKnown now asks C_SpellBook.IsSpellInSpellBook first too, and its comment was wrong: the bare IsSpellKnown is a deprecation fallback on Classic Era as well, not only on Forever (Deprecated_SpellBook.lua:16 in the classic_era tree). The specs feed spells through the harness's wow.spells / wow.knownSpells instead of stubbing bare globals, and Tests/compat_spec.lua drops its hand-built C_Spell / C_SpellBook for the harness's, with a Forever case. Tests/wowapi moved to 0e90e7d. Location: Compat.lua, Tests/compat_spec.lua, Tests/scanner_scan_spec.lua, Tests/scanner_cooldowns_spec.lua, Tests/scanner_names_spec.lua, Tests/recipedetails_spec.lua.
• The Professions tab's recipe list and shopping list are LibAceGUIWidgets RowLists (LAGW adoption step 5b). The last two hand-built row pools in the addon. The recipe list was 35 raw frames over an AceGUI ScrollFrame, with its own header bar, index arithmetic and a container LayoutFinished hook that anchored the scroll to that header bar and the detail panel -- the hook behind v1.0.6's "the list is drawn over the game world after opening Settings". It is now a RowList on a host the tab owns for the session (addon.GUI.ParkList), anchored inside its own group only, with the header, hover highlight, selection tint (matched by recipe id, so it survives a guild-data rewarm), scroll memory (ListScroll key browser) and a [Bank] button column. The crafter column is fitted with the list's own cell width and measure (fitCrafterText, now pure), so the WINDOW_RESIZED repaint nudge is gone. The shopping list is a second RowList: a +/- expander per recipe with reagents, reagent rows indented under it, [Bank] / [AH] / "!" / [-] qty [+] / <input disabled="disabled" type="checkbox" checked="checked" /> as button columns, capped at its share of the tab in whole rows and scrolling inside. The tab lays out with Flow so the list and detail panel take the rest of its height. Removed: BuildPool, DestroyPool, UpdateVirtualRows, EnsureHeaderBar, EnsureShoppingListScroll, DetachShoppingListPool and the shopping list's OnRelease callback on the pooled InlineGroup. The row tooltip is unchanged, now BrowserTab:ShowRowTooltip. Not yet tried in game. Location: GUI/BrowserTab.lua, Tests/browservirtual_spec.lua, Tests/browserdetail_spec.lua, Tests/mainwindow_spec.lua, Tests/tooltipwrapflag_spec.lua.
• The [Bank] request dialog is a LibAceGUIWidgets form dialog (LAGW adoption step 6, first part). addon.Bank.ShowRequestDialog built its own frame, backdrop, close button, UIDropDownMenu banker picker and fixed heights (165, or 205 with the shop line). It is now W:CreateFormDialog (MINOR 36, TOGPM contract 6cf3b4e4): an item row, the library's dropdown box for the banker, a digits-only quantity; the stock line is the hint, "/ max N" a note under the quantity, TOGBank's shop line the body, and the dialog sizes itself to what shows. It opens where it did (below a given anchor, else beside the main window, else centred) through the dialog's own SetAnchor. The allowance, view-only, shop-order and Send logic is unchanged, and so are the field names the rest of the file reads. Not yet tried in game. Location: Compat.lua, Tests/compat_spec.lua.
• Every tab's toolbar uses LibAceGUIWidgets controls, and the window's "don't redraw under the player's hands" check is the library's (LAGW adoption step 6). The Professions, Cooldowns, Missing Recipes, Profit Planner and Crafting toolbars had 14 AceGUI Dropdowns, 4 AceGUI EditBoxes styled as search fields and 6 AceGUI CheckBoxes. They are now the library's dropdown box (single choice, or a tick-box menu that stays open with Select All / Clear All rows that act and close it), LAGW-SearchBox and CreateCheckbox, through three shared helpers in GUI/SharedWidgets.lua (ToolbarDropdown, ToolbarSearch, ToolbarCheckbox, plus MenuItems). The raw dropdown and check-box frames are built once per tab and parked in each draw's slot, so the redraw every guild sync causes creates none. A labelled control and its unlabelled neighbours share one height, so a row of them lines up. MainWindow:Refresh now asks W:IsMenuOpenFor (and redraws once, through W:OnMenuClosed, when the menu closes) and W:IsInputFocusedIn instead of walking AceGUI's global AceGUI30Pullout<N> frames -- which every AceGUI addon shares, and which once held this window's redraws forever on another addon's leaked pullout. Removed: addon.GUI.IsAnyDropdownPulloutOpen, OffsetInputLabel, StyleSearchBox, IsAnySearchFocused. Not yet tried in game. Location: GUI/SharedWidgets.lua, GUI/MainWindow.lua, GUI/BrowserTab.lua, GUI/CooldownsTab.lua, GUI/MissingRecipesTab.lua, GUI/AHProfitTab.lua, GUI/CraftingTab.lua.
• The hand-rolled list and pool helpers are gone (LAGW adoption step 7). With every list a library RowList and every toolbar control the library's, nothing called them any more. Removed from GUI/SharedWidgets.lua: addon.GUI.DetachPool, PersistentScroll (Acquire / Restore / Reset and its LayoutFinished repair), RowStripe / ApplyRowStripe, the whole addon.GUI.Sort table (SetIndicator, Indicator, Next, NextOrNone, ConfigureHeaderIcon, ConfigureCenteredHeaderIcon), LiftAboveSizers, MakeHeaderHoverGlow, and MakeColumnHeader's justifyH / onClick / hoverGlow options (its only caller, the Guild tab, passes none of them). The new Tests/sharedwidgets_spec.lua (16 cases) covers the paths nothing else drove, bringing GUI/SharedWidgets.lua to 100% line coverage (661/661). The Cooldowns group popup, the last DetachPool caller, is parented to UIParent for its whole life, so its release now just hides it and its click-outside overlay. addon.GUI.ListScroll keeps its store in the same db.char.frames.scrollTabs table. The specs for the removed helpers (gui_pool_spec, gui_scroll_spec, bottomrow_spec) are rewritten or dropped; gui_scroll_spec now specs ListScroll. Location: GUI/SharedWidgets.lua, GUI/CooldownsTab.lua.
• The last hand-built row lists are library RowLists (finishing the LibAceGUIWidgets adoption). Three panels still drew their rows from their own frame pools, and each is now a RowList, like every other list in the addon:
  • Professions tab, recipe details: the panel's own ScrollFrame, slider, reagent-row pool and crafter-row pool are gone. Reagents and Known By are one list under the header and shopping controls, with the two headings as the library's group-heading rows. The list scrolls itself. Reagent counts read the shopping-list quantity when drawn, so +/- restates them with a refresh. The crafter right-click whisper goes through the list's click handler.
  • Crafting tab, reagents: the fixed pool of 12 reagent rows and the hand-placed "Reagents" / "Cost" labels are replaced by a list whose own header carries both headings and their tooltips. The list sizes itself to its rows (fitContent), and the panel's auto-height reads that. The header bar moves the first reagent row down 2 px.
  • Cooldowns, the group popup: its pooled rows (name, reagent, [AH], [Bank], mail) are a list whose columns switch with SetColumns between a transmute group and a plain one. The reagent's white/grey is now a colour code in the cell rather than a font colour. The popup moves from TOOLTIP strata to the main window's FULLSCREEN_DIALOG, with a frame level 100 above the window's set on every open, so tooltips draw above it without the old frame-level bump. (A first cut set the strata alone and the popup opened behind the window, in game 2026-10-01.) The Crafting tab's "Missing Materials" label sits inside the reagent list's header bar, right-aligned and centred on it (in game it floated above the bar and ran past its end). None of it has been tried in game. New specs: the Crafting reagent list (7 cases in craftingtab_draw_spec), the popup's colours and mail icon, and the Known By whisper. Four browserdetail_spec and three cooldowndraw_spec cases that read the old frames now read the list's rows and cells instead (test changes approved by the operator 2026-09-30). Location: GUI/BrowserTab.lua, GUI/CraftingTab.lua, GUI/CooldownsTab.lua, Tests/browserdetail_spec.lua, Tests/craftingtab_draw_spec.lua, Tests/cooldowndraw_spec.lua.
• The [Bank] dialog's item keeps hold-to-compare and the chat-link click. Moving the dialog onto the library's form dialog lost both: the library's item row drew its own tooltip and click. LibAceGUIWidgets MINOR 39 (TOGPM contract 1d7a76ba) lets the row take the consumer's onEnter / onLeave / onClick, and the dialog now passes TOGPM's own ItemLink.SetItem / EndHover / Click. Against an older library the fields are ignored and the built-in hover and click stay. Not yet tried in game. Location: Compat.lua, Tests/compat_spec.lua.
• A spec corrupted AceGUI's shared widget pool for every file after it. settingsbleed_spec's "never re-anchors" case released the Professions list section while it was still listed in its TabGroup's children, so the window's later close released that pooled widget a second time. A TabGroup went back to the pool with a nil child, and every later spec file's Professions draw died in ReleaseChildren -- the seven toolbar_spec failures that appeared only in a full run. The case now takes the section out of the list, as AceGUI's own ReleaseChildren does. No production code releases a child on its own (only MainWindow releases, and only the root window). toolbar_spec now names the error when the Professions draw fails, instead of failing on a missing toolbar. Location: Tests/settingsbleed_spec.lua, Tests/toolbar_spec.lua.
• The offline suite no longer keeps every reload of every tab alive (WoWAPITesting thread 83f92459). Its heap climbed to 377-720 MB and ~85% of a 321 s median run was garbage collection. The new Tests/reloadleak_spec.lua reloads each tab and asserts the old copy is collected, printing the reference chain when it is not; five of six were held. Two holders, both in the test env: containers env.drawTab drew were never released, so AceGUI's _G-named frames kept the tabs' callbacks; and each module reload re-registered on the addon's, AceEvent's and LibGuildRoster's callback registries under its own (new) owner key. The env now releases drawn containers at each install (collecting every failure and raising them together at the end), re-shows a pooled container before drawing into it, and drops a replaced module's registrations. Full suite 1676/0 in 219 s; the heap still ended near 334 MB. A second pass extended the spec to every module and found the third holder: Scanner, CraftingEngine, MainWindow and MinimapButton hooksecurefunc the addon's OnEnable (Scanner also OnPlayerEnteringWorld) at load -- once per session in game -- and each spec file's reload wrapped the previous wrapper, keeping every older copy alive down the chain. Fixed first with a layer in the env, then moved into the harness on TOGPM's request (pin 515873b: a hook re-made from the same source line after a reset replaces the earlier one) and the env layer deleted; the leak spec now resets between its two loads, as between two spec files. 16 module cases added, all collected. Two more found by the full run: AHScanner's LibItemDB callback (the env's reload cleanup now covers LibItemDB's registry, and the leak spec loads the real LibItemDB), and a reloaded GUI/MinimapButton.lua got nil from LDB:NewDataObject for its own, already-held name, so the next OnEnable handed LibDBIcon nothing; the file now reuses the existing object (in game it loads once, so nothing a player sees changes). Full suite 1700/0. A spec use of a container after the env released it is NOT guarded: AceGUI's pool is private to the library, so a released widget cannot be marked without breaking its next owner. tooltipwrapflag_spec's floor comments now list each file's remaining tooltip calls. Harness pin 0e90e7d -> 515873b. docs/AUDIT.md and Tests/HARNESS_CONTRACT.md are deleted; findings and requests travel through writ's inbox, and git history keeps both boards. Location: Tests/env_togpm.lua, Tests/reloadleak_spec.lua, Tests/tooltipwrapflag_spec.lua, CLAUDE.md.
• The offline suite runs one full garbage collection per spec file, not per example (WoWAPITesting thread 48754b3d). Its telemetry put the suite at a 251 s median, the slowest in the fleet, with 89% of example time inside the reset's full collections. The harness reset already ran exactly one per example, after the env's own release, so its opt-in deferral alone would save nothing; the env now turns it on (wow.deferCollection(true)) and calls wow.collect() the first time each spec file installs. Known cost, the harness's: inside one spec file, a frame an earlier example discarded is still alive and still hears events until the next file. reloadleak_spec collects for itself and is unaffected. Full suite 1712/0; the whole background run, agent overhead included, took 64 s against 206-331 s for the day's earlier runs. Location: Tests/env_togpm.lua.

Older versions (v0.0.1 to v1.1.2) are in CHANGELOG_ARCHIVE.md.