Geoffrey Booth

Adding don't land label for now because I think the other loaders PR should be ready soon and it would be best to release them together.

> So excited for this. Any indication of whether this will make it into LTS / 18.x? Yes, it should.

@nodejs/releasers per https://github.com/nodejs/loaders/issues/103#issuecomment-1397323186 can this please go out in the next release? Thanks. Also this change is arguably notable, so I put a short release note in the [top post](https://github.com/nodejs/node/pull/43772#issue-1300576475)....

And/or this one can go on its own. There haven’t been too many ESM loader PRs lately, other than some test improvements. This was @arcanis’s PR, @arcanis do you mind...

The workflow that ran for this PR passed with a retry for v18 Windows, but didn’t run for 21 (?). When I run it manually I get failures: https://github.com/nodejs/citgm/actions/runs/6843202973/job/18605533429. Are...

I’m not very knowledgeable of how CITGM works. Perhaps @targos knows?

> @targos Interesting. Then perhaps `citgm` should utilize corepack. Or at least manually read the [`packageManger`](https://nodejs.org/api/packages.html#packagemanager) field in package.json to select the correct version of the package manger. Why does...

Okay, do you want to update CITGM accordingly?

They're just dependencies of CITGM, so they'd get installed via that.

This feels like the wrong place for this. I would think these keywords should be defined near where we define things like `for` and `throw`.