From 1c117e2091bf2769102a52bda7bb626d9c50bac3 Mon Sep 17 00:00:00 2001 From: Hermes Bot Date: Sat, 4 Jul 2026 13:10:09 -0400 Subject: [PATCH 1/2] comfyui-mcp: stop baking generate_song workflow into the image The song workflow was baked at /app/workflows/generate_song.json (originally the upstream v1 3.5b graph, which hardcoded a model and used image-gen sampler defaults that produced low-quality output). Remove it so nothing song-specific is baked: skills/agents now compose the graph and submit it via queue_prompt(workflow_json=...), keeping the workflow as editable data (operators own the model/params) rather than an image rebuild. Matches how generate_image was already handled. Co-Authored-By: Claude Opus 4.8 (1M context) --- comfyui-mcp/Dockerfile | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/comfyui-mcp/Dockerfile b/comfyui-mcp/Dockerfile index e035259..e290cea 100644 --- a/comfyui-mcp/Dockerfile +++ b/comfyui-mcp/Dockerfile @@ -32,9 +32,12 @@ if "_bi.print" not in t: s.write_text(t) PY -# Ordo AI Stack: remove upstream bundled workflows — agents compose their own via queue_prompt. -# The upstream generate_image.json hardcodes v1-5-pruned-emaonly.ckpt which doesn't exist. -RUN rm -f workflows/generate_image.json workflows/basic_api_test.json +# Ordo AI Stack: remove ALL upstream bundled workflows — nothing is baked here. Skills/agents +# compose their own graph and submit it via queue_prompt(workflow_json=...). In particular the +# song workflow lives in Hermes' private data +# (data/hermes/skills/creative/song-generation/generate_song.workflow.json) so it can be edited +# without an image rebuild. (generate_image.json also hardcoded a nonexistent checkpoint.) +RUN rm -f workflows/generate_image.json workflows/basic_api_test.json workflows/generate_song.json # Ordo AI Stack: accept flat run_workflow args + optional default workflow_id (see tools/workflow.py) COPY tools/workflow.py tools/workflow.py From 2fb5ac692896bb9c9808838a7691c8f93b973aa3 Mon Sep 17 00:00:00 2001 From: Hermes Bot Date: Sat, 4 Jul 2026 13:46:25 -0400 Subject: [PATCH 2/2] compose: document ComfyUI >= v0.19.0 requirement for ACE-Step 1.5 XL The ComfyUI code is the bind-mounted git checkout (data/comfyui-storage/ComfyUI), not the image; note the version floor + rollback command so a fresh clone knows XL song generation needs >= 0.19.0. Co-Authored-By: Claude Opus 4.8 (1M context) --- docker-compose.yml | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/docker-compose.yml b/docker-compose.yml index fcb7d31..7367f50 100644 --- a/docker-compose.yml +++ b/docker-compose.yml @@ -496,6 +496,10 @@ services: comfyui: image: ${COMFYUI_IMAGE:-yanwk/comfyui-boot:cpu} + # NOTE: this image supplies only the Python/CUDA runtime. The ComfyUI application code is + # the git checkout bind-mounted from data/comfyui-storage/ComfyUI (see volumes below). + # It must be >= v0.19.0 for ACE-Step 1.5 XL support (local song generation via generate_song); + # roll back with `git -C data/comfyui-storage/ComfyUI checkout ` if a version breaks a node. # No fixed container_name — avoids "name already in use" when another project # owns `comfyui`; Docker DNS still resolves the service name `comfyui` on this network. restart: unless-stopped