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 main2026-10-12 19:39:45 +00:00
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
Summary
默认使用浅色简洁界面,常规问答保持轻量;任务摘要、待确认事项和文件预览渐进展开。窄屏可在上下文、对话和工作间切换。异步事件、确认反馈和预览意图由既有 store 统一处理。
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。