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

Aperta Adatta ai principianti
  • Qualcosa non va?
#61,436 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
2/5
Tempo stimato
1-3 ore
Idoneità per principianti
62/100
Tipo di issue
Bug
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
python, rust, shell
Ambito
tooling

Direzione di ricerca

Inizia dall'installer in codex-rs/skills/src/assets/samples/skill-installer/scripts/install-skill-from-github.py, in particolare dall'estrazione dello ZIP intorno alle righe 108-116 dove viene usato ZipFile.extractall. Riproduci il problema con i passaggi shell della issue e confronta i permessi di playwright/scripts/playwright_cli.sh tra le destinazioni auto e git. È fatto quando il wrapper scaricato ha il bit di esecuzione (755) come nell'installazione Git, con una verifica che i due percorsi di installazione producano gli stessi permessi.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

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

Lingua principale
Rust
Stelle
135k
Fork
21.4k
Merge medio
1m
PR unite (30g)
997

Preparare l'ambiente

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di openai/codex

Tutte le issue di openai/codex

Issue simili

Altre issue su Rust

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.