Repository navigation
fix(goals): 修复验收标准、会话跳转与看板刷新 (#578, #579, #580) - #581
Merged
Merged
Conversation
PeterGuy326
approved these changes
Oct 9, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #578
Closes #579
Closes #580
无验收标准的项目可以执行任务,但随后无法通过人工验收完成;阻塞目标的快捷入口可能打开无关会话;Agent 看板在任务事件到达后仍显示旧快照。本 PR 为目标/项目提供验收标准编辑入口,限制快捷跳转到真实绑定的会话,并使可见看板随任务事件和重连刷新。
Changes
updateGoal契约和 shared 校验,保留 1–16 项及单项 4096 字符限制。保存使用expectedUpdatedAt;409 保留草稿,显式加载最新标准后可重新核对并保存,传输失败可重试。保存期间防重复提交,切换目标/工作区后丢弃旧界面响应。之前审查的跨工作区文档草稿串用已由主干 #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通过。goals、goal-task-acceptance、task-board、task-board-http、acceptance,低并发 62 项全部通过。npm audit --audit-level=high:0 漏洞。http://127.0.0.1:5178/__review,1440×900 和 640×900,中文及英文。验证补充标准后验收准确来源的真实交付、缺失会话隐藏入口、后续绑定分支的准确跳转、事件及重连刷新。页面身份正确,无空白、框架覆盖层、应用运行错误或横向溢出。Browser 插件不可用,使用已有 Playwright;保留原有 Ant Drawerwidth弃用提示,截图和临时 QA 记录位于仓库外。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 门禁。