react / react/react-native

Key error shows internal code frames

Ouverte
#41,945 6 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

DX Impact: Regression LogBox Needs: Attention Never gets stale Tech: Bundler 📦
Langage dominant
C++
Étoiles
127k
Forks
25.3k
Merge moyen
1 j 23 h
PR mergées (30 j)
4

Description

Description

In LogBox, when there's a key error from React, the code frame should point to the application code causing the issue, but in 0.73 (and maybe other versions) it points to the react-jsx-runtime:

Screenshot 2023-12-14 at 1 05 06 PM

The fix is to add '/node_modules/react/cjs/react-jsx-runtime.+\\.js', to the config here:

https://github.com/facebook/react-native/blob/main/packages/metro-config/index.js#L15

Really we should probably have all of '/node_modules/react/.+\\.js$', there.

Steps to reproduce
  1. trigger key error
  2. open logbox warning
  3. see wrong code frame
React Native Version

0.73.0

Affected Platforms

Runtime - Android, Runtime - iOS

Output of npx react-native info
No
Stacktrace or Logs
No
Reproducer

No

Screenshots and Videos

Above

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.

Piste de recherche

Commencez dans packages/metro-config/index.js, autour de la ligne 15, où la configuration de code-frame est définie. Reproduisez l’erreur principale dans LogBox et vérifiez quel fichier du runtime React est sélectionné à la place du code source de l’application. C’est terminé lorsque l’avertissement pointe vers le code de l’application plutôt que vers un fichier interne react-jsx-runtime.

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

Évaluation

Stack technique
react, react-native
Domaine
mobile, tooling
Type d'issue
Bug
Difficulté
2/5
Temps estimé
1-3 heures
Activité
À l'abandon
Clarté
Clairement spécifiée
Accessibilité débutants
45/100

Recevez les nouvelles issues par e-mail

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