No way to read a workflow without creating a build

Abierto
  • ¿Algo no está bien?
#787 1 comentario 0 reacciones 0 asignados Ver en GitHub

Los mantenedores suelen responder en 1 día

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
3/5
Tiempo estimado
1-2 días
Aptitud para principiantes
72/100
Tipo de issue
Nueva funcionalidad
Claridad
Bastante claro
Estado de actividad
Activo
Stack tecnológico
python
Área
api, cli

Línea de trabajo

Comienza con el comando existente comfy build from-workflow y el endpoint POST /v1/workflows/resolve descrito en el issue. Traza cómo la CLI lee un workflow e imprime la respuesta del build, y después expón una ruta de solo lectura que informe de la definición del workflow y del informe de importación. La tarea estará terminada cuando se pueda inspeccionar un workflow desconocido sin crear un build.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

enhancement

comfy build from-workflow is the only verb that reads a workflow file, and it creates a build as a side effect. There is no way to ask what a workflow contains first.

The builder already serves the read-only door. POST /v1/workflows/resolve returns the same definition and report and creates nothing; its own description says it is "for a caller that wants to show a person what the workflow means first". The CLI exposes only the create door.

Why it matters: an agent handed an unfamiliar workflow is told to describe what will be created before creating it, and cannot. It has to write a build to find out whether the build is worth writing. On a graph carrying a class nobody publishes, that record is pure litter: the definition comes back {} and the right answer is not to build at all.

Observed, staging, a workflow with one unpublishable class:

$ comfy build from-workflow --from sdxl-with-unknown.json --name probe
{"ok": true, "data": {"build": {"id": "b0fed96c-...", "definition": {},
 "workflowImportReport": {"unresolvedClasses": ["TotallyMadeUpNodeXYZ42"], ...}}}}

A build now exists that should never have been created, and the only way to have known that was to create it.

Suggested: a comfy build resolve-workflow --from <file> (or --dry-run on from-workflow) that posts to /v1/workflows/resolve and prints the report without writing anything.

Found while testing the comfy-build skill's workflow path end to end.

Lenguaje dominante
Python
Estrellas
1k
Forks
176
Merge medio
1 d 11 h
PR fusionados (30 d)
87

Preparar el entorno

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de Comfy-Org/comfy-cli

Todos los issues de Comfy-Org/comfy-cli

Issues similares

Más issues de Python

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.