fix(node): remove deprecation warnings

[bartlomieju]

Closes #22116

[mmastrac]

LGTM

[iuioiua]

Suggested change

	const DENO_RID_SYMBOL = Symbol.for("Deno.internal.rid");
	const DENO_RID_SYMBOL = SymbolFor("Deno.internal.rid");

Nit: primordials.

[irbull]

I don't get it, although maybe take my opinion lightly.

1. If rid is really depreciated and going away, doesn't this "fix" really just hide it now? If / when rid is removed, this will now just fail at runtime.
2. There were several deprecations listed in 1.40. This fix only seems to address the one (rid). Maybe this is the only one that the node compat layer uses.

[bartlomieju]

I don't get it, although maybe take my opinion lightly.

1. If rid is really depreciated and going away, doesn't this "fix" really just hide it now? If / when rid is removed, this will now just fail at runtime.

Resource IDs are here to stay for some foreseeable future, but we want to make them an internal notion, so that we can migrate APIs to other mechanisms (like GCable objects like in #21999). This is a stop-gap solution to stop the warnings. I'm working on a better solution that will use a unique symbol to store resource IDs (this symbol will not be reachable by user code so you won't have a way to access the value of the field).

2. There were several deprecations listed in 1.40. This fix only seems to address the one (rid). Maybe this is the only one that the node compat layer uses.

These appear to be the only ones that we missed :)