util: fix inspect indentation of detached DataView

[Kjubikstronk]

util.inspect() of a DataView whose buffer was detached comes out indented too far, and so does everything after it in the same output:

const util = require('node:util');
const ab = new ArrayBuffer(4);
const dv = new DataView(ab);
structuredClone(ab, { transfer: [ab] });
console.log(util.inspect(dv));

DataView {
      [byteLength]: 0,
      [byteOffset]: undefined,
      [buffer]: ArrayBuffer { (detached), [byteLength]: 0 }
    }

The cause was formatExtraProperties(): it raised ctx.indentationLvl before reading value[key]. On a detached DataView the byteLength and byteOffset getters throw, so the -= 2 never runs and 2 spaces leak per key, 4 in total.

The bug came in with #60131, and the test added there asserted the leaked output.

Fix: read the value before indenting.

I built Node on Windows and tested it: the changed test fails without the fix and passes with it, as do 428 related tests.

I used Claude Code to find the bug and write the fix. I reviewed the diff and the changed assertion myself.

Refs: #60131

[github-actions]

Welcome to Node.js, and thank you for your first contribution!

Before review, please take a moment to read:

• the guide for first-time contributors
• the contribution and automation policies
• the pull request guide
• the AI use policy
• the Code of Conduct

Please make sure every commit is signed off. For a first pull request, GitHub Actions require collaborator approval and Jenkins CI must be started by a collaborator or triager, so an initial wait is normal.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 90.37%. Comparing base (d7ea02d) to head (c90d878).
⚠️ Report is 12 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main   #66421   +/-   ##
=======================================
  Coverage   90.37%   90.37%           
=======================================
  Files         792      792           
  Lines      275683   275686    +3     
  Branches    52854    52854           
=======================================
+ Hits       249147   249159   +12     
+ Misses      16947    16935   -12     
- Partials     9589     9592    +3     

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

... and 21 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/78126/

[nodejs-github-bot]

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

[MikeMcC399]

Only failing in smartos, and with different flaky tests:

https://ci.nodejs.org/job/node-test-pull-request/78126/
https://ci.nodejs.org/job/node-test-commit-smartos/68875/
failure in: test-debugger-probe-startup-disconnect

https://ci.nodejs.org/job/node-test-pull-request/78131/
https://ci.nodejs.org/job/node-test-commit-smartos/68883/
failure in: test-worker-init-failure
Also reported in https://github.com/nodejs/reliability/blob/main/reports/2026-10-02.md

[nodejs-github-bot]

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