facebookarchive/prepack

Review/enforce possible values / types of tracked object properties

Open

#1.124 aberto em 1 de nov. de 2017

Ver no GitHub
 (1 comment) (0 reactions) (0 assignees)JavaScript (520 forks)batch import
bootcampeddesign neededhelp wantedlevel 3 (medium)test needed

Métricas do repositório

Stars
 (14.268 stars)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

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

Guia do colaborador

Direção de pesquisa
Examine o arquivo ObjectValue.js para entender a lista atual de trackedPropertyNames e seus tipos esperados. Veja setupTrackedPropertyAccessors para entender como as propriedades são injetadas e se verificações de tipo podem ser adicionadas. Além disso, revise como objetos semelhantes a arrays, como $MapData, usam propriedades rastreadas para garantir consistência.
Pilha de tecnologia
javascript
Domain
backend
Tipo Issue
Documentação
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-2 dias
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Antigo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Precisa de investigação
Prerequisites
JavaScriptFlow
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
35