mm_eslint 1.5.3 → 1.5.5

package/lib/detector/const_name.js

@@ -68,12 +68,24 @@ ConstName.prototype.VariableDeclaration = function (node) {
           continue;
         }
 
+        // 排除require()调用，因为导入的模块类型无法确定
+        if (this._isRequireCall(decl.init)) {
+          continue;
+        }
+
         // 判断是否为常量（包括 Object.freeze() 等情况）
         if (this._isConstantValue(decl.init)) {
           // 检测常量名
           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) {
@@ -590,6 +602,60 @@ ConstName.prototype.ExportNamedDeclaration = function (node) {
   }
 }
 
+/**
+ * 判断节点是否为require()调用
+ * @param {Object} node - AST节点
+ * @returns {boolean} 是否为require()调用
+ */
+ConstName.prototype._isRequireCall = function (node) {
+  if (!node) {
+    return false;
+  }
+
+  // 情况1：直接的require()调用，如 require('module')
+  if (node.type === 'CallExpression' &&
+      node.callee && node.callee.type === 'Identifier' && 
+      node.callee.name === 'require') {
+    return true;
+  }
+
+  // 情况2：require().property，如 require('module').property
+  if (node.type === 'MemberExpression' &&
+      node.object && node.object.type === 'CallExpression' &&
+      node.object.callee && node.object.callee.type === 'Identifier' &&
+      node.object.callee.name === 'require') {
+    return true;
+  }
+
+  // 情况3：require('module')[property]，如 require('module')['property']
+  if (node.type === 'MemberExpression' &&
+      node.object && node.object.type === 'CallExpression' &&
+      node.object.callee && node.object.callee.type === 'Identifier' &&
+      node.object.callee.name === 'require') {
+    return true;
+  }
+
+  return false;
+}
+
+/**
+ * 判断名称是否符合变量命名规则
+ * @param {string} name - 名称
+ * @returns {boolean} 是否符合变量命名规则
+ */
+ConstName.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);
+}
+
 module.exports = {
   ConstName
 };

package/lib/detector/variable_name.js

@@ -147,6 +147,20 @@ VariableName.prototype.VariableDeclaration = function (node) {
             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;
       }
@@ -354,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.3",
+  "version": "1.5.5",
   "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": [