Skip to content

chore(deps)!: require node 22 and upgrade dependencies - #14

Merged
thatonevikash merged 1 commit into
mainfrom
chore/node-22-dependency-upgrade
Sep 19, 2026
Merged

thatonevikash merged 1 commit into
mainfrom
chore/node-22-dependency-upgrade

Conversation

@thatonevikash

@thatonevikash thatonevikash commented Sep 19, 2026

Copy link
Copy Markdown
Owner

BREAKING CHANGE: Node.js 22 or newer is now required.

Summary by CodeRabbit

  • Compatibility

    • Minimum supported Node.js version is now 22. Node.js 24 LTS is recommended.
    • Package and test workflows now validate against current Node.js versions, including 22 and 24.
  • Documentation

    • Installation and technology guidance now reflects the updated Node.js requirement.
  • Bug Fixes

    • Added coverage to verify supported SDK integrations can be loaded successfully without making API requests.

BREAKING CHANGE: Node.js 22 or newer is now required.
@coderabbitai

coderabbitai Bot commented Sep 19, 2026

Copy link
Copy Markdown

Review Change StackReview Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Advanced

Run ID: 312532fc-431f-4ffb-83cf-57f344f31769

📥 Commits

Reviewing files that changed from the base of the PR and between 0d7e964 and e55db0e.

⛔ Files ignored due to path filters (1)
  • package-lock.json is excluded by !**/package-lock.json
📒 Files selected for processing (5)
  • .github/workflows/package.yml
  • .github/workflows/testing.yml
  • README.md
  • package.json
  • test/sdk-imports.test.js

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.


📝 Walkthrough

Walkthrough

The project now requires Node.js 22 or newer, recommends Node.js 24 LTS, updates four dependencies, validates SDK imports, and tests package workflows with Node.js 22 and 24.

Changes

Node.js runtime upgrade

Layer / File(s) Summary
Runtime contract and SDK validation
package.json, test/sdk-imports.test.js
The package requires Node.js 22 or newer and updates four dependency constraints. A test verifies that the Anthropic and Google SDK adapters export functions without making API requests.
CI runtime coverage
.github/workflows/package.yml, .github/workflows/testing.yml
The package workflow tests Node.js 22 and 24. The testing workflow uses Node.js 24 and actions/setup-node@v6.
Runtime requirement documentation
README.md
The installation and technology sections document Node.js 22 or newer and recommend Node.js 24 LTS.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Other

🚥 Pre-merge checks | ✅ 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 summarizes the two primary changes: the Node.js 22 minimum requirement and dependency upgrades. The breaking-change marker is appropriate.
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 1…
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 💡 1
🛠️ Fix failing CI checks 💡
  • Commit to this branch
  • Create a new PR
📝 Generate docstrings
  • Commit to this branch
  • Create a new PR

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

Nova lifts the runtime high
Twenty-two starts the flight
Twenty-four joins the testing sky
SDK imports pass in sight
Dependencies align bright
The package paths take light

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

@thatonevikash
thatonevikash merged commit f3fb0f3 into main Sep 19, 2026
14 of 15 checks passed
@thatonevikash
thatonevikash deleted the chore/node-22-dependency-upgrade branch September 19, 2026 10:09
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