lib: add legacy built-in functions to primordials

PR-URL: #42049 Reviewed-By: Darshan Sen <raisinten@gmail.com> Reviewed-By: Mestery <mestery@protonmail.com>
nodejs · Apr 24, 2022 · 91a06bd · 91a06bd
1 parent 11f747b
commit 91a06bd
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 0 deletions.
diff --git a/lib/.eslintrc.yaml b/lib/.eslintrc.yaml
@@ -134,6 +134,8 @@ rules:
     - name: decodeURIComponent
     - name: encodeURI
     - name: encodeURIComponent
+    - name: escape
+    - name: eval
     - name: Error
       ignore:
         - prepareStackTrace
@@ -177,6 +179,7 @@ rules:
     - name: Uint32Array
     - name: Uint8Array
     - name: Uint8ClampedArray
+    - name: unescape
     - name: URIError
     - name: WeakMap
       into: Safe

diff --git a/lib/internal/freeze_intrinsics.js b/lib/internal/freeze_intrinsics.js
@@ -121,7 +121,9 @@ const {
   decodeURIComponent,
   encodeURI,
   encodeURIComponent,
+  escape,
   globalThis,
+  unescape,
 } = primordials;
 
 const {
@@ -232,6 +234,7 @@ module.exports = function() {
 
     // 19 The Global Object
     // 19.2 Function Properties of the Global Object
+    // eslint-disable-next-line node-core/prefer-primordials
     eval,
     // eslint-disable-next-line node-core/prefer-primordials
     isFinite,

diff --git a/lib/internal/per_context/primordials.js b/lib/internal/per_context/primordials.js
@@ -154,6 +154,15 @@ function copyPrototype(src, dest, prefix) {
   primordials[fn.name] = fn;
 });
 
+// Create copies of legacy functions
+[
+  escape,
+  eval,
+  unescape,
+].forEach((fn) => {
+  primordials[fn.name] = fn;
+});
+
 // Create copies of the namespace objects
 [
   'JSON',