Comfy-Org / Comfy-Org/comfy-cli

jobs wait omits structured node failure details from failed rows

Open
#834 0 comments 0 reactions 1 assignee Claimed by @skishore23 View on GitHub
Dominant language
Python
Stars
968
Forks
151
Avg merge
1d 9h
Merged PRs (30d)
77

Description

`jobs wait` now reports an error message, but its per-job row still drops the structured node failure fields already available from job status. Carry the structured execution error into each failed row so an agent can diagnose the node and exception without a second status call.

- Preserve `prompt_id`, `status`, and `ok`.
- Add the structured node/type/exception payload when present.
- Keep the existing aggregate summary and exit behavior.

Paper-code audit evidence

Linear BE-10026 is partially stale against `main@99c5a596`: `run --wait` now preserves `prompt_id`, and the documented NDJSON event stream has coverage. The residual is in `comfy_cli/command/jobs.py:1564-1576`, where each row copies only outputs and optional `error_message`; `:1582-1605` then emits that reduced row set as the failure details.

The affected row-construction surface blames to `skishore23`. Independently re-verified from scratch for OPP-48 before filing.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.