std/node: Node compatible setTimeout, setInterval

setTimeout in Node.js returns Timeout object

ref(Node.js):

• https://github.com/nodejs/node/blob/8ee4e672eca6826e7b6f2c1ad36ecbe6e570d412/lib/timers.js#L140
• https://github.com/nodejs/node/blob/ee9e2a2eb6143fa7b4b1454f7aed009a8703d4d7/lib/internal/timers.js#L167

We have Timeout class implemented in std/node, but doesn’t seem implementing setTimeout correctly

• Timeout https://github.com/denoland/deno_std/blob/ce0ac6ec4b489ec3eab7bbf4991efb8a0c9fe827/node/_timers.ts#L31
• setTimeout https://github.com/denoland/deno_std/blob/ce0ac6ec4b489ec3eab7bbf4991efb8a0c9fe827/node/timers.ts

[lucacasonato]

I’d say it is probably safe to override them in compat mode, if Symbo.toPrimitive is implemented on the timers object. Very ugly though (and very painful). See responses to https://twitter.com/lcasdev/status/1462766579196862470

[bnoordhuis]

Channeling @lucacasonato: idea is to monkey-patch setTimeout/setInterval globals and return objects instead of numbers in --compat mode.

(My recollection was to return objects always but I might be misremembering and I don’t have a strong preference anyway.)