src: remove deprecated node debug command

The `node debug` command has been deprecated for a while now. There's
really no good reason to keep it around. Move to end of life.

PR-URL: #33648
Reviewed-By: Matteo Collina <[email protected]>
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Сковорода Никита Андреевич <[email protected]>
Reviewed-By: Shelley Vohr <[email protected]>

Author: jasnell

doc/api/deprecations.md

@@ -1435,12 +1435,15 @@ an officially supported API.
 ### DEP0068: `node debug`
 <!-- YAML
 changes:
+  - version: REPLACEME
+    pr-url: https://github.com/nodejs/node/pull/33648
+    description: The legacy `node debug` command was removed.
   - version: v8.0.0
     pr-url: https://github.com/nodejs/node/pull/11441
     description: Runtime deprecation.
 -->
 
-Type: Runtime
+Type: End-of-life
 
 `node debug` corresponds to the legacy CLI debugger which has been replaced with
 a V8-inspector based CLI debugger available through `node inspect`.

lib/internal/main/inspect.js

@@ -8,11 +8,6 @@ const {
 
 prepareMainThreadExecution();
 
-if (process.argv[1] === 'debug') {
-  process.emitWarning(
-    '`node debug` is deprecated. Please use `node inspect` instead.',
-    'DeprecationWarning', 'DEP0068');
-}
 
 markBootstrapComplete();
 

src/node.cc

@@ -477,7 +477,7 @@ MaybeLocal<Value> StartExecution(Environment* env, StartExecutionCallback cb) {
     first_argv = env->argv()[1];
   }
 
-  if (first_argv == "inspect" || first_argv == "debug") {
+  if (first_argv == "inspect") {
     return StartExecution(env, "internal/main/inspect");
   }
 

test/parallel/test-debug-usage.js

@@ -1,56 +1,28 @@
 'use strict';
 const common = require('../common');
 common.skipIfInspectorDisabled();
+
+if (common.isWindows)
+  common.skip('unsupported function on windows');
+
 const assert = require('assert');
 const spawn = require('child_process').spawn;
 
 let buffer = '';
 
 // Connect to debug agent
-const interfacer = spawn(process.execPath, ['debug', '-p', '655555']);
+const interfacer = spawn(process.execPath, ['inspect', '-p', '655555']);
 
 interfacer.stdout.setEncoding('utf-8');
 interfacer.stderr.setEncoding('utf-8');
 const onData = (data) => {
   data = (buffer + data).split('\n');
   buffer = data.pop();
-  data.forEach(function(line) {
-    interfacer.emit('line', line);
-  });
+  data.forEach((line) => interfacer.emit('line', line));
 };
 interfacer.stdout.on('data', onData);
 interfacer.stderr.on('data', onData);
 
-let lineCount = 0;
-interfacer.on('line', function(line) {
-  let expected;
-  const pid = interfacer.pid;
-  switch (++lineCount) {
-    case 1:
-      expected =
-        new RegExp(`^\\(node:${pid}\\) \\[DEP0068\\] DeprecationWarning: `);
-      assert.match(line, expected);
-      break;
-    case 2:
-      assert.match(line, /Use `node --trace-deprecation \.\.\.` to show where /);
-      break;
-    case 3:
-      // Doesn't currently work on Windows.
-      if (!common.isWindows) {
-        expected = "Target process: 655555 doesn't exist.";
-        assert.strictEqual(line, expected);
-      }
-      break;
-
-    default:
-      if (!common.isWindows)
-        assert.fail(`unexpected line received: ${line}`);
-  }
-});
-
-interfacer.on('exit', function(code, signal) {
-  assert.strictEqual(code, 1, `Got unexpected code: ${code}`);
-  if (!common.isWindows) {
-    assert.strictEqual(lineCount, 3);
-  }
-});
+interfacer.on('line', common.mustCall((line) => {
+  assert.strictEqual(line, 'Target process: 655555 doesn\'t exist.');
+}));