fix(drive): --latest 扫描不完整时拒绝产出 Top-N 并杜绝 sortTime 泄露

[haofeng0705]

背景

本 PR 取代 #885。#885 原本要把内部 wukong 上游的 drive list --latest 整分支同步下来,但核对后发现 main 已有一份独立实现(玉澜 via #868,commit 58dfbc5b,同上游不同港)—— 核心 runDriveListLatest/applyDriveListLatest 字节级一致。整分支移植会覆盖同事已合入的骨架,得不偿失。

因此关闭 #885,改在本 PR 中只搬两个 main 上真实成立的行为修复,不动已有实现骨架。

修复

P1-a sortTime 泄露进输出契约 —— runDriveListDepth 采集端无条件写内部排序字段 sortTime,而 emit 仅在单层 latest(reqDepth==1)路径经 stripDriveDepthDecorations 删除。--depth 2(无 latest)与 depth>1 --latest 两条路径都把 sortTime 泄露进输出。修法:在 emitDriveDepthResult 尾部无条件 delete,一处覆盖正常 emit / SIGINT 取消 / unrecoverable partial 三条路径。采集端保持不动(内部字段,排序时才读)。

P1-b 不完整扫描仍以退出码 0 产出「Top-N」 —— 尾部拒绝 guard 只拦全局截断(truncated),不拦递归途中目录读取失败;后者把可恢复失败记进 errs[] 后照常 emit,Top-N 落在漏扫子树的不完整集合上却冒充全局最新。修法:guard 扩为 latest>0 && (truncated || len(errs)>0),走新增 driveLatestIncompleteError(LATEST_SCAN_TRUNCATED / LATEST_SCAN_INCOMPLETE 双 token,后者带首个失败的 folder/depth/reason);unrecoverable 分支在 latest>0 时不吐 partial,直接回根因错误。同步 --latest flag help 反映新行为。

明确不搬

第三条疑似缺陷(别名互斥漏 --page-size/--page-token)经逐行核对为 main 上的伪缺陷:drive list 从未注册这两个 flag,传入会被 cobra 直接 unknown flag 拒绝,不存在静默失效。搬过去需凭空引入 crossProductAliasPeers 架构 + 注册隐藏别名,违反最小影响原则。

同样不动:main 已有的 runDriveListLatest/applyDriveListLatest/drive_latest.go 骨架(字节级同源)。

测试

新增 internal/helpers/drive_latest_incomplete_test.go 独立锁定两个 P1(避开 main 的 pr868_*_test.go/drive_depth_test.go,杜绝命名撞车):

• sortTime 不泄露(depth>1 --latest 与 --depth 2 无 latest 两条路径)
• 目录读失败拒绝产出 Top-N(断言 LATEST_SCAN_INCOMPLETE 且 stdout 无 items)
• driveLatestIncompleteError 截断/目录失败双分支 + folder 回落
• Suggestion 每子句自洽(「去掉 --latest」子句的示例命令不带 --latest)

改代码前已在纯 origin/main 上验证这些断言对旧代码为红(sortTime 泄露、INCOMPLETE 拒绝均失败),修后转绿 —— 证明测试确实盖得住缺陷。

Reviewer 须知

• 关联:关闭 feat: drive list 支持 --latest 按修改时间取最新 N 个文件 #885,与 fix: sync missing CLI surfaces from wukong cli_to_mcp eval #868 同上游不同港。
• 全部改动只针对 main 的文件布局,单个 commit,+217/-4,4 文件。
• 覆盖率门禁将在 CI 正式跑;本地复刻已在跑(改动为删除语句 + 纯构造器 + 单 if 分支,新分支由测试覆盖)。