Solid ESLint Plugin

This package contains Solid-specific linting rules for ESLint. It can ease Solid's learning curve by finding and fixing problems around Solid's reactivity system, and can migrate some React patterns to Solid code.

It's approaching a 1.0.0 release, and it's well tested and should be helpful in Solid projects today.

Installation

Install eslint and eslint-plugin-solid locally.

npm install --save-dev eslint eslint-plugin-solid
# or
pnpm add --save-dev eslint eslint-plugin-solid
yarn add --dev eslint eslint-plugin-solid

# optional, to create an ESLint config file
npx eslint --init
# or
pnpm eslint --init
yarn eslint --init

If you're using VSCode, you'll want to install the ESLint extension. You're encouraged to enable auto-fixing problems on save by adding the following to your settings.json file.

{
  "editor.codeActionsOnSave": {
    "source.fixAll": true
  }
}

If you're using Vite, you may want to install vite-plugin-eslint.

You may also want to check out eslint-plugin-jsx-a11y, which provides useful rules for writing accessible HTML.

Configuration

Use the "plugin:solid/recommended" configuration to get reasonable defaults as shown below.

{
  "plugins": ["solid"],
  "extends": ["eslint:recommended", "plugin:solid/recommended"]
}

TypeScript

If you're using TypeScript, use the "plugin:solid/typescript" configuration instead. This disables some features that overlap with type checking.

{
  "parser": "@typescript-eslint/parser",
  "plugins": ["solid"],
  "extends": ["eslint:recommended", "plugin:solid/typescript"]
}

Manual Configuration

If you don't want to use a preset, you can configure rules individually. Add the "solid" plugin, enable JSX with the parser options (or use the equivalent options for @typescript-eslint/parser or @babel/eslint-parser), and configure the rules you would like to use. Some rules have additional options you can set.

{
  "plugins": ["solid"],
  "parserOptions": {
    "ecmaFeatures": {
      "jsx": true
    }
  },
  "rules": {
    "solid/reactivity": "warn",
    "solid/no-destructure": "warn",
    "solid/jsx-no-undef": "error"
  }
}

Disabling Rules

The rules in this plugin provide sensible guidelines, but there may be occasions where you know what you're doing and want to ignore a warning. Just add a comment like the following:

// eslint-disable-next-line solid/reactivity
const [editedValue, setEditedValue] = createSignal(props.value);

Rules

✔: Enabled in the recommended configuration.

🔧: Fixable with eslint --fix/IDE auto-fix.

✔	🔧	Rule	Description
✔	🔧	solid/components-return-once	Disallow early returns in components. Solid components only run once, and so conditionals should be inside JSX.
✔	🔧	solid/event-handlers	Enforce naming DOM element event handlers consistently and prevent Solid's analysis from misunderstanding whether a prop should be an event handler.
✔	🔧	solid/imports	Enforce consistent imports from "solid-js", "solid-js/web", and "solid-js/store".
✔		solid/jsx-no-duplicate-props	Disallow passing the same prop twice in JSX.
✔		solid/jsx-no-script-url	Disallow javascript: URLs.
✔	🔧	solid/jsx-no-undef	Disallow references to undefined variables in JSX. Handles custom directives.
✔		solid/jsx-uses-vars	Prevent variables used in JSX from being marked as unused.
✔	🔧	solid/no-destructure	Disallow destructuring props. In Solid, props must be used with property accesses (`props.foo`) to preserve reactivity. This rule only tracks destructuring in the parameter list.
✔	🔧	solid/no-innerhtml	Disallow usage of the innerHTML attribute, which can often lead to security vulnerabilities.
✔	🔧	solid/no-react-specific-props	Disallow usage of React-specific `className`/`htmlFor` props, which were deprecated in v1.4.0.
✔		solid/no-unknown-namespaces	Enforce using only Solid-specific namespaced attribute names (i.e. `'on:'` in `<div on:click={...} />`).
✔	🔧	solid/prefer-classlist	Enforce using the classlist prop over importing a classnames helper. The classlist prop accepts an object `{ [class: string]: boolean }` just like classnames.
✔	🔧	solid/prefer-for	Enforce using Solid's `<For />` component for mapping an array to JSX elements.
✔	🔧	solid/prefer-show	Enforce using Solid's `<Show />` component for conditionally showing content. Solid's compiler covers this case, so it's a stylistic rule only.
✔		solid/reactivity	Enforce that reactive expressions (props, signals, memos, etc.) are only used in tracked scopes; otherwise, they won't update the view as expected.
✔	🔧	solid/self-closing-comp	Disallow extra closing tags for components without children.
✔	🔧	solid/style-prop	Require CSS properties in the `style` prop to be valid and kebab-cased (ex. 'font-size'), not camel-cased (ex. 'fontSize') like in React, and that property values with dimensions are strings, not numbers with implicit 'px' units.

Versioning

Pre-1.0.0, the rules and the recommended and typescript configuations will be stable across patch (0.0.x) versions, but may change across minor (0.x) versions. If you want to pin a minor version, use a tilde in your package.json.

- "eslint-plugin-solid": "^0.7.4"
+ "eslint-plugin-solid": "~0.7.4"

ajstrand/eslint-plugin-solid