chore(chat): 删除 reassign-bar.tsx 死组件 —— 留着它下一个 agent 很可能把假按钮接上线
- Dominant language
- TypeScript
- Stars
- 0
- Forks
- 0
- Avg merge
- 1h 7m
- Merged PRs (30d)
- 969
Description
从 #715 的裁决里拆出来(一个 issue 不干两件事)。
`apps/web/components/chat/reassign-bar.tsx` 吃 `apps/web/lib/mock/chat.ts` 的
`REASSIGN_SUGGESTION` 假数据,不在任何生产路由闭包里,已登记在
`apps/web/tests/session/chat-dead-mock-cluster.test.ts` 的死代码台账中。
#715 已裁决「委派/转派功能不做」(无真实后端能力,`suggestRedispatch` 域逻辑存在但
缺 HTTP 路由,且「改派在执行层面意味着什么」尚未定义)。
**留着这个组件的具体代价**:它长得很完整,下一个 agent 看到它很可能顺手接到某条路由上,
于是一个假按钮就上线了——这正是 `chat-ux-acceptance-criteria.md` 第 1 条硬性要求
(「没有真实数据支撑的能力,不做假 UI」)要防的。
## 做什么
1. 删 `apps/web/components/chat/reassign-bar.tsx`
2. 删 `lib/mock/chat.ts` 里只被它使用的 `REASSIGN_SUGGESTION`(确认无其它引用再删)
3. 同步更新 `chat-dead-mock-cluster.test.ts` 的台账
4. 确认 `verify:base` 仍绿
## 不要顺手做的事
不要因为「域逻辑都在了」就顺手把 `suggestRedispatch` 接上路由——#715 已裁决那需要先有
产品/契约决策(ADR-023 签核),不是实现细节。
Contributor guide
No contributing guide indexed for this repository
Research direction
Start with apps/web/components/chat/reassign-bar.tsx and check references to REASSIGN_SUGGESTION in apps/web/lib/mock/chat.ts before removing either. Update apps/web/tests/session/chat-dead-mock-cluster.test.ts to keep its dead-code ledger accurate, then run verify:base; done means the dead component and unused mock are gone and verification remains green.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- typescript
- Domain
- frontend, testing-qa
- Issue type
- Refactor
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Quiet
- Clarity
- Clearly specified
- Newbie friendliness
- 82/100