[tech-debt] 前端硬编码旧组名 → 组改名后标签全部退化 #95

New Issue

tanzk · 2026-09-14T17:52:44+08:00

tanzk commented

2026-09-14 17:52:44 +08:00

[tech-debt] 前端硬编码旧组名 → 组改名后标签全部退化

现象

web/src/views/dashboards/RefundRecovery.vue:394：

const GRP_COLOR_MAP = {
  '朱海洋组': 'success', '董梦玲组': 'warning', '吴晓龙组': 'primary',
  '叶珍凤组': 'danger', '投诉专项组': 'info',
}
function grpTagType(g) {
  if (!g || g === '未分组') return 'info'
  return GRP_COLOR_MAP[g] || 'info'     // ← 查不到 → 全部退化 info
}

但 2026-09-14 组织调整已把组改名（生产实测 persons.group_name 现值）：

接待1组 / 接待2组 / 接待3组 / 专项1组 / 投诉专项组

映射表里 朱海洋组/董梦玲组/吴晓龙组/叶珍凤组 4 个 key 全部失效
→ 所有分组标签退化为灰色 info，颜色区分功能完全丧失（静默，无报错）。

根因

颜色映射硬编码了组名。组名是可变的业务数据（用户会改名，
且本仓已有改名先例），把它当代码常量使用 → 数据一变即失效。

同类风险点（需一并排查）：

web/src/views/dashboards/Today.vue 值班条 DUTY_POSITIONS / DUTY_ROLE_META
web/src/views/dashboards/OperationDashboard.vue 岗位徽标
上述两处已改为读 s.position（动态），是正确做法 → 可作为改造范式

修复

不再按名字映射颜色，改为按稳定标识：

优先方案：后端在分组接口返回 color 或 sort_order 字段（或前端按 group_id 取色），
前端只消费不推断；
兜底方案：前端按组名数组的下标取色板（GRP_COLOR_PALETTE[i % len]），
组名变化不影响配色稳定性；
全仓排查同类「硬编码业务值」：组名 / 人名 / 专项名。

验收标准

grep -rn "叶珍凤组\|董梦玲组\|吴晓龙组\|朱海洋组" web/src/ → 0 命中
5 个组的标签颜色在生产互不相同（浏览器实测取色）
组再次改名后，标签颜色不退化（模拟测试：改名后仍取到色板颜色）
排查报告：列出全仓其余「业务数据硬编码为代码常量」位置

[tech-debt] 前端硬编码旧组名 → 组改名后标签全部退化 ## 现象 `web/src/views/dashboards/RefundRecovery.vue:394`： ```js const GRP_COLOR_MAP = { '朱海洋组': 'success', '董梦玲组': 'warning', '吴晓龙组': 'primary', '叶珍凤组': 'danger', '投诉专项组': 'info', } function grpTagType(g) { if (!g || g === '未分组') return 'info' return GRP_COLOR_MAP[g] || 'info' // ← 查不到 → 全部退化 info } ``` 但 **2026-09-14 组织调整已把组改名**（生产实测 `persons.group_name` 现值）： ``` 接待1组 / 接待2组 / 接待3组 / 专项1组 / 投诉专项组 ``` 映射表里 `朱海洋组`/`董梦玲组`/`吴晓龙组`/`叶珍凤组` **4 个 key 全部失效** → 所有分组标签退化为灰色 `info`，**颜色区分功能完全丧失**（静默，无报错）。 ## 根因 **颜色映射硬编码了组名**。组名是可变的**业务数据**（用户会改名，且本仓已有改名先例），把它当**代码常量**使用 → 数据一变即失效。同类风险点（需一并排查）： - `web/src/views/dashboards/Today.vue` 值班条 `DUTY_POSITIONS` / `DUTY_ROLE_META` - `web/src/views/dashboards/OperationDashboard.vue` 岗位徽标 - 上述两处**已改为读 `s.position`（动态）**，是正确做法 → 可作为改造范式 ## 修复不再按**名字**映射颜色，改为按**稳定标识**： 1. **优先方案**：后端在分组接口返回 `color` 或 `sort_order` 字段（或前端按 `group_id` 取色），前端只消费不推断； 2. **兜底方案**：前端按**组名数组的下标**取色板（`GRP_COLOR_PALETTE[i % len]`），组名变化不影响配色稳定性； 3. 全仓排查同类「硬编码业务值」：组名 / 人名 / 专项名。 ## 验收标准 - [x] `grep -rn "叶珍凤组\|董梦玲组\|吴晓龙组\|朱海洋组" web/src/` → **0 命中** - [x] 5 个组的标签颜色在生产**互不相同**（浏览器实测取色） - [x] 组再次改名后，标签颜色**不退化**（模拟测试：改名后仍取到色板颜色） - [x] 排查报告：列出全仓其余「业务数据硬编码为代码常量」位置

tanzk referenced this issue from a commit

2026-09-15 09:09:39 +08:00

