Ignore specific warnings of svelte-check

[ptrxyz]

Description

For eslint-plugin-svelte3 there is svelte3/compiler-options or svelte3/ignore-warnings to ignore some specified compiler warnings, for example all the a11y warnings.

For this plugin I can't seem to find something similar. I can disable the valid-compile rule all together, yet that's a bit unspecific. I could try to disable the warning in my svelte.config.js by defining a config.onwarn function but this does not seem to work with vite/kit.

I suggest to provide a way to ignore specific warnings of the svelte compiler, as I think it might be quite handy.

[ota-meshi]

Thank you for posting issue.

How about using <!-- svelte-ignore a11y-autofocus --> comments?
https://svelte.dev/docs#template-syntax-comments
I think it works for the valid-compile rule as well.

The svelte compiler doesn't seem to have an option to ignore specific warnings, so I don't think there's anything we can do with rule.
https://svelte.dev/docs#compile-time-svelte-compile
If you want to add an option to a rule, first ask svelte's compiler to add the ignore option.

[ptrxyz]

Yep, the comments work.
I don't know if that's helpful but svelte-check allows to pass a list of warnings to be ignored. Maybe that's just an output filter then?

[baseballyama]

vite-Plugin-svelte has such a configuration.
Is it enough for you?

https://github.com/sveltejs/vite-plugin-svelte/blob/main/docs/config.md#onwarn

[ptrxyz]

No, I would like to silence the linter and it doesn't seem to care :D

[ota-meshi]

Hmm... svelte-check seems to filter on its own.

https://github.com/dummdidumm/language-tools/blob/6bd57b018c7ae37b9bbdb4d7a9393be115c96c4f/packages/language-server/src/plugins/svelte/features/getDiagnostics.ts#L48

I found a related issue, but I'm not sure why decided to add the option in the language tools instead of adding it to svelte's compiler 🤔.
sveltejs/language-tools#473

[ptrxyz]

I would like to follow up on this real quick. I have no new insight or anything, but to move things forward, could we expose this array to the rule's settings? Just like the language tools have an ignore list to configure, maybe we can have one too until the Svelte compiler offers a solution?

https://github.com/ota-meshi/eslint-plugin-svelte/blob/78c1fbc7a379e8f709660c0cb5072b845c161a5d/src/rules/valid-compile.ts#L30

[ota-meshi]

When does the Svelte compiler offer that solution? I think you should ask the Svelte compiler to add an option to ignore warnings first. If there's a reason the Svelte team doesn't provide an ignore option, I don't think we should add it either.

[ptrxyz]

Svelte offers command line flags to disable certain warnings for the svelte-check tool, like so:

svelte-check --tsconfig ./tsconfig.json --compiler-warnings 'a11y-aria-attributes:ignore,a11y-click-events-have-key-events:ignore'

I think this is the closes the user usually gets in touch with the compiler, so I would argue that they already provide a way to the user to ignore specific warnings in their tooling. However I do not know if they also do through the compiler API.

Further, I understand your point, but I was hoping we could still have this. It could be through some flag that makes sure it's simply filtering out the error codes and the errors are still there but hidden. It could be something like internal.ignore_errors = ['a11y-aria-attributes', 'a11y-click-events-have-key-events']. This would transport that it's only used internally and not by the compiler.

[ota-meshi]

I looked into svelte's issues.

It seems to me that it is up to the compiler API caller to decide how to handle the warning.
sveltejs/svelte#2040
sveltejs/svelte#2093

So if the user wants to suppress warnings globally, I think it is intended to be controlled by the compiler API caller.

Also, since most bundler svelte plugins seem to be designed to globally suppress warnings with the onwarn option, it would be good to provide a similar option if possible. However, I have no good ideas. (Current) the eslint config file may not be .js.

[wtachau]

I am also interested in this. See: sveltejs/language-tools#650

[baseballyama]

How about this?

sveltejs/svelte#8558 (comment)

[ota-meshi]

ESLint configuration files can also be defined in JSON or YAML, so I still have no idea how to define isIgnore (or onwarn) nicely.

[ota-meshi]

By the way, JSON and YAML configuration files may be deprecated after ESLint v9. But for now we support ESLint>=v7.