Commit 5be2364
committed
fix: SyncResult 语义/格式修复
1. ruff format 报错:SyncResult 与下一行常量之间少一行空行(PEP 8 要求顶层定义间两空行),补上。
2. ruff format 报错:sync_maps 内 logger.info 行长 107 > 100,拆多行参数。
3. SyncResult.message 语义歧义:sync_maps 之前把 _download_maps_remote 的 ready 当成 done 填进 SyncResult。但 ready 其实是"已是最新被跳过 + 本次新下载成功"的累计数,不是"本次下载条数"。当 sync 没有新文件可下时(manifest 全部 sha 匹配),message 会输出"地图更新完成(3309 条)"——用户会以为下了 3309 个文件,但实际什么都没下载。
改造:
- sync_maps 改为 done=done(本次实际下载成功条数),skipped=ready-done。
- SyncResult.message 区分三种成功路径:
- 0 新下载 + 全 skipped: "{name}已是最新(N 条)"
- 部分 done + 部分 skipped: "{name}更新完成(新 X 条,已最新 Y 条)"
- 全新下载无 skipped: "{name}更新完成(X 条)"
修复后行为示例:
- 全部 sha 命中, 无新文件 → "地图已是最新(3309 条)"
- 100 新下载, 3209 已最新 → "地图更新完成(新 100 条,已最新 3209 条)"
- 部分失败 → "地图部分失败(成功 3200,失败 109)"1 parent 76671e7 commit 5be2364
1 file changed
Lines changed: 17 additions & 4 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
44 | 44 | | |
45 | 45 | | |
46 | 46 | | |
| 47 | + | |
| 48 | + | |
| 49 | + | |
| 50 | + | |
47 | 51 | | |
48 | 52 | | |
49 | 53 | | |
| 54 | + | |
50 | 55 | | |
51 | 56 | | |
52 | 57 | | |
| |||
300 | 305 | | |
301 | 306 | | |
302 | 307 | | |
303 | | - | |
| 308 | + | |
| 309 | + | |
| 310 | + | |
| 311 | + | |
| 312 | + | |
| 313 | + | |
| 314 | + | |
| 315 | + | |
| 316 | + | |
304 | 317 | | |
305 | 318 | | |
306 | 319 | | |
307 | | - | |
308 | | - | |
| 320 | + | |
| 321 | + | |
309 | 322 | | |
310 | | - | |
| 323 | + | |
311 | 324 | | |
312 | 325 | | |
313 | 326 | | |
| |||
0 commit comments