debugger: surface inspector failures in probe mode

[joyeecheung]

This addresses a few TODOs left from the initial implementation around the error handling when the inspector session fails mid-probe - for example because a probe expression terminates the target, severs the inspector connection, or hangs the evaluation. Before this patch, these failures would cause the probe report to silently miss hits or hit the session timeout. A consumer of these reports could not tell whether the recorded hits were reliable or a partial trace cut short by an inspector-side failure.

This patch surfaces those cases as a new probe_failure terminal error event on the report, with best-effort attribution to the implicated probe via error.probe and additional context on error.details. Together with probe_timeout, it marks the report as best-effort, and the probing process exits 1 in both cases. probe_target_exit and clean completed/miss reports continue to exit 0 because the recorded hits in those cases remain ground truth, so tooling can use the exit code as a quick trustworthiness signal.

The per-hit error is reshaped to { message, details? } so that the per-hit and terminal errors share a top-level shape. Both fields are informative-only, only their top-level type is stable.

Refs: #62713

[codecov]

Codecov Report

❌ Patch coverage is 69.80519% with 93 lines in your changes missing coverage. Please review.
✅ Project coverage is 90.03%. Comparing base (b5659d1) to head (6f5c4a8).
⚠️ Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
lib/internal/debugger/inspect_probe.js	69.80%	91 Missing and 2 partials ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main   #63437      +/-   ##
==========================================
- Coverage   90.06%   90.03%   -0.03%     
==========================================
  Files         714      714              
  Lines      225918   226098     +180     
  Branches    42729    42796      +67     
==========================================
+ Hits       203466   203573     +107     
- Misses      14231    14320      +89     
+ Partials     8221     8205      -16     

Files with missing lines	Coverage Δ
lib/internal/debugger/inspect_probe.js	79.89% <69.80%> (+1.17%)	⬆️

... and 48 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/73591/

[nodejs-github-bot]

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

[nodejs-github-bot]

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

[nodejs-github-bot]

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