Hacktoberfest 2026: los issues que los mantenedores marcaron para octubre, abiertos y aptos para principiantes. Explorar issues de Hacktoberfest

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

Abierto Apto para principiantes
  • ¿Algo no está bien?
#4,679 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Los mantenedores suelen responder en 1 día

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
2/5
Tiempo estimado
1-3 horas
Aptitud para principiantes
78/100
Tipo de issue
Documentación
Claridad
Bien especificado
Estado de actividad
Activo
Stack tecnológico
github, javascript
Área
documentation

Línea de trabajo

Empieza con workshop/side-quest-17-06-output-injection.md, especialmente con el bloque de detalles alrededor de la línea 70 y el ejercicio “Inspect the Validation Source” alrededor de las líneas 92–94. Comprueba el comportamiento descrito en actions/setup/js/add_comment.cjs, comment_limit_helpers.cjs y pkg/workflow/safe_outputs_parser.go; actualiza los enlaces o las instrucciones de búsqueda y aclara que deben coincidir todas las etiquetas obligatorias enumeradas. El trabajo estará listo cuando las referencias del taller dirijan a los alumnos a las comprobaciones actuales.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

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
Lenguaje dominante
JavaScript
Estrellas
53
Forks
35
Merge medio
1 d 5 h
PR fusionados (30 d)
16

Preparar el entorno

Abrir en Codespaces

Inicia el contenedor de desarrollo del proyecto en tu navegador, con tu propia cuenta de GitHub.

  • Sin Dockerfile ni archivo de Docker Compose
  • Sin plantilla de pull request
  • Sin guía de contribución

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de githubnext/gh-aw-workshop

Todos los issues de githubnext/gh-aw-workshop

Issues similares

Más issues de JavaScript

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.