Skip to content

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

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

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

修复内容

修复 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

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