Simplify the workbench for everyday conversations #74

Merged
weichen merged 6 commits from feature/work-companion-ui into main 2026-10-12 19:39:45 +00:00
Owner

Summary

默认使用浅色简洁界面,常规问答保持轻量;任务摘要、待确认事项和文件预览渐进展开。窄屏可在上下文、对话和工作间切换。异步事件、确认反馈和预览意图由既有 store 统一处理。

Conversation → 当前工作 → 同一文件预览

Evidence

SELF:Frontend 210 项测试、类型与 knip,Dev 全量 make test + make build 已通过;19 项浏览器/真实 HTTP 用例覆盖展示、导航和三附件空文字发送。REPLAY:最终固定 Frontend 43d1933 的正式独立复审与原 #1–6 整改闭合见 Dev 任务卡。组件没有原生工作流;GATE:Dev run2724 的全三作业成功,精确 b0424f563ede624b92bc4d87344fa481c71b44d2 固定 Frontend43/UI0da/Backend0f,home-runner/runner_id=3。

对象:43d1933b8c3377ae548eb495305e9f0de29dcb59;完整脱敏原始收据见 Dev 任务卡。合入前按最新 base/head、独立 Review 和固定组合平台 GATE 核验。

Merge Danger

Door: two-way

Blast Radius: presentation

沿用既有后端能力、权限和数据;不部署服务。可通过组件版本回退。

Why: 让非专业用户先看工作结果,再按需查看依据和执行记录。
Evidence: Dev 任务卡中的固定对象、原始 evidence 收据和复审记录。
Impact: 展示与前端交互状态;不改变模型路由或业务权限。
Refs: UI01–UI13,U1–U5,work-companion-ui r0.2 / r1。

## Summary 默认使用浅色简洁界面,常规问答保持轻量;任务摘要、待确认事项和文件预览渐进展开。窄屏可在上下文、对话和工作间切换。异步事件、确认反馈和预览意图由既有 store 统一处理。 ```text Conversation → 当前工作 → 同一文件预览 ``` ## Evidence SELF:Frontend 210 项测试、类型与 knip,Dev 全量 make test + make build 已通过;19 项浏览器/真实 HTTP 用例覆盖展示、导航和三附件空文字发送。REPLAY:最终固定 Frontend 43d1933 的正式独立复审与原 #1–6 整改闭合见 Dev 任务卡。组件没有原生工作流;GATE:Dev [run2724](https://git-home.u1680832.nyat.app:30319/weichen/Agent-System-Dev/actions/runs/2724) 的全三作业成功,精确 b0424f563ede624b92bc4d87344fa481c71b44d2 固定 Frontend43/UI0da/Backend0f,home-runner/runner_id=3。 对象:`43d1933b8c3377ae548eb495305e9f0de29dcb59`;完整脱敏原始收据见 [Dev 任务卡](https://git-home.u1680832.nyat.app:30319/weichen/Agent-System-Dev/src/branch/main/.task/archive/feature-work-companion-ui.md)。合入前按最新 base/head、独立 Review 和固定组合平台 GATE 核验。 ## Merge Danger **Door:** two-way **Blast Radius:** presentation 沿用既有后端能力、权限和数据;不部署服务。可通过组件版本回退。 Why: 让非专业用户先看工作结果,再按需查看依据和执行记录。 Evidence: Dev 任务卡中的固定对象、原始 evidence 收据和复审记录。 Impact: 展示与前端交互状态;不改变模型路由或业务权限。 Refs: UI01–UI13,U1–U5,work-companion-ui r0.2 / r1。
weichen added 6 commits 2026-10-12 19:20:03 +00:00
Why: make material, background work, confirmations and files accessible to nontechnical users across desktop and mobile.

Evidence: Dev SELF receipts for Frontend tsc, 190 tests, knip baseline, build, nine fixture browser cases and two real HTTP/browser smoke cases using MockRunner.

Impact: existing API and permissions remain authoritative; stale events/queries and duplicate effects are guarded, while one file preview and reading state are preserved.

Refs: work-companion-ui design r0.2 and plan r1 U2-U5; UI01-UI13.
Why: Opening the auxiliary work pane should keep the conversation compact.

Evidence: Existing Playwright browser suite passed all 10 cases, including finished Collapsible transition.

Impact: Test and screenshot timing only; product behavior is unchanged.

Refs: Dev feature/work-companion-ui, UI03
Why: Retained React nodes alone did not prove actual browser scrolling was preserved.

Evidence: The settled Chromium case failed on the previous bundle, then all 11 browser cases passed with exact material, scroll and focus assertions.

Impact: Browser regression coverage only; fixture transport stays outside the production bundle.

Refs: UI10, ce-code-review UI finding #3, work-companion-ui
Why: Background frames blocked terminal reconciliation, stale failures left confirmations busy, and delayed page hydration changed mobile navigation.

Evidence: Review findings #1/#2/#3 have recorded red-to-green checks; all 210 Frontend tests, tsc and knip pass, plus six real navigation browser checks.

Impact: Keep sequence high-water and operation ownership, merge observations by object, and use an explicit preview-opening intent.

Refs: ce-code-review 20261012-165912-cb241b46 #1/#2/#3, UI04/UI09/UI10/UI11
Why: Ordinary explanations were folded with tool activity and the oldest result was selected as current work.

Evidence: Findings #5/#6 red-to-green actual-component and live-projection checks are retained; current full Frontend suite passes 210 cases.

Impact: Preserve business text, errors and file references in reading order; retain active/pinned priorities and newest-first host ordering.

Refs: ce-code-review 20261012-165912-cb241b46 #5/#6, UI03/UI05
Why: Upload inventory alone did not prove that the attachment-only user message included and persisted all three materials.

Evidence: Real upload IDs match the empty-text POST and the same user artifact_ref parts before and after reload; both live cases pass.

Impact: Stronger acceptance evidence only; retain mock-runner model-quality limits.

Refs: ce-code-review 20261012-165912-cb241b46 #4, UI01/UI04/U5
weichen marked the pull request as ready for review 2026-10-12 19:39:44 +00:00
weichen merged commit 43d1933b8c into main 2026-10-12 19:39:45 +00:00
Sign in to join this conversation.
No Reviewers
No labels
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: weichen/Agent-System-Frontend#74