Merge pull request #1377 from danielaskdd/main

Fix: disable LLM cache recording while enable_llm_cache is disabled

lightrag/utils.py

@@ -731,6 +731,9 @@ async def save_to_cache(hashing_kv, cache_data: CacheData):
         hashing_kv: The key-value storage for caching
         cache_data: The cache data to save
     """
+    if not hashing_kv.global_config.get("enable_llm_cache"):
+        return
+
     # Skip if storage is None or content is a streaming response
     if hashing_kv is None or not cache_data.content:
         return

lightrag_webui/src/App.tsx

@@ -25,6 +25,7 @@ function App() {
   const [apiKeyAlertOpen, setApiKeyAlertOpen] = useState(false)
   const [initializing, setInitializing] = useState(true) // Add initializing state
   const versionCheckRef = useRef(false); // Prevent duplicate calls in Vite dev mode
+  const healthCheckInitializedRef = useRef(false); // Prevent duplicate health checks in Vite dev mode
 
   const handleApiKeyAlertOpenChange = useCallback((open: boolean) => {
     setApiKeyAlertOpen(open)
@@ -70,6 +71,14 @@ function App() {
       }
     };
 
+    // On first mount or when enableHealthCheck becomes true and apiKeyAlertOpen is false,
+    // perform an immediate health check
+    if (!healthCheckInitializedRef.current) {
+      healthCheckInitializedRef.current = true;
+      // Immediate health check on first load
+      // performHealthCheck();
+    }
+
     // Set interval for periodic execution
     const interval = setInterval(performHealthCheck, healthCheckInterval * 1000);
     return () => clearInterval(interval);