[PR #31] [MERGED] Remove comfy-build: moved into comfy-cli's own skills #30

New issue

Closed

opened 2026-10-26 22:34:10 +00:00 by bsakaguchi · 0 comments

bsakaguchi commented

2026-10-26 22:34:10 +00:00

Owner

📋 Pull Request Information

Original PR: https://github.com/Comfy-Org/comfy-skills/pull/31
Author: @claude[bot]
Created: 8/26/2026
Status: ✅ Merged
Merged: 8/26/2026
Merged by: @james00012

Base: main ← Head: move-build-skill-to-comfy-cli

📝 Commits (1)

1c66af0 Remove comfy-build: moved into comfy-cli's own skills (BE-9486)

📊 Changes

3 files changed (+2 additions, -702 deletions)

View changed files

➖ .github/workflows/validate-skills.yml (+0 -45)
📝 README.md (+2 -1)
➖ comfy-cli/comfy-build/SKILL.md (+0 -656)

📄 Description

Requested by James Kwon · Slack thread

Before

comfy-build lived here, under comfy-cli/comfy-build/SKILL.md, and was fetched over the network by comfy skills install --skill comfy-build at install time — the only skill installed that way. That meant comfy skills show didn't work for it (unlike every other comfy-cli skill), and its content could drift out of sync with whatever CLI version a user actually had installed.

After

comfy-build ships bundled inside comfy-cli itself (comfy_cli/skills/comfy-build/), versioned with each CLI release, and is discoverable/usable through the CLI's normal skills commands (list, show, install, etc.) like any other skill.

How

Removed comfy-cli/comfy-build/SKILL.md (and the now-empty comfy-cli/ directory).
Removed .github/workflows/validate-skills.yml, which existed solely to run comfy skills validate over the directories under comfy-cli/ — with comfy-build gone, that directory is empty and the workflow has nothing left to check.
Updated the README's repo-layout section to drop the comfy-cli/ bullet and point readers at comfy-cli's own comfy_cli/skills/ for its bundled skills going forward.

Companion PR adding the skill to comfy-cli: https://github.com/Comfy-Org/comfy-cli/pull/788

Linear: https://linear.app/comfyorg/issue/BE-9486/integrate-builder-skills-into-clis-existing-skills-command

Generated by Claude Code

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/Comfy-Org/comfy-skills/pull/31 **Author:** [@claude[bot]](https://github.com/apps/claude) **Created:** 8/26/2026 **Status:** ✅ Merged **Merged:** 8/26/2026 **Merged by:** [@james00012](https://github.com/james00012) **Base:** `main` ← **Head:** `move-build-skill-to-comfy-cli` --- ### 📝 Commits (1) - [`1c66af0`](https://github.com/Comfy-Org/comfy-skills/commit/1c66af0675310c2728f2ff1ddc2e73f41f02542c) Remove comfy-build: moved into comfy-cli's own skills (BE-9486) ### 📊 Changes **3 files changed** (+2 additions, -702 deletions) <details> <summary>View changed files</summary> ➖ `.github/workflows/validate-skills.yml` (+0 -45) 📝 `README.md` (+2 -1) ➖ `comfy-cli/comfy-build/SKILL.md` (+0 -656) </details> ### 📄 Description  _Requested by **James Kwon** · [Slack thread](https://comfy-organization.slack.com/archives/C0BD6BR53QX/p1787681847332999?thread_ts=1787681847.332999&cid=C0BD6BR53QX)_ ## Before `comfy-build` lived here, under `comfy-cli/comfy-build/SKILL.md`, and was fetched over the network by `comfy skills install --skill comfy-build` at install time — the only skill installed that way. That meant `comfy skills show` didn't work for it (unlike every other comfy-cli skill), and its content could drift out of sync with whatever CLI version a user actually had installed. ## After `comfy-build` ships bundled inside [comfy-cli](https://github.com/Comfy-Org/comfy-cli) itself (`comfy_cli/skills/comfy-build/`), versioned with each CLI release, and is discoverable/usable through the CLI's normal skills commands (`list`, `show`, `install`, etc.) like any other skill. ## How - Removed `comfy-cli/comfy-build/SKILL.md` (and the now-empty `comfy-cli/` directory). - Removed `.github/workflows/validate-skills.yml`, which existed solely to run `comfy skills validate` over the directories under `comfy-cli/` — with `comfy-build` gone, that directory is empty and the workflow has nothing left to check. - Updated the README's repo-layout section to drop the `comfy-cli/` bullet and point readers at comfy-cli's own `comfy_cli/skills/` for its bundled skills going forward. Companion PR adding the skill to comfy-cli: https://github.com/Comfy-Org/comfy-cli/pull/788 Linear: https://linear.app/comfyorg/issue/BE-9486/integrate-builder-skills-into-clis-existing-skills-command --- _Generated by [Claude Code](https://claude.ai/code/session_01PbEixFDWBUXz45TdED6zui)_ --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

bsakaguchi

2026-10-26 22:34:10 +00:00

closed this issue
added the
pull-request
label

No labels

pull-request

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

Comfy-Org/comfy-skills#30

No description provided.

Rows
Columns