Skip to content

CI ladder: idle drafts, ci:draft / ci:spinel, four unit shards - #12

Closed
thomasklemm wants to merge 2 commits into
mainfrom
cursor/draft-pr-ci-opt-b5c3
Closed

thomasklemm wants to merge 2 commits into
mainfrom
cursor/draft-pr-ci-opt-b5c3

Conversation

@thomasklemm

Copy link
Copy Markdown
Owner

Summary

Single meta-PR for the CI cost ladder (not folded into rubys#451):

  1. Draft idle by default — no selected jobs until ci:draft, ci:spinel, or ci:full is applied (plan still routes; fixture/unit are selection-gated).
  2. ci:draft — fixture + unit only.
  3. ci:spinel — Ruby floor + full Spinel suite; skips other-language SDKs, WASM, Writebook.
  4. Ready / non-draft — existing path-selected Ruby floor (unchanged ownership rules); ci:full still expands everything.
  5. Four unit shards — shorter unit critical path on every non-idle run.

Docs: docs/ci/README.md ladder table + label request section.

Labels to create (if missing on the repo)

  • ci:draft
  • ci:spinel
  • ci:full (already exists)

Verification

  • python3 -B tests/ci_plan_test.py — 50 passed
  • cargo test --test ci_fixture_workflow
  • cargo test --test ci_policy_workflow -- --skip resource_and_harness
  • cargo test --test workflow_yaml_parses draft_transitions

Commits

  1. Opt-in draft CI and add a ci:spinel lane
  2. Widen unit CI to four parallel shards

Target times and remaining cost centers are in the agent report (internal/draft-pr-ci.md).

Open in Web Open in Cursor 

cursoragent and others added 2 commits October 5, 2026 19:02
Draft PRs stay idle until ci:draft, ci:spinel, or ci:full is applied.
ci:draft runs fixture + unit only; ci:spinel runs the Ruby floor plus
the full Spinel suite without other-language SDKs. Ready PRs keep
path-selected coverage. Gate generate-fixture and unit on planner
selection so idle drafts do not burn runners.

Co-Authored-By: Cursor <cursoragent@cursor.com>

Co-authored-by: Thomas Klemm <github@tklemm.eu>
Spread integration targets across four shards (max-parallel 4) to shorten
the unit critical path on ready, ci:draft, and ci:spinel runs.

Co-Authored-By: Cursor <cursoragent@cursor.com>

Co-authored-by: Thomas Klemm <github@tklemm.eu>
@coderabbitai

coderabbitai Bot commented Oct 5, 2026

Copy link
Copy Markdown

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

  • Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Comment @coderabbitai help to get the list of available commands.

@thomasklemm thomasklemm closed this Oct 5, 2026
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.

2 participants