Skip to content

ci: 增 push/PR 触发 + OpenAPI 漂移门可见性 + e2e 每周安全网#30

Merged
XZZKANY merged 1 commit into
masterfrom
ci/pr-triggers-and-openapi-drift
Jun 29, 2026
Merged

ci: 增 push/PR 触发 + OpenAPI 漂移门可见性 + e2e 每周安全网#30
XZZKANY merged 1 commit into
masterfrom
ci/pr-triggers-and-openapi-drift

Conversation

@XZZKANY

@XZZKANY XZZKANY commented Jun 29, 2026

Copy link
Copy Markdown
Owner

背景(next-step-plan 阶段0-2)

ci.yml/e2e.yml 原仅 workflow_dispatch,PR/合并 master 不自动跑门禁,"绿"全靠个人本地纪律。这让随后落主干的全部代码(质量引擎 + 前端)缺系统保护。

改动

  • ci.yml:增 push/pull_request(到 master) 触发 + concurrency(取消同 ref 在途 run)。核心 job 仍跑 pnpm run verify:local——其内已含 OpenAPI 契约漂移检查(verify-local.mjs 刷新后比对 sha256,漂移 exit 1);加注释明示"改路由未刷 openapi 的 PR 在此自动失败"。
  • e2e.yml:增每周 cron 安全网(契约/迁移漂移),保留 workflow_dispatch,不压每个 PR。

验证

  • yaml.safe_load 两文件 → YAML OK
  • 合并后由 push 触发首个自动 CI run 实测核心门禁。

说明

未设置分支保护/required checks,故不阻断合并,仅提供自动信号;快门禁子集化留待后续(verify:local 仍为全量核心门禁)。

🤖 Generated with Claude Code

ci.yml 由仅 workflow_dispatch 增加 push/pull_request(到 master) 触发与
concurrency;核心 job 的 verify:local 已内含 OpenAPI 契约漂移检查
(verify-local.mjs 刷新后比对 sha256,漂移 exit 1),加注释说明改路由未刷
openapi 的 PR 会在此自动失败。e2e.yml 增每周 cron 安全网,保留手动触发。

让"绿"从个人本地纪律变成系统门禁,保护随后落主干的全部代码。
验证:ci.yml/e2e.yml YAML safe_load OK。

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@XZZKANY
XZZKANY merged commit ead3c3b into master Jun 29, 2026
@XZZKANY
XZZKANY deleted the ci/pr-triggers-and-openapi-drift branch June 29, 2026 14:26
XZZKANY added a commit that referenced this pull request Jun 29, 2026
* chore(desktop): prettier 格式化 9 个文件,修复 CI 启用后暴露的既有格式债

PR #30 启用 push/PR CI 后首个 master run 在 prettier --check 失败:#27/
hidden-agent-jobs 合并引入 9 个未格式化文件,因旧 CI 仅 workflow_dispatch 未被
发现。对这些文件跑 prettier --write,纯格式化无逻辑变更。

验证:pnpm run lint exit 0;desktop typecheck exit 0;pnpm openapi 无漂移。

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

* chore(ci): 移除 GitHub Actions CI 工作流

按需求去除 CI:删除 ci.yml(Core verification)与 e2e.yml(每周安全网),
不再以 GitHub Actions 作为 PR 门禁。本地验证仍走 pnpm verify / pnpm test。

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.8 <noreply@anthropic.com>
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