nodeVersion: false doesn't disable the related rules

[XhmikosR]

C:\Users\xmr\Desktop\foo>node -v && npm -v && ver
v14.17.5
6.14.14
Microsoft Windows [Version 10.0.19043.1165]

C:\Users\xmr\Desktop\foo>npm ls xo
foo@1.0.0 C:\Users\xmr\Desktop\foo
`-- xo@0.44.0

I was trying to disable the following two rules:

• "node/no-unsupported-features/es-builtins"
• "node/no-unsupported-features/es-syntax"

but it doesn't have any effect. So then I saw that there's the nodeVersion option that could control this. I tried setting it to false but it doesn't work either.

A quick patch would be to change

xo/lib/options-manager.js

Line 205 in bc6e05d

...(enginesOptions && enginesOptions.node && semver.validRange(enginesOptions.node) ? {nodeVersion: enginesOptions.node} : {}),

-		...(enginesOptions && enginesOptions.node && semver.validRange(enginesOptions.node) ? {nodeVersion: enginesOptions.node} : {}),
+		...(xoOptions.nodeVersion !== false && enginesOptions && enginesOptions.node && semver.validRange(enginesOptions.node) ? {nodeVersion: enginesOptions.node} : {}),

But shouldn't it be possible to override the related rules?

[sindresorhus]

A quick patch would be to change

Pull request welcome.

[XhmikosR]

I can make a PR but I'm still not sure if it's the proper solution since IMHO it should be possible to override those rules only without overriding nodeVersion.

[sindresorhus]

But shouldn't it be possible to override the related rules?

Ideally, yes, but it's complicated. Rule overrides can come from your package.json, but it can also come from a shareable config, and we override before all the config is resolved, so we cannot reliably check whether it's already set.

xo/lib/options-manager.js

Lines 309 to 313 in bc6e05d

	if (options.nodeVersion) {
	config.baseConfig.rules['node/no-unsupported-features/es-builtins'] = ['error', {version: options.nodeVersion}];
	config.baseConfig.rules['node/no-unsupported-features/es-syntax'] = ['error', {version: options.nodeVersion, ignores: ['modules']}];
	config.baseConfig.rules['node/no-unsupported-features/node-builtins'] = ['error', {version: options.nodeVersion}];
	}

[fregante]

It looks like this was not fixed in #761, "nodeVersion": false, won't disable n rules

source/github-events/on-react-page-update.tsx:8:25
  ✖    8:25  The AbortSignal.any is still an experimental feature and is not supported until Node.js 20.3.0 (backported: ^18.17.0). The configured version range is >= 20.  n/no-unsupported-features/node-builtins

It did fix #613 though.

cc @melusc

[fregante]

Pretty sure nodeVersion: false is being overridden here:

xo/lib/options-manager.js

Line 120 in b537b21

...(enginesOptions && enginesOptions.node && semver.validRange(enginesOptions.node) ? {nodeVersion: enginesOptions.node} : {}),

because xoOptions.nodeVersion is false but options.nodeVersion is still set to what's in the package.json after this line, so that's what's received by the rest of the code

[fregante]

Confirmed to be fixed in v0.59.2

• Update XO to v0.59.2 refined-github/refined-github#7588