@madgex/eslint-config-madgex 2.2.0 → 2.4.0

package/CHANGELOG.md

@@ -3,6 +3,18 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+# [2.4.0](https://github.com/wiley/madgex-eslint-config-madgex/compare/@madgex/eslint-config-madgex@2.0.0...@madgex/eslint-config-madgex@2.4.0) (2026-02-12)
+
+### Bug Fixes
+
+- add some default ignores ([c5d58a7](https://github.com/wiley/madgex-eslint-config-madgex/commit/c5d58a79e9cb3a874a658556cd6690b3d5c86489))
+- package dependancies lock major ([3a78107](https://github.com/wiley/madgex-eslint-config-madgex/commit/3a78107b7628713a4a1efac55e2f57a602e4445a))
+
+### Features
+
+- add no-unused-vars & no-param-reassign allowances ([19a8f33](https://github.com/wiley/madgex-eslint-config-madgex/commit/19a8f334c7039bd9990545631a7450469758ddc4))
+- enforce extra eslint rules ([d00f939](https://github.com/wiley/madgex-eslint-config-madgex/commit/d00f939939891b118c18c8dfaf8d46f5844bb8c3))
+
 # [2.0.0](https://github.com/wiley/madgex-eslint-config-madgex/compare/@madgex/eslint-config-madgex@1.4.1...@madgex/eslint-config-madgex@2.0.0) (2024-04-18)
 
 -fix!: eslint 9, using modern eslint-plugin-n setup

package/README.md

@@ -10,28 +10,36 @@ npm install eslint@9 @madgex/eslint-config-madgex -save-dev
 
 ## VSCode extension
 
-Use VSCode Extension v3.0.5+ (you might need to switch to pre-release version)
+Use the official VSCode [ESLint Extension](https://marketplace.visualstudio.com/items?itemName=dbaeumer.vscode-eslint) v3.0.5+ (you might need to switch to pre-release version)
 
 ## Usage
 
 > ⚠️ Recomended to use only 1 type of source file in your repo, either `commonjs` or `module` (esm). `eslint` & `eslint-plugin-n` (this config depends on) has difficulty supporting both at the same time in a monorepo.
 
-```json
+`eslint-plugin-n` which is a part of our config, reads the package.json for node version, and what `type` of JS files you use (ESM or commonjs).
+
+```json5
 // package.json
-// ESM import/export modules, set to commonjs if you are using `require/module.exports`, you cant use both
-"type": "module",
 {
-  "engines": {
-    "node": ">=18"
-  }
+  ...
+  "type": "module", // "module" (ESM), or "commonjs" - you cant use both
+  {
+    "engines": {
+      "node": ">=18"
+    }
+  },
+  ...
 }
 ```
 
+`eslint-plugin-n` also recognises `.npmignore` files which will silence warnings of missing dependancies when they are installed as `devDependancies`.
+
 ### Node/Browser
 
 ```js
 // eslint.config.js
 import configMadgex from '@madgex/eslint-config-madgex';
+
 export default [...configMadgex];
 ```
 
@@ -43,6 +51,23 @@ config order matters.
 // eslint.config.js
 import configMadgex from '@madgex/eslint-config-madgex';
 import pluginVue from 'eslint-plugin-vue';
+// pluginVue.configs['flat/recommended'] has prettier conflicts
 
-export default [...configMadgex, ...pluginVue.configs['flat/recommended']];
+export default [...configMadgex, ...pluginVue.configs['flat/essential']];
 ```
+
+### Rules
+
+Notes regarding specific rules enforced in this config:
+
+- [`no-param-reassign`](https://eslint.org/docs/latest/rules/no-param-reassign) - Disallow reassigning function parameters, except for **`accu`** which is the accumulator in a `.reduce()` e.g.
+
+  ```js
+  Array.reduce((accu, key) => {
+    accu[key] = getVal(key);
+
+    return accu;
+  }, {});
+  ```
+
+- [`no-unused-vars`](https://eslint.org/docs/latest/rules/no-unused-vars) - Disallow unused variables, except for `request` and `h` as found on hapi.js route handlers.

package/index.js

@@ -37,7 +37,6 @@ module.exports = [
       // https://eslint.org/docs/latest/rules/
       'no-debugger': process.env.NODE_ENV === 'production' ? 2 : 0,
       'no-console': process.env.NODE_ENV === 'production' ? 2 : 0,
-      'no-param-reassign': ['error', { props: true }],
       eqeqeq: ['error', 'always', { null: 'ignore' }],
       'no-shadow': 'error',
       'no-lonely-if': 'error',
@@ -46,6 +45,14 @@ module.exports = [
       'default-case': 'error',
       'consistent-return': 'error',
       'func-names': 'warn',
+      'no-param-reassign': [
+        'error',
+        {
+          props: true,
+          ignorePropertyModificationsFor: ['accu'],
+        },
+      ],
+      'no-unused-vars': ['error', { argsIgnorePattern: 'request|h' }],
     },
   },
 ];

package/package.json

@@ -1,27 +1,27 @@
 {
   "name": "@madgex/eslint-config-madgex",
-  "version": "2.2.0",
+  "version": "2.4.0",
   "type": "commonjs",
   "description": "Madgex ESLint config - based on eslint-config-airbnb",
   "main": "index.js",
   "scripts": {
     "build": "echo 'build is not required' || exit 0",
-    "lint": "eslint --config index.js *.js",
-    "test": "echo \"Error: no test specified\" && exit 1"
+    "lint": "eslint --config index.js *.js"
   },
   "keywords": [],
   "author": "James Wragg <james.wragg@madgex.com> (https://madgex.com/)",
   "license": "MIT",
   "peerDependencies": {
-    "@eslint/js": ">=9",
-    "eslint": ">=9",
+    "@eslint/js": "^9.39.2",
+    "eslint": "^9.39.2",
     "eslint-config-prettier": "^9.1.0",
-    "eslint-plugin-n": "^17.2.1",
-    "eslint-plugin-prettier": "^5.1.3",
-    "globals": ">=15",
-    "prettier": ">=3"
+    "eslint-plugin-n": "^17.23.2",
+    "eslint-plugin-prettier": "^5.5.5",
+    "globals": "^15",
+    "prettier": "^3.8.1"
   },
   "engines": {
     "node": ">=18"
-  }
+  },
+  "gitHead": "0a2d2c26e7303483af175a958488648de0cd3696"
 }