@lsby/eslint-plugin 0.0.14 → 0.0.16

package/index.js

@@ -3,8 +3,7 @@ module.exports = {
     'no-broken-link': require('./lib/rules/no-broken-link'),
     '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'),
     'no-definite-assignment-assertion': require('./lib/rules/no-definite-assignment-assertion'),
+    'no-else': require('./lib/rules/no-else'),
   },
 }

package/lib/rules/no-broken-link.js

@@ -1,3 +1,5 @@
+// jsdoc的link必须存在
+
 module.exports = {
   meta: {
     type: 'problem', // 规则类型，可以是 "problem", "suggestion", 或 "layout"

package/lib/rules/no-definite-assignment-assertion.js

@@ -1,3 +1,5 @@
+// 禁止类属性使用确定赋值断言语法
+
 module.exports = {
   meta: {
     type: 'problem',
@@ -6,7 +8,7 @@ module.exports = {
       recommended: 'error',
     },
     messages: {
-      noDefiniteAssignment: '类属性 "{{name}}" 不允许使用确定赋值断言 (!:)',
+      noDefiniteAssignment: '类属性不允许使用确定赋值断言 (!:)',
     },
     schema: [],
   },
@@ -14,27 +16,17 @@ module.exports = {
     return {
       ClassProperty(node) {
         if (node.definite) {
-          let name = '未知属性名'
-          if (node.key && node.key.type === 'Identifier') {
-            name = node.key.name
-          }
           context.report({
             node,
             messageId: 'noDefiniteAssignment',
-            data: { name },
           })
         }
       },
       PropertyDefinition(node) {
         if (node.definite) {
-          let name = '未知属性名'
-          if (node.key && node.key.type === 'Identifier') {
-            name = node.key.name
-          }
           context.report({
             node,
             messageId: 'noDefiniteAssignment',
-            data: { name },
           })
         }
       },

package/lib/rules/no-else.js

@@ -0,0 +1,30 @@
+// 禁止 else
+// else 表示"除当前条件外的所有可能"
+// 当状态集合未来扩展时, 依然会被包含在else分支里, 导致状态遗漏却无任何报错
+// 应当使用提早返回 或 switch + 穷尽检查
+
+module.exports = {
+  meta: {
+    type: 'problem',
+    docs: {
+      description: '禁止使用 else, 避免不稳定的否定剩余分支',
+    },
+    messages: {
+      noElse: '禁止使用 else, 请改用 early return 或 switch',
+    },
+    schema: [],
+  },
+
+  create(context) {
+    return {
+      IfStatement(node) {
+        if (node.alternate !== null) {
+          context.report({
+            node: node.alternate,
+            messageId: 'noElse',
+          })
+        }
+      },
+    }
+  },
+}

package/lib/rules/no-negation.js

@@ -1,3 +1,6 @@
+// 禁止对非布尔值使用取反
+// 对于 number | null 的值x, if(!x)在x等于null和0时都会触发, 这可能是非预期的
+
 module.exports = {
   create(context) {
     return {

package/lib/rules/prefer-let.js

@@ -1,3 +1,9 @@
+// 永远使用let, 拒绝var和const, 并自动修复
+// 禁止const是因为它有抽象泄漏
+// 它只对原语值和指针不变, 不对引用值本身不变
+// 要搞懂什么可变什么不可变, 必须理解原语值和引用值的区别, 以及变量在内存上的机制
+// 如果真的需要表达不变, 应该在类型等级写递归只读
+
 module.exports = {
   meta: {
     type: 'suggestion',

package/package.json

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

package/lib/rules/no-null.js

@@ -1,27 +0,0 @@
-module.exports = {
-  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/require-has-check.js

@@ -1,50 +0,0 @@
-module.exports = {
-  meta: {
-    type: 'problem',
-    docs: {
-      description: 'Enforce using `hasOwnProperty` instead of unsafe `null` checks',
-      category: 'Best Practices',
-      recommended: false,
-    },
-    fixable: 'code', // 自动修复支持
-    messages: {
-      useHas: 'Avoid comparing with `null` directly. Use `hasOwnProperty` for existence checks.',
-    },
-    schema: [], // No options
-  },
-  create(context) {
-    return {
-      BinaryExpression(node) {
-        // 检查是不是 a[x] === null 或 a[x] !== null
-        if (
-          ['===', '!=='].includes(node.operator) &&
-          node.left.type === 'MemberExpression' &&
-          node.right.raw === 'null'
-        ) {
-          // 检查是不是类实例的 null 比较, 比如 this.x === null
-          const isClassPropertyCheck = node.left.object.type === 'ThisExpression'
-          if (isClassPropertyCheck) {
-            // 如果是类实例的 null 检查，则跳过修复
-            return
-          }
-
-          // 默认的 hasOwnProperty 检查
-          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)
-
-              let replacement = `${objectCode}.hasOwnProperty(${propertyCode})`
-              replacement = `!${replacement}`
-
-              return fixer.replaceText(node, replacement)
-            },
-          })
-        }
-      },
-    }
-  },
-}