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

🤖 ci: wait_pr_codex.sh usage-limit check trusts any author's request comment

Aperta Adatta ai principianti
  • Qualcosa non va?
#7,733 1 commento 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, github
Ambito
ci-cd

Direzione di ricerca

Inizia da scripts/wait_pr_codex.sh, vicino al controllo del limite di utilizzo, e da scripts/lib/codex_comments.jq, dove vengono filtrati gli autori delle richieste attendibili. Trova i test esistenti per lo script di attesa e aggiungi il caso descritto: il commento di richiesta di una persona esterna è seguito da una notifica del limite di utilizzo; lo script dovrebbe comunque poter pubblicare la propria richiesta. Il lavoro è completato quando questo caso passa e la correlazione del limite di utilizzo applica lo stesso filtro OWNER, MEMBER o COLLABORATOR.

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

Descrizione

Problem

scripts/wait_pr_codex.sh can fail early (exit 1) on a public PR when an outsider posts a comment that names the head and the words of a Codex security review request, and Codex later posts a usage-limit notice. The wait then reports "Codex usage limits reached: no security receipt for " before it posts its own request. It never passes a head without a receipt: the effect is an early failure in the local wait script only.

Cause

The usage-limit check added in #7732 (wait_pr_codex.sh, near line 468) finds the latest comment that names the head and the request words, from any author. The check that suppresses the script's own request (codex_security_review_requested_for in scripts/lib/codex_comments.jq) counts only authors with OWNER, MEMBER or COLLABORATOR association. The two checks disagree.

Fix direction

Apply the same OWNER/MEMBER/COLLABORATOR filter to the usage-limit correlation, for example by reusing one jq def for "a trusted request that names the head". Add a test case: an outsider's comment plus a later usage-limit notice still lets the script post its request.

Found by Codex round 5 on #7732 (thread PRRT_kwDOPxxmWM6xxG98). Refs #7731.

Generated with xum • Model: anthropic:claude-opus-5-5 • Thinking: high • Cost: $273.17

Lingua principale
TypeScript
Stelle
2.1k
Fork
139
Merge medio
3h 45m
PR unite (30g)
953

Preparare l'ambiente

  • Include un Dockerfile o un file Docker Compose
  • Nessun modello di pull request
  • Nessuna guida per i contributori

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/xum

Tutte le issue di coder/xum

Issue simili

Altre issue su TypeScript

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.