@lunit/eslint-config 0.5.0 → 1.1.0

package/LICENSE

@@ -1,21 +1,24 @@
-MIT License
+@lunit/eslint-config
+
+Copyright (c) Lunit Inc. All rights reserved.
 
-Copyright (c) 2020-present Lunit.inc
+MIT License
 
-Permission is hereby granted, free of charge, to any person obtaining a copy
-of this software and associated documentation files (the "Software"), to deal
-in the Software without restriction, including without limitation the rights
-to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
-copies of the Software, and to permit persons to whom the Software is
-furnished to do so, subject to the following conditions:
+Permission is hereby granted, free of charge, to any person obtaining
+a copy of this software and associated documentation files (the
+"Software"), to deal in the Software without restriction, including
+without limitation the rights to use, copy, modify, merge, publish,
+distribute, sublicense, and/or sell copies of the Software, and to
+permit persons to whom the Software is furnished to do so, subject to
+the following conditions:
 
-The above copyright notice and this permission notice shall be included in all
-copies or substantial portions of the Software.
+The above copyright notice and this permission notice shall be
+included in all copies or substantial portions of the Software.
 
-THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
-IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
-FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
-AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
-LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
-OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
-SOFTWARE.
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
+EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
+MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
+NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE
+LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION
+OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION
+WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.

package/README.md

@@ -1,38 +1,257 @@
-# ESLint config
+# @lunit/eslint-config
 
