Skip to content

chore: upgrade CI pipeline#38

Merged
best merged 1 commit into
masterfrom
chore/ci-upgrade
Feb 22, 2026
Merged

chore: upgrade CI pipeline#38
best merged 1 commit into
masterfrom
chore/ci-upgrade

Conversation

@best
Copy link
Copy Markdown
Contributor

@best best commented Feb 22, 2026

Closes #36

Changes

  • Concurrency cancellation: same PR/branch new push auto-cancels the old run
  • Scope detection: if only docs/markdown files changed, heavy jobs (typecheck, build, test) are skipped automatically
  • Split job chain: typecheckbuildtest — each is a separate job; early failure stops the chain
  • release-check job: runs on PRs, checks for version bump or changeset file; placeholder that can be extended later
  • release.yml cleanup: removed duplicate pnpm test step (CI workflow already handles it)

- Add concurrency cancellation (same PR/branch cancels old runs)
- Add scope detection: docs-only commits skip heavy jobs
- Split monolithic job into typecheck → build → test chain
- Add release-check job (PR-only, checks version bump / changeset)
- Remove duplicate pnpm test from release.yml

Closes #36
Copy link
Copy Markdown
Contributor Author

@best best left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🤖 开发经理 AI Review — PR #38 CI Upgrade

Informational only, 不阻断合并。

Checklist

项目 说明
类型安全 YAML 配置,无类型问题
错误处理 scope 检测有 fallback(zero SHA → HEAD~1),稳健
测试覆盖 基础设施变更,CI 本身即验证
⚠️ API 一致性 test job 从单一 job 拆分为 4 个 — branch protection 的 required status check 名称需同步更新(原 test,现在是 Test (unit + integration)
破坏性变更 无 breaking change,release.yml 清理合理

具体建议

  1. Branch protection 需更新:master 的 required status check 目前配置的是 test,CI 拆分后 job name 变为 Test (unit + integration),建议在 PR merge 后同步更新 branch protection rule。

  2. artifact 路径upload-artifact 上传 packages/*/distdownload-artifact 下载到 packages,路径层级需确认解压后结构是否正确(应为 packages/*/dist/...)。

  3. release-check 是占位符:当前只打印不失败,后续可接入 changeset 强校验。

总体评价

结构清晰,分层合理,是一次高质量的 CI 重构。关注 branch protection status check 名称同步即可。


由开发经理巡逻 AI Review · openlinkos/agent

@best best enabled auto-merge February 22, 2026 13:43
@best best merged commit e09368b into master Feb 22, 2026
4 of 5 checks passed
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.

[Infra] Upgrade CI pipeline to follow OpenClaw patterns

1 participant