@lsby/eslint-plugin 0.0.17 → 0.0.18

package/dist/index.js

@@ -4,7 +4,7 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
 };
 Object.defineProperty(exports, "__esModule", { value: true });
 const no_definite_assignment_assertion_1 = __importDefault(require("./lib/rules/no-definite-assignment-assertion"));
-const no_else_1 = __importDefault(require("./lib/rules/no-else"));
+const no_else_on_equality_1 = __importDefault(require("./lib/rules/no-else-on-equality"));
 const no_negation_1 = __importDefault(require("./lib/rules/no-negation"));
 const no_switch_default_1 = __importDefault(require("./lib/rules/no-switch-default"));
 const prefer_let_1 = __importDefault(require("./lib/rules/prefer-let"));
@@ -14,7 +14,7 @@ module.exports = {
         'prefer-let': prefer_let_1.default,
         'no-negation': no_negation_1.default,
         'no-definite-assignment-assertion': no_definite_assignment_assertion_1.default,
-        'no-else': no_else_1.default,
+        'no-else-on-equality': no_else_on_equality_1.default,
         'no-switch-default': no_switch_default_1.default,
         'prefer-switch-over-multi-if': prefer_switch_over_multi_if_1.default,
     },

package/dist/lib/rules/no-else-on-equality.js

@@ -0,0 +1,40 @@
+"use strict";
+// 禁止 else（当条件是等于或不等于时）
+// 对于等于/不等于的条件判断，else 表示"除当前条件外的所有可能"
+// 当状态集合未来扩展时, 依然会被包含在else分支里, 导致状态遗漏却无任何报错
+// 应当使用提早返回 或 switch + 穷尽检查
+Object.defineProperty(exports, "__esModule", { value: true });
+const rule = {
+    meta: {
+        type: 'problem',
+        docs: { description: '当条件是等于或不等于时，禁止使用 else 分支，防止隐含的状态遗漏' },
+        messages: {
+            noElse: '禁止在等于/不等于条件下使用 else 分支。当条件状态扩展时, else 分支可能遗漏新的状态。请改用提前返回(early return)或 switch 语句',
+        },
+        schema: [],
+    },
+    defaultOptions: [],
+    create(context) {
+        /**
+         * 检查条件中是否包含等于或不等于操作符
+         */
+        function hasEqualityOperator(node) {
+            if (node.type === 'BinaryExpression') {
+                return ['===', '!==', '==', '!='].includes(node.operator);
+            }
+            if (node.type === 'LogicalExpression') {
+                return hasEqualityOperator(node.left) || hasEqualityOperator(node.right);
+            }
+            return false;
+        }
+        return {
+            IfStatement(node) {
+                // 仅在 if 条件是等于或不等于的情况下，禁止 else
+                if (node.alternate !== null && hasEqualityOperator(node.test)) {
+                    context.report({ node: node.alternate, messageId: 'noElse' });
+                }
+            },
+        };
+    },
+};
+exports.default = rule;

package/package.json

@@ -1,19 +1,19 @@
 {
   "name": "@lsby/eslint-plugin",
-  "version": "0.0.17",
+  "version": "0.0.18",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",
   "files": [
     "dist"
   ],
   "scripts": {
-    "build": "tsc",
+    "build": "rimraf dist && tsc",
     "check:all": "npm run check:format && npm run check:type",
     "check:format": "prettier --write .",
     "check:type": "tsc --noEmit",
     "dev": "tsc --watch",
     "pub:public": "npm run build && npm run test && bumpp && npm publish --access public",
-    "test": "mocha test/**/*.test.js"
+    "test": "npm run build && mocha test/**/*.test.js"
   },
   "devDependencies": {
     "@ianvs/prettier-plugin-sort-imports": "^4.2.1",
@@ -27,6 +27,7 @@
     "mocha": "^10.7.3",
     "prettier": "^3.7.4",
     "prettier-plugin-packagejson": "^2.5.20",
+    "rimraf": "^6.1.2",
     "typescript": "^5.3.0"
   },
   "peerDependencies": {

package/dist/lib/rules/no-else.js

@@ -1,27 +0,0 @@
-"use strict";
-// 禁止 else
-// else 表示"除当前条件外的所有可能"
-// 当状态集合未来扩展时, 依然会被包含在else分支里, 导致状态遗漏却无任何报错
-// 应当使用提早返回 或 switch + 穷尽检查
-Object.defineProperty(exports, "__esModule", { value: true });
-const rule = {
-    meta: {
-        type: 'problem',
-        docs: { description: '禁止使用 else 分支, 防止隐含的状态遗漏' },
-        messages: {
-            noElse: '禁止使用 else 分支。当条件状态扩展时, else 分支可能遗漏新的状态。请改用提前返回(early return)或 switch 语句',
-        },
-        schema: [],
-    },
-    defaultOptions: [],
-    create(context) {
-        return {
-            IfStatement(node) {
-                if (node.alternate !== null) {
-                    context.report({ node: node.alternate, messageId: 'noElse' });
-                }
-            },
-        };
-    },
-};
-exports.default = rule;