[WIP] chore(eslint): prefer node protocol

[virtuoushub]

currently blocked

closing as blocked by downstream dependency on jest. see also: jestjs/jest#14297

once jest supports this, or redwood uses a different test framework that supports the node protocol we can revisit.

source

---

When importing builtin modules, it's better to use the node: protocol as it makes it perfectly clear that the package is a Node.js builtin module.

https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/prefer-node-protocol.md

Preliminary work to aid in ESM migration.

[virtuoushub]

Failing due to jestjs/jest#14040

[thedavidprice]

@virtuoushub I just read through jestjs/jest#14332 Doesn't look like this is going to come through anytime soon. Any thoughts about initiating a Jest patch following advice in this comment? If it works, you could open a PR back into Jest.

[virtuoushub]

Failing due to jestjs/jest#14040

unblocked this via yarn patching in jest draft PR: jestjs/jest#14332

this approach is brittle, and ideally jest fixes it, but in the meantime this should work. It does mean anytime jest is updated we need to propagate the patch forward until jest fixes it in upstream. didn't work.

[virtuoushub]

closing as blocked by downstream dependency on jest. see also: jestjs/jest#14297

once jest supports this, or redwood uses a different test framework that supports the node protocol we can revisit.