Skip to content

ci: 按 PR 来源分流 Bug Server 构建,避免重复构建 - #4689

Merged
xile611 merged 1 commit into
developfrom
codex/bugserver-pr-routing
Sep 17, 2026
Merged

xile611 merged 1 commit into
developfrom
codex/bugserver-pr-routing

Conversation

@xile611

@xile611 xile611 commented Sep 17, 2026

Copy link
Copy Markdown
Contributor

🤔 这个分支是...

  • Workflow

💡 问题的背景&解决方案

目前同一个 PR 会同时执行 Bug Server CI 和 Bug Server PR Bundle 的四步构建。通过比较 PR head 仓库与当前仓库,在构建 job 上添加互斥条件:

  • 仓库内 PR:仅执行 Bug Server CI,构建并提交测试。
  • 外部 fork PR:仅执行 Bug Server PR Bundle,生成产物后由维护者通过现有手动入口提交测试。

每个 PR 事件只实际构建一次。GitHub 仍可能生成两个 workflow 的运行记录,但不适用的构建 job 会显示 skipped,不启动 runner。main push 自动测试和 fork PR 的手动校验、上传流程保留。

同步更新中文说明:仓库内 PR 需要重测时重跑 CI;手动入口依赖 fork PR 的 bundle 产物。

🔗 相关的 PR 链接

#4688

📝 Changelog

Language Changelog
🇺🇸 English Avoid duplicate Bug Server builds by routing same-repository PRs to CI and fork PRs to bundle builds.
🇨🇳 Chinese 按 PR 来源分流 Bug Server 构建:仓库内 PR 自动测试,fork PR 构建产物后手动测试,消除重复构建。

☑️ 自测

  • actionlint 校验两个 workflow。
  • 实际条件求值覆盖仓库内 PR、外部 fork PR、同组织不同仓库 PR、main push、手动触发,共 5 种场景。
  • 现有 Node 来源校验测试 36 项、Python ZIP 校验测试 6 项通过。
  • pre-push 包测试:VChart 87 个套件、482 项测试,vutils-extension 1 项测试全部通过。测试结束后清理了未退出的 Electron 子进程,Rush 随后以 warnings 状态完成,推送成功。
  • Prettier、git diff --check、commitlint 通过。
  • 中文文档已更新;无需 Demo 或 TypeScript 类型变更。
  • Changelog 已提供。

构建步骤和提交脚本未变,本次未重跑完整构建;线上 Actions 结果以本 PR 检查为准。


🚀 Summary

copilot:summary

🔍 Walkthrough

copilot:walkthrough

@skie1997

Copy link
Copy Markdown
Contributor

Review 基于当前 head 11ae2a2c3b8e23be1e64fca7f9ef25dbcb1929a0

本次审查未发现阻断问题。两个构建 job 的条件与预期分流一致,手动校验、提交链路保持不变。

已从实际 workflow YAML 读取条件,验证以下五种场景:

场景 Bug Server CI 构建 PR Bundle 构建
仓库内 PR 执行 跳过
外部 fork PR 跳过 执行
同组织不同仓库 PR 跳过 执行
push 到 main 执行 不触发
workflow_dispatch 跳过 不触发

本地 36 项来源校验测试和 6 项 ZIP 校验测试全部通过。当前 PR 的线上 PR Bundle job 也已正确跳过。

验证边界:未重跑完整构建,也未触发新的 Bug Server 测试。审查时线上 Bug Server 图片测试为 2061/2093 通过,size 检查未通过;未将这些失败归因于本次分流改动,合并前仍需另行确认。

@xile611
xile611 merged commit 0ceb8a8 into develop Sep 17, 2026
10 of 12 checks passed
@xile611
xile611 deleted the codex/bugserver-pr-routing branch September 17, 2026 09:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants