Open phorks opened 2 years ago
This is intentional, but I have added an option useCurrentGlobal
and some factory functions createRequireFromString
etc. in the pre-released version 3.2.0-0
. You can install it and give it a try.
If it works well, I will update the README and publish a new minor version tomorrow.
Please use the newly added option useCurrentGlobal
in 3.2.0
.
Thank you. Just a question about the implementation though, did you consider the compileFunction
approach that the default nodejs cjs loader takes (here)? That way, there would be no need to shallow copy the entire global
object each time.
Thank you for the suggestion, I haven't actually considered this approach.
I will try refactoring the code in the next few days.
The
instanceof
operator always returnsfalse
when used with the values from imported modules.It outputs:
true
false
false
Or:
Outputs:
true
false
Using
importFromString
yields the same results. It's a side effect of usingrunInNewContext
instead ofrunInThisContext
(or providing context in case of thevm.Module
). If this behavior is not intentional, I would be glad to fix it.