e.preventDefault() on unhandledrejection/error events doesn't work

[jespertheend]

When I saw that #16928 was fixed I wanted to try it out for my use case, but it seems like I can still reproduce it under certain conditions. Specifically when importing std/node/process.ts.

import "https://deno.land/[email protected]/node/process.ts";
import {delay} from "std/async/delay.ts";

/**
 * @typedef FailedTestsData
 * @property {string} path
 * @property {string[]} failedTestNames
 */

globalThis.addEventListener("unhandledrejection", e => {
	console.log("unhandled!");
	e.preventDefault();
});
globalThis.addEventListener("error", e => {
	console.log("caught!");
	e.preventDefault();
});

(async () => {
    throw 'failed';
})();

await delay(5_000);

console.log("done");

[jespertheend]

Ah hmm, seems like this issue is exactly what was fixed in #19235, I'm simply importing an old std version which still has this issue.
Unfortunately third party code is importing this version, so there's not a lot I can do :/

[bartlomieju]

@jespertheend yeah, there's not much we can do with old version of deno_std. The issue at hand is fixed when using node:process import, but I won't be able to fix it for old code.

[jespertheend]

Yeah makes sense, sorry I thought I closed this.
I have forked deno-puppeteer and manually replaced https://esm.sh/[email protected] with npm:[email protected].
Seems to work so far.