[Medium] 编码规范不统一：ES6+ 与旧语法混用

[Colin4k1024]

问题描述

项目中存在混合使用 ES6+ 和旧 JavaScript 语法的情况。

问题示例

// 新语法
const result = selectGene(...);
const signals = [];

// 旧语法 (var)
var OPPORTUNITY_SIGNALS = [...];
var list = Array.isArray(signals) ? signals : [];
for (var i = 0; i < OPPORTUNITY_SIGNALS.length; i++) {

影响

• 代码风格不一致
• 维护困难
• 开发者困惑

建议

1. 统一使用 ES6+ 语法
2. 使用 ESLint 进行代码风格检查
3. 考虑迁移到 TypeScript

---

严重程度: Medium
分类: Code Quality

[autogame-17]

Thank you @Colin4k1024 for flagging this. The inconsistency between var and const/let is a valid code quality concern.

Context:

Evolver uses CommonJS (require/module.exports) and some modules were written in an earlier style. The var usage is concentrated in a few files (skillDistiller.js, a2aProtocol.js, issueReporter.js) that were authored or generated in earlier development phases.

Action plan:

1. New code will use const/let exclusively
2. We plan to add ESLint with the no-var rule (initially as a warning) to enforce this going forward
3. Existing var usage will be migrated incrementally -- this is safe for CommonJS modules and doesn't change behavior

Regarding TypeScript migration:

This is out of scope for the foreseeable future. Evolver is designed to be lightweight, self-contained, and runnable without a build step. TypeScript would add a compilation dependency that conflicts with the project's design goals.

Thank you for the suggestions.