增量质检报告 · code-scan-hub v1.0.15

• 项目：codescan/code-scan-hub
• 分支：v1.0.15
• 基准版本（base）：7fdeaadf5bd55cb2c386ac7eaefa6d7bd68656de
• 目标版本（head）：f43f3de0b2b7ed2a8764a2b68cec26bf5ade3f45
• 数据来源：codescan.issues_diff（增量问题对比）+ codescan.git_diff（变更面）+ codescan.quality_list（严重度核对）
• 严重度分布（head 全量）：Critical = 0，High = 0，其余均为 Medium / Low（质量与风格类，无安全/正确性高危项）

本文「修复 bug / 新增问题」两章完全基于 codescan.issues_diff 的真实返回数据；变更面描述基于 git_diff。未做任何推断性编造。

---

一、变更概览

本版本是**「push 历史回填」特性**的完整落地（spec 2026-08-31-push-backfill），并顺带修复了数据缺口、钉钉推送白名单、分析时段多时段等问题。

1.1 变更文件清单（12 个）

---

二、修复的问题（修复 654 条）

issues_diff 给出的「修复」= base 有、head 无，共 654 条。

2.1 结论：绝大多数是「行号漂移」造成的统计位移，而非语义修复

scheduler.rs、repos.rs、gitea.rs、insight_settings.rs 在本版本被大规模插入新代码，导致同一批存量告警的行号整体后移。质检按行号匹配问题，旧行号位置消失即被计入「修复」，新行号位置则被计入「新增」。直接证据（同一问题、仅行号偏移）：

因此本章「修复 654 条」不宜解读为该版本真实修掉了 654 个缺陷。

2.2 真实发生的语义修复（源码注释可佐证，共 6 项）

1. 提交/文件/问题落库幂等（repos.rs::insert_commits / insert_commit_files / insert_issues_with_commit）—— 新增「先 SELECT 已存在、再 INSERT」去重，修复重复执行 persist_insight 导致的提交量/代码产出/问题数重复计数（注释标注 2026-09-02 重复事故，既有 36 例同 push 重复 commit）。
2. 回填补齐效能统计缺口（backfill_service.rs）—— 回填任务合成 push_id，使 persist_insight 不再早退，回填数据与正常 push 同口径落 commits/commit_files。
3. 员工 username 污染（repos.rs::bind_employee_username）—— 语义由「相同即返回」改为「仅当 username 为空才回填」，避免回填合成 push 的 pusher=backfill / 代推场景覆盖真实 Gitea login（2026-09-02 事故）。
4. 个人账号仓库可见性（gitea.rs::list_personal_repos）—— /user/repos（仅 owner/协作者）改为 /repos/search?q=（admin 全量），补回「公开/非协作者」仓库。
5. 回填不被分析时段闸拦住（scheduler.rs + repos.rs::exists_queued_backfill_task）—— 管理员主动补数据动作豁免时段约束（2026-09-06 实测 36 任务 queued 8 分钟无派发）。
6. /repos/search 响应体对齐（insight_webhook.rs 测试）—— mock 改为 {"data":[...]}，并同步 gitea.rs 新增 GiteaRepoSearchResult。

---

三、新增问题（新增 716 条）

issues_diff 给出的「新增」= head 有、base 无，共 716 条（严重度全部为 Medium / Low，无 Critical / High）。

3.1 按类别聚合（主导类别）

3.2 真正由「本次新增代码」引入的问题（需重点关注）

以下条目确实落在本次新增文件/新增函数上，不属于行号漂移：

① src/api/backfill.rs（新文件，14 条）

• [Medium/Dependency] 高耦合 @104：backfill handler 直接依赖 12 个函数（超阈值 10）
• [Low/Maintainability] 重复代码块 @125 / @232 / @245：401/400/JSON 错误响应块重复 3~5 次 → 建议抽 render_error(res, status, msg) 帮助函数
• [Medium/ErrorHandling] 无安全注释的 unwrap @237/238/333/334/348/349/363/364/378/379：集中在 #[cfg(test)] 域（take_json().unwrap()、as_str().unwrap()），生产代码本身未新增 unwrap

② src/orchestrator/backfill_service.rs（新文件，17 条）

