Support Babel 8 in babelParser (loadPartialConfigSync)

Ouverte Adaptée aux débutants
#1,102 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

Évaluation

Difficulté
2/5
Temps estimé
1-3 heures
Accessibilité débutants
78/100
Type d'issue
Bug
Clarté
Clairement spécifiée
Activité
Calme
Stack technique
react, typescript
Domaine
tooling

Piste de recherche

Commencez par le chemin synchrone de babelParser indiqué dans dist/babelParser.js et examinez l'appel à loadPartialConfig. Utilisez le repro.mjs fourni avec @babel/core 8.0.1, puis vérifiez que l'analyse syntaxique réussit avec Babel 8 tout en conservant la compatibilité avec Babel 7.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Description

Description

react-docgen@8.0.3 cannot parse components when its @babel/core dependency is resolved to Babel 8.

babelParser calls loadPartialConfig synchronously. Starting with Babel 8, that API expects a callback; synchronous callers must use loadPartialConfigSync.

This surfaced through Storybook's react-docgen plugin when a workspace-wide override resolved Babel to 8.0.1. Storybook is currently tracking Babel 8 support and explicitly waiting for react-docgen compatibility: https://github.com/storybookjs/storybook/issues/35295

Minimal reproduction

package.json:

{
  "private": true,
  "type": "module",
  "packageManager": "pnpm@11.13.0",
  "dependencies": {
    "@babel/core": "8.0.1",
    "react-docgen": "8.0.3"
  }
}

pnpm-workspace.yaml:

packages:
  - .

overrides:
  '@babel/core': 8.0.1

repro.mjs:

import { parse } from 'react-docgen';

parse('export const Button = ({ label }) => <button>{label}</button>;');

Run:

pnpm install
node repro.mjs

Actual result

Error: Starting from Babel 8.0.0, the 'loadPartialConfig' function expects a callback. If you need to call it synchronously, please use 'loadPartialConfigSync'.
    at loadPartialConfig (.../@babel/core/lib/index-shared.js:2207:11)
    at buildPluginList (.../react-docgen/dist/babelParser.js:34:27)

Expected result

Parsing succeeds with Babel 8, while retaining Babel 7 compatibility.

Versions

  • react-docgen: 8.0.3
  • @babel/core: 8.0.1
  • Node.js: 24.18.0
  • pnpm: 11.13.0

The likely compatibility change is to use loadPartialConfigSync for this synchronous code path; that API is also available in Babel 7.

Langage dominant
TypeScript
Étoiles
3.8k
Forks
316
Merge moyen
3 h 28 min
PR mergées (30 j)
6

Guide de contribution

Ouvrir le guide de contribution

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de reactjs/react-docgen

Toutes les issues de reactjs/react-docgen

Issues similaires

Plus d'issues TypeScript

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.