* feat(automations): workflow JSON validator + LTX 2.3 stage-2-only findings
Adds scripts/validate_workflow.py — a stdlib-only CLI that checks a
ComfyUI workflow JSON for internal consistency: link-table declarations
vs node-IO references, slot index bounds and types, endpoint cross-
checks, last_node_id/last_link_id bounds, and duplicate IDs.
Catches the class of bugs that come from hand-editing workflow JSONs
(e.g. when an agent re-wires a graph to build a slim variant from a
Director workflow). Comfy's loader silently drops or mis-wires bad
parts, so the cost of an undetected drift is "graph looks fine but
behaves wrong" — exactly the failure mode this script blocks.
Exit codes: 0 clean, 1 problems on stderr, 2 file missing or bad JSON.
Docs:
- docs/automations/CONVENTIONS.md §11 — when and how to run the
validator, full list of checks performed.
- docs/automations/LTX-2.3/findings.md — two new sections covering the
spatial upscaler's architectural facts (ltx-2.3-spatial-upscaler-x2
is hardcoded x2, the brief enters only via Director > guide_data >
DirectorGuide, attention scales (W * H * T)^2) plus three open
hypotheses for a stage-2-only refine path.
- CHANGELOG.md [Unreleased] entry.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
* fix(scripts): make validate_workflow.py robust against malformed JSON
Addresses PR #170 review (Codex GPT-5.3) — the validator crashed with
AttributeError on malformed-but-valid JSON, defeating its goal as a safe
preflight tool. Two repros from the review:
{"nodes":[1],"links":[],"last_node_id":0,"last_link_id":0}
-> AttributeError: 'int' object has no attribute 'get'
[1,2]
-> AttributeError: 'list' object has no attribute 'get'
Both confirmed locally. Now every shape assumption is guarded:
- Top-level must be a JSON object; non-object short-circuits with one
clear problem line.
- 'nodes' and 'links' must be lists; non-list reported and treated as
empty so subsequent checks still run.
- Each node entry must be a dict; bad entries reported and dropped.
- Each input/output entry must be a dict (extracted into a
_sanitize_slots helper, cached per node so duplicates aren't reported
when the same node is touched by multiple links).
- Each link entry must be a 6-element list; malformed link entries
reported and skipped without crashing later phases.
- node_by_id only keys on int node IDs (non-int IDs can't match a
link's int src/dst, so we don't mask bad data by accepting them).
Adds tests/scripts/test_validate_workflow.py — 24 tests covering both
review repros, every new guard, and a sampling of the existing
consistency checks (link-to-missing-node, type mismatch, endpoint
desync, duplicate IDs, last_node_id bound, CLI exit codes). Full suite:
168 passed in 1.59s.
CHANGELOG entry updated to reflect the defensive behavior + test
coverage.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
---------
Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
