From e48dcd11f12b41580be737ae7f42bde042f2f730 Mon Sep 17 00:00:00 2001 From: dante01yoon Date: Tue, 19 May 2026 22:14:38 +0900 Subject: [PATCH] ci(temp): enable --enable-assets on Playwright ComfyUI server Temporary unblock for FE-729 e2e while BE-786 is in flight. FE-729 removes the legacy /api/models fallback in modelStore, so the assets endpoint must be reachable in CI. BE-786 will make assets always-on in OSS core; once that ships in the CI ComfyUI image, this flag (and this commit) MUST be reverted before merging FE-729 to main. --- .github/actions/start-comfyui-server/action.yaml | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/.github/actions/start-comfyui-server/action.yaml b/.github/actions/start-comfyui-server/action.yaml index 2af727508d..c7accf7125 100644 --- a/.github/actions/start-comfyui-server/action.yaml +++ b/.github/actions/start-comfyui-server/action.yaml @@ -19,5 +19,8 @@ runs: run: | set -euo pipefail cp -r ./tools/devtools/* /ComfyUI/custom_nodes/ComfyUI_devtools/ - cd /ComfyUI && python3 main.py --cpu --multi-user --front-end-root "${{ inputs.front_end_root }}" & + # TODO(FE-729): remove --enable-assets once BE-786 lands in the CI ComfyUI image + # (BE-786 removes the gate so /api/assets is always on). Until then, FE-729 + # routes modelStore through assetService, which 503s without this flag. + cd /ComfyUI && python3 main.py --cpu --multi-user --enable-assets --front-end-root "${{ inputs.front_end_root }}" & wait-for-it --service 127.0.0.1:8188 -t ${{ inputs.timeout }}