debugger: reduce scope of eslint disable comment

Current code masks setInterval and setTimeout with promisified versions.
This can be confusing to read and causes lint errors. Replace masking
with use of pSetInterval and pSetTimeout instead.

Move disabling of lint rule from entire file to the one remaining line
(after the above changes) that still needs it.

PR-URL: #38946
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>

Author: Trott

lib/internal/inspector/_inspect.js

@@ -20,9 +20,6 @@
  * IN THE SOFTWARE.
  */
 
-// TODO(aduh95): remove restricted syntax errors
-/* eslint-disable no-restricted-syntax */
-
 'use strict';
 
 const {
@@ -53,8 +50,8 @@ const { EventEmitter } = require('events');
 const net = require('net');
 const util = require('util');
 const {
-  setInterval,
-  setTimeout,
+  setInterval: pSetInterval,
+  setTimeout: pSetTimeout,
 } = require('timers/promises');
 const {
   AbortController,
@@ -85,13 +82,13 @@ async function portIsFree(host, port, timeout = 9999) {
   const ac = new AbortController();
   const { signal } = ac;
 
-  setTimeout(timeout).then(() => ac.abort());
+  pSetTimeout(timeout).then(() => ac.abort());
 
-  const asyncIterator = setInterval(retryDelay);
+  const asyncIterator = pSetInterval(retryDelay);
   while (true) {
     await asyncIterator.next();
     if (signal.aborted) {
-      throw new StartupError(
+      throw new StartupError( // eslint-disable-line no-restricted-syntax
         `Timeout (${timeout}) waiting for ${host}:${port} to be free`);
     }
     const error = await new Promise((resolve) => {
@@ -251,7 +248,7 @@ class NodeInspector {
         return;
       } catch (error) {
         debuglog('connect failed', error);
-        await setTimeout(1000);
+        await pSetTimeout(1000);
       }
     }
     this.stdout.write(' failed to connect, please retry\n');