-## Basic Usage
+A TypeScript ESLint ruleset designed for Lunit projects based on [@rushstack/eslint-config](https://github.com/microsoft/rushstack/tree/main/eslint/eslint-config).
 
-Install dependencies.
+This ESLint configuration is only available for TypeScript projects.
 
-```sh
-npm install eslint @lunit/eslint-config eslint-config-prettier --save-dev
+## Installation
+
+```shell
+npm install --save-dev @lunit/eslint-config
 ```
 
-Add this config into package.json if the project is a React application.
-
-```json
-{
-  "eslintConfig": {
-    "extends": [
-      "@lunit",
-      "prettier",
-      "prettier/react",
-      "prettier/@typescript-eslint"
-    ]
+## Usage
+
+### Getting Started
+
+`@lunit/eslint-config v2.x` is configured using the ESLint Flat Config approach.
+
+For information regarding ESLint Flat Config, please refer to the relevant ESLint Blog post.
+- https://eslint.org/blog/2022/08/new-config-system-part-2/
+
+
+To use it, add @lunit/eslint-config to your eslint.config.js file as follows.
+If you are using TypeScript ESLint rules, please remove them.
+This is already being used in Rushstack ESLint, causing conflict issues.
+
+
+#### ESM
+
+```javascript
+// eslint.config.js
+import js from "@eslint/js";
+import globals from "globals";
+import reactHooks from "eslint-plugin-react-hooks";
+import reactRefresh from "eslint-plugin-react-refresh";
+import { defineConfig, globalIgnores } from "eslint/config";
+import eslintConfig from '@lunit/eslint-config';
+
+export default defineConfig([
+  globalIgnores(["dist"]),
+  {
+    files: ["**/*.{ts,tsx}"],
+    extends: [
+      js.configs.recommended,
+      reactHooks.configs.flat["recommended-latest"],
+      reactRefresh.configs.vite,
+    ],
+    languageOptions: {
+      ecmaVersion: 2020,
+      globals: globals.browser,
+    },
+  },
+  lunitEslintConfig,
+]);
+```
+
+#### CommonJS
+
+```javascript
+// eslint.config.cjs
+const js = require("@eslint/js");
+const  globals = require("globals");
+const reactHooks = require("eslint-plugin-react-hooks");
+const reactRefresh = require("eslint-plugin-react-refresh");
+const { defineConfig, globalIgnores } = require("eslint/config");
+const lunitEslintConfig = require('@lunit/eslint-config');
+
+module.exports = defineConfig([
+  globalIgnores(["dist"]),
+  {
+    files: ["**/*.{ts,tsx}"],
+    extends: [
+      js.configs.recommended,
+      reactHooks.configs.flat["recommended-latest"],
+      reactRefresh.configs.vite,
+    ],
+    languageOptions: {
+      ecmaVersion: 2020,
+      globals: globals.browser,
+    },
+  },
+  // You can add the ESLint Config rules as follows.
+  lunitEslintConfig,
+]);
+```
+
+> If you created the app using vite,
+> please refer to the information below.
+
+If the tsconfig.json file is separate and the actual rules are applied in `tsconfig.app.json`,
+please configure it as follows.
+
+rushstack eslint config internally follows the rules set in tsconfig,
+so you must link the corresponding JSON file.
+
+#### ESM
+```js
+// eslint.config.js
+import js from "@eslint/js";
+import globals from "globals";
+import reactHooks from "eslint-plugin-react-hooks";
+import reactRefresh from "eslint-plugin-react-refresh";
+import { defineConfig, globalIgnores } from "eslint/config";
+import eslintConfig from '@lunit/eslint-config';
+
+export default defineConfig([
+  globalIgnores(["dist"]),
+  {
+    files: ["**/*.{ts,tsx}"],
+    extends: [
+      js.configs.recommended,
+      reactHooks.configs.flat["recommended-latest"],
+      reactRefresh.configs.vite,
+    ],
+    languageOptions: {
+      ecmaVersion: 2020,
+      globals: globals.browser,
+    },
+  },
+  {
+    ...eslintConfig,
+    // You must integrate the tsconfig.app.json file applied to the actual ts and tsx files as shown below.
+    languageOptions: {
+      parserOptions: {
+        project: ["./tsconfig.app.json"],
+        tsconfigRootDir: import.meta.dirname,
+      },
+    },
+  }
+]);
+```
+
+#### CommonJS
+
+```js
+// eslint.config.cjs
+const js = require("@eslint/js");
+const  globals = require("globals");
+const reactHooks = require("eslint-plugin-react-hooks");
+const reactRefresh = require("eslint-plugin-react-refresh");
+const { defineConfig, globalIgnores } = require("eslint/config");
+const lunitEslintConfig = require('@lunit/eslint-config');
+
+module.exports = defineConfig([
+  globalIgnores(["dist"]),
+  {
+    files: ["**/*.{ts,tsx}"],
+    extends: [
+      js.configs.recommended,
+      reactHooks.configs.flat["recommended-latest"],
+      reactRefresh.configs.vite,
+    ],
+    languageOptions: {
+      ecmaVersion: 2020,
+      globals: globals.browser,
+    },
+  },
+  {
+    ...lunitEslintConfig,
+    // You must integrate the tsconfig.app.json file applied to the actual ts and tsx files as shown below.
+    languageOptions: {
+      parserOptions: {
+        project: ["./tsconfig.app.json"],
+        tsconfigRootDir: __dirname,
+      },
+    },
+  }
+]);
+```
+
+### Advanced
+
+#### Override rushstack eslint rule
+
+If you wish to modify the Rushstack ESLint rule, please refer to the code below.
+
+#### ESM
+```js
+// eslint.config.js
+import js from "@eslint/js";
+import globals from "globals";
+import reactHooks from "eslint-plugin-react-hooks";
+import reactRefresh from "eslint-plugin-react-refresh";
+import { defineConfig, globalIgnores } from "eslint/config";
+import eslintConfig from '@lunit/eslint-config';
+
+export default defineConfig([
+  globalIgnores(["dist"]),
+  {
+    files: ["**/*.{ts,tsx}"],
+    extends: [
+      js.configs.recommended,
+      reactHooks.configs.flat["recommended-latest"],
+      reactRefresh.configs.vite,
+    ],
+    languageOptions: {
+      ecmaVersion: 2020,
+      globals: globals.browser,
+    },
+  },
+  {
+    // Using Spread Operator lunit Eslint Config
+    ...lunitEslintConfig,
+    rules: {
+      // Using Spread Operator lunit Eslint Config Rules
+      ...lunitEslintConfig.rules,
+      // And apply the target rule
+      '@rushstack/no-new-null': 'error',
+    },
   }
-}
+]);
 ```
 
-Or add this config into package.json if the project is not a React application. (like a Node.js application)
+#### CommonJS
+
+```js
+// eslint.config.cjs
+const js = require("@eslint/js");
+const  globals = require("globals");
+const reactHooks = require("eslint-plugin-react-hooks");
+const reactRefresh = require("eslint-plugin-react-refresh");
+const { defineConfig, globalIgnores } = require("eslint/config");
+const lunitEslintConfig = require('@lunit/eslint-config');
 
-```json
-{
-  "eslintConfig": {
-    "extends": [
-      "@lunit/eslint-config/without-react",
-      "prettier",
-      "prettier/@typescript-eslint"
-    ]
+module.exports = defineConfig([
+  globalIgnores(["dist"]),
+  {
+    files: ["**/*.{ts,tsx}"],
+    extends: [
+      js.configs.recommended,
+      reactHooks.configs.flat["recommended-latest"],
+      reactRefresh.configs.vite,
+    ],
+    languageOptions: {
+      ecmaVersion: 2020,
+      globals: globals.browser,
+    },
+  },
+  {
+    // Using Spread Operator lunit Eslint Config
+    ...lunitEslintConfig,
+    rules: {
+      // Using Spread Operator lunit Eslint Config Rules
+      ...lunitEslintConfig.rules,
+      // And apply the target rule
+      '@rushstack/no-new-null': 'error',
+    },
   }
-}
-```  
+]);
+```
+
+#### Migrate 1.x to 2.x
+
+Since migration to the ESLint flat config is required,
+please refer to the official ESLint documentation guide below.
+
+- https://eslint.org/docs/latest/use/configure/migration-guide

package/dist/index.js

@@ -0,0 +1,110 @@
+"use strict";
+const react = require("@rushstack/eslint-config/flat/mixins/react.js");
+const webApp = require("@rushstack/eslint-config/flat/profile/web-app.js");
+function _interopNamespaceDefault(e) {
+  const n = Object.create(null, { [Symbol.toStringTag]: { value: "Module" } });
+  if (e) {
+    for (const k in e) {
+      if (k !== "default") {
+        const d = Object.getOwnPropertyDescriptor(e, k);
+        Object.defineProperty(n, k, d.get ? d : {
+          enumerable: true,
+          get: () => e[k]
+        });
+      }
+    }
+  }
+  n.default = e;
+  return Object.freeze(n);
+}
+const react__namespace = /* @__PURE__ */ _interopNamespaceDefault(react);
+const webApp__namespace = /* @__PURE__ */ _interopNamespaceDefault(webApp);
+function getDefaultExportFromCjs(x) {
+  return x && x.__esModule && Object.prototype.hasOwnProperty.call(x, "default") ? x["default"] : x;
+}
+function getAugmentedNamespace(n) {
+  if (Object.prototype.hasOwnProperty.call(n, "__esModule")) return n;
+  var f = n.default;
+  if (typeof f == "function") {
+    var a = function a2() {
+      var isInstance = false;
+      try {
+        isInstance = this instanceof a2;
+      } catch {
+      }
+      if (isInstance) {
+        return Reflect.construct(f, arguments, this.constructor);
+      }
+      return f.apply(this, arguments);
+    };
+    a.prototype = f.prototype;
+  } else a = {};
+  Object.defineProperty(a, "__esModule", { value: true });
+  Object.keys(n).forEach(function(k) {
+    var d = Object.getOwnPropertyDescriptor(n, k);
+    Object.defineProperty(a, k, d.get ? d : {
+      enumerable: true,
+      get: function() {
+        return n[k];
+      }
+    });
+  });
+  return a;
+}
+const require$$0 = /* @__PURE__ */ getAugmentedNamespace(react__namespace);
+const require$$1 = /* @__PURE__ */ getAugmentedNamespace(webApp__namespace);
+var eslintConfig;
+var hasRequiredEslintConfig;
+function requireEslintConfig() {
+  if (hasRequiredEslintConfig) return eslintConfig;
+  hasRequiredEslintConfig = 1;
+  const rushStackReactConfig = require$$0;
+  const rushStackWebAppConfig = require$$1;
+  function cleanConfig(config) {
+    if (Array.isArray(config)) {
+      return config.map(cleanConfig);
+    }
+    if (config && typeof config === "object" && config.default) {
+      return cleanConfig(config.default);
+    }
+    return config;
+  }
+  const cleanedExtends = [rushStackWebAppConfig, rushStackReactConfig].map(cleanConfig);
+  eslintConfig = {
+    files: ["**/*.ts", "**/*.tsx"],
+    extends: cleanedExtends,
+    rules: {
+      /**
+       * RATIONALE : The null type is universally used, so it is excluded.
+       * docs: https://www.npmjs.com/package/@rushstack/eslint-plugin
+       */
+      "@rushstack/no-new-null": "off",
+      /**
+       * RATIONALE : Rushstack's policy enforces explicit types for top-level variables for readability,
+       * but our project prefers to leverage TypeScript's type inference.
+       * docs: https://github.com/microsoft/rushstack/tree/main/eslint/eslint-plugin#rushstacktypedef-var
+       */
+      "@rushstack/typedef-var": "off",
+      /**
+       * RATIONALE : According to the documentation, it can be used in projects that use many classes,
+       * but it is unnecessary because the development pattern is functional.
+       * docs: https://typescript-eslint.io/rules/explicit-member-accessibility/
+       */
+      "@typescript-eslint/explicit-member-accessibility": "off",
+      /**
+       * RATIONALE : Depending on the situation, it may be better to leave the return type to type inference.
+       * docs: https://typescript-eslint.io/rules/explicit-function-return-type/
+       */
+      "@typescript-eslint/explicit-function-return-type": "off"
+    },
+    settings: {
+      react: {
+        version: "detect"
+      }
+    }
+  };
+  return eslintConfig;
+}
+var eslintConfigExports = requireEslintConfig();
+const index = /* @__PURE__ */ getDefaultExportFromCjs(eslintConfigExports);
+module.exports = index;

package/dist/index.mjs

@@ -0,0 +1,91 @@
+import * as react from "@rushstack/eslint-config/flat/mixins/react.js";
+import * as webApp from "@rushstack/eslint-config/flat/profile/web-app.js";
+function getDefaultExportFromCjs(x) {
+  return x && x.__esModule && Object.prototype.hasOwnProperty.call(x, "default") ? x["default"] : x;
+}
+function getAugmentedNamespace(n) {
+  if (Object.prototype.hasOwnProperty.call(n, "__esModule")) return n;
+  var f = n.default;
+  if (typeof f == "function") {
+    var a = function a2() {
+      var isInstance = false;
+      try {
+        isInstance = this instanceof a2;
+      } catch {
+      }
+      if (isInstance) {
+        return Reflect.construct(f, arguments, this.constructor);
+      }
+      return f.apply(this, arguments);
+    };
+    a.prototype = f.prototype;
+  } else a = {};
+  Object.defineProperty(a, "__esModule", { value: true });
+  Object.keys(n).forEach(function(k) {
+    var d = Object.getOwnPropertyDescriptor(n, k);
+    Object.defineProperty(a, k, d.get ? d : {
+      enumerable: true,
+      get: function() {
+        return n[k];
+      }
+    });
+  });
+  return a;
+}
+const require$$0 = /* @__PURE__ */ getAugmentedNamespace(react);
+const require$$1 = /* @__PURE__ */ getAugmentedNamespace(webApp);
+var eslintConfig;
+var hasRequiredEslintConfig;
+function requireEslintConfig() {
+  if (hasRequiredEslintConfig) return eslintConfig;
+  hasRequiredEslintConfig = 1;
+  const rushStackReactConfig = require$$0;
+  const rushStackWebAppConfig = require$$1;
+  function cleanConfig(config) {
+    if (Array.isArray(config)) {
+      return config.map(cleanConfig);
+    }
+    if (config && typeof config === "object" && config.default) {
+      return cleanConfig(config.default);
+    }
+    return config;
+  }
+  const cleanedExtends = [rushStackWebAppConfig, rushStackReactConfig].map(cleanConfig);
+  eslintConfig = {
+    files: ["**/*.ts", "**/*.tsx"],
+    extends: cleanedExtends,
+    rules: {
+      /**
+       * RATIONALE : The null type is universally used, so it is excluded.
+       * docs: https://www.npmjs.com/package/@rushstack/eslint-plugin
+       */
+      "@rushstack/no-new-null": "off",
+      /**
+       * RATIONALE : Rushstack's policy enforces explicit types for top-level variables for readability,
+       * but our project prefers to leverage TypeScript's type inference.
+       * docs: https://github.com/microsoft/rushstack/tree/main/eslint/eslint-plugin#rushstacktypedef-var
+       */
+      "@rushstack/typedef-var": "off",
+      /**
+       * RATIONALE : According to the documentation, it can be used in projects that use many classes,
+       * but it is unnecessary because the development pattern is functional.
+       * docs: https://typescript-eslint.io/rules/explicit-member-accessibility/
+       */
+      "@typescript-eslint/explicit-member-accessibility": "off",
+      /**
+       * RATIONALE : Depending on the situation, it may be better to leave the return type to type inference.
+       * docs: https://typescript-eslint.io/rules/explicit-function-return-type/
+       */
+      "@typescript-eslint/explicit-function-return-type": "off"
+    },
+    settings: {
+      react: {
+        version: "detect"
+      }
+    }
+  };
+  return eslintConfig;
+}
+var eslintConfigExports = requireEslintConfig();
+const index = /* @__PURE__ */ getDefaultExportFromCjs(eslintConfigExports);
+export default index;

package/package.json

@@ -1,26 +1,40 @@
 {
   "name": "@lunit/eslint-config",
-  "version": "0.5.0",
-  "description": "Lunit eslint config",
-  "main": "index.js",
+  "version": "1.1.0",
+  "description": "Lunit ESLint Config",
+  "type": "commonjs",
+  "main": "dist/index.js",
+  "files": ["dist"],
+  "exports": {
+    ".": {
+      "import": "./dist/index.mjs",
+      "require": "./dist/index.js"
+    }
+  },
   "repository": "lunit-io/eslint-config",
   "author": {
-    "name": "Seoyeon Lee",
-    "email": "i@ssen.name"
+    "name": "Larry Kim",
+    "email": "larry@lunit.io"
   },
   "publishConfig": {
     "access": "public"
   },
+  "scripts": {
+    "build": "vite build"
+  },
   "license": "MIT",
+  "peerDependencies": {
+    "eslint": ">= 8",
+    "prettier": ">= 3",
+    "typescript": ">= 5"
+  },
   "dependencies": {
-    "@typescript-eslint/eslint-plugin": "2.x",
-    "@typescript-eslint/parser": "2.x",
-    "babel-eslint": "10.x",
-    "confusing-browser-globals": "^1.0.9",
-    "eslint": "6.x",
-    "eslint-plugin-import": "2.x",
-    "eslint-plugin-jsx-a11y": "6.x",
-    "eslint-plugin-react": "7.x",
-    "eslint-plugin-react-hooks": "2.3.0"
+    "@rushstack/eslint-config": "^4.4.0"
+  },
+  "devDependencies": {
+    "@rollup/plugin-commonjs": "^28.0.6",
+    "eslint": "^9.36.0",
+    "typescript": "^5.8.3",
+    "vite": "^7.1.7"
   }
 }

package/index.js

@@ -1,299 +0,0 @@
-const restrictedGlobals = require('confusing-browser-globals');
-const {typescript} = require('./rules');
-
-module.exports = {
-  root: true,
-  
-  parser: 'babel-eslint',
-  
-  plugins: ['import', 'jsx-a11y', 'react', 'react-hooks'],
-  
-  env: {
-    browser: true,
-    commonjs: true,
-    es6: true,
-    jest: true,
-    node: true,
-  },
-  
-  parserOptions: {
-    ecmaVersion: 2018,
-    sourceType: 'module',
-    ecmaFeatures: {
-      jsx: true,
-    },
-  },
-  
-  settings: {
-    react: {
-      version: 'detect',
-    },
-  },
-  
-  overrides: [
-    {
-      files: ['**/*.ts?(x)'],
-      parser: '@typescript-eslint/parser',
-      parserOptions: {
-        ecmaVersion: 2018,
-        sourceType: 'module',
-        ecmaFeatures: {
-          jsx: true,
-        },
-        
-        // typescript-eslint specific options
-        warnOnUnsupportedTypeScriptVersion: true,
-      },
-      plugins: ['@typescript-eslint'],
-      // If adding a typescript-eslint version of an existing ESLint rule,
-      // make sure to disable the ESLint rule here.
-      rules: {
-        // TypeScript's `noFallthroughCasesInSwitch` option is more robust (#6906)
-        'default-case': 'off',
-        // 'tsc' already handles this (https://github.com/typescript-eslint/typescript-eslint/issues/291)
-        'no-dupe-class-members': 'off',
-        // 'tsc' already handles this (https://github.com/typescript-eslint/typescript-eslint/issues/477)
-        'no-undef': 'off',
-        
-        // Add TypeScript specific rules (and turn off ESLint equivalents)
-        '@typescript-eslint/consistent-type-assertions': 'warn',
-        'no-array-constructor': 'off',
-        '@typescript-eslint/no-array-constructor': 'warn',
-        'no-use-before-define': 'off',
-        '@typescript-eslint/no-use-before-define': [
-          'warn',
-          {
-            functions: false,
-            classes: false,
-            variables: false,
-            typedefs: false,
-          },
-        ],
-        'no-unused-expressions': 'off',
-        '@typescript-eslint/no-unused-expressions': [
-          'error',
-          {
-            allowShortCircuit: true,
-            allowTernary: true,
-            allowTaggedTemplates: true,
-          },
-        ],
-        'no-unused-vars': 'off',
-        '@typescript-eslint/no-unused-vars': [
-          'warn',
-          {
-            args: 'none',
-            ignoreRestSiblings: true,
-          },
-        ],
-        'no-useless-constructor': 'off',
-        '@typescript-eslint/no-useless-constructor': 'warn',
-  
-        ...typescript,
-      },
-    },
-    {
-      files: ['**/*.stories.{js,jsx,ts,tsx}'],
-      rules: {
-        'react-hooks/rules-of-hooks': 'off',
-      }
-    }
-  ],
-  
-  // NOTE: When adding rules here, you need to make sure they are compatible with
-  // `typescript-eslint`, as some rules such as `no-array-constructor` aren't compatible.
-  rules: {
-    // http://eslint.org/docs/rules/
-    'array-callback-return': 'warn',
-    'default-case': ['warn', {commentPattern: '^no default$'}],
-    'dot-location': ['warn', 'property'],
-    eqeqeq: ['warn', 'smart'],
-    'new-parens': 'warn',
-    'no-array-constructor': 'warn',
-    'no-caller': 'warn',
-    'no-cond-assign': ['warn', 'except-parens'],
-    'no-const-assign': 'warn',
-    'no-control-regex': 'warn',
-    'no-delete-var': 'warn',
-    'no-dupe-args': 'warn',
-    'no-dupe-class-members': 'warn',
-    'no-dupe-keys': 'warn',
-    'no-duplicate-case': 'warn',
-    'no-empty-character-class': 'warn',
-    'no-empty-pattern': 'warn',
-    'no-eval': 'warn',
-    'no-ex-assign': 'warn',
-    'no-extend-native': 'warn',
-    'no-extra-bind': 'warn',
-    'no-extra-label': 'warn',
-    'no-fallthrough': 'warn',
-    'no-func-assign': 'warn',
-    'no-implied-eval': 'warn',
-    'no-invalid-regexp': 'warn',
-    'no-iterator': 'warn',
-    'no-label-var': 'warn',
-    'no-labels': ['warn', {allowLoop: true, allowSwitch: false}],
-    'no-lone-blocks': 'warn',
-    'no-loop-func': 'warn',
-    'no-mixed-operators': [
-      'warn',
-      {
-        groups: [
-          ['&', '|', '^', '~', '<<', '>>', '>>>'],
-          ['==', '!=', '===', '!==', '>', '>=', '<', '<='],
-          ['&&', '||'],
-          ['in', 'instanceof'],
-        ],
-        allowSamePrecedence: false,
-      },
-    ],
-    'no-multi-str': 'warn',
-    'no-native-reassign': 'warn',
-    'no-negated-in-lhs': 'warn',
-    'no-new-func': 'warn',
-    'no-new-object': 'warn',
-    'no-new-symbol': 'warn',
-    'no-new-wrappers': 'warn',
-    'no-obj-calls': 'warn',
-    'no-octal': 'warn',
-    'no-octal-escape': 'warn',
-    // TODO: Remove this option in the next major release of CRA.
-    // https://eslint.org/docs/user-guide/migrating-to-6.0.0#-the-no-redeclare-rule-is-now-more-strict-by-default
-    'no-redeclare': ['warn', {builtinGlobals: false}],
-    'no-regex-spaces': 'warn',
-    'no-restricted-syntax': ['warn', 'WithStatement'],
-    'no-script-url': 'warn',
-    'no-self-assign': 'warn',
-    'no-self-compare': 'warn',
-    'no-sequences': 'warn',
-    'no-shadow-restricted-names': 'warn',
-    'no-sparse-arrays': 'warn',
-    'no-template-curly-in-string': 'warn',
-    'no-this-before-super': 'warn',
-    'no-throw-literal': 'warn',
-    'no-undef': 'error',
-    'no-restricted-globals': ['error'].concat(restrictedGlobals),
-    'no-unreachable': 'warn',
-    'no-unused-expressions': [
-      'error',
-      {
-        allowShortCircuit: true,
-        allowTernary: true,
-        allowTaggedTemplates: true,
-      },
-    ],
-    'no-unused-labels': 'warn',
-    'no-unused-vars': [
-      'warn',
-      {
-        args: 'none',
-        ignoreRestSiblings: true,
-      },
-    ],
-    'no-use-before-define': [
-      'warn',
-      {
-        functions: false,
-        classes: false,
-        variables: false,
-      },
-    ],
-    'no-useless-computed-key': 'warn',
-    'no-useless-concat': 'warn',
-    'no-useless-constructor': 'warn',
-    'no-useless-escape': 'warn',
-    'no-useless-rename': [
-      'warn',
-      {
-        ignoreDestructuring: false,
-        ignoreImport: false,
-        ignoreExport: false,
-      },
-    ],
-    'no-with': 'warn',
-    'no-whitespace-before-property': 'warn',
-    'react-hooks/exhaustive-deps': 'warn',
-    'require-yield': 'warn',
-    'rest-spread-spacing': ['warn', 'never'],
-    strict: ['warn', 'never'],
-    'unicode-bom': ['warn', 'never'],
-    'use-isnan': 'warn',
-    'valid-typeof': 'warn',
-    'no-restricted-properties': [
-      'error',
-      {
-        object: 'require',
-        property: 'ensure',
-        message:
-          'Please use import() instead. More info: https://facebook.github.io/create-react-app/docs/code-splitting',
-      },
-      {
-        object: 'System',
-        property: 'import',
-        message:
-          'Please use import() instead. More info: https://facebook.github.io/create-react-app/docs/code-splitting',
-      },
-    ],
-    'getter-return': 'warn',
-    
-    // https://github.com/benmosher/eslint-plugin-import/tree/master/docs/rules
-    'import/first': 'error',
-    'import/no-amd': 'error',
-    'import/no-webpack-loader-syntax': 'error',
-    
-    // https://github.com/yannickcr/eslint-plugin-react/tree/master/docs/rules
-    'react/forbid-foreign-prop-types': ['warn', {allowInPropTypes: true}],
-    'react/jsx-no-comment-textnodes': 'warn',
-    'react/jsx-no-duplicate-props': 'warn',
-    'react/jsx-no-target-blank': 'warn',
-    'react/jsx-no-undef': 'error',
-    'react/jsx-pascal-case': [
-      'warn',
-      {
-        allowAllCaps: true,
-        ignore: [],
-      },
-    ],
-    'react/jsx-uses-react': 'warn',
-    'react/jsx-uses-vars': 'warn',
-    'react/no-danger-with-children': 'warn',
-    // Disabled because of undesirable warnings
-    // See https://github.com/facebook/create-react-app/issues/5204 for
-    // blockers until its re-enabled
-    // 'react/no-deprecated': 'warn',
-    'react/no-direct-mutation-state': 'warn',
-    'react/no-is-mounted': 'warn',
-    'react/no-typos': 'error',
-    'react/react-in-jsx-scope': 'error',
-    'react/require-render-return': 'error',
-    'react/style-prop-object': 'warn',
-    
-    // https://github.com/evcohen/eslint-plugin-jsx-a11y/tree/master/docs/rules
-    'jsx-a11y/accessible-emoji': 'warn',
-    'jsx-a11y/alt-text': 'warn',
-    'jsx-a11y/anchor-has-content': 'warn',
-    'jsx-a11y/anchor-is-valid': [
-      'warn',
-      {
-        aspects: ['noHref', 'invalidHref'],
-      },
-    ],
-    'jsx-a11y/aria-activedescendant-has-tabindex': 'warn',
-    'jsx-a11y/aria-props': 'warn',
-    'jsx-a11y/aria-proptypes': 'warn',
-    'jsx-a11y/aria-role': ['warn', {ignoreNonDOM: true}],
-    'jsx-a11y/aria-unsupported-elements': 'warn',
-    'jsx-a11y/heading-has-content': 'warn',
-    'jsx-a11y/iframe-has-title': 'warn',
-    'jsx-a11y/img-redundant-alt': 'warn',
-    'jsx-a11y/no-access-key': 'warn',
-    'jsx-a11y/no-distracting-elements': 'warn',
-    'jsx-a11y/no-redundant-roles': 'warn',
-    'jsx-a11y/role-has-required-aria-props': 'warn',
-    'jsx-a11y/role-supports-aria-props': 'warn',
-    'jsx-a11y/scope': 'warn',
-    
-    // https://github.com/facebook/react/tree/master/packages/eslint-plugin-react-hooks
-    'react-hooks/rules-of-hooks': 'error',
-  },
-};

package/rules.js

@@ -1,15 +0,0 @@
-exports.typescript = {
-  '@typescript-eslint/no-explicit-any': 'error',
-  '@typescript-eslint/typedef': [
-    'error',
-    {
-      'arrayDestructuring': false,
-      'arrowParameter': false,
-      'objectDestructuring': false,
-      'parameter': true,
-      'propertyDeclaration': true,
-      'memberVariableDeclaration': false,
-      'variableDeclaration': false,
-    },
-  ],
-};

package/without-react.js

@@ -1,231 +0,0 @@
-const restrictedGlobals = require('confusing-browser-globals');
-const {typescript} = require('./rules');
-
-module.exports = {
-  root: true,
-  
-  parser: 'babel-eslint',
-  
-  plugins: ['import'],
-  
-  env: {
-    browser: true,
-    commonjs: true,
-    es6: true,
-    jest: true,
-    node: true,
-  },
-  
-  parserOptions: {
-    ecmaVersion: 2018,
-    sourceType: 'module',
-    ecmaFeatures: {
-      jsx: false,
-    },
-  },
-  
-  overrides: [
-    {
-      files: ['**/*.ts?(x)'],
-      parser: '@typescript-eslint/parser',
-      parserOptions: {
-        ecmaVersion: 2018,
-        sourceType: 'module',
-        ecmaFeatures: {
-          jsx: false,
-        },
-        
-        // typescript-eslint specific options
-        warnOnUnsupportedTypeScriptVersion: true,
-      },
-      plugins: ['@typescript-eslint'],
-      // If adding a typescript-eslint version of an existing ESLint rule,
-      // make sure to disable the ESLint rule here.
-      rules: {
-        // TypeScript's `noFallthroughCasesInSwitch` option is more robust (#6906)
-        'default-case': 'off',
-        // 'tsc' already handles this (https://github.com/typescript-eslint/typescript-eslint/issues/291)
-        'no-dupe-class-members': 'off',
-        // 'tsc' already handles this (https://github.com/typescript-eslint/typescript-eslint/issues/477)
-        'no-undef': 'off',
-        
-        // Add TypeScript specific rules (and turn off ESLint equivalents)
-        '@typescript-eslint/consistent-type-assertions': 'warn',
-        'no-array-constructor': 'off',
-        '@typescript-eslint/no-array-constructor': 'warn',
-        'no-use-before-define': 'off',
-        '@typescript-eslint/no-use-before-define': [
-          'warn',
-          {
-            functions: false,
-            classes: false,
-            variables: false,
-            typedefs: false,
-          },
-        ],
-        'no-unused-expressions': 'off',
-        '@typescript-eslint/no-unused-expressions': [
-          'error',
-          {
-            allowShortCircuit: true,
-            allowTernary: true,
-            allowTaggedTemplates: true,
-          },
-        ],
-        'no-unused-vars': 'off',
-        '@typescript-eslint/no-unused-vars': [
-          'warn',
-          {
-            args: 'none',
-            ignoreRestSiblings: true,
-          },
-        ],
-        'no-useless-constructor': 'off',
-        '@typescript-eslint/no-useless-constructor': 'warn',
-        
-        ...typescript,
-      },
-    },
-  ],
-  
-  // NOTE: When adding rules here, you need to make sure they are compatible with
-  // `typescript-eslint`, as some rules such as `no-array-constructor` aren't compatible.
-  rules: {
-    // http://eslint.org/docs/rules/
-    'array-callback-return': 'warn',
-    'default-case': ['warn', {commentPattern: '^no default$'}],
-    'dot-location': ['warn', 'property'],
-    eqeqeq: ['warn', 'smart'],
-    'new-parens': 'warn',
-    'no-array-constructor': 'warn',
-    'no-caller': 'warn',
-    'no-cond-assign': ['warn', 'except-parens'],
-    'no-const-assign': 'warn',
-    'no-control-regex': 'warn',
-    'no-delete-var': 'warn',
-    'no-dupe-args': 'warn',
-    'no-dupe-class-members': 'warn',
-    'no-dupe-keys': 'warn',
-    'no-duplicate-case': 'warn',
-    'no-empty-character-class': 'warn',
-    'no-empty-pattern': 'warn',
-    'no-eval': 'warn',
-    'no-ex-assign': 'warn',
-    'no-extend-native': 'warn',
-    'no-extra-bind': 'warn',
-    'no-extra-label': 'warn',
-    'no-fallthrough': 'warn',
-    'no-func-assign': 'warn',
-    'no-implied-eval': 'warn',
-    'no-invalid-regexp': 'warn',
-    'no-iterator': 'warn',
-    'no-label-var': 'warn',
-    'no-labels': ['warn', {allowLoop: true, allowSwitch: false}],
-    'no-lone-blocks': 'warn',
-    'no-loop-func': 'warn',
-    'no-mixed-operators': [
-      'warn',
-      {
-        groups: [
-          ['&', '|', '^', '~', '<<', '>>', '>>>'],
-          ['==', '!=', '===', '!==', '>', '>=', '<', '<='],
-          ['&&', '||'],
-          ['in', 'instanceof'],
-        ],
-        allowSamePrecedence: false,
-      },
-    ],
-    'no-multi-str': 'warn',
-    'no-native-reassign': 'warn',
-    'no-negated-in-lhs': 'warn',
-    'no-new-func': 'warn',
-    'no-new-object': 'warn',
-    'no-new-symbol': 'warn',
-    'no-new-wrappers': 'warn',
-    'no-obj-calls': 'warn',
-    'no-octal': 'warn',
-    'no-octal-escape': 'warn',
-    // TODO: Remove this option in the next major release of CRA.
-    // https://eslint.org/docs/user-guide/migrating-to-6.0.0#-the-no-redeclare-rule-is-now-more-strict-by-default
-    'no-redeclare': ['warn', {builtinGlobals: false}],
-    'no-regex-spaces': 'warn',
-    'no-restricted-syntax': ['warn', 'WithStatement'],
-    'no-script-url': 'warn',
-    'no-self-assign': 'warn',
-    'no-self-compare': 'warn',
-    'no-sequences': 'warn',
-    'no-shadow-restricted-names': 'warn',
-    'no-sparse-arrays': 'warn',
-    'no-template-curly-in-string': 'warn',
-    'no-this-before-super': 'warn',
-    'no-throw-literal': 'warn',
-    'no-undef': 'error',
-    'no-restricted-globals': ['error'].concat(restrictedGlobals),
-    'no-unreachable': 'warn',
-    'no-unused-expressions': [
-      'error',
-      {
-        allowShortCircuit: true,
-        allowTernary: true,
-        allowTaggedTemplates: true,
-      },
-    ],
-    'no-unused-labels': 'warn',
-    'no-unused-vars': [
-      'warn',
-      {
-        args: 'none',
-        ignoreRestSiblings: true,
-      },
-    ],
-    'no-use-before-define': [
-      'warn',
-      {
-        functions: false,
-        classes: false,
-        variables: false,
-      },
-    ],
-    'no-useless-computed-key': 'warn',
-    'no-useless-concat': 'warn',
-    'no-useless-constructor': 'warn',
-    'no-useless-escape': 'warn',
-    'no-useless-rename': [
-      'warn',
-      {
-        ignoreDestructuring: false,
-        ignoreImport: false,
-        ignoreExport: false,
-      },
-    ],
-    'no-with': 'warn',
-    'no-whitespace-before-property': 'warn',
-    'require-yield': 'warn',
-    'rest-spread-spacing': ['warn', 'never'],
-    strict: ['warn', 'never'],
-    'unicode-bom': ['warn', 'never'],
-    'use-isnan': 'warn',
-    'valid-typeof': 'warn',
-    'no-restricted-properties': [
-      'error',
-      {
-        object: 'require',
-        property: 'ensure',
-        message:
-          'Please use import() instead. More info: https://facebook.github.io/create-react-app/docs/code-splitting',
-      },
-      {
-        object: 'System',
-        property: 'import',
-        message:
-          'Please use import() instead. More info: https://facebook.github.io/create-react-app/docs/code-splitting',
-      },
-    ],
-    'getter-return': 'warn',
-    
-    // https://github.com/benmosher/eslint-plugin-import/tree/master/docs/rules
-    'import/first': 'error',
-    'import/no-amd': 'error',
-    'import/no-webpack-loader-syntax': 'error',
-  },
-};