Review requested:

• @nodejs/loaders

implementing the ESM resolution algorithm from scratch.

We've been meaning to expose this logic in a helper function. Is that something you'd be interested in implementing?

@GeoffreyBooth, I don't know if I'd have the time to maintain a pure reference implementation, but my implementation here might be a good starting point for someone. Aside from using paths instead of URLs, it follows the documented algorithms as closely as possible.

Force pushed another typo I forgot about. PACKAGE_TARGET_RESOLVE 2.1 should say target instead of exports.

Section 3.3 of the same function also has confusing wording (I had to check the node.js code to implement it properly). I need to consider how to reword it though.

I just meant exposing ours, like import { defaultResolve } from 'node:module'.

- Loading data for nodejs/node/pull/50898
✔  Done loading data for nodejs/node/pull/50898
----------------------------------- PR info ------------------------------------
Title      doc: fix some errors in esm resolution algorithms (#50898)
Author     Christopher Jeffrey (JJ)  (@chjj)
Branch     chjj:fix-esm-docs -> nodejs:main
Labels     doc, esm, author ready
Commits    1
 - doc: fix some errors in esm resolution algorithms
Committers 1
 - Christopher Jeffrey 
PR-URL: https://github.com/nodejs/node/pull/50898
Reviewed-By: Guy Bedford 
Reviewed-By: Geoffrey Booth 
Reviewed-By: Luigi Pinca 
Reviewed-By: Antoine du Hamel 
------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/50898
Reviewed-By: Guy Bedford 
Reviewed-By: Geoffrey Booth 
Reviewed-By: Luigi Pinca 
Reviewed-By: Antoine du Hamel 
--------------------------------------------------------------------------------
   ⚠  Commits were pushed since the last approving review:
   ⚠  - doc: fix some errors in esm resolution algorithms
   ℹ  This PR was created on Fri, 24 Nov 2023 17:13:06 GMT
   ✔  Approvals: 4
   ✔  - Guy Bedford (@guybedford): https://github.com/nodejs/node/pull/50898#pullrequestreview-1748266109
   ✔  - Geoffrey Booth (@GeoffreyBooth) (TSC): https://github.com/nodejs/node/pull/50898#pullrequestreview-1748269208
   ✔  - Luigi Pinca (@lpinca): https://github.com/nodejs/node/pull/50898#pullrequestreview-1748269330
   ✔  - Antoine du Hamel (@aduh95) (TSC): https://github.com/nodejs/node/pull/50898#pullrequestreview-1748269950
   ✔  Last GitHub CI successful
   ℹ  Green GitHub CI is sufficient
--------------------------------------------------------------------------------
   ✔  Aborted `git node land` session in /home/runner/work/node/node/.ncu

@GeoffreyBooth, isn't this already going to be more-or-less exposed once import.meta.resolve accepts a parentURL?

I suppose CJS modules could access it if it were part of node:module?

Landed in 59f57d2