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

No error handling around execSync + JSON.parse in whitespace.js (ESLint 9 path)

Abierto
  • ¿Algo no está bien?
#3,238 8 comentarios 0 reacciones 0 asignados Ver en GitHub

@dataCenter430 ya está trabajando en esto.

Desde el 6/3/2026.

  • #3239 de @dataCenter430 — abierto
  • #3306 de @loulanyue — cerrado sin fusionar
  • #3319 de @bodapatisaikrishna — abierto

Evaluación

Dificultad
2/5
Tiempo estimado
1-3 horas
Aptitud para principiantes
48/100
Tipo de issue
Error
Claridad
Bien especificado
Estado de actividad
Estancado
Stack tecnológico
eslint, javascript
Área
tooling

Línea de trabajo

Comienza en la línea 55 de packages/eslint-config-airbnb-base/whitespace.js y packages/eslint-config-airbnb/whitespace.js, siguiendo la ruta que usa execSync y JSON.parse cuando CLIEngine no está disponible. El trabajo estará terminado cuando los fallos de los scripts secundarios y las salidas no válidas produzcan un diagnóstico claro, mientras la carga correcta de la configuración siga funcionando.

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

Descripción

Summary

When CLIEngine is not available (e.g. ESLint 9), whitespace.js loads the config by running whitespace-async.js with execSync and then doing JSON.parse(String(...)) on the result. There is no try/catch around this.

Impact

  • If the child script exits non-zero, prints errors, or prints non-JSON (e.g. ESLint timing or debug output), execSync may throw or return non-JSON.
  • JSON.parse will then throw and the config load fails with a generic parsing error, making it hard to see that the real failure was in the child script.

Affected files

  • packages/eslint-config-airbnb-base/whitespace.js (line 55)
  • packages/eslint-config-airbnb/whitespace.js (line 55)

Suggested fix

Wrap the execSync call and JSON.parse in try/catch and rethrow with a clear message (and optionally cause: err) so failures are easier to diagnose.

screenshot

Image

I'd love to work on this.

Lenguaje dominante
JavaScript
Estrellas
148k
Forks
26.6k
Sin PR fusionados en 30 d

Preparar el entorno

Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.

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 airbnb/javascript

Todos los issues de airbnb/javascript

Issues similares

Más issues de JavaScript

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.