@1stg/eslint-config 9.1.0 → 10.1.0

package/CHANGELOG.md

@@ -1,5 +1,26 @@
 # Change Log
 
+## 10.1.0
+
+### Minor Changes
+
+- [#377](https://github.com/1stG/configs/pull/377) [`ed8b7e5`](https://github.com/1stG/configs/commit/ed8b7e56fedf3c99b99b64631d675f67a7265825) Thanks [@JounQin](https://github.com/JounQin)! - feat: enable `eslint-plugin-node-dependencies` conditionally
+
+### Patch Changes
+
+- [#377](https://github.com/1stG/configs/pull/377) [`ed8b7e5`](https://github.com/1stG/configs/commit/ed8b7e56fedf3c99b99b64631d675f67a7265825) Thanks [@JounQin](https://github.com/JounQin)! - fix: declare node engines range correctly
+
+- Updated dependencies [[`ed8b7e5`](https://github.com/1stG/configs/commit/ed8b7e56fedf3c99b99b64631d675f67a7265825)]:
+  - @1stg/config@1.1.0
+
+## 10.0.0
+
+### Major Changes
+
+- [#371](https://github.com/1stG/configs/pull/371) [`3cbd4b7`](https://github.com/1stG/configs/commit/3cbd4b734f2297e58f4f917165924a903100d655) Thanks [@JounQin](https://github.com/JounQin)! - feat!: migrate `eslint-plugin-unicorn` to `eslint-plugin-unicorn-x`
+
+- [#372](https://github.com/1stG/configs/pull/372) [`f6e2755`](https://github.com/1stG/configs/commit/f6e27555b4ece8266418cfb371b0165b8749b42d) Thanks [@JounQin](https://github.com/JounQin)! - feat!: use `n.convertPath` setting for hashbang rule
+
 ## 9.1.0
 
 ### Minor Changes

package/_util.js

@@ -2,7 +2,7 @@
 
 import path from 'node:path'
 
-import { preferPrettier } from '@1stg/config'
+import { isCI, isEnvDisabled, isEnvEnabled, preferPrettier } from '@1stg/config'
 import {
   isMonorepo,
   isPkgAvailable,
@@ -25,7 +25,7 @@ if (isMonorepo()) {
       return acc
     }
     const { name, peerDependencies = {}, dependencies = {} } = pkgJson
-    // eslint-disable-next-line unicorn/prefer-spread
+    // eslint-disable-next-line unicorn-x/prefer-spread
     return acc.concat(
       name,
       Object.keys(peerDependencies),
@@ -78,3 +78,9 @@ export const nonSourceRules = /** @type {const} */ ({
   'n/no-extraneous-require': 0,
   'n/no-unsupported-features/es-builtins': 0,
 })
+
+const ESLINT_NODE_DEPS_ENV = 'ESLINT_NODE_DEPS'
+
+export const isEslintNodeDepsEnabled = isCI
+  ? !isEnvDisabled(ESLINT_NODE_DEPS_ENV)
+  : isEnvEnabled(ESLINT_NODE_DEPS_ENV)

package/angular.js

@@ -43,6 +43,6 @@ export const angular = tseslint.config(
   },
   {
     files: ['**/*inline-template-*.component.html'],
-    rules: { 'unicorn/no-empty-file': 0 },
+    rules: { 'unicorn-x/no-empty-file': 0 },
   },
 )

package/base.js

@@ -2,6 +2,7 @@
 
 import eslint from '@eslint/js'
 import eslintCommentsConfigs from '@eslint-community/eslint-plugin-eslint-comments/configs'
+import { isPkgAvailable, tryRequirePkg } from '@pkgr/utils'
 import { createTypeScriptImportResolver } from 'eslint-import-resolver-typescript'
 import css from 'eslint-plugin-css'
 import importX from 'eslint-plugin-import-x'
@@ -11,11 +12,12 @@ import promise from 'eslint-plugin-promise'
 import regexp from 'eslint-plugin-regexp'
 import simpleImportSort from 'eslint-plugin-simple-import-sort'
 import sonarjs from 'eslint-plugin-sonarjs'
-import unicorn from 'eslint-plugin-unicorn'
+import unicornX from 'eslint-plugin-unicorn-x'
 import globals from 'globals'
 import tseslint from 'typescript-eslint'
 
 import {
+  isEslintNodeDepsEnabled,
   isTsAvailable,
   isWebpackAvailable,
   magicNumbers,
@@ -32,8 +34,15 @@ export const base = tseslint.config([
   promise.configs['flat/recommended'],
   regexp.configs['flat/recommended'],
   sonarjs.configs.recommended,
-  unicorn.configs.recommended,
+  unicornX.configs.recommended,
   prettierExtends,
+  isEslintNodeDepsEnabled && isPkgAvailable('eslint-plugin-node-dependencies')
+    ? [
+        /** @type {typeof import('eslint-plugin-node-dependencies')} */ (
+          tryRequirePkg('eslint-plugin-node-dependencies')
+        ).configs['flat/recommended'],
+      ]
+    : [],
   {
     name: '@1stg/node',
     files: ['**/*.{cjs,cts}'],
@@ -52,7 +61,13 @@ export const base = tseslint.config([
         'node_modules/@d-ts',
         'node_modules/@types',
       ],
-      'import-x/resolver-next': [createTypeScriptImportResolver()],
+      'import-x/resolver-next': createTypeScriptImportResolver(),
+      n: {
+        convertPath: {
+          '**/bin.*ts': [String.raw`^(src/)?bin\.([cm]?)ts$`, 'lib/bin.$2js'],
+          '**/cli.*ts': [String.raw`^(src/)?cli\.([cm]?)ts$`, 'lib/cli.$2js'],
+        },
+      },
     },
     languageOptions: {
       sourceType: 'module',
@@ -116,9 +131,9 @@ export const base = tseslint.config([
         },
       ],
       'no-negated-condition': 2,
-      'no-process-exit': 0, // suspended by unicorn/no-process-exit
+      'no-process-exit': 0, // suspended by unicorn-x/no-process-exit
 
-      // The following rules are duplicate with `eslint-plugin-import`
+      // The following rules are duplicate with `eslint-plugin-import-x`
       'n/no-extraneous-import': 0,
       'n/no-extraneous-require': 0,
       'n/no-missing-import': 0,
@@ -131,15 +146,15 @@ export const base = tseslint.config([
       'prefer-const': [2, { destructuring: 'all' }],
       'prefer-object-spread': 2,
       'simple-import-sort/exports': 2,
-      'unicorn/catch-error-name': [
+      'unicorn-x/catch-error-name': [
         2,
         {
           name: 'error',
           ignore: ['^e(rr)?$'],
         },
       ],
-      'unicorn/consistent-function-scoping': 0,
-      'unicorn/filename-case': [
+      'unicorn-x/consistent-function-scoping': 0,
+      'unicorn-x/filename-case': [
         2,
         {
           cases: {
@@ -150,13 +165,13 @@ export const base = tseslint.config([
           ignore: [/^[A-Z](([\dA-Z]+_)*[\dA-Z]+)?\.(mdx?|ya?ml)$/],
         },
       ],
-      'unicorn/no-array-reduce': 0,
-      'unicorn/no-null': 0,
-      'unicorn/no-unreadable-array-destructuring': 0, // conflict with `no-unused-vars`
-      'unicorn/prefer-module': 0,
-      'unicorn/prefer-object-from-entries': 0,
-      'unicorn/prevent-abbreviations': 0,
-      'unicorn/prefer-export-from': [2, { ignoreUsedVariables: true }],
+      'unicorn-x/no-array-reduce': 0,
+      'unicorn-x/no-null': 0,
+      'unicorn-x/no-unreadable-array-destructuring': 0, // conflict with `no-unused-vars`
+      'unicorn-x/prefer-module': 0,
+      'unicorn-x/prefer-object-from-entries': 0,
+      'unicorn-x/prevent-abbreviations': 0,
+      'unicorn-x/prefer-export-from': [2, { ignoreUsedVariables: true }],
     },
   },
   {

package/loose.js

@@ -28,12 +28,12 @@ export const loose = tseslint.config([
       'no-negated-condition': 0, // not auto-fixable
       'promise/always-return': 0,
       'promise/catch-or-return': 0,
-      'unicorn/explicit-length-check': 0,
-      'unicorn/no-array-for-each': 0,
-      'unicorn/no-fn-reference-in-iterator': 0,
-      'unicorn/prefer-number-properties': 0,
-      'unicorn/prefer-prototype-methods': 0,
-      'unicorn/prefer-spread': 0,
+      'unicorn-x/explicit-length-check': 0,
+      'unicorn-x/no-array-for-each': 0,
+      'unicorn-x/no-fn-reference-in-iterator': 0,
+      'unicorn-x/prefer-number-properties': 0,
+      'unicorn-x/prefer-prototype-methods': 0,
+      'unicorn-x/prefer-spread': 0,
     },
   },
 ])

package/overrides.js

@@ -26,11 +26,6 @@ import { resolveSettings, tsBase } from './ts-base.js'
 
 export const ts = tseslint.config(
   tsBase,
-  {
-    name: '@1stg/cli',
-    files: ['**/{bin,cli}.ts'],
-    rules: { 'n/hashbang': 0 },
-  },
   {
     name: '@1stg/code-block',
     files: ['**/*.{md,mdx}/**/*.{cts,mts,ts,tsx}'],

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@1stg/eslint-config",
-  "version": "9.1.0",
+  "version": "10.1.0",
   "type": "module",
   "description": "Personal but Shareable ESLint Configuration for all 1stG.me projects",
   "repository": "git+https://github.com/1stG/configs.git",
@@ -9,7 +9,7 @@
   "funding": "https://opencollective.com/configs",
   "license": "MIT",
   "engines": {
-    "node": ">=18.18"
+    "node": "^18.20.0 || ^20.19.0 || >=22.12.0"
   },
   "main": "index.js",
   "exports": {
@@ -29,6 +29,7 @@
     "angular-eslint-template-parser": "^0.1.0",
     "eslint": ">=8.0.0",
     "eslint-plugin-jest": "^28.0.0",
+    "eslint-plugin-node-dependencies": "^1.0.0",
     "eslint-plugin-react-hooks": "^5.0.0",
     "eslint-plugin-react-refresh": "^0.4.0",
     "eslint-plugin-vue": "^10.0.0"
@@ -55,6 +56,9 @@
     "eslint-plugin-jest": {
       "optional": true
     },
+    "eslint-plugin-node-dependencies": {
+      "optional": true
+    },
     "eslint-plugin-react-hooks": {
       "optional": true
     },
@@ -66,31 +70,31 @@
     }
   },
   "dependencies": {
-    "@1stg/config": "^1.0.5",
+    "@1stg/config": "^1.1.0",
     "@eslint-community/eslint-plugin-eslint-comments": "^4.5.0",
-    "@eslint/js": "^9.27.0",
-    "@pkgr/utils": "^3.1.2",
-    "@typescript-eslint/eslint-plugin": "^8.32.1",
-    "@typescript-eslint/parser": "^8.32.1",
+    "@eslint/js": "^9.28.0",
+    "@pkgr/utils": "^4.0.0",
+    "@typescript-eslint/eslint-plugin": "^8.33.0",
+    "@typescript-eslint/parser": "^8.33.0",
     "eslint-config-prettier": "^10.1.5",
-    "eslint-import-resolver-typescript": "^4.3.5",
+    "eslint-import-resolver-typescript": "^4.4.2",
     "eslint-plugin-css": "^0.11.0",
-    "eslint-plugin-import-x": "^4.12.2",
-    "eslint-plugin-jsdoc": "^50.6.17",
+    "eslint-plugin-import-x": "^4.15.0",
+    "eslint-plugin-jsdoc": "^50.7.0",
     "eslint-plugin-jsonc": "^2.20.1",
     "eslint-plugin-markup": "^1.0.1",
     "eslint-plugin-mdx": "^3.4.2",
     "eslint-plugin-n": "^17.18.0",
-    "eslint-plugin-prettier": "^5.4.0",
+    "eslint-plugin-prettier": "^5.4.1",
     "eslint-plugin-promise": "^7.2.1",
     "eslint-plugin-regexp": "^2.7.0",
     "eslint-plugin-simple-import-sort": "^12.1.1",
     "eslint-plugin-sonarjs": "^3.0.2",
     "eslint-plugin-toml": "^0.12.0",
-    "eslint-plugin-unicorn": "^59.0.1",
+    "eslint-plugin-unicorn-x": "^1.0.8",
     "eslint-plugin-yml": "^1.18.0",
-    "globals": "^16.1.0",
-    "typescript-eslint": "^8.32.1"
+    "globals": "^16.2.0",
+    "typescript-eslint": "^8.33.0"
   },
   "publishConfig": {
     "access": "public"

package/ts-base.js

@@ -129,10 +129,10 @@ export const tsBase = tseslint.config({
     'promise/always-return': 0,
     'promise/catch-or-return': 0,
     // ts itself has guaranteed it
-    'unicorn/no-array-callback-reference': 0,
+    'unicorn-x/no-array-callback-reference': 0,
     // covered by @typescript-eslint/no-extraneous-class
-    'unicorn/no-static-only-class': 0,
+    'unicorn-x/no-static-only-class': 0,
     // covered by @typescript-eslint/no-this-alias
-    'unicorn/no-this-assignment': 0,
+    'unicorn-x/no-this-assignment': 0,
   },
 })