From b89aa364833c1535a41c36d6ab95cfad08081fd8 Mon Sep 17 00:00:00 2001
From: Kelsi <kelsihates2fa@gmail.com>
Date: Fri, 20 Mar 2026 09:14:53 -0700
Subject: [PATCH] fix: clear spell visual negative cache on world entry

The spell visual failed-model cache was never cleared across world
changes, so models that failed to load during initial asset loading
(before MPQ/CASC data was fully indexed) would never retry. Now clears
spellVisualFailedModels_ in resetCombatVisualState() alongside the
active spell visual cleanup, giving failed models a fresh attempt on
each world entry.
---
 src/rendering/renderer.cpp | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/src/rendering/renderer.cpp b/src/rendering/renderer.cpp
index d9520348..11c37bab 100644
--- a/src/rendering/renderer.cpp
+++ b/src/rendering/renderer.cpp
@@ -3023,6 +3023,8 @@ void Renderer::resetCombatVisualState() {
         if (m2Renderer) m2Renderer->removeInstance(sv.instanceId);
     }
     activeSpellVisuals_.clear();
+    // Reset the negative cache so models that failed during asset loading can retry.
+    spellVisualFailedModels_.clear();
 }
 
 bool Renderer::isMoving() const {