You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Change API of module.lnk((specifier) => module) to module.link(specifiers, modules) so that it has fewer C++ -> JS calls and can be reused to implement require(esm).
According to @SimenB, here are the issues that should probably be taken care of before we unflag and mark vm modules as stable:
vm.Script'simportModuleDynamicallyshould get a reference to thecontextused to run it (#35714)vmmodules #31658import()instead of callingimportModuleDynamically(#35889)vm.SourceTextModulememory leak #59118SourceTextModule: Using dynamic imports while passingcachedDatathrowsTypeError: Invalid host defined options#43663module.lnk((specifier) => module)tomodule.link(specifiers, modules)so that it has fewer C++ -> JS calls and can be reused to implementrequire(esm).Tracking issue in Jest: jestjs/jest#9430
/cc @devsnek