Skip to content

fix: 修复 fork PR 的 Windows bootstrap 取源逻辑 - #47

Closed
534A4D21 wants to merge 1 commit into
Qjzn:mainfrom
534A4D21:codex/contrib-fork-pr-bootstrap-source
Closed

fix: 修复 fork PR 的 Windows bootstrap 取源逻辑#47
534A4D21 wants to merge 1 commit into
Qjzn:mainfrom
534A4D21:codex/contrib-fork-pr-bootstrap-source

Conversation

@534A4D21

Copy link
Copy Markdown
Contributor

修复内容

修复 Windows locked in-place upgrade smoke 在 fork PR 场景下使用错误源码仓库的问题。

此前 workflow 只向 bootstrap 传递 PR 的 head branch,但没有传递 fork head repository 的 owner 和 name。bootstrap 因此继续使用默认的 Qjzn/CX-Codex,尝试从上游仓库下载仅存在于 fork 的分支,最终返回 404。

本次修改:

  • pull request 场景使用 fork head repository 和 head branch
  • push 场景回退到当前 repository 和 ref
  • 校验源仓库必须符合 owner/name 格式
  • RepoOwnerRepoNameBranch 一起传给现有 bootstrap 脚本
  • 不修改 bootstrap 脚本公共接口

验证

  • npm.cmd run verify:windows-productization
  • npm.cmd run verify:release -- -SchemaAudit skip
  • npm.cmd run verify:governance
  • workflow YAML 解析
  • fork PR 与 push 参数输入模拟
  • git diff --check

@Qjzn

Qjzn commented Aug 6, 2026

Copy link
Copy Markdown
Owner

感谢 @534A4D21 补充 fork PR 的取源修复。提交 fdf5896 已保留原作者信息并进入当前 main,冲突处理同时保留了上游新增的 -ForceActiveTaskUpgrade 夹具。PR #54windows-bootstrap-smoke 已完整通过本地 bootstrap、生成文件、产品化、锁定升级和官方卸载步骤,并随 v2.7.5 发布。

由于功能已通过后续发布 PR 集成,这里关闭原 PR;感谢对失败根因和修复范围的清晰说明。

@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