@wistia/eslint-config 0.16.1 → 0.17.0

package/configs/eslint/storybook.js

@@ -0,0 +1,9 @@
+require('@rushstack/eslint-patch/modern-module-resolution');
+
+module.exports = {
+  env: {
+    node: true,
+  },
+
+  extends: ['../../rules/eslint/storybook'].map(require.resolve),
+};

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@wistia/eslint-config",
-  "version": "0.16.1",
+  "version": "0.17.0",
   "description": "Wistia's ESLint configurations",
   "main": "react.js",
   "exports": {
@@ -11,6 +11,7 @@
     "./node": "./configs/eslint/node.js",
     "./prettier": "./configs/eslint/prettier.js",
     "./react": "./configs/eslint/react.js",
+    "./storybook": "./configs/eslint/storybook.js",
     "./strict": "./configs/eslint/strict.js",
     "./styled-components": "./configs/eslint/styled-components.js",
     "./stylelint": "./configs/stylelint/default.js",
@@ -53,6 +54,7 @@
     "eslint-plugin-react": "^7.31.10",
     "eslint-plugin-react-hooks": "^4.6.0",
     "eslint-plugin-sonarjs": "^0.16.0",
+    "eslint-plugin-storybook": "^0.6.7",
     "eslint-plugin-styled-components-a11y": "^1.0.0",
     "eslint-plugin-testing-library": "^5.9.1",
     "postcss": "^8.4.18",

package/rules/eslint/storybook.js

@@ -0,0 +1,66 @@
+// only add storybook rules
+// see: https://github.com/storybookjs/eslint-plugin-storybook#supported-rules-and-configurations
+
+module.exports = {
+  plugins: ['eslint-plugin-storybook'],
+
+  rules: {
+    // storybook 6 requires default exports in stories
+    'import/no-anonymous-default-export': 'off',
+    'import/no-default-export': 'off',
+    'import/no-extraneous-dependencies': ['error', { devDependencies: true }],
+    'import/prefer-default-export': 'error',
+
+    // Interactions should be awaited
+    // https://github.com/storybookjs/eslint-plugin-storybook/blob/main/docs/rules/storybook/await-interactions.md
+    'storybook/await-interactions': 'error',
+
+    // Pass a context when invoking play function of another story
+    // https://github.com/storybookjs/eslint-plugin-storybook/blob/main/docs/rules/storybook/context-in-play-function.md
+    'storybook/context-in-play-function': 'error',
+
+    // The component property should be set
+    // https://github.com/storybookjs/eslint-plugin-storybook/blob/main/docs/rules/storybook/csf-component.md
+    'storybook/csf-component': 'error',
+
+    // Story files should have a default export
+    // https://github.com/storybookjs/eslint-plugin-storybook/blob/main/docs/rules/storybook/default-exports.md
+    'storybook/default-exports': 'error',
+
+    // Deprecated hierarchy separator in title property
+    // https://github.com/storybookjs/eslint-plugin-storybook/blob/main/docs/rules/storybook/hierarchy-separator.md
+    'storybook/hierarchy-separator': 'error',
+
+    // A story should not have a redundant name property
+    // https://github.com/storybookjs/eslint-plugin-storybook/blob/main/docs/rules/storybook/no-redundant-story-name.md
+    'storybook/no-redundant-story-name': 'error',
+
+    // storiesOf is deprecated and should not be used
+    // https://github.com/storybookjs/eslint-plugin-storybook/blob/main/docs/rules/storybook/no-stories-of.md
+    'storybook/no-stories-of': 'error',
+
+    // Do not define a title in meta
+    // https://github.com/storybookjs/eslint-plugin-storybook/blob/main/docs/rules/storybook/no-title-property-in-meta.md
+    'storybook/no-title-property-in-meta': 'error',
+
+    // This rule identifies storybook addons that are invalid because they are either not installed or contain a typo in their name
+    // https://github.com/storybookjs/eslint-plugin-storybook/blob/main/docs/rules/storybook/no-uninstalled-addons.md
+    'storybook/no-uninstalled-addons': 'error',
+
+    // Stories should use PascalCase
+    // https://github.com/storybookjs/eslint-plugin-storybook/blob/main/docs/rules/storybook/prefer-pascal-case.md
+    'storybook/prefer-pascal-case': 'error',
+
+    // A story file must contain at least one story export
+    // https://github.com/storybookjs/eslint-plugin-storybook/blob/main/docs/rules/storybook/story-exports.md
+    'storybook/story-exports': 'error',
+
+    // Use expect from @storybook/jest
+    // https://github.com/storybookjs/eslint-plugin-storybook/blob/main/docs/rules/storybook/use-storybook-expect.md
+    'storybook/use-storybook-expect': 'error',
+
+    // Do not use testing-library directly on stories
+    // https://github.com/storybookjs/eslint-plugin-storybook/blob/main/docs/rules/storybook/use-storybook-testing-library.md
+    'storybook/use-storybook-testing-library': 'error',
+  },
+};

package/rules/eslint/typescript-imports.js

@@ -22,6 +22,8 @@ module.exports = {
 
   settings: {
     'import/parsers': {
+      // omit `.d.ts` because 1) TypeScript compilation already confirms that types are resolved, and
+      // 2) it would mask an unresolved `.ts`/`.tsx`/`.js`/`.jsx` implementation.
       '@typescript-eslint/parser': ['.ts', '.tsx'],
     },
 

package/rules/eslint/typescript.js

@@ -307,7 +307,7 @@ module.exports = {
     // Require function parameters to be typed as <code>readonly</code> to prevent accidental mutation of inputs
     // https://typescript-eslint.io/rules/prefer-readonly-parameter-types
     // disabled due to development friction and lackluster linting hints to resolve violations
-    '@typescript-eslint/prefer-readonly-parameter-types': 'error',
+    '@typescript-eslint/prefer-readonly-parameter-types': 'off',
 
     // Enforce using type parameter when calling <code>Array#reduce</code> instead of casting
     // https://typescript-eslint.io/rules/prefer-reduce-type-parameter