Click Casting: skip secret values when scanning Blizzard frames for status bars

FindHealthManaBars recursively walks a Blizzard frame's whole table
graph looking for HealthBar/ManaBar keys, using each visited table as a
key in a 'checked' dedupe set. Under 12.x secret values, a protected
frame/unit reference stored on the frame passes the type()=='table'
check but throws 'attempted to index a table that cannot be indexed
with secret keys' the moment it's used as that key.

The throw propagated up through registerBlizzardFrame and aborted the
RegisterBlizzardFrames loop on OnCombatEnd, so frames after the failing
one never got RegisterFrame() - click-casting silently stopped working
on Blizzard frames until reload (matches user reports of binds not
working / 'frame doesn't accept my bind').

Guard traverse() with issecretvalue() before the table is used as a
key. Health/mana bars are plain child frames, so skipping secret
subtrees loses nothing. PropagateMouseOnChildren (the sibling walker)
uses GetChildren() + IsForbidden and was already safe; the registration
validator already bails on secret protected/name values.

Author: Krathe82

CHANGELOG.md

@@ -8,6 +8,10 @@
 * (Interface) Opening a colour picker no longer counts as a colour change — the picker fired its change handlers once during setup, which could commit settings (such as a Text Designer element's colour override) without any edit. (by Krathe)
 * (Aura Designer) Fixed expiring border animations staying stuck on after the tracked aura was refreshed (for example re-casting a HoT in its pandemic window). (by Krathe)
 
+### Bug Fixes
+
+* (Click Casting) Fixed a Lua error walking Blizzard unit frames after combat that could stop click-casting bindings from working on the default frames until reload — the frame scan now skips protected (secret) values introduced by recent client versions. (by Krathe)
+
 ## [4.7.0]
 
 ### Bug Fixes

ClickCasting/Frames.lua

@@ -924,8 +924,16 @@ function CC:FindHealthManaBars(obj)
 
     local function traverse(current)
         if type(current) ~= "table" then return end
+        -- 12.x secret values: a protected frame/unit reference stored on a
+        -- Blizzard frame passes the type()=="table" check but throws the moment
+        -- it's used as a table key ("cannot be indexed with secret keys"). That
+        -- aborted RegisterBlizzardFrames on combat end, so Blizzard frames never
+        -- finished click-cast setup and bindings stopped working on them. We
+        -- never need to descend into secret subtrees (health/mana bars are plain
+        -- child frames), so skip them before `current` is used as a key below.
+        if issecretvalue(current) then return end
         if checked[current] then return end
-        
+
         checked[current] = true
         if not pcall(next, current) then return end
         for key, value in pairs(current) do