eslint-plugin-crisp 1.4.4 → 1.4.7

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-crisp",
-  "version": "1.4.4",
+  "version": "1.4.7",
   "description": "Custom ESLint Rules for Crisp",
   "author": "Crisp IM SAS",
   "main": "index.js",

package/rules/import-group-comment.js

@@ -1,3 +1,51 @@
+function isTypeComment(comment) {
+  return (
+    comment.type === "Block" &&
+    comment.value.startsWith("*") &&
+    (comment.value.includes("@import") || comment.value.includes("@typedef"))
+  );
+}
+
+function isTypesGroupComment(comment) {
+  return comment.type === "Line" && comment.value.trim() === "TYPES";
+}
+
+function isGroupComment(comment) {
+  return (
+    comment.type === "Line" &&
+    (
+      comment.value.trim().startsWith("PROJECT:") ||
+      comment.value.trim().startsWith("NPM") ||
+      comment.value.trim().startsWith("TYPES")
+    )
+  );
+}
+
+function isIgnoreComment(comment) {
+  return (
+    comment.type === "Line" &&
+    comment.value.trim().startsWith("@ts-ignore")
+  );
+}
+
+function getDirectoryName(filePath) {
+  const pathParts = filePath.split("/");
+
+  return pathParts[pathParts.length - 2].toUpperCase();
+}
+
+function generateExpectedComment(group) {
+  if (group === "NPM") {
+    return group;
+  }
+
+  if (group === "SRC") {
+    group = "MAIN";
+  }
+
+  return `PROJECT: ${group}`;
+}
+
 export default {
   meta: {
     type: "suggestion",
@@ -21,18 +69,12 @@ export default {
     const customGroups = context.options[0] || {};  // Get custom groups from options
 
     // Pre-compile custom regexes once instead of on every import
-    const compiledCustomGroups = Object.entries(customGroups).map(
-      ([regexStr, group]) => [new RegExp(regexStr), group]
-    );
+    const compiledCustomGroups = Object.entries(customGroups).map(([regexStr, group]) => {
+      return [new RegExp(regexStr), group];
+    });
 
     let currentGroupComment = null;
 
-    // Extract the directory name from the file path
-    function getDirectoryName(filePath) {
-      const pathParts = filePath.split("/");
-      return pathParts[pathParts.length - 2].toUpperCase(); // Directory name
-    }
-
     function extractGroupFromPath(path, filePath) {
       // Check custom regexes first (using pre-compiled patterns)
       for (const [regex, group] of compiledCustomGroups) {
@@ -60,36 +102,6 @@ export default {
       return "NPM";
     }
 
-    function generateExpectedComment(group) {
-      if (group === "NPM") {
-        return group;
-      }
-
-      if (group === "SRC") {
-        group = "MAIN";
-      }
-
-      return `PROJECT: ${group}`;
-    }
-
-    function isGroupComment(comment) {
-      return (
-        comment.type === "Line" &&
-        (
-          comment.value.trim().startsWith("PROJECT:") ||
-          comment.value.trim().startsWith("NPM")
-        )
-      );
-    }
-
-    function isIgnoreComment(comment) {
-      // Skip some comments
-      return (
-        comment.type === "Line" &&
-        comment.value.trim().startsWith("@ts-ignore")
-      );
-    }
-
     function updateCurrentGroupComment(node) {
       const comments = context.getSourceCode().getCommentsBefore(node);
 
@@ -119,7 +131,7 @@ export default {
       // Get comment for current group
       updateCurrentGroupComment(node);
 
-      if (!currentGroupComment || !currentGroupComment.value.toUpperCase().includes(expectedComment.toUpperCase())) {
+      if (!currentGroupComment || !currentGroupComment.value.includes(expectedComment)) {
         context.report({
           node,
           message: `Import "${importPath}" should be in the "${expectedComment}" group.`,
@@ -127,8 +139,49 @@ export default {
       }
     }
 
+    const sourceCode = context.sourceCode || context.getSourceCode();
+    const allComments = sourceCode.getAllComments();
+    const typeComments = allComments.filter(c => isTypeComment(c));
+
+    function checkTypeCommentsGroup() {
+      const filename = context.getFilename();
+
+      // Only check for // TYPES in .vue files (block header is checked by header-comments-check rule for .js/.ts)
+      if (!filename.endsWith(".vue")) {
+        return;
+      }
+
+      if (typeComments.length === 0) {
+        return;
+      }
+
+      const firstTypeComment = typeComments[0];
+      const typeCommentStart = firstTypeComment.range[0];
+
+      let comment = null;
+
+      for (let i = allComments.length - 1; i >= 0; i--) {
+        if (allComments[i].range[1] < typeCommentStart) {
+          comment = allComments[i];
+
+          break;
+        }
+      }
+
+      if (!comment || !isTypesGroupComment(comment)) {
+        context.report({
+          loc: firstTypeComment.loc,
+          message: "Type comments group must be preceded by a '// TYPES' comment",
+        });
+      }
+    }
+
     return {
       ImportDeclaration: checkImportGroup,
+
+      "Program:exit": () => {
+        checkTypeCommentsGroup();
+      }
     };
   }
 };