Conversation
Owner
Author
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.
对应 A06,基于 #10 的堆叠 PR。
Start-Process 的 ArgumentList 会重新合并数组,造成空参数、含空格值及引号丢失。Host 启动改用 Rust 原生 Command::args + CREATE_NEW_CONSOLE;保留 npm .cmd/.bat shim 回退。直接返回子进程 PID/系统启动错误,因此仍满足 A04/A05。移除不再需要的 PowerShell 启动脚本及 PID 文本解析。
Rust 对 batch 参数采用专门转义;无法安全表示的特殊值返回错误,不自行拼接 cmd shell 字符串。
验证:Linux 可运行的 4 个 launcher tests、rustfmt/diff check 通过。新增 Windows 实际 C# 子进程 argv probe,覆盖 exe 的空值、空格、引号、反斜杠、Unicode、换行及 shell 字符;另覆盖 cmd shim 和无效 exe/cwd。原生测试在本环境未执行,需 Windows CI/实机验证,保留草稿。
参考:https://doc.rust-lang.org/std/process/index.html#windows-argument-splitting
2026-09-23 repair verification
f5a3851d3453ca313b26e0a3e6032b8b8f758670.