-
Notifications
You must be signed in to change notification settings - Fork 638
Cloud/1.36 - cloud release branch [DO NOT MERGE] #7967
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Closed
Closed
Changes from 34 commits
Commits
Show all changes
39 commits
Select commit
Hold shift + click to select a range
9f3bbb9
[backport cloud/1.36] fix: restore mask editor compatibility with Imp…
comfy-pr-bot 4957cd3
[backport cloud/1.36] Guard downgrades via billing portal (#7820)
comfy-pr-bot 8eb2ba0
[backport cloud/1.36] [feat] Filter out nlf model type from Upload Mo…
comfy-pr-bot e9c47e8
[backport cloud/1.36] [feat] Add async model upload with WebSocket pr…
comfy-pr-bot 1de23b8
[backport cloud/1.36] feat: split asset_update_options_enabled into s…
comfy-pr-bot f1b874e
[backport cloud/1.36] feat: Stale-while-revalidate pattern for AssetB…
comfy-pr-bot 2a15325
[backport cloud/1.36] Prevent nav item shrink (#7890)
comfy-pr-bot b4e4ccc
[backport cloud/1.36] Fix run badge anchoring (#7915)
comfy-pr-bot aecb841
[backport cloud/1.36] feature: model browser folder grouping (#7916)
comfy-pr-bot e912b42
[backport cloud/1.36] feat: add model download progress dialog (#7917)
comfy-pr-bot b9e6f3d
[backport cloud/1.36] feat: add HoneyToast component for persistent p…
luke-mino-altherr be8ee3d
[backport cloud/1.36] fix: Button sizing in modals and asset browser …
comfy-pr-bot ec7a3a9
[backport cloud/1.36] perf(AssetBrowserModal): virtualize asset grid …
comfy-pr-bot b53976c
[backport cloud/1.36] Feat(cloud)/new top up dialog (#7932)
simula-r 4165f52
[backport cloud/1.36] fix: UX nits and styles (#7936)
comfy-pr-bot 348d674
[backport cloud/1.36] fix: Model upload UI improvements (#7943)
comfy-pr-bot e84b6f3
[backport cloud/1.36] fix(UploadModel): truncate long filenames in wi…
comfy-pr-bot 1abe3f2
[backport cloud/1.36] fix(upload-model): UI/UX improvements for Uploa…
comfy-pr-bot 53d76c4
[backport cloud/1.36] fix: PrimitiveNode combo widget value not persi…
comfy-pr-bot 7158e81
[backport cloud/1.36] disable workflow validation warnings by default…
comfy-pr-bot 89a00fe
[backport cloud/1.36] CI: Use custom container for E2E tests (#7983)
comfy-pr-bot 2d04cf4
[backport cloud/1.36] fix 3d-min-resize (#7985)
comfy-pr-bot 7f83af3
[backport cloud/1.36] feat: add polling fallback for stale asset down…
DrJKL 4963f59
[backport cloud/1.36] refactor: simplify asset download state and fix…
comfy-pr-bot 19db196
[backport cloud/1.36] fix: Improve legacy widget compatibility in vue…
comfy-pr-bot fe48900
[backport cloud/1.36] fix: disable frustum culling for SkinnedMesh to…
comfy-pr-bot 056308a
[backport cloud/1.36] fix: continue rendering when 3D animation is pl…
comfy-pr-bot 9bf5176
[backport cloud/1.36] fix: prevent image preview resize issues when s…
comfy-pr-bot 4a6a128
[backport cloud/1.36] fix(price-badges): improve Gemini and OpenAI ch…
comfy-pr-bot bc4b3d0
[backport cloud/1.36] fix(price-badges): add missing badge for WanRef…
comfy-pr-bot 74ebfec
[backport cloud/1.36] Fix linked asset widget promotion in vue (#8003)
comfy-pr-bot dc0d859
[backport cloud/1.36] fix: respect node resizable property in vueNode…
comfy-pr-bot c28e60e
[backport cloud/1.36] fix: remove negative margin from legacy widget …
comfy-pr-bot eeb0fd3
[backport cloud/1.36] fix: enable immediate file saving for i18n tran…
comfy-pr-bot fca4931
[backport cloud/1.36] feat(price-badges): add price badges for Vidu2 …
comfy-pr-bot 8e62359
[backport cloud/1.36] Fix reactivity washing in refreshNodeSlots (#8019)
comfy-pr-bot 15955bd
[backport cloud/1.36] fix: wrap image preview navigation dots when ov…
comfy-pr-bot 2da8338
[backport cloud/1.36] Dynamic input fixes (#8008)
comfy-pr-bot 99f5b36
[backport cloud/1.36] feat: local/legacy settings dialog fix (#8004)
comfy-pr-bot File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,23 @@ | ||
| name: Start ComfyUI Server | ||
| description: 'Start ComfyUI server in a container environment (assumes ComfyUI is pre-installed)' | ||
|
|
||
| inputs: | ||
| front_end_root: | ||
| description: 'Path to frontend dist directory' | ||
| required: false | ||
| default: '$GITHUB_WORKSPACE/dist' | ||
| timeout: | ||
| description: 'Timeout in seconds for server startup' | ||
| required: false | ||
| default: '600' | ||
|
|
||
| runs: | ||
| using: 'composite' | ||
| steps: | ||
| - name: Copy devtools and start server | ||
| shell: bash | ||
| 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 }}" & | ||
| wait-for-it --service 127.0.0.1:8188 -t ${{ inputs.timeout }} | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🧩 Analysis chain
🏁 Script executed:
Repository: Comfy-Org/ComfyUI_frontend
Length of output: 770
🏁 Script executed:
# Check the action.yml file to understand input definitions cat -n .github/actions/start-comfyui-server/action.ymlRepository: Comfy-Org/ComfyUI_frontend
Length of output: 986
🏁 Script executed:
# Search for Dockerfile and container-related files fd -e dockerfile -e Dockerfile -e docker-compose.yml -e docker-compose.yamlRepository: Comfy-Org/ComfyUI_frontend
Length of output: 52
🏁 Script executed:
Repository: Comfy-Org/ComfyUI_frontend
Length of output: 104
🏁 Script executed:
Repository: Comfy-Org/ComfyUI_frontend
Length of output: 104
🏁 Script executed:
Repository: Comfy-Org/ComfyUI_frontend
Length of output: 86
🏁 Script executed:
Repository: Comfy-Org/ComfyUI_frontend
Length of output: 241
🏁 Script executed:
Repository: Comfy-Org/ComfyUI_frontend
Length of output: 258
🏁 Script executed:
Repository: Comfy-Org/ComfyUI_frontend
Length of output: 2463
🏁 Script executed:
Repository: Comfy-Org/ComfyUI_frontend
Length of output: 99
Create the devtools directory before copying files.
The
cpcommand on line 21 assumes/ComfyUI/custom_nodes/ComfyUI_devtools/already exists. The relatedsetup-comfyui-serveraction properly handles this withmkdir -pfirst. Without creating the directory, the copy will fail. Addmkdir -pbefore the copy or add error handling.Quote the timeout input for shell safety.
The
timeoutinput on line 23 should be quoted to prevent word-splitting issues if unexpected characters are introduced:Suggested fix
📝 Committable suggestion
🤖 Prompt for AI Agents