fix: bind project acceptance to the requested workspace - #42
Merged
Merged
Conversation
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.
两个可选 TavernDesk 验收脚本原先在临时目录启动固定工作区 Gateway,随后打开目标仓库,导致 WORKSPACE_MISMATCH。改为启动时绑定目标仓库,缓存和回收站继续使用独立临时目录,并核对 hello 返回的工作区。
原有 stdio 协议测试将整个开发仓库作为输入,且独立缓存配置未传给实际 CLI。现改用三个固定文件组成的临时项目,并复用已安装 MCP SDK 的握手、传输与关闭。保留原有 12 个场景、8000 ms 请求限制和协议错误断言;失败附有界 stderr。生产源码、依赖和客户端配置不变。
验证:
当前 head 的 Node 24 和三项 CodeQL 已通过,Node 22 正在执行最后的设计时输出隔离验收。TavernDesk 用当前未修改源码重建后,专用窗口的 6 个 UI 产品任务也已通过;原始 UnauthorizedAccessException 未复现,原因未定。双 Gateway UI 测试新增代码和实际客户端 Roslyn 验收记录在后续独立提交中,不扩大本 PR 的代码范围。