Post-merge cleanup from the PR #221 review

Six non-blocking items from the review that were safe to fix directly.

- /dfauras <unknown-unit> opened a second DF:Out writer, printing the
  separator rule and title twice. Reuses the one already open.
- The roster monitor told the user to type /dfroster, which is no longer a
  bind after the one-command rework. Routed through DF:CmdPath so it stays
  right if roster moves between the everyday and debug listings.
- COMMAND_SIBLINGS.secure.dev was missing showbutton and hidebutton, which
  the comment above it says must stay in step with SECURE_MUTATORS. Now 14
  against 14.
- L["Back"] had no readers left once the wizard runtime went; the pruning
  note claiming otherwise is corrected rather than just deleted.
- Dangling TEST COMMANDS section header in Popup.lua with nothing under it.
- Orphaned raidRoleCache comment in StatusIcons.lua.

Checks: zero L[...] reads without a definition (2094 keys, 2059 uses); TOC
clean both directions.

Author: mergetest

Core.lua

@@ -394,7 +394,8 @@ DF.COMMAND_SIBLINGS = {
     -- SECURE_MUTATORS in Features/SecureSort.lua, which is what refuses them.
     -- "init" leads the dev list because every other one needs the handler.
     secure    = { "help", "status", dev = { "init", "party", "raid", "all", "register",
-                  "test", "swap", "swapback", "debug", "ui", "show", "hide" } },
+                  "test", "swap", "swapback", "debug", "ui", "show", "hide",
+                  "showbutton", "hidebutton" } },
     flatraid  = { "info", "reinit", "test" },
     -- (No "cc" entry.) /df debug cc's BARE form already prints its full subcommand
     -- table — that is its entire job — so a Siblings footer would repeat it.

Features/Auras.lua

@@ -3598,7 +3598,8 @@ SlashCmdList["DFAURAS"] = function(msg)
         end)
     end
     if not target then
-        local o = DF:Out("Aura Pipeline", "unit " .. unit)
+        -- Reuse the writer opened above; a second DF:Out here printed the
+        -- separator rule and the title twice on the not-found path.
         -- IterateAllFrames covers party/raid/arena, not pets or pinned sets.
         o:Line("No DF party/raid/arena frame is currently driving that unit.", "WARN")
         o:Line("Try player, party1..4 or raid1..40, and make sure the frames are shown.", "NEUTRAL")

Frames/Headers.lua

@@ -504,7 +504,11 @@ SlashCmdList["DFROSTER"] = function(msg)
         wipe(DF.RosterDebug.events)
         DF.RosterDebug.startTime = GetTime()
         DF.RosterDebug.enabled = true
-        DF:Say("Started monitoring. Join/leave groups, then type /dfroster again to see results.")
+        -- Via CmdPath, not a hardcoded spelling: /dfroster is no longer a bind
+        -- after the one-command rework, and the path stays right if roster ever
+        -- moves between the everyday and debug listings.
+        DF:Say("Started monitoring. Join/leave groups, then run "
+            .. DF:CmdPath("roster") .. " again to see results.")
     else
         -- Stop and report
         DF.RosterDebug.enabled = false

Frames/StatusIcons.lua

@@ -900,8 +900,6 @@ end
 -- Shows Main Tank / Main Assist assignment
 -- ============================================================
 
--- Cache for raid role assignments (unit -> role string)
-
 function DF:UpdateRaidRoleIcon(frame)
     if not frame or not frame.unit or not frame.raidRoleIcon then return end
 

Locales/enUS.lua

@@ -471,7 +471,6 @@ L["Auto-switched to profile: %s"] = true
 L["Auto-switching disabled"] = true
 L["Automatically add players by role when they join your group."] = true
 L["Available Profiles"] = true
-L["Back"] = true
 L["Background"] = true
 L["Background Alpha"] = true
 L["Background Color"] = true
@@ -1657,8 +1656,10 @@ L["• Text Designer (Name, Health, Status & custom text)\n• Buff Stack & Dura
 -- the rest of the locale orphans. ⚠ The reason given for leaving them — "pruning them
 -- is the same edit across all 11 locale files" — was WRONG: the 10 translated files
 -- are 9-line packager stubs holding zero keys, so a key lives in enUS.lua only.
--- "Back", "Cancel", "Next", "None", "Party" and "Raid" survived the cut because they
+-- "Cancel", "Next", "None", "Party" and "Raid" survived the cut because they
 -- have real readers (Popup.lua, ClickCasting/UI/BindingEditor.lua, DesignerPresets.lua).
+-- "Back" was on that list in error — its only reader was the wizard runtime's step
+-- navigation, which went with Popup.lua's trim, so the key is now gone too.
 L["Name"] = true
 L["Notice"] = true
 L["Test Mode"] = true

Popup.lua

@@ -620,7 +620,3 @@ end
 
 -- (Removed) DF:IsPopupShown — no callers. Consumers that care whether a dialog is
 -- up check their own state; nothing ever asked the popup system.
-
--- ============================================================
--- TEST COMMANDS (temporary, for validation)
--- ============================================================