Commit 3daa486 inserted frame_rate into the required widget list at
position 5 (directly after duration). ComfyUI's positional graph loader
maps saved widgets_values arrays by index, so every save from before that
commit mis-mapped: the old ref_image_size ('match') landed on frame_rate
(crashing the queue with float('match')), the timeline JSON landed on
ref_image_size, the builder_state JSON landed on timeline_data, and the
builder_state slot stayed empty -> the Director node showed an empty
prompt on load.
Reorder the schema so the eight legacy widgets keep their original
positional order and frame_rate is appended last. Old saves now map 1:1
onto the right slots; the stale 9th value (the removed external_prompt
widget, '') lands on frame_rate instead of shifting the real fields.
Make build_guide coerce a non-numeric frame_rate to the 24.0 default
instead of raising, so the queue never crashes on a stale shifted value;
genuinely out-of-range numbers still raise.
Add regression tests: a 9-value legacy save loads without crashing and
preserves its REF2VA prompt, and the input-type order invariant is
enforced. Update the frame_rate position test to lock in the new order.
Darksidewalker
ed5d39308d