Merge pull request #819 from ioBroker/copilot/add-check-for-obsolete-config-files

Add W5048: warn about obsolete eslint/prettier config files when using @iobroker/eslint-config

Author: mcm1957

README.md

@@ -26,6 +26,7 @@ Example:
 	### **WORK IN PROGRESS**
 -->
 ### **WORK IN PROGRESS**
+- (@copilot) Added [W5048] check: warns about obsolete eslint/prettier config files (`.eslintignore`, `.eslintrc.json`, `.prettierignore`, `.prettierrc.js`, `.prettierrc.json`) when `@iobroker/eslint-config` is used as a devDependency.
 - (@copilot) Added [W4047]: warn when adapter is found in the latest repository but not yet available in the stable repository. Related to [#820].
 - (@copilot) Added `[E9506]`: error when an i18n directory is explicitly excluded by `.npmignore`, which would cause translations to be missing from the npm package.
 - (@copilot) Added `[E9507]`: error when an i18n directory is present in the repository but not covered by the `"files"` field in `package.json`, which would cause translations to be missing from the npm package.

lib/M5000_Code.js

@@ -899,6 +899,24 @@ async function checkCode(context) {
         }
     }
 
+    // W5048: Check for obsolete eslint/prettier config files when @iobroker/eslint-config is used
+    if (context.packageJson.devDependencies && context.packageJson.devDependencies['@iobroker/eslint-config']) {
+        const obsoleteFiles = [
+            '/.eslintignore',
+            '/.eslintrc.json',
+            '/.prettierignore',
+            '/.prettierrc.js',
+            '/.prettierrc.json',
+        ];
+        for (const file of obsoleteFiles) {
+            if (context.filesList.includes(file)) {
+                context.warnings.push(
+                    `[W5048] "${file.slice(1)}" is most likely obsolete when using "@iobroker/eslint-config". Please remove it.`,
+                );
+            }
+        }
+    }
+
     if (context.packageJson.devDependencies && context.packageJson.devDependencies['@alcalzone/release-script']) {
         const version = context.packageJson.devDependencies['@alcalzone/release-script'];
         if (compareVersions.compareVersions(version, '3.0.0') >= 0) {
@@ -1450,6 +1468,7 @@ exports.checkCode = checkCode;
 // [E5045] "/admin/jsonTab.json[5]" not found, but tab support is declared
 // [W5046] "admin/jsonConfig.json" or "admin/jsonConfig.json5" found but "common.adminUI.config" is not set to "json" in io-package.json. Please add "common.adminUI": { "config": "json" }.
 // [W5047] "${file}" is most likely outdated since jsonConfig is used. Please consider removing it.
+// [W5048] "${file}" is most likely obsolete when using "@iobroker/eslint-config". Please remove it.
 // [W5515] Schema validation skipped for "${configFile}" due to parse error: ${e.message}
 
 // [5500 - ] see M5500_jsonConfig.js