• [Medium/Complexity] 函数过长 @53：enumerate_and_backfill 135 行 + 嵌套过深
• [Low/Dependency] 扇入扇出异常 / [Medium/Dependency] 高耦合 @53：fan-out 26（组织展开 + 枚举 + 过滤 + 聚簇 + upsert 项目 + 建任务 + 合成 push 全塞一处）→ 建议按「枚举层 / 落库层」拆函数
• [Low/Maintainability] 重复代码块 @106、行过长 @144
• [Medium/ErrorHandling] 无安全注释的 unwrap @292/304/305/307/308/314/316/317/391：均在 #[cfg(test)] 测试域

③ src/orchestrator/backfill.rs（新文件，1 条）

• [Medium/ErrorHandling] 无安全注释的 unwrap @81：测试域

④ src/client/gitea.rs（新增 commits() + 改造 list_personal_repos，约 41 条）

• [Medium/Dependency] 高耦合：commits @413 依赖 15 个函数；list_personal_repos @369 依赖 15 个；list_orgs @320 依赖 16 个
• [Low/Dependency] 扇入扇出异常：mock_server @492（fan-in 19 / fan-out 16）
• [Low/Maintainability] 行过长 @513
• [Medium/ErrorHandling] 无安全注释的 unwrap @496/501/532/…/752/753（约 29 条，绝大多数为测试断言 unwrap）

⑤ src/orchestrator/scheduler.rs（新增 notify_push_task_types 等）

• [Medium/Dependency] 高耦合 @436：notify_push_task_types 依赖 12 个函数
• [Medium/Complexity] 函数过长：complete_task @454（95 行）、persist_insight @560（314 行 / 圈复杂度 24）——注意 persist_insight 的膨胀与本次无关，属存量
• [Low/Maintainability] 注释代码过多 @191、重复代码块 @262

⑥ src/store/repos.rs（新增 3 个回填函数 + 幂等改造）

• [Medium/Dependency] 高耦合 @263：create_backfill_task 依赖 17 个函数（幂等判定 + 建任务链路过长）
• [Low/Complexity] 参数过多：upsert_version(9) @571、upsert_push_by_delivery(8) @712
• 新增/位移 unwrap 集中在 #[cfg(test)] 域（@2699 起）

说明：scheduler.rs（约 300 条）与 repos.rs（约 350 条）的「新增」绝大部分是测试域 unwrap 告警的位移（见第二章证据），并非新逻辑引入，故归于行号漂移，仅其中少量挂在新函数上。

3.3 代码审阅发现的「扫描器未覆盖」的残留风险（供参考，非 issues_diff 数据）

以下为人工读码观察，不在质检问题清单内，仅提示：

1. 幂等去重存在读-写竞态：insert_commits / insert_commit_files / insert_issues_with_commit 采用「SELECT 已存在 → INSERT」而非 DB 唯一约束。注释针对的是 persist_insight 重复（串行）执行；若同一 push 被并发触发两次，两次 SELECT 可能都通过，仍会双写。建议补 (push_id, sha) / (commit_id, path) / 问题键唯一索引兜底。
2. 回填计数可能高估：backfill_service.rs 中合成 push 失败时仍 report.tasks_created += 1（注释声明为有意，任务已建、仅该簇落库跳过）。调用方（insight）若以 tasks_created 为「成功回填数」展示，需知悉口径。
3. 根提交簇被静默跳过：base_sha 为空的簇 continue（初始提交无法增量扫描），注释说明为「可接受缺口」，建议在响应 BackfillResponse 中显式暴露 skipped_root 计数，避免用户误以为全量覆盖。
4. backfill.rs::backfill 用 expect("AppState 未注入")：与其余 handler 风格一致（依赖注入保证），非阻塞项。

---

四、结论与建议

1. 版本定位：v1.0.15 为「push 历史回填」特性的功能版本，变更面集中在回填链路（3 个新文件 + 5 处关联改造），并附带 6 项数据缺口/推送策略修复。
2. 质量水位：无 Critical / High；增量 716 条新增告警无一是新逻辑的正确性/安全缺陷，主导类别为「测试域 unwrap 无 SAFETY 注释」与「长函数高耦合」，属可维护性债。
3. 统计口径提醒：修复 654 / 新增 716 两个数字受行号漂移严重放大（同问题新旧位置各计一次），不应直接当作缺陷净变化量。
4. 优先处理建议：
   • P1：enumerate_and_backfill（135 行 / fan-out 26）与 create_backfill_task（17 依赖）抽函数/分层，降低耦合；
   • P1：为 insert_* 幂等路径补唯一索引，消除并发双写竞态；
   • P2：backfill.rs 抽公共错误渲染函数，消除 3 处重复块；
   • P3：BackfillResponse 增补 skipped_root 字段，提升回填结果可观测性。