Support Babel 8 in babelParser (loadPartialConfigSync)

Abierto Apto para principiantes
#1,102 0 comentarios 0 reacciones 0 asignados Ver en GitHub

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
Error
Claridad
Bien especificado
Estado de actividad
Tranquilo
Stack tecnológico
react, typescript
Área
tooling

Línea de trabajo

Comienza por la ruta síncrona de babelParser mostrada en dist/babelParser.js e inspecciona la llamada a loadPartialConfig. Usa el repro.mjs proporcionado con @babel/core 8.0.1 y, después, verifica que el análisis sintáctico se realice correctamente con Babel 8 manteniendo la compatibilidad con Babel 7.

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

Descripción

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.

Lenguaje dominante
TypeScript
Estrellas
3.8k
Forks
316
Merge medio
5 h 7 min
PR fusionados (30 d)
4

Guía de contribución

Abrir la 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 reactjs/react-docgen

Todos los issues de reactjs/react-docgen

Issues similares

Más issues de TypeScript

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.