fixup --expose-internals esm facade

nodejs · Oct 7, 2019 · 10d7585 · 10d7585
1 parent d85bb91
commit 10d7585
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/lib/internal/bootstrap/loaders.js b/lib/internal/bootstrap/loaders.js
@@ -155,6 +155,7 @@ function NativeModule(id) {
   this.loaded = false;
   this.loading = false;
   this.canBeRequiredByUsers = !id.startsWith('internal/');
+  this.defaultOnly = false;
 }
 
 // To be called during pre-execution when --expose-internals is on.
@@ -164,6 +165,7 @@ NativeModule.exposeInternals = function() {
     // Do not expose this to user land even with --expose-internals.
     if (id !== loaderId) {
       mod.canBeRequiredByUsers = true;
+      mod.defaultOnly = true;
     }
   }
 };
@@ -219,7 +221,7 @@ NativeModule.prototype.compileForPublicLoader = function() {
   }
   this.compile();
   if (!this.exportKeys) {
-    this.exportKeys = Object.keys(this.exports);
+    this.exportKeys = this.defaultOnly ? [] : Object.keys(this.exports);
   }
   this.getESMFacade();
   this.syncExports();