Skip to content

Add CI build matrix with Node 20, 22, and 24#234

Merged
himerus merged 2 commits intodevfrom
feature/ci-node-matrix-20-22-24
Mar 28, 2026
Merged

Add CI build matrix with Node 20, 22, and 24#234
himerus merged 2 commits intodevfrom
feature/ci-node-matrix-20-22-24

Conversation

@himerus
Copy link
Copy Markdown
Contributor

@himerus himerus commented Mar 28, 2026

Summary

Summary

Replicate the CI Matrix parity pattern from HELiX (reference implementation at /Volumes/Development/booked/helix) to helixir. This ensures helixir tests pass across Node 20, 22, and 24 — matching GitHub CI and catching forward-compatibility issues locally.

Reference Files (helix template)

  • /Volumes/Development/booked/helix/.github/workflows/ci-matrix.yml — GitHub CI Matrix workflow
  • /Volumes/Development/booked/helix/.github/workflows/act-ci.yml — Local CI with `test-full...

Recovered automatically by Automaker post-agent hook

Summary by CodeRabbit

  • Chores
    • Added comprehensive CI/testing pipelines for improved code quality assurance
    • Extended Node.js version support to include versions 20, 22, and 24

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Mar 28, 2026

Warning

Rate limit exceeded

@himerus has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 18 minutes and 6 seconds before requesting another review.

Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 18 minutes and 6 seconds.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: 18c95662-8da4-4fca-94bd-f18028bd5696

📥 Commits

Reviewing files that changed from the base of the PR and between 4cf81ea and 129d7ec.

📒 Files selected for processing (3)
  • .automaker/context/agent-push-protocol.md
  • scripts/act-ci.sh
  • scripts/preflight.sh

Walkthrough

Three configuration files were added or modified to expand CI/CD infrastructure. Two new GitHub Actions workflows—act-ci.yml for local CI execution with quality gates and matrix testing, and ci-matrix.yml for remote matrix-based CI across multiple Node.js versions—were introduced. Additionally, Node.js engine requirements in package.json were broadened to support versions 20, 22, and 24.

Changes

Cohort / File(s) Summary
GitHub Actions Workflows
.github/workflows/act-ci.yml, .github/workflows/ci-matrix.yml
Added two new CI workflows: act-ci.yml defines quality-gates and conditional test-full jobs with pnpm dependency management and a Vitest hang watchdog; ci-matrix.yml runs matrix tests across Node versions 20, 22, 24 with conditional skip logic for staging-to-main promotion PRs.
Package Configuration
package.json
Updated engines.node from >=20.0.0 to `^20.0.0

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The PR description is incomplete, missing required sections like Type of change, Tests added, and Checklist verification items. Complete the PR description by filling in all required template sections including type of change, whether tests were added, and checklist items.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately summarizes the main change: adding CI build matrix support for Node 20, 22, and 24.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feature/ci-node-matrix-20-22-24

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 and usage tips.

- Add scripts/act-ci.sh with --full, --matrix, and --help flags
  (--matrix sets ACT_MATRIX_TESTS=true + ACT_FULL_TESTS=true for Node 20/22/24 parity)
- Add scripts/preflight.sh with Gate 8 (full test suite with vitest hang watchdog)
- Add .automaker/context/agent-push-protocol.md documenting push sequence and
  the new --matrix flag with guidance on when to run it

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@himerus himerus merged commit 83d1171 into dev Mar 28, 2026
10 of 17 checks passed
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