Skip to content

fix: 隐藏 Windows Codex 后台控制台窗口 - #46

Open
534A4D21 wants to merge 1 commit into
Qjzn:mainfrom
534A4D21:codex/contrib-windows-appserver-hide
Open

fix: 隐藏 Windows Codex 后台控制台窗口#46
534A4D21 wants to merge 1 commit into
Qjzn:mainfrom
534A4D21:codex/contrib-windows-appserver-hide

Conversation

@534A4D21

Copy link
Copy Markdown

变更说明

  • 在 Windows 启动 Codex app-server 子进程时设置 windowsHide
  • 避免 detached 后台服务弹出可见控制台窗口
  • 非 Windows 平台保持现有行为不变

验证结果

  • npm run build:cli
  • npm run verify:server-modules
  • npm run verify:windows-productization

@534A4D21

Copy link
Copy Markdown
Author

补充说明一下当前失败的 CI / windows-bootstrap-smoke

该 job 中的本地 bootstrap、生成文件检查和 Windows 产品化验证均已通过,
只有 Run locked in-place upgrade smoke testacquire_repository 阶段失败。

原因是 workflow 在 fork PR 场景下传入了 GITHUB_HEAD_REF 作为分支名,
但没有同时传入 PR head 仓库的 owner/name。bootstrap 因而继续使用默认
Qjzn/CX-Codex,尝试从上游仓库下载仅存在于 fork
534A4D21/CX-Codex 中的分支归档,最终返回 404。

CI / build 已通过,并且失败发生在源码归档获取阶段,尚未进入本 PR
修改的 app-server 运行路径。为保持本 PR 的单一修复范围,我没有在这里
混入 CI workflow 修改;如有需要,我可以另行提交一个只修复 fork PR
取源逻辑的独立 PR。

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.

1 participant