Repository navigation
feat(web-studio): workspace-first setup and dual-provider experience - #6066
Merged
Merged
Conversation
yufeng201
reviewed
Oct 14, 2026
yufeng201
left a comment
Collaborator
There was a problem hiding this comment.
审查确认 4 项问题(2 项 P1、2 项 P2),具体复现和建议见各行内评论。规范轴未发现额外需阻塞的硬性违反;需求/行为轴存在工作区隔离、配置可访问性和会话历史兼容问题。原有 1107 个测试通过,补充定向复现覆盖了上述缺口。
yufeng201
reviewed
Oct 14, 2026
yufeng201
left a comment
Collaborator
There was a problem hiding this comment.
复核最新提交后发现两个新的行为问题,具体位置见行内评论。已有的技能导入失效操作、身份切换旧闭包、配置列表可见性和 system 会话消息问题已有对应保护,不重复评论。
added 6 commits
October 15, 2026 11:41
Queued uploads outlived their provider and resolved later requests through the mutable global client. Capture an independent client for the source connection, abort pending operations on unmount, and verify ownership before queued uploads and import continuations. Cover two-file workspace switches, same-service credential changes, delayed remote responses, normal sequential uploads, and StrictMode task loading.
yufeng201
approved these changes
Oct 15, 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.
Summary
Demo
Validation
web-studio: 148 test files, 1107 tests passednpm run lint: 0 errors (8 existing warnings)npm run build: passedThis PR is based on the current upstream
main. The fork's previousmain-only commits were intentionally excluded.