When the responses endpoint fails and the chat.completions fallback also
fails (raises or returns empty), the original responses error was captured
in fallback_reason but never surfaced. Wrap the fallback call in its own
try/except and re-raise a RuntimeError that chains both errors with
context, so users see why both endpoints failed.
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
When the responses endpoint fails and execution falls back to
chat.completions, only the first image in a batch can be forwarded
(chat.completions accepts a single image_url per message in this
implementation). Previously extra frames were silently dropped.
Now we record batch_size before entering the fallback and append a plain-
language warning to fallback_reason when batch_size > 1, so the status
preview shown in ComfyUI makes the degradation explicit.
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
connect_node.py
- Fix UI label typo: "Enable Reasonning" → "Enable Reasoning"
- Use normalize_api_key() instead of an inline (api_token or "-").strip()
expression; the helper also checks DEFAULT_API_KEY_PLACEHOLDER so both
code paths now agree on the canonical fallback value
client.py
- Remove build_responses_input_with_image(): dead code; image_to_text_node
builds its own multi-image payload directly in _responses_kwargs() and
this function was never imported or called
routes.py
- _test_handler default timeout: 600 s → 15 s, consistent with
_models_handler. Both handlers probe connectivity identically; a 10-min
default on a "Test Connection" button is a foot-gun when called without
a timeout_seconds query param.
- Drop now-unused DEFAULT_TIMEOUT_SECONDS import
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- coerce_seed("") returns None, not ValueError; move "" out of the
invalid-values parametrize and give it its own explicit assertion
- strip_think_content tests used \\n (literal backslash-n) instead of
real newlines; .strip() cannot remove backslash or 'n' so the dangling-
tag test always failed. Replace with actual \n to match real LLM output.
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
