review-pipe-type-namespace.md 4.5 KB

管网编码与设施命名空间:收尾审查报告

审查日期:2026-10-12。初始范围为后端提交 740d896,固定比较 git diff 94971b5...740d896;规格为管网编码 0/1 映射与设备边界设施类型命名空间及维护者已确认的两项裁定。规范与规格两个轴独立审查,随后对发现项补齐并回归。

Standards(规范轴)

未发现适用文档规范违规或值得阻塞的代码结构问题。数字编码 0/1/2 与现行业务 GIS 规则一致,设施使用 GAS_PIPE/WATER_PIPE/DRAINAGE_PIPE,与监测设备类型主键隔离。无法映射的管网上行到既有未知项计数,而不是在适配器静默丢失。

Spec(规格轴)

原提交存在两项规格完整性缺口,目前已补齐:

优先级 原问题 补齐
P2 票面要求跨端点管网总量一致的契约测试,原测试主要逐适配器验证,无法锁住五端点的共同口径 新增 CockpitPipeNetworkCrossEndpointContractTest,同一份管网表数据经过五套真实适配器与服务,只 mock 数据源;逐行业精确比较基础档案、管理信息、运行信息、概览和设备边界的管网计数,并验证 3/99/null 未知项的五处排除计数
P3 基础档案的三类管网名称和行业仍独立硬编码,类型单源部分完成 改为从 CockpitPipeNetworkType 直接取得类型码、中文名及行业,保留原有列表顺序与当前值

这两项没有证实当前线上映射错误;它们是自动回归与单源传播的交付缺口。补齐未修改数据库、前端代码或业务写流程。

验证与实际部署边界

  • 补充两个跨端点用例均通过。
  • 全量 reactor BUILD SUCCESS:zksy-system 187 项、zksy-admin 441 项,合计 628 项,0 失败、0 错误、0 跳过。
  • 更正此前计数表述:“439/439 全量”是 zksy-admin 模块小计,并非所有模块合计;本次分别记录两个模块,避免口径混用。
  • 真实运行实例 PID 25880 已验证主修复 740d896 的行为。补齐只将一致的显示字典定义改为共享读取、增加测试,当前部署的主修复现场证据与补齐源码回归分别登记,不凭进程启动时间推定后续补齐已经部署。
  • 本次未做自动浏览器走查或截图;界面只采用维护者“粗验未发现问题”的反馈,不宣称完整人工验收通过。

现场复验摘要

对象域 设施总数 管网计数核对
基础档案 5278 燃气 537/供水 996/排水 3735
总体概览 5276 同上;排除未知类型 0
设备边界 5268 同上;名称正确,没有管网被标为“供水设备/排水设备”
管理信息 5273 含 5268 管网 + 3 窨井 + 2 水源地;管网归未分区
运行信息 不按设施总数比较 三行业管网长度指标条数 537/996/3735

不同端点对象域不同,设施总数无需相等;共同的管网集合为 5268。运行信息的管网长度单位为 m,不能将原始 value 当作 km。

远端并行更新后的最终验证

审查补齐提交 29d1ae2,整合远端报警描述更新后的合并提交 0a6b108 均已推送。规范和规格补齐再经只读复核,P2/P3 闭环,无新增阻塞。

  • 同步远端前:zksy-system 187 + zksy-admin 441,合计 628 项全部通过。
  • 同步远端报警描述提交 86b0c8d 后:全量在 zksy-system 的 AlarmLifecycleServiceTest.insertWritesLogBeforeCommittingWithoutTrigger 出现 1 个 VerificationInOrderFailure(190 项、1 失败),停在该模块,不能宣称当前整合树全量通过。报警日志断言涉及的源码与测试与远端完全一致,本次管网补齐没有改动它们;本轮只记录,不扩展到报警领域修复。
  • 整合后的管网专项回归:29 项全部通过(包含五端点同源的正常/未知值两组测试)。
  • 并行会话的工单读适配器与其测试改动留在工作树,未暂存、未代提交。
  • 已部署主修复为 740d896;审查补齐为 29d1ae2,并未声称当前运行实例加载了后续合并的所有报警改动。单源补齐保持现有管网名称、行业和排列不变。

规范轴未报告问题;规格轴初始发现 P2、P3 两项并完成补齐。人工界面核验按人工核验步骤继续,整体阶段仍归前端真实接口联调与阶段收口。