chore(ci): modernize

[avivkeller]

Modernize the CI with improvements for brevity and speed

The Required Checks

• CI / Lint
• CI / Smoketests
• CI / Test - macos-latest (Node.js 18.x)
• CI / Test - macos-latest (Node.js 20.x)
• CI / Test - macos-latest (Node.js 22.x)
• CI / Test - macos-latest (Node.js 24.x)
• CI / Test - ubuntu-latest (Node.js 18.x)
• CI / Test - ubuntu-latest (Node.js 20.x)
• CI / Test - ubuntu-latest (Node.js 22.x)
• CI / Test - ubuntu-latest (Node.js 24.x)
• CI / Test - windows-latest (Node.js 18.x)
• CI / Test - windows-latest (Node.js 20.x)
• CI / Test - windows-latest (Node.js 22.x)
• CI / Test - windows-latest (Node.js 24.x)
• Validate PR / Lint Commit Messages

[Copilot]

Pull request overview

This PR modernizes the CI infrastructure by replacing commitlint with GitHub Actions-based PR title validation and consolidating CI workflows. Key changes include:

• Removal of commitlint dependencies and local commit message validation
• Migration from nodejs.yml to a simplified ci.yml workflow
• Addition of automated PR title validation using GitHub Actions

Reviewed changes

Copilot reviewed 9 out of 10 changed files in this pull request and generated 9 comments.

Show a summary per file

File	Description
package.json	Removes @commitlint/cli and @commitlint/config-conventional dependencies
package-lock.json	Removes commitlint and related dependency entries; also removes "peer" flags from multiple packages
commitlint.config.js	Deletes commitlint configuration file
.husky/commit-msg	Removes husky hook for commit message validation
.github/workflows/validate-pr-title.yml	Adds new workflow for PR title validation using amannn/action-semantic-pull-request
.github/workflows/nodejs.yml	Removes old CI workflow
.github/workflows/ci.yml	Creates new consolidated CI workflow with lint and test jobs
.github/workflows/dependency-review.yml	Updates workflow name and step descriptions
.github/dependabot.yml	Changes GitHub Actions update schedule from daily to monthly and adds grouping
.cspell.json	Adds "amannn" and removes "commitlint" from dictionary

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.72%. Comparing base (6a70fee) to head (879b17e).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #4620   +/-   ##
=======================================
  Coverage   89.72%   89.72%           
=======================================
  Files           9        9           
  Lines        1401     1401           
  Branches      448      448           
=======================================
  Hits         1257     1257           
  Misses        144      144           

---

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6a70fee...879b17e. Read the comment docs.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[alexander-akait]

@avivkeller I make some valiuable change in our CI from your PR here - #4623, feel free to rebase and improve more if you have something, I prefer to use matrix everywhere because it is using variable and less hard coded values

[avivkeller]

Perhaps it's a personal preference, but I find redundant matrices to be a bad practice, since they are meant to run multiple variations, not single ones.

[alexander-akait]

@avivkeller We use this value in two places - in run-on and node-version, we've previously encountered problems where a contributors forgot to update both (and more than both), so we decided to use a matrix because it allows you to change the value in only one place and not forget about the others

[alexander-akait]

Anyway in future we are planning to reuse workflows from webpack/.github, so in future when we will have only a couple workflows it will not be a problem, we can avoid matrix because we will catch such things

[avivkeller]

I see. In that case, I'm happy with your PR for now