Skip to content

扩大 CI 触发范围,避免直接 push 或手动发布绕过 smoke test #85

@ShadyUnderLight

Description

@ShadyUnderLight

摘要

当前 .github/workflows/ci.yml 只在 pull_request 且特定 paths 变化时触发。直接 push、手动验证、定期健康检查都不会运行 smoke test,可能让主分支在没有完整验证的情况下漂移。

失败模式

当前触发条件:

on:
  pull_request:
    paths:
      - 'scripts/**'
      - 'schemas/**'
      - 'examples/**'
      - 'requirements.txt'
      - '.github/workflows/ci.yml'

风险:

  • 直接 push 到分支 / main 不触发 CI。
  • 文档或 skill 规则变化可能影响运行行为,但不一定在 paths 中触发完整 smoke。
  • 无法手动触发验证。
  • 没有定期检查依赖安装、Playwright browser、PDF pipeline 是否仍可用。

为什么重要

这个仓库越来越依赖脚本 + 文档规则共同保证 skill 运行。如果 CI 只覆盖 PR 子集,真实运行健康度会被低估。

建议修复

  • 增加:
on:
  pull_request:
  push:
  workflow_dispatch:
  schedule:
    - cron: '0 3 * * 1'
  • 考虑将 SKILL.mdROUTING-MATRIX.mdreferences/**checklists/** 加入触发 paths,或对 smoke workflow 不使用 paths 过滤。
  • 保留 quick job 和 smoke job,但允许手动运行完整 smoke。

相关文件

  • .github/workflows/ci.yml

关联

验收标准

  • pushworkflow_dispatch 能触发 CI。
  • 每周 schedule 能跑一次 smoke test。
  • 重要 skill 文档变化不会绕过必要检查。

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions