fix: remove auto-save from put() to prevent test hanging

The test_cache_memory_limit test was adding 10,000 entries, each triggering a file write. This caused the CI to hang for 2+ hours.

Now save_cache() must be called explicitly after batch operations.

Author: sshekhar563

demo_cache_persistence.py

@@ -39,6 +39,9 @@ def demo_cache_persistence():
         cache1.put(query, result)
         time.sleep(0.1)  # Simulate processing time
 
+    # Save cache after adding all entries
+    cache1.save_cache()
+    
     stats1 = cache1.get_cache_stats()
     print(f"\n📊 Cache populated: {stats1['total_entries']} entries")
     print(f"   Cache file: {stats1['cache_file']}")

nexum_ai/optimizer.py

@@ -94,9 +94,6 @@ def put(self, query: str, result: str) -> None:
             'result': result
         })
         print(f"Cached query: {query[:50]}...")
-        
-        # Auto-save cache after adding new entry
-        self.save_cache()
 
     def clear(self) -> None:
         """Clear the cache"""
@@ -346,6 +343,9 @@ def test_cache_persistence() -> Dict[str, Any]:
     for query, result in test_queries:
         cache1.put(query, result)
 
+    # Save cache after adding entries
+    cache1.save_cache()
+    
     stats1 = cache1.get_cache_stats()
     print(f"Cache stats after adding entries: {stats1}")
 

test_cache_integration.py

@@ -39,6 +39,9 @@ def test_cache_persistence_lifecycle():
         for query, result in test_data:
             cache1.put(query, result)
 
+        # Save cache after adding entries
+        cache1.save_cache()
+        
         stats1 = cache1.get_cache_stats()
         print(f"   ✅ Added {stats1['total_entries']} entries")
 
@@ -113,6 +116,7 @@ def test_environment_variable_config():
             # Create cache (should use env variable)
             cache = SemanticCache()
             cache.put("SELECT 1", "test result")
+            cache.save_cache()
 
             # Verify it used the custom path
             assert os.path.exists(custom_cache), "Should use environment variable path"