Rule Change: no-restricted-imports should honor a package's declared exports about eslint

[fasttime]

No problem :) For usage questions around ESLint there is also a Discord chat, they are usually quick to respond. Good luck!

[fasttime]

Thanks for the issue @JeremyLoy. I don't think that ESLint can support this feature in a core rule because it's about a Node.js-specific API. Also note that the no-restricted-imports rule is designed to work without knowledge of the file system. It even works in a browser because it expects a list of import specifiers to be provided by the user, without checking if those imports correspond to existing or accessible modules.

That said, eslint-plugin-n has a rule called n/no-missing-import that should report a problem in this case (I tried it with your repro and it does, although there's another plugin in your config that doesn't recognize the subpath export and reports a problem).

[JeremyLoy]

Thanks for the response @fasttime, I appreciate it. I'll close this since it makes sense to not have Node only concepts in ESLint proper.

Also, thank you for pointing me towards eslint-plugin-n, I hadn't seen it before.

Is that the de-facto plugin for all things NodeJS specific nowadays? I see it referenced a few times in this repo next to deprecated rules

[JeremyLoy]

hey @fasttime - I updated my repo with eslint-plugin-n, but I'm still not seeing the lint error like you said.

Do you mind taking a look and seeing what I have misconfigured?

[fasttime]

Hm, your npm "lint" script in the package.json file looks incomplete if you are running npm run lint. It should be along the lines of "eslint --ext .mjs .".

In the eslintrc config system, no files are linted by default, and only .js files are linted if a directory is specified. To lint a directory of .mjs files, one would pass both a directory name (. for the current directory) and the file extension with --ext .mjs on the command line. When using the flat config system, which will become the default in ESLint 9, neither is required.

[JeremyLoy]

@fasttime thanks 🤦 Can't believe I forgot to do that. Its been a while since I set up a project from scratch.

My PoC is working now, correctly failing the impossible import. But now its also failing to find the local workspace module. Unless you think thats an issue with eslint specifically, I'll start a conversation over at eslint-plugin-n.

Thanks again for your help!