From 0141af07864b5f345b679a0f570bf8c19aa6ec09 Mon Sep 17 00:00:00 2001 From: Deep Mehta Date: Fri, 30 Jan 2026 00:46:10 +0530 Subject: [PATCH] refactor: rename _is_cacheable_value to _is_external_cacheable_value MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Clearer name since objects are also cached locally - this specifically checks for external caching eligibility. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude --- comfy_execution/caching.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/comfy_execution/caching.py b/comfy_execution/caching.py index 05b7871f2..c666c4dc1 100644 --- a/comfy_execution/caching.py +++ b/comfy_execution/caching.py @@ -238,7 +238,7 @@ class BasicCache: return if not has_cache_providers(): return - if not self._is_cacheable_value(value): + if not self._is_external_cacheable_value(value): return if contains_nan(cache_key): return @@ -302,8 +302,8 @@ class BasicCache: return None - def _is_cacheable_value(self, value): - """Check if value is a CacheEntry (not objects cache).""" + def _is_external_cacheable_value(self, value): + """Check if value is a CacheEntry suitable for external caching (not objects cache).""" return hasattr(value, 'outputs') and hasattr(value, 'ui') def _get_class_type(self, node_id):