Skip to content

ci: run independent steps in parallel - #1375

Merged
Makisuo merged 1 commit into
mainfrom
ci/parallel-steps
Oct 10, 2026
Merged

Makisuo merged 1 commit into
mainfrom
ci/parallel-steps

Conversation

@Makisuo

@Makisuo Makisuo commented Oct 10, 2026 •

Copy link
Copy Markdown
Collaborator

Uses the new step-level `parallel:` group in Actions.

  • TypeScript: `web` and `build-other` fold into `quality`. One turbo build covers every bundle the checks read, then 8 read-only checks run as a parallel group (schema, iOS OpenAPI, tokens, browser size, alchemy tsc, web bundle budget, web typecheck, knip). Two fewer jobs pay ~30s setup and hold a runner slot. Web typecheck uses `--only` so no turbo run inside the group restores dist/ under the others.
  • Local checkpoint native: the three probes (83s + 21s + 37s on main) run in parallel. They already use separate TMPDIRs and ports. Expected job time ~190s to ~135s.

Watch for: probe flakiness under CPU contention, and memory pressure from tsc + knip + web typecheck sharing one runner.

🤖 Generated with Claude Code


View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.

Summary by CodeRabbit

  • Chores
    • Updated behind-the-scenes build and validation workflows. These changes do not alter app features or functionality.
    • Automated checks now run in parallel, and native checkpoint probes run independently, helping streamline validation without changing the experience for end users.

Fold the web and build-other TypeScript shards into quality: one turbo
build, then the read-only checks as a parallel step group. Two fewer
jobs pay setup and hold a runner slot on every TS PR.

Run the three native checkpoint probes in parallel. They already own
separate TMPDIRs and ports.
@coderabbitai

coderabbitai Bot commented Oct 10, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 896a819b-69cc-447c-a8d5-b857a632145b

📥 Commits

Reviewing files that changed from the base of the PR and between 6ed598a and 8c56514.


📒 Files selected for processing (1)
  • .github/workflows/ci.yml

Included review availability: This review used your included allowance. Your plan provides up to 4 included reviews per hour; 3 remain after this review.



📝 Walkthrough

Walkthrough

The CI workflow consolidates TypeScript production builds and quality checks in a shared shard. It also runs three native probes as parallel steps while retaining their individual settings.

Changes

CI workflow

Layer / File(s) Summary
TypeScript quality shard
.github/workflows/ci.yml
The matrix removes the web and build-other shards. The quality shard builds production bundles for web, Alchemy, browser, effect-sdk, landing, and local-ui, then runs schema, OpenAPI, token, bundle-budget, typecheck, and Knip checks in parallel. Web typechecking uses --only.
Parallel native probes
.github/workflows/ci.yml
The checkpoint, checkpoint refresh/crash-recovery, and local-store migration probes run as parallel steps. Each retains its timeout, environment, temporary directory setup, and script.

Priority: ⬇️ Low

Estimated code review effort: 3 (Moderate) | ~20 minutes

Change: Other

Suggested reviewers: jeremyfunk


Merge Risk: ⚪ Minimal · up to 8c565

The CI workflow’s parallel step groups use supported syntax; no identified issue blocks merging after normal checks.

Pre-merge checks | Passed 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed The title clearly and concisely describes the main change: running independent CI steps in parallel.
Docstring Coverage Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR

  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@Makisuo
Makisuo marked this pull request as ready for review October 10, 2026 21:27
@maple-review-bot

maple-review-bot Bot commented Oct 10, 2026 •

Copy link
Copy Markdown

Maple review

🟢 Confidence 9/10 · safe to merge
Every check the two removed shards ran survives in the merged shard, and both parallel groups ran green on this head.
quality 100/100 · no findings · tests not needed · risk low

Folds the web and build-other TypeScript shards into quality and runs the eight read-only checks and the three native checkpoint probes as step-level parallel groups. The same checks and builds still run, with no shared mutable state between parallel steps.

  • web and build-other matrix shards removed; their builds fold into quality
  • One turbo build covers web, landing, local-ui, browser, effect-sdk and alchemy
  • Eight read-only checks run in a parallel: group, web typecheck with --only
  • The three local checkpoint probes run in parallel, each with its own TMPDIR and port
What was checked
  • No check was lost: all eight group steps and both former web steps are present at ci.yml:260-311
  • The three probes keep distinct ports (45231, 45233, 45241) and TMPDIRs, so nothing collides
  • Removing shard names is safe: branch protection requires only CI passed (ci.yml:846-852)

8c56514 · Updated on every push. Reply "won't fix" to dismiss a finding, or mention @maple-review-bot to ask about one.

@Makisuo
Makisuo merged commit 8391581 into main Oct 10, 2026
43 checks passed
@Makisuo
Makisuo deleted the ci/parallel-steps branch October 10, 2026 21:36
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