Return correct DeleteQueryResponse shape for empty attributes

- Changed delete_by_attributes() to return {"deleted_entries_count": 0} instead of {}
- Changed adelete_by_attributes() to return {"deleted_entries_count": 0} instead of {}
- Updated tests to verify correct response shape
- This matches the DeleteQueryResponse structure from LangCache SDK

Co-authored-by: abrookins <[email protected]>

Author: Copilot

redisvl/extensions/cache/llm/langcache.py

@@ -588,8 +588,8 @@ def delete_by_attributes(self, attributes: Dict[str, Any]) -> Dict[str, Any]:
             Dict[str, Any]: Result of the deletion operation.
         """
         if not attributes:
-            # No attributes provided, return empty result without calling API
-            return {}
+            # No attributes provided, return result with zero deletions
+            return {"deleted_entries_count": 0}
         result = self._client.delete_query(attributes=attributes)
         # Convert DeleteQueryResponse to dict
         return result.model_dump() if hasattr(result, "model_dump") else {}
@@ -605,8 +605,8 @@ async def adelete_by_attributes(self, attributes: Dict[str, Any]) -> Dict[str, A
             Dict[str, Any]: Result of the deletion operation.
         """
         if not attributes:
-            # No attributes provided, return empty result without calling API
-            return {}
+            # No attributes provided, return result with zero deletions
+            return {"deleted_entries_count": 0}
         result = await self._client.delete_query_async(attributes=attributes)
         # Convert DeleteQueryResponse to dict
         return result.model_dump() if hasattr(result, "model_dump") else {}

tests/unit/test_langcache_semantic_cache.py

@@ -458,7 +458,7 @@ def test_delete_by_attributes_with_valid_attributes(self, mock_langcache_client)
     def test_delete_by_attributes_with_empty_attributes_returns_empty(
         self, mock_langcache_client
     ):
-        """Test that delete_by_attributes returns empty dict with empty attributes."""
+        """Test that delete_by_attributes returns correct shape with empty attributes."""
         _, mock_client = mock_langcache_client
 
         cache = LangCacheSemanticCache(
@@ -470,8 +470,8 @@ def test_delete_by_attributes_with_empty_attributes_returns_empty(
 
         result = cache.delete_by_attributes({})
 
-        # Should return empty dict without calling delete_query
-        assert result == {}
+        # Should return dict with correct shape without calling delete_query
+        assert result == {"deleted_entries_count": 0}
         mock_client.delete_query.assert_not_called()
 
     @pytest.mark.asyncio
@@ -503,7 +503,7 @@ async def test_adelete_by_attributes_with_valid_attributes(
     async def test_adelete_by_attributes_with_empty_attributes_returns_empty(
         self, mock_langcache_client
     ):
-        """Test that async delete_by_attributes returns empty dict with empty attributes."""
+        """Test that async delete_by_attributes returns correct shape with empty attributes."""
         _, mock_client = mock_langcache_client
 
         cache = LangCacheSemanticCache(
@@ -515,8 +515,8 @@ async def test_adelete_by_attributes_with_empty_attributes_returns_empty(
 
         result = await cache.adelete_by_attributes({})
 
-        # Should return empty dict without calling delete_query_async
-        assert result == {}
+        # Should return dict with correct shape without calling delete_query_async
+        assert result == {"deleted_entries_count": 0}
         mock_client.delete_query_async.assert_not_called()
 
     def test_update_not_supported(self, mock_langcache_client):