增量质检报告 · dzs/dzs-admin v1.0.103

• 项目：dzs/dzs-admin
• 分支：v1.0.103
• 对比区间：fb3d3f411b585cbd9f325cfdde8c73523cc1b651（base，即 v1.0.102 发布提交）→ 8cf7013235afb2b77430affeff7b291fd16dc29f（head）
• 质检口径：issues_diff 真实数据（修复 = base 有 head 无；新增 = head 有 base 无）+ 变更面复核
• 质检结论：修复 8 条 / 新增 11 条，无 Critical、无 High；新增问题全部集中在 e2e 测试与既有复杂函数，业务代码未见新缺陷

---

一、版本主题与变更规模

本次变更是一个纯前端 UI 交互改造：把全后台表格行操作的「type="link" 平铺按钮组」升级为 「主操作常显 + 『更多』下拉」（新增 RowMore 组件），把低频 / 危险动作收进下拉，从而真正收窄操作列整体宽度 （第一轮「紧凑相连」只省按钮之间的间隙）。文档自称覆盖「全后台 46 处操作列」。

核心实现（src/components/common/RowActions.tsx）

• RowActions：Space.Compact 紧凑相连按钮组（沿用）。
• RowMore：仅图标按钮（30px，aria-label="更多操作"）+ Dropdown，菜单项 onClick 里若带 confirm 则改用命令式 Modal.confirm（替代原 Popconfirm，因为菜单一点即卸载）。
• 约定：RowMore 必须作为 RowActions 的最后一个子元素，且 items 为空时调用方不渲染（length > 0 &&）， 否则紧凑组首尾圆角类名会算错；confirm 与业务 handler 内已有的 Modal.confirm 不得重复。

---

二、修复的问题（8 条，均 Low/Medium，无 High/Critical）

说明：#5~#8 与新增 #7~#10 是同一函数 AttrValuesDrawer 因编辑导致行号从 662 漂移到 698， 属于「修复/新增」成对误报，并非本次新增或真修复。评审时按同一处历史欠账看待即可。

---

三、新增的问题（11 条，均 Medium/Low，无 High/Critical）

小结：11 条中有 6 条为行号漂移的误报（集中在 attributes/page.tsx 的 AttrValuesDrawer）， 5 条为真实的低风险新增——其中 4 条在 e2e（重复块/耦合），唯一落在产品源码的是 RowActions.tsx:76 的嵌套深度。无任何 Critical/High，无功能性缺陷。

---

四、变更面复核（补充发现）

对 43 个变更文件做了针对性复核，重点排查「重构后遗留的死代码 / 未使用导入 / 悬空引用」：

1. 未使用导入 / 悬空变量：未发现问题。 逐符号核查了 9 个高改动量页面：metrics、tags、commodities、columns、content-checks、 topics、article-types、event-data、industries。

   • 被删的 Popconfirm 在各文件中确已既未 import 也未使用（columns/content-checks/article-types）； 其余「仍保留 Popconfirm」的文件（commodities 属性/合约表、event-data 批量操作、industries 产能图）用法合法。
   • 被误以为会变「未使用」的 Tooltip/Space/theme(token)，经确认在各自文件中仍有其它使用点，import 合法。
   • 删除的 deleteCheckingId 状态（commodities、metrics）已彻底清除，仅剩注释说明，无 setDeleteCheckingId 残留。 → 不会引发 tsc/eslint 报错。

2. RowMore 空数组约定一致性：部分调用方用 （如 、、、…）， 另一部分直接 （、、、、、）。 后者当前 items 均为， 内部 不会触发，首尾圆角正确。 → 属风格不统一，非缺陷；但若日后这些页面改成条件渲染，需改用 写法（组件注释已约定）。

---

五、重点风险与回归关注点

---

六、结论

• 本次 v1.0.103 是一次边界清晰的纯前端重构（引入 RowMore，全后台操作列二次收窄），无后端影响。
• 质量趋势可控：修复 8 / 新增 11，其中新增里 6 条为行号漂移误报、5 条为低风险新增， 无 Critical / High，未发现功能性缺陷、未使用导入或悬空引用。
• 唯一落在产品源码的新增问题是 RowActions.tsx:76 的嵌套深度（Medium，建议抽取 handleMenuClick）。
• 最需要人工关注的是「交互动线变更」与「scroll.x 列宽同步」这两类回归风险，建议在后续批次 补一条权限可见性 e2e 与 RowMore 轻量单测。

附：codescan_quality_list 全量 1708 条中含 1 条 Critical/Security（e2e/helpers/db.ts:78 SQL 拼接）， 但不在本次 diff 变更范围内，属历史存量，非本版本引入，此处仅备注。