feat: 新增对话界面锚点,支持点击跳转对应提问 - #875
Conversation
PR quality triageChanged areas: area:desktop CLI core policy: No CLI-core policy block detected. Missing-test policy: Blocked by policy until a maintainer applies Coverage baseline policy: No coverage-baseline policy block detected. CLI core files:
Coverage policy files:
Expected checks:
Test coverage signals:
Risk notes:
Hard merge gates still come from GitHub Actions, not AI review. Dosu handoff: Dosu can be used as the AI reviewer for risk explanation, missing-test prompts, and maintainer Q&A. If it does not comment automatically from the PR template, ask: @dosubot review this PR for changed-area risk, missing tests, docs impact, desktop startup risk, and CLI core impact. |
|
Hi @adrve! I'm Dosu and I'm helping the Relakkes's Org team. Here's my review of this PR across the requested dimensions: Changed-Area Risk: Low Missing Tests: Medium Concern
Given that no dedicated UI test infrastructure exists in the indexed repos yet, this may be an accepted gap — but it should be explicitly documented in the "Known risk / rollback" section. Docs Impact: None Desktop Startup Risk: Negligible CLI Core Impact: None Summary: The feature is well-scoped and low-risk. The main gap is the absence of test coverage documentation. I'd recommend either adding lightweight component tests or explicitly noting in the PR why they're deferred (e.g., pending test infra setup) so reviewers have the full picture. |
Summary
新增对话锚点功能,在聊天消息导航组件增加跳转锚点,点击可快速定位对应提问消息,仅改动桌面端聊天 UI 组件,无服务端、底层逻辑变更。
改动文件:
MessageNavigation.tsx(新增)— 右侧书签面板 UI
MessageList.tsx(修改)— 引入组件 + 添加 data-message-id 属性 + 跳转逻辑
如下图,对话界面右侧新增了锚点,点击可以跳转到指定的提问:

Feature Quality Contract
Verification
bun run verifyfor code changes, including the coverage gate.Risk
allow-cli-core-change.allow-missing-tests.allow-coverage-baseline-change.@dosubot review this PR for changed-area risk, missing tests, docs impact, desktop startup risk, and CLI core impact.