@etchteam/eslint-config 3.1.7 → 3.2.0

package/CHANGELOG.md

@@ -1,6 +1,10 @@
-## <small>3.1.7 (2026-04-16)</small>
+## 3.2.0 (2026-04-20)
 
-* Merge pull request #563 from etchteam/dependabot/npm_and_yarn/npm_and_yarn-da47bb892b ([ef34c14](https://github.com/etchteam/eslint/commit/ef34c14)), closes [#563](https://github.com/etchteam/eslint/issues/563)
-* Merge pull request #564 from etchteam/fix/webc-browser-globals ([b10f47a](https://github.com/etchteam/eslint/commit/b10f47a)), closes [#564](https://github.com/etchteam/eslint/issues/564)
-* fix: add browser globals to webc virtual JS files ([b487521](https://github.com/etchteam/eslint/commit/b487521))
-* chore: Bump hono in the npm_and_yarn group across 1 directory ([1d5c097](https://github.com/etchteam/eslint/commit/1d5c097))
+* Merge pull request #565 from etchteam/dependabot/npm_and_yarn/next/eslint-plugin-next-16.2.4 ([7e658c6](https://github.com/etchteam/eslint/commit/7e658c6)), closes [#565](https://github.com/etchteam/eslint/issues/565)
+* Merge pull request #566 from etchteam/dependabot/npm_and_yarn/darraghor/eslint-plugin-nestjs-typed-7 ([330438f](https://github.com/etchteam/eslint/commit/330438f)), closes [#566](https://github.com/etchteam/eslint/issues/566)
+* Merge pull request #567 from etchteam/dependabot/npm_and_yarn/eslint-plugin-react-hooks-7.1.0 ([5939065](https://github.com/etchteam/eslint/commit/5939065)), closes [#567](https://github.com/etchteam/eslint/issues/567)
+* Merge pull request #568 from etchteam/feat/add-sonarjs-plugin ([2ebd744](https://github.com/etchteam/eslint/commit/2ebd744)), closes [#568](https://github.com/etchteam/eslint/issues/568)
+* feat: add eslint-plugin-sonarjs to base config ([a111ccc](https://github.com/etchteam/eslint/commit/a111ccc))
+* chore: Bump @darraghor/eslint-plugin-nestjs-typed from 7.1.29 to 7.1.30 ([2d938f5](https://github.com/etchteam/eslint/commit/2d938f5))
+* chore: Bump @next/eslint-plugin-next from 16.2.3 to 16.2.4 ([eb7a485](https://github.com/etchteam/eslint/commit/eb7a485))
+* chore: Bump eslint-plugin-react-hooks from 7.0.1 to 7.1.0 ([607d6c6](https://github.com/etchteam/eslint/commit/607d6c6))

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@etchteam/eslint-config",
-  "version": "3.1.7",
+  "version": "3.2.0",
   "description": "Etch's standard eslint config",
   "type": "module",
   "main": "src/index.mjs",
@@ -63,6 +63,7 @@
     "eslint-plugin-json": "^4.0.1",
     "eslint-plugin-prettier": "^5.5.0",
     "eslint-plugin-security": "^4.0.0",
+    "eslint-plugin-sonarjs": "^3.0.7",
     "eslint-plugin-unused-imports": "^4.1.3",
     "eslint-plugin-yml": "^3.0.0",
     "eslint-plugin-you-dont-need-lodash-underscore": "^6.14.0",
@@ -77,8 +78,8 @@
     "eslint-plugin-jsx-a11y": ">=6.0.0",
     "eslint-plugin-lit": ">=2.0.0",
     "eslint-plugin-react": ">=7.0.0",
-    "eslint-plugin-storybook": ">=10.0.0",
     "eslint-plugin-react-hooks": ">=5.0.0",
+    "eslint-plugin-storybook": ">=10.0.0",
     "prettier": ">=3.0.0",
     "typescript": ">=4.8.4"
   },

package/src/base.mjs

@@ -2,12 +2,37 @@ import js from '@eslint/js';
 import importPlugin from 'eslint-plugin-import';
 import prettier from 'eslint-plugin-prettier/recommended';
 import security from 'eslint-plugin-security';
+import sonarjs from 'eslint-plugin-sonarjs';
 import unusedImports from 'eslint-plugin-unused-imports';
 import youDontNeedLodash from 'eslint-plugin-you-dont-need-lodash-underscore';
 import tseslint from 'typescript-eslint';
 
 import { fixupPluginRules } from './utils/fixup.mjs';
 
+const JS_FILES = ['**/*.{js,mjs,cjs,jsx,ts,mts,cts,tsx}'];
+
+/**
+ * Catch some common sonar issues locally, this is JS/TS only and not a 1:1 mirror
+ * of Sonar's web-UI Quality Profile. Everything is downgraded to 'warn' because it
+ * doesn't recognise when issues have been marked as safe in the Sonar web UI.
+ *
+ * Rule updates:
+ * - `file-header` is off because we never us a header template and it warns on every file.
+ * - `sonarjs/arrow-function-convention` is off because it conflicts with prettier.
+ */
+const sonarConfig = {
+  ...sonarjs.configs.recommended,
+  files: JS_FILES,
+  rules: {
+    ...Object.fromEntries(
+      Object.keys(sonarjs.configs.recommended.rules).map((r) => [r, 'warn']),
+    ),
+    'sonarjs/file-header': 'off',
+    'sonarjs/arrow-function-convention': 'off',
+    'sonarjs/declarations-in-global-scope': 'off',
+  },
+};
+
 /**
  * Base ESLint configuration for all projects.
  * Includes: JavaScript, TypeScript, security, import ordering, prettier, and common rules.
@@ -17,8 +42,9 @@ export default [
   js.configs.recommended,
   ...tseslint.configs.recommended,
   security.configs.recommended,
+  sonarConfig,
   {
-    files: ['**/*.{js,mjs,cjs,jsx,ts,mts,cts,tsx}'],
+    files: JS_FILES,
     plugins: {
       'unused-imports': unusedImports,
       import: fixupPluginRules(importPlugin),