@lsby/eslint-plugin 0.0.7 → 0.0.9

package/index.js

@@ -4,5 +4,6 @@ module.exports = {
     'prefer-let': require('./lib/rules/prefer-let'),
     'no-negation': require('./lib/rules/no-negation'),
     'no-null': require('./lib/rules/no-null'),
+    'require-has-check': require('./lib/rules/require-has-check'),
   },
 }

package/lib/rules/no-null.js

@@ -1,31 +1,27 @@
 module.exports = {
-  rules: {
-    'no-null': {
-      meta: {
-        type: 'suggestion', // 表示这是个建议类型的规则
-        docs: {
-          description: 'Disallow usage of null and replace it with undefined',
-          category: 'Best Practices',
-          recommended: false,
-        },
-        fixable: 'code', // 支持自动修复
-        schema: [], // 无额外配置
-      },
-      create(context) {
-        return {
-          Literal(node) {
-            if (node.value === null) {
-              context.report({
-                node,
-                message: '不允许使用‘null’。使用‘undefined’代替。',
-                fix(fixer) {
-                  return fixer.replaceText(node, 'undefined')
-                },
-              })
-            }
-          },
+  meta: {
+    type: 'suggestion', // 表示这是个建议类型的规则
+    docs: {
+      description: 'Disallow usage of null and replace it with undefined',
+      category: 'Best Practices',
+      recommended: false,
+    },
+    fixable: 'code', // 支持自动修复
+    schema: [], // 无额外配置
+  },
+  create(context) {
+    return {
+      Literal(node) {
+        if (node.value === null) {
+          context.report({
+            node,
+            message: '不允许使用‘null’。使用‘undefined’代替。',
+            fix(fixer) {
+              return fixer.replaceText(node, 'undefined')
+            },
+          })
         }
       },
-    },
+    }
   },
 }

package/lib/rules/prefer-let.js

@@ -6,15 +6,15 @@ module.exports = {
       category: 'Best Practices',
       recommended: false,
     },
-    fixable: 'code', // 支持修复
-    schema: [], // 没有配置选项
+    fixable: 'code',
+    schema: [],
   },
 
   create(context) {
     return {
       VariableDeclaration(node) {
-        // 检查变量声明的类型
-        if (node.kind === 'const' || node.kind === 'var') {
+        // 检查变量声明的类型（排除 declare）
+        if ((node.kind === 'const' || node.kind === 'var') && !node.declare) {
           // 使用修复功能将 const 或 var 替换为 let
           context.report({
             node,

package/lib/rules/require-has-check.js

@@ -0,0 +1,41 @@
+module.exports = {
+  meta: {
+    type: 'problem',
+    docs: {
+      description: 'Enforce using `hasOwnProperty` or `Map.has` instead of unsafe index checks',
+      category: 'Best Practices',
+      recommended: false,
+    },
+    fixable: 'code', // 自动修复支持
+    messages: {
+      useHas:
+        'Avoid comparing with `undefined` or `null` directly. Use `hasOwnProperty` or `Map.has` for existence checks.',
+    },
+    schema: [], // No options
+  },
+  create(context) {
+    return {
+      BinaryExpression(node) {
+        if (
+          ['===', '!=='].includes(node.operator) &&
+          node.left.type === 'MemberExpression' &&
+          ['undefined', 'null'].includes(node.right.type === 'Identifier' ? node.right.name : node.right.raw)
+        ) {
+          context.report({
+            node,
+            messageId: 'useHas',
+            fix(fixer) {
+              // 自动修复逻辑
+              const sourceCode = context.getSourceCode()
+              const objectCode = sourceCode.getText(node.left.object)
+              const propertyCode = sourceCode.getText(node.left.property)
+              const isNegative = node.operator === '!=='
+              const replacement = `${objectCode}.hasOwnProperty(${propertyCode})`
+              return fixer.replaceText(node, isNegative ? `!${replacement}` : replacement)
+            },
+          })
+        }
+      },
+    }
+  },
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lsby/eslint-plugin",
-  "version": "0.0.7",
+  "version": "0.0.9",
   "main": "index.js",
   "scripts": {
     "pub:public": "bumpp && npm publish --access public",