CC 评审结论(L3 §20–21)
总评
§20–21 整体可据以推进 §22–29。代码事实核对全过:§21/§15/§16 三处反馈缺口(webhook 仅 received、retry_count 不自增、无置信字段)与代码一致,无 stale。承接 L2 §12 五条 + gap-map 准确,FinBayes 边界清楚(不变量 1,DH 评感知质量不评认知),无残留,守工程层。问题集中在 §20 测试现状精度。
必修 blocking
- B1 §20「约 10+ test」夸大、掩盖核心链路零测试:实有 11 个
*_test.go,但分布是 pkg/ 工具层 6 个(cleaner/llm selector/redislock/txno/cron_human/xprivacy)、standardize 2 个(仅 TestParseLLMResult + JSON 修复)、logic/agent + logic/open(多为 JSON 序列化/config 结构体测试)。internal/job/与internal/agent/零 test 文件——核心数据流(job→agent→push)完全无测试。现表述「单元层=有零散单元测试」给"基本有覆盖"错觉,与事实相反。→ 改:工具层有零散测试,处理/分发/推送主链路无测试,单列缺口行(§22、WP-P1-01)。
建议 non-blocking
- N1 §20
pkg/llmtest 是 provider 选择/配额(selector/image_download),非 LLM 抽取质量测试;括注点明,避免与「标准化质量缺口」混淆。 - N2 §20 契约测试行引 ADR-候选-04(=交付幂等/重试/回放/确认)偏交付契约;与「最小契约(粒度/时效/证据)」是两件事,§22–29 拆 ADR 时明确口径。
- N3 §21「内部质量证据未成统一视图」与 §18 重复,直接锚 §18 减少漂移点。
- N4 §20 测试数据/夹具来源可点一句「回归基线可承接 DH-WP-001 S0 导出样本(生产脱敏 3322 样本)」,利于 §28 任务。
亮点
- 反馈缺口三处与代码逐字吻合:webhook_logic.go 仅 Infof + 返回 received;MaxRetryCount=3 无调用方、RetryCount 恒插 0、retry_count<3 因失败转终态永不触发(精准代码级判断);standardize/raw_news 无 confidence/置信字段。
- FinBayes 边界处理干净(§21 引言 + 末节不变量 1 收口),透明名(「质量与反馈闭环」非照搬「评估闭环」)。
- §15↔§11.6↔§21 协调点闭环自洽(挂账→落定→承接链条完整)。
- L2 §12 五条验收 + gap-map WP 标签逐条对得上。
核验文件:webhook_logic.go、push_consts.go、push_log_model.go、standardize_test.go、agent_logic_test.go、internal/job//internal/agent/(无 test)。