Default skill installation strips executable bits and breaks the curated Playwright launcher

Đang mở Phù hợp với người mới
  • Có gì không ổn?
#61,436 0 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ó
2/5
Thời gian dự kiến
1-3 giờ
Mức phù hợp với người mới
62/100
Loại issue
Lỗi
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
python, rust, shell
Lĩnh vực
tooling

Hướng nghiên cứu

Bắt đầu từ trình cài đặt tại codex-rs/skills/src/assets/samples/skill-installer/scripts/install-skill-from-github.py, cụ thể là phần giải nén ZIP quanh dòng 108-116, nơi dùng ZipFile.extractall. Tái hiện lỗi bằng các bước shell trong issue và so sánh quyền (mode) của playwright/scripts/playwright_cli.sh giữa đích auto và git. Hoàn thành khi wrapper đã tải về có bit thực thi (755) giống bản cài đặt từ Git, và có kiểm tra xác nhận hai đường cài đặt cho cùng quyền.

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

Mô tả

bug CLI skills

The bundled skill installer reports that the Playwright skill installed successfully, but the wrapper in its quick start cannot be executed. The same upstream script remains executable when the installer uses Git.

Steps to reproduce On Linux x86_64 with Python 3.12.14, from the Codex repository root at de83e7674a69b9ae18544dd1fbf53f31bfe8c640:

dest=$(mktemp -d)
script=codex-rs/skills/src/assets/samples/skill-installer/scripts/install-skill-from-github.py
skill=skills/.curated/playwright
wrapper=playwright/scripts/playwright_cli.sh

python3 "$script" --repo openai/skills --path "$skill" --method auto --dest "$dest/auto"
python3 "$script" --repo openai/skills --path "$skill" --method git --dest "$dest/git"
cmp "$dest/auto/$wrapper" "$dest/git/$wrapper"
stat -c '%a %n' "$dest/auto/$wrapper" "$dest/git/$wrapper"
"$dest/auto/$wrapper" --help

Observed Both installs exit 0 and the two scripts have identical contents. Their modes are 644 (default download path) and 755 (Git). Directly invoking the downloaded script returns Permission denied before Playwright starts. The openai/skills revision used was 49f948faa9258a0c61caceaf225e179651397431.

Expected A tracked executable should remain executable through either install method. The skill's quick start invokes this wrapper directly. Playwright usage

Suggested fix Restore the Git executable bits for regular files when extracting the archive and test mode parity between the download and Git installation paths. The ZIP path currently delegates extraction to ZipFile.extractall. Archive extraction

Ngôn ngữ chính
Rust
Star
135k
Fork
21.4k
Merge trung bình
1 phút
Pull request đã merge (30 ngày)
997

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 openai/codex

Tất cả issue của openai/codex

Issue tương tự

Thêm issue về Rust

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.