facebookarchive/prepack

realm.$TemplateMap is persistent throughout effects

Open

#2.129 geöffnet am 18. Juni 2018

Auf GitHub ansehen
 (0 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (14.268 Stars) (520 Forks)batch import
abstractbughelp wantedinterpreterlevel 2 (easy)

Beschreibung

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.

Contributor Guide

Tech Stack
javascript
Domain
build systemperformance
Issue Type
bug
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
4
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-2 days
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
stale
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
mostly clear
Voraussetzungen
Understanding of JavaScript template literalsFamiliarity with prepack's effect system
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
20
Research-Richtung
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.