Relax node requirement to allow v20?

[Floriferous]

Hi there,

I just had my entire npm install fail because this package was imported by MCP inspector and it has a node v22 requirement.

Is it possible to allow node 20, as it's also still LTS?

Thanks!

[bendrucker]

I just had my entire npm install fail

That part is your choice:

https://docs.npmjs.com/cli/v8/using-npm/config#engine-strict

IIRC I just went with 22 to spare myself having to worry about v20 at all.

If you want to dig through and confirm Node 20 compatibility, I'll take a PR. Node 20.0 is not compatible since require can't load ESM modules without an experimental flag, but a later v20 does enable it.

[Floriferous]

Okay, then what about allowing the node 20.x version that does enable it?

The reason I created this issue is we have quite a lot of npm packages in our repo, (and we do indeed use engine-strict), and this is the only package out of all that prevents our node 20 setup to pass. Maybe the >22 requirement is a bit strict in comparison to the current ecosystem of npm packages.

[bendrucker]

Yep that's fine!

[MalekBouba]

@bendrucker also related/similar #6419