From 9c53442a85a5aaffa10c4422790ea1de2bfb0275 Mon Sep 17 00:00:00 2001 From: bymyself Date: Wed, 4 Feb 2026 23:14:29 -0800 Subject: [PATCH] fix: address code review feedback - Fix StyleModelLoader and GLIGENLoader missing placeholders - Fix function values called without node context - Fix ellipsis formatting and Vue props destructuring Amp-Thread-ID: https://ampcode.com/threads/T-019c2c7e-2ac1-7114-9147-b41e6334faa9 --- folder_paths.py | 2 +- nodes.py | 8 ++++++-- 2 files changed, 7 insertions(+), 3 deletions(-) diff --git a/folder_paths.py b/folder_paths.py index b9022fbba..ff9230021 100644 --- a/folder_paths.py +++ b/folder_paths.py @@ -481,7 +481,7 @@ def get_model_placeholder(folder_name: str) -> str: Returns: A user-friendly placeholder string indicating where models should be placed. """ - return f"No models found in ComfyUI/models/{folder_name} folder . . ." + return f"No models found in ComfyUI/models/{folder_name} folder..." def get_input_subfolders() -> list[str]: diff --git a/nodes.py b/nodes.py index 3f08adc47..c661d164d 100644 --- a/nodes.py +++ b/nodes.py @@ -1091,7 +1091,9 @@ class CLIPVisionEncode: class StyleModelLoader: @classmethod def INPUT_TYPES(s): - return {"required": { "style_model_name": (folder_paths.get_filename_list("style_models"), )}} + return {"required": { "style_model_name": (folder_paths.get_filename_list("style_models"), { + "placeholder": folder_paths.get_model_placeholder("style_models") + })}} RETURN_TYPES = ("STYLE_MODEL",) FUNCTION = "load_style_model" @@ -1190,7 +1192,9 @@ class unCLIPConditioning: class GLIGENLoader: @classmethod def INPUT_TYPES(s): - return {"required": { "gligen_name": (folder_paths.get_filename_list("gligen"), )}} + return {"required": { "gligen_name": (folder_paths.get_filename_list("gligen"), { + "placeholder": folder_paths.get_model_placeholder("gligen") + })}} RETURN_TYPES = ("GLIGEN",) FUNCTION = "load_gligen"