No error handling around execSync + JSON.parse in whitespace.js (ESLint 9 path)
@dataCenter430 y travaille déjà.
Depuis le 6/3/2026.
Évaluation
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Accessibilité débutants
- 48/100
- Type d'issue
- Bug
- Clarté
- Clairement spécifiée
- Activité
- À l'abandon
- Stack technique
- eslint, javascript
- Domaine
- tooling
Piste de recherche
Commencez à la ligne 55 de packages/eslint-config-airbnb-base/whitespace.js et packages/eslint-config-airbnb/whitespace.js, en suivant le chemin utilisant execSync et JSON.parse lorsque CLIEngine n’est pas disponible. Le travail est terminé lorsque les échecs des scripts enfants et les sorties invalides produisent un diagnostic clair, tout en permettant au chargement réussi de la configuration de continuer à fonctionner.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
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),
execSyncmay throw or return non-JSON. JSON.parsewill 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
I'd love to work on this.
- Langage dominant
- JavaScript
- Étoiles
- 148k
- Forks
- 26.5k
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Préparer son environnement
Ce projet ne fournit ni conteneur de développement, ni Dockerfile, ni guide de contribution : l'installation est à votre charge. Commencez par son README, et consultez notre guide de la première contribution pour les étapes générales.
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de airbnb/javascript
-
Severity: Unhandled promise rejection in `whitespace-async.js` when ESLint async path is usedPeut-être pris @bodapatisaikrishna l’a pris il y a 39 jours. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
airbnb/javascript#3237 · 6 commentaires ·
-
Inconsistent semicolon usage in examples (Arrays vs Functions)Peut-être pris @Developer-shivamMishra l’a pris il y a 29 jours. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
airbnb/javascript#3152 · 4 commentaires ·
-
Upgrading eslint-plugin-react-hooksPeut-être pris @weihongyu12 l’a pris il y a 379 jours. Ouverte
Difficulté 3/5 1-2 jours Accessibilité débutants 38/100
airbnb/javascript#3186 · 3 commentaires · 2 réactions ·
-
Difficulté 3/5 1-2 jours Accessibilité débutants 45/100
airbnb/javascript#3173 · 9 commentaires · 1 réaction ·
-
Isn't section 5.1 "Use object destructuring" conflicting with 3.3 "Use object method shorthand" ?Ouverte
Difficulté 4/5 3-5 jours Accessibilité débutants 35/100
airbnb/javascript#3136 · 11 commentaires ·
Toutes les issues de airbnb/javascript
Issues similaires
-
Clinic sharing: fast sync refusal message suggests Delete Data, which only developer builds haveOuvertebug
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
avniproject/avni-client#2228 ·
-
bug confirmed issue
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
open-webui/open-webui#33196 · 2 commentaires ·
Les mainteneurs répondent en général sous 1 jour
-
`printf "%o"` prints decimalOuverte
Difficulté 2/5 1-3 heures Accessibilité débutants 76/100
fable-compiler/Fable#5157 ·
Les mainteneurs répondent en général sous 1 jour
-
New tool: PaperBananaPeut-être pris @github-actions l’a pris aujourd’hui. Ouvertesubmission triage
Difficulté 1/5 Moins d'une heure Accessibilité débutants 75/100
kausalflow/tools#243 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
area: apple-music area: build-release bug effort: small priority: high
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
ChristianTabor/Argon#32 ·
Les mainteneurs répondent en général sous 1 jour