debugger: wait for V8 debugger to be enabled

targos · danielleadams · commit ee1056da6022 · 2021-05-31T15:34:57.000-04:00
Refs: #38273 (comment) PR-URL: #38811 Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By: Richard Lau <rlau@redhat.com> Reviewed-By: Beth Griggs <bgriggs@redhat.com> Reviewed-By: Gerhard Stöbich <deb2001-github@yahoo.de> Reviewed-By: Rich Trott <rtrott@gmail.com>
diff --git a/lib/internal/inspector/_inspect.js b/lib/internal/inspector/_inspect.js
@@ -199,8 +199,8 @@ class NodeInspector {
     process.once('SIGTERM', exitCodeZero);
     process.once('SIGHUP', exitCodeZero);
 
-    PromisePrototypeCatch(PromisePrototypeThen(this.run(), () => {
-      const repl = startRepl();
+    PromisePrototypeCatch(PromisePrototypeThen(this.run(), async () => {
+      const repl = await startRepl();
       this.repl = repl;
       this.repl.on('exit', exitCodeZero);
       this.paused = false;
diff --git a/lib/internal/inspector/inspect_repl.js b/lib/internal/inspector/inspect_repl.js
@@ -1146,14 +1146,17 @@ function createRepl(inspector) {
     return Runtime.runIfWaitingForDebugger();
   }
 
-  return function startRepl() {
+  return async function startRepl() {
     inspector.client.on('close', () => {
       resetOnStart();
     });
     inspector.client.on('ready', () => {
       initAfterStart();
     });
 
+    // Init once for the initial connection
+    await initAfterStart();
+
     const replOptions = {
       prompt: 'debug> ',
       input: inspector.stdin,
@@ -1172,9 +1175,6 @@ function createRepl(inspector) {
       repl.emit('SIGINT');
     });
 
-    // Init once for the initial connection
-    initAfterStart();
-
     return repl;
   };
 }
