DarkiT / zcli

55%

Build Type

push

github

Committed by cnb

Commit Message

fix(service): 保证强制退出定时器的排布先于运行收尾

Stop 原本在 stopMu 临界区之外排强制退出定时器，而 Run 的启动同样要持有
stopMu。两者没有顺序保证时，Run 可能在定时器排布之前就收尾：它的
cancelForceExit 空跑，随后排下的定时器再没有运行来取消，最终误触发强制
退出——把一个正常的进程杀掉。

「停止意图」修复（d58e0f7）之后这个问题被放大：Run 领受意图时会立即返回，
收尾大幅提早，竞争窗口随之变宽。实测 TestForceExit_CancelledBeforeTimeout
失败率 4/10（GitHub 线 2/10），此前 3 轮全绿只是运气。

修复方式是把 scheduleForceExit 移进 stopMu 临界区内，用锁序保证
「先排定时器、后收尾」，而不是用 running 之类的状态判断——后者会误伤
daemon 模式下「服务从未启动也要兜底强杀」的场景
（TestForceExit_DaemonOnly/daemon_schedules_force_exit 会失败）。

验证：
- go test -race -count=10 -run TestForceExit . → 0 失败（修复前 4/10）
- go test -race -count=3 ./... → 全绿，含 tests/e2e

Coverage Stats

6 of 6 new or added lines in 1 file covered. (100.0%)

226 existing lines in 5 files now uncovered.

2641 of 4776 relevant lines covered (55.3%)

50.67 hits per line