Microsoft/TypeScript

The parsed identifier is incomplete due to updateSourceFile.

Open

#60 773 ouverte le 16 déc. 2024

Voir sur GitHub
 (2 commentaires) (0 réactions) (0 assignés)TypeScript (6 726 forks)batch import
BugDomain: ParserHelp Wanted

Métriques du dépôt

Stars
 (48 455 stars)
Métriques de merge PR
 (Merge moyen 6j 17h) (9 PRs mergées en 30 j)

Description

🔎 Search Terms

updateSourceFile

🕗 Version & Regression Information

5.6.0

⏯ Playground Link

No response

💻 Code

// Your code here

🙁 Actual behavior

class Test { testMethod() { let newLocal = "kk" let a = 123 let b = 12 } } After the value of variable b is changed, the identifiers set of sourceFile parsed by the updateSourceFile method contains only Test, testMethod, and b. Variables a and newLocal are lost.

🙂 Expected behavior

the identifiers set of sourceFile contains Test, testMethod, b, a and newLocal.

Additional information about the issue

No response

Guide contributeur

Direction de recherche
Étudiez la fonction `updateSourceFile` dans l'analyseur syntaxique de TypeScript (probablement dans `src/compiler/parser.ts`). Concentrez vous sur la façon dont elle construit l'ensemble des identifiants après l'analyse incrémentale. Le bug suggère que lorsque la valeur d'une variable change, les identifiants des nœuds inchangés (comme `newLocal` et `a`) sont supprimés. Vérifiez comment `SyntaxList` et `NodeArray` sont gérés pendant les mises à jour incrémentales. Comparez potentiellement la logique de collecte des identifiants dans `updateSourceFile` avec le chemin d'analyse complet. Configurez un cas de test reproduisant le problème et parcourez l'analyse incrémentale étape par étape pour voir où les identifiants sont perdus.
Stack technique
typescript
Domaine
backend
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Récente
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Plutôt claire
Prérequis
TypeScript
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
45