Repository navigation
Fix web remote authentication, Git Review paths, and state recovery - #130
Merged
Merged
Conversation
Reject plaintext business messages and prevent outbound encryption downgrade. Preserve literal Git paths, align rename statistics, and fix SSH tilde quoting. Repair stale pane layouts and workspace selections during state recovery. Add browser, real Git/shell, and isolated Swift recovery regressions to CI. Validation: 42 browser tests, 23 Git checks, and 19 recovery cases passed. Swift/JavaScript syntax and diff checks passed. Full Xcode/UI validation was not run because the local Xcode and Ghostty build dependencies are unavailable.
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
修复 Web Remote、Git Review 和存档恢复中的六项问题:
补充回归测试,并将独立 Git/Swift 恢复校验纳入 CI。持久化 schema 不变。
验证方式 / Testing
node --test scripts/test-web-remote-*.mjs scripts/test-git-review.mjs scripts/test-persistence-recovery.mjs:44 个外层测试入口通过,0 失败(Git 和恢复脚本各算一个入口)。git diff --check通过。本机缺完整 Xcode 和 Ghostty 构建依赖,完整应用构建、XCTest runner 和真实 AppKit/Ghostty UI 未运行。原有完整 GlintTests CI 步骤保留。
截图或录屏 / Screenshots or recordings
暂无。本次为协议、文件路径和存档恢复逻辑修复;真实 UI 恢复后的分屏及焦点仍需完整开发环境验证。
关联议题 / Related issues
本次代码审查发现,未关联仓库 issue。