solidjs-community / eslint-plugin-solid

Solid-specific linting rules for ESLint.
MIT License
206 stars 24 forks source link

`solid/reactivity` does not track imported signals #127

Open CreativeTechGuy opened 5 months ago

CreativeTechGuy commented 5 months ago

Describe the bug

I am making SolidJS libraries for my organization. Some of the exports from this library are Signals. These signals aren't caught in the ESLint rule when using them improperly.

Note, this also applies to any other imported signal, even from within the same package.

To Reproduce

Expected behavior

In a perfect world, the ESLint rule would follow imports to find out which objects are signals. This could also possibly be accomplished via TypeScript type checking for Accessor types. But more realistically, it'd be great if the rule could be configured where we can manually list names of methods which should be treated as signals always.

Screenshots

Environment (please complete the following information):

Additional context

N/A