eslint-plugin-mgw-eslint-rules 2.3.26 → 2.3.28

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, ...rule10 }) {
-        const ruleWithDocs = createRule10({
+      return function createNamedRule({ meta, name, ...rule11 }) {
+        const ruleWithDocs = createRule11({
           meta: {
             ...meta,
             docs: {
@@ -3271,12 +3271,12 @@ var require_RuleCreator = __commonJS({
             }
           },
           name,
-          ...rule10
+          ...rule11
         });
         return ruleWithDocs;
       };
     }
-    function createRule10({
+    function createRule11({
       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 createRule10(args);
+      return createRule11(args);
     };
   }
 });
@@ -20621,13 +20621,13 @@ ${ctx.toSourceMapGenerator(sourceUrl, headerLines).toJsComment()}`;
       }
       _scopeKeyframesRelatedCss(cssText, scopeSelector) {
         const unscopedKeyframesSet = /* @__PURE__ */ new Set();
-        const scopedKeyframesCssText = processRules(cssText, (rule10) => this._scopeLocalKeyframeDeclarations(rule10, scopeSelector, unscopedKeyframesSet));
-        return processRules(scopedKeyframesCssText, (rule10) => this._scopeAnimationRule(rule10, scopeSelector, unscopedKeyframesSet));
+        const scopedKeyframesCssText = processRules(cssText, (rule11) => this._scopeLocalKeyframeDeclarations(rule11, scopeSelector, unscopedKeyframesSet));
+        return processRules(scopedKeyframesCssText, (rule11) => this._scopeAnimationRule(rule11, scopeSelector, unscopedKeyframesSet));
       }
-      _scopeLocalKeyframeDeclarations(rule10, scopeSelector, unscopedKeyframesSet) {
+      _scopeLocalKeyframeDeclarations(rule11, scopeSelector, unscopedKeyframesSet) {
         return {
-          ...rule10,
-          selector: rule10.selector.replace(/(^@(?:-webkit-)?keyframes(?:\s+))(['"]?)(.+)\2(\s*)$/, (_, start, quote, keyframeName, endSpaces) => {
+          ...rule11,
+          selector: rule11.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(rule10, scopeSelector, unscopedKeyframesSet) {
-        let content = rule10.content.replace(/((?:^|\s+|;)(?:-webkit-)?animation\s*:\s*),*([^;]+)/g, (_, start, animationDeclarations) => start + animationDeclarations.replace(this._animationDeclarationKeyframesRe, (original, leadingSpaces, quote = "", quotedName, nonQuotedName) => {
+      _scopeAnimationRule(rule11, scopeSelector, unscopedKeyframesSet) {
+        let content = rule11.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 {
-          ...rule10,
+          ...rule11,
           content
         };
       }
@@ -20661,8 +20661,8 @@ ${ctx.toSourceMapGenerator(sourceUrl, headerLines).toJsComment()}`;
       }
       _insertPolyfillRulesInCssText(cssText) {
         return cssText.replace(_cssContentRuleRe, (...m) => {
-          const rule10 = m[0].replace(m[1], "").replace(m[2], "");
-          return m[4] + rule10;
+          const rule11 = m[0].replace(m[1], "").replace(m[2], "");
+          return m[4] + rule11;
         });
       }
       _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 rule10 = m[0].replace(m[2], "").replace(m[1], m[4]);
-          r += rule10 + "\n\n";
+          const rule11 = m[0].replace(m[2], "").replace(m[1], m[4]);
+          r += rule11 + "\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, (rule10) => {
-          let selector = rule10.selector;
-          let content = rule10.content;
-          if (rule10.selector[0] !== "@") {
+        return processRules(cssText, (rule11) => {
+          let selector = rule11.selector;
+          let content = rule11.content;
+          if (rule11.selector[0] !== "@") {
             selector = this._scopeSelector({
               selector,
               scopeSelector,
               hostSelector,
               isParentSelector: true
             });
-          } else if (scopedAtRuleIdentifiers.some((atRule) => rule10.selector.startsWith(atRule))) {
-            content = this._scopeSelectors(rule10.content, scopeSelector, hostSelector);
-          } else if (rule10.selector.startsWith("@font-face") || rule10.selector.startsWith("@page")) {
-            content = this._stripScopingSelectors(rule10.content);
+          } else if (scopedAtRuleIdentifiers.some((atRule) => rule11.selector.startsWith(atRule))) {
+            content = this._scopeSelectors(rule11.content, scopeSelector, hostSelector);
+          } else if (rule11.selector.startsWith("@font-face") || rule11.selector.startsWith("@page")) {
+            content = this._stripScopingSelectors(rule11.content);
           }
           return new CssRule(selector, content);
         });
       }
       _stripScopingSelectors(cssText) {
-        return processRules(cssText, (rule10) => {
-          const selector = rule10.selector.replace(_shadowDeepSelectors, " ").replace(_polyfillHostNoCombinatorRe, " ");
-          return new CssRule(selector, rule10.content);
+        return processRules(cssText, (rule11) => {
+          const selector = rule11.selector.replace(_shadowDeepSelectors, " ").replace(_polyfillHostNoCombinatorRe, " ");
+          return new CssRule(selector, rule11.content);
         });
       }
       _safeSelector;
@@ -21051,8 +21051,8 @@ ${ctx.toSourceMapGenerator(sourceUrl, headerLines).toJsComment()}`;
           suffix = suffix.substring(BLOCK_PLACEHOLDER.length + 1);
           contentPrefix = "{";
         }
-        const rule10 = ruleCallback(new CssRule(selector, content));
-        return `${m[1]}${rule10.selector}${m[3]}${contentPrefix}${rule10.content}${suffix}`;
+        const rule11 = ruleCallback(new CssRule(selector, content));
+        return `${m[1]}${rule11.selector}${m[3]}${contentPrefix}${rule11.content}${suffix}`;
       });
       return unescapeInStrings(escapedResult);
     }
@@ -44056,11 +44056,9 @@ var rule3 = createRule3({
     return {
       Identifier(node) {
         const variableNode = services.esTreeNodeToTSNodeMap?.get(node);
-        console.log(node);
         if (variableNode) {
           const type = checker.getTypeAtLocation(variableNode);
           const typeName = checker.typeToString(type);
-          console.log(type, typeName, variableNode);
           if (isSignal(typeName)) {
             const parent = node.parent;
             if (!parent || parent.type !== import_utils3.AST_NODE_TYPES.CallExpression) {
@@ -44248,15 +44246,109 @@ var rule6 = createRule6({
   }
 });
 
+// src/rules/reactive-naming-convention.ts
+var import_utils7 = __toESM(require_dist4());
+var RULE_NAME7 = "reactive-naming-convention";
+var createRule7 = import_utils7.ESLintUtils.RuleCreator((name) => `https://github.com/developpement-megao/mgw-eslint-rules/docs/rules/${name}.md`);
+var rule7 = createRule7({
+  name: RULE_NAME7,
+  meta: {
+    type: "suggestion",
+    fixable: "code",
+    docs: {
+      description: "Naming convention for reactive variables and properties (name must end with $ and $$ for WritableSignals and Subjects)",
+      recommended: true
+    },
+    messages: {
+      reactiveNamingConvention: "Element {{ typeName }} names {{ name }} must end with `{{ suffix }}`."
+    },
+    schema: []
+    // No options
+  },
+  defaultOptions: [],
+  create(context) {
+    const parserServices = context.sourceCode.parserServices;
+    if (!parserServices?.program || !parserServices.esTreeNodeToTSNodeMap) {
+      return {};
+    }
+    const listeReactiveWritableTypes = ["Subject", "BehaviorSubject", "ReplaySubject", "AsyncSubject", "WritableSignal", "InputSignal"];
+    const listeReactiveReadonlyTypes = ["Signal", "Observable", "Promise"];
+    const checker = parserServices.program.getTypeChecker();
+    function checkNode(node) {
+      const name = node.name;
+      if (!name || name === "_" || !parserServices?.esTreeNodeToTSNodeMap) {
+        return;
+      }
+      const tsNode = parserServices.esTreeNodeToTSNodeMap.get(node);
+      const type = checker.getTypeAtLocation(tsNode);
+      const typeName = checker.typeToString(type);
+      const isReactiveReadonlyTypes = listeReactiveReadonlyTypes.includes(typeName);
+      const suffix = isReactiveReadonlyTypes ? "$" : "$$";
+      if ((isReactiveReadonlyTypes || listeReactiveWritableTypes.includes(typeName)) && (!name.endsWith(suffix) || suffix === "$" && name.endsWith("$$"))) {
+        context.report({
+          node,
+          messageId: "reactiveNamingConvention",
+          data: {
+            typeName,
+            name,
+            suffix
+          },
+          fix(fixer) {
+            const baseName = name.replace(/\$+$/, "");
+            const newName = `${baseName}${suffix}`;
+            const sourceCode = context.sourceCode;
+            const scope = sourceCode.getScope(node);
+            const fixes = [];
+            fixes.push(fixer.replaceText(node, newName));
+            let variable;
+            let current = scope;
+            while (current) {
+              variable = current.variables.find((v) => v.name === name);
+              if (variable) {
+                for (const ref of variable.references) {
+                  const refNode = ref.identifier;
+                  if (refNode !== node) {
+                    fixes.push(fixer.replaceText(refNode, newName));
+                  }
+                }
+                break;
+              }
+              if (!current.upper) {
+                break;
+              }
+              current = current.upper;
+            }
+            return fixes;
+          }
+        });
+      }
+    }
+    return {
+      // Déclarations de variables : const x = signal(0)
+      VariableDeclarator(node) {
+        if (node.id?.type === "Identifier") {
+          checkNode(node.id);
+        }
+      },
+      // Propriétés de classe : count = signal(0)
+      PropertyDefinition(node) {
+        if (node.key?.type === "Identifier") {
+          checkNode(node.key);
+        }
+      }
+    };
+  }
+});
+
 // src/rules/service-class-suffix.ts
-var import_utils7 = __toESM(require_dist6());
-var import_utils8 = __toESM(require_dist4());
-var RULE_NAME7 = "service-class-suffix";
+var import_utils8 = __toESM(require_dist6());
+var import_utils9 = __toESM(require_dist4());
+var RULE_NAME8 = "service-class-suffix";
 var SUFFIXES_DEFAULT = ["Service"];
 var DOCS_RULES3 = "https://github.com/developpement-megao/mgw-eslint-rules/blob/main/docs/rules";
-var createRule7 = import_utils8.ESLintUtils.RuleCreator((name) => `${DOCS_RULES3}/${name}.md`);
-var rule7 = createRule7({
-  name: RULE_NAME7,
+var createRule8 = import_utils9.ESLintUtils.RuleCreator((name) => `${DOCS_RULES3}/${name}.md`);
+var rule8 = createRule8({
+  name: RULE_NAME8,
   meta: {
     type: "suggestion",
     docs: {
@@ -44288,7 +44380,7 @@ var rule7 = createRule7({
   ],
   create(context, [{ suffixes }]) {
     return {
-      [import_utils7.Selectors.INJECTABLE_CLASS_DECORATOR](node) {
+      [import_utils8.Selectors.INJECTABLE_CLASS_DECORATOR](node) {
         const classDeclaration = node.parent;
         if (!classDeclaration.id) {
           return;
@@ -44311,11 +44403,11 @@ var rule7 = createRule7({
 });
 
 // src/rules/template/attributes-naming-convention.ts
-var import_utils9 = __toESM(require_dist6());
-var import_utils10 = __toESM(require_dist4());
+var import_utils10 = __toESM(require_dist6());
+var import_utils11 = __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_NAME8 = "template/attributes-naming-convention";
+var RULE_NAME9 = "template/attributes-naming-convention";
 var conventionPatterns = {
   camelCase: /^[a-z][a-zA-Z0-9]*$/,
   strictCamelCase: /^(?!.*[A-Z]{2})[a-z][a-zA-Z0-9]*$/,
@@ -44326,9 +44418,9 @@ var conventionPatterns = {
   "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 createRule8 = import_utils10.ESLintUtils.RuleCreator((name) => `${DOCS_RULES4}/${name}.md`);
-var rule8 = createRule8({
-  name: RULE_NAME8,
+var createRule9 = import_utils11.ESLintUtils.RuleCreator((name) => `${DOCS_RULES4}/${name}.md`);
+var rule9 = createRule9({
+  name: RULE_NAME9,
   meta: {
     type: "suggestion",
     docs: {
@@ -44387,7 +44479,7 @@ var rule8 = createRule8({
     }
   ],
   create(context, [options]) {
-    const parserServices = (0, import_utils9.getTemplateParserServices)(context);
+    const parserServices = (0, import_utils10.getTemplateParserServices)(context);
     const listeAttributes = new Map(
       Object.entries(options).map(([a, n]) => {
         const listeNommage = typeof n === "string" ? [n] : n;
@@ -44436,13 +44528,13 @@ var rule8 = createRule8({
 });
 
 // src/rules/template/prefer-class-style-binding.ts
-var import_utils11 = __toESM(require_dist6());
-var import_utils12 = __toESM(require_dist4());
-var RULE_NAME9 = "template/prefer-class-style-binding";
+var import_utils12 = __toESM(require_dist6());
+var import_utils13 = __toESM(require_dist4());
+var RULE_NAME10 = "template/prefer-class-style-binding";
 var DOCS_RULES5 = "https://github.com/developpement-megao/mgw-eslint-rules/tree/main/docs/rules";
-var createRule9 = import_utils12.ESLintUtils.RuleCreator((name) => `${DOCS_RULES5}/${name}.md`);
-var rule9 = createRule9({
-  name: RULE_NAME9,
+var createRule10 = import_utils13.ESLintUtils.RuleCreator((name) => `${DOCS_RULES5}/${name}.md`);
+var rule10 = createRule10({
+  name: RULE_NAME10,
   meta: {
     type: "suggestion",
     docs: {
@@ -44457,7 +44549,7 @@ var rule9 = createRule9({
   },
   defaultOptions: [],
   create(context) {
-    const parserServices = (0, import_utils11.getTemplateParserServices)(context);
+    const parserServices = (0, import_utils12.getTemplateParserServices)(context);
     return {
       // Détecte les attributs ngClass et ngStyle et inputs [ngClass] et [ngStyle] dans les templates
       Element(node) {
@@ -44483,10 +44575,11 @@ var rules = {
   [RULE_NAME3]: rule3,
   [RULE_NAME4]: rule4,
   [RULE_NAME]: rule,
-  [RULE_NAME9]: rule9,
-  [RULE_NAME7]: rule7,
+  [RULE_NAME10]: rule10,
+  [RULE_NAME8]: rule8,
   [RULE_NAME2]: rule2,
-  [RULE_NAME8]: rule8
+  [RULE_NAME9]: rule9,
+  [RULE_NAME7]: rule7
 };
 // Annotate the CommonJS export names for ESM import in node:
 0 && (module.exports = {