util: inspect objects with throwing Symbol.toStringTag

[BridgeAR]

util.inspect() should handle all kinds of input, even if it is not well defined. Throwing is something that is meant to be worked around in all known cases. This fixes issues inspecting objects where accessing the Symbol.toStringTag would cause an error. The symbol is just ignored in that case.

Refs: #55539
Refs: #55544

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 88.28%. Comparing base (f1b56d6) to head (6ab2ff6).
⚠️ Report is 183 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main   #59860      +/-   ##
==========================================
+ Coverage   88.27%   88.28%   +0.01%     
==========================================
  Files         702      702              
  Lines      206706   206691      -15     
  Branches    39757    39761       +4     
==========================================
+ Hits       182469   182487      +18     
+ Misses      16231    16225       -6     
+ Partials     8006     7979      -27     

Files with missing lines	Coverage Δ
lib/internal/util/inspect.js	99.93% <100.00%> (+<0.01%)	⬆️

... and 42 files with indirect coverage changes

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[nodejs-github-bot]

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

[nodejs-github-bot]

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

[nodejs-github-bot]

Landed in 60c1fbc

[aduh95]

Test failure on v22.x-staging when trying to include this change:

Path: parallel/test-util-inspect
node:assert:152
  throw new AssertionError(obj);
  ^

AssertionError [ERR_ASSERTION]: Expected values to be strictly equal:
+ actual - expected

+ '<ref *1> {\n' +
+   '  y: { x: [Circular *1], [Symbol(Symbol.toStringTag)]: [Getter] }\n' +
+   '}'
- '<ref *1> {\n  y: { x: [Circular *1], Symbol(Symbol.toStringTag): [Getter] }\n}'

    at Object.<anonymous> (…/test/parallel/test-util-inspect.js:1680:10)
    at Module._compile (node:internal/modules/cjs/loader:1706:14)
    at Object..js (node:internal/modules/cjs/loader:1839:10)
    at Module.load (node:internal/modules/cjs/loader:1441:32)
    at Function._load (node:internal/modules/cjs/loader:1263:12)
    at TracingChannel.traceSync (node:diagnostics_channel:328:14)
    at wrapModuleLoad (node:internal/modules/cjs/loader:237:24)
    at Function.executeUserEntryPoint [as runMain] (node:internal/modules/run_main:171:5)
    at node:internal/main/run_main_module:36:49 {
  generatedMessage: true,
  code: 'ERR_ASSERTION',
  actual: '<ref *1> {\n' +
    '  y: { x: [Circular *1], [Symbol(Symbol.toStringTag)]: [Getter] }\n' +
    '}',
  expected: '<ref *1> {\n' +
    '  y: { x: [Circular *1], Symbol(Symbol.toStringTag): [Getter] }\n' +
    '}',
  operator: 'strictEqual',
  diff: 'simple'
}

Node.js v22.20.1-pre
Command: out/Release/node --expose-internals --test-reporter=./test/common/test-error-reporter.js --test-reporter-destination=stdout …/test/parallel/test-util-inspect.js

We would need a manual backport if we want to include it there.