eslint-plugin-mgw-eslint-rules 2.3.41 → 2.3.43

package/dist/index.js

@@ -3261,8 +3261,8 @@ var require_RuleCreator = __commonJS({
     exports2.RuleCreator = RuleCreator;
     var applyDefault_1 = require_applyDefault();
     function RuleCreator(urlCreator) {
-      return function createNamedRule({ meta, name, ...rule12 }) {
-        const ruleWithDocs = createRule12({
+      return function createNamedRule({ meta, name, ...rule13 }) {
+        const ruleWithDocs = createRule13({
           meta: {
             ...meta,
             docs: {
@@ -3271,12 +3271,12 @@ var require_RuleCreator = __commonJS({
             }
           },
           name,
-          ...rule12
+          ...rule13
         });
         return ruleWithDocs;
       };
     }
-    function createRule12({
+    function createRule13({
       create,
       // Keep accepting deprecated defaultOptions for backward compatibility.
       // eslint-disable-next-line @typescript-eslint/no-deprecated
@@ -3296,7 +3296,7 @@ var require_RuleCreator = __commonJS({
       };
     }
     RuleCreator.withoutDocs = function withoutDocs(args) {
-      return createRule12(args);
+      return createRule13(args);
     };
   }
 });
@@ -20621,13 +20621,13 @@ ${ctx.toSourceMapGenerator(sourceUrl, headerLines).toJsComment()}`;
       }
       _scopeKeyframesRelatedCss(cssText, scopeSelector) {
         const unscopedKeyframesSet = /* @__PURE__ */ new Set();
-        const scopedKeyframesCssText = processRules(cssText, (rule12) => this._scopeLocalKeyframeDeclarations(rule12, scopeSelector, unscopedKeyframesSet));
-        return processRules(scopedKeyframesCssText, (rule12) => this._scopeAnimationRule(rule12, scopeSelector, unscopedKeyframesSet));
+        const scopedKeyframesCssText = processRules(cssText, (rule13) => this._scopeLocalKeyframeDeclarations(rule13, scopeSelector, unscopedKeyframesSet));
+        return processRules(scopedKeyframesCssText, (rule13) => this._scopeAnimationRule(rule13, scopeSelector, unscopedKeyframesSet));
       }
-      _scopeLocalKeyframeDeclarations(rule12, scopeSelector, unscopedKeyframesSet) {
+      _scopeLocalKeyframeDeclarations(rule13, scopeSelector, unscopedKeyframesSet) {
         return {
-          ...rule12,
-          selector: rule12.selector.replace(/(^@(?:-webkit-)?keyframes(?:\s+))(['"]?)(.+)\2(\s*)$/, (_, start, quote, keyframeName, endSpaces) => {
+          ...rule13,
+          selector: rule13.selector.replace(/(^@(?:-webkit-)?keyframes(?:\s+))(['"]?)(.+)\2(\s*)$/, (_, start, quote, keyframeName, endSpaces) => {
             unscopedKeyframesSet.add(unescapeQuotes(keyframeName, quote));
             return `${start}${quote}${scopeSelector}_${keyframeName}${quote}${endSpaces}`;
           })
@@ -20640,8 +20640,8 @@ ${ctx.toSourceMapGenerator(sourceUrl, headerLines).toJsComment()}`;
         });
       }
       _animationDeclarationKeyframesRe = /(^|\s+|,)(?:(?:(['"])((?:\\\\|\\\2|(?!\2).)+)\2)|(-?[A-Za-z][\w\-]*))(?=[,\s]|$)/g;
-      _scopeAnimationRule(rule12, scopeSelector, unscopedKeyframesSet) {
-        let content = rule12.content.replace(/((?:^|\s+|;)(?:-webkit-)?animation\s*:\s*),*([^;]+)/g, (_, start, animationDeclarations) => start + animationDeclarations.replace(this._animationDeclarationKeyframesRe, (original, leadingSpaces, quote = "", quotedName, nonQuotedName) => {
+      _scopeAnimationRule(rule13, scopeSelector, unscopedKeyframesSet) {
+        let content = rule13.content.replace(/((?:^|\s+|;)(?:-webkit-)?animation\s*:\s*),*([^;]+)/g, (_, start, animationDeclarations) => start + animationDeclarations.replace(this._animationDeclarationKeyframesRe, (original, leadingSpaces, quote = "", quotedName, nonQuotedName) => {
           if (quotedName) {
             return `${leadingSpaces}${this._scopeAnimationKeyframe(`${quote}${quotedName}${quote}`, scopeSelector, unscopedKeyframesSet)}`;
           } else {
@@ -20650,7 +20650,7 @@ ${ctx.toSourceMapGenerator(sourceUrl, headerLines).toJsComment()}`;
         }));
         content = content.replace(/((?:^|\s+|;)(?:-webkit-)?animation-name(?:\s*):(?:\s*))([^;]+)/g, (_match, start, commaSeparatedKeyframes) => `${start}${commaSeparatedKeyframes.split(",").map((keyframe) => this._scopeAnimationKeyframe(keyframe, scopeSelector, unscopedKeyframesSet)).join(",")}`);
         return {
-          ...rule12,
+          ...rule13,
           content
         };
       }
@@ -20661,8 +20661,8 @@ ${ctx.toSourceMapGenerator(sourceUrl, headerLines).toJsComment()}`;
       }
       _insertPolyfillRulesInCssText(cssText) {
         return cssText.replace(_cssContentRuleRe, (...m) => {
-          const rule12 = m[0].replace(m[1], "").replace(m[2], "");
-          return m[4] + rule12;
+          const rule13 = m[0].replace(m[1], "").replace(m[2], "");
+          return m[4] + rule13;
         });
       }
       _scopeCssText(cssText, scopeSelector, hostSelector) {
@@ -20683,8 +20683,8 @@ ${ctx.toSourceMapGenerator(sourceUrl, headerLines).toJsComment()}`;
         let m;
         _cssContentUnscopedRuleRe.lastIndex = 0;
         while ((m = _cssContentUnscopedRuleRe.exec(cssText)) !== null) {
-          const rule12 = m[0].replace(m[2], "").replace(m[1], m[4]);
-          r += rule12 + "\n\n";
+          const rule13 = m[0].replace(m[2], "").replace(m[1], m[4]);
+          r += rule13 + "\n\n";
         }
         return r;
       }
@@ -20769,28 +20769,28 @@ ${ctx.toSourceMapGenerator(sourceUrl, headerLines).toJsComment()}`;
         return _shadowDOMSelectorsRe.reduce((result, pattern) => result.replace(pattern, " "), cssText);
       }
       _scopeSelectors(cssText, scopeSelector, hostSelector) {
-        return processRules(cssText, (rule12) => {
-          let selector = rule12.selector;
-          let content = rule12.content;
-          if (rule12.selector[0] !== "@") {
+        return processRules(cssText, (rule13) => {
+          let selector = rule13.selector;
+          let content = rule13.content;
+          if (rule13.selector[0] !== "@") {
             selector = this._scopeSelector({
               selector,
               scopeSelector,
               hostSelector,
               isParentSelector: true
             });
-          } else if (scopedAtRuleIdentifiers.some((atRule) => rule12.selector.startsWith(atRule))) {
-            content = this._scopeSelectors(rule12.content, scopeSelector, hostSelector);
-          } else if (rule12.selector.startsWith("@font-face") || rule12.selector.startsWith("@page")) {
-            content = this._stripScopingSelectors(rule12.content);
+          } else if (scopedAtRuleIdentifiers.some((atRule) => rule13.selector.startsWith(atRule))) {
+            content = this._scopeSelectors(rule13.content, scopeSelector, hostSelector);
+          } else if (rule13.selector.startsWith("@font-face") || rule13.selector.startsWith("@page")) {
+            content = this._stripScopingSelectors(rule13.content);
           }
           return new CssRule(selector, content);
         });
       }
       _stripScopingSelectors(cssText) {
-        return processRules(cssText, (rule12) => {
-          const selector = rule12.selector.replace(_shadowDeepSelectors, " ").replace(_polyfillHostNoCombinatorRe, " ");
-          return new CssRule(selector, rule12.content);
+        return processRules(cssText, (rule13) => {
+          const selector = rule13.selector.replace(_shadowDeepSelectors, " ").replace(_polyfillHostNoCombinatorRe, " ");
+          return new CssRule(selector, rule13.content);
         });
       }
       _safeSelector;
@@ -21051,8 +21051,8 @@ ${ctx.toSourceMapGenerator(sourceUrl, headerLines).toJsComment()}`;
           suffix = suffix.substring(BLOCK_PLACEHOLDER.length + 1);
           contentPrefix = "{";
         }
-        const rule12 = ruleCallback(new CssRule(selector, content));
-        return `${m[1]}${rule12.selector}${m[3]}${contentPrefix}${rule12.content}${suffix}`;
+        const rule13 = ruleCallback(new CssRule(selector, content));
+        return `${m[1]}${rule13.selector}${m[3]}${contentPrefix}${rule13.content}${suffix}`;
       });
       return unescapeInStrings(escapedResult);
     }
@@ -44309,13 +44309,101 @@ var rule6 = createRule6({
   }
 });
 
-// src/rules/prefer-const-enum.ts
+// src/rules/no-restricted-variable-names.ts
 var import_utils7 = __toESM(require_dist4());
-var RULE_NAME7 = "prefer-const-enum";
-var DOCS_RULES2 = "https://github.com/developpement-megao/mgw-eslint-rules/tree/main/docs/rules";
+var RULE_NAME7 = "no-restricted-variable-names";
+var RESTRICTED_NAMES_DEFAULT = ["data", "list", "obj", "id", "code", "name"];
+var DOCS_RULES2 = "https://github.com/developpement-megao/mgw-eslint-rules/blob/main/docs/rules";
 var createRule7 = import_utils7.ESLintUtils.RuleCreator((name) => `${DOCS_RULES2}/${name}.md`);
 var rule7 = createRule7({
   name: RULE_NAME7,
+  meta: {
+    type: "suggestion",
+    docs: {
+      description: "Disallow the use of generic or vague variable names (e.g., `data`, `list`, `obj`, `id`, `code`, `name`).",
+      recommended: true
+    },
+    messages: {
+      restrictedName: "Avoid using `{{ name }}` as a variable name. Use a more descriptive name instead."
+    },
+    schema: [
+      {
+        type: "object",
+        properties: {
+          restrictedNames: {
+            type: "array",
+            items: {
+              type: "string"
+            },
+            description: "List of restricted variable names."
+          }
+        },
+        additionalProperties: false
+      }
+    ]
+  },
+  defaultOptions: [
+    {
+      restrictedNames: RESTRICTED_NAMES_DEFAULT
+    }
+  ],
+  create(context, [{ restrictedNames }]) {
+    function checkNode(node) {
+      const name = node.name;
+      if (!name || name === "_") {
+        return;
+      }
+      const restrictedNamesDef = restrictedNames || RESTRICTED_NAMES_DEFAULT;
+      if (restrictedNamesDef.includes(name)) {
+        context.report({
+          node,
+          messageId: "restrictedName",
+          data: { name }
+        });
+      }
+    }
+    return {
+      // Déclarations de variables : const x = signal(0)
+      VariableDeclarator(node) {
+        if (node.id.type === "Identifier") {
+          checkNode(node.id);
+        }
+      },
+      // Pour les paramètres de fonction
+      FunctionDeclaration(node) {
+        node.params.forEach((param) => {
+          if (param.type === "Identifier") {
+            checkNode(param);
+          }
+        });
+      },
+      // Propriétés de classe : count = signal(0)
+      PropertyDefinition(node) {
+        if (node.key.type === "Identifier") {
+          checkNode(node.key);
+        }
+      },
+      // Pour les paramètres des méthodes des classes
+      MethodDefinition(node) {
+        if (node.value.type === "FunctionExpression") {
+          node.value.params.forEach((param) => {
+            if (param.type === "Identifier") {
+              checkNode(param);
+            }
+          });
+        }
+      }
+    };
+  }
+});
+
+// src/rules/prefer-const-enum.ts
+var import_utils8 = __toESM(require_dist4());
+var RULE_NAME8 = "prefer-const-enum";
+var DOCS_RULES3 = "https://github.com/developpement-megao/mgw-eslint-rules/tree/main/docs/rules";
+var createRule8 = import_utils8.ESLintUtils.RuleCreator((name) => `${DOCS_RULES3}/${name}.md`);
+var rule8 = createRule8({
+  name: RULE_NAME8,
   meta: {
     type: "problem",
     docs: {
@@ -44363,12 +44451,12 @@ var rule7 = createRule7({
 });
 
 // src/rules/reactive-naming-convention.ts
-var import_utils8 = __toESM(require_dist4());
+var import_utils9 = __toESM(require_dist4());
 var import_eslint_utils3 = __toESM(require_eslint_utils3());
-var RULE_NAME8 = "reactive-naming-convention";
-var createRule8 = import_utils8.ESLintUtils.RuleCreator((name) => `https://github.com/developpement-megao/mgw-eslint-rules/docs/rules/${name}.md`);
-var rule8 = createRule8({
-  name: RULE_NAME8,
+var RULE_NAME9 = "reactive-naming-convention";
+var createRule9 = import_utils9.ESLintUtils.RuleCreator((name) => `https://github.com/developpement-megao/mgw-eslint-rules/docs/rules/${name}.md`);
+var rule9 = createRule9({
+  name: RULE_NAME9,
   meta: {
     type: "suggestion",
     fixable: "code",
@@ -44464,14 +44552,14 @@ var rule8 = createRule8({
 });
 
 // src/rules/service-class-suffix.ts
-var import_utils9 = __toESM(require_dist6());
-var import_utils10 = __toESM(require_dist4());
-var RULE_NAME9 = "service-class-suffix";
+var import_utils10 = __toESM(require_dist6());
+var import_utils11 = __toESM(require_dist4());
+var RULE_NAME10 = "service-class-suffix";
 var SUFFIXES_DEFAULT = ["Service"];
-var DOCS_RULES3 = "https://github.com/developpement-megao/mgw-eslint-rules/blob/main/docs/rules";
-var createRule9 = import_utils10.ESLintUtils.RuleCreator((name) => `${DOCS_RULES3}/${name}.md`);
-var rule9 = createRule9({
-  name: RULE_NAME9,
+var DOCS_RULES4 = "https://github.com/developpement-megao/mgw-eslint-rules/blob/main/docs/rules";
+var createRule10 = import_utils11.ESLintUtils.RuleCreator((name) => `${DOCS_RULES4}/${name}.md`);
+var rule10 = createRule10({
+  name: RULE_NAME10,
   meta: {
     type: "suggestion",
     docs: {
@@ -44503,7 +44591,7 @@ var rule9 = createRule9({
   ],
   create(context, [{ suffixes }]) {
     return {
-      [import_utils9.Selectors.INJECTABLE_CLASS_DECORATOR](node) {
+      [import_utils10.Selectors.INJECTABLE_CLASS_DECORATOR](node) {
         const classDeclaration = node.parent;
         if (!classDeclaration.id) {
           return;
@@ -44526,11 +44614,11 @@ var rule9 = createRule9({
 });
 
 // src/rules/template/attributes-naming-convention.ts
-var import_utils11 = __toESM(require_dist6());
-var import_utils12 = __toESM(require_dist4());
+var import_utils12 = __toESM(require_dist6());
+var import_utils13 = __toESM(require_dist4());
 var NAMING_CONVENTIONS = ["camelCase", "strictCamelCase", "PascalCase", "StrictPascalCase", "snake_case", "UPPER_CASE", "kebab-case"];
 var ATTRIBUTE_CONVENTION_POSSIBLE_VALUES = [...NAMING_CONVENTIONS];
-var RULE_NAME10 = "template/attributes-naming-convention";
+var RULE_NAME11 = "template/attributes-naming-convention";
 var conventionPatterns = {
   camelCase: /^[a-z][a-zA-Z0-9]*$/,
   strictCamelCase: /^(?!.*[A-Z]{2})[a-z][a-zA-Z0-9]*$/,
@@ -44540,10 +44628,10 @@ var conventionPatterns = {
   UPPER_CASE: /^[A-Z][A-Z0-9]*(?:_[A-Z0-9]+)*$/,
   "kebab-case": /^[a-z][a-z0-9]*(?:-[a-z0-9]+)*$/
 };
-var DOCS_RULES4 = "https://github.com/developpement-megao/mgw-eslint-rules/tree/main/docs/rules";
-var createRule10 = import_utils12.ESLintUtils.RuleCreator((name) => `${DOCS_RULES4}/${name}.md`);
-var rule10 = createRule10({
-  name: RULE_NAME10,
+var DOCS_RULES5 = "https://github.com/developpement-megao/mgw-eslint-rules/tree/main/docs/rules";
+var createRule11 = import_utils13.ESLintUtils.RuleCreator((name) => `${DOCS_RULES5}/${name}.md`);
+var rule11 = createRule11({
+  name: RULE_NAME11,
   meta: {
     type: "suggestion",
     docs: {
@@ -44602,7 +44690,7 @@ var rule10 = createRule10({
     }
   ],
   create(context, [options]) {
-    const parserServices = (0, import_utils11.getTemplateParserServices)(context);
+    const parserServices = (0, import_utils12.getTemplateParserServices)(context);
     const listeAttributes = new Map(
       Object.entries(options).map(([a, n]) => {
         const listeNommage = typeof n === "string" ? [n] : n;
@@ -44651,13 +44739,13 @@ var rule10 = createRule10({
 });
 
 // src/rules/template/prefer-class-style-binding.ts
-var import_utils13 = __toESM(require_dist6());
-var import_utils14 = __toESM(require_dist4());
-var RULE_NAME11 = "template/prefer-class-style-binding";
-var DOCS_RULES5 = "https://github.com/developpement-megao/mgw-eslint-rules/tree/main/docs/rules";
-var createRule11 = import_utils14.ESLintUtils.RuleCreator((name) => `${DOCS_RULES5}/${name}.md`);
-var rule11 = createRule11({
-  name: RULE_NAME11,
+var import_utils14 = __toESM(require_dist6());
+var import_utils15 = __toESM(require_dist4());
+var RULE_NAME12 = "template/prefer-class-style-binding";
+var DOCS_RULES6 = "https://github.com/developpement-megao/mgw-eslint-rules/tree/main/docs/rules";
+var createRule12 = import_utils15.ESLintUtils.RuleCreator((name) => `${DOCS_RULES6}/${name}.md`);
+var rule12 = createRule12({
+  name: RULE_NAME12,
   meta: {
     type: "suggestion",
     docs: {
@@ -44672,7 +44760,7 @@ var rule11 = createRule11({
   },
   defaultOptions: [],
   create(context) {
-    const parserServices = (0, import_utils13.getTemplateParserServices)(context);
+    const parserServices = (0, import_utils14.getTemplateParserServices)(context);
     return {
       // Détecte les attributs ngClass et ngStyle et inputs [ngClass] et [ngStyle] dans les templates
       Element(node) {
@@ -44693,17 +44781,18 @@ var rule11 = createRule11({
 
 // src/index.ts
 var rules = {
-  [RULE_NAME7]: rule7,
+  [RULE_NAME8]: rule8,
   [RULE_NAME5]: rule5,
   [RULE_NAME3]: rule3,
   [RULE_NAME4]: rule4,
   [RULE_NAME]: rule,
+  [RULE_NAME12]: rule12,
+  [RULE_NAME10]: rule10,
+  [RULE_NAME2]: rule2,
   [RULE_NAME11]: rule11,
   [RULE_NAME9]: rule9,
-  [RULE_NAME2]: rule2,
-  [RULE_NAME10]: rule10,
-  [RULE_NAME8]: rule8,
-  [RULE_NAME6]: rule6
+  [RULE_NAME6]: rule6,
+  [RULE_NAME7]: rule7
 };
 // Annotate the CommonJS export names for ESM import in node:
 0 && (module.exports = {