@foray1010/eslint-config 7.7.2 → 7.8.0

package/CHANGELOG.md

@@ -3,6 +3,19 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+## [7.8.0](https://github.com/foray1010/common-presets/compare/@foray1010/eslint-config@7.7.3...@foray1010/eslint-config@7.8.0) (2022-10-03)
+
+### Features
+
+- **eslint-config:** prefer Number static properties over global ones ([532df43](https://github.com/foray1010/common-presets/commit/532df43987f464ea3f3cfe1e5507b1077e14bced))
+- **eslint-config:** re-enable functional/prefer-tacit ([9f312cc](https://github.com/foray1010/common-presets/commit/9f312cc3ac05429a969f271c46e02d915bf44e41))
+
+## [7.7.3](https://github.com/foray1010/common-presets/compare/@foray1010/eslint-config@7.7.2...@foray1010/eslint-config@7.7.3) (2022-10-02)
+
+### Bug Fixes
+
+- **eslint-config:** prefer isomorphic global variables ([fa098bc](https://github.com/foray1010/common-presets/commit/fa098bc9890d86facd9c592f1ed42515bc94d2af))
+
 ## [7.7.2](https://github.com/foray1010/common-presets/compare/@foray1010/eslint-config@7.7.1...@foray1010/eslint-config@7.7.2) (2022-10-01)
 
 ### Bug Fixes

package/package.json

@@ -1,7 +1,7 @@
 {
   "$schema": "https://json.schemastore.org/package",
   "name": "@foray1010/eslint-config",
-  "version": "7.7.2",
+  "version": "7.8.0",
   "homepage": "https://github.com/foray1010/common-presets/tree/master/packages/eslint-config#readme",
   "bugs": "https://github.com/foray1010/common-presets/issues",
   "repository": {
@@ -30,7 +30,7 @@
     "eslint-config-prettier": "^8.3.0",
     "eslint-plugin-compat": "^4.0.0",
     "eslint-plugin-eslint-comments": "^3.2.0",
-    "eslint-plugin-functional": "^4.4.0",
+    "eslint-plugin-functional": "^4.4.1",
     "eslint-plugin-import": "^2.22.1",
     "eslint-plugin-jest": "^27.0.4",
     "eslint-plugin-jest-dom": "^4.0.0",
@@ -40,7 +40,8 @@
     "eslint-plugin-react": "^7.31.8",
     "eslint-plugin-react-hooks": "^4.2.0",
     "eslint-plugin-simple-import-sort": "^8.0.0",
-    "eslint-plugin-testing-library": "^5.7.0"
+    "eslint-plugin-testing-library": "^5.7.0",
+    "eslint-plugin-unicorn": "^42.0.0"
   },
   "devDependencies": {
     "@types/confusing-browser-globals": "1.0.0",
@@ -56,5 +57,5 @@
   "publishConfig": {
     "access": "public"
   },
-  "gitHead": "1fae3d076965b166094c89e62957cdb81e78338e"
+  "gitHead": "728b8351b1895f7af4b9b476461f8a3fd6f34a60"
 }

package/presets/base.js

@@ -64,6 +64,7 @@ module.exports = {
     'eslint-plugin-import',
     'eslint-plugin-jsdoc',
     'eslint-plugin-prettier',
+    'eslint-plugin-unicorn',
   ],
   env: {
     // should align with parserOptions.ecmaVersion
@@ -126,6 +127,8 @@ module.exports = {
     'jsdoc/valid-types': 'off',
     // avoid assigning anonymous function to object key which is harder to trace when debug
     'object-shorthand': ['error', 'always'],
+    // prefer Number static properties over global ones
+    'unicorn/prefer-number-properties': 'error',
   },
   overrides: [
     {
@@ -272,8 +275,7 @@ module.exports = {
                 },
               ],
               // forbid unnecessary callback wrapper
-              // disabled due to https://github.com/eslint-functional/eslint-plugin-functional/issues/486
-              // 'functional/prefer-tacit': 'error',
+              'functional/prefer-tacit': 'error',
               // @typescript-eslint/eslint-plugin suggests to disable it: https://github.com/typescript-eslint/typescript-eslint/blob/2588e9ea55f78352fdd6ae92a306135aabb49a1a/docs/linting/TROUBLESHOOTING.md#i-get-errors-from-the-no-undef-rule-about-global-variables-not-being-defined-even-though-there-are-no-typescript-errors
               // It is disabled in recommended config but re-enabled here to enforce a subset of global variables that supported by both node.js and browsers
               'no-undef': 'error',

package/presets/node.js

@@ -22,10 +22,20 @@ module.exports = {
     'n/no-unpublished-bin': 'error',
     // disallow unsupported Node.js built-in APIs on the specified version
     'n/no-unsupported-features/node-builtins': 'error',
-    // prefer `import { Buffer } from 'node:buffer'`
+    // prefer `import { Buffer } from 'node:buffer'` as it is not isomorphic
     'n/prefer-global/buffer': ['error', 'never'],
-    // prefer `import process from 'node:process'`
+    // prefer global `console` to be isomorphic
+    'n/prefer-global/console': ['error', 'always'],
+    // prefer `import process from 'node:process'` as it is not isomorphic
     'n/prefer-global/process': ['error', 'never'],
+    // prefer global `TextDecoder` to be isomorphic
+    'n/prefer-global/text-decoder': ['error', 'always'],
+    // prefer global `TextEncoder` to be isomorphic
+    'n/prefer-global/text-encoder': ['error', 'always'],
+    // prefer global `URLSearchParams` to be isomorphic
+    'n/prefer-global/url-search-params': ['error', 'always'],
+    // prefer global `URL` to be isomorphic
+    'n/prefer-global/url': ['error', 'always'],
     // make `process.exit()` expressions the same code path as `throw`
     'n/process-exit-as-throw': 'error',
     // enforce shebang on the entry bin file