From 5482878edcec4bdbfb762b0cfa19307efabb09f3 Mon Sep 17 00:00:00 2001 From: Jason Hoku Date: Fri, 8 May 2026 12:52:43 -0700 Subject: [PATCH] fix: drop seed/denoise/clip_type from schema-drift validator These keys legitimately aren't set in pre-orchestrator configs: - seed: filled by expand_configs (node-level seed widget or per-entry override) - denoise: filled by run_generation_loop from the denoise widget - clip_type: only set when model_type != "checkpoint" The validator was a stale leftover from the pre-single-source-of-truth era. Trimming false positives. Still warns on actual typos / dropped fields. Co-Authored-By: Claude Opus 4.7 (1M context) --- config_builder_node.py | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/config_builder_node.py b/config_builder_node.py index 1050944..31acb61 100644 --- a/config_builder_node.py +++ b/config_builder_node.py @@ -768,10 +768,14 @@ class UltimateConfigBuilder: # If neither, omit "positive"/"negative" keys - node inputs will be used as fallback # --- CONFIG SCHEMA VALIDATION --- - # Catches JS/Python sync drift — warns if expected keys are missing + # Sanity check for typos / dropped fields. Only includes keys this + # function ALWAYS sets unconditionally — fields that are filled in + # later by the orchestrator (seed, denoise) or only set for + # specific model types (clip_type, text_encoders, gguf_options, + # ltx_*) are intentionally excluded so they don't fire false alarms. _EXPECTED_CONFIG_KEYS = { - "sampler", "scheduler", "steps", "cfg", "denoise", "seed", "seed_behavior", - "model", "model_type", "lora", "vae", "clip_type", + "sampler", "scheduler", "steps", "cfg", "seed_behavior", + "model", "model_type", "lora", "vae", "model_sampling_override", "use_advanced_sampling", "use_flux_guidance", "model_prompt_prefix", "model_prompt_suffix", "attention_mode", }