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

Đang mở
  • Có gì không ổn?
#797 1 bình luận 0 reaction 0 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 1 ngày

Chưa có ai nhận issue này.

Đánh giá

Độ khó
3/5
Thời gian dự kiến
1-2 ngày
Mức phù hợp với người mới
58/100
Loại issue
Lỗi
Độ rõ ràng
Khá rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
python
Lĩnh vực
cli, release

Hướng nghiên cứu

Tái hiện lỗi bằng comfy skills install --skill comfy-build --dry-run, sau đó kiểm tra comfy_cli/skills/__init__.py và mục nhập REMOTE_SKILLS cho đường dẫn được ghim. Kiểm tra commit 4a86b7c và cây v1.18.0 đã phát hành để hiểu sự không khớp của release. Hoàn thành khi một CLI đã phát hành có thể cài đặt comfy-build mà không gặp 404 hoặc kết quả bỏ qua gây hiểu nhầm.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

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.

Ngôn ngữ chính
Python
Star
1k
Fork
176
Merge trung bình
1 ngày 11 giờ
Pull request đã merge (30 ngày)
87

Chuẩn bị môi trường

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của Comfy-Org/comfy-cli

Tất cả issue của Comfy-Org/comfy-cli

Issue tương tự

Thêm issue về Python

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.