GeezerVison

Review requested:

• @nodejs/loaders

See also: #53822

@RafaelGSS I disagree, I think it makes sense to land this to fix the behavior on release lines (also TSC consensus is not exactly relevant, it's up to Node.js collaborators, not the TSC, unless collaborators can't find consensus)

I think it makes sense to land this to fix the behavior on release lines

I think in that case it should just be targeting release lines, instead of main?

Also, it seems the diff here isn't working, there are multiple relevant test failures, maybe there's some git error (OP mentioned a test case in test/parallel/test-esm-data-url-import.js, but it's not showing up in the diff), because the diff here clearly doesn't work (making use of an undefined variable).

I tried checking out this branch and fixing the tests, unfortunately it's apparent that --experimental-network-imports is really broken, and since we're removing it, it seems not worth spending time fixing it.

Thanks for sending this PR, and sorry it won't work out this time!
Closing since the PR removing the feature from main has now landed.