增量代码质检报告 · huanyan/server v0.1.43

一、版本与对比范围

二、变更概览

本次提交为员工端（mp）补齐缺口文档 §3 剩余 3 条接口，属于功能增量：

1. 新增 src/modules/app/about.rs（112 行）：GET /api/v1/app/about 帮助与关于页。
   • 品牌名取全局参数 brand.name（param_override.store_id IS NULL），未配置回落「量星衍」；
   • 版本取 crate::app_version()（与 /healthz 同源）；
   • 门店电话/名称/地址取会话门店档案；statement 为 PRD §20.2 原句；copyright 恒 null。
2. 新增 src/modules/app/appointments.rs（318 行）：
   • GET /api/v1/app/appointments/{id} 预约详情（含动作回显 canArrive/canCancel/canReschedule，手机号恒脱敏）；
   • POST /api/v1/app/appointments/{id}/arrive 到店确认（条件更新 status='已登记'→'已到店'，0 行回 1005 并回报现状；待分配单同动作接单并写当日委派授权 grant_today；落审计）。
3. src/router.rs：挂载 /app/about（AppFuncGuardAny(STAFF_APP_ALLOWED)）与 /app/appointments/{id}＋/arrive（AppFuncGuard(appointment.register)）。
4. src/modules/app/mod.rs：声明 pub mod about; pub mod appointments;。

安全复核（人工）：详情接口使用 mask_mobile，不下发完整手机号；越权路径区分「无令牌 2001 / 缺功能点 2003 / 跨店或非当事人 2004」；到店确认使用条件更新 + 事务，具备并发/重复点击幂等兜底；待分配单接单后回填护理师并写委派授权。未发现越权或 SQL 注入等安全缺陷。

三、修复的问题（base 有、head 无）

固定对比数据：0 条（codescan_issues_diff 真实结果）。

本批以「新增功能 + 新增测试」为主，未收敛存量质检告警；唯一实质修复是测试夹具的时区口径（CURRENT_DATE → 北京日），属测试稳定性修复，未计入质量告警口径。

四、新增的问题（head 有、base 无）

共 20 条：Medium 10 条 / Low 10 条，无 High / Critical。

4.1 按文件分布

4.2 明细清单

五、人工复核与风险分析

以下为结合源码做出的补充判断（与自动告警区分）：

• 高耦合/扇出告警（#2/#3/#8/#9/#11/#12）：本质是单函数内串联 entity 查询与 DTO 组装，扇出计数偏高但逻辑线性、无副作用，实际风险低；属可维护性建议而非缺陷。
• detail_inner 过长（#7）：87 行内混合客户/项目/门店/员工批量查询与展示串计算，建议抽出 load_names / build_detail 等私有 helper，降低后续改动回归面。
• 重复代码块（#5/#10）：两处为相似的 entity 查询/DTO 组装片段，可合并，收益有限。
• 测试函数过长（#14）与 4 处 unwrap（#17–#20）：位于 tests/，unwrap 用于取登录 token，测试场景失败即 panic 属可接受，但建议按「详情 / 到店 / 关于页」拆成多个测试函数并补 expect("...") 说明。
• arrive_inner 事务 0 行分支：begin() 后直接 return Err(...)，依赖 DatabaseTransaction 的 Drop 回滚（未显式 rollback()）；语义正确，但建议显式回滚以提升可读性与确定性。
• 功能正确性：到店确认以 status='已登记' 作条件更新闸门，重复点击回 1005 并回报当前状态；待分配单接单回填 staff_id 并复用 grant_today（ON CONFLICT 幂等），设计自洽。about 端点品牌名查询与兜底、版本同源、手机号脱敏均符合 PRD/文档口径。

六、建议

1. 优先（本批可顺手）：arrive_inner 的 0 行分支改为显式 txn.rollback().await? 后再返回，避免依赖 Drop 语义。
2. 建议（后续重构）：拆分 detail_inner 与 about_inner 的查询/组装逻辑；把 app_appointment_detail_arrive_about_flow 按场景拆为 3–4 个测试函数。
3. 可选：为测试中的 .unwrap() 补 expect("...")；合并 appointments.rs 中的重复查询块。
4. 无需处理：所有新增告警均为 Medium/Low 的可维护性、耦合度与测试风格项，无阻断发版的 High/Critical 问题。

七、结论

本次 v0.1.43 增量以「补齐员工端关于页 / 预约详情 / 到店确认」为主线，新增 3 个接口、路由挂载、模块声明与端到端测试，并顺带修复了测试夹具的北京日时区口径。安全与越权路径经人工复核无缺陷。新增质检问题 20 条（Medium 10 / Low 10，0 High / 0 Critical），集中在新增文件与测试的可维护性与耦合度，不构成发版阻断，建议按第六节优先级在后续迭代中收敛。