comfy skills install --skill comfy-build silently skips on every released CLI

未关闭
#797 1 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看

维护者通常 1 天内回复

还没有人认领这个 Issue。

评估

难度
3/5
预计耗时
1-2 天
新手友好度
58/100
Issue 类型
缺陷
描述清晰度
基本清楚
活跃度
活跃
技术栈
python
领域
cli, release

调研方向

使用 comfy skills install --skill comfy-build --dry-run 重现故障,然后检查 comfy_cli/skills/__init__.py 以及对应固定路径的 REMOTE_SKILLS 条目。检查提交 4a86b7c 和已发布的 v1.18.0 树,以了解 release 不匹配的原因。当已发布的 CLI 可以安装 comfy-build,且不会出现 404 或误导性的跳过结果时,即视为完成。

由索引模型根据 Issue 内容生成。

描述

bug

comfy skills install --skill comfy-build cannot install the skill on any published release. It reports skipped, exit 0.

What happens

$ comfy skills install --skill comfy-build --dry-run
│  comfy-build  │ claude-code  │ skipped  │ ~/.claude/skills/comfy-build/SKILL.md  │
│  comfy-build  │ cursor       │ skipped  │ ~/.cursor/rules/comfy-build.mdc        │
│  comfy-build  │ agents-md    │ skipped  │ ~/AGENTS.md                           │
│  comfy-build skipped: could not fetch
│  https://raw.githubusercontent.com/Comfy-Org/comfy-skills/main/comfy-cli/comfy-build/SKILL.md: HTTP Error 404: Not Found

Why

v1.18.0 fetches the skill at install time from a path that no longer exists:

# comfy_cli/skills/__init__.py (v1.18.0)
REMOTE_SKILLS = (RemoteSkill(name="comfy-build", repo="Comfy-Org/comfy-skills",
                             ref="main", path="comfy-cli/comfy-build/SKILL.md"),)

The skill moved into this repository in 4a86b7c (BE-9486), which deleted it from Comfy-Org/comfy-skills. main is fine because the bundling commit dropped the remote fetch, but v1.18.0 is the version on PyPI and it still points at the dead path. Verified: that URL returns 404, and git ls-tree v1.18.0 comfy_cli/skills/ has no comfy-build.

Impact

Anyone on pip install comfy-cli today asks for the skill and is told skipped with a 404 they cannot act on. comfy skills list compounds it by showing a description that reads as though the skill is available: Fetched from Comfy-Org/comfy-skills (main) by 'comfy skills install'.

Suggested fix

A release carrying 4a86b7c resolves it. If a release is not imminent, consider restoring comfy-cli/comfy-build/SKILL.md in Comfy-Org/comfy-skills as a pointer or a copy so the pinned ref="main" fetch stops 404ing for users already on 1.18.0.

Found while driving the skill end to end against a real builder.

主要语言
Python
星标
1k
派生
176
平均合并
1 天 11 小时
30 天内合并 PR
87

环境准备

  • 没有 Dockerfile 或 Docker Compose 文件
  • 没有 Pull Request 模板
  • 阅读贡献指南

从这里开始

  1. 先读完整个 Issue,再读项目的贡献指南。
  2. 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
  3. Fork 仓库,在一个分支上完成修改。
  4. 提交 Pull Request,并在描述里引用这个 Issue 编号。

Comfy-Org/comfy-cli 的其他 Issue

查看 Comfy-Org/comfy-cli 的全部 Issue

相似的 Issue

更多 Python Issue

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。