Archived
snowy-yang
65a81af94a
为什么：_task_monitor_loop 是一个 170 行 / 圈复杂度 58 / 嵌套 8 层的单函数，把
7 组互不相关的周期性作业塞在一起——终态回收、超时取消、缓存清扫、限流桶清理、
账本对账、孤儿目录补扫、心跳离线——整轮共用一个 try/except。任何一处改动都要
通读全部 7 组逻辑；上一轮恢复它的启动点时，也正是这个体量让判读很吃力。
改法：一组作业一个具名函数，另抽 _run_monitor_tick 只负责编排顺序（循环本身
只剩「sleep + try + 限频记录」三件事）。
复杂度（同口径 AST 实测，重构前后）：
_task_monitor_loop 170 行 / 圈 58 / 嵌套 8 → 17 行 / 圈 3 / 嵌套 2
文件内最大函数 170 行 → 66 行（_reap_finished_tasks）
文件内 >80 行函数 1 → 0 · 嵌套 ≥5 层 1 → 0
代价：总行数 231 → 291（+60，全部是新的函数签名与 docstring）
等价性证据（重构的分内之事，不是附赠）：
1. AST **数值字面量多重集**比对：旧 16 种 / 新 16 种，计数零差异——确认搬运
中没有改任何阈值（4h 硬超时 / 8 次重试 / 60s / 24h / 5h / 90 天 / 600s …）。
2. 剔除 docstring 后的**字符串字面量**比对：旧版有的一处不少——确认用户可见
状态文案与日志格式没被抄错。
3. 新增 20 例行为契约测试（tests/test_monitor_jobs.py），按「调用后世界里发生
了什么」断言，不引用实现细节，故可跨后续重构复用。
刻意保留的语义（是决定，不是疏漏）：
- 单个 tick 仍共用一个 try/except：前段作业抛异常会跳过本轮剩余作业。改为
逐作业隔离属于**行为变更**，本提交不夹带；已用 test_tick_exception_skips_
rest_of_tick 如实钉住现状，让那次变更有据可依、不会悄悄发生。
- _run_periodic_ledger_jobs / _sweep_orphan_task_dirs 的局部 try/except 原样
保留（原注释即写明「下轮重试」意图），其兜底粒度各有独立用例。
- 代结算的前提仍是「worker_thread 字段存在且线程已死」：没有线程句柄的任务
不退款——拿不到句柄就无法判断 worker 是否已在别处结算过，贸然退会双退。
该约束由新增用例固化（原实现如此，此前无测试记录）。
验证：ruff 全过 · pyright 0/0 · pytest 209 passed（189→209）·
生产 data/ 未被触碰（全部用例走 tmp_path 沙箱）。