Skip to content

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

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

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
Contributor

变更说明

  • 在 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
Contributor 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。

@Qjzn

Qjzn commented Aug 6, 2026

Copy link
Copy Markdown
Owner

感谢 @534A4D21 的贡献。这个修复已在当前 main 中以原作者提交 3c57062 保留,并随 v2.7.5 发布。发布前后已通过 Windows App Server/CLI 构建、服务模块检查,以及远端 Windows bootstrap、锁定升级和官方卸载烟测。

由于功能已通过发布 PR #54 集成,这里按“已被后续集成替代”关闭;贡献者署名保留在 Git 历史和 2.7.5 发布说明中。

@Qjzn Qjzn closed this Aug 6, 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.

2 participants