Hacktoberfest 2026: le issue che i maintainer hanno segnato per ottobre, aperte e adatte ai principianti. Sfoglia le issue Hacktoberfest

🤖 verification skills: harden check.sh and verify.sh argument and file handling (#391 follow-ups)

Aperta Adatta ai principianti
#392 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
76/100
Tipo di issue
Bug
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
bash
Ambito
devops, tooling

Direzione di ricerca

Leggi hack/formal/aigatewaykey/check.sh e .mux/skills/verify-coder-k8s/scripts/verify.sh. Controlla come check.sh analizza --config, --out e --set, poi esamina load_run e le trappole EXIT in verify.sh. Il lavoro è completato quando i valori delle opzioni mancanti causano l’uscita con codice 2, run.env viene rifiutato se è un collegamento simbolico o ha permessi di scrittura per il gruppo o per gli altri, e la gestione delle trappole funziona con un apostrofo in VERIFY_RUN_DIR; l’issue non menziona alcun comando di test o di esecuzione.

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

Descrizione

Follow-ups from Codex review round 4 on #391. Under the review limit for #391 (rounds 4 to 6 fix blockers only), I judged these minor and deferred them.

  1. hack/formal/aigatewaykey/check.sh: --config, --out or --set as the last argument exits 1 ("unbound variable"). Exit 1 means a rule violation. A malformed call must exit 2. Fix: check that each option has a value before reading $2.
  2. .mux/skills/verify-coder-k8s/scripts/verify.sh load_run: before it sources run.env, check that the file is a regular file (not a symlink) with no group or other write bits. Launch creates it with mode 0600 in a private run dir, so this matters only after someone changes its mode.
  3. verify.sh EXIT traps (coder-api port-forward, temp files): the traps build shell code from expanded paths. A ' in VERIFY_RUN_DIR breaks the trap and leaves the port-forward running. Use a trap function that reads already-set variables.

Trigger: the next change to either script.


Generated with mux • Model: anthropic:claude-opus-5-5 • Thinking: high

Lingua principale
Go
Stelle
5
Fork
3
Merge medio
2h 56m
PR unite (30g)
163

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 coder/coder-k8s

Tutte le issue di coder/coder-k8s

Issue simili

Altre issue su Go

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.