facebookarchive/prepack

Review/enforce possible values / types of tracked object properties

Open

#1124 aperta il 1 nov 2017

Vedi su GitHub
 (1 commento) (0 reazioni) (0 assegnatari)JavaScript (520 fork)batch import
bootcampeddesign neededhelp wantedlevel 3 (medium)test needed

Metriche repository

Star
 (14.268 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

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

Guida contributor

Direzione di ricerca
Esamina il file ObjectValue.js per comprendere l'attuale elenco di trackedPropertyNames e i loro tipi previsti. Osserva setupTrackedPropertyAccessors per vedere come vengono iniettate le proprietà e se è possibile aggiungere controlli di tipo. Inoltre, rivedi come oggetti simili ad array come $MapData utilizzano le proprietà tracciate per garantire la coerenza.
Tech stack
javascript
Dominio
backend
Tipo issue
Documentazione
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-2 giorni
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Richiede indagine
Prerequisiti
JavaScriptFlow
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
35