eslint-plugin-perfectionist 项目中静态属性依赖检测的修复

在 JavaScript/TypeScript 类定义中，属性之间的依赖关系是一个需要特别注意的问题。eslint-plugin-perfectionist 是一个用于帮助开发者保持代码风格一致性的 ESLint 插件，其中的 sort-classes 规则负责对类成员进行排序。

最近发现该插件在处理静态属性依赖时存在一个重要的边界情况未被正确处理。具体表现为：当静态属性通过类名（如 MyClass.x）引用其他静态属性时，这些依赖关系没有被正确识别，导致可能产生错误的排序结果。

问题分析

正常情况下，sort-classes 规则能够识别类实例属性之间通过 this.x 形式的依赖关系。例如：

class Example {
  b = 1;
  a = this.b; // 正确识别为a依赖于b
}

然而，对于静态属性的处理存在缺陷：

class Class {
  static b = 1;
  static a = Class.b; // 未能识别a依赖于b
}

这种静态属性间的引用关系在实际开发中相当常见，特别是在需要共享常量或配置的场景下。插件未能识别这种依赖关系，可能导致自动排序后产生运行时错误。

技术实现细节

静态属性引用与实例属性引用在语法结构上有明显差异：

1. 实例属性通过 this 关键字访问
2. 静态属性通过类名直接访问

在 AST（抽象语法树）解析层面，这两种访问方式属于不同的节点类型，需要分别处理。原始实现可能只考虑了 this 形式的引用，而忽略了类名直接引用的情况。

修复方案

修复此问题需要在依赖检测逻辑中增加对静态属性引用的支持。具体需要：

1. 识别 MemberExpression 节点
2. 检查对象是否为当前类名
3. 收集被引用的属性名作为依赖

同时需要考虑以下边界情况：

• 类名可能通过别名引用
• 可能存在多层属性访问（如 Class.config.value）
• 需要处理 TypeScript 的类型断言语法

对开发者的影响

此修复确保了类成员排序时能正确处理所有形式的属性依赖关系，包括：

• 实例属性间的依赖
• 静态属性间的依赖
• 可能的跨层级引用

开发者现在可以更安全地使用自动排序功能，不必担心静态属性间的依赖关系被破坏。

最佳实践建议

尽管工具已经能够自动处理依赖关系，仍建议开发者在组织类成员时：

1. 将基础/独立的属性放在前面
2. 将有依赖关系的属性分组放置
3. 对于复杂依赖，考虑添加注释说明
4. 定期运行 lint 检查保持一致性

这个修复体现了静态代码分析工具在维护代码质量方面的重要性，也展示了开源社区通过 issue 跟踪和协作解决问题的典型流程。