Skip to content

fix(v2): keep inspection review, task results, and workflow stages consistent - #352

Merged
wanglongan587 merged 12 commits into
ora-space:mainfrom
wanglongan587:fix/1012-daily-inspection-ui
Nov 9, 2026
Merged

wanglongan587 merged 12 commits into
ora-space:mainfrom
wanglongan587:fix/1012-daily-inspection-ui

Conversation

@wanglongan587

@wanglongan587 wanglongan587 commented Nov 6, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

每日巡检的确认表单曾替换所有节点视图,判定、批量操作、任务回执与页面状态也存在不一致。本 PR 保留各节点自己的内容,并让报告确认、任务创建/关联与结果查看形成连续的前端路径。

测试报告 → 问题确认 → 创建/关联协作任务 → 任务详情与来源报告
             ├─ 当前筛选范围内批量判定、撤销、草稿保存
             ├─ 必填项直接可见,提交时显示遗漏
             └─ 内容独立滚动,操作页脚不覆盖按钮
  • 问题卡片直接展示“是问题 / 不是问题”,报告可折叠,长描述和证据可展开。筛选后批量操作只影响当前结果,空结果禁用;全部排除显示“本次无需创建任务”。
  • 新任务默认使用确认人为负责人;已有任务沿用原负责人,控件只读并提供任务详情入口。提交后以结论展示,非确认人只读,失败节点和任务失败重试有明确入口。
  • c00f53d0 修复本地验收发现的三项问题:运行已成功但表单回执未到齐时显示“正在同步任务结果”,不误报“无需处理”;载入最新草稿时清除旧冲突错误、校验和撤销记录;表单主体独立滚动,页脚保留布局空间。
  • 只改 frontend-v2/;v2,不涉及 specs。没有修改后端、接口、数据库或外部系统写入语义。

Refs ora-space/circus#1012

Evidence

验证提交:c00f53d0b51afced9187e4a52ded46b272c6dfaa。

  • Before: 组件回归测试复现“已确认问题但回执暂为空 → 无需处理”及载入草稿后旧冲突警告残留;浏览器布局断言复现可滚动区域延伸至页脚覆盖区。
    After: run-form.test.tsx 17 个测试通过;实际浏览器两标签页冲突恢复、任务回执自动更新,以及 1280×800 / 390×844 按钮点击与页脚布局均通过。
  • API 生成漂移检查、TypeScript、lint、build、git diff --check 通过。
  • 全量单测 104 文件 / 787 用例:首轮 786 通过,未改动的技能设置页用例出现一次 5 秒超时;该文件单独复跑 5 个用例全部通过。未修改超时或跳过测试。
  • 真实 v2-server + PostgreSQL 的 workflow-review.spec.ts / workflow-forms.spec.ts 共 13 个相关 e2e 用例均已验证通过,覆盖草稿、批量/筛选、权限、跨次任务复用、取消、窄屏、节点视图、并发冲突、长列表、定时记录、页脚布局和延迟回执。首轮末项遇到并行 Playwright 运行共用产物目录的 ENOENT,使用独立输出目录复跑通过。
  • 延迟回执用例仅暂缓浏览器读取到的任务回执;提交和任务创建仍访问真实后端。
  • CI:最新提交的 server、frontend、e2e 全部通过:v2 workflow。frontend 的完整检查包含 104 个测试文件全部通过;e2e 为 60 passed、10 skipped(未将跳过项计作通过)。GitHub 当前为 MERGEABLE / CLEAN。

最新浏览器截图均来自 c00f53d0,存放在独立 screenshots/1012 分支,固定到截图提交 e42e38e1,不进入产品 diff。截图使用本地 fixture 数据;任务保存/创建/关联为真实本地后端行为,DTS 回执仅演示,未写入外部 DTS;本次未执行实时 Agent 采集。

场景 截图
待确认:报告、必填原因、已有任务、固定可见的操作区 桌面待确认
390px:第三项判定和页脚不重叠 移动端操作区
提交后:正确显示新建 2 个、关联 1 个 提交后任务统计
任务详情入口与明确标记的 DTS 演示回执 任务与DTS演示回执

Merge Danger

Door: two-way。纯前端变更,可回退;无数据迁移。

Blast Radius: 巡检界面。影响表单与任务节点的展示、交互及滚动布局;其他节点视图、已有任务负责人和后端建单语义保持原有规则。表单等待仍无截止时间、待办聚合或通知;真实 DTS 集成不在本 PR 范围内。

@wanglongan587
wanglongan587 force-pushed the fix/1012-daily-inspection-ui branch 2 times, most recently from 7b4249e to 4f84537 Compare November 6, 2026 07:47
@wanglongan587
wanglongan587 force-pushed the fix/1012-daily-inspection-ui branch from 4f84537 to 63925d8 Compare November 6, 2026 07:58
- 一键判定每项发现(是问题/不是问题),并把确认人预填为负责人,替代藏在原生下拉里的第三种状态
- 支持批量标记与撤销;底栏明确写出提交后会新建几个、关联几个任务
- 报告收进左栏、窄屏移到列表之后,Markdown 列表与表格恢复可读
- 提交后按结论阅读:不再渲染禁用的输入框,判为不是问题的条目说明不会建单
- 没有发现问题时只需一次确认;表单节点失败时说明原因,而不是让人反复刷新
- 任务结果页按冻结的发现补上标题与严重级别,失败时回到确认节点重试
- 删掉空操作的「应用表单配置」按钮;配置改动本来就是即时的
- 给每个附加字段标注标识、显示名、类型和必填,替换只有 aria-label 的裸输入框
- 说明报告/问题列表来源变量各需要什么形状,选中错误不必等到运行才暴露
复用已有任务的条目也显示“负责人(沿用) <姓名>”,新建任务的条目才给下拉框;
之前只有“会被新建”的那一行出现负责人控件,看起来像别的行漏了字段。

- 负责人槽位对每一行都在同一位置出现:沿用任务只读,新建任务可选
- 任务关联信息不再重复负责人,改为“关联到已有任务 #id · 状态 · 查看任务”
- 去掉关联标签的底色,避免看着像可点击的按钮
复用任务的行也给出同一个负责人下拉框,默认带出该任务的负责人;之前只有
“会新建任务”的行有控件,复用的行是只读文字,同一张页面里看起来像漏字段。

- 判定为问题后每行都有负责人,位置与宽度一致
- 复用任务仍显示“关联到已有任务 #id · 状态 · 查看任务”,不再重复负责人
- 结论态继续按文本展示负责人,不渲染禁用的控件
- 失败在确认节点的运行直接落在表单节点上,先看到“确认表单没有生成”的原因,而不是一张通用失败页
- 非确认人看到只读说明位于标题下方,底部不再渲染一整套禁用按钮;运行状态改回中性的“等待处理”,不再对只读成员说“待你处理”
十个场景跑在真实 v2-server + PostgreSQL 上:判定/批量/撤销/筛选、草稿与刷新、
建单与任务页回链、跨天复用、无问题、只读成员、等待中取消、390px、失败绑定、
双人并发冲突、30 条长列表、每段节点各自视图、定时任务停在表单并可再次触发。
@wanglongan587 wanglongan587 changed the title fix(v2): keep workflow run nodes on their own stages and redesign inspection review fix(v2): keep inspection review, task results, and workflow stages consistent Nov 9, 2026
@wanglongan587
wanglongan587 merged commit ec08752 into ora-space:main Nov 9, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant