Skip to content

feat(launch): 增加自定义环境变量配置选项#2500

Closed
Harry-hq wants to merge 5 commits intoPCL-Community:devfrom
Harry-hq:dev2
Closed

feat(launch): 增加自定义环境变量配置选项#2500
Harry-hq wants to merge 5 commits intoPCL-Community:devfrom
Harry-hq:dev2

Conversation

@Harry-hq
Copy link
Contributor

抱歉第一次 PR 眼睛瞎了没看见这寄吧 AI 瞎改 #2490
QQ_1771986168869

Harry-hq and others added 2 commits February 24, 2026 15:50
@pcl-ce-automation pcl-ce-automation bot added 🛠️ 等待审查 Pull Request 已完善,等待维护者或负责人进行代码审查 size: M PR 大小评估:中型 labels Feb 25, 2026
Signed-off-by: Harry-hq <dq114144@126.com>
Signed-off-by: Harry-hq <dq114144@126.com>
Copy link

@DotnetInstall DotnetInstall left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

必须修改

1.tokenizer 仍有缺陷:当前双引号解析在引号不配对时会静默出错,用户毫无感知。更根本的问题是,环境变量值本身可以含换行符,空格分隔方案从设计上就不健全。建议改用换行分隔(TextBox 设为多行),或按 ruattd 的建议使用 JsonSerializer 存储,彻底规避歧义。

2.PATH 覆盖风险:_ApplyInstanceEnvVariables 在启动器已设好 Path 之后执行,用户若填写 Path=... 会直接覆盖掉 PCL 拼好的 Java 路径,导致游戏启动失败且报错完全不直观。至少需要对 Path/PATH 做特殊处理(拒绝覆盖或改为追加),并在 Tooltip 中明确警告。

3.Tooltip 示例有误导性:示例 JAVA_HOME=C:\Java 没有空格,但真实场景几乎都是 C:\Program Files\Java\jdk-21。应在示例中展示含空格路径的正确写法(加双引号),否则用户遇到含空格路径时会被静默截断。

应改进

4.缺少注入日志:环境变量注入成功与否在启动日志中完全不可见,出问题时排查困难,建议至少记录注入了哪些 key(值可脱敏)。

5.Key 未做合法性校验:若用户误输入导致 key 含空格等非法字符,会被静默注入,建议加 Trim() 并跳过含空格的 key。

Signed-off-by: Harry-hq <dq114144@126.com>
@pcl-ce-automation pcl-ce-automation bot added size: L PR 大小评估:大型 and removed size: M PR 大小评估:中型 labels Feb 27, 2026
@Harry-hq
Copy link
Contributor Author

必须修改

1.tokenizer 仍有缺陷:当前双引号解析在引号不配对时会静默出错,用户毫无感知。更根本的问题是,环境变量值本身可以含换行符,空格分隔方案从设计上就不健全。建议改用换行分隔(TextBox 设为多行),或按 ruattd 的建议使用 JsonSerializer 存储,彻底规避歧义。

2.PATH 覆盖风险:_ApplyInstanceEnvVariables 在启动器已设好 Path 之后执行,用户若填写 Path=... 会直接覆盖掉 PCL 拼好的 Java 路径,导致游戏启动失败且报错完全不直观。至少需要对 Path/PATH 做特殊处理(拒绝覆盖或改为追加),并在 Tooltip 中明确警告。

3.Tooltip 示例有误导性:示例 JAVA_HOME=C:\Java 没有空格,但真实场景几乎都是 C:\Program Files\Java\jdk-21。应在示例中展示含空格路径的正确写法(加双引号),否则用户遇到含空格路径时会被静默截断。

应改进

4.缺少注入日志:环境变量注入成功与否在启动日志中完全不可见,出问题时排查困难,建议至少记录注入了哪些 key(值可脱敏)。

5.Key 未做合法性校验:若用户误输入导致 key 含空格等非法字符,会被静默注入,建议加 Trim() 并跳过含空格的 key。

现在遇到了问题:日志系统不支持输出换行字符串
我先把目前已完成的代码提交上来

@ruattd
Copy link
Contributor

ruattd commented Feb 27, 2026

根据社区开发者的讨论,我们仍然无法接受你现在 vibe coding 的结果,抱歉

@ruattd ruattd closed this Feb 27, 2026
@pcl-ce-automation pcl-ce-automation bot removed 🛠️ 等待审查 Pull Request 已完善,等待维护者或负责人进行代码审查 size: L PR 大小评估:大型 labels Feb 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants