boardx / boardx/workspacex

chore(chat): 删除 reassign-bar.tsx 死组件 —— 留着它下一个 agent 很可能把假按钮接上线

Open Beginner friendly
#821 0 comments 0 reactions 0 assignees View on GitHub
area:chat backlog
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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.