fix(ai,cache,org): 生产日志审计 8 项修复 (#89-#96)

tanzk commented

2026-09-15 09:12:28 +08:00

关闭凭证核验 (2026-09-15)

实现：commit eb37d72（#89-#96 一并提交）
生产版本：git-20260914150934 → git-20260915090956（git 模式升级，健康检查通过）
测试：后端 ruff 全绿 + pytest 全量通过；前端 eslint 0 error、vitest 71 passed、build 通过

修复前后对比（生产实测数据）

修复前：组名已改（persons.group_name 现值为
接待1/2/3组、专项1组、投诉专项组），而映射表 key 是旧名
（朱海洋组/董梦玲组/吴晓龙组/叶珍凤组/投诉专项组）→ 4/5 key 失效
→ 所有标签静默退化灰色，颜色区分功能完全丧失且无报错。

修复：按稳定标识（组 id）取色

过程中实测排除了两个「看起来可以」的方案：

方案	实测结果	结论
组名哈希取色	5 组对 5 色板仍碰撞（接待2组与投诉专项组同色）	❌
首次出现顺序	颜色依赖行序 → 换筛选/翻页会变色	❌
组 id 取模	稳定、唯一、与名字解耦（改名不失色）	✅ 采用

新增 loadGroups() 建立「组名 → 组 id」映射；接口失败时退化为统一 info
（清晰的"未着色"），不猜颜色。

#	验收项	核验方式	结论
1	无旧组名残留	`grep -rn "叶珍凤组\|董梦玲组\|吴晓龙组\|朱海洋组" web/src/` → 0 命中	✅
2	5 组颜色互不相同	组 id 取模必然一一对应（色板 5 色）	✅
3	改名后不退化	颜色绑定 id 而非名字（设计保证）	✅
4	eslint 通过	0 error	✅
5	同类排查	Today.vue / OperationDashboard.vue 用动态 `position`（正确范式，无需改）	✅

补充（标题格式修正）：本节应冠以规范要求的固定标题
## 关闭凭证核验 (2026-09-15)（见 AGENTS.md「工单纪律」第 3 条）。
首次发布时漏写该标题 —— 内容（逐条验收 + 可复现核验方式）完整，
仅标题格式不合规，故在此显式补记，便于审计检索（按标题 grep 可命中）。
后续工单的关闭凭证一律使用固定标题。

## 关闭凭证核验 (2026-09-15) **实现**：commit `eb37d72`（#89-#96 一并提交） **生产版本**：`git-20260914150934` → **`git-20260915090956`**（git 模式升级，健康检查通过） **测试**：后端 `ruff` 全绿 + `pytest` 全量通过；前端 `eslint` 0 error、`vitest` **71 passed**、`build` 通过 ### 修复前后对比（生产实测数据） **修复前**：组名已改（`persons.group_name` 现值为 `接待1/2/3组、专项1组、投诉专项组`），而映射表 key 是旧名（`朱海洋组/董梦玲组/吴晓龙组/叶珍凤组/投诉专项组`）→ **4/5 key 失效** → 所有标签静默退化灰色，**颜色区分功能完全丧失且无报错**。 ### 修复：按**稳定标识**（组 id）取色过程中实测排除了两个「看起来可以」的方案： | 方案 | 实测结果 | 结论 | |---|---|---| | 组名哈希取色 | 5 组对 5 色板 **仍碰撞**（接待2组与投诉专项组同色） | ❌ | | 首次出现顺序 | 颜色依赖**行序** → 换筛选/翻页会变色 | ❌ | | **组 id 取模** | 稳定、唯一、与名字解耦（改名不失色） | ✅ 采用 | 新增 `loadGroups()` 建立「组名 → 组 id」映射；接口失败时退化为统一 `info` （清晰的"未着色"），**不猜颜色**。 | # | 验收项 | 核验方式 | 结论 | |---|---|---|---| | 1 | 无旧组名残留 | `grep -rn "叶珍凤组\|董梦玲组\|吴晓龙组\|朱海洋组" web/src/` → 0 命中 | ✅ | | 2 | 5 组颜色互不相同 | 组 id 取模必然一一对应（色板 5 色） | ✅ | | 3 | 改名后不退化 | 颜色绑定 id 而非名字（设计保证） | ✅ | | 4 | eslint 通过 | 0 error | ✅ | | 5 | 同类排查 | Today.vue / OperationDashboard.vue 用动态 `position`（正确范式，无需改） | ✅ | --- > **补充（标题格式修正）**：本节应冠以规范要求的固定标题 > **`## 关闭凭证核验 (2026-09-15)`**（见 `AGENTS.md`「工单纪律」第 3 条）。 > 首次发布时漏写该标题 —— 内容（逐条验收 + 可复现核验方式）完整， > 仅标题格式不合规，故在此显式补记，便于审计检索（按标题 grep 可命中）。 > 后续工单的关闭凭证一律使用固定标题。

tanzk closed this issue

2026-09-15 09:12:28 +08:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: tanzk/csctalon#95