refactor(tsc): drop redundant timer globals from Deno libs

[bartlomieju]

lib.node is now in the default lib chain since #33823, so this removes Deno-side declarations for setTimeout, setInterval, clearTimeout, and clearInterval from lib.deno.shared_globals.d.ts. lib.node already provides compatible timer declarations, and Deno timers already return NodeJS.Timeout.

The scope is intentionally narrower than removing the global-partitioning machinery entirely. NODE_ONLY_GLOBALS is trimmed to the entries that still need segregation or context-specific behavior: console/Console, crypto, gc, localStorage, sessionStorage, RequestInit, and ResponseInit. The console declaration stays in Deno libs because the bundled lib.node can be skipped when a project has its own @types/node installed.

This also lets Buffer, setImmediate, clearImmediate, and queueMicrotask flow through the shared global table instead of being hidden behind NODE_ONLY_GLOBALS, matching globals that Deno exposes at runtime. Tests are updated for the newly visible globals, and timer tests that depended on the removed Deno-specific overloads or unsupported string callbacks are removed.

[fibibot]

Lib.node provides setTimeout/setInterval/clearTimeout/clearInterval with NodeJS.Timeout return type, so dropping the Deno copies in lib.deno.shared_globals.d.ts is a real dedup, not just a textual move — the lib.node overloads are now the single source. The corresponding trim of NODE_ONLY_GLOBALS in cli/tsc/mod.rs is consistent: console/Console/crypto/localStorage/sessionStorage/RequestInit/ResponseInit/gc stay partitioned (Deno still declares those with different shapes), while the four timers, Buffer*, setImmediate/clearImmediate/queueMicrotask, __dirname/__filename, ErrorConstructor, Global drop out and are picked up from lib.node directly.

The node_globals_check.out flip from a 2-error fail to a clean pass is the visible win — setImmediate/clearImmediate type-check under --unstable-node-globals now, and the stale "TODO: type checking doesn't work properly … wait till Deno 3" line goes with them. compare_globals/main.ts is updated accordingly to assert Buffer is on globalThis. CI is fully green across all targets including wpt release linux-x86_64.

• nit: dropping timeoutArgs from tests/unit/timers_test.ts loses coverage for the setTimeout(cb, ms, ...args) variadic form — @types/node's <TArgs extends any[]> overload supports it, so a one-line typed-callback version could keep the runtime assertion. The two Deno.test.ignore removals are pure cleanup, fine.

[lunadogbot]

NODE_ONLY_GLOBALS isn't actually emptied — console, Console, crypto, gc, localStorage, RequestInit, ResponseInit, sessionStorage remain. And the body says localStorage/sessionStorage move into TYPES_NODE_IGNORABLE_NAMES, but that list is byte-identical to main. Title says "drop redundant timer/console globals" but only the four timer declarations come out — declare var console: Console; at lib.deno.shared_globals.d.ts:490 is kept. Worth aligning the description with what actually ships.

One code question: why drop timeoutArgs in tests/unit/timers_test.ts? Node's timers do pass extra args through to the callback at runtime, so removing the test loses coverage without an obvious reason. Dropping the two .ignore'd string-callback tests alongside it is fine — they were already disabled.