resolve 'delete' edge case

nodejs · May 13, 2024 · 0c46beb · 0c46beb
1 parent e186d94
commit 0c46beb
Show file tree

Hide file tree

Showing 2 changed files with 20 additions and 6 deletions.
diff --git a/lib/internal/repl/experimental/index.js b/lib/internal/repl/experimental/index.js
@@ -36,6 +36,7 @@ const {
   isStaticIdentifier,
   isScopeOpen,
   stripVTControlCharacters: stripAnsi,
+  acornParseOptions,
 } = require('internal/repl/experimental/util');
 const getHistory = require('internal/repl/experimental/history');
 const {
@@ -192,7 +193,7 @@ class ExperimentalREPLServer extends EventEmitter {
     }
 
     let { node: expression } = walk.findNodeAround(
-      acornParse(line, { ecmaVersion: 2021 }),
+      acornParse(line, acornParseOptions),
       line.length,
       (type) => type === 'MemberExpression' || type === 'Identifier',
     );
@@ -278,7 +279,7 @@ class ExperimentalREPLServer extends EventEmitter {
   preview(line) {
     try {
       const { node: expression } = walk.findNodeAround(
-        acornParse(line, { ecmaVersion: 2021 }),
+        acornParse(line, acornParseOptions),
         line.length,
         (type) => type === 'MemberExpression' || type === 'Identifier',
       );

diff --git a/lib/internal/repl/experimental/util.js b/lib/internal/repl/experimental/util.js
@@ -18,14 +18,23 @@ const {
 } = require('internal/deps/acorn/acorn/dist/acorn');
 const walk = require('internal/deps/acorn/acorn-walk/dist/walk');
 
+const acornParseOptions = {
+  __proto__: null,
+  ecmaVersion: 2021,
+  allowReserved: true,
+  allowReturnOutsideFunction: true,
+  allowImportExportEverywhere: true,
+  allowAwaitOutsideFunction: true,
+  allowSuperOutsideMethod: true,
+  allowHashBang: true,
+  checkPrivateFields: false,
+};
+
 function isStaticIdentifier(exp) {
   let isStatic = true;
 
   try {
-    walk.simple(acornParse(exp, {
-      ecmaVersion: 2021,
-      allowImportExportEverywhere: true,
-    }), {
+    walk.simple(acornParse(exp, acornParseOptions), {
       VariableDeclaration() {
         isStatic = false;
       },
@@ -71,6 +80,9 @@ function isStaticIdentifier(exp) {
       ExportAllDeclaration() {
         isStatic = false;
       },
+      UnaryExpression(node) {
+        isStatic = node.operator !== 'delete';
+      }
     });
   } catch {
     // We ignore the error because we don't care about invalid syntax.
@@ -299,4 +311,5 @@ module.exports = {
   stripVTControlCharacters,
   isStaticIdentifier,
   isScopeOpen,
+  acornParseOptions,
 };