ESLint Plugin Perfectionist 排序规则导致 Switch Case 语句行为异常问题分析

在 JavaScript/TypeScript 开发中，ESLint 插件 Perfectionist 的 sort-switch-case 规则被发现存在一个可能导致代码逻辑改变的重要缺陷。这个缺陷特别出现在处理连续枚举(ENUM) case 语句的场景中。

问题现象

当开发者使用 switch 语句处理多个连续的枚举值时，Perfectionist 的自动排序功能会错误地重构代码结构，导致原本的连续 case 逻辑被拆分成独立的 case 语句块。这种重构不仅改变了代码的可读性，更重要的是改变了代码的实际执行逻辑。

原始代码示例：

switch (restMethod) {
  case ApiRestMethodEnum.POST:
  case ApiRestMethodEnum.GET:
    console.log("Print me")
    break;
}

被错误重构为：

switch (restMethod) {
  case ApiRestMethodEnum.GET:
    console.log("Print me")
    break;
  case ApiRestMethodEnum.POST:
}

技术影响分析

在 JavaScript/TypeScript 中，连续 case 语句是一种常见的模式，它表示多个 case 共享同一段执行逻辑。这种写法不仅简洁，而且明确表达了开发者的意图：当值为 POST 或 GET 时都执行相同的代码块。

Perfectionist 的错误重构会导致：

1. 只有 GET 情况会执行 console.log
2. POST 情况变成了一个空的 case 语句
3. 完全改变了代码的原始语义

问题根源

这个问题的根本原因在于排序规则没有正确处理连续 case 语句的特殊语法结构。在 AST (抽象语法树) 分析阶段，插件可能将连续的 case 节点视为完全独立的节点进行排序，而没有考虑它们之间的逻辑关联性。

解决方案

该问题已在 Perfectionist 的 3.1.0 版本中得到修复。修复方案主要包括：

1. 增强 AST 分析逻辑，识别连续的 case 语句块
2. 在排序时保持连续 case 语句的完整性
3. 将整个连续 case 块作为一个单元进行排序

最佳实践建议

对于开发者而言，在使用代码排序/格式化工具时应注意：

1. 始终检查自动重构后的代码逻辑是否与原始意图一致
2. 对于关键逻辑部分，考虑暂时禁用自动格式化规则
3. 定期更新 linting 工具以获取最新的错误修复
4. 在团队中建立代码审查流程，特别关注自动工具修改的部分

这个案例也提醒我们，即使是成熟的代码质量工具也可能存在边界情况的问题，保持警惕和定期更新是保证代码质量的重要环节。