@asd14/eslint-config 9.2.0 → 10.0.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@asd14/eslint-config",
-  "version": "9.2.0",
+  "version": "10.0.0",
   "description": "JavaScript ESLint rules",
   "homepage": "https://github.com/asd-xiv/eslint-config",
   "main": "targets/node.js",
@@ -34,7 +34,7 @@
   "scripts": {
     "----UTIL": "",
     "prepare": "git config --local core.hooksPath .githook",
-    "setup": "npm ci --also=dev --prefer-offline --no-audit",
+    "setup": "npm ci --also=dev --cache .npm --prefer-offline --no-audit",
     "update": "ncu --interactive",
     "postupdate": "npm audit fix",
     "format": "npm run lint.js -- --fix",
@@ -46,17 +46,18 @@
     "release": "semantic-release"
   },
   "lint-staged": {
-    "*.{js,json}": "eslint --quiet",
+    "*.{js}": "eslint --quiet",
+    "*.{json}": "prettier",
     "*.md": "markdownlint --ignore CHANGELOG.md",
     ".circleci/config.yml": "circleci config validate"
   },
   "dependencies": {
-    "@typescript-eslint/eslint-plugin": "^5.51.0",
-    "@typescript-eslint/parser": "^5.51.0",
-    "eslint-config-prettier": "^8.6.0",
+    "@typescript-eslint/eslint-plugin": "^5.57.0",
+    "@typescript-eslint/parser": "^5.57.0",
+    "eslint-config-prettier": "^8.8.0",
     "eslint-plugin-html": "^7.1.0",
     "eslint-plugin-import": "^2.27.5",
-    "eslint-plugin-jsdoc": "^39.8.0",
+    "eslint-plugin-jsdoc": "^40.1.1",
     "eslint-plugin-json": "^3.1.0",
     "eslint-plugin-no-inferred-method-name": "^2.0.0",
     "eslint-plugin-no-relative-import-paths": "^1.5.2",
@@ -64,23 +65,23 @@
     "eslint-plugin-promise": "^6.1.1",
     "eslint-plugin-react": "^7.32.2",
     "eslint-plugin-react-hooks": "^4.6.0",
-    "eslint-plugin-unicorn": "^45.0.2"
+    "eslint-plugin-unicorn": "^46.0.0"
   },
   "devDependencies": {
-    "@commitlint/cli": "^17.4.2",
-    "@commitlint/config-conventional": "^17.4.2",
-    "@semantic-release/changelog": "^6.0.2",
+    "@commitlint/cli": "^17.5.1",
+    "@commitlint/config-conventional": "^17.4.4",
+    "@semantic-release/changelog": "^6.0.3",
     "@semantic-release/git": "^10.0.1",
-    "lint-staged": "^13.1.1",
+    "lint-staged": "^13.2.0",
     "markdownlint-cli": "^0.33.0",
-    "npm-check-updates": "^16.7.4",
-    "semantic-release": "^20.1.0"
+    "npm-check-updates": "^16.9.0",
+    "semantic-release": "^21.0.0"
   },
   "peerDependencies": {
-    "eslint": "^8.33.0",
-    "prettier": "^2.8.3"
+    "eslint": "^8.37.0",
+    "prettier": "^2.8.7"
   },
   "engines": {
-    "node": ">=16.0.0"
+    "node": ">=18"
   }
 }

package/rules/best-practices.js

@@ -0,0 +1,11 @@
+module.exports = {
+  rules: {
+    // Encourages use of dot notation whenever possible
+    "dot-notation": [
+      "error",
+      {
+        allowKeywords: true,
+      },
+    ],
+  },
+}

package/rules/comments.js

