Hacktoberfest 2026: die Issues, die Maintainer für den Oktober markiert haben – offen und einsteigerfreundlich. Hacktoberfest-Issues durchsuchen

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

Offen
  • Stimmt etwas nicht?
#3,238 8 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

@dataCenter430 arbeitet bereits daran.

Seit 06.3.2026.

  • #3239 von @dataCenter430 — offen
  • #3319 von @bodapatisaikrishna — offen

Bewertung

Schwierigkeit
2/5
Geschätzter Aufwand
1-3 Stunden
Anfängerfreundlichkeit
48/100
Issue-Typ
Bug
Klarheit
Klar beschrieben
Aktivitätsstatus
Veraltet
Tech-Stack
eslint, javascript
Bereich
tooling

Rechercherichtung

Beginne bei Zeile 55 in packages/eslint-config-airbnb-base/whitespace.js und packages/eslint-config-airbnb/whitespace.js und verfolge den von execSync und JSON.parse verwendeten Pfad, wenn CLIEngine nicht verfügbar ist. Die Arbeit ist abgeschlossen, wenn Fehler von Child-Skripten und ungültige Ausgaben eine klare Diagnose erzeugen, während das erfolgreiche Laden der Konfiguration weiterhin funktioniert.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

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.

Vorherrschende Sprache
JavaScript
Sterne
148k
Forks
26.5k
Keine gemergten PRs in 30 T.

Entwicklungsumgebung

Dieses Projekt bietet weder Dev-Container noch Dockerfile noch Beitragsleitfaden – die Einrichtung liegt bei Ihnen. Beginnen Sie mit der README; die allgemeinen Schritte stehen in unserem Leitfaden für den ersten Beitrag.

Erste Schritte

  1. Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
  3. Forken Sie das Repository und arbeiten Sie in einem Branch.
  4. Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus airbnb/javascript

Alle Issues in airbnb/javascript

Ähnliche Issues

Weitere Issues zu JavaScript

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.