@jsenv/eslint-config-relax 1.6.12 → 1.7.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jsenv/eslint-config-relax",
-  "version": "1.6.12",
+  "version": "1.7.1",
   "description": "A pleaseant ESLint configuration for any project",
   "repository": {
     "type": "git",
@@ -24,14 +24,15 @@
   ],
   "scripts": {},
   "dependencies": {
-    "@babel/eslint-parser": "7.28.0",
-    "@jsenv/eslint-import-resolver": "8.4.1",
+    "@babel/eslint-parser": "7.28.4",
+    "@jsenv/eslint-import-resolver": "8.4.2",
+    "@jsenv/eslint-plugin": "1.0.0",
     "@jsenv/urls": "2.9.1",
     "eslint-plugin-html": "8.1.3",
     "eslint-plugin-import-x": "4.16.1",
     "eslint-plugin-react": "7.37.5",
-    "eslint-plugin-regexp": "2.9.0",
-    "globals": "16.3.0"
+    "eslint-plugin-regexp": "2.10.0",
+    "globals": "16.4.0"
   },
   "devDependencies": {},
   "engines": {

package/src/eslint_config_relax.js

@@ -1,4 +1,5 @@
 import babelParser from "@babel/eslint-parser";
+import jsenvPlugin from "@jsenv/eslint-plugin";
 import { urlToRelativeUrl } from "@jsenv/urls";
 import html from "eslint-plugin-html";
 import pluginImportX from "eslint-plugin-import-x";
@@ -35,6 +36,8 @@ export const eslintConfigRelax = ({
   prettierSortImport,
   jsxPragmaAuto = false,
   preact,
+  reactVersion = "detect",
+  reactVersionForPreact = "19.2.0",
   importResolutionLogLevel,
 
   browserFiles = [],
@@ -122,6 +125,15 @@ export const eslintConfigRelax = ({
         "import-x/ignore": ["node_modules/playwright/"],
       },
     },
+    // jsenv plugin
+    {
+      plugins: {
+        jsenv: jsenvPlugin,
+      },
+      rules: {
+        "jsenv/no-unknown-params": ["off"], // Default: only report likely typos
+      },
+    },
     // node "module" files
     {
       files: ["**/*.js", "**/*.mjs", "**/*.jsx"],
@@ -180,7 +192,7 @@ export const eslintConfigRelax = ({
       },
       settings: {
         react: {
-          version: "detect",
+          version: preact ? reactVersionForPreact : reactVersion,
         },
       },
       rules: {

package/src/explicit_globals.js

@@ -4,6 +4,7 @@ export const explicitGlobals = {
   blur: "off",
   btoa: "off",
   caches: "off",
+  constructor: "off",
   close: "off",
   closed: "off",
   crypto: "off",

package/src/rules_relax.js

@@ -51,7 +51,7 @@ export const rulesRelax = {
   "consistent-return": ["error"],
   "constructor-super": ["error"],
   "curly": ["error"],
-  "default-case": ["error"],
+  "default-case": ["off"], // not alawys needed so it's annoying
   "dot-location": ["error", "property"],
   "dot-notation": ["off"],
   "eol-last": ["off"],
@@ -225,7 +225,19 @@ export const rulesRelax = {
   "no-unused-expressions": ["error"],
   "no-unused-labels": ["off"], // https://gist.github.com/getify/706e5e10822a298375da40f9cc1fa295
   "no-unused-private-class-members": ["error"],
-  "no-unused-vars": ["error"],
+  "no-unused-vars": [
+    "error",
+    {
+      // it's better to ensure rest siblings are reported
+      // because otherwise it's easy to miss that they get stripped of param forwaring
+      // which happens a lot in rect components
+      // const { importantParam, ...rest} =props;
+      // <OtherComponent {...rest} />
+      // If "importantParam" is not used, rest will not contains it anymore
+      // which can lead to bugs if OtherComponent expect "importantParam" to be there
+      ignoreRestSiblings: false,
+    },
+  ],
   "no-use-before-define": [
     "error",
     /*