@@ -1,5 +1,3 @@
-/* eslint-env node */
-
 module.exports = {
   rules: {
     /*

package/rules/errors.js

@@ -1,5 +1,3 @@
-/* eslint-env node */
-
 module.exports = {
   rules: {
     /*

package/rules/es6.js

@@ -1,5 +1,3 @@
-/* eslint-env node */
-
 module.exports = {
   rules: {
     // Require braces in arrow function body

package/rules/html.js

@@ -1,5 +1,3 @@
-/* eslint-env node */
-
 module.exports = {
   plugins: ["html"],
   settings: {

package/rules/import-ts.js

@@ -1,5 +1,3 @@
-/* eslint-env node */
-
 module.exports = {
   extends: ["plugin:import/typescript"],
 

package/rules/import.js

@@ -1,5 +1,3 @@
-/* eslint-env node */
-
 module.exports = {
   plugins: ["import", "no-relative-import-paths"],
   extends: ["plugin:import/recommended"],

package/rules/jsdoc-ts.js

@@ -1,5 +1,3 @@
-/* eslint-env node */
-
 module.exports = {
   rules: {
     "valid-jsdoc": "off",

package/rules/jsdoc.js

@@ -1,5 +1,3 @@
-/* eslint-env node */
-
 /*
  * JSDoc is a JavaScript API documentation generator. It uses
  * specially-formatted comments inside of code to generate API

package/rules/json.js

@@ -1,5 +1,3 @@
-/* eslint-env node */
-
 module.exports = {
   plugins: ["json"],
 

package/rules/node.js

@@ -1,5 +1,3 @@
-/* eslint-env node */
-
 module.exports = {
   rules: {
     // Enforce return after a callback

package/rules/prettier.js

@@ -1,5 +1,3 @@
-/* eslint-env node */
-
 module.exports = {
   extends: ["plugin:prettier/recommended"],
 

package/rules/promise.js

@@ -1,5 +1,3 @@
-/* eslint-env node */
-
 module.exports = {
   plugins: ["promise"],
 

package/rules/react.js

@@ -1,5 +1,3 @@
-/* eslint-env node */
-
 module.exports = {
   plugins: ["react", "react-hooks"],
 

package/rules/style.js

@@ -1,5 +1,3 @@
-/* eslint-env node */
-
 module.exports = {
   /*
    * Disabled by prettier

package/rules/unicorn.js

@@ -1,5 +1,3 @@
-/* eslint-env node */
-
 module.exports = {
   plugins: ["unicorn"],
 

package/rules/variables.js

@@ -1,5 +1,3 @@
-/* eslint-env node */
-
 module.exports = {
   rules: {
     // Enforce or disallow variable initializations at definition

package/targets/node-esm.js

@@ -1,18 +1,14 @@
-/* eslint-env node */
-
 module.exports = {
   parserOptions: {
     ecmaVersion: 2021,
     sourceType: "module",
   },
-
   env: {
     node: true,
   },
-
   extends: [
     "../rules/json",
-    "../rules/best.practices",
+    "../rules/best-practices",
     "../rules/comments",
     "../rules/errors",
     "../rules/es6",
@@ -24,8 +20,6 @@ module.exports = {
     "../rules/variables",
     "../rules/prettier",
   ].map(require.resolve),
-
   plugins: ["no-inferred-method-name"],
-
   settings: {},
 }

package/targets/node-ts.js

@@ -1,22 +1,17 @@
-/* eslint-env node */
-
 module.exports = {
   parser: "@typescript-eslint/parser",
-
   parserOptions: {
     ecmaVersion: 2021,
     sourceType: "module",
   },
-
   env: {
     node: true,
     es6: true,
   },
-
   extends: [
     ...[
       "../rules/json",
-      "../rules/best.practices",
+      "../rules/best-practices",
       "../rules/comments",
       "../rules/errors",
       "../rules/es6",
@@ -31,12 +26,9 @@ module.exports = {
       "../rules/variables",
       "../rules/prettier",
     ].map(require.resolve),
-
     "plugin:@typescript-eslint/recommended",
   ],
-
   plugins: ["no-inferred-method-name"],
-
   rules: {
     "@typescript-eslint/no-var-requires": "off",
     "@typescript-eslint/ban-ts-comment": "warn",

package/targets/node.js

@@ -1,18 +1,14 @@
-/* eslint-env node */
-
 module.exports = {
   parserOptions: {
     ecmaVersion: 2021,
     sourceType: "module",
   },
-
   env: {
     node: true,
   },
-
   extends: [
     "../rules/json",
-    "../rules/best.practices",
+    "../rules/best-practices",
     "../rules/comments",
     "../rules/errors",
     "../rules/es6",
@@ -25,9 +21,7 @@ module.exports = {
     "../rules/variables",
     "../rules/prettier",
   ].map(require.resolve),
-
   plugins: ["no-inferred-method-name"],
-
   settings: {
     "import/resolver": {
       node: {

package/targets/react-ts.js

@@ -1,6 +1,5 @@
-/* eslint-env node */
-
 module.exports = {
+  parser: "@typescript-eslint/parser",
   parserOptions: {
     ecmaVersion: 2021,
     sourceType: "module",
@@ -8,17 +7,15 @@ module.exports = {
       jsx: true,
     },
   },
-
   env: {
     browser: true,
     es6: true,
     node: false,
   },
-
   extends: [
     ...[
       "../rules/json",
-      "../rules/best.practices",
+      "../rules/best-practices",
       "../rules/comments",
       "../rules/errors",
       "../rules/es6",
@@ -37,9 +34,7 @@ module.exports = {
 
     "plugin:@typescript-eslint/recommended",
   ],
-
   plugins: ["no-inferred-method-name"],
-
   rules: {
     "react/prop-types": "off",
 
@@ -70,7 +65,6 @@ module.exports = {
       },
     ],
   },
-
   settings: {
     "react": {
       version: "detect",

package/targets/react.js

@@ -1,5 +1,3 @@
-/* eslint-env node */
-
 module.exports = {
   parserOptions: {
     ecmaVersion: 2021,
@@ -19,7 +17,7 @@ module.exports = {
 
   extends: [
     "../rules/json",
-    "../rules/best.practices",
+    "../rules/best-practices",
     "../rules/comments",
     "../rules/errors",
     "../rules/es6",

package/rules/best.practices.js

@@ -1,200 +0,0 @@
-/* eslint-env node */
-
-module.exports = {
-  rules: {
-    // Enforces getter/setter pairs in objects
-    "accessor-pairs": 0,
-
-    // Treat var statements as if they were block scoped
-    "block-scoped-var": "error",
-
-    // Specify the maximum cyclomatic complexity allowed in a program
-    "complexity": [0, 5],
-
-    // Require return statements to either always or never specify values
-    "consistent-return": "warn",
-
-    // Require default case in switch statements
-    "default-case": "error",
-
-    // Encourages use of dot notation whenever possible
-    "dot-notation": [
-      "error",
-      {
-        allowKeywords: true,
-      },
-    ],
-
-    // Require the use of === and !==
-    "eqeqeq": "error",
-
-    // Make sure for-in loops have an if statement
-    "guard-for-in": "error",
-
-    // Disallow the use of alert, confirm, and prompt
-    "no-alert": 1,
-
-    // Disallow use of arguments.caller or arguments.callee
-    "no-caller": "error",
-
-    // Disallow lexical declarations in case clauses
-    "no-case-declarations": "error",
-
-    /*
-     * Disallow division operators explicitly at beginning of regular
-     * expression
-     */
-    "no-div-regex": 0,
-
-    // Disallow else after a return in an if
-    "no-else-return": [
-      "error",
-      {
-        allowElseIf: false,
-      },
-    ],
-
-    // Disallow use of empty destructuring patterns
-    "no-empty-pattern": 0,
-
-    // Disallow comparisons to null without a type-checking operator
-    "no-eq-null": 0,
-
-    // Disallow use of eval()
-    "no-eval": "error",
-
-    // Disallow adding to native types
-    "no-extend-native": "error",
-
-    // Disallow unnecessary function binding
-    "no-extra-bind": "error",
-
-    // Disallow fallthrough of case statements
-    "no-fallthrough": "error",
-
-    // Disallow the type conversions with shorter notations
-    "no-implicit-coercion": 0,
-
-    // Disallow use of eval()-like methods
-    "no-implied-eval": "error",
-
-    // Disallow this keywords outside of classes or class-like objects
-    "no-invalid-this": 0,
-
-    // Disallow usage of __iterator__ property
-    "no-iterator": "error",
-
-    // Disallow use of labels for anything other then loops and switches
-    "no-label": 0,
-
-    // Disallow use of labeled statements
-    "no-labels": "error",
-
-    // Disallow unnecessary nested blocks
-    "no-lone-blocks": "warn",
-
-    // Disallow creation of functions within loops
-    "no-loop-func": "error",
-
-    // Disallow the use of magic numbers
-    "no-magic-numbers": 0,
-
-    // Disallow use of multiline strings
-    "no-multi-str": "error",
-
-    // Disallow reassignments of native objects
-    "no-native-reassign": "error",
-
-    /*
-     * Disallow use of new operator when not part of the assignment or
-     * Comparison
-     */
-    "no-new": "error",
-
-    // Disallow use of new operator for Function object
-    "no-new-func": "error",
-
-    // Disallows creating new instances of String,Number, and Boolean
-    "no-new-wrappers": "error",
-
-    // Disallow use of (old style) octal literals
-    "no-octal": "error",
-
-    /*
-     * Disallow use of octal escape sequences in string literals, such as
-     * Var foo = "Copyright \251";
-     */
-    "no-octal-escape": "error",
-
-    // Disallow reassignment of function parameters
-    "no-param-reassign": [
-      "error",
-      {
-        props: false,
-      },
-    ],
-
-    // Disallow use of process.env
-    "no-process-env": 0,
-
-    // Disallow usage of __proto__ property
-    "no-proto": "error",
-
-    // Disallow declaring the same variable more then once
-    "no-redeclare": "error",
-
-    // Disallow use of assignment in return statement
-    "no-return-assign": "error",
-
-    // Disallow use of `javascript:` urls.
-    "no-script-url": "error",
-
-    // Disallow comparisons where both sides are exactly the same
-    "no-self-compare": "error",
-
-    // Disallow use of comma operator
-    "no-sequences": "error",
-
-    // Restrict what can be thrown as an exception
-    "no-throw-literal": "error",
-
-    // Disallow usage of expressions in statement position
-    "no-unused-expressions": [
-      "error",
-      {
-        allowShortCircuit: true,
-        allowTernary: true,
-      },
-    ],
-
-    // Disallow unnecessary .call() and .apply()
-    "no-useless-call": 0,
-
-    // Disallow unnecessary concatenation of literals or template literals
-    "no-useless-concat": 0,
-
-    // Disallow use of void operator
-    "no-void": 0,
-
-    // Disallow usage of configurable warning terms in comments: e.g. todo
-    "no-warning-comments": [
-      0,
-      {
-        location: "start",
-        terms: ["todo", "fixme", "xxx"],
-      },
-    ],
-
-    // Disallow use of the with statement
-    "no-with": "error",
-
-    // Require use of the second argument for parseInt()
-    "radix": "error",
-
-    // Requires to declare all vars on top of their containing scope
-    "vars-on-top": "error",
-
-    // Require or disallow Yoda conditions
-    "yoda": 0,
-  },
-}