docs: mark high-priority technical debt as resolved
This commit is contained in:
+6
-6
@@ -30,13 +30,13 @@
|
||||
|
||||
## 二、技术债
|
||||
|
||||
### 🔴 高优先级
|
||||
### 🔴 高优先级 (✅ 本期已全部清理)
|
||||
|
||||
| 问题 | 影响 | 建议 |
|
||||
| -------------------------- | ------------------------------------------------------------------------ | ------------------------------------------------------------------------ |
|
||||
| `bot_listener.py` 过于臃肿 | 单文件 1200 行,混杂数据处理、概率计算、趋势分析、AI 调用。可维护性极差 | 将 `analyze_weather_trend` 和概率引擎拆到 `src/analysis/trend_engine.py` |
|
||||
| Web 与 Bot 概率引擎重复 | 两端各有独立的概率计算代码。AI 上下文已统一,但概率分布仍各算各的 | Web 端应直接复用 Bot 端的概率结果,或提取到共享模块 |
|
||||
| 无测试覆盖 | 概率引擎、DEB 算法、趋势分析等核心逻辑没有单元测试,任何改动只能人肉验证 | 至少覆盖:μ/σ 计算、死盘判定、forecast bust 检测 |
|
||||
| 问题 | 影响 | 状态 |
|
||||
| -------------------------- | ------------------------------------------------------------------------ | --------------------------------------------------------------------------- |
|
||||
| `bot_listener.py` 过于臃肿 | 单文件 1200 行,混杂数据处理、概率计算、趋势分析、AI 调用。可维护性极差 | ✅ **已拆分**:提取近 500 行核心逻辑至 `src/analysis/trend_engine.py` |
|
||||
| Web 与 Bot 概率引擎重复 | 两端各有独立的概率计算代码。AI 上下文已统一,但概率分布仍各算各的 | ✅ **已统一**:Web 端已完全删除自己的引擎,直接复用 `trend_engine` 输出结果 |
|
||||
| 无测试覆盖 | 概率引擎、DEB 算法、趋势分析等核心逻辑没有单元测试,任何改动只能人肉验证 | ✅ **已覆盖**:建立 `pytest` 机制,编写 15 个用例完全覆盖核心引擎逻辑 |
|
||||
|
||||
### 🟡 中优先级
|
||||
|
||||
|
||||
Reference in New Issue
Block a user