Skip to content

fix(web): 打磨会话工作台表现细节(ZER-712) - #47

Merged
ZeroPointSix merged 30 commits into
mainfrom
fix/zer-712-workbench-polish
Aug 15, 2026
Merged

fix(web): 打磨会话工作台表现细节(ZER-712)#47
ZeroPointSix merged 30 commits into
mainfrom
fix/zer-712-workbench-polish

Conversation

@ZeroPointSix

Copy link
Copy Markdown
Owner

概要

打磨 Plus 只读会话工作台的第一屏表现,对齐 ZER-712。视觉参考 AionUi / Factory 的活动流密度:思考合成一条折叠,工具默认一行摘要,左栏改成按 Agent 分组的列表树。

本 PR 叠在 #46 的只读工作台分支上,不整页重构,也不混入 W6 写能力。

Review Contract

Goal

让现网只读会话工作台不再把思考链拆成一词一块,连接中不再同时亮 Live,左栏按 Agent 名分组并显示完整会话标题,工具调用默认收成一行且能看到工具名,右侧不再是空的事件列表。

Non-goals

  • 不整页搬 AionUi / Factory 官网。
  • 不实现发消息、插话、Stop / Cancel、Agent 热切换。
  • 不改 transcript 后端协议,也不做跨进程持久化。
  • 不把本 PR 直接合进 main;基线是 整合:重建 ZER-404 只读 Agent 会话工作台 #46 工作台分支。

Accepted Residual Risks

会话列表快照没有独立的「用户原话」标题字段,主行目前使用完整 session_id,thread 作为副行。若后续后端补 title / first user message,再单独替换主行文案。右侧日志仍消费既有生命周期事件,不是真实 PTY。

Acceptance Criteria

  1. 连续 thinking 合成一条「思考过程」折叠;空文本不占行;刷新 / 重连后仍是一条。
  2. 顶栏和底栏共用同一套连接态;connecting 只显示连接中,不亮 Live;会话 idle 显示「空闲」。
  3. 左栏只按 Agent 名称分组,不写 Profile / 模型名;主行不再截 12 字。
  4. 工具调用默认一行摘要,必须显示工具名(execute 回退为 bash),点击展开参数和结果,颜色为淡墨低饱和。
  5. 右侧「事件」改为只读执行日志,不再是空事件列表。
  6. pnpm --dir web lintpnpm --dir web testpnpm --dir web build 通过。

Follow-ups

  • 若后端补会话标题 / 首条用户消息,用它替换左栏主行。
  • 若要把右侧日志做成真实终端回放,需要单独的终端事件源。
  • 合入前先让 整合:重建 ZER-404 只读 Agent 会话工作台 #46 工作台分支落地,再把本 PR 一起带上。

验证

  • pnpm --dir web lint
  • pnpm --dir web test:11 个文件、61 项通过
  • pnpm --dir web build

关联:ZER-712、ZER-404。

cursoragent and others added 29 commits August 13, 2026 11:48
Replace ProTable sessions list and detail pages with a read-only
three-column SessionWorkbenchPage using Ant Design X Conversations for
the sidebar, status bar + transcript placeholder in the main panel,
and metadata/alerts/events tabs in the inspector.

Co-authored-by: ZeroPointSix <ZeroPointSix@users.noreply.github.com>
- 左栏新增 New chat 禁用占位(会话创建 W6 开放)
- 中栏状态条补齐 Profile / workdir 字段
- 底部只读条内嵌 SSE 连接态:新增 StreamStatusContext,复用 AdminLayout 同一 SSE,不新建连接
- 新增 useMediaQuery,≤1100px 左右栏折叠为 Drawer,390 下顶栏不溢出
- 版本号 0.2.3 → 0.3.0
- 重新构建并提交 web/app.js 与 web/styles.css 产物
Merge consecutive thinking chunks, unify connection status,
group the sidebar by Agent name, and restyle tool rows and
the inspector log after AionUi activity-stream density.
@linear-code

linear-code Bot commented Aug 15, 2026

Copy link
Copy Markdown

ZER-712

@ZeroPointSix ZeroPointSix left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

首轮审查结论:存在阻塞项,暂不建议合并。已按 Review Contract 审查提交 47082e4

发现 2 个 ORIGINAL 阻塞项,分别影响 Acceptance Criteria 1 和 4。Admin Web CI 与 Review Contract 检查均已通过,但现有测试没有覆盖这两种输入序列。修复后请补对应回归测试,再进行 fix verification。

说明:当前 GitHub 连接账号是 PR 作者,GitHub 不允许作者对自己的 PR 提交 REQUEST_CHANGES,因此本次以 COMMENT 形式记录阻塞结论。

Comment thread web/src/lib/transcript.ts Outdated
Comment thread web/src/components/activity/normalizeToolCall.ts Outdated
Store consecutive thinking fragments by stable entry_id so later
revisions replace the same chunk. Map explicit execute/exec names
to bash and cover both sequences with regression tests.

@ZeroPointSix ZeroPointSix left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fix verification 通过,未发现新增阻塞项。

  • P1:不同 thinking entry_id 继续保留为可修订分片,仅在活动流视图模型合并;回归测试覆盖 partial → same-ID revision,并得到 Hello world
  • P2:显式 execute / exec 与 kind 回退统一规范化为 bash,对应测试已补齐。
  • 回归:仅检查 47082e4..afea34b 增量;未发现相关行为回归。
  • CI:Review Contract 成功;Admin Web run #78 的 type-check、Web tests、profile schema smoke、build、生成产物校验和 Gateway embedding 校验全部成功。

两条审查线程已标记 resolved。基于当前 head afea34b,本轮无剩余阻塞项,可以进入最终合并判断。

@ZeroPointSix
ZeroPointSix changed the base branch from agent/zer-404-clean-readonly-workbench-20260813 to main August 15, 2026 04:34
@ZeroPointSix
ZeroPointSix merged commit 95d7253 into main Aug 15, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants