Provide built-in implementations of module linker, initializeImportMeta, importModuleDynamically
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- JavaScript
- Sterne
- 122k
- Forks
- 37.3k
- Ø Merge
- 4 T. 2 Std.
- Gemergte PRs (30 T.)
- 283
Beschreibung
Is your feature request related to a problem? Please describe.
When using vm.SourceTextModule we have to implement a linker, initializeImportMeta, and importModuleDynamically, even if we just want the default behavior of built-in module support. There are many subtle ways to get that wrong.
Describe the solution you'd like
Default implementations that do basically what built-in module support does:
- A linker function that:
- Reads files from disk, within a specified root.
- Uses Node module resolution as implemented for modules
- Uses a specified module cache
initializeImportMetathat setsurlproperty- By configuration, allows access to built-in modules.
Describe alternatives you've considered
Userland libraries can take the first cuts at easy-to-use vm.Modules, but I think something for common use cases should likely be included.
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne mit dem Einstiegspunkt vm.SourceTextModule und dem im Issue referenzierten Node-Verhalten bei der Modulauflösung. Arbeite die angeforderten Standardwerte für linker, initializeImportMeta und importModuleDynamically durch, einschließlich der Konfiguration für root, den Modul-Cache und integrierte Module. Fertig ist die Aufgabe, wenn gängige vm.Modules integrierte Implementierungen ohne benutzerdefinierte Callbacks verwenden können.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- javascript, node.js
- Bereich
- backend
- Issue-Typ
- Feature
- Schwierigkeit
- 5/5
- Geschätzter Aufwand
- Über eine Woche
- Aktivitätsstatus
- Ruhig
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 45/100