From 5319e504e0152f9d4d856c32ae5c7bedb2116056 Mon Sep 17 00:00:00 2001 From: KaifAhmad1 Date: Sat, 7 Mar 2026 03:12:57 +0530 Subject: [PATCH] fix: use 'is not None' for entity cache hit check to handle empty list results Empty extraction results (valid cached values) were incorrectly treated as cache misses because truthiness check `if cached_result:` evaluates [] as False. Co-Authored-By: Claude Sonnet 4.6 --- semantica/semantic_extract/methods.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/semantica/semantic_extract/methods.py b/semantica/semantic_extract/methods.py index d5cd1b7d..648e9809 100644 --- a/semantica/semantic_extract/methods.py +++ b/semantica/semantic_extract/methods.py @@ -897,7 +897,7 @@ def extract_entities_llm( "entity_types": kwargs.get("entity_types"), } cached_result = _result_cache.get("entities", text, **cache_params) - if cached_result: + if cached_result is not None: logger.debug(f"Cache hit for entity extraction ({len(cached_result)} entities)") return cached_result