@boundaries/eslint-plugin 5.3.0 → 5.3.1

package/dist/Settings/Validations.js

@@ -18,7 +18,6 @@ const Helpers_1 = require("./Helpers");
 const Settings_1 = require("./Settings");
 const Settings_types_1 = require("./Settings.types");
 const { TYPES, ALIAS, ELEMENTS, DEPENDENCY_NODES, ADDITIONAL_DEPENDENCY_NODES, VALID_DEPENDENCY_NODE_KINDS, DEFAULT_DEPENDENCY_NODES, VALID_MODES, } = Settings_types_1.SETTINGS;
-const invalidMatchers = [];
 const DEFAULT_MATCHER_OPTIONS = {
     type: "object",
 };
@@ -108,23 +107,50 @@ function rulesOptionsSchema(options = {}) {
     ];
 }
 function isValidElementTypesMatcher(matcher, settings) {
-    const matcherToCheck = (0, Common_1.isArray)(matcher) ? matcher[0] : matcher;
-    const typeMatcherToCheck = (0, Common_1.isString)(matcherToCheck)
-        ? matcherToCheck
-        : matcherToCheck.type;
-    return (!typeMatcherToCheck ||
-        (typeMatcherToCheck &&
-            micromatch_1.default.some(settings.elementTypeNames, typeMatcherToCheck)));
+    const matcherToCheck = (0, Common_1.isArray)(matcher)
+        ? matcher[0]
+        : matcher;
+    return (!matcherToCheck ||
+        (matcherToCheck &&
+            micromatch_1.default.some(settings.elementTypeNames, matcherToCheck)));
+}
+/**
+ * Checks if the value is a single matcher with options (tuple of [string, object])
+ */
+function isSingleMatcherWithOptions(value) {
+    return ((0, Common_1.isArray)(value) &&
+        value.length === 2 &&
+        (0, Common_1.isString)(value[0]) &&
+        (0, Common_1.isObject)(value[1]));
 }
 // TODO: Remove this validation. Selectors should not be limited to element types defined in settings when using selector objects
 function validateElementTypesMatcher(elementsMatcher, settings) {
-    const [matcher] = (0, Common_1.isArray)(elementsMatcher)
-        ? elementsMatcher
-        : [elementsMatcher];
-    if (!invalidMatchers.includes(matcher) &&
-        !isValidElementTypesMatcher(matcher, settings)) {
-        invalidMatchers.push(matcher);
-        (0, Debug_1.warnOnce)(`Option '${matcher}' does not match any element type from '${ELEMENTS}' setting`);
+    // Handle empty array case
+    if ((0, Common_1.isArray)(elementsMatcher) && elementsMatcher.length === 0) {
+        return;
+    }
+    // Determine if it's a single matcher or an array of matchers
+    let matcher;
+    if ((0, Common_1.isString)(elementsMatcher)) {
+        matcher = elementsMatcher;
+    }
+    else if (isSingleMatcherWithOptions(elementsMatcher)) {
+        // It's a single matcher with options: ["type", { option: value }]
+        matcher = elementsMatcher;
+    }
+    else if ((0, Common_1.isArray)(elementsMatcher)) {
+        // It's an array of matchers: ["helpers", "components"] or [["helpers", {...}], "components"]
+        // NOTE: Validate only the first matcher. It is wrong, but we don't want to impact performance, and anyway it was already validating only the first one.
+        // In next major version, validation will be removed, because schema validation will handle it.
+        matcher = elementsMatcher[0];
+    }
+    else {
+        (0, Debug_1.warnOnce)(`Option is not a valid elements selector: '${JSON.stringify(elementsMatcher)}'`);
+        return;
+    }
+    // Validate the matcher
+    if (!isValidElementTypesMatcher(matcher, settings)) {
+        (0, Debug_1.warnOnce)(`Option '${JSON.stringify(matcher)}' does not match any element type from '${ELEMENTS}' setting`);
     }
 }
 function isValidElementAssigner(element) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@boundaries/eslint-plugin",
-  "version": "5.3.0",
+  "version": "5.3.1",
   "description": "Eslint plugin checking architecture boundaries between elements",
   "keywords": [
     "eslint",
@@ -53,14 +53,16 @@
     "eslint-import-resolver-node": "0.3.9",
     "eslint-module-utils": "2.12.1",
     "micromatch": "4.0.8",
-    "@boundaries/elements": "1.1.1"
+    "@boundaries/elements": "1.1.2"
   },
   "engines": {
     "node": ">=18.18"
   },
   "devDependencies": {
     "@types/estree": "1.0.8",
-    "@types/micromatch": "4.0.9"
+    "@types/micromatch": "4.0.9",
+    "eslint-plugin-eslint-plugin": "7.2.0",
+    "eslint-plugin-local-rules": "3.0.2"
   },
   "scripts": {
     "eslint": "eslint",