Skip to content

fix(goals): 修复验收标准、会话跳转与看板刷新 (#578, #579, #580) - #581

Merged
PeterGuy326 merged 1 commit into
mainfrom
fix/client-goals-review-followup
Oct 9, 2026
Merged

PeterGuy326 merged 1 commit into
mainfrom
fix/client-goals-review-followup

Conversation

@Bindy-lbb

@Bindy-lbb Bindy-lbb commented Oct 9, 2026 •

Copy link
Copy Markdown
Collaborator

Closes #578
Closes #579
Closes #580

无验收标准的项目可以执行任务,但随后无法通过人工验收完成;阻塞目标的快捷入口可能打开无关会话;Agent 看板在任务事件到达后仍显示旧快照。本 PR 为目标/项目提供验收标准编辑入口,限制快捷跳转到真实绑定的会话,并使可见看板随任务事件和重连刷新。

Changes

  • 验收标准编辑复用既有 updateGoal 契约和 shared 校验,保留 1–16 项及单项 4096 字符限制。保存使用 expectedUpdatedAt;409 保留草稿,显式加载最新标准后可重新核对并保存,传输失败可重试。保存期间防重复提交,切换目标/工作区后丢弃旧界面响应。
  • 已执行任务继续通过真实交付的人工验收完成,不通过修改任务状态绕过验收。增加“无标准项目 → 补充标准 → 核对真实产出 → 验收完成”的组件集成回归。
  • 阻塞入口跳过仅有岗位的分支,选择同时绑定岗位和非空会话的分支;缺少导航能力时隐藏无效按钮。
  • 看板收到目标/任务事件或 SSE 重连后刷新任务。请求版本与组件生命周期防止旧响应覆盖新快照;加载失败保留上次成功数据并显示重试入口。
  • 中英文文案保持键一致;更新 Unreleased changelog。

之前审查的跨工作区文档草稿串用已由主干 #575 的 Vault 路由和 vaultId + workspaceKey 存储范围解决。本 PR 不重复创建该已解决问题的 Issue;新增真实 VaultModule 回归,验证同 Vault、同路径、同版本的两个工作区不会串用草稿或触发错误自动保存。

Validation

基线:main@64a6026;head:d4d7601。Linux / Node 22.23.2,Playwright 1.62.1 / Chromium;隔离 IPC fixture,不操作真实工作区或 Agent。

  • npm run build、npm run typecheck:renderer、npm run typecheck:ui、生产 renderer 构建及 git diff --check 通过。
  • 相关 renderer 回归:8 个文件、121 项通过,包括新增目标/看板 10 项、Vault 隔离 1 项,以及 i18n 键一致性和 CJK 门禁。
  • 相关服务端契约:goals、goal-task-acceptance、task-board、task-board-http、acceptance,低并发 62 项全部通过。
  • 桌面主进程:402 项通过。脚本测试:229 项通过。UI 测试:50 项通过。npm audit --audit-level=high:0 漏洞。
  • 完整 App 以外的 renderer 全量复核(4 workers):79 个文件、1005 项全部通过。
  • 全 renderer(4 workers):80 个文件、1119 项通过、2 项完整 App 用例超过既有 5 秒预算;单独以 15 秒预算复核相关 App 用例,3 项通过、113 项未选中。两条超时用例涉及旧模型保存和群会话选择,本 PR 未改动这些实现。
  • Playwright:http://127.0.0.1:5178/__review,1440×900 和 640×900,中文及英文。验证补充标准后验收准确来源的真实交付、缺失会话隐藏入口、后续绑定分支的准确跳转、事件及重连刷新。页面身份正确,无空白、框架覆盖层、应用运行错误或横向溢出。Browser 插件不可用,使用已有 Playwright;保留原有 Ant Drawer width 弃用提示,截图和临时 QA 记录位于仓库外。
  • OCR host review:4/4 可审文件完成复核,无剩余高/中严重度问题;测试与服务端验收契约也已人工检查。

Full-check limitations

npm run check 未全绿:本执行环境临时端口范围仅为 60500–60800,批量服务端 HTTP 测试出现 EADDRINUSE;root 身份还使已有 chmod(0555) 写入失败测试无法模拟权限拒绝。另有既有进程时限用例在负载下超时,以及旧 checkout 遗留的已删除 Jev 测试构建产物(已从本地生成目录移除)。已将相关服务端检查降为单文件并发验证通过;完整 App 的两个超时用例单独复核通过。不能将这些复核描述为整个 npm run check 已通过,最终门禁由本 PR CI 在干净 checkout、Node 24 环境验证。

未验证 macOS/Windows 安装包的实际用户交互;本次没有修改服务端 schema、IPC 契约或依赖,也没有绕过项目的 CI 门禁。

@PeterGuy326
PeterGuy326 merged commit 223add9 into main Oct 9, 2026
11 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

2 participants