@foray1010/eslint-config 10.0.5 → 10.2.0

package/CHANGELOG.md

@@ -3,6 +3,26 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+## [10.2.0](https://github.com/foray1010/common-presets/compare/@foray1010/eslint-config@10.1.0...@foray1010/eslint-config@10.2.0) (2023-03-29)
+
+### Features
+
+- enable @typescript-eslint/no-unused-vars ([ff31b94](https://github.com/foray1010/common-presets/commit/ff31b94884796f00723f396469d53ea747109ff9))
+- **eslint-config:** enable n/no-process-exit ([9e9ddce](https://github.com/foray1010/common-presets/commit/9e9ddce3d43dad54fc9570e7f99bf7a6d3be009e))
+- **eslint-config:** enable react/jsx-no-leaked-render ([d65d8b4](https://github.com/foray1010/common-presets/commit/d65d8b4a83c50b050ce2ab68701c691b9c885cbf))
+- **eslint-config:** enable react/no-object-type-as-default-prop ([79c1639](https://github.com/foray1010/common-presets/commit/79c163963a3eaac503e79376ccf880b0165e6556))
+- **eslint-config:** prefer inline exports ([2a9de59](https://github.com/foray1010/common-presets/commit/2a9de5940a042607bc9556989937647d98236f86))
+
+### Reverts
+
+- Revert "fix(eslint-config): disable n/prefer-global hacks as they do not work with flat config" ([1e02034](https://github.com/foray1010/common-presets/commit/1e02034f6a5ec5d7d19230ddebcbcbf64e682022))
+
+## [10.1.0](https://github.com/foray1010/common-presets/compare/@foray1010/eslint-config@10.0.5...@foray1010/eslint-config@10.1.0) (2023-03-27)
+
+### Features
+
+- **eslint-config:** use inline type imports to avoid duplicate imports ([b6741bb](https://github.com/foray1010/common-presets/commit/b6741bbe0a95a579d1ae85ee32a1c75c309d20a9))
+
 ## [10.0.5](https://github.com/foray1010/common-presets/compare/@foray1010/eslint-config@10.0.4...@foray1010/eslint-config@10.0.5) (2023-03-24)
 
 ### Bug Fixes

package/bases/base.mjs

@@ -79,12 +79,19 @@ async function generateTypeScriptConfig() {
           },
         ],
         // separate type exports which allow certain optimizations within compilers
-        '@typescript-eslint/consistent-type-exports': 'error',
+        '@typescript-eslint/consistent-type-exports': [
+          'error',
+          {
+            fixMixedExportsWithInlineTypeSpecifier: true,
+          },
+        ],
         '@typescript-eslint/consistent-type-imports': [
           'error',
           {
             // separate type imports which allow certain optimizations within compilers
             prefer: 'type-imports',
+            // Use inline type imports to avoid duplicate imports, but can be overridden by @typescript-eslint/no-import-type-side-effects
+            fixStyle: 'inline-type-imports',
           },
         ],
         // disable the base rule as it can report incorrect errors, use @typescript-eslint/dot-notation instead
@@ -122,6 +129,13 @@ async function generateTypeScriptConfig() {
         ],
         // declaration merging between classes and interfaces is unsafe
         '@typescript-eslint/no-unsafe-declaration-merging': 'error',
+        '@typescript-eslint/no-unused-vars': [
+          'error',
+          {
+            // error is optional now
+            caughtErrors: 'all',
+          },
+        ],
         // do not block functions referring to other functions
         '@typescript-eslint/no-use-before-define': [
           'error',

package/bases/node.mjs

@@ -20,11 +20,23 @@ const nodeConfig = [
     plugins: {
       n: eslintPluginN,
     },
+    languageOptions: {
+      globals: {
+        // hack to mute no-undef error, and show n/prefer-global/buffer error instead
+        // @ts-expect-error
+        Buffer: 'writable',
+        // hack to mute no-undef error, and show n/prefer-global/process error instead
+        // @ts-expect-error
+        process: 'writable',
+      },
+    },
     rules: {
       // disallow deprecated node APIs
       'n/no-deprecated-api': 'error',
       // disallow the assignment to `exports`
       'n/no-exports-assign': 'error',
+      // use process.exitCode instead
+      'n/no-process-exit': 'error',
       // disallow `bin` files that npm ignores
       'n/no-unpublished-bin': 'error',
       // disallow unsupported Node.js built-in APIs on the specified version

package/bases/react.mjs

@@ -31,6 +31,8 @@ const reactConfig = [
       ...eslintPluginReactHooks.configs['recommended']?.rules,
       // avoid unexpected form submits
       'react/button-has-type': 'error',
+      // avoid displaying unexpected value, and align coding style with react native
+      'react/jsx-no-leaked-render': 'error',
       'react/jsx-no-useless-fragment': [
         'error',
         {
@@ -50,6 +52,8 @@ const reactConfig = [
           shorthandLast: false,
         },
       ],
+      // avoid unexpected react hook dead loops
+      'react/no-object-type-as-default-prop': 'error',
       // rely on typescript instead, and it does not work well with types that are imported from elsewhere
       'react/prop-types': 'off',
       // avoid unnecessary closing tags

package/package.json

@@ -1,7 +1,7 @@
 {
   "$schema": "https://json.schemastore.org/package",
   "name": "@foray1010/eslint-config",
-  "version": "10.0.5",
+  "version": "10.2.0",
   "homepage": "https://github.com/foray1010/common-presets/tree/master/packages/eslint-config#readme",
   "bugs": "https://github.com/foray1010/common-presets/issues",
   "repository": {
@@ -22,7 +22,7 @@
   "dependencies": {
     "@eslint-community/eslint-plugin-eslint-comments": "^3.2.1",
     "@eslint/js": "^8.36.0",
-    "@foray1010/common-presets-utils": "^7.0.1",
+    "@foray1010/common-presets-utils": "^7.0.2",
     "@typescript-eslint/eslint-plugin": "^5.56.0",
     "@typescript-eslint/parser": "^5.56.0",
     "confusing-browser-globals": "^1.0.10",
@@ -64,5 +64,5 @@
   "publishConfig": {
     "access": "public"
   },
-  "gitHead": "f74fc1c607b33237dd85fede7b8f76f6d298dac6"
+  "gitHead": "d413c514ed4534c77b3ca52b11b7bdd2084f0185"
 }