DarkiT / zcli / 32234253943

55%

push

github

fix: 对抗审查修复三处边界问题并固化契约测试

全模块对抗测试审查发现并修复：

1. CombineErrors 单错误不再原样返回——errors.Join 对单个错误会额外包装
   一层 joinError，破坏与原始错误的同一性（==/指针比较失效，v0.5.0 引入）。
   现在：无错误返回 nil、单非 nil 错误原样返回、多个用 errors.Join 聚合。

2. (*Cli).AddCommand(nil) 直接 panic——cobra AddCommand 不防御 nil 命令，
   库透传导致崩溃。现在静默忽略 nil 命令。

3. ErrorAggregator 并发 Add 存在 data race 且丢错误——内部无锁。补
   sync.Mutex 保护全部方法，-race 下并发 Add 不丢错误。

审查覆盖：Builder 边界、配置克隆深拷贝、errors chain、i18n 回退、
命令管理、flag 导出去重、Config 独立克隆、Args 校验器、nil 接收者、
未安装服务 start 等边界，均验证通过并固化为对抗契约测试。

验证：go build / go vet / go test . ./tests/e2e / go test -race . 全绿。

31 of 31 new or added lines in 2 files covered. (100.0%)

2575 of 4683 relevant lines covered (54.99%)

19.24 hits per line