repl: fix top level await with surrogate characters

Fixes: #39929
nodejs · Aug 29, 2021 · 458a9fd · 458a9fd
1 parent 449147e
commit 458a9fd
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 1 deletion.
diff --git a/lib/internal/repl/await.js b/lib/internal/repl/await.js
@@ -155,7 +155,7 @@ for (const nodeType of ObjectKeys(walk.base)) {
 function processTopLevelAwait(src) {
   const wrapPrefix = '(async () => { ';
   const wrapped = `${wrapPrefix}${src} })()`;
-  const wrappedArray = ArrayFrom(wrapped);
+  const wrappedArray = StringPrototypeSplit(wrapped, '');
   let root;
   try {
     root = parser.parse(wrapped, { ecmaVersion: 'latest' });

diff --git a/test/parallel/test-repl-preprocess-top-level-await.js b/test/parallel/test-repl-preprocess-top-level-await.js
@@ -9,17 +9,29 @@ const { processTopLevelAwait } = require('internal/repl/await');
 // This test was created based on
 // https://cs.chromium.org/chromium/src/third_party/WebKit/LayoutTests/http/tests/inspector-unit/preprocess-top-level-awaits.js?rcl=358caaba5e763e71c4abb9ada2d9cd8b1188cac9
 
+const surrogate = '"\u{1F601}\u{1f468}\u200d\u{1f469}\u200d\u{1f467}\u200d\u{1f466}"';
+
 const testCases = [
   [ '0',
     null ],
   [ 'await 0',
     '(async () => { return (await 0) })()' ],
+  [ `await ${surrogate}`,
+    `(async () => { return (await ${surrogate}) })()` ],
   [ 'await 0;',
     '(async () => { return (await 0); })()' ],
+  [ `await ${surrogate};`,
+    `(async () => { return (await ${surrogate}); })()` ],
+  [ `await ${surrogate};`,
+    `(async () => { return (await ${surrogate}); })()` ],
   [ '(await 0)',
     '(async () => { return ((await 0)) })()' ],
+  [ `(await ${surrogate})`,
+    `(async () => { return ((await ${surrogate})) })()` ],
   [ '(await 0);',
     '(async () => { return ((await 0)); })()' ],
+  [ `(await ${surrogate});`,
+    `(async () => { return ((await ${surrogate})); })()` ],
   [ 'async function foo() { await 0; }',
     null ],
   [ 'async () => await 0',
@@ -28,8 +40,12 @@ const testCases = [
     null ],
   [ 'await 0; return 0;',
     null ],
+  [ `await ${surrogate}; await ${surrogate};`,
+    `(async () => { await ${surrogate}; return (await ${surrogate}); })()` ],
   [ 'var a = await 1',
     'var a; (async () => { void (a = await 1) })()' ],
+  [ `var a = await ${surrogate}`,
+    `var a; (async () => { void (a = await ${surrogate}) })()` ],
   [ 'let a = await 1',
     'let a; (async () => { void (a = await 1) })()' ],
   [ 'const a = await 1',