Make SendChatMessage reclaim cycle-safe and cover LVL

The previous reclaim stack-overflowed when LeafVillageAchievements loaded
after us (its captured "original" was our own function, so reclaiming onto
the top created an ours<->LVA loop). Add a reentrancy guard plus a base
SendChatMessage reference so a hook cycle routes straight to the base
function instead of recursing.

Gate the reclaim on LeafVillageAchievements OR LeafVillageLegends (both hook
SendChatMessage and can bump ours off the top -- LVA orphans it, LVL wraps
and re-installs on timers). Keep the anchored ^#showtooltip match since being
on top means we see the pristine line.

Core.lua

@@ -3935,10 +3935,14 @@ function CleveRoids.OnUpdate(self)
 
     CR.lastUpdate = time
 
-    -- Reclaim the top of the SendChatMessage hook chain if another addon has
+    -- LeafVillageAchievements/Legends also hook SendChatMessage and can bump ours
-    -- displaced our #showtooltip filter (see EnsureSendChatMessageHook). Cheap
+    -- off the top of the chain (orphaning it / decorating #showtooltip). Reclaim
-    -- identity check; only re-hooks when actually displaced.
+    -- the top when they're present so #showtooltip stays filtered.
-    if CR.EnsureSendChatMessageHook then
+    if not CR._watchChatHooks then
+        CR._watchChatHooks = (C_AddOns.IsAddOnLoaded("LeafVillageAchievements")
+                or C_AddOns.IsAddOnLoaded("LeafVillageLegends"))
+    end
+    if CR._watchChatHooks then
         CR.EnsureSendChatMessageHook()
     end
 
@@ -5997,27 +6001,27 @@ function CleveRoids.Frame:KEY_UP()
     CleveRoids.isActionUpdateQueued = true
 end
 
--- Filter out the #showtooltip line that Blizzard's native macro executor sends
+-- Base SendChatMessage captured at first hook; used to break a hook cycle.
--- to chat when a macro runs from an action button. Named (not anonymous) so we
+local baseSendChatMessage = SendChatMessage
--- can detect displacement and reclaim the top of the hook chain.
+local sendingChatMessage = false
+
+-- Swallow #showtooltip lines Blizzard's macro executor sends to chat.
 local function CleveRoids_SendChatMessage(msg, ...)
     if msg and string.find(msg, "^#showtooltip") then
         return
     end
-    -- Call whatever we chained over (another addon's hook, or the real function)
+    if sendingChatMessage then
-    CleveRoids.Hooks.SendChatMessage(msg, unpack(arg))
+        -- Re-entered via a hook cycle; go straight to base to avoid recursion.
+        return baseSendChatMessage(msg, unpack(arg))
+    end
+    sendingChatMessage = true
+    local ok, err = pcall(CleveRoids.Hooks.SendChatMessage, msg, unpack(arg))
+    sendingChatMessage = false
+    if not ok and geterrorhandler then geterrorhandler()(err) end
 end
 
--- (Re-)assert our filter as the OUTERMOST SendChatMessage hook. This is a no-op
+-- Reassert our filter as the outermost SendChatMessage hook (no-op if already on
--- once we're already on top. It matters because some addons snapshot
+-- top). See the OnUpdate caller for why this is needed.
--- SendChatMessage at their file-load and later install a hook that calls that
--- snapshot DIRECTLY -- if they loaded before us, their snapshot predates our
--- filter, so calling it directly orphans us and #showtooltip leaks to chat.
--- (LeafVillageAchievements does exactly this at PLAYER_ENTERING_WORLD+3s; and
--- because the fork sorts after it alphabetically, we load too late to be in its
--- snapshot -- upstream "CleveRoidMacros" sorted before it and wasn't affected.)
--- OnUpdate calls this so we reclaim the top within a frame of being displaced;
--- being outermost also guarantees we see the pristine line for the anchor match.
 function CleveRoids.EnsureSendChatMessageHook()
     if SendChatMessage ~= CleveRoids_SendChatMessage then
         CleveRoids.Hooks.SendChatMessage = SendChatMessage