@coko/lint 3.0.0-alpha.3 → 3.0.0-alpha.30

package/src/eslint.bak.js

@@ -1,106 +1,106 @@
-module.exports = {
-  parser: '@babel/eslint-parser',
-  parserOptions: {
-    requireConfigFile: false,
-    babelOptions: {
-      presets: ['@babel/preset-react'],
-    },
-  },
-  overrides: [
-    {
-      files: ['*.jsx'],
-    },
-  ],
-  env: {
-    browser: true,
-    es6: true,
-    jest: true,
-    node: true,
-  },
-  extends: [
-    'airbnb',
-    'standard',
-    'standard-react',
-    'plugin:jest/recommended',
-    'plugin:cypress/recommended',
-    'prettier',
-  ],
-  plugins: ['jest'],
-  ignorePatterns: [
-    '_build',
-    'dist',
-    '!.storybook',
-    '!.commitlintrc.js',
-    '!.cz-config.js',
-    '!.eslintrc.js',
-    '!.jest.config.js',
-    '!.lintstagedrc.js',
-    '!.prettierrc.js',
-    '!.stylelintrc.js',
-    '**/node_modules',
-  ],
-  rules: {
-    'arrow-body-style': 0,
-    'import/no-extraneous-dependencies': [
-      'error',
-      {
-        devDependencies: [
-          // storybook
-          '.storybook/*',
-          '**/stories/**/*.js',
+// module.exports = {
+//   parser: '@babel/eslint-parser',
+//   parserOptions: {
+//     requireConfigFile: false,
+//     babelOptions: {
+//       presets: ['@babel/preset-react'],
+//     },
+//   },
+//   overrides: [
+//     {
+//       files: ['*.jsx'],
+//     },
+//   ],
+//   env: {
+//     browser: true,
+//     es6: true,
+//     jest: true,
+//     node: true,
+//   },
+//   extends: [
+//     'airbnb',
+//     'standard',
+//     'standard-react',
+//     'plugin:jest/recommended',
+//     'plugin:cypress/recommended',
+//     'prettier',
+//   ],
+//   plugins: ['jest'],
+//   ignorePatterns: [
+//     '_build',
+//     'dist',
+//     '!.storybook',
+//     '!.commitlintrc.js',
+//     '!.cz-config.js',
+//     '!.eslintrc.js',
+//     '!.jest.config.js',
+//     '!.lintstagedrc.js',
+//     '!.prettierrc.js',
+//     '!.stylelintrc.js',
+//     '**/node_modules',
+//   ],
+//   rules: {
+//     'arrow-body-style': 0,
+//     'import/no-extraneous-dependencies': [
+//       'error',
+//       {
+//         devDependencies: [
+//           // storybook
+//           '.storybook/*',
+//           '**/stories/**/*.js',
 
-          // tests
-          'cypress/**',
-          '**/*.spec.js',
-          '**/*.test.js',
+//           // tests
+//           'cypress/**',
+//           '**/*.spec.js',
+//           '**/*.test.js',
 
-          // webpack
-          'webpack/**',
+//           // webpack
+//           'webpack/**',
 
-          // configs
-          '.commitlintrc.js',
-          '.cz-config.js',
-          '.eslintrc.js',
-          '.jest.config.js',
-          '.lintstagedrc.js',
-          '.prettierrc.js',
-          '.stylelintrc.js',
+//           // configs
+//           '.commitlintrc.js',
+//           '.cz-config.js',
+//           '.eslintrc.js',
+//           '.jest.config.js',
+//           '.lintstagedrc.js',
+//           '.prettierrc.js',
+//           '.stylelintrc.js',
 
-          // other
-          'dev/**',
-        ],
-      },
-    ],
-    'jsx-a11y/anchor-is-valid': [
-      'error',
-      {
-        components: ['Link'],
-        specialLink: ['to', 'hrefLeft', 'hrefRight'],
-        aspects: ['noHref', 'invalidHref', 'preferButton'],
-      },
-    ],
-    'no-console': ['error', { allow: ['warn', 'error'] }],
-    'padding-line-between-statements': [
-      'error',
-      { blankLine: 'always', prev: '*', next: 'block' },
-      { blankLine: 'always', prev: 'block', next: '*' },
-      { blankLine: 'always', prev: '*', next: 'block-like' },
-      { blankLine: 'always', prev: 'block-like', next: '*' },
-      { blankLine: 'always', prev: '*', next: 'multiline-const' },
-      { blankLine: 'always', prev: 'multiline-const', next: '*' },
-    ],
-    // 'prettier/prettier': 'error',
-    'react/function-component-definition': [
-      2,
-      { namedComponents: 'arrow-function' },
-    ],
-    'react/jsx-filename-extension': [2, { extensions: ['.js'] }], // disallows .jsx files
-    'react/jsx-props-no-spreading': 0,
-    'react/jsx-sort-props': [1, { ignoreCase: true }],
-    'react/prop-types': [
-      2,
-      { ignore: ['children', 'className', 'onClick', 'theme'] },
-    ],
-    // 'sort-keys': [1, 'asc', { caseSensitive: false }],
-  },
-}
+//           // other
+//           'dev/**',
+//         ],
+//       },
+//     ],
+//     'jsx-a11y/anchor-is-valid': [
+//       'error',
+//       {
+//         components: ['Link'],
+//         specialLink: ['to', 'hrefLeft', 'hrefRight'],
+//         aspects: ['noHref', 'invalidHref', 'preferButton'],
+//       },
+//     ],
+//     'no-console': ['error', { allow: ['warn', 'error'] }],
+//     'padding-line-between-statements': [
+//       'error',
+//       { blankLine: 'always', prev: '*', next: 'block' },
+//       { blankLine: 'always', prev: 'block', next: '*' },
+//       { blankLine: 'always', prev: '*', next: 'block-like' },
+//       { blankLine: 'always', prev: 'block-like', next: '*' },
+//       { blankLine: 'always', prev: '*', next: 'multiline-const' },
+//       { blankLine: 'always', prev: 'multiline-const', next: '*' },
+//     ],
+//     // 'prettier/prettier': 'error',
+//     'react/function-component-definition': [
+//       2,
+//       { namedComponents: 'arrow-function' },
+//     ],
+//     'react/jsx-filename-extension': [2, { extensions: ['.js'] }], // disallows .jsx files
+//     'react/jsx-props-no-spreading': 0,
+//     'react/jsx-sort-props': [1, { ignoreCase: true }],
+//     'react/prop-types': [
+//       2,
+//       { ignore: ['children', 'className', 'onClick', 'theme'] },
+//     ],
+//     // 'sort-keys': [1, 'asc', { caseSensitive: false }],
+//   },
+// }

