eslint-plugin-better-codes 1.1.6 → 1.1.9

package/README.md

@@ -59,8 +59,9 @@ module.exports = {
 不校验的情况：
 
 - React组件函数
-- 函数体只有一行代码的函数
+- 函数体只有一行代码(一条执行语句)的函数
 - 用于map/forEach中返回元素/组件用于渲染的箭头函数
+- 嵌套函数内的子函数,，如果父函数已经包含try-catch，则子函数不进行校验
 
 <!-- TODO:待解决的问题：针对返回结果为true/false的函数，使用try...catch进入catch分支之后改返回false还是true还是不返回(undefined)，因为按照之前的逻辑报错之后后续的代码不应再执行的，但是现在会直接继续走false/true的逻辑，造成逻辑错误 -->
 

package/libs/function-with-try-catch-code.js

@@ -182,6 +182,29 @@ function isArrayMethodCallback(node) {
   }
 }
 
+// 判断任一父级函数是否包含 try...catch
+function ancestorHasTry(node) {
+  try {
+    let p = node.parent;
+    while (p) {
+      if (
+        p.type === "FunctionDeclaration" ||
+        p.type === "FunctionExpression" ||
+        p.type === "ArrowFunctionExpression" ||
+        p.type === "MethodDefinition"
+      ) {
+        // 对于函数节点，检查其 body 是否包含 TryStatement
+        const body = p.body || (p.value && p.value.body);
+        if (body && containsTryStatement(body)) return true;
+      }
+      p = p.parent;
+    }
+  } catch (err) {
+    // ignore
+  }
+  return false;
+}
+
 module.exports =  {
   meta: {
     type: "suggestion",
@@ -214,6 +237,8 @@ module.exports =  {
         const stmts = body.body || [];
         // 如果函数体为空或只有注释/空，则跳过
         if (stmts.length === 0) return;
+        // 如果任一父级函数已包含 try/catch，跳过（避免重复校验内部回调）
+        if (ancestorHasTry(node)) return;
         // 如果函数体中包含 TryStatement，说明已处理，跳过
         if (containsTryStatement(body)) return;
         context.report({

package/libs/no-commented-code.js

@@ -91,78 +91,82 @@ module.exports = {
         Program() {
           const comments = sourceCode.getAllComments();
           comments.forEach(comment => {
-            if (comment.loc) {
-              // 单行注释
-              if (comment.type === "Line") {
-                if (isFunctionComment(comment.value)) {
-                  return;
-                }
-                singleLineCount++;
+            try {
+              if (comment.loc) {
+                // 单行注释
+                if (comment.type === "Line") {
+                  if (isFunctionComment(comment.value)) {
+                    return;
+                  }
+                  singleLineCount++;
 
-                const lines = comment.value.split("\n");
-                // 过滤空行，只计算有实际内容的行
-                const codeLines = lines.filter(line => {
-                  const trimmed = line.trim();
-                  return trimmed.length > 0;
-                });
-                const isIgnored = codeLines[0].endsWith("ignore-eslint"); // 因默认添加了ignore-eslint，进行忽略校验
-                if (isIgnored && ignoredCount < minLines) {
-                  ignoredCount++;
-                  singleLineCount--;
-                }
-                // 优先检查忽略次数：当忽略累计次数达到阈值时，优先提示并禁止继续忽略
-                if (ignoredCount >= minLines) {
-                  context.report({
-                    node: comment,
-                    message: `总忽略次数大于等于${minLines}次，请删除或保留为有效代码`,
-                    data: {
-                      lines: ignoredCount,
-                    },
-                  });
-                  return;
-                }
-                // 其次检查单行注释总数
-                if (singleLineCount >= minLines) {
-                  context.report({
-                    node: comment,
-                    message: `单行注释大于等于 ${minLines} 行代码，请删除或保留为有效代码`,
-                    data: {
-                      lines: singleLineCount,
-                    },
+                  const lines = comment.value.split("\n");
+                  // 过滤空行，只计算有实际内容的行
+                  const codeLines = lines.filter(line => {
+                    const trimmed = line.trim();
+                    return trimmed.length > 0;
                   });
-                  return;
-                }
-                // 如果注释块包含多行代码逻辑
-                if (codeLines.length >= minLines) {
-                  context.report({
-                    node: comment,
-                    message: `注释掉了 ${codeLines.length} 行代码，请删除或保留为有效代码`,
-                    data: {
-                      lines: codeLines.length,
-                    },
-                  });
-                }
-              }
-              // 多行注释
-              if (comment.type === "Block") {
-                if (isFunctionComment(comment.value, true)) {
-                  return;
+                  const isIgnored = codeLines[0]?.endsWith("ignore-eslint"); // 因默认添加了ignore-eslint，进行忽略校验
+                  if (isIgnored && ignoredCount < minLines) {
+                    ignoredCount++;
+                    singleLineCount--;
+                  }
+                  // 优先检查忽略次数：当忽略累计次数达到阈值时，优先提示并禁止继续忽略
+                  if (ignoredCount >= minLines) {
+                    context.report({
+                      node: comment,
+                      message: `总忽略次数大于等于${minLines}次，请删除或保留为有效代码`,
+                      data: {
+                        lines: ignoredCount,
+                      },
+                    });
+                    return;
+                  }
+                  // 其次检查单行注释总数
+                  if (singleLineCount >= minLines) {
+                    context.report({
+                      node: comment,
+                      message: `单行注释大于等于 ${minLines} 行代码，请删除或保留为有效代码`,
+                      data: {
+                        lines: singleLineCount,
+                      },
+                    });
+                    return;
+                  }
+                  // 如果注释块包含多行代码逻辑
+                  if (codeLines.length >= minLines) {
+                    context.report({
+                      node: comment,
+                      message: `注释掉了 ${codeLines.length} 行代码，请删除或保留为有效代码`,
+                      data: {
+                        lines: codeLines.length,
+                      },
+                    });
+                  }
                 }
-                const lines = comment.value.split("\n");
-                const codeLines = lines.filter(line => {
-                  const trimmed = line.trim();
-                  return trimmed !== "*" && trimmed.length > 0 && trimmed; // 过滤掉空行和只包含*的行
-                });
-                if (codeLines.length >= minLines) {
-                  context.report({
-                    node: comment,
-                    message: `注释掉了 ${codeLines.length} 行代码，请删除或保留为有效代码`,
-                    data: {
-                      lines: codeLines.length,
-                    },
+                // 多行注释
+                if (comment.type === "Block") {
+                  if (isFunctionComment(comment.value, true)) {
+                    return;
+                  }
+                  const lines = comment.value.split("\n");
+                  const codeLines = lines.filter(line => {
+                    const trimmed = line.trim();
+                    return trimmed !== "*" && trimmed.length > 0 && trimmed; // 过滤掉空行和只包含*的行
                   });
+                  if (codeLines.length >= minLines) {
+                    context.report({
+                      node: comment,
+                      message: `注释掉了 ${codeLines.length} 行代码，请删除或保留为有效代码`,
+                      data: {
+                        lines: codeLines.length,
+                      },
+                    });
+                  }
                 }
               }
+            } catch (err) {
+              console.error(err);
             }
           });
         },

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-better-codes",
-  "version": "1.1.6",
+  "version": "1.1.9",
   "main": "index.js",
   "scripts": {
     "test": "npx eslint ./test.js"