mm_eslint 1.5.2 → 1.5.4

package/lib/detector/const_name.js

@@ -74,6 +74,13 @@ ConstName.prototype.VariableDeclaration = function (node) {
           let name = decl.id.name;
           let original_type = 'constant';
           let error = this.check(node, name, original_type);
+          
+          // 如果常量名检测有错误，但符合变量命名规则，则排除此错误
+          if (error && this._isValidVariableName(name)) {
+            // 符合变量命名规则，跳过常量检测，由变量检测器处理
+            continue;
+          }
+          
           this.report(node, error, original_type);
 
           if (error) {
@@ -227,6 +234,11 @@ ConstName.prototype._isConstantValue = function (value_node) {
     return false; // await表达式的结果通常是可变的，不是常量
   }
 
+  // 排除条件表达式（三元运算符）
+  if (value_node.type === 'ConditionalExpression') {
+    return false; // 条件表达式的结果在运行时确定，不是常量
+  }
+
   // 默认情况下，如果不是排除的类型，就认为是常量
   return true;
 }
@@ -585,6 +597,19 @@ ConstName.prototype.ExportNamedDeclaration = function (node) {
   }
 }
 
+/**
+ * 判断名称是否符合变量命名规则
+ * @param {string} name - 名称
+ * @returns {boolean} 是否符合变量命名规则
+ */
+ConstName.prototype._isValidVariableName = function (name) {
+  // 变量命名规则：小写蛇形命名法 (snake_case)
+  const variable_regex = /^[a-z][a-z0-9]*(_[a-z0-9]+)*$/;
+  
+  // 检查是否符合变量命名风格
+  return variable_regex.test(name);
+}
+
 module.exports = {
   ConstName
 };

package/lib/detector/variable_name.js

@@ -134,7 +134,33 @@ VariableName.prototype.VariableDeclaration = function (node) {
             this.report(node, error, 'variable');
             return error;
           }
+          
+          // 对于ConditionalExpression（条件表达式/三元运算符），直接处理
+          else if (decl.init.type === 'ConditionalExpression') {
+            let error = {
+              message: `变量'${name}'应该使用let标签声明而不是const标签声明`,
+              fix: function(fixer) {
+                return fixer.replaceTextRange([node.range[0], node.range[0] + 5], 'let');
+              }
+            };
+            this.report(node, error, 'variable');
+            return error;
+          }
         }
+        
+        // 新增：检测const声明但符合变量命名规则的情况
+        // 如果名称符合变量命名规则（小写蛇形命名法），则提示使用let
+        if (this._isValidVariableName(name) && !this._isValidConstantName(name)) {
+          let error = {
+            message: `变量'${name}'应该使用let标签声明而不是const标签声明`,
+            fix: function(fixer) {
+              return fixer.replaceTextRange([node.range[0], node.range[0] + 5], 'let');
+            }
+          };
+          this.report(node, error, 'variable');
+          return error;
+        }
+        
         // 对于const声明的其他情况，跳过变量规则检测（由常量规则处理）
         return;
       }
@@ -342,6 +368,42 @@ VariableName.prototype._isInForLoop = function (node) {
   return false;
 }
 
+/**
+ * 判断名称是否符合变量命名规则
+ * @param {string} name - 名称
+ * @returns {boolean} 是否符合变量命名规则
+ */
+VariableName.prototype._isValidVariableName = function (name) {
+  // 变量命名规则：小写蛇形命名法 (snake_case)
+  const variable_regex = /^[a-z][a-z0-9]*(_[a-z0-9]+)*$/;
+  
+  // 检查名称长度（1-20字符）
+  if (name.length < 1 || name.length > 20) {
+    return false;
+  }
+  
+  // 检查是否符合变量命名风格
+  return variable_regex.test(name);
+}
+
+/**
+ * 判断名称是否符合常量命名规则
+ * @param {string} name - 名称
+ * @returns {boolean} 是否符合常量命名规则
+ */
+VariableName.prototype._isValidConstantName = function (name) {
+  // 常量命名规则：大写蛇形命名法 (UPPER_SNAKE_CASE)
+  const constant_regex = /^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$/;
+  
+  // 检查名称长度（1-20字符）
+  if (name.length < 1 || name.length > 20) {
+    return false;
+  }
+  
+  // 检查是否符合常量命名风格
+  return constant_regex.test(name);
+}
+
 module.exports = {
   VariableName
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "mm_eslint",
-  "version": "1.5.2",
+  "version": "1.5.4",
   "description": "ESLint plugin for personal naming conventions - supports PascalCase, camelCase, snake_case, and UPPER_SNAKE_CASE naming rules with intelligent recommendations",
   "main": "index.js",
   "keywords": [