🐛 Bug: no-redundant-files conflicts with n/no-unpublished-bin

[cylewaitforit]

Bug Report Checklist

• I have tried restarting my IDE and the issue persists.
• I have searched for related issues and found none that matched my issue.

Expected

When upgrading to 0.56.0 in which no-redundant-files is a part of the recommended config the no-redundant-files rule flagged "bin/index.js", in the package.json as redundant. Removing it should result in lint passing.

Actual

After removing "bin/index.js", in the package.json files, the no-unpublished-bin rule from eslint-plugin-n throws

error  npm ignores 'bin/index.js'. Check 'files' field of 'package.json' or '.npmignore'  n/no-unpublished-bin

Version

0.56.0

ESLint Version

9.34.0

Additional Info

"eslint-plugin-n": "17.21.0"

The no-unpublished-bin documentation states:

If npm ignores the files in bin field, this rule warns the files.

• If files field does not includes the files in bin field.

I don't know which rule is correct. Filing the issue here first because enabling this rule is how the discrepancy was found.

You can recreate this in octoguide/bot#260.

[michaelfaith]

That seems like a bug in eslint-plugin-n, right? If you don't include the file referenced as the bin in you files array, it's still included when you run npm publish or npm pack. So, I'm not sure that's an issue with this rule, so much as it is an unnecessary thing to enforce on theirs.

[michaelfaith]

I see there's activity in the issue you updated for the other plugin. I'll go ahead and close this one on our side.