refactor: move resume on start logic to repl

Author: Denis Semenenko

lib/_inspect.js

@@ -170,19 +170,7 @@ class NodeInspector {
     this.domainNames.forEach((domain) => {
       this[domain] = createAgentProxy(domain, this.client);
     });
-    let startBreakReached = false;
     this.handleDebugEvent = (fullName, params) => {
-      if (!startBreakReached &&
-          process.env.NODE_INSPECT_RESUME_ON_START === '1' &&
-          fullName === 'Debugger.paused' &&
-          params && params.reason === 'Break on start') {
-        startBreakReached = true;
-        debuglog('Paused on start, but NODE_INSPECT_RESUME_ON_START' +
-                 ' environment variable is set to 1, resuming');
-        this.client.callMethod('Debugger.resume');
-        return;
-      }
-
       const [domain, name] = fullName.split('.');
       if (domain in this) {
         this[domain].emit(name, params);

lib/internal/inspect_repl.js

@@ -782,6 +782,14 @@ function createRepl(inspector) {
   }
 
   Debugger.on('paused', ({ callFrames, reason /* , hitBreakpoints */ }) => {
+    if (process.env.NODE_INSPECT_RESUME_ON_START === '1' &&
+        reason === 'Break on start') {
+      debuglog('Paused on start, but NODE_INSPECT_RESUME_ON_START' +
+              ' environment variable is set to 1, resuming');
+      inspector.client.callMethod('Debugger.resume');
+      return;
+    }
+
     // Save execution context's data
     currentBacktrace = Backtrace.from(callFrames);
     selectedFrame = currentBacktrace[0];