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

[workshop-sync] workshop/side-quest-17-06-output-injection.md: stale add_comment.cjs line references

Chiusa Adatta ai principianti
  • Qualcosa non va?
#4,679 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
78/100
Tipo di issue
Documentazione
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
github, javascript
Ambito
documentation

Direzione di ricerca

Inizia da workshop/side-quest-17-06-output-injection.md, in particolare dal blocco dei dettagli intorno alla riga 70 e dall’esercizio “Inspect the Validation Source” intorno alle righe 92–94. Verifica il comportamento descritto consultando actions/setup/js/add_comment.cjs, comment_limit_helpers.cjs e pkg/workflow/safe_outputs_parser.go; aggiorna i link o le istruzioni di ricerca e chiarisci che devono corrispondere tutte le etichette obbligatorie elencate. Il lavoro è concluso quando i riferimenti del workshop indirizzano gli studenti ai controlli attuali.

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

Descrizione

documentation

Workshop file reviewed

workshop/side-quest-17-06-output-injection.md (reviewed together with side-quest-17-05-supply-chain-mcp.md and side-quest-17-07-repo-poisoning.md; the other two had no accuracy issues). Checked against gh-aw release v0.92.0.

Problem

The file links to hard-coded line ranges in actions/setup/js/add_comment.cjs on main, and they no longer point at the described code:

add_comment.cjs#L582-L650, #L582-L583 ("the required-labels target check") and #L646-L650 ("comment sanitization and limits")

In the current main file (1184 lines), lines 582-583 and 646-650 are inside handleAddComment setup and comment-ID/target resolution code. They do not contain the label check or sanitization.

Current behaviour

  • The required-labels filter (does not match required-labels filter) is at about lines 724-762.
  • sanitizeContent(processedBody, ...) is at about line 817.
  • Mention and link limits come from comment_limit_helpers.cjs (enforceCommentLimits).
  • required-labels is still valid and is documented as "ALL must match" in pkg/workflow/safe_outputs_parser.go.

Suggested fix

  • Update the line anchors in the details block (line 70) and in the "Inspect the Validation Source" exercise (lines 92-94) to the current locations.
  • Better, pin the links to a release tag or commit SHA instead of main so they stay stable. Alternatively, link to the file without line anchors and tell learners to search for required-labels and sanitizeContent.
  • Line 93 says the check is "required-labels target check". The text could also note that all listed labels must be present.

Generated by 🔍 Workshop Sync Check · copilot · auto · 23.4 AIC · ⌖ 0.552 AIC · ⊞ 9.2K · ◷

  • expires on Oct 18, 2026, 8:39 AM UTC
Lingua principale
JavaScript
Stelle
53
Fork
35
Merge medio
1g 6h
PR unite (30g)
15

Preparare l'ambiente

Apri in Codespaces

Avvia il container di sviluppo del progetto nel browser, con il tuo account GitHub.

  • Nessun Dockerfile né 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 githubnext/gh-aw-workshop

Tutte le issue di githubnext/gh-aw-workshop

Issue simili

Altre issue su JavaScript

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.