bootstrap: run preload prior to frozen-intrinsics

This is used to allow people to run polyfills. Co-Authored-By: Anna Henningsen <[email protected]> PR-URL: nodejs#28940 Reviewed-By: Guy Bedford <[email protected]> Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Gus Caplan <[email protected]> Reviewed-By: Rich Trott <[email protected]> Reviewed-By: James M Snell <[email protected]>
yashLadha · Aug 20, 2019 · 85898e0 · 85898e0
1 parent 9fd9efa
commit 85898e0
Show file tree

Hide file tree

Showing 7 changed files with 49 additions and 2 deletions.
diff --git a/doc/api/cli.md b/doc/api/cli.md
@@ -218,6 +218,9 @@ Support is currently only provided for the root context and no guarantees are
 currently provided that `global.Array` is indeed the default intrinsic
 reference. Code may break under this flag.
 
+`--require` runs prior to freezing intrinsics in order to allow polyfills to
+be added.
+
 ### `--heapsnapshot-signal=signal`
 <!-- YAML
 added: v12.0.0

diff --git a/lib/internal/bootstrap/pre_execution.js b/lib/internal/bootstrap/pre_execution.js
@@ -48,10 +48,10 @@ function prepareMainThreadExecution(expandArgv1 = false) {
   initializeClusterIPC();
 
   initializeDeprecations();
-  initializeFrozenIntrinsics();
   initializeCJSLoader();
   initializeESMLoader();
   loadPreloadModules();
+  initializeFrozenIntrinsics();
 }
 
 function patchProcessObject(expandArgv1) {

diff --git a/lib/internal/main/worker_thread.js b/lib/internal/main/worker_thread.js
@@ -106,10 +106,10 @@ port.on('message', (message) => {
       require('internal/process/policy').setup(manifestSrc, manifestURL);
     }
     initializeDeprecations();
-    initializeFrozenIntrinsics();
     initializeCJSLoader();
     initializeESMLoader();
     loadPreloadModules();
+    initializeFrozenIntrinsics();
     publicWorker.parentPort = publicPort;
     publicWorker.workerData = workerData;
 

diff --git a/test/fixtures/intrinsic-mutation.js b/test/fixtures/intrinsic-mutation.js
@@ -0,0 +1,10 @@
+'use strict';
+Object.defineProperty(
+  Object.prototype,
+  'flatten', {
+    enumerable: false,
+    // purposefully named something that
+    // would never land in JS itself
+    value: function smoosh() {}
+  }
+);
diff --git a/test/fixtures/print-intrinsic-mutation-name.js b/test/fixtures/print-intrinsic-mutation-name.js
@@ -0,0 +1,2 @@
+'use strict';
+console.log({}.flatten.name);
diff --git a/test/fixtures/worker-from-argv.js b/test/fixtures/worker-from-argv.js
@@ -0,0 +1,3 @@
+'use strict';
+const {Worker} = require('worker_threads');
+new Worker(process.argv[2]).on('exit', process.exit);
diff --git a/test/parallel/test-preload.js b/test/parallel/test-preload.js
@@ -23,6 +23,9 @@ const fixtureA = fixtures.path('printA.js');
 const fixtureB = fixtures.path('printB.js');
 const fixtureC = fixtures.path('printC.js');
 const fixtureD = fixtures.path('define-global.js');
+const fixtureE = fixtures.path('intrinsic-mutation.js');
+const fixtureF = fixtures.path('print-intrinsic-mutation-name.js');
+const fixtureG = fixtures.path('worker-from-argv.js');
 const fixtureThrows = fixtures.path('throws_error4.js');
 
 // Test preloading a single module works
@@ -62,6 +65,32 @@ childProcess.exec(
   }
 );
 
+// Test that preload can be used with --frozen-intrinsics
+childProcess.exec(
+  `"${nodeBinary}" --frozen-intrinsics ${
+    preloadOption([fixtureE])
+  } ${
+    fixtureF
+  }`,
+  function(err, stdout) {
+    assert.ifError(err);
+    assert.strictEqual(stdout, 'smoosh\n');
+  }
+);
+childProcess.exec(
+  `"${
+    nodeBinary
+  }" --frozen-intrinsics ${
+    preloadOption([fixtureE])
+  } ${
+    fixtureG
+  } ${fixtureF}`,
+  function(err, stdout) {
+    assert.ifError(err);
+    assert.strictEqual(stdout, 'smoosh\n');
+  }
+);
+
 // Test that preload can be used with stdin
 const stdinProc = childProcess.spawn(
   nodeBinary,