TypeScript ESLint 项目中移除 no-loop-func 扩展规则的思考

在 TypeScript ESLint 项目中，开发者们最近讨论了一个关于 @typescript-eslint/no-loop-func 扩展规则是否还有必要存在的技术问题。这个规则最初是为了解决 TypeScript 类型引用在循环语句中的特殊处理而创建的，但随着 ESLint 核心规则的改进，这个扩展规则可能已经完成了它的技术使命。

规则的历史背景

@typescript-eslint/no-loop-func 扩展规则最初是在项目的 #2448 和 #2490 两个 PR 中引入的。当时，这个规则的主要目的是正确处理循环语句中对类型的引用。在 TypeScript 中，开发者可能会在循环内部引用类型，而原始的 ESLint 规则无法正确处理这种情况。

当前的技术现状

随着 ESLint 核心规则的不断改进，特别是在处理未声明的全局变量方面有了更好的支持，这个扩展规则的必要性受到了质疑。开发者发现，即使注释掉规则中专门处理类型引用的代码（.isTypeReference 检查），现有的测试用例仍然全部通过。这表明核心规则已经能够正确处理 TypeScript 中的各种情况。

技术讨论与决策

项目成员对此进行了深入讨论，提出了几种可能的处理方案：

1. 保守方案：先修改扩展规则使其直接运行基础规则，同时添加废弃通知，观察是否有错误报告
2. 激进方案：直接删除扩展规则，因为 ESLint 现在更倾向于在其核心规则中直接支持 TypeScript 语法

经过讨论，团队倾向于采用第二种方案。他们认为，随着 ESLint 对 TypeScript 语法支持度的提高，这类语法不兼容问题应该被视为上游规则的 bug 来处理，而不是通过扩展规则解决。

对开发者的影响

这个变化对使用 TypeScript ESLint 的开发者来说是一个积极的改进：

• 减少了规则集的复杂性
• 提高了与 ESLint 核心规则的一致性
• 简化了项目的维护工作

总结

这个讨论反映了开源项目中一个常见的技术演进过程：随着上游项目的改进，一些专门为解决特定问题而创建的扩展可能会变得不再必要。TypeScript ESLint 团队通过审慎的技术评估，决定简化规则集，这既体现了对项目质量的追求，也展示了开源社区协作的价值。