util: reduce inspection depth default to five

[BridgeAR]

To minimize potential problems with the higher util.inspect() depth default it is lowered to five as a intermediate step.

I added the don't-land-on labels instead of marking this as semver-major since this relies on a semver-major commit.

Refs: #22846 (comment)

Checklist

• make -j4 test (UNIX), or vcbuild test (Windows) passes
• tests and/or benchmarks are included
• documentation is changed or added
• commit message follows commit guidelines

[nodejs-github-bot]

@BridgeAR build started: https://ci.nodejs.org/blue/organizations/jenkins/node-test-pull-request-lite-pipeline/detail/node-test-pull-request-lite-pipeline/1177/pipeline

[BridgeAR]

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

[sam-github]

Maybe I'm confused about where this default is used, but since this is changing the default depth, doesn't this PR have to touch the docs where it describes the default depth as 20?

[BridgeAR]

@sam-github absolutely! I totally overlooked to change that again.

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

[addaleax]

@BridgeAR Looks like this needs a rebase?

[jasnell]

This will need to land by Saturday in order to make it in to 11.0.0

[devsnek]

can this please make it into a 11x release.

[Trott]

Since the change this builds on was reverted, should this be closed? Or rebased? Opened as a new PR?