mirror of
https://github.com/storybookjs/storybook.git
synced 2025-02-06 03:33:16 +08:00
251 lines
7.6 KiB
JavaScript
251 lines
7.6 KiB
JavaScript
const path = require('path');
|
|
const fs = require('fs');
|
|
|
|
const scriptPath = path.join(__dirname, '..', 'scripts');
|
|
|
|
const addonsPackages = fs
|
|
.readdirSync(path.join(__dirname, 'addons'))
|
|
.filter((p) => fs.statSync(path.join(__dirname, 'addons', p)).isDirectory());
|
|
const libPackages = fs
|
|
.readdirSync(path.join(__dirname, 'lib'))
|
|
.filter((p) => fs.statSync(path.join(__dirname, 'lib', p)).isDirectory());
|
|
const uiPackages = fs
|
|
.readdirSync(path.join(__dirname, 'ui'))
|
|
.filter((p) => fs.statSync(path.join(__dirname, 'ui', p)).isDirectory())
|
|
.filter((p) => !p.startsWith('.'));
|
|
|
|
module.exports = {
|
|
root: true,
|
|
extends: [path.join(scriptPath, '.eslintrc.cjs')],
|
|
parserOptions: {
|
|
tsconfigRootDir: __dirname,
|
|
project: ['./tsconfig.json'],
|
|
},
|
|
plugins: ['local-rules'],
|
|
rules: {
|
|
'eslint-comments/disable-enable-pair': ['error', { allowWholeFile: true }],
|
|
'eslint-comments/no-unused-disable': 'error',
|
|
'react-hooks/rules-of-hooks': 'off',
|
|
'import/extensions': 'off', // for mjs, we sometimes need extensions
|
|
'jest/no-done-callback': 'off',
|
|
'@typescript-eslint/dot-notation': [
|
|
'error',
|
|
{
|
|
allowIndexSignaturePropertyAccess: true,
|
|
},
|
|
],
|
|
'@typescript-eslint/no-restricted-imports': [
|
|
'error',
|
|
{
|
|
paths: [
|
|
{
|
|
name: 'vite',
|
|
message: 'Please dynamically import from vite instead, to force the use of ESM',
|
|
allowTypeImports: true,
|
|
},
|
|
],
|
|
},
|
|
],
|
|
},
|
|
overrides: [
|
|
{
|
|
// this package depends on a lot of peerDependencies we don't want to specify, because npm would install them
|
|
files: ['**/frameworks/angular/template/**/*'],
|
|
rules: {
|
|
'@typescript-eslint/no-useless-constructor': 'off',
|
|
'@typescript-eslint/dot-notation': 'off',
|
|
},
|
|
},
|
|
{
|
|
// this package depends on a lot of peerDependencies we don't want to specify, because npm would install them
|
|
files: ['**/addons/docs/**/*'],
|
|
rules: {
|
|
'import/no-extraneous-dependencies': 'off',
|
|
},
|
|
},
|
|
{
|
|
files: [
|
|
'*.js',
|
|
'*.jsx',
|
|
'*.json',
|
|
'*.html',
|
|
'**/.storybook/*.ts',
|
|
'**/.storybook/*.tsx',
|
|
'setup-jest.ts',
|
|
],
|
|
parserOptions: {
|
|
project: null,
|
|
},
|
|
rules: {
|
|
'@typescript-eslint/dot-notation': 'off',
|
|
'@typescript-eslint/no-implied-eval': 'off',
|
|
'@typescript-eslint/no-throw-literal': 'off',
|
|
'@typescript-eslint/return-await': 'off',
|
|
},
|
|
},
|
|
{
|
|
// this package depends on a lot of peerDependencies we don't want to specify, because npm would install them
|
|
files: ['**/*.ts', '**/*.tsx'],
|
|
rules: {
|
|
'no-shadow': 'off',
|
|
'@typescript-eslint/ban-types': 'warn', // should become error, in the future
|
|
},
|
|
},
|
|
{
|
|
// this package depends on a lot of peerDependencies we don't want to specify, because npm would install them
|
|
files: ['**/builder-vite/**/*.html'],
|
|
rules: {
|
|
'@typescript-eslint/no-unused-expressions': 'off', // should become error, in the future
|
|
},
|
|
},
|
|
{
|
|
// these packages use pre-bundling, dependencies will be bundled, and will be in devDepenencies
|
|
files: ['frameworks/**/*', 'builders/**/*', 'deprecated/**/*', 'renderers/**/*'],
|
|
excludedFiles: ['frameworks/angular/**/*', 'frameworks/ember/**/*', 'lib/core-server/**/*'],
|
|
rules: {
|
|
'import/no-extraneous-dependencies': [
|
|
'error',
|
|
{ bundledDependencies: false, devDependencies: true },
|
|
],
|
|
},
|
|
},
|
|
{
|
|
files: ['**/ui/.storybook/**'],
|
|
rules: {
|
|
'import/no-extraneous-dependencies': [
|
|
'error',
|
|
{ packageDir: [__dirname], devDependencies: true },
|
|
],
|
|
},
|
|
},
|
|
...addonsPackages.map((directory) => ({
|
|
files: [path.join('**', 'addons', directory, '**', '*.*')],
|
|
rules: {
|
|
'import/no-extraneous-dependencies': [
|
|
'error',
|
|
{
|
|
packageDir: [__dirname, path.join(__dirname, 'addons', directory)],
|
|
devDependencies: true,
|
|
},
|
|
],
|
|
},
|
|
})),
|
|
...uiPackages.map((directory) => ({
|
|
files: [path.join('**', 'ui', directory, '**', '*.*')],
|
|
rules: {
|
|
'import/no-extraneous-dependencies': [
|
|
'error',
|
|
{ packageDir: [__dirname, path.join(__dirname, 'ui', directory)], devDependencies: true },
|
|
],
|
|
},
|
|
})),
|
|
...libPackages.map((directory) => ({
|
|
files: [path.join('**', 'lib', directory, '**', '*.*')],
|
|
rules: {
|
|
'import/no-extraneous-dependencies': [
|
|
'error',
|
|
{
|
|
packageDir: [__dirname, path.join(__dirname, 'lib', directory)],
|
|
devDependencies: true,
|
|
},
|
|
],
|
|
},
|
|
})),
|
|
{
|
|
files: [
|
|
'**/__tests__/**',
|
|
'**/__testfixtures__/**',
|
|
'**/*.test.*',
|
|
'**/*.stories.*',
|
|
'**/storyshots-*/**/stories/**',
|
|
],
|
|
rules: {
|
|
'@typescript-eslint/no-empty-function': 'off',
|
|
'import/no-extraneous-dependencies': 'off',
|
|
},
|
|
},
|
|
{
|
|
files: ['**/__testfixtures__/**'],
|
|
rules: {
|
|
'react/forbid-prop-types': 'off',
|
|
'react/no-unused-prop-types': 'off',
|
|
'react/require-default-props': 'off',
|
|
},
|
|
},
|
|
{
|
|
files: ['**/*.stories.*'],
|
|
rules: {
|
|
'no-console': 'off',
|
|
},
|
|
},
|
|
{
|
|
files: ['**/renderers/preact/**/*'],
|
|
rules: {
|
|
'react/react-in-jsx-scope': 'off',
|
|
'react/prop-types': 'off',
|
|
},
|
|
},
|
|
{
|
|
files: ['**/*.tsx', '**/*.ts'],
|
|
rules: {
|
|
'no-shadow': 'off',
|
|
'@typescript-eslint/ban-types': 'warn', // should become error, in the future
|
|
'react/require-default-props': 'off',
|
|
'react/prop-types': 'off', // we should use types
|
|
'react/forbid-prop-types': 'off', // we should use types
|
|
'no-dupe-class-members': 'off', // this is called overloads in typescript
|
|
'react/no-unused-prop-types': 'off', // we should use types
|
|
'react/default-props-match-prop-types': 'off', // we should use types
|
|
'import/no-named-as-default': 'warn',
|
|
'import/no-named-as-default-member': 'warn',
|
|
'react/destructuring-assignment': 'warn',
|
|
|
|
// This warns about importing interfaces and types in a normal import, it's arguably better to import with the `type` prefix separate from the runtime imports,
|
|
// I leave this as a warning right now because we haven't really decided yet, and the codebase is riddled with errors if I set to 'error'.
|
|
// It IS set to 'error' for JS files.
|
|
'import/named': 'warn',
|
|
},
|
|
},
|
|
{
|
|
files: ['**/*.d.ts'],
|
|
rules: {
|
|
'vars-on-top': 'off',
|
|
'no-var': 'off', // this is how typescript works
|
|
'spaced-comment': 'off',
|
|
},
|
|
},
|
|
{
|
|
files: ['**/e2e-tests/**/*'],
|
|
rules: {
|
|
'jest/no-test-callback': 'off', // These aren't jest tests
|
|
},
|
|
},
|
|
{
|
|
files: ['**/builder-vite/input/iframe.html'],
|
|
rules: {
|
|
'no-undef': 'off', // ignore "window" undef errors
|
|
},
|
|
},
|
|
{
|
|
// Because those templates reference css files in other directory.
|
|
files: ['**/template/cli/**/*'],
|
|
rules: {
|
|
'import/no-unresolved': 'off',
|
|
},
|
|
},
|
|
{
|
|
files: ['**/*.ts', '!**/*.test.*', '!**/*.spec.*'],
|
|
rules: {
|
|
'local-rules/no-uncategorized-errors': 'warn',
|
|
},
|
|
},
|
|
{
|
|
files: ['**/core-events/src/**/*'],
|
|
excludedFiles: ['**/*.test.*'],
|
|
rules: {
|
|
'local-rules/no-duplicated-error-codes': 'error',
|
|
},
|
|
},
|
|
],
|
|
};
|