facebookarchive/prepack

Review/enforce possible values / types of tracked object properties

Open

#1 124 ouverte le 1 nov. 2017

Voir sur GitHub
 (1 commentaire) (0 réactions) (0 assignés)JavaScript (520 forks)batch import
bootcampeddesign neededhelp wantedlevel 3 (medium)test needed

Métriques du dépôt

Stars
 (14 268 stars)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

In ObjectValue.js, there's a list of trackedPropertyNames.

Tracked objects must be void or a (subtype of a) Value. After state joins, an AbstractValue may appear (encoding a conditional expression), or possibly other Value kinds due to simplification rules. Maybe there's some special logic in place for arrays (otherwise, how does $MapData and $SetData and $WeakMapData and $WeakSetData work as tracked objects?)

We should...

  • Document the actual requirements where trackedPropertyNames is defined
  • Consider including type information in trackedPropertyNames and issuing type invariant checks as part of the property injection in setupTrackedPropertyAccessors.
  • Manually review all static Flow types of tracked properties to check if they conform

Guide contributeur

Direction de recherche
Examinez le fichier ObjectValue.js pour comprendre la liste actuelle des trackedPropertyNames et leurs types attendus. Regardez setupTrackedPropertyAccessors pour voir comment les propriétés sont injectées et si des vérifications de type peuvent être ajoutées. De plus, examinez comment les objets de type tableau comme $MapData utilisent les propriétés suivies pour garantir la cohérence.
Stack technique
javascript
Domaine
backend
Type d'issue
Documentation
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-2 jours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Ancienne
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Investigation requise
Prérequis
JavaScriptFlow
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
35