node:diagnostics_channel: forward all runStores callback arguments

[andrewtdiz]

Summary

Channel#runStores(context, fn[, thisArg[, ...args]]) should forward every argument after thisArg to fn. Perry currently wires the native method through a fixed five-argument entrypoint and only preserves two callback arguments, so additional arguments are dropped.

Node Behavior

Probe run with Node v25.9.0:

const { channel } = require("node:diagnostics_channel");
const { AsyncLocalStorage } = require("node:async_hooks");
const ch = channel("runstores-args");
const als = new AsyncLocalStorage();
ch.bindStore(als);
const ret = ch.runStores({ value: 1 }, function (...args) {
  console.log(JSON.stringify(args), this && this.tag, JSON.stringify(als.getStore()));
  return "ret";
}, { tag: "ctx" }, "a", "b", "c");
console.log(ret);

Node prints the complete callback argument list:

["a","b","c"] ctx {"value":1}
ret

Perry Behavior

• docs/runtime-parity.md documents the Node signature as channel.runStores(context, fn[, thisArg[, ...args]]).
• crates/perry-runtime/src/node_submodules/diagnostics.rs attaches runStores with method_closure(cast5(diag_channel_run_stores), 5, id).
• diag_channel_run_stores accepts only data, fn_value, this_arg, a, and b, then calls run_store_wrapped(..., &[a, b]) when no stores are bound.
• The bound-store path allocates store_next_thunk with captures for only a and b, and store_next_thunk also calls run_store_wrapped(..., &[a, b]).
• The granular fixture test-parity/node-suite/diagnostics_channel/stores/bind-run-unbind.ts only exercises two forwarded arguments.

Suggested PR Cut

Batch this with related issues in:
node:diagnostics_channel: tracing and store semantics parity cut (#3242)

Good batch candidates:

• node:diagnostics_channel: implement Channel.withStoreScope disposable scopes #2632
• node:diagnostics_channel: preserve non-callable bindStore transform errors #3085
• node:diagnostics_channel: honor traceCallback position and arguments #3086

Do not batch with:

• node:trace_events category tracing
• broad node:async_hooks redesign outside store binding needs
• worker/thread diagnostics subscriber policy changes

Acceptance

• parity/regression test proves runStores() forwards at least three callback arguments with a bound store
• parity/regression test proves the same rest arguments are preserved when no stores are bound
• thisArg binding and AsyncLocalStorage store behavior continue matching Node
• related known-failure/docs/manifest entries updated if touched

[andrewtdiz]

Taking this in a diagnostics_channel tracing/store semantics PR cut with #3085, #3086, and #3084.

[andrewtdiz]

PR opened: #3503