package/src/eslint.mjs

@@ -1,49 +1,17 @@
-// import path from 'path'
-
 import { defineConfig, globalIgnores } from 'eslint/config'
 import js from '@eslint/js'
-// import { FlatCompat } from '@eslint/eslintrc'
-
 import globals from 'globals'
 import tseslint from 'typescript-eslint'
-// import babelParser from '@babel/eslint-parser'
-
-import nodePlugin from 'eslint-plugin-n'
 import importPlugin from 'eslint-plugin-import'
-// import jest from 'eslint-plugin-jest'
-// import cypressPlugin from 'eslint-plugin-cypress'
-
-// const __filename = fileURLToPath(import.meta.url)
-// const __dirname = path.dirname(__filename)
-
-// const compat = new FlatCompat({
-//   // baseDirectory: __dirname,
-//   resolvePluginsRelativeTo: import.meta.url,
-//   // recommendedConfig: js.configs.recommended,
-//   // allConfig: js.configs.all,
-// })
-
-/**
- * TO DO
- *
- * explicit rules from old config
- * jest
- * cypress
- *
- * => client
- * no console
- * jsx a11y
- * airbnb react
- * react recommended
- * react hooks
- * confusing-browser-globals
- *
- * => drop dependencies
- * airbnb
- * @eslint/eslintrc (if not using flat compat)
- * babel eslint parser
- *
- */
+import workspaces from 'eslint-plugin-workspaces'
+import pluginPromise from 'eslint-plugin-promise'
+import nodePlugin from 'eslint-plugin-n'
+import pluginCypress from 'eslint-plugin-cypress'
+import vitest from '@vitest/eslint-plugin'
+import react from 'eslint-plugin-react'
+import reactHooks from 'eslint-plugin-react-hooks'
+import jsxA11y from 'eslint-plugin-jsx-a11y'
+import confusingBrowserGlobals from 'confusing-browser-globals'
 
 const commonRules = {
   'array-callback-return': 'error',
@@ -87,7 +55,6 @@ const commonRules = {
   'no-template-curly-in-string': 'warn',
   'no-throw-literal': 'error',
   'no-undef-init': 'warn',
-  'no-undefined': 'error',
   'no-unneeded-ternary': ['error', { defaultAssignment: false }],
   'no-unused-expressions': 'error',
   'no-useless-computed-key': 'error',
@@ -109,7 +76,9 @@ const commonRules = {
     'ignorePackages',
     {
       js: 'never',
+      jsx: 'never',
       ts: 'never',
+      tsx: 'never',
     },
   ],
   'import/no-absolute-path': 'error',
@@ -120,25 +89,33 @@ const commonRules = {
       devDependencies: [
         // storybook
         '.storybook/*',
-        '**/stories/**/*.js',
+        '**/stories/**/*',
 
         // tests
         'cypress/**',
         '**/*.spec.js',
+        '**/*.spec.ts',
         '**/*.test.js',
+        '**/*.test.ts',
         '**/__tests__/**/*',
-
-        // webpack
-        'webpack/**',
+        '**/vitest.config.*',
 
         // configs
-        '.commitlintrc.js',
         '.cz-config.js',
-        '.jest.config.js',
+        '.cz-config.ts',
+        '.cz-config.mjs',
+        '.cz-config.cjs',
         '.lintstagedrc.js',
+        '.lintstagedrc.ts',
+        '.lintstagedrc.mjs',
         '.prettierrc.js',
+        '.prettierrc.ts',
+        '.prettierrc.mjs',
         '.stylelintrc.js',
+        '.stylelintrc.ts',
+        '.stylelintrc.mjs',
         'eslint.config.js',
+        'eslint.config.ts',
         'eslint.config.mjs',
 
         // other
@@ -151,9 +128,75 @@ const commonRules = {
   'import/order': ['error', { groups: [['builtin', 'external', 'internal']] }],
 }
 
+const workspacesConfig = {
+  plugins: { workspaces },
+  rules: {
+    'workspaces/no-relative-imports': 'error',
+    'workspaces/require-dependency': 'warn',
+  },
+}
+
+const typescriptConfig = {
+  files: ['**/*.{ts,tsx}'],
+  languageOptions: {
+    parser: tseslint.parser,
+  },
+  plugins: {
+    '@typescript-eslint': tseslint.plugin,
+  },
+  rules: {
+    ...tseslint.configs.recommended.rules,
+    '@typescript-eslint/explicit-function-return-type': 'error',
+
+    'no-redeclare': 'off',
+    '@typescript-eslint/no-redeclare': 'error',
+
+    'no-unused-vars': 'off',
+    '@typescript-eslint/no-unused-vars': [
+      'error',
+      {
+        argsIgnorePattern: '^_',
+        varsIgnorePattern: '^_',
+        caughtErrorsIgnorePattern: '^_',
+      },
+    ],
+  },
+}
+
+const viteConfig = {
+  files: ['**/__tests__/**/*.test.ts'],
+  plugins: {
+    vitest,
+  },
+  // settings: {
+  //   vitest: {
+  //     typecheck: true,
+  //   },
+  // },
+  rules: {
+    ...vitest.configs.recommended.rules,
+    'vitest/no-focused-tests': ['error', { fixable: false }],
+  },
+}
+
+const globalIgnoreList = [
+  '**/_build',
+  '**/dist',
+  '**/docs',
+  '!**/.storybook',
+  '!**/.cz-config.js',
+  '!**/.cz-config.cjs',
+  '!**/.lintstagedrc.js',
+  '!**/.prettierrc.js',
+  '!**/.stylelintrc.js',
+  '**/node_modules',
+]
+
 const server = [
   js.configs.recommended,
   importPlugin.flatConfigs.recommended,
+  pluginPromise.configs['flat/recommended'],
+  workspacesConfig,
 
   {
     files: ['**/*.{js,mjs,ts}'],
@@ -175,6 +218,7 @@ const server = [
     rules: {
       ...commonRules,
       'no-console': 'error',
+      // 'n/no-process-exit': 'off',
       'import/no-unresolved': ['error', { commonjs: true }],
     },
   },
@@ -189,28 +233,6 @@ const server = [
     },
   },
 
-  {
-    files: ['**/*.mjs', '**/*.ts'],
-    languageOptions: {
-      sourceType: 'module',
-    },
-    rules: {
-      'no-restricted-globals': [
-        'error',
-        {
-          name: '__dirname',
-          message:
-            'Do not use __dirname; use import.meta.url or process.cwd().',
-        },
-        {
-          name: '__filename',
-          message:
-            'Do not use __filename in ES modules or TypeScript; use import.meta.url or path.resolve() instead.',
-        },
-      ],
-    },
-  },
-
   {
     files: ['**/*.mjs'],
     rules: {
@@ -218,33 +240,76 @@ const server = [
     },
   },
 
+  typescriptConfig,
+  viteConfig,
+  globalIgnores(globalIgnoreList),
+]
+
+const client = [
+  js.configs.recommended,
+  importPlugin.flatConfigs.recommended,
+  pluginPromise.configs['flat/recommended'],
+  workspacesConfig,
+  react.configs.flat.recommended,
+  react.configs.flat['jsx-runtime'],
+  reactHooks.configs.flat.recommended,
+  jsxA11y.flatConfigs.recommended,
+
   {
-    files: ['**/*.ts'],
+    files: ['**/*.{js,jsx,mjs,cjs,ts,tsx}'],
     languageOptions: {
-      parser: tseslint.parser,
+      globals: {
+        ...globals.browser,
+        process: 'readonly',
+      },
+      ecmaVersion: 'latest',
+      parserOptions: {
+        ecmaFeatures: {
+          jsx: true,
+        },
+      },
     },
-    plugins: {
-      '@typescript-eslint': tseslint.plugin,
+    settings: {
+      'import/resolver': {
+        typescript: {
+          alwaysTryTypes: true,
+        },
+      },
+      react: {
+        version: 'detect',
+      },
     },
     rules: {
-      ...tseslint.configs.recommended.rules,
-      '@typescript-eslint/explicit-function-return-type': 'error',
+      ...commonRules,
+      'no-console': ['error', { allow: ['warn', 'error'] }],
+      'import/no-unresolved': 'error',
+      'react/jsx-sort-props': [1, { ignoreCase: true }],
+      'no-restricted-globals': ['error'].concat(confusingBrowserGlobals),
     },
   },
 
-  globalIgnores([
-    '**/_build',
-    '**/dist',
-    '**/docs',
-    '!**/.storybook',
-    '!**/.commitlintrc.js',
-    '!**/.cz-config.js',
-    '!**/.jest.config.js',
-    '!**/.lintstagedrc.js',
-    '!**/.prettierrc.js',
-    '!**/.stylelintrc.js',
-    '**/node_modules',
-  ]),
+  {
+    files: ['**/*.cjs'],
+    languageOptions: { sourceType: 'commonjs' },
+  },
+
+  typescriptConfig,
+  viteConfig,
+  globalIgnores(globalIgnoreList),
 ]
 
-export { defineConfig as defineEslintConfig, server as serverEslintConfig }
+const root = [
+  ...server,
+
+  {
+    files: ['cypress/**/*.{js,mjs,ts}'],
+    extends: [pluginCypress.configs.recommended],
+  },
+]
+
+export {
+  defineConfig as defineEslintConfig,
+  server as serverEslintConfig,
+  client as clientEslintConfig,
+  root as rootEslintConfig,
+}

package/src/index.mjs

@@ -1,12 +1,22 @@
-import commitizen from './commitizen'
-import { defineEslintConfig, serverEslintConfig } from './eslint.mjs'
-import prettier from './prettier'
-import stylelint from './stylelint'
-import lintstaged from './lintstaged'
+/* eslint-disable-next-line import/extensions */
+import commitizen from './commitizen.js'
+
+import {
+  defineEslintConfig,
+  serverEslintConfig,
+  clientEslintConfig,
+  rootEslintConfig,
+} from './eslint.mjs'
+
+import prettier from './prettier.mjs'
+import stylelint from './stylelint.mjs'
+import lintstaged from './lintstaged.mjs'
 
 export {
   defineEslintConfig,
   serverEslintConfig,
+  clientEslintConfig,
+  rootEslintConfig,
   stylelint,
   prettier,
   commitizen,

package/src/{lintstaged.js → lintstaged.mjs}

@@ -1,4 +1,4 @@
-module.exports = {
+export default {
   '*.js': ['eslint', 'stylelint'],
   '*.{js,graphql,json,yml,md,html}': ['prettier --check'],
 }

package/src/logger.mjs

@@ -0,0 +1,21 @@
+/* eslint-disable no-console */
+
+import chalk from 'chalk'
+
+const pre = chalk.cyan.underline('[coko lint]')
+const primary = chalk.cyan
+const error = chalk.red
+
+const logger = {
+  error: str => {
+    console.log(error(`${pre} ${str}`))
+  },
+  info: str => {
+    console.log(primary(`${pre} ${str}`))
+  },
+  newLine: () => {
+    console.log('')
+  },
+}
+
+export default logger

package/src/{prettier.js → prettier.mjs}

@@ -1,4 +1,4 @@
-module.exports = {
+export default {
   arrowParens: 'avoid',
   semi: false,
   singleQuote: true,

package/src/stylelint.mjs

@@ -0,0 +1,17 @@
+export default {
+  extends: ['stylelint-config-standard'],
+  overrides: [
+    {
+      files: ['**/*.{js,jsx,mjs,mts,ts,tsx}'],
+      customSyntax: 'postcss-styled-syntax',
+    },
+  ],
+  rules: {
+    'comment-empty-line-before': null,
+    'declaration-empty-line-before': null,
+    'declaration-block-no-duplicate-properties': true,
+    'declaration-no-important': true,
+    'no-descending-specificity': null,
+    'no-empty-source': null,
+  },
+}

package/src/airbnb/variables.mjs

@@ -1,26 +0,0 @@
-// const confusingBrowserGlobals = require('confusing-browser-globals')
-
-export default {
-  rules: {
-    // disallow specific globals
-    'no-restricted-globals': [
-      'error',
-      {
-        name: 'isFinite',
-        message:
-          'Use Number.isFinite instead https://github.com/airbnb/javascript#standard-library--isfinite',
-      },
-      {
-        name: 'isNaN',
-        message:
-          'Use Number.isNaN instead https://github.com/airbnb/javascript#standard-library--isnan',
-      },
-    ]
-      .concat
-      // confusingBrowserGlobals.map(g => ({
-      //   name: g,
-      //   message: `Use window.${g} instead. https://github.com/facebook/create-react-app/blob/HEAD/packages/confusing-browser-globals/README.md`,
-      // })),
-      (),
-  },
-}