After upgrade to version 3.x, json files are broken

[yp05327]

It works well in 2.4.4, but after I upgrade to the latest 3.x.x, it seems that it will lint all .json as js/ts file
In 2.4.4:

In 3.0.8 and 3.0.10:

[dbaeumer]

@yp05327 can you provide me with a GitHub repository I can clone that demos this. Problems like this are sensitive to versions (elint, plugins, ...) and I want to ensure we are using the same setup.

[dbaeumer]

I tried to reproduce this but failed. A GitHub repository to clone is highly appreciated.

[yp05327]

I'm using a payed Vue template, so I can't push all of them. I will try to provide a demo later.

[yp05327]

@dbaeumer
I have done. You can clone this repo:
https://github.com/yp05327/eslinterror

Reproduce:

• Open it in dev container
• Run pnpm install in terminal
• Reload window (press F1 and select Developer: Reload Window)

Then you can see this issue:

I notice that the following two things effecting this issue:

• You should run pnpm install, and reload the window
• You should not delete .eslintrc.cjs

I have tested it in my local PC in a clean space again, so I'm sure you can reproduce it.

I didn't modify .eslintrc.cjs, but I removed several custom commands in packages.json which will cause error during the installation as not all files are uploaded, and this is not related to this issue

[dbaeumer]

I was able to reproduce, however the same errors seem to be reported on the command line as well (see screen shot).

So there seems to be a more general problem with the setup since this happens independent of the ESLint extension.

[yp05327]

So this issue comes from eslint self but not the extension?
One thing I couldn't understand, why after I downgrade the extension, it worked.
And eslint's version is hard coding (in package.json), so eslint version and configs are all the same.
The only difference between these are the extension's version. This seems strange. 🤔

[dbaeumer]

It sounds strange. May be the extension doesn't work when downgraded :-). Does it work in general?

[yp05327]

These two conditions are required, or you will not see the errors.

[dbaeumer]

I followed these steps and still see the errors both in the editor AND when running eslint directly in the terminal. Since these errors are present in the terminal it is IMO correct that they are shown in the extension as well.

[yp05327]

If I downgrade the extension, you will see nothing happened, but eslint still reports errors.

Maybe it is better to report this issue to eslint not here

So I guess, maybe eslint extension will ignore these no-related errors in 2.x, but will not do that in 3.x.

[yp05327]

I notice that the missing rules valid-appcardcode-code-prop are user defined rules.
I uploaded missing files in yp05327/eslinterror@59b10bb

valid-appcardcode-code-prop is defined in eslint-internal-rules, and imported by .vscode/settings.json
so if you just run eslint in command line, it should display errors, as this rule is not imported.

So the correct command is ./node_modules/.bin/eslint --rulesdir eslint-internal-rules/ test.json :

There are still some errors but rule not found error disappeared. So I think, at least, the settings options eslint.options.rulePaths is broken in 3.x.

[yp05327]

I found the solution:
the lint command defined in packages.json is

"lint": "eslint . -c .eslintrc.cjs --fix --rulesdir eslint-internal-rules/ --ext .ts,.js,.cjs,.vue,.tsx,.jsx"

Then I added these options in .vscode/settings.json

And then there's no errors any more.

--fix is not related, but config is related I think.
So maybe the root reason is that .eslintrc.cjs can not be automatically loaded by the extension by default.

[yp05327]

I confirmed that this is partly caused by #1787
In this PR, json files are added. So in 2.x, json files will not be triggered, but in 3.x, it will.

And then because of the bad logic here:

If user didn't even enable the plugins in .eslintrc.cjs, extension will still validate these files.
In my case, this project don't use json lint, but it will be detected by the extension.

Maybe a fix can be checking whether user enabled the plugin in eslint config file, then mark settings.validate to on or off.