diagnostics_channel: replace using with try-finally

[ayush23chaudhary]

Fixes: #64230

Description

This PR replaces instances of the experimental using syntax with try...finally blocks inside lib/diagnostics_channel.js.

Since Explicit Resource Management is still a flagged feature in V8, encountering using in core causes a segfault when running Node with the --no-js-explicit-resource-management flag. This refactor maintains the exact same resource cleanup semantics by manually calling [SymbolDispose]() in a finally block, avoiding the flagged syntax entirely.

Example of Refactor

Before:

using scope = this.withStoreScope(data);
return ReflectApply(fn, thisArg, args);

After

const scope = this.withStoreScope(data);
try {
  return ReflectApply(fn, thisArg, args);
} finally {
  scope[SymbolDispose]();
}

[Copilot]

Pull request overview

This PR removes the experimental using syntax from lib/diagnostics_channel.js, replacing it with equivalent try...finally disposal to avoid crashes when Node is run with --no-js-explicit-resource-management.

Changes:

• Replaced using statements with explicit try...finally blocks that invoke [SymbolDispose]().
• Updated scope handling in ActiveChannel, BoundedChannel, and TracingChannel paths to ensure disposal occurs reliably without relying on flagged syntax.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[anonrig]

Please add at least a test

[ayush23chaudhary]

Thanks for the thorough review, @Renegade334 and @anonrig!

1. I've updated RunStoresScope to perfectly match the suggested error handling and disposal order (NodeAggregateError).

2. I've added a test in test/parallel/test-diagnostics-channel-no-erm.js to ensure Node does not segfault when run with --no-js-explicit-resource-management.

3. I squashed everything into a single commit to resolve the commit linter failure.

[Renegade334]

Thanks for your changes! Some more comments 👍

[ayush23chaudhary]

@Renegade334!

1. I completely flattened the nested try...finally blocks in traceSync, tracePromise, and traceCallback.

2. I removed the empty try...finally blocks around the continuation window disposals.

3. I updated the test script to use the // Flags: directive, completely removing the spawnSync boilerplate

4. As noted, I also reverted RunStoresScope back to the simpler reverse-loop without the synchronous error logic

[Renegade334]

LGTM, but please run git commit --amend --no-edit --signoff to indicate acceptance of the Developer's Certificate of Origin, and make lint-js-fix to satisfy the linter.

[ayush23chaudhary]

Thanks for the LGTM!

I ran make lint-js-fix to resolve the formatting, and squashed/amended with --signoff to accept the DCO. Everything should be green and good to go

[Qard]

One minor nit but otherwise LGTM.

[nodejs-github-bot]

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