fix: fixed react version finding, added 2 new logger settings

Signed-off-by: prisis <d.bannert@anolilab.de>

Author: prisis

packages/eslint-config/README.md

@@ -259,6 +259,42 @@ To disable the info, set the value to `false`.
 }
 ```
 
+#### info_on_testing_library_framework
+
+Type: `boolean`
+
+Default: `undefined`
+
+To disable the info, set the value to `false`.
+
+```json
+{
+    "anolilab": {
+        "eslint-config": {
+            "info_on_testing_library_framework": false
+        }
+    }
+}
+```
+
+#### info_on_found_react_version
+
+Type: `boolean`
+
+Default: `undefined`
+
+To disable the info, set the value to `false`.
+
+```json
+{
+    "anolilab": {
+        "eslint-config": {
+            "info_on_found_react_version": false
+        }
+    }
+}
+```
+
 ### Let [Prettier](https://prettier.io/) handle style-related rules
 
 Prettier is a code formatting tool that offers fewer options but is more professional than the style-related rules in ESLint.

packages/eslint-config/src/config/plugins/react.ts

@@ -1,5 +1,6 @@
 // @see https://github.com/yannickcr/eslint-plugin-react
 import { hasAnyDep, pkg } from "@anolilab/package-json-utils";
+import { getPackageSubProperty } from "@anolilab/package-json-utils/src";
 import type { Linter } from "eslint";
 import findUp from "find-up";
 import { env } from "node:process";
@@ -67,8 +68,17 @@ const hasJsxRuntime = (() => {
     return global.hasAnolilabEsLintConfigReactRuntimePath;
 })();
 
-const packageDependencies = pkg?.dependencies ?? {};
-const reactDependency = (packageDependencies?.["react"] ?? "").split(".");
+// eslint-disable-next-line @typescript-eslint/no-unsafe-assignment,@typescript-eslint/no-unsafe-call
+let reactVersion: string | undefined = getPackageSubProperty<string>("dependencies")("react")
+
+if (reactVersion === undefined) {
+    // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment,@typescript-eslint/no-unsafe-call
+    reactVersion = getPackageSubProperty<string | undefined>("devDependencies")("react");
+}
+
+if (reactVersion !== undefined && anolilabEslintConfig?.["info_on_found_react_version"] !== false) {
+    consoleLog(`\n@anolilab/eslint-config found the version ${reactVersion} of react in your dependencies, this version ${reactVersion} will be used to setup the "eslint-plugin-react"\n`);
+}
 
 const config: Linter.Config = {
     overrides: [
@@ -97,7 +107,7 @@ const config: Linter.Config = {
                     // The default value is "detect". Automatic detection works by loading the entire React library
                     // into the linter's process, which is inefficient. It is recommended to specify the version
                     // explicity.
-                    version: reactDependency[0] ? `${reactDependency[0]}.${reactDependency[1] ?? "0"}` : "detect",
+                    version: reactVersion ?? "detect",
                 },
                 propWrapperFunctions: [
                     "forbidExtraProps", // https://www.npmjs.com/package/airbnb-prop-types

packages/eslint-config/src/config/plugins/testing-library.ts

@@ -1,4 +1,4 @@
-import { hasAnyDep } from "@anolilab/package-json-utils";
+import { hasAnyDep, pkg } from "@anolilab/package-json-utils";
 import type { Linter } from "eslint";
 
 import { consolePlugin } from "../../utils/loggers";
@@ -9,8 +9,18 @@ if (hasAnyDep(["react"])) {
     ruleset = "react";
 }
 
+let anolilabEslintConfig: { [key: string]: boolean | undefined } = {};
+
+if (pkg) {
+    // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment,@typescript-eslint/no-unsafe-member-access
+    anolilabEslintConfig = pkg?.["anolilab"]?.["eslint-config"];
+}
+
+// Workaround VS Code trying to run this file twice!
 if (!global.hasAnolilabEsLintTestConfigLoaded) {
-    consolePlugin(`testing-library/${ruleset}`);
+    if (anolilabEslintConfig?.["info_on_testing_library_framework"] !== false) {
+        consolePlugin(`testing-library/${ruleset}`);
+    }
 
     global.hasAnolilabEsLintTestConfigLoaded = true;
 }
@@ -19,19 +29,19 @@ const config: Linter.Config = {
     extends: [`plugin:testing-library/${ruleset}`],
     rules: {
         // Not included in jest/recommended
-        "testing-library/await-fire-event": 0,
-        "testing-library/consistent-data-testid": 0,
-        "testing-library/no-debug": 0,
-        "testing-library/no-dom-import": 0,
-        "testing-library/no-manual-cleanup": 0,
-        "testing-library/no-render-in-setup": 0,
-        "testing-library/no-await-sync-events": 0,
-        "testing-library/no-wait-for-empty-callback": 0,
-        "testing-library/no-wait-for-snapshot": 0,
-        "testing-library/prefer-explicit-assert": 0,
-        "testing-library/prefer-presence-queries": 0,
-        "testing-library/prefer-screen-queries": 0,
-        "testing-library/prefer-wait-for": 0,
+        "testing-library/await-fire-event": "off",
+        "testing-library/consistent-data-testid": "off",
+        "testing-library/no-debug": "off",
+        "testing-library/no-dom-import": "off",
+        "testing-library/no-manual-cleanup": "off",
+        "testing-library/no-render-in-setup": "off",
+        "testing-library/no-await-sync-events": "off",
+        "testing-library/no-wait-for-empty-callback": "off",
+        "testing-library/no-wait-for-snapshot": "off",
+        "testing-library/prefer-explicit-assert": "off",
+        "testing-library/prefer-presence-queries": "off",
+        "testing-library/prefer-screen-queries": "off",
+        "testing-library/prefer-wait-for": "off",
     },
 };