deps: Add node-inspect 1.10.6

[hybrist]

This updates the bundled node-inspect to 1.10.6.

Highlights

• node --debug-port=1234 inspect respects the custom port.
• Test stability improvements on various platforms.

Compare: nodejs/node-inspect@v1.10.4...v1.10.6

Rendered Changelog

1.10.6

• chore: Fix usage text for embedded mode - @addaleax#20
  • b0779f5chore: Fix usage text for embedded mode
• print 'ok' after connection - @ofrobots#25
  • 2a47125fix: print 'ok' after connection
• Make autocompletion in REPL work - @aqrln#28
  • ccab737fix: Make autocompletion in REPL work
• Remove console.error() statement - @aqrln#30
  • 032b045style: Remove console.error() statement
• Take --debug-port into account - @jkrems#26
  • 054d4b1fix: Take --debug-port into account
• Delay run until breakpoints are restored - @jkrems#34
  • 802b88cfix: Delay run until breakpoints are restored
  • 2b93173fix: Use single string for paused notice
  • b4d5ee2fix: Work around inconsistent handling of strict directive
  • f6ccfc7fix: Only restart after port is free
  • 8b101bftest: Skip exact match on AIX
• a4e4b6fchore: Fix repo info in package.json

1.10.5

• docs: minor edits to governance docs - @joshgav#17
  • a70fe04docs: minor edits to governance docs

Checklist

• make test-node-inspect passes
• tests and/or benchmarks are included
• commit message follows commit guidelines

Affected core subsystem(s)

node-inspect

[hybrist]

Previous PR: #10187

Changeset generated via:

rm -rf deps/node-inspect node-inspect-*&& curl -sSL "https://github.com/nodejs/node-inspect/archive/v1.10.6.tar.gz"| tar -xzvf - && mv node-inspect-* deps/node-inspect

[hybrist]

One test fails because of https://github.com/nodejs/node/pull/6171/files#r106263519 - <enter> is swallowed inside of repl.

[hybrist]

This hack appear in multiple places in bootstrap. What I saw before adding it: The pingPort function calls net.connect and setTimeout. Unless the latter was removed or moved to the next tick, none of the connect events would ever fire.

[addaleax]

Rubber-stamp-ish LGTM (lib: might be a better subsystem for the second commit, though)

CI: https://ci.nodejs.org/job/node-test-commit/8488/

[hybrist]

s/deps:/lib: for that second commit message. Guess I went with deps: because I was still thinking about merging the two commits while writing the commit message. Thanks for clarifying! 👍

[hybrist]

Since make test-node-inspect doesn't pass on master right now, I backed that bullet point out of the issue description (working on making them pass again in parallel).

[Trott]

Rubber-stamp LGTM if CI is ✅

[jasnell]

Rubber stamp LGTM here as well. Tried it out and it appears to work as expected :-)

[hybrist]

I think my force-push to reword the commit is why those CI builds failed (Could not checkout 7f0e8337eaf1198a9f23a2de25d2b9418800f84e).

[jasnell]

New CI: https://ci.nodejs.org/job/node-test-pull-request/6901/

[hybrist]

🍏

[jasnell]

Landed in b57f9ee...7042631