docs(comfyui): add system detection + hardware-aware recommendations to onboarding - #26400
Closed
Master-Rensei wants to merge 5 commits into
Closed
docs(comfyui): add system detection + hardware-aware recommendations to onboarding#26400Master-Rensei wants to merge 5 commits into
Master-Rensei wants to merge 5 commits into
Conversation
…y dependency Complete rewrite of the ComfyUI skill to use: - comfy-cli (official, Comfy-Org/comfy-cli) for lifecycle management: install, launch, stop, node management, model downloads - Direct REST API + helper scripts for workflow execution: parameter injection, submission, monitoring, output download - No dependency on comfyui-skill-cli or any unofficial tool New files: - SKILL.md: full rewrite with two-layer architecture, decision tree, pitfalls - references/official-cli.md: complete comfy-cli command reference - references/rest-api.md: all REST endpoints (local + cloud) - references/workflow-format.md: API format spec, common nodes, param mapping - scripts/extract_schema.py: analyze workflow → extract controllable params - scripts/run_workflow.py: inject args, submit, poll, download outputs - scripts/check_deps.py: check missing nodes/models against running server - scripts/comfyui_setup.sh: full setup automation with official CLI Removed: - references/cli-reference.md (was for unofficial comfyui-skill-cli) - references/api-notes.md (replaced by rest-api.md) Addresses feedback from PR #17316 comment: - Correct author attribution - Remove references to unofficial OpenClaw project - License field reflects hermes-agent repo (MIT)
…links, cloud setup Adds structured onboarding flow to SKILL.md: - Decision table: which install path for which situation - Path A: Comfy Cloud (zero setup, API key, pricing) - Path B: Desktop app (Windows/macOS, one-click) - Path C: Portable build (Windows, extract-and-run) - Path D: comfy-cli (recommended for agents, all platforms) - Path E: Manual install (advanced, all hardware types) - Post-install: model downloads, custom nodes, verification All paths link to official docs: - https://docs.comfy.org/installation - https://docs.comfy.org/comfy-cli/getting-started - https://docs.comfy.org/get_started/cloud - https://docs.comfy.org/installation/desktop - https://docs.comfy.org/installation/comfyui_portable_windows - https://docs.comfy.org/installation/manual_install
…ring sanitization' Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
…ring sanitization' Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
…to onboarding Instead of asking the user what they have, the agent now: 1. Runs system detection commands (OS, GPU, VRAM, RAM, disk, Python) 2. Checks if ComfyUI is already installed/running 3. Recommends the best path based on findings Adds: - Step 1: detection script block (nvidia-smi, system_profiler, etc.) - Step 2: decision table mapping detected system → recommended path - Hardware requirements table (VRAM tiers, RAM, disk) - Specific recommendations per platform: macOS → Desktop app, Linux+NVIDIA → comfy-cli, no GPU → Cloud, etc.
daimon-nous
Bot
added
type/docs
Contributor
|
@christian-byrne Tagging you on this ComfyUI item. |
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
Summary
Single commit adding automated system-detection steps to the ComfyUI skill onboarding. Instead of asking users what hardware they have, the agent now:
Changes
optional-skills/creative/comfyui/SKILL.md— 69 insertions, 2 deletionsContext
This is the remaining commit from the original
feat/comfyui-skill-v3branch after the first 4 commits were cherry-picked onto main via #17610. Note that the path (optional-skills/) differs from the current main (skills/via #17631) — this branch would need a merge conflict resolution or rebase.Closes # (no related issue)