facebookarchive/prepack

realm.$TemplateMap is persistent throughout effects

Open

#2.129 aberto em 18 de jun. de 2018

Ver no GitHub
 (0 comments) (0 reactions) (0 assignees)JavaScript (14.268 stars) (520 forks)batch import
abstractbughelp wantedinterpreterlevel 2 (easy)

Description

Using ES2015 tagged template literals makes use of a $TemplateMap on the realm. When we evaluateForEffects and create a tagged template inside some effects, then revert the effects, it's expected that the newly created array objects in the template map are also stored on effects and properly reverted.

Guia do colaborador

Pilha de tecnologia
javascript
Domain
build systemperformance
Tipo Issue
bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
4
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-2 days
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
stale
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
mostly clear
Prerequisites
Understanding of JavaScript template literalsFamiliarity with prepack's effect system
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
20
Direção de pesquisa
The issue is about tagged template literals and the `$TemplateMap` on the realm. Investigate the effect system in prepack, specifically how effects are created and reverted when evaluating tagged templates. Look at the relevant source files in the repository, likely in the effects or realm modules, to understand how `$TemplateMap` entries are managed. There are no comments or linked PRs, so the fix requires a thorough understanding of the codebase.