Skip to content

ci: Split CI checks by changed area#798

Open
dcramer wants to merge 1 commit into
mainfrom
improve-ci-workflow
Open

ci: Split CI checks by changed area#798
dcramer wants to merge 1 commit into
mainfrom
improve-ci-workflow

Conversation

@dcramer

@dcramer dcramer commented Jul 8, 2026

Copy link
Copy Markdown
Member

CI now runs as focused, path-gated jobs instead of one monolithic workflow. Pull requests run only the relevant release, skills, lint, typecheck, test, build, docs, and dashboard e2e checks, while pushes still run the full set and package artifacts.

The workflow also centralizes Node/pnpm setup in a local composite action, adds concurrency cancellation for stale PR runs, limits Playwright installation to dashboard e2e, and uses a single ci / required aggregator so branch protection can accept intentionally skipped jobs without losing failure signal.

After this lands, branch protection should require ci / required instead of the old monolithic CI job.

Run CI checks as focused jobs behind path-based gates so pull requests avoid unrelated work while pushes keep full coverage.

Add shared Node and pnpm setup, workflow concurrency, a required aggregator for skipped jobs, and scoped dashboard/eval setup.

Co-Authored-By: GPT-5 Codex <codex@openai.com>
@vercel

vercel Bot commented Jul 8, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
junior-docs Ready Ready Preview, Comment Jul 8, 2026 7:05pm

Request Review

@dcramer dcramer marked this pull request as ready for review July 8, 2026 19:05
@github-actions github-actions Bot added the risk: medium PR risk score: medium label Jul 8, 2026

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 275ee22. Configure here.

Comment thread .github/workflows/ci.yml
"${{ needs.pack-release-artifacts.result }}"; do
if [[ "$result" != "success" && "$result" != "skipped" ]]; then
failed=1
fi

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cancelled jobs fail required check

Medium Severity

The ci / required job treats cancelled the same as failure, while the workflow enables cancel-in-progress for pull requests. When a newer push cancels an in-flight run, sibling jobs often end as cancelled, so the aggregator can report a failed required status even though a newer run is still valid.

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit 275ee22. Configure here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

risk: medium PR risk score: